🐛 [firebase_app_check] FlutterFire Realtime Database calls blocked by firebase_app_check for Android only

Bug report
Describe the bug
On Android only, error message received as 'Client is offline', even though the client is online and the same code is working perfectly fine on iOS. After a while it suddenly starts working fine on Android though, but it takes around 10-20 min before it suddenly works.
Steps to reproduce
When running a simple query on a Firebase Realtime Database it works perfectly fine on iOS both in simulator and on device. However on Android it doesn't work. The code below attempts to get the user 'YYYY', whether or not the user already exists in the database. Seems security rules are not the issue, as this working on iOS. The code is as follows:
DatabaseReference _databaseRef = FirebaseDatabase(databaseURL:'https://XXXX.europe-west1.firebasedatabase.app/').reference();

DatabaseReference _userRef = _databaseRef.child('users/YYYY');
DataSnapshot userData = await _userRef.once();
DataSnapshot userData = await _userRef.get();

For the call await _userRef.once(), nothing happens and the function invocation stalls and never moves forward. The emulator will just continue to provide a stream of app_time_stats updates:
D/EGL_emulation: app_time_stats: avg=67.81ms min=7.44ms max=980.53ms count=19
D/EGL_emulation: app_time_stats: avg=16.53ms min=9.48ms max=19.88ms count=61
D/EGL_emulation: app_time_stats: avg=16.64ms min=14.06ms max=19.05ms count=61
D/EGL_emulation: app_time_stats: avg=11.45ms min=2.64ms max=65.59ms count=58

For the call await _userRef.get(), I get back the following error message on both Android emulator and physical device:
I/RepoOperation: get for query /users/YYYY falling back to disk cache after error: Client is offline
E/flutter: [ERROR:flutter/lib/ui/ui_dart_state.cc(199)] Unhandled Exception: PlatformException(get-failed, Client is offline, null, null)

If I try to interact with the database by creating a new user, no error message is received but no data appears in the realtime database. If I try to retrieve the user again, it thinks that the user creation was successful and returns me data. I suppose this is because it thinks the database is offline and it then just creates the data offline.
Expected behavior
I would expect the code above to work with milliseconds delay on both Android and iOS. Currently, this is works as expected on iOS, but not on Android.

Additional context
All other Firebase related frameworks, such as Cloud Firestore, work fine for both iOS and Android in the sample project.
After 10-20 min the calls to Firebase Database sometimes suddenly starts working fine on Android, just as it does on iOS. However, I am not doing anything in particular that I can identify as the reason for it suddenly working.

Flutter doctor
Run flutter doctor and paste the output below:
Click To Expand
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 2.2.3, on macOS 11.5 20G71 darwin-arm)
[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
[✓] Xcode - develop for iOS and macOS
[✓] Chrome - develop for the web
[✓] Android Studio (version 2020.3)
[✓] VS Code (version 1.59.0)
[✓] Connected device (4 available)

• No issues found!

Flutter dependencies
Run flutter pub deps -- --style=compact and paste the output below:
Click To Expand
- cloud_firestore 2.4.0 [cloud_firestore_platform_interface cloud_firestore_web firebase_core firebase_core_platform_interface flutter meta]
- firebase_analytics 8.2.0 [firebase_analytics_platform_interface firebase_analytics_web firebase_core flutter meta]
- firebase_app_check 0.0.1+3 [firebase_app_check_platform_interface firebase_app_check_web firebase_core firebase_core_platform_interface flutter]
- firebase_auth 3.0.1 [firebase_auth_platform_interface firebase_auth_web firebase_core firebase_core_platform_interface flutter meta]
- firebase_core 1.4.0 [firebase_core_platform_interface firebase_core_web flutter meta]
- firebase_database 7.1.2 [firebase_core flutter]
- firebase_performance 0.7.0+7 [firebase_core flutter]
- firebase_storage 10.0.1 [firebase_core firebase_core_platform_interface firebase_storage_platform_interface firebase_storage_web flutter]
- flutter 0.0.0 [characters collection meta typed_data vector_math sky_engine]
- http 0.13.3 [async http_parser meta path pedantic]
- intl 0.17.0 [clock path]

@mzkai
Since you are getting this error:
falling back to disk cache after error: Client is offline
PlatformException(get-failed, Client is offline, null, null)
It indicates, for some reason, The events being fired from native Android to Dart may be taking long time and hence during this wait period, dart is throwing this exception. So the issue could be in Native Android or in Flutter, because Flutterfire plugins just wraps native SDKs.
You may check this relevant issue opened in android-firebase-sdk talking about realtime database not getting updates on wifi, https://github.com/firebase/firebase-android-sdk/issues/1258.
Can you check from your end if it only occurs in wifi or with cellular data too ?

Just checked, and with cellular data the problem is still the same. I believe the following information was shown in the logs around 20 min later when the connection started working again. Maybe they can help to figure out what is wrong:
[FirebaseAuth:] Preparing to create service connection to fallback implementation
W/System: Ignoring header X-Firebase-Locale because its value was null.
D/FirebaseAuth: Notifying id token listeners about user ( YYYY ).

@mzkai
I am wondering if it's related to the way you are accessing the database url: databaseURL:'https://XXXX.europe-west1.firebasedatabase.app.  Is it pointing to the url which is in different timezone ?
Moreover, can you try the same using the official plugin example and see if it results in same error ?

@darshankawar
Still facing the same issue after all the mentioned fixes

@darshankawar
I have also tried the same, and still have the same problem. I tried creating a https://XXXX.firebaseio.com/ database instead, but gives me the same error.

After a lot of trial and error I found out that it is an issue with firebase_app_check: ^0.0.2. After removing the package it is now working fine also for Android. Seems that app check is somehow hindering or blocking communication with the firebase realtime database.
I have changed the title of this issue to clarify that this is concerning firebase_app_check.

I'm experiencing this issue and I don't even user firebase_app_check

Estou tendo esse problema e nem mesmo uso firebase_app_check

Doesn't work without using firebase_app_check
bool  exists = await ref.get().then((data) => data.exists);

Doesn't work without using firebase_app_check
bool exists = await ref.get().then((data) => data.exists);

log:
get for query /users/irsxHzDYM5WdKK2IXTKD5DGK2WN2 falling back to disk cache after error: Client is offline
I/flutter ( 2048): PlatformException(get-failed, Client is offline, null, null

flutter doctor -v
[✓] Flutter (Channel stable, 2.5.3, on macOS 12.0.1 21A559 darwin-arm, locale ru-RU)
    • Flutter version 2.5.3 at /opt/homebrew/Caskroom/flutter/2.2.3/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 18116933e7 (7 weeks ago), 2021-10-15 10:46:35 -0700
    • Engine revision d3ea636dc5
    • Dart version 2.14.4

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/vladislavponomarev/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • Java binary at: /Applications/Android Studio Preview.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 13.1, Build version 13A1030d
    • CocoaPods version 1.11.2

[!] Android Studio
    • Android Studio at /Applications/Android Studio Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to find bundled Java version.
    • Try updating or re-installing Android Studio.

[✓] VS Code (version 1.62.3)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.29.0

[✓] Connected device (2 available)
    • sdk gphone64 arm64 (mobile) • emulator-5554 • android-arm64 • Android 12 (API 31) (emulator)
    • macOS (desktop)             • macos         • darwin-arm64  • macOS 12.0.1 21A559 darwin-arm

! Doctor found issues in 1 category.

Hey @mzkai, this appears to be a bug with firebase-android-sdk. This was ocurrring on Firebase android BOM 28.1.0, and we're now on 30.1.0. If it is still happening, please provide an MCVE so I can easily debug and I will take a look. Thanks.


[INLONG-9223][Sort] TubeMQ source support InlongAudit

Prepare a Pull Request
(Change the title refer to the following example)

Title Example: [INLONG-XYZ][Component] Title of the pull request

(The following XYZ should be replaced by the actual GitHub Issue number)

Fixes #9223

Motivation
Explain here the context, and why you're making that change. What is the problem you're trying to solve?
Modifications
Describe the modifications you've done.
Verifying this change
(Please pick either of the following options)

[ ] This change is a trivial rework/code cleanup without any test coverage.

[ ] This change is already covered by existing tests, such as:
(please describe tests)

[ ] This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a follow-up issue for adding the documentation

Please fix test cases


fix empty facet bug

When the facet domain does not include any value from the actual contents, the facet index is undefined instead of [].
I'm not sure this is the correct fix, I have an alternative that moves a bit more things but seems better in #1216
closes #1212

Superseded by #1216.


Lithops fails when runtime not specified

In case of selecting VPC backend if runtime not specified Lithops fails to setup master node.
Probably, either documentation should be updated to have runtime as mandatory or code should be fixed to select default runtime, based on user python version.

@Could you provide context formation, debug logs, and an example code that fails?
For example I ran lithops test -b ibm_vpc -d and Lithops correctly detected my python3.8 version:
2021-08-29 12:01:48,522 [INFO] lithops.config -- Lithops v2.4.2.dev0
2021-08-29 12:01:48,522 [DEBUG] lithops.config -- Loading configuration from /home/josep/.lithops/config
2021-08-29 12:01:48,545 [DEBUG] lithops.config -- Loading Standalone backend module: ibm_vpc
2021-08-29 12:01:48,991 [DEBUG] lithops.config -- Loading Storage backend module: ibm_cos
2021-08-29 12:01:49,031 [DEBUG] lithops.storage.backends.ibm_cos.ibm_cos -- Creating IBM COS client
2021-08-29 12:01:49,031 [DEBUG] lithops.storage.backends.ibm_cos.ibm_cos -- Set IBM COS Endpoint to https://s3.eu-de.cloud-object-storage.appdomain.cloud
2021-08-29 12:01:49,031 [DEBUG] lithops.storage.backends.ibm_cos.ibm_cos -- Using access_key and secret_key
2021-08-29 12:01:49,058 [INFO] lithops.storage.backends.ibm_cos.ibm_cos -- IBM COS Storage client created - Region: eu-de
2021-08-29 12:01:49,058 [DEBUG] lithops.standalone.backends.ibm_vpc.ibm_vpc -- Creating IBM VPC client
2021-08-29 12:01:49,058 [DEBUG] lithops.standalone.backends.ibm_vpc.ibm_vpc -- Setting VPC endpoint to: https://us-east.iaas.cloud.ibm.com
2021-08-29 12:01:49,059 [INFO] lithops.standalone.backends.ibm_vpc.ibm_vpc -- IBM VPC client created - Region: us-east
2021-08-29 12:01:49,059 [DEBUG] lithops.standalone.standalone -- Standalone handler created successfully
2021-08-29 12:01:49,059 [DEBUG] lithops.invokers -- ExecutorID 90a8a7-0 - Total workers: 100
2021-08-29 12:01:49,059 [DEBUG] lithops.standalone.backends.ibm_vpc.ibm_vpc -- Initializing IBM VPC backend (Consume mode)
2021-08-29 12:01:54,271 [DEBUG] lithops.executors -- Function executor for ibm_vpc created with ID: 90a8a7-0
2021-08-29 12:01:54,271 [INFO] lithops.invokers -- ExecutorID 90a8a7-0 | JobID M000 - Selected Runtime: python3.8
2021-08-29 12:01:54,272 [DEBUG] lithops.storage.storage -- Runtime metadata not found in local cache. Retrieving it from storage
2021-08-29 12:01:54,272 [DEBUG] lithops.storage.storage -- Trying to download runtime metadata from: ibm_cos://lithops-data-eu-de/lithops.runtimes/2.4.2.dev0/ibm_vpc/0767_b14ffff6-d61f-4ff6-9e99-22945892b347/python3.8.meta.json
2021-08-29 12:01:55,248 [DEBUG] lithops.storage.storage -- Runtime metadata not found in storage
2021-08-29 12:01:55,248 [INFO] lithops.invokers -- Runtime python3.8 is not yet installed
2021-08-29 12:01:55,248 [DEBUG] lithops.standalone.standalone -- Checking if VM instance lithops-master-ad73 (<IP_ADDRESS>) is ready
2021-08-29 12:01:57,251 [DEBUG] lithops.standalone.standalone -- VM instance lithops-master-ad73 (<IP_ADDRESS>) not ready
2021-08-29 12:01:57,251 [DEBUG] lithops.standalone.backends.ibm_vpc.ibm_vpc -- Starting VM instance lithops-master-ad73
2021-08-29 12:01:58,160 [DEBUG] lithops.standalone.backends.ibm_vpc.ibm_vpc -- VM instance lithops-master-ad73 started successfully
2021-08-29 12:01:58,160 [INFO] lithops.standalone.standalone -- Waiting VM instance lithops-master-ad73 (<IP_ADDRESS>) to become ready
2021-08-29 12:02:41,298 [DEBUG] lithops.util.ssh_client -- <IP_ADDRESS> ssh client created
2021-08-29 12:02:44,083 [DEBUG] lithops.standalone.standalone -- VM instance lithops-master-ad73 (<IP_ADDRESS>) ready in 45.92 seconds
2021-08-29 12:02:44,083 [DEBUG] lithops.standalone.standalone -- Installing Lithops in VM instance lithops-master-ad73 (<IP_ADDRESS>)
2021-08-29 12:02:44,083 [DEBUG] lithops.utils -- Creating function handler zip in /home/josep/data/dev-workspace/lithops/lithops/lithops_standalone.zip
2021-08-29 12:02:44,127 [DEBUG] lithops.standalone.standalone -- Uploading lithops files to VM instance lithops-master-ad73 (<IP_ADDRESS>)
2021-08-29 12:02:46,109 [DEBUG] lithops.standalone.standalone -- Executing lithops installation process on VM instance lithops-master-ad73 (<IP_ADDRESS>)
2021-08-29 12:02:46,109 [DEBUG] lithops.standalone.standalone -- Be patient, initial installation process may take up to 5 minutes
2021-08-29 12:02:47,156 [DEBUG] lithops.standalone.standalone -- Lithops installation process completed
2021-08-29 12:02:47,156 [INFO] lithops.standalone.standalone -- Waiting Lithops service to become ready on VM instance lithops-master-ad73 (<IP_ADDRESS>)
2021-08-29 12:02:50,205 [DEBUG] lithops.standalone.standalone -- VM instance lithops-master-ad73 (<IP_ADDRESS>) ready in 3.05 seconds
2021-08-29 12:02:50,205 [DEBUG] lithops.standalone.standalone -- Extracting runtime metadata information
2021-08-29 12:02:51,530 [DEBUG] lithops.storage.storage -- Uploading runtime metadata to: ibm_cos://lithops-data-eu-de/lithops.runtimes/2.4.2.dev0/ibm_vpc/0767_b14ffff6-d61f-4ff6-9e99-22945892b347/python3.8.meta.json
2021-08-29 12:02:52,247 [DEBUG] lithops.storage.backends.ibm_cos.ibm_cos -- PUT Object lithops.runtimes/2.4.2.dev0/ibm_vpc/0767_b14ffff6-d61f-4ff6-9e99-22945892b347/python3.8.meta.json - Size: 7.9KiB - OK
2021-08-29 12:02:52,248 [DEBUG] lithops.storage.storage -- Storing runtime metadata into local cache: /home/josep/.lithops/cache/lithops.runtimes/2.4.2.dev0/ibm_vpc/0767_b14ffff6-d61f-4ff6-9e99-22945892b347/python3.8.meta.json
2021-08-29 12:02:52,248 [DEBUG] lithops.job.job -- ExecutorID 90a8a7-0 | JobID M000 - Serializing function and data
2021-08-29 12:02:52,249 [DEBUG] lithops.job.serialize -- Referenced modules: time
2021-08-29 12:02:52,249 [DEBUG] lithops.job.serialize -- Modules to transmit: None
2021-08-29 12:02:52,249 [DEBUG] lithops.job.job -- ExecutorID 90a8a7-0 | JobID M000 - Uploading function and modules to the storage backend
2021-08-29 12:02:52,349 [DEBUG] lithops.storage.backends.ibm_cos.ibm_cos -- PUT Object lithops.jobs/90a8a7-0/06518eda31bd4fb2ec00f302ba908242.func.pickle - Size: 665.0B - OK
2021-08-29 12:02:52,349 [DEBUG] lithops.job.job -- ExecutorID 90a8a7-0 | JobID M000 - Uploading data to the storage backend
2021-08-29 12:02:52,957 [DEBUG] lithops.storage.backends.ibm_cos.ibm_cos -- PUT Object lithops.jobs/90a8a7-0-M000/aggdata.pickle - Size: 42.0B - OK
2021-08-29 12:02:52,957 [INFO] lithops.invokers -- ExecutorID 90a8a7-0 | JobID M000 - Starting function invocation: my_map_function() - Total: 2 activations
2021-08-29 12:02:52,957 [DEBUG] lithops.invokers -- ExecutorID 90a8a7-0 | JobID M000 - Chunksize: 1 - Worker processes: 4
2021-08-29 12:02:52,957 [DEBUG] lithops.standalone.standalone -- ExecutorID 90a8a7-0 | JobID M000 - Going to run 2 activations in 1 workers
2021-08-29 12:02:52,957 [DEBUG] lithops.standalone.standalone -- Checking if VM instance lithops-master-ad73 (<IP_ADDRESS>) is ready
2021-08-29 12:02:53,790 [DEBUG] lithops.standalone.standalone -- Job invoked on VM instance lithops-master-ad73 (<IP_ADDRESS>)
2021-08-29 12:02:53,791 [DEBUG] lithops.invokers -- ExecutorID 90a8a7-0 | JobID M000 - Job invoked (0.834s) - Activation ID: 90a8a7-0-M000
2021-08-29 12:02:53,791 [INFO] lithops.invokers -- ExecutorID 90a8a7-0 | JobID M000 - View execution logs at /tmp/lithops/logs/90a8a7-0-M000.log
2021-08-29 12:02:53,791 [DEBUG] lithops.monitor -- ExecutorID 90a8a7-0 - Starting Storage job monitor
2021-08-29 12:02:53,791 [INFO] lithops.wait -- ExecutorID 90a8a7-0 - Getting results from functions
2021-08-29 12:02:57,027 [DEBUG] lithops.monitor -- ExecutorID 90a8a7-0 - Pending: 2 - Running: 0 - Done: 0
2021-08-29 12:03:00,038 [DEBUG] lithops.monitor -- ExecutorID 90a8a7-0 - Pending: 0 - Running: 2 - Done: 0
2021-08-29 12:03:02,801 [DEBUG] lithops.future -- ExecutorID 90a8a7-0 | JobID M000 - Got status from call 00000 - Activation ID: f57f5dcab323 - Time: 5.74 seconds
2021-08-29 12:03:03,064 [DEBUG] lithops.monitor -- ExecutorID 90a8a7-0 - Pending: 0 - Running: 1 - Done: 1
2021-08-29 12:03:04,541 [DEBUG] lithops.monitor -- ExecutorID 90a8a7-0 - Pending: 0 - Running: 0 - Done: 2
2021-08-29 12:03:04,541 [DEBUG] lithops.monitor -- ExecutorID 90a8a7-0 - Storage job monitor finished
2021-08-29 12:03:04,803 [DEBUG] lithops.future -- ExecutorID 90a8a7-0 | JobID M000 - Got status from call 00001 - Activation ID: f57f5dcab323 - Time: 6.76 seconds
2021-08-29 12:03:05,685 [DEBUG] lithops.util.ssh_client -- <IP_ADDRESS> ssh client created
2021-08-29 12:03:06,502 [INFO] lithops.executors -- ExecutorID 90a8a7-0 - Cleaning temporary data
2021-08-29 12:03:06,506 [DEBUG] lithops.executors -- ExecutorID 90a8a7-0 - Finished getting results

@kpavel is this still present? or solved?

Probably the issue was in outdated lithops on master or some outdated cached data.
Resolved by manual delete of master vm and:
lithops clean
rm -rf ~/.lithops


flake.lock: update nixpkgs and xdph

Describe your PR, what does it fix/add?
Update nixpkgs and xdph to v1.3.8
Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)
Does not pass CI
Is it ready for merging, or does it need work?
Not ready

Can you update xdph again? This time it should build.

Done, it build correctly now.

@fufexan This should be good to go. I removed the nixpkgs update since it has already been done in another PR.


what is index action in controller in API being used for

what is index action in controller in API being used for?
public function index(IndexStore $request)
{
$query = Store::query(); // You can extend this however you want.
$cols = [
Column::name('id')->title('Id')->sort()->searchable(),
Column::name('location_name')->title('Location Name')->sort()->searchable(),
Column::name('address_1')->title('Address 1')->sort()->searchable(),
Column::name('address_2')->title('Address 2')->sort()->searchable(),
Column::name('city')->title('City')->sort()->searchable(),
Column::name('province')->title('Province')->sort()->searchable(),
Column::name('email')->title('Email')->sort()->searchable(),
Column::name('phone_no')->title('Phone No')->sort()->searchable(),
Column::name('updated_at')->title('Updated At')->sort()->searchable(),
        Column::name('actions')->title('')->raw()
    ];
    $data = Pagetables::of($query)->columns($cols)->make(true);
    return $this->api->success()->message("List of Stores")->payload($data)->send();
}

public function dt(Request $request) {
    $query = Store::query()->select(Store::getModel()->getTable().'.*'); // You can extend this however you want.
    return $this->repo::dt($query);
}

it returns the default columns for the datatable for the respective models

But when this action is called?
I can't find any event
On Wed, Feb 9, 2022, 3:57 PM bryan rakoi @.***> wrote:

it returns the default columns for the datatable for the respective models
—
Reply to this email directly, view it on GitHub
https://github.com/coolsam726/jetstream-inertia-generator/issues/48#issuecomment-1033604720,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA6DATQHNTV6I6TEYK3XXZDU2I6RBANCNFSM5NZ5HXTA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi @abhitheawesomecoder,
Initially it was being used to actually render the Datatables, but since we moved to using Yajra datatables, this index function now is used mainly for fetching paginated lists of models and searching the models. If you check closely you will realize that all the InfiniteSelect input components call this method  (e.g route('api.roles.index')) in order to fetch a paginated list of models, while at the same time offering search. The $cols variable sets the fields which can be searched within the model. You can even set relationships using dots. I hope this helps.


chore: upgrade rule_nodejs, rules_js, bazel-lib

bazel-lib and rules_js upgrades landed https://github.com/aspect-build/rules_ts/commit/16499c2696392ffa2f2f21ba417f868cb77e70c7 and https://github.com/aspect-build/rules_ts/commit/6b6face94e8cfdd86f22c24a70481258ce1b92c5 so just the rules_nodejs 5.5.2 upgrade remains to be done from this
the bzlmod to 1.0.0-rc.2 won't work since bzlmod only supports vanilla "1.2.3" style versions 🤷

Closing for now since most of this was done in other PRs. Will probably open another PR soon 🤷


lambda: Code.file unneccessary repeats S3 bucket parameter

For every call to Code.file two parameters are added to the stack, one for the object and one for the bucket.
The bucket parameter should be re-used since it will be the same for all.

Unless you're hitting CloudFormation limits because of this, I'm going to close it as "works as intended" for now.
Doing it this way affords us the flexibility to retrieve assets from different locations in the future. In any case, the assets feature will need to be redesigned shortly anyway.


Exposing Transforms.rotationMatrixFromPositionVelocity

Also added tests for Transforms.rotationMatrixFromPositionVelocity
NOTE: I wasn't able to figure out how to get the tests to pass since Transforms.rotationMatrixFromPositionVelocity sometimes return 0 or -0 for it's 0 values. I ended up just using the Matrix3.abs function for now but if there is a better way to handle this let me know.
Handles bullet 3 from #8927

Thanks for the pull request @easternmotors!

:heavy_check_mark: Signed CLA found.
:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.
[ ] Works (or fails gracefully) in IE11.

Updated CHANGES.md

Thanks @easternmotors, just the one comment to address the abs usage and hopefully this will be good to go.  Thanks.

Thanks!


change input type=number in send and recieve amount

as referenced in #229 the inputs for where only numbers should be are actually text currently.  Added minimum to keep some weird ux from happening along with zero and negative error catches for the user to understand what went wrong

I just briefly went over the error handling again for the toast strings and I realized they refuse to display on the receive page. they work everywhere else but to see them on receive you must first navigate home. I'll need to figure out why it is happening on this page before this pr is ready

sorry for the wave of commits, I kept thinking of more and more edge cases...
It should now accept only numbers while allowing you to still create zero-amount invoices along with adding commas for when you need to keep track of typing larger numbers.
As well on phones it appears to be a number text field but still behaves like a text field so any non-numeric characters pasted in accidentally will still be treated with the same logic as a on desktop.
Shout out to chatGPT for coming in clutch.


SOLR-17065: Absorb the Cross DC project

https://issues.apache.org/jira/browse/SOLR-17065

[x] Absorb the CrossDC Consumer/Producer/Common
[x] Have the build work

[x] Fix the compilation warning and uncomment the lines from javac.gradle

[x] Have the unit tests work
[x] Integrate with Solr packaging (deduplicating dependency jars and all)
[ ] Add an integration test
[ ] Add a docker test?
[x] Add documentation in the repo
[x] Add ref guide documentation
[x] Make sure the smoke tests work
[x] Rename the configurations so that they make sense with the whole project

We can't upgrade to kafka 3.8.0, because of https://issues.apache.org/jira/browse/KAFKA-17227, but 3.8.1 will be released in early october and we should be good to upgrade to that.


Make cartodb-postgresql python3 compatible (again)

encoding string with python3
adapt HTTPConnection signature to python3

Next time please add a reviewer :pray:

Next time please add a reviewer

This was a copy of another PR which was merged before and Raul reviewed but you're right, I should add a reviewer always. Sorry about that! :sweat_smile:


Add OpenMPI?

Is it possible to add an OpenMPI-based version as well?

Sorry for the delayed response. Would adding support for OpenMPI provide anything beyond what is currently available using MPICH? When we set up the Meep Conda package around 2018, it seemed the MPICH Conda packages were more up to date and better maintained than OpenMPI although the situation could have changed since then.

The main advantage of adding the OpenMPI implementation would be to provide compatibility with other packages that use OpenMPI instead of MPICH. Other than that, though, I'm not entirely sure what the state of the two packages are. I was able to get the package to build by just adding the openmpi flag to the mpi options so it might be straightforward to do this now?
For context, I initially opened this because I wanted to make this conda package (which recommends using OpenMPI) compatible with pymeep. I'm building that with MPICH now as well though so I closed this issue.
I do still think it would be good to have both implementations for pymeep regardless though, if possible. What do you think?


Py3 workers changes

Mostly fixing meta.

Current coverage is 67.60% (diff: 80.00%)

Merging #192 into master will not change coverage

@@             master       #192   diff @@
==========================================
  Files            70         70
  Lines          4451       4451
  Methods           0          0
  Messages          0          0
  Branches        531        531
==========================================
  Hits           3009       3009
  Misses         1308       1308
  Partials        134        134

Powered by Codecov. Last update 50a36ac...e7d44b2


auto sync with specific fields is not happening properly.

I am trying to sync 4 collections to separate indexes in ES with following command
mongo-connector -m localhost:27017 -t 1xx.xxx.xxx.xxx:9200 -o /opt/mongo-connector.oplog --auto-commit-interval=0 -d elastic2_doc_manager -n dbname.first,dbname.second,dbname.third,dbname.fourth
It is working fine for first 3 collections, and syncs data for those 3 collections. But not for the last one dbname.fourth.
while checking, i noticed that, I have used "birthdate": "" Or "birthdate": "2\/1\/1993", this is causing problem. If i removes this birthdate, then it properly syncs the data of dbname.fourth collections as well.
I have checked oplog , that also seems to be normal like
"birthdate" : "", .......... in case of empty
"birthdate" : "2/1/1993", ............ in case of data

Any idea on this ? Or is there any way to skip such fields from auto syncing ?

On the mongo-connector side, you can exclude the "birthdate" field entirely: https://github.com/mongodb-labs/mongo-connector/wiki/Configuration-Options#filtering-documents-per-namespace
See also: http://stackoverflow.com/a/15950434.


ERROR: torchao-0.6.1+git-cp312-cp312-win_amd64.whl is not a supported wheel on this platform.

Not sure why I get this when trying to install Torchao. I am on Windows 10, have CUDA 12.4 and Portable Comfyui with python 3.11

cp312 refers to python 3.12, so this wheel won't install on 3.11


Amplify Console hosting gives 403 Access Denied error on refresh

Describe the bug
I hosted a react web application on amplify console. I am using react-router for routing. Whenever I refresh a page on the app, I get a 403 access denied. The website is working fine on my local and there are no issues on refresh.
<Error>
AccessDenied
<Message>Access Denied</Message>
<RequestId>7B11E37EDFFD3126</RequestId>
<HostId>tqvP88WnIXKktzlGH+NRf88iF33bsMOe7BqosnSXrrPoZvyFh80M6gdGaHo+fpoA0gUcnSI94G8=</HostId>
</Error>

amplify status output:
Current Environment: dev

| Category | Resource name        | Operation | Provider plugin   |
| -------- | -------------------- | --------- | ----------------- |
| Api      | Test Newsletters App | Create    |                   |
| Auth     | letterreaderd755019a | No Change | awscloudformation |
| Hosting  | amplifyhosting       | No Change |                   |

To Reproduce
Steps to reproduce the behavior:

Go to https://www.letterdash.io/
Click on "Get Started Button"
Signup and log in
Refresh any page

Expected behavior
There should be no error on refresh
Screenshots

Desktop (please complete the following information):

Chrome

Additional context
Add any other context about the problem here.

I resolved this using 200 redirects for SPA - https://docs.aws.amazon.com/amplify/latest/userguide/redirects.html#redirects-for-single-page-web-apps-spa


Higher FID in Comparision to Paper

Dear Authors, thank you for your work and for publicizing the code. We tried replicating the results of AFHQ_Cat, but we got the FID score around 62.03. Also, we observed that running eval on every 50th epoch makes the training very slow. We would like to know if is it because of eval frequency, if we reduce it, we can get paper-reported FID, or if something else is the reason.

Hi: thanks for the interest. Yes you could lower the eval gap and train a bit longer (say >2k iterations) to see the results. Meanwhile, you can also share the train/eval setup in your trials if necessary.


AutoGraph could not transform <function simple_ml_inference_op_with_handle at 0x7fa17d6e85e0> and will run it as-is

Click to expand!
Issue Type
Bug
Have you reproduced the bug with TF nightly?
No
Source
binary
Tensorflow Version
2.12.0
Custom Code
Yes
OS Platform and Distribution
Linux Ubuntu 22.04
Mobile device
No response
Python version
3.10.6
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
CPU only
GPU model and memory
No response
Current Behaviour?
Running tensorflow_decision_forests I saw a warning in the output while running model.fit() which asked me to file a bug report.
Prediction seems okay, but I cannot be 100% sure about it.
Dataset files needed to reproduce the case:
train.csv
test.csv
List of Python modules:
pip_list.txt
Standalone code to reproduce the issue
import pandas as pd
import tensorflow_decision_forests as tfdf

def type_conversion_tfdf(df_orig):
    df = df_orig.copy(deep=True)

    df[["PassengerId_1", "PassengerId_2"]] = df["PassengerId"].str.split(
        pat="_", n=1, expand=True
    )
    df[["Name_First", "Name_Last"]] = df["Name"].str.split(pat=" ", n=1, expand=True)
    df[["Deck", "Cabin_Num", "Side"]] = df["Cabin"].str.split(pat="/", n=2, expand=True)

    bool_to_int_columns = ["Transported", "CryoSleep", "VIP"]
    for c in bool_to_int_columns:
        if c in df.columns.to_list():
            df[c] = df[c].fillna(value=0).astype(bool).astype(int)

    categorical_columns = [
        "HomePlanet",
        "Destination",
        "PassengerId_2",
        "Name_First",
        "Name_Last",
        "Deck",
        "Side",
    ]
    for c in categorical_columns:
        # df[c] = df[c].astype("category")
        pass

    int_columns = ["Cabin_Num"]
    for c in int_columns:
        df[c] = df[c].fillna(value=0).astype(int)

    return df.drop(columns=["PassengerId", "PassengerId_1", "Cabin", "Name"])

pd.options.mode.copy_on_write = True

train_df = pd.read_csv("train.csv")
test_df = pd.read_csv("test.csv")

train_df_tfdf = type_conversion_tfdf(train_df)
test_df_tfdf = type_conversion_tfdf(test_df)

train_ds_tfdf = tfdf.keras.pd_dataframe_to_tf_dataset(
    train_df_tfdf, label="Transported"
)
test_ds_tfdf = tfdf.keras.pd_dataframe_to_tf_dataset(test_df_tfdf)

rf = tfdf.keras.RandomForestModel()
rf.fit(x=train_ds_tfdf)

predictions = rf.predict(test_ds_tfdf)

Relevant log output
2023-04-27 10:53:42.381793: I tensorflow/core/platform/cpu_feature_guard.cc:182] This TensorFlow binary is optimized to use available CPU instructions in performance-critical operations.
To enable the following instructions: AVX2 FMA, in other operations, rebuild TensorFlow with the appropriate compiler flags.
2023-04-27 10:53:42.848387: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Could not find TensorRT
2023-04-27 10:53:43.514073: I tensorflow/compiler/xla/stream_executor/cuda/cuda_gpu_executor.cc:996] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero. See more at https://github.com/torvalds/linux/blob/v6.0/Documentation/ABI/testing/sysfs-bus-pci#L344-L355
2023-04-27 10:53:43.540254: W tensorflow/core/common_runtime/gpu/gpu_device.cc:1956] Cannot dlopen some GPU libraries. Please make sure the missing libraries mentioned above are installed properly if you would like to use GPU. Follow the guide at https://www.tensorflow.org/install/gpu for how to download and setup the required libraries for your platform.
Skipping registering GPU devices...
Use /tmp/tmpsuc8grc3 as temporary training directory
Reading training dataset...
2023-04-27 10:53:43.603559: I tensorflow/core/common_runtime/executor.cc:1197] [/device:CPU:0] (DEBUG INFO) Executor start aborting (this does not indicate an error and you can ignore this message): INVALID_ARGUMENT: You must feed a value for placeholder tensor 'Placeholder/_8' with dtype string and shape [8693]
         [[{{node Placeholder/_8}}]]
Training dataset read in 0:00:02.534852. Found 8693 examples.
Training model...
[INFO 23-04-27 10:54:06.0499 PDT kernel.cc:1242] Loading model from path /tmp/tmpsuc8grc3/model/ with prefix fd4ad06ed85c449e
[INFO 23-04-27 10:54:06.5441 PDT decision_forest.cc:660] Model loaded with 300 root(s), 292454 node(s), and 16 input feature(s).
[INFO 23-04-27 10:54:06.5441 PDT abstract_model.cc:1311] Engine "RandomForestGeneric" built
[INFO 23-04-27 10:54:06.5441 PDT kernel.cc:1074] Use fast generic engine
Model trained in 0:00:20.491433
Compiling model...
2023-04-27 10:54:06.630203: I tensorflow/core/common_runtime/executor.cc:1197] [/device:CPU:0] (DEBUG INFO) Executor start aborting (this does not indicate an error and you can ignore this message): INVALID_ARGUMENT: You must feed a value for placeholder tensor 'Placeholder/_0' with dtype double and shape [8693]
         [[{{node Placeholder/_0}}]]
WARNING:tensorflow:AutoGraph could not transform <function simple_ml_inference_op_with_handle at 0x7fa17d6e85e0> and will run it as-is.
Please report this to the TensorFlow team. When filing the bug, set the verbosity to 10 (on Linux, `export AUTOGRAPH_VERBOSITY=10`) and attach the full output.
Cause: could not get source code
To silence this warning, decorate the function with @tf.autograph.experimental.do_not_convert
WARNING:tensorflow:AutoGraph could not transform <function simple_ml_inference_op_with_handle at 0x7fa17d6e85e0> and will run it as-is.
Please report this to the TensorFlow team. When filing the bug, set the verbosity to 10 (on Linux, `export AUTOGRAPH_VERBOSITY=10`) and attach the full output.
Cause: could not get source code
To silence this warning, decorate the function with @tf.autograph.experimental.do_not_convert
Model compiled.
2023-04-27 10:54:07.749903: I tensorflow/core/common_runtime/executor.cc:1197] [/device:CPU:0] (DEBUG INFO) Executor start aborting (this does not indicate an error and you can ignore this message): INVALID_ARGUMENT: You must feed a value for placeholder tensor 'Placeholder/_13' with dtype double and shape [4277]
         [[{{node Placeholder/_13}}]]
5/5 [==============================] - 0s 60ms/step

Hi @FlorinAndrei ,
I have replicated the behaviour and we have to dig into more whether this is a bug or not. Reference gist here.
I request you to set verbosity to 10 using command export AUTOGRAPH_VERBOSITY=10 and submit the logs for us, As colab not generating complete logs(particularly at C++ level). This might be helpful to analyse the issue. Thanks!

@SuryanarayanaY I did export AUTOGRAPH_VERBOSITY=10 before launching the script with python3 script_name.py. This was on an Ubuntu 22.04 machine that I have ssh-ed into. The log output you see is all the output I got from the script.
Is there anything else I should do? Just let me know what you need.

Hi @FlorinAndrei ,
From the error log,it seems this warning is due to the reason that may The Tensorflow seems does not have the access to the source code of the some of the function from TensorFlow Decision Forests (TF-DF) which is powered by Yggdrasil Decision Forests (YDF).Please refer this source.
WARNING:tensorflow:AutoGraph could not transform <function simple_ml_inference_op_with_handle at 0x7fa17d6e85e0> and will run it as-is.
Please report this to the TensorFlow team. When filing the bug, set the verbosity to 10 (on Linux, `export AUTOGRAPH_VERBOSITY=10`) and attach the full output.
Cause: could not get source code

Also Tensorflow couldn't create Autograph for lambda functions which is known issue.
In fact this is just a warning to users let them know that autograph is not constructed for a particular function and running it as it is without graph.Yeah this have some performance implications but not sure this can be fixable or not especially when there are other libraries involved.For other library functions TF might have not access to Source code and this is causing this warning. This warning is visible in few official Tensorflow Tutorials also and I am attaching few for your references.Refer source1,source2,source3 etc.
I hope this will provide some additional info for you. Thanks!

The warning literally tells people to inform the TF team about it, which I did.
Either this requires some action from the TF team, or the warning needs to be removed (which is also "action required").

Hi @FlorinAndrei ,
I appreciate that you have reported as per the logs. I will escalate this to concern team and let you updated. Thanks!

@FlorinAndrei ,
AFAIK Autograph cannot convert those functions for which source code is not available or those functions from third party libraries. However the log message also needs to look into whether we need to change this.

The warning literally tells people to inform the TF team about it, which I did.
Either this requires some action from the TF team, or the warning needs to be removed (which is also "action required").

But you can't really expect as it did require

Hi @FlorinAndrei ,
From the error log,it seems this warning is due to the reason that may The Tensorflow seems does not have the access to the source code of the some of the function from TensorFlow Decision Forests (TF-DF) which is powered by Yggdrasil Decision Forests (YDF).Please refer this source.
WARNING:tensorflow:AutoGraph could not transform <function simple_ml_inference_op_with_handle at 0x7fa17d6e85e0> and will run it as-is.
Please report this to the TensorFlow team. When filing the bug, set the verbosity to 10 (on Linux, `export AUTOGRAPH_VERBOSITY=10`) and attach the full output.
Cause: could not get source code

Also Tensorflow couldn't create Autograph for lambda functions which is known issue.
In fact this is just a warning to users let them know that autograph is not constructed for a particular function and running it as it is without graph.Yeah this have some performance implications but not sure this can be fixable or not especially when there are other libraries involved.For other library functions TF might have not access to Source code and this is causing this warning. This warning is visible in few official Tensorflow Tutorials also and I am attaching few for your references.Refer source1,source2,source3 etc.
I hope this will provide some additional info for you. Thanks!

Well this would make it quite obvious of the fact that the warning was referencing to action

@SuryanarayanaY I did export AUTOGRAPH_VERBOSITY=10 before launching the script with python3 script_name.py. This was on an Ubuntu 22.04 machine that I have ssh-ed into. The log output you see is all the output I got from the script.
Is there anything else I should do? Just let me know what you need.

we will also need to ensure smooth functionality

Hi everyone,
TF-DF developer here. Thank you very much for reporting this! We just fixed the issue at head and the next version of TF-DF will include the fix.

awesome, thanks! Close the issue for now as the issue with TF-DF is fixed.


Parse (a - b) as sub BinaryOp instead of add+negate

Sub is currently represented by add + negate. This is unnecessary confusion, we should instead add a sub binary op.

I can take this one if you haven't worked on it yet. Please confirm.

At the parse level,
(A- B) is parsed as Add(A, Minus(b)), rather than Sub(A,B)
At the parser level:

Parse Tree's BinaryOp has a 'Add' but no 'Sub'
https://github.com/microsoft/Power-Fx/blob/main/src/libraries/Microsoft.PowerFx.Core/Lexer/BinaryOp.cs
And Unary op has the minus: https://github.com/microsoft/Power-Fx/blob/main/src/libraries/Microsoft.PowerFx.Core/Lexer/UnaryOp.cs

The parser tree then goes through Binding, and then converted into an "execution tree" (called the IR Tree).
See IRTranslator.cs.
The IR's binary operators also just have Add, but no sub:
https://github.com/microsoft/Power-Fx/blob/main/src/libraries/Microsoft.PowerFx.Core/IR/Nodes/BinaryOpKind.cs
Note that Syntax Binary op just has a single add operator because it maps to the + symbol. But the IR has multiple Add operators because now we know the types and can determine what kind of add it is.

Once this is fixed, then we can clean up this spot:
https://github.com/microsoft/Power-Fx/blob/4b8a3e1e7bbe48f7e5870367cebbaaea10d633f7/src/libraries/Microsoft.PowerFx.Core/IR/IRTranslator.cs#L756
Introduced in #356.

Thanks for taking the time to explain.


[WIP] Change the way NDArrayIter handle the last batch

Description
Update some behaviors of NDArrayIter.

Change the functionality of class NDArrayIter last_batch_handle = roll_over.
Change the timing of shuffling. Previously, it shuffles only during the initialization, which didn't meet training needs.

Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[x] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[x] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[x] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

shuffle when calling the reset.
when last_batch_handle is roll_over, the last batch should be rolled over to next epoch
add the unit test for three last_batch_handle params

Comments
N/A

What's the behavior of 'roll_over'? I think the doc is not very clear.

roll_over means the remaining elements of the last batch would be rolled over to next iteration if the last batch is less than the batch size


Different versions installed with pip depending on python version

py -3.10 -m pip install survey installs survey==3.4.0
whereas
py -3.11 -m pip install survey installs survey==4.3.0
Is this intended?

That's curious, it's not intended but I can understand why it may be happening.
The 4.* release requires python 3.11, whereas the 3.* is happy with python 3.10. I guess pip is trying to install the latest viable version, rather than the latest (and if the respective python version is not supported, fail with an error). I'm unsure on how to force the latter behavior from the package's end.

That's curious, it's not intended but I can understand why it may be happening.
The 4.* release requires python 3.11, whereas the 3.* is happy with python 3.10. I guess pip is trying to install the latest viable version, rather than the latest (and if the respective python version is not supported, fail with an error). I'm unsure on how to force the latter behavior from the package's end.

Why it requires Python 3.11?

A few features from python 3.11 are used:

StrEnum
weakref_slot for dataclasses.dataclass
typing.Unpack

That being said, most of these have workarounds to achieve 3.10 compatibility:

Can be replaced with (str, enum.Enum)
Can be replaced with a simple class implementing __weakref__ in __slots__.

However, [3.] has no way of being handled in python 3.10. But, since it is only used in 2 instances - 1 of them being a private function and the other being the signature typehint for Visual.funnel_enter (an internal mechanism) - They could be removed.
These changes are now part v4.4.0, which is on pypi.


Databricks integration example

A new integration example showing to enable Infrastructure Monitoring and optionally Apache Spark metrics on the controller or worker. An installation script example has been included to install the splunk OpenTelemetry collector.  The readme now includes more details into executing the start-up script included.
CLA:
I have read the CLA Document and I hereby sign the CLA
Code of Conduct:
I have read the Code of Conduct and I hereby accept the Terms

I also added a link in the script for reference to https://docs.databricks.com/clusters/init-scripts.html


Build error

When I try to build the latest wasmoon, it comes the following error messages.
root@5d128954f212:/wasmoon# npm run build

><EMAIL_ADDRESS>build
> rollup -c && tsc -d --emitDeclarationOnly --rootDir src --declarationDir dist

[!] SyntaxError: Unexpected identifier
    at Loader.moduleStrategy (internal/modules/esm/translators.js:145:18)
    at link (internal/modules/esm/module_job.js:67:21)

Hey! To build wasmoon you need node 16+. Which version are you using?

I update my nodejs from 14 to 18 and run the build. There appear the following warnings.
root@5d128954f212:/wasmoon# npm run build

><EMAIL_ADDRESS>build
> rollup -c && tsc -d --emitDeclarationOnly --rootDir src --declarationDir dist

(node:444) ExperimentalWarning: Importing JSON modules is an experimental feature. This feature could change at any time
(Use `node --trace-warnings ...` to show where the warning was created)

./src/index.ts → dist/index.js...
created dist/index.js in 3.4s

Just ignore it, it's just a warning about an experimental feature we are using


style: linter error fixes

A few random linter error fixes.

I don't see anything in our Makefile to run this. Do you think you can add a command?

Maybe we should try this: https://github.com/golangci/golangci-lint-action


fix route_reflector_client in get_bgp_config()

This fixes the route_reflector_client key in get_bgp_config(), as far as I can see it's always set to False. This should work when applying the cluster command in group or neighbors level, but not in global config. As far as I can see all commands applied globally are ignored, not sure if there is a reason for that.
With the following configuration example:
group internal {
    type internal;
    local-address <IP_ADDRESS>;
    cluster <IP_ADDRESS>;
    neighbor <IP_ADDRESS> {
        description routerA;
    }
    neighbor <IP_ADDRESS> {
        description routerB;
    }
}

both neighbors are route reflectors clients:
carles@vmx# run show bgp neighbor | display xml | match "peer-address|client"
            <peer-address><IP_ADDRESS></peer-address>
            <route-reflector-client/>
            <peer-address><IP_ADDRESS></peer-address>
            <route-reflector-client/>

so the output from get_bgp_config() should be:
        "neighbors": {
            "<IP_ADDRESS>": {
                "description": "routerA",
                "route_reflector_client": true
            "<IP_ADDRESS>": {
                "description": "routerB",
                "route_reflector_client": true

Please let me know if I missed something or if it's breaking other things...thanks

Coverage decreased (-0.3%) to 78.514% when pulling 4623a1132377f104730a72c81ae47e3e6b1d2722 on ckishimo:devel-rr into b087b8b0ea90d0b8fb88a3315bc72e97b36d5d44 on napalm-automation:develop.


Add payload example to readme

Add explanation of how to add extra data to tree nodes. It wasn't immediately obvious how to do this, even though it's quite simple.

I am not sure if this should be the documented (and therefore recommended) way of adding payload to nodes.
One alternative way would be to index not tuples but other data structures that already contain the payload. It might also be worth considering to provide the payload through the add() method.
Currently I'm leaning more towards recommending to have the payload in the indexed data structure.

A defined data type for each node sounds good. I guess it just needs to be indexable, right? (My Python's not that great).
I noticed rebalance() recreates nodes, so any extra attached data is lost. I fixed this without specifying a node data type, but I don't think it's an elegant solution.


"Animation Example" link leads to a non-existant page

In Introduction -> READ THE RESOURCES there is a link: "Animations Example" which leads to https://github.com/facebook/react-native/blob/master/Examples/UIExplorer/AnimatedExample.js which doesn't exist

In the same section, "Gratuitous Animation Examples" leads to https://github.com/facebook/react-native/tree/master/Examples/UIExplorer/AnimatedGratuitousApp which also doesn't exist

Thank you, I'll fix those links. they have moved in the git repo


 The idea behind this change is that we want to simplify the use of params and put all info under the job-spec param.
Notable changes:

I've removed the pr_author param as I believe it was redundant since we already have the pull_request_author param
Shortened git-url and git-revision to be just url and revision. Since they are nested under git {} in the job-spec json they will be used as git.url and git.revision in tasks. This avoids the git.git-url situation.

LGTM
@ascerra feel free to merge


feat: deprecate -f

closes #539
Summary:
This PR provides support to make -f an optional argument when running the validator. This is required to run the validator programatically for:

integration to the MobilityDatabase
integration tests purpose

Expected behavior:
if -f  is now optional. If provided, phone numbers will be validated according to the country code, else phone number validation is skipped.
No NPEs should be thrown.
Please make sure these boxes are checked before submitting your pull request - thanks!

[x] Run the unit tests with gradle test to make sure you didn't break anything
[x] Format the title like "feat: [new feature short description]". Title must follow the Conventional Commit Specification(https://www.conventionalcommits.org/en/v1.0.0/).
[x] Linked all relevant issues
~[ ] Include screenshot(s) showing how this pull request works and fixes the issue(s)~

@barbeau  Question: should we modify -f to be -c (meaning country code)?

Thanks for working in that direction!
One more thing came to my mind. Let's remove the monolithic ValidationContext and inject only direct dependencies.
"Inject only direct dependencies" is one of the main DI principles at Google. Here is what it means.
This approach is not recommended:
public class ShowBudgets {
   private final Account account;

   @Inject
   ShowBudgets(Customer customer) {
     account = customer.getPurchasingAccount();
   }

This one is preferred:
public class ShowBudgets {
   private final Account account;

   @Inject
   ShowBudgets(Account account) {
     this.account = account;
   }

So, instead of injecting ValidationContext, we should inject the CountryCode instead.
We probably should not inject ZonedDateTime but we need a wrapper class for it (say, CurrentDateTime) because ZonedDateTime does not have a semantic that it is the current datetime.
How does that sound?

How does that sound?

Good to me! 👍🏾

Oh, if this is a breaking change, the commit/PR title should include a ! - https://www.conventionalcommits.org/en/v1.0.0/

Removing mocks from CliParametersAnalyzerTest in #854

Thanks @aababilov  for reviewing! I provided the changes you suggested in the previous commits. At the exception of this one:  https://github.com/MobilityData/gtfs-validator/pull/851#discussion_r615135074

private static final String ZZ = "ZZ";

I made this constant public, so that we can access from the main method in order to avoid NPE.

Beautiful! Please format the code and submit it, I can't wait to see it in upstream!
google-java-format -i $(find . -name '*.java')

Thanks @aababilov for this feedback!


Upgrade to jbang 0.107

We have in one Dockefile
FROM docker.io/jbangdev/jbang-action
And in another
FROM jbangdev/jbang-action:0.106.1
I wonder if we can align this. And either use latest or use a specific version

Base images sync
The latest version of image in dockerhub is [0.106.1]


Feature/ searchbar for idle members and tasks (updated)

Extends #231

Good work Vinayak, Added a few comments however I have not reviewed the code completely as I observe a lot of indention issues here.

@harshith-venkatesh
On the frontend side of things right now I'm planning to do a lot of refactoring and that is in the process.
Right now only this file has been refactor.
@ankurnarkhede recommended I do small incremental refactoring so that the review process is easy for all.
Regardless of that Thank you for your review, I will keep these points in mind.

Thanks for the update @ivinayakg


Give a Feedback!!

Welcome!!
Thanks for visiting SOHW! Want to ask or say something? Yaay!! share it here or ask us a question. We are glad to answer. More questions creates new ideas and solve big problems 😃
Please comment here if you're not sure where to put your thoughts!
Comments can include, but are not limited to:
Clarifications: If something that you've read on the repository or website isn't clear please let us know!
Questions: No question is too tiny to add here!
Suggestions: No suggestion is too general or early in conception to add here!
Mad props: If you like what we're doing and just want to say 👍, ✨ or 😃 please do so!
Cheers!!

Awesome job @FahimaZulfath 👍 Also do we have to add ourselves to contributors.yml?

@Tlazypanda Thanks for fixing the bug. Yes, add yourself to the contributor.yml and it will glad if you introduce yourself to the issue


Can't run the workspace

i have complete all instalation and creat new workspace but i can't run the workspace

http://prntscr.com/n02hzw
http://prntscr.com/n02ihr

There could be 2 different things going on here. First, make sure you have properly installed the cloud9sdk. This includes running the install script (this is the most likely cause of your issue).
The other problem may have been that you did not include the proper path in the config. Make sure the path is to the directory that contains all the cloud9 scripts. For example, if the script itself was ~/c9sdk/server.js, your path should be ~/c9sdk.

i have installed the cloud9sdk
check this out http://prntscr.com/n032nu

You have installed it to /root/c9sdk
Change that in your config to /root/c9sdk

i have change the path config , but i got error , and what is the mean of crypto in config ?
https://prnt.sc/n03p2o
i'm sorry i'm bad in english

Crypto is the key used for encrypt the passwords that you use when you launch instances of cloud9.
If cryptr is giving you issues, try checking your nodejs version. I created this in NodeJS v8.

i'm using nodeJS v8.15.1,please check this one again , and thank you before 👍
http://prntscr.com/n041nb

Solved , Thank you !!!

Glad you fixed it. Crypto should be any random string of characters, but it cannot be changed after you create a workspace.


Font size - text gets cut/displayed improperly.

Bug Report
Description

For higher font-sizes , text in the grid gets cut/hidden/not proper.

Steps to Reproduce

Terra Clinical DataGrid example
Set the body font-size to 30px.

Additional Context / Screenshots

Expected Behavior

Text is properly displayed

Possible Solution
Environment

Component: terra-clinical-data-grid and Version:2.12.0
Browser: Chrome , Version : 79
MacOS

@ Mentions

[](url)
##Watch
Kurian, Habel

@chaitu9916

The recommended UX pattern is to have base font-size of 14px for open-source theme and for orion-fusion-theme, it is12px.
Is there a reason for you to set it to 30px and have you considered using the default zoom instead?

In my opinion, this may not be a valid issue. @neilpfeiffer @tbiethman any thoughts?

We dont have a use case of 30 px today. But we were hoping the row height (supplied as rems) for clinical data grid will adjust based on the base font size.

@neilpfeiffer, @jeremyfuksa  could you guys please confirm is this the valid workflow ?

This is not a valid bug as currently described. There are two issues needing to be corrected:

In order for the rem sizing to correctly cascade and adjust UI elements that are not text -- consumers need to adjust the font-size not on the body, but on the html element (body should always be set to 1rem). Try it and you will see the data-grid row heights correctly resize (if set to rems not pixels). cc: @chaitu9916 @habelk

Restating for clarity -- someone new to the data-grid may assume that it is a flexible table that should respond to it's content, however, the data-grid is by design a fixed-size-concept component - where it does not respond or grow to its content, but has rigid heights set by the consumer. If a consumer needs larger row heights due to larger content, it is the consumers responsibility to set accordingly. Using rem sizing will allow for both content and row height to appropriately adjust if the html font-size is changed.  (describing this for if any other consumers who might see this and might also be confused)

Closing as not an issue.


Preconditioner and Factorization config

This pr contains the config interface for preconditioner and factorization.
TODO:

[x] how to distinguish the ILU between preconditioner and factorization. I currently use Factorization as prefix. (it is only internal)

@thoasm can you either approve the PR or request changes?

I wonder whether the instantiation of preconditioner Ic/Ilu in config will leads the duplicated symbol if user also use the same instantiation in their own code.
For function, we can use inline. but I was not sure for the class.
from https://en.cppreference.com/w/cpp/language/inline

A function defined entirely inside a class/struct/union definition, whether it's a member function or a non-member friend function, is implicitly an inline function unless it is attached to a named module(since C++20).

should it be safe?

For function, we can use inline. but I was not sure for the class.

template functions automatically emit weak symbols, so we don't need to worry about that


migration files not published under Laravel 5.4

Under Laravel 5.4 database migration files are not published when installing
running this command:
php artisan vendor:publish only publishes config file
$ php artisan vendor:publish
Copied Directory [\vendor\laravel\framework\src\Illuminate\Not
ifications\resources\views] To [\resources\views\vendor\n
otifications]
Copied Directory [\vendor\laravel\framework\src\Illuminate\Pag
ination\resources\views] To [\resources\views\vendor\pagi
nation]
Copied File [\vendor\santigarcor\laratrust\src\config\config.p
hp] To [\config\laratrust.php]
Copied File [\vendor\santigarcor\laratrust\src\config\laratrus
t_seeder.php] To [\config\laratrust_seeder.php]
Copied Directory [\vendor\laravel\framework\src\Illuminate\Mai
l\resources\views] To [\resources\views\vendor\mail]m

Copied File [\vendor\waavi\translation\config\translator.php]
 To [\config\translator.php]
Publishing complete.

+1

I have installed some packages that have validations in their vendor folders, previously one of the packages I used published these migrations to migrations folder by running:
php artisan vendor:publish only publishes config file
Now I found alteast 2 packages that no longer published migrations when running this command, so I went into vendor folder and grabbed migration file and manually moved it and when I run
composer dump-autoload I got a warrning message from it that said: Warning: Ambiguous class resolution, "CreateRevisionsTable" was found in both "$baseDir . '/database/migrations/2013_04_09_062329_create_revisions_table.php" and "C:\xampp\htdocs\example\vendor/venturecraft/revisionable/src/migrations\2013_04_09_062329_create_revisions_table.php", the first will be used.
why is this happening? Has something changed in 5.4?

+1 me to

Laravel 5.4 changed the way migrations are managed.
You do not need to publish the migration files, php artisan migrate will automatically run this package's migrations.


update style to follow D211

https://hg.python.org/peps/rev/9b715d8246db
Which is actually being checked for by newer versions of pep257 / pydocstyle.

+1


Any way to load custom fonts when calling SvgDocument.Draw?

We need to load custom fonts for rendering images, but I can't find any way to pass a System.Drawing PrivateFontCollection into the rendering - it looks like there's a hardcoded reference to System.Drawing.FontFamily.Families here, blocking out the ability to load custom fonts from disk/memory.
Given our deployment scenario, we cannot simply install the font into Windows to add the custom font to the drawing context and it seems like a PrivateFontCollection is the only exposed interface that would allow us to do so.
Is there some other way I'm missing? Or will this require a refactoring of SvgElementStyle's ValidateFontFamily method? https://github.com/vvvv/SVG/blob/44173b47e29429f98db60413f5058b016d1024f4/Source/SvgElementStyle.cs#L386

hope this is closed now with the mentioned pull request. otherwise please reopen it.


[iOS] Connect API using KMP module

Issue

close #ISSUE_NUMBER

Overview (Required)

Links

Screenshot (Optional if screenshot test is present or unrelated to UI)

Before
After

Movie (Optional)

Before
After

I was wondering if you could try this. But I'll leave it to you 🙇 https://github.com/rickclephas/KMP-NativeCoroutines#swift-swift-package-manager #862

I think introducing KMP-NativeCoroutines is not only in iOS, so I didn't in this PR. But I'll try it!


fix: raise primary owner removal upgrader order

this upgrader must run after the default system role upgrader to avoid failures causing the process to be skipped.

🚀 CI was able to deploy the build of this PR, so you can now try it directly here
Notes: The deployed app is linked to the management API of APIM master. (Same login and password as APIM master)

📚  View the storybook of this branch here

should be fixed when https://github.com/gravitee-io/gravitee-api-management/pull/2600 is merged back on master


Add workflow and OpenSSF badges to README

Only add badges.
This should further improve OpenSSF score.

@ekomarova @xaleryb Please approve since merging without such approval would negatively impact the OpenSSF score.

This is how badges look like now:


S3 resume support

Reopening because of auto-close of PR implementing partial feature.

Resume not possible at the moment since the crate providing the S3 interface needs to provide the stream object used in async io.


Reinforcement learning

reinforcement part i have made using python

Kindly add your name and github in README

@Akshat111111 done


Support for Steem

Description
Adds support for Steem blockchain. We have reused a lot of Bravo's code to add the support, thanks to their similarities.
Testing instructions
We have added tests in src/tests as well as swift and kotlin tests in their respective directories
Types of changes

New feature (non-breaking change which adds functionality)

Checklist

[ ] Prefix PR title with [WIP] if necessary.
[x] Add tests to cover changes as needed.
[ ] Update documentation as needed.

we have noticed indentation issues in src/Coin.cpp and src/interface/TWCoinTypeConfiguration.cpp files. we will fix it together with review changes if any.

Looks good other than indentation.

@alejandro-isaza Not sure about the CI status "Waiting for status to be reported". Do we need to rebase again?

CI is not triggered somehow, and we refactor a bit on master, could you please rebase and fix the indentation?

Indentation is fixed. Let me rebase again.

@alejandro-isaza we rebased again but for some reason, CI is not spinning up.

/AzurePipelines run

@hewigovens It is done. can you please merge if everything is okay. we will raise another for EOS after merge.

/AzurePipelines run


Changing queue overflow response code to 503

Changing from custom response code 513 to RFC7231 https://tools.ietf.org/html/rfc7231#section-6.6.4 response code 503 "Service Unavailable"

Can one of the admins verify this patch?

👍

This is ok to test

@matoikar thanks for your PR! I created a Jira for ir, can you please add it to the commit message?
https://issues.jboss.org/browse/UNDERTOW-1537
Just add [UNDERTOW-1537] to the beginning of the message. Thanks!

@fl4via Thanks Flavia, updated the commit message to start with the ticket id.


chore: update to react native 67

This PR resolves MOPLAT-664
Description

PR Checklist

[ ] I have tested my changes on iOS and Android.
[ ] I hid my changes behind a feature flag, or they don't need one.
[ ] I have included screenshots or videos, or I have not changed the UI.
[ ] I have added tests, or my changes don't require any.
[ ] I added an app state migration, or my changes do not require one.
[ ] I have documented any follow-up work that this PR will require, or it does not require any.
[ ] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

think this is circle funkiness, just gonna close and hope it gets resolved.


Unified State Future Infrastructure Projects/Planning Coordination

Benefit
Working with Laura Hansen and Evan Curtis at GOPB on a map similar to Wasatch Choice 2050 or UDOT's unified plan:

Layers:

Future Transportation Projects/needs
Energy Transmission Corridors
Water Projects
UIPA
New state parks
any future physical infrastructure project

Displayed in an app (ExB if at all possible) that a legislator could pull up in a meeting and say "We've got x dollars, where should we spend it?"
Current focus on state agencies, but future dream is to pull in regional and local plans (wasatch choice 2050, etc)

Acceptance Criteria
For this quarter:

[ ] Proof of concept ExB app
[ ] gather as many datasets as we can

Notes
Goal is to have the final product by Aug of 2025, so we've got time to build a proof of concept and iterate through, hopefully build momentum amongst agencies by showing what we've got.
Risks

Getting data from agencies
ExB not providing all the functionality we want

Issue Reference
refs #

25.1.2

[ ] POC App based on wireframe from Laura
[ ] Start gathering datasets

25.1.4
Waiting on Laura Hansen and co to provide list of desired data and identify potential sources.

25.1.6
Laura and Co are working on datasets and collaboration. We're just sitting in a holding pattern to advise during this stage.


create a new blog

Hi @zzzyer, why do you just commit your profile without any blog? I guess you are going to post a blog, and looking forward to your blog. When you commit your new blog, I will review your PR.


model format availability (tflite, ONNX)

back before the main release of 2023, all the models were available as .tflite which allows us user to easilly convert them into .onnx which is widely used.
As by now the models are avilable only as .task, is it possibile to have have a feature to convert the model to a tflite, hence to onnx?
Thanks

@valsecchi,
The task file encompasses the .tflite model, and you can extract the TFLite files by executing the following command, exemplified by the face landmarker.task:

Upon executing the above command, you will obtain the TFLite files as outlined below:
- Extracting: face_detector.tflite
- Extracting: face_landmarks_detector.tflite
- Extracting: face_blendshapes.tflite
Regrettably, we do not offer support for ONNX input or its corresponding conversion. Additionally, the MediaPipe team neither recommends nor supports the conversion of models from TFLite. Thank you

Thanks a lot!


ccs 6.4.0 reproducibility with 6.3.0 and 6.2.0

Hi,
I have a complex pipeline developed with ccs 6.2.0.
I have refrained from upgrading to the latest version of ccs due to concern that consensus sequence and quality scores won't be exactly reproducible, specifically for "--by-strand ccs"
=> Question: Is ccs 6.4.0 --by-strand producing the same exact 1) consensus sequences 2) base quality scores, 3) rq scores as ccs 6.2.0 ?
Assume this run command: ccs --by-strand --min-rq 0.99 --top-passes 0
Thanks.

They might differ slightly as we are constantly swapping internal algorithms and their implementations to enable future speedups.

Ok thanks.


Fix upgrade stuck when using incorrect isoChecksum in version CR

Problem:
The upgrade stuck when isoChecksum isn't in SHA512 format in Version CR. The root cause is in createBackingImageAndStorageClass we missed to invoke handleFail thus the max retry never reached to maximum count and lead to the infinite loop of vm_image_controller.go#onChanged
Solution:
Applied two things here.

Add Version isoChecksum validation, make it fail fast if the checksum isn't in SHA512 format.
Invoke handleFail when BackingImage, StorageClass creation fails.
though 1. could avoid invalid isoChecksum passed to BackingImage, the VMImage required two things, StorageClass and BackingImage, and there might have other potential failures in BackingImage/StorageClass creation other than invalid isoChecksum.

Related Issue:
related to #5480
Test plan:

Test version with incorrect isoChecksum
After applying the following version.yaml, the console should return error message like admission webhook "validator.harvesterhci.io" denied the request: invalid isoChecksum asdf since asdf isn't in sha512 format.apiVersion: harvesterhci.io/v1beta1
kind: Version
metadata:
  name: master
  namespace: harvester-system
spec:
  isoURL: http://<IP_ADDRESS>:8000/harvester-v1.3.0-amd64.iso
  isoChecksum: asdf

Test version with correct isoChecksum and run upgrade

Apply the correct version.yaml to K8s
Click the upgrade button in harvester dashboard
Everything should works fine

Test vm image

Go to harvester UI images and create image with mismatch SHA512 checksum
The vm image should download 4 times and then failed with error message like Failed to process sync file: the expected checksum ... doesn't match the file actual checksum ...

This pr is based on master branch, while the issue also happens in v1.3 & v1.2 branch, so we should backport this pr after this pr merged to master.

@brandboat Can you provide another test plan purely in the image path? Some changes have been made in the vm image controller, I'd prefer if there is also a test plan in the image path only.

@brandboat Can you provide another test plan purely in the image path? Some changes have been made in the vm image controller, I'd prefer if there is also a test plan in the image path only.

@bk201, many thanks ! I've updated the vm image test plan.

@brandboat Due to the revert https://github.com/harvester/harvester/commit/18973f46329dd06587c3d98f0abae126e7bdb1fd#diff-271c0b1384d0424ab45c719482f3c867166e7c48319fee9754bf8938f56f67ba
The PR can't build correctly due to some missing fields, can you help rebase again, thanks!
you can check the output here: https://drone-pr.rancher.io/harvester/harvester/4726/1/2

@brandboat Due to the revert https://github.com/harvester/harvester/commit/18973f46329dd06587c3d98f0abae126e7bdb1fd#diff-271c0b1384d0424ab45c719482f3c867166e7c48319fee9754bf8938f56f67ba
The PR can't build correctly due to some missing fields, can you help rebase again, thanks!
you can check the output here: https://drone-pr.rancher.io/harvester/harvester/4726/1/2

Thanks for the reminder, already rebased to the latest commit in master branch :smiley:

@brandboat I'm trying to create a virtual machine with the change, but it seems the field status.storageClassName is not set within an image. GUI and the Harvester Terraform provider are now relying on this field to get the correct storage class name.

@brandboat I'm trying to create a virtual machine image with the change, but it seems the field status.storageClassName is not set within an image. GUI and the Harvester Terraform provider are now relying on this field to get the correct storage class name.

I can reproduce the error. @brandboat, we have to call h.images.Update(toUpdate) if it's happy path. Currently, we only call Update when there is error.

Already fixed the errors... Thanks you @bk201 , @FrankYang0529 ! :smiley:

@mergifyio backport v1.2 v1.3


Add r2d2-odbc to README.md

Hi, I've recently made a simple adapter which allows to use ODBC connections with r2d2 connection pool.
It would be great if you could add link to this adapter to README so other people could find this adapter easily.

Thanks!


Absurd high fee rate for a Funded Send All transaction

Here's a “Send All” transaction that was created by omni_funded_sendall.
https://blockchair.com/bitcoin/transaction/4310934654fedc2b2a3e5c9949cf3efd4065bc1bec7c4517402ef36b364dec58
That's a whopping 530 satoshis/byte.
This is the config:
  "paytxfee": 0.00000000,
  "relayfee": 0.00001000,

Average tx fee is significantly lower:

Excuse me, but wth? How is that possible?

Hi @dexX7 ,
We have set -txconfirmtarget to 6, still we got sendall txs paying >1,000 sat/B fees.
An example: https://www.blockchain.com/btc/tx/67aaa681fe5b87b0978cb42a559b96f2163e110ab143e7ed094cb1d3522f6dea
this never happen for simple send txs.
Is there any other setting that could help us limit the maximum txfee per tx?
Thanks!

@dexX7 we have same problem. We are seeing this situation sendall tx's.
https://www.blockchain.com/btc/tx/6f1cfeaadd033fb179b82131886d80a56e98a1f8963cf4fb1d313202da4f4c08

The same problem happens with the omni_funded_send method, but never with omni_send.
Sometimes fee exceeds 0.02 BTC for transaction.
@dexX7, can you reopen this issue?

@efpies @fomo8 @letreset
Did you find the cause of this problem?
Do you know steps to reproduce the problem?
In my case, the problem arises absolutely arbitrarily, and I don't know how to debug it.

@du2zy Same, no steps to reproduce. Normal omni_send txs have normal fees, so I don't see how playing with -txconfirmtarget would help. Maybe it implies that omni_funded_sendall uses it and omni_send doesn't. Maybe fees fluctuate like crazy, so 2 consequental tx have fees that are 7 times different from each other.
The tx in my question is literally the most expensive in the whole block.

@efpies
In my case, when using omni_funded_send, some transactions cost big money, some - as usual. I can not understand what it depends on. I have a few ideas, but I’m not ready to test with real money, but I’m not able to catch the problem in testnet.
How did you solve this problem? Ready to pay a high commission if you need to use a funded send?

Yes, we're ready to pay a commission since we don't want to make the tx manually. But we may opt to calculate fees manually based on the mempool state and previous blocks. Maybe it won't be based on fancy formulae, but it will be times cheaper.

Hey guys,
we noticed that some funded transactions had absurd high fees and dig deeper: in these cases the address had a lot of unspent outputs, which created a more and more expensive transaction during the coin selection.
Does this somehow apply to your cases as well?

Looks like this is the case for 1K5M6LqWhLbpse74TezgnWuUx2g4oDhuEc and 1Jkvb8QxUb5EyBpoPaC5rfGTHfG6uYNiQE.
We have a fix for this in the work.

Looks like this is the case for 1K5M6LqWhLbpse74TezgnWuUx2g4oDhuEc and 1Jkvb8QxUb5EyBpoPaC5rfGTHfG6uYNiQE.
We have a fix for this in the work.

Hey @dexX7 ,
Thanks for picking this up, if it is any help, I can provide some more detail about 1Jkvb8QxUb5EyBpoPaC5rfGTHfG6uYNiQE:
We have a product that allow our users to deposit USDT to our system for later commercial activities.
We used omni_funded_sendall to move all the deposits, and 1Jkvb8QxUb5EyBpoPaC5rfGTHfG6uYNiQE was the account we used at that time to pay for the fees.
We used -txconfirmtarget and set it to 6
The from accounts involved in those transactions are new accounts that has no prior activity, except being send some USDTs.
One thing worth mentioning that most of our users are sending USDTs our way through a crypto exchange withdraw, which involve some highly active accounts.

Looks like this is the case for 1K5M6LqWhLbpse74TezgnWuUx2g4oDhuEc and 1Jkvb8QxUb5EyBpoPaC5rfGTHfG6uYNiQE.
We have a fix for this in the work.

sir，may I ask have you fixed it?


IdentityServer4 to redirect back to Angular app with claims

Hi,
Thanks a ton for the quick reply.
As you suggested to do this with an identity server4, so will be great if you suggest me any sample where it is redirecting back to the angular application with all claims from the identity server4 after successful login.
I have created the identity server 4 app and able to redirect to it from my angular app but it's breaking when I am trying to redirect back to my angular application after successful login. Found few examples on google but they are not so clear and not working in my case.
Thanks and regards.

Thanks a ton for the quick reply.

I think you meant to reply to someone, somewhere? You've opened an issue with the angular-oauth2-oidc library just now?

I have created the identity server 4 app and able to redirect to it from my angular app but it's breaking when I am trying to redirect back to my angular application after successful login. Found few examples ... but they're not so clear...

Not sure if you'd find the best help or answers for such a broad and generic question in the issues list of the Angular library though?
If you feel you have a bug or a problem with this library, feel free to open a fresh issue. But please, do add full details, a way to reproduce the situation you're in, and enough details. - Or consider teaming up with a colleague, friend, or consultant (your situation might benefit from some more interactive help)?


requiredConfig and nested config objects

Hi,
I'm writing a plugin for ember-cli-deploy and have a situation where the config can accept nested configuration objects. For instance, in my deploy.js, the config could look like:
ENV['dukstra-sftp']: {
        sftpOptions: {
        username: 'secret',
        .........
        .........
        },
       pluginFeatureA: true,
        .........
        .........

}

It seems like in index.js for my plugin, I can only check for simple parameters and not complex ones:
requiredConfig: ['sftpOptions']  //This works
requiredConfig: ['sftpOptions', 'pluginFeatureA'] //Also works
requiredConfig:['sftpOptions.username'] //Does not work.

I can easily override the ensureConfigPropertySet method in the plugin, but was wondering if it should be  feature available to plugin developers or if I should re-structure my plugin? Give then current state of the project, is overriding the ensureConfigPropertySet the only option?

We have no religious opposition to nested config objects, but there should be a pretty compelling reason for it, IMO. Simple properties are easier to work with for both plugin devs and end-users.

Closing due to no "compelling reason" presented for discussion.

A compelling reason is to preserve config options from other tools, which the plugin wraps.
For example, I'm writing a gcloud-storage plugin and that takes a config like so:
var gcloud = require('gcloud')({
  credentials: {
    'private_key': auth.key,
    'client_email': auth.email
  },
  projectId: auth.project
});

I want to preserve this for consistency.


Async Support

Hey,
I really like the library but I was just wondering do you have async/non IO blocking image loading?
Cheers :)

yes!


Update system-requirements.md

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@MicrosoftHeidi PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


Faster, smaller, (almost) ready for overlays

New features:

Supports DT_FLOAT64.
Uses compute shader to reorient volumes (both background and overlays). Order of magnitude faster than prior CPU-based method.
Scroll-wheel and pinch gestures work on multi-plane view.
Global overlayItem replaced with back properties that describe dimensions and orientation of background image.
webgl.md expanded to describe compute shader rotation implementation, performance and limitations.

Limitations:

Reslicing overlays is always NEAREST, not LINEAR (only normalized values are Texture filterable). Not major concern as NEAREST is often preferable for statistical overlays. Not a problem when 1-to-1 mapping exists between overlay and background.

To Do:

Check rendering on Google Chrome for Windows (ANGLE uses DirectX)
Compute spatial transform for overlay to background (a few lines of code, need functioning overlay loading).
First call to loadVolume() should await completion of init(). At the moment, first volume loads before font is ready.
Colorbar only draws colormap for top-most layer.

New features:

Supports DT_FLOAT64.
Uses compute shader to reorient volumes (both background and overlays). Order of magnitude faster than prior CPU-based method.
Scroll-wheel and pinch gestures work on multi-plane view.
Global overlayItem replaced with back properties that describe dimensions and orientation of background image.

I kept back but just set it as volumes[0] (it references the first item in the volume array)

webgl.md expanded to describe compute shader rotation implementation, performance and limitations.

Limitations:

Reslicing overlays is always NEAREST, not LINEAR (only normalized values are Texture filterable). Not major concern as NEAREST is often preferable for statistical overlays. Not a problem when 1-to-1 mapping exists between overlay and background.

To Do:

Check rendering on Google Chrome for Windows (ANGLE uses DirectX)

DONE. I tested on my Windows Dell XPS using Brave (browser based on Chrome). Also tested Firefox on that same machine. No formal tests, but things seem to work ok.

Compute spatial transform for overlay to background (a few lines of code, need functioning overlay loading).
First call to loadVolume() should await completion of init(). At the moment, first volume loads before font is ready.

I believe I handled this, or at least some related side effects when loading more than one volume at a time. see loadVolumes and refresh

Colorbar only draws colormap for top-most layer.

I have manually fixed all the merge conflicts between our two branches (font, and prototype). Main branch now reflects the state of the art combination of our efforts. Niivue is now an object (pretty much as class). All Niivue properties and methods are part of a Niivue instance. Those properties and methods can be accessed within the Niivue library as this.MyProperty, or this.MyMethod. When creating new methods to be used in Niivue you should assign the method to the Niivue object within the Niivue library as Niivue.prototype.MyNewMethod = function(arg1, argN) { //content}. If you use a nested function within a Niivue prototype you will need to bind this this context to that nested function like so: myNestedFunc = function(arg1, argN) { // content}.bind(this).
Closing this PR since main now has all of these additions after merging both font and prototype


Improved MIDI Keyboard

closes #74

looking nice but would be real great if we got an octave on mobile

k adjusted a few things incl. considering dvh instead of vh to ensure the correct keyboard sizing

haha u are a brave soul :)


Update middleware.ts

There are many duplicate blocks in the code for different URL paths. You can consider putting these blocks in a separate function or dictionary for cleaner and more supported code.
If your project is growing and adding new URLs, you need to ensure their support in this middleware.

@cypherpepe excellent cleanup, using data structures instead of control flow 😘

@cypherpepe I don't think this code will work for
'/discord': 'discord.gg/buildonbase',
'/blog': 'base.mirror.xyz/',
Because they have different hosts, could U revise?

Review Error for zencephalon @ 2024-07-09 16:41:23 UTC
User failed mfa authentication, see go/mfa-help

Hm, I think the way the two dictionaries is defined is kind of the inverse of what I'd expect, we should only need to specify the host for the two that are different.


feat(tag): dark theme

Description
Enables tag dark theme color scheme. Previously: https://github.com/uber-web/baseui/pull/1132
Early on considered taking a light touch approach to applying the correct colors in dark mode, but found that the color modulations between light/dark themes to be inconsistent. Because of that, simply applying correct gradients would not work nicely. Also realized that although figma displays color values as overlays, they actually map one to one with standardized color values. Check out the figma doc I put together to map everything precisely:

Since all of the color values are just modulations from the 400 color ramp unit, I explored programmatically lightening and darkening for each tag state but found that it led to inconsistent results. Notice that most of the colors look fine, but the positive text color looks rather 'neon'. Because of these inconsistencies I chose to exactly define each color value. While this approach can have rather large line counts, I believe it is better to provide a perfect component.

I still could not avoid from color modulations entirely because this component supports custom colors. To perform the shifts, I pulled in the polished package. It doesn't work for all provided colors because it relies on a middle value between light/dark, but developers should be able to test out what works for them.

Scope

[x] Minor: New Feature

Nice, that's a lot of manual work! The disabled states might not have enough contrast but that might be acceptable for that use-case?

Nice, that's a lot of manual work! The disabled states might not have enough contrast but that might be acceptable for that use-case?

I thought the same thing, but assume that this was reviewed by design folks. Given the current implementation it should be simple to update

Minor thing:

for closable/non-clickable outlined tags would make sense to have the hovered action's background color to match the border color:

It's a lot of tag specific color props. Wonder if designers can simplify it slightly.
Great job!


Protect PoGoMap with password

Allows user to define a password to protect the website.
This does not conflict with the configuration password.
Also renamed is_authenticated to is_admin because only admins can change the config and add/delete points to/from the map.

Very nice feature, would like to see this merged!

@favll is this of your interested at all?

@DPr00f Can you add a js check on the config page and a check in app.py to ensure that the admin cannot specify the same password for config/admin and site authentication. This is to avoid confusions and mess-ups?


Request for pretrained models

Thank you very much for sharing your excellent work.
May I inquire if you could provide the pretrained models? In any case, thanks for your help.

We will release the pre-trained model once accepted. Sorry for that.


jks to pem

Codecov Report

Merging #76 (f7d34fc) into main (0f48d05) will decrease coverage by 2.72%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##               main      #76      +/-   ##
============================================
- Coverage     34.69%   31.97%   -2.73%
  Complexity      146      146
============================================
  Files            69       72       +3
  Lines           879      954      +75
  Branches         45       57      +12
============================================
  Hits            305      305
- Misses          558      633      +75
  Partials         16       16

Flag
Coverage Δ

unittests
31.97% <0.00%> (-2.73%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

.../java/com/github/shoothzj/dev/secret/Constant.java
0.00% <0.00%> (ø)

.../com/github/shoothzj/dev/secret/ConvertSecret.java
0.00% <0.00%> (ø)

...github/shoothzj/dev/secret/SecretResponseBody.java
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0f48d05...f7d34fc. Read the comment docs.


Use qualifiers from e.g. 'journal' when parsing Wikidata

Use qualifiers, for example on P1433, which can hold info on issue, volume and pages.

Relevant qualifiers: P577, P478, P433, P304. Relevant item: Q22116856.


fix volume parameter

Coverage remained the same at 100.0% when pulling 34e325975e6f6aa1a44c4a91acb3a22086cd32b3 on gregwebs:master into 02252d352256a1a2e8bd4ea606e1578d82406631 on softprops:master.


revert mars-protocol coingecko-id

the longer string works, the shorter one shows a price, but it is incorrect. If you would like to verify it is displayed - https://www.coingecko.com/en/coins/mars-protocol

Turns out mars-protocol is the old terra classic mars token


ITM threshold

HI, I am very fortunate to read your paper BLIP.  It's very exciting.
I wonder how to set the ITM threshold when filtering?
Thanks!
Looking forward to your reply.

We simply set it as 0.5, thanks.

In train_retrieval.py, the score of ITM did not go through softmax. Did you consider using softmax at that time?
When filtering, is the score of 0.5 used without softmax?
Thanks!

When filtering, the score of 0.5 uses softmax?
During inference of retrieval, softmax does not affect the ranking of ITM predictions, but it does affect the relative scale w.r.t the ITC similarity. If softmax is used for ITM scores, some additional scaling on the ITC score is also needed.


Compute and display distances using two node picks

I would like to be able to compute distances between two "picks". Preferably, when a new node pick is made, it is compared to the previous pick, and distance is displayed somewhere in the pick gui. Thanks!Mike
-----------------------REDMINE MIGRATION-----------------------
This ticket was migrated from Redmine. As such, not all
information was able to be captured in the transition. Below is
a complete record of the original redmine ticket.
Ticket number: 803
Status: Pending
Project: VisIt
Tracker: Feature
Priority: Normal
Subject: Compute and display distances using two node picks
Assigned to: -
Category: -
Target version: -
Author: Michael Barad
Start: 08/03/2011
Due date:
% Done: 0%
Estimated time:
Created: 08/03/2011 03:04 pm
Updated: 08/16/2011 02:01 pm
Likelihood:
Severity:
Found in version: 2.12.3
Impact: 3 - Medium
Expected Use: 3 - Occasional
OS: All
Support Group: Any
Description:
I would like to be able to compute distances between two "picks". Preferably, when a new node pick is made, it is compared to the previous pick, and distance is displayed somewhere in the pick gui. Thanks!Mike
Comments:

This was requested by another user.
If we had a check-box in pick output to compute distance to previous pick, that might be sufficient.


Support for XSD Date type in TransformationRules

Is your feature request related to a problem? Please describe.
When trying to add a property with XSD Date type (by entering "date" in the Type column of the excel sheet), you get the following error:
Errors: UndefinedObjectsAsExpectedValueTypes

Following classes date defined as classes in the 'Classes' sheet have no properties defined in the 'Properties' sheet or their validation as objects failed!

Describe the solution you'd like
Support for XSD Date with the possibility to enter "date" in the Type column of the excelsheet, and get the correct datatype for DM instances
Describe alternatives you've considered
Using the implemented Datetime type does not work:
Input should be a valid datetime, invalid datetime separator, expected T, t, _ or space [type=datetime_parsing, input_value='2014-01-01', input_type=str]
Additional context

Fix merged


You cannot open files and folders in second-level subfolders

I tried to open a local version of my repository (link). I can only read one level of subfolders.
Example: testServeFolder.zip
testServeFolder/
       you-can-read-me.txt
       first-subfolder/
              you-can-read-me.txt
              second-subfolder/
                     you-can-NOT-read-me.txt

In the servefolder.dev console there is this error:
index.js:239 Unable to serve file 'first-subfolder/first-subfolder/second-subfolder/you-can-NOT-read-me.txt':  DOMException: A requested file or directory could not be found at the time an operation was processed.

In the host there is this error:

Thank you very much 😄

Should work now - clear cache to make sure it updates if you try again.


fix(chat): Add indicator for disabled emoji extension

What this PR does 📖

Adds an indicator in form of a tooltip upon hovering the emoji button in quickprofile that tells the user the extension is disabled (if true)

Which issue(s) this PR fixes 🔨

Resolve #1083

after discussing with the team, we will merge 🔨


Error upon install

Helloooo!
I think an error was recently introduced in 935ea57
Currently cloning the main branch (head is 3e1667e), I get this error upon first starting nvim:
Error detected while processing /home/fargus/.config/nvim/init.lua:
E5113: Error while calling lua chunk: /home/fargus/.config/nvim/init.lua:7: attempt to index field 'uv' (a nil value)
stack traceback:
        /home/fargus/.config/nvim/init.lua:7: in main chunk
Press ENTER or type command to continue

If I run
cd ~/.config/nvim/
git checkout 0c7d9ce # (the commit before 935ea57)
nvim

It works just fine

update to neovim v0.10


[GHSA-qrg7-hfx7-95c5] Please correct the credit for the advisory

Following https://github.com/github/advisory-database/pull/1683, the global advisory https://github.com/advisories/GHSA-qrg7-hfx7-95c5 was wrongly credited to aruneko. The local (repository level) advisory https://github.com/rancher/wrangler/security/advisories/GHSA-qrg7-hfx7-95c5 was credited to a security researcher, which hasn't yet accepted the credit in its GH profile. I'm requesting such correction in order to be transparent with the person that actually reported the issue.

@macedogm thanks for writing in and apologies for the confusion here.
We automatically credit anyone who submits a community contribution to improve an advisory. However, this has caused consternation (such as this issue) due to the lack of differentiation between types of credits.
As such, we'll soon be releasing a new feature allowing for different types of credits on advisories. Researchers will be able to be credited as the discovering party, and contributors will be credited separately.
Does that resolve your concern?

Hi @KateCatlin. Thank you for the context and explanation. Yes, that will resolve my concern. Just to reiterate, I created this issue to be fair with the discovering party and not to be unfair with the contributor party. The improvement that you mentioned will be a nice one for this. Feel free to close this issue.

@macedogm perfect thank you so much! Glad we've resolved it for you. I'll go ahead and close this, keep an eye on our changelog for the upcoming credit differentiation release.

Has this feature already been introduced? I'm facing a similar problem, but for some advisories I see people credited as "Finder" and "Analyst". Though, for such advisories, there's no matching JSON entries. Is it only supported if the issue wasn't imported from another database?
I'd like to add credits to advisories from Rust database and NVD.