🐛 [FirebaseAuth] "The audience in ID Token does not match the expected audience" when I tried sign-in with Apple ID on Android Device

Bug report
I'm facing this issue after I entered any Apple ID account and press Continue using an Android device:

Additional context
If I use an iPhone, it works successfully, the problem is after I entered any Apple ID account in the AppleId web view using any Android device.
Here is my code:
static Future<UserCredential> signInWithApple() async {
    final appleCredential = await SignInWithApple.getAppleIDCredential(
      scopes: [
        AppleIDAuthorizationScopes.email,
        AppleIDAuthorizationScopes.fullName,
      ],
      webAuthenticationOptions: WebAuthenticationOptions(
          clientId: FIREBASE_BUNDLE_ID //I have this data according to package documentation
          redirectUri: Uri.parse(
              'https://MI_PROJECT_ID.cloudfunctions.net/handleAppleSignIn')),
    );

    final oauthCredential = OAuthProvider('apple.com').credential(
      idToken: appleCredential.identityToken,
      accessToken: appleCredential.authorizationCode,
    );

    final UserCredential appleUser =
        await FirebaseAuth.instance.signInWithCredential(oauthCredential);

    return appleUser;
  }

Im using sign_in_with_apple: ^4.3.0 , firebase_core: ^2.3.0 and firebase_auth: ^4.4.0
I have my project registered in firebase and developer.apple.com, the Apple provider as Sign-In provider also OAuth 2.0 Client IDs registration and all the setup of the https://pub.dev/packages/sign_in_with_apple package, the intent://callback?${PARAMETERS_FROM_CALLBACK_BODY}#Intent;package=YOUR.PACKAGE.IDENTIFIER;scheme=signinwithapple;end I also configured the Apple ServiceId with the return URL for using Firebase Cloud Functions
If I use the Apple Id sign in option on any Android device, is it considered a Web method and do I need to use FirebaseAuth.instance.signInWithPopup() as the page https://firebase.google.com/docs/auth/flutter/federated-auth?hl=en&authuser=0#android_1 says?

Flutter doctor
Run flutter doctor and paste the output below:
Click To Expand
[✓] Flutter (Channel stable, 3.7.11, on macOS 13.2.1 22D68 darwin-arm64,
    locale en-MX)
[✓] Android toolchain - develop for Android devices (Android SDK version
    30.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 14.3)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2021.3)
[✓] VS Code (version 1.71.0)
[✓] Connected device (4 available)
[✓] HTTP Host Availability

• No issues found!

Flutter dependencies
Run flutter pub deps -- --style=compact and paste the output below:
Click To Expand
art SDK 2.19.6
Flutter SDK 3.7.11
mentoro 0.6.0+1

dependencies:
- auto_size_text 3.0.0 [flutter]
- cached_network_image 3.2.3 [flutter flutter_cache_manager octo_image cached_network_image_platform_interface cached_network_image_web]
- crypto 3.0.2 [typed_data]
- cupertino_icons 1.0.5
- firebase_auth 4.4.1 [firebase_auth_platform_interface firebase_auth_web firebase_core firebase_core_platform_interface flutter meta]
- firebase_core 2.9.0 [firebase_core_platform_interface firebase_core_web flutter meta]
- firebase_messaging 14.4.0 [firebase_core firebase_core_platform_interface firebase_messaging_platform_interface firebase_messaging_web flutter meta]
- firebase_storage 11.1.0 [firebase_core firebase_core_platform_interface firebase_storage_platform_interface firebase_storage_web flutter]
- floating_bottom_navigation_bar 1.5.2 [flutter]
- flutter 0.0.0 [characters collection js material_color_utilities meta vector_math sky_engine]
- flutter_local_notifications 13.0.0 [clock flutter flutter_local_notifications_linux flutter_local_notifications_platform_interface timezone]
- flutter_native_splash 2.2.19 [args flutter flutter_web_plugins js html image meta path universal_io xml yaml]
- font_awesome_flutter 10.4.0 [flutter]
- google_fonts 4.0.3 [flutter http path_provider crypto]
- google_sign_in 6.1.0 [flutter google_sign_in_android google_sign_in_ios google_sign_in_platform_interface google_sign_in_web]
- hive 2.2.3 [meta crypto]
- hive_flutter 1.1.0 [flutter hive path_provider path]
- http 0.13.5 [async http_parser meta path]
- icons_launcher 2.1.0 [args image path universal_io yaml]
- intl 0.17.0 [clock path]
- path_provider 2.0.14 [flutter path_provider_android path_provider_foundation path_provider_linux path_provider_platform_interface path_provider_windows]
- pmvvm 4.0.1 [provider flutter_hooks tint flutter]
- pod_player 0.1.0 [flutter lottie video_player http get wakelock universal_html youtube_explode_dart]
- provider 6.0.5 [collection flutter nested]
- sign_in_with_apple 4.3.0 [flutter meta sign_in_with_apple_platform_interface sign_in_with_apple_web]
- smooth_page_indicator 1.0.1 [flutter]
- uuid 3.0.7 [crypto]
- video_player 2.6.1 [flutter html video_player_android video_player_avfoundation video_player_platform_interface video_player_web]

dev dependencies:
- build_runner 2.3.3 [args async analyzer build build_config build_daemon build_resolvers build_runner_core code_builder collection crypto dart_style frontend_server_client glob graphs http_multi_server io js logging meta mime package_config path pool pub_semver pubspec_parse shelf shelf_web_socket stack_trace stream_transform timing watcher web_socket_channel yaml]
- flutter_lints 2.0.1 [lints]
- flutter_test 0.0.0 [flutter test_api path fake_async clock stack_trace vector_math async boolean_selector characters collection js matcher material_color_utilities meta source_span stream_channel string_scanner term_glyph]
- hive_generator 2.0.0 [build source_gen hive analyzer source_helper]
- integration_test 0.0.0 [flutter flutter_driver flutter_test path vm_service archive async boolean_selector characters clock collection crypto fake_async file js matcher material_color_utilities meta source_span stack_trace stream_channel string_scanner sync_http term_glyph test_api typed_data vector_math webdriver]
- test 1.22.0 [analyzer async boolean_selector collection coverage http_multi_server io js node_preamble package_config path pool shelf shelf_packages_handler shelf_static shelf_web_socket source_span stack_trace stream_channel typed_data web_socket_channel webkit_inspection_protocol yaml test_api test_core]

transitive dependencies:
- _fe_analyzer_shared 58.0.0 [meta]
- _flutterfire_internals 1.1.0 [collection firebase_core firebase_core_platform_interface flutter meta]
- analyzer 5.10.0 [_fe_analyzer_shared collection convert crypto glob meta package_config path pub_semver source_span watcher yaml]
- archive 3.3.2 [crypto path]
- args 2.4.0
- async 2.10.0 [collection meta]
- boolean_selector 2.1.1 [source_span string_scanner]
- build 2.3.1 [analyzer async convert crypto glob logging meta path]
- build_config 1.1.1 [checked_yaml json_annotation path pubspec_parse yaml]
- build_daemon 3.1.1 [built_collection built_value http_multi_server logging path pool shelf shelf_web_socket stream_transform watcher web_socket_channel]
- build_resolvers 2.2.0 [analyzer async build collection crypto graphs logging path package_config pool pub_semver stream_transform yaml]
- build_runner_core 7.2.7 [async build build_config build_resolvers collection convert crypto glob graphs json_annotation logging meta path package_config pool timing watcher yaml]
- built_collection 5.1.1
- built_value 8.4.4 [built_collection collection fixnum meta]
- cached_network_image_platform_interface 2.0.0 [flutter flutter_cache_manager]
- cached_network_image_web 1.0.2 [flutter flutter_cache_manager cached_network_image_platform_interface]
- characters 1.2.1
- charcode 1.3.1
- checked_yaml 2.0.2 [json_annotation source_span yaml]
- clock 1.1.1
- code_builder 4.4.0 [built_collection built_value collection matcher meta]
- collection 1.17.0
- convert 3.1.1 [typed_data]
- coverage 1.6.3 [args logging package_config path source_maps stack_trace vm_service]
- csslib 0.17.2 [source_span]
- dart_style 2.3.0 [analyzer args path pub_semver source_span]
- dbus 0.7.8 [args ffi meta xml]
- fake_async 1.3.1 [clock collection]
- ffi 2.0.1
- file 6.1.4 [meta path]
- firebase_auth_platform_interface 6.13.0 [_flutterfire_internals collection firebase_core flutter meta plugin_platform_interface]
- firebase_auth_web 5.3.1 [firebase_auth_platform_interface firebase_core firebase_core_web flutter flutter_web_plugins http_parser intl js meta]
- firebase_core_platform_interface 4.6.0 [collection flutter flutter_test meta plugin_platform_interface]
- firebase_core_web 2.3.0 [firebase_core_platform_interface flutter flutter_web_plugins js meta]
- firebase_messaging_platform_interface 4.3.0 [_flutterfire_internals firebase_core flutter meta plugin_platform_interface]
- firebase_messaging_web 3.3.0 [_flutterfire_internals firebase_core firebase_core_web firebase_messaging_platform_interface flutter flutter_web_plugins js meta]
- firebase_storage_platform_interface 4.2.0 [collection firebase_core flutter meta plugin_platform_interface]
- firebase_storage_web 3.4.0 [_flutterfire_internals async firebase_core firebase_core_web firebase_storage_platform_interface flutter flutter_web_plugins http js meta]
- fixnum 1.1.0
- flutter_blurhash 0.7.0 [flutter]
- flutter_cache_manager 3.3.0 [clock collection file flutter http path path_provider pedantic rxdart sqflite uuid]
- flutter_driver 0.0.0 [file flutter flutter_test fuchsia_remote_debug_protocol path meta vm_service webdriver archive async boolean_selector characters clock collection crypto js matcher material_color_utilities platform process source_span stack_trace stream_channel string_scanner sync_http term_glyph test_api typed_data vector_math]
- flutter_hooks 0.18.6 [flutter]
- flutter_local_notifications_linux 3.0.0+1 [dbus ffi flutter flutter_local_notifications_platform_interface path xdg_directories]
- flutter_local_notifications_platform_interface 6.0.0 [flutter plugin_platform_interface]
- flutter_web_plugins 0.0.0 [flutter js characters collection material_color_utilities meta vector_math]
- freezed_annotation 2.2.0 [collection json_annotation meta]
- frontend_server_client 3.2.0 [async path]
- fuchsia_remote_debug_protocol 0.0.0 [process vm_service file meta path platform]
- get 4.6.5 [flutter]
- glob 2.1.1 [async collection file path string_scanner]
- google_identity_services_web 0.2.1 [js meta]
- google_sign_in_android 6.1.9 [flutter google_sign_in_platform_interface]
- google_sign_in_ios 5.6.1 [flutter google_sign_in_platform_interface]
- google_sign_in_platform_interface 2.4.0 [flutter plugin_platform_interface quiver]
- google_sign_in_web 0.12.0 [flutter flutter_web_plugins google_identity_services_web google_sign_in_platform_interface http js]
- graphs 2.2.0 [collection]
- html 0.15.2 [csslib source_span]
- http_multi_server 3.2.1 [async]
- http_parser 4.0.2 [collection source_span string_scanner typed_data]
- image 4.0.15 [archive meta xml]
- io 1.0.4 [meta path string_scanner]
- js 0.6.5 [meta]
- json_annotation 4.8.0 [meta]
- lints 2.0.1
- logging 1.1.1
- lottie 1.4.3 [archive flutter path vector_math]
- matcher 0.12.13 [meta stack_trace]
- material_color_utilities 0.2.0
- meta 1.8.0
- mime 1.0.4
- nested 1.0.0 [flutter]
- node_preamble 2.0.2
- octo_image 1.0.2 [flutter flutter_blurhash]
- package_config 2.1.0 [path]
- path 1.8.2
- path_provider_android 2.0.24 [flutter path_provider_platform_interface]
- path_provider_foundation 2.2.2 [flutter path_provider_platform_interface]
- path_provider_linux 2.1.10 [ffi flutter path path_provider_platform_interface xdg_directories]
- path_provider_platform_interface 2.0.6 [flutter platform plugin_platform_interface]
- path_provider_windows 2.1.5 [ffi flutter path path_provider_platform_interface win32]
- pedantic 1.11.1
- petitparser 5.1.0 [meta]
- platform 3.1.0
- plugin_platform_interface 2.1.4 [meta]
- pool 1.5.1 [async stack_trace]
- process 4.2.4 [file path platform]
- pub_semver 2.1.3 [collection meta]
- pubspec_parse 1.2.2 [checked_yaml collection json_annotation pub_semver yaml]
- quiver 3.2.1 [matcher]
- rxdart 0.27.7
- shelf 1.4.0 [async collection http_parser path stack_trace stream_channel]
- shelf_packages_handler 3.0.1 [path shelf shelf_static]
- shelf_static 1.1.1 [convert http_parser mime path shelf]
- shelf_web_socket 1.0.3 [shelf stream_channel web_socket_channel]
- sign_in_with_apple_platform_interface 1.0.0 [flutter plugin_platform_interface meta]
- sign_in_with_apple_web 1.0.1 [flutter flutter_web_plugins sign_in_with_apple_platform_interface js]
- sky_engine 0.0.99
- source_gen 1.2.7 [analyzer async build dart_style glob path source_span yaml]
- source_helper 1.3.3 [analyzer collection source_gen]
- source_map_stack_trace 2.1.1 [path source_maps stack_trace]
- source_maps 0.10.12 [source_span]
- source_span 1.9.1 [collection path term_glyph]
- sqflite 2.2.6 [flutter sqflite_common path]
- sqflite_common 2.4.3 [synchronized path meta]
- stack_trace 1.11.0 [path]
- stream_channel 2.1.1 [async]
- stream_transform 2.1.0
- string_scanner 1.2.0 [source_span]
- sync_http 0.3.1
- synchronized 3.0.1
- term_glyph 1.2.1
- test_api 0.4.16 [async boolean_selector collection meta source_span stack_trace stream_channel string_scanner term_glyph matcher]
- test_core 0.4.20 [analyzer async args boolean_selector collection coverage frontend_server_client glob io meta package_config path pool source_map_stack_trace source_maps source_span stack_trace stream_channel vm_service yaml matcher test_api]
- timezone 0.9.1 [path]
- timing 1.0.1 [json_annotation]
- tint 2.0.1
- typed_data 1.3.1 [collection]
- universal_html 2.2.1 [async csslib charcode collection html meta source_span typed_data universal_io]
- universal_io 2.2.0 [collection meta typed_data]
- vector_math 2.1.4
- video_player_android 2.4.4 [flutter video_player_platform_interface]
- video_player_avfoundation 2.4.4 [flutter video_player_platform_interface]
- video_player_platform_interface 6.1.0 [flutter plugin_platform_interface]
- video_player_web 2.0.16 [flutter flutter_web_plugins video_player_platform_interface]
- vm_service 9.4.0
- wakelock 0.6.2 [flutter meta wakelock_macos wakelock_platform_interface wakelock_web wakelock_windows]
- wakelock_macos 0.4.0 [flutter flutter_web_plugins wakelock_platform_interface]
- wakelock_platform_interface 0.3.0 [flutter meta]
- wakelock_web 0.4.0 [flutter flutter_web_plugins js wakelock_platform_interface]
- wakelock_windows 0.2.1 [flutter wakelock_platform_interface win32]
- watcher 1.0.2 [async path]
- web_socket_channel 2.4.0 [async crypto stream_channel]
- webdriver 3.0.1 [archive matcher path stack_trace sync_http]
- webkit_inspection_protocol 1.2.0 [logging]
- win32 3.1.4 [ffi]
- xdg_directories 0.2.0+3 [meta path process]
- xml 6.2.2 [collection meta petitparser]
- yaml 3.1.1 [collection source_span string_scanner]
- youtube_explode_dart 1.12.4 [collection freezed_annotation html http http_parser json_annotation meta xml]

Hi @GuerreroC, does the package id in your app match what is on firebase? See https://developer.apple.com/forums/thread/663461 for a similar issue that occurred in native ios

Hi @danagbemava-nc! Thanks for attend my issue, regarding to this link, is focussed on bundle ID, and yes I already have the same bundle Id registered on xcode and firebase.

Solved:
I got confused when putting the Apple OAuth cliend ID in the Google Cloud credentials instead of the correct Service ID of the Sign with Apple setup in dev apple page.
My apologies, I'm going to close this but I attached the image if someone has the same confusion.


simple-bar correctly alligned on one monitor, not both

I'll just continue filing bug reports until I find one that's actually a real issue ;)
I have connected an external display, and the simple-bar bar is not showing correctly on one of them.

If I click übersicht --> refresh all widgets on the built-in monitor, that monitor's bar is looking good...

...but there is a gap between the menu bar and the simple-bar bar on the external monitor:

If I do the same on the external monitor, it looks good there...

...but on the external monitor, the simple-bar bar is partially hidden behind the menu bar:

I didn't find any setting to change this.

Solution : https://github.com/Jean-Tinland/simple-bar/issues/360

Hey Emile,
I had seen that issue, but I don't think it's applicable here. There, it is described how to add a buffer for the bar to each space. But I don't see how to change the position of the bar to always be directly below the menu bar, nor how to make the distance dependent on the display (instead of the space).
Let me know if I'm mistaken!

I think the issue is caused by the fact that the menu bar on the built-in monitor is drawn taller than on the external monitor. If you compare the images in each set, the distance between the top of the screen and the top of the simple-bar is very similar.

I think I should be resolved as stated in #372 with a fix in Übersicht.
Feel free to open a new issue if you still have this issue.


Crash in dart vm when running test.py

Twice in the past day we have seen failures where the Dart vm running the test scripts (the test runner, test.py) has failed in native code, with the message
Dart_StringToCString expects argument 'object' to be of type String.
There is no stack trace in either Dart or C for this, only this line and an exit code of 255.
Neither the test scripts nor the checked-in version of the Dart SDK has changed recently, though I plan to update the Dart SDK very soon.  The present version is 1.25.0-dev.7.0, and we are now at dev version 2.0.0-dev.1.0.
If this continues to fail flakily like this, it will be very hard to debug, because we aren't getting a trace.
In that case, our best hope is to update the checked-in SDK.  We will wait and see, but if you have any ideas about how to get a stack trace, let me know.
The failure log is at
https://ci.chromium.org/swarming/task/38d68b7090fc4710?server=chromium-swarm.appspot.com

Can this be closed in favor of https://github.com/dart-lang/sdk/issues/30990 ?

Duplicate.


Remove catchAll listener from flow to prevent multiple event listener registration.

Similar to issue encountered in #135 - multiple catchall event handlers were being added to the existing flow object.
Don't expect this would be a breaking change unless someone is adding catchall event handlers directly to a pre-initialized flow object.
Would probably prefer to see this initialization done by default in factory or config block, but broadcasts containing the flow object become problematic without larger changes.

Great, thanks, but I am not sure if test is suitable for this specific case. You should test that apply is called only once and not if catchall event is registered.
How about other approach, once flow-init directive is created, we can assign new property to flow object
if (!flow.ngFlowInit) {
flow.ngFlowInit = true;
flow.on('catchAll', function (eventName) {
....
});
}

What do you think?

Both seem reasonable.
The extra flag is probably not great, but solves the issue I mentioned above.
Will update the request with both changes later today.

Updated.
I think this apply is the one you wanted me to test.
Let me know if you would like me to modify or squash the commits.
Cheers.

Looks great, but test still can be improved, because it is tightly coupled to the implementation of the directive.  What if broadcast is called with call method instead of apply? It should probably be changed to:
spyOn(elementScope, '$broadcast').andCallThrough();

I have one concern, what if we use flow-init with existing flow objects in distinct, non related scopes. With current pull request only one flow-init directive will fire a broadcast. Because of this only one scope tree will be updated with changes.
The only issue I see here is that $apply is called on every flow-init instance, it creates a bit of performance loss.
What we are trying to solve with this pull request?

Am fine with the test change.
Yes, it would cause the issue you mention. I think in that, case adding a destroy listener to the scope and removing the catchall event listener would solve my issue.
Which is that I have a persistent flow instance.
When all uploads are complete, I refresh the contents of the uploads list (remote GET) so the user can see their file, and the view is in sync.
However, because of the above behavior of init, every time the user navigates to the file upload area, another event handler is added to the causing 2 or 5 or 10 requests after upload.
This is what the proposed solution would look like:
var catchAllHandler = function(eventName){
  var args = Array.prototype.slice.call(arguments);
  ...
};
flow.on('catchAll', catchAllHandler);

$scope.on('$destroy', function(){
    flow.off('catchAll', catchAllHandler);
});

Awesome, you found a real issue, thanks! Can you close this pull request and open a new one with a fix?


NobodyNada: Watch rposervicesin.blogspot.com

NobodyNada requests the watch of the watch_keyword rposervicesin.blogspot.com. See the Metasmoke search here

!!/approve


Welcome Card not downloaded automatically in Mobile cards client

Tutorial URL: https://developers.sap.com/tutorials/cp-mobile-cards-welcome.html
Please specify the step you are referring to: Step 4: View the Welcome Card in Mobile Cards client
Issue:
Here it is given that "Welcome cards are auto subscribed by default and downloaded automatically after registration."
But sometimes the app will not be auto downloaded and we have to reload the mobile card client for the Welcome card to be visible.
So from a end user perspective, please add a line in the tutorial that if the card is not visible automatically then we have to reload the Mobile card client.
Regards,
Priyanka

Hi @priyankars2210,
Thank you so much for raising this concern.
Can you please elaborate on the following statement you made in your comment:

But sometimes the app will not be auto downloaded and we have to reload the mobile card client for the Welcome card to be visible

Did you do a pull-refresh or did you unsubscribe and subscribe?
Thank you for providing additional information.
Warm Regards
Sandeep T D S

Hi Sandeep,
I had to do a pull refresh for the card to be visible.
But in the document it is said that the card would be auto downloaded. So from an end user perspective that means that we should be able to see the card without doing a pull request.
Hope this helps...
Regards,
Priyanka

Hi Priyanka,
Thank you so much for providing the details; your feedback is very helpful.
We will address the same in our documentation.
Warm Regards
Sandeep T D S

@sandeep-tds Is this something that has to be updated in the tutorial?

If yes, please update.
if no, I can close the issue.

Hi @thecodester,
I have made the changes and will be submitting the Pull Request along with other changes tomorrow.

Thank you for your feedback. The issue seems to be resolved, so I am closing the issue.
If you still have questions, feel free to reopen the issue.


Fixing the link colors

Fixing the link colors by replacing rgba with rgb whenever no alpha channel value is set.

Looks good


"Load More Labs" button does not reset color

When loading lots of labs and using the button multiple times, the visuals for the "Load More Labs" button stays on/selected-looking every time after the first, even after the extra labs have been loaded. I'd expect it to revert back to the original color. Tested on 1.4.1 on iPhone 8.

Duplicate of #29


The system should be able to map intent to conceptual legal entities that user will be interacting with.

From @czdiana2321 on June 15, 2018 1:51
Copied from original issue: Microsoft/Access2Justice#149

@KellyeDC  - Not needed for pilot.  This is an enhancement.

Not sure of intent for this user story.


Use env variable to disable clearConsole (#2495)

This PR allows disabling clearConsole by providing the env variable REACT_APP_NO_CLEAR_CONSOLE (suggested here).
How this works?
create-react-app my-app
cd my-app
REACT_APP_NO_CLEAR_CONSOLE=1 yarn start
# or: REACT_APP_NO_CLEAR_CONSOLE=1 npm start

Hello! I'm a bot that helps facilitate testing pull requests.
Your pull request (commit bf995a845a3bf2ae5b8b50d16a4004c9e96f3087) has been released on npm for testing purposes.
npm i<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS># or
create-react-app<EMAIL_ADDRESS>folder/

Note that the package has not been reviewed or vetted by the maintainers. Only install it at your own risk!
Thanks for your contribution!

@gaearon sounds legit.
Usually it is meant for a more verbose output so I would suggest to rename it to DEBUG or VERBOSE

Environment variables prefixed with REACT_APP_ can be embedded during build time, so the variable name probably shouldn't use that prefix. Also note that none of the other configuration variables use the prefix.

Any news on this?
Please can we add REACT_APP_NO_CLEAR_CONSOLE env var. Using Windows, OSX, Linux is a nightmare!

You can do the following trick: https://github.com/facebook/create-react-app/issues/2495#issuecomment-344537005
Not sure about Windows though


cloudflare_branding field not available in Go struct

Confirmation

[X] My issue isn't already found on the issue tracker.
[X] I have replicated my issue using the latest version of the library and it is still present.

cloudflare-go version
v0.110.0
Go environment
GO111MODULE=''
GOARCH='arm64'
GOBIN=''
GOCACHE='/Users/limbo/Library/Caches/go-build'
GOENV='/Users/limbo/Library/Application Support/go/env'
GOEXE=''
GOEXPERIMENT=''
GOFLAGS=''
GOHOSTARCH='arm64'
GOHOSTOS='darwin'
GOINSECURE=''
GOMODCACHE='/Users/limbo/.go/pkg/mod'
GONOPROXY='github.com/AfterShip/*'
GONOSUMDB='github.com/AfterShip/*'
GOOS='darwin'
GOPATH='/Users/limbo/.go'
GOPRIVATE='github.com/AfterShip/*'
GOPROXY='https://proxy.golang.org,direct'
GOROOT='/nix/store/44g95npjwgjyrxm1sx54g22w5sfmkz1i-go-1.21.6/share/go'
GOSUMDB='sum.golang.org'
GOTMPDIR=''
GOTOOLCHAIN='auto'
GOTOOLDIR='/nix/store/44g95npjwgjyrxm1sx54g22w5sfmkz1i-go-1.21.6/share/go/pkg/tool/darwin_arm64'
GOVCS=''
GOVERSION='go1.21.6'
GCCGO='gccgo'
AR='ar'
CC='clang'
CXX='clang++'
CGO_ENABLED='1'
GOMOD='/Users/limbo/Source/AfterShip/pltf-domains.as-in.com/go.mod'
GOWORK=''
CGO_CFLAGS='-O2 -g'
CGO_CPPFLAGS=''
CGO_CXXFLAGS='-O2 -g'
CGO_FFLAGS='-O2 -g'
CGO_LDFLAGS='-O2 -g'
PKG_CONFIG='pkg-config'
GOGCCFLAGS='-fPIC -arch arm64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -ffile-prefix-map=/var/folders/tl/vdx9psv541bc4271016rwshc0000gn/T/go-build559841223=/tmp/go-build -gno-record-gcc-switches -fno-common'

Expected output
Create custom hostname successfully.
Actual output
{
  "result": {
    "ssl": null
  },
  "success": false,
  "errors": [
    {
      "code": 1469,
      "message": "Since no host is 64 characters or fewer, Cloudflare Branding is required. Please check your input and try again."
    }
  ],
  "messages": []
}

Code demonstrating the issue
// there is no way to set `"cloudflare_branding": true` here
	cloudflare.CustomHostname{
		Hostname: hostname,
		SSL: &cloudflare.CustomHostnameSSL{
			Method: "http",
			Type:   "dv",
			Settings: cloudflare.CustomHostnameSSLSettings{
				HTTP2:         "on",
				MinTLSVersion: "1.2",
				TLS13:         "on",
			},
		},
	}

Steps to reproduce
Just write the code and you will see.
References
No response

best submit this as a feature request to cloudflare support or your account team to have it triaged by the service team.

If I read it right, this is official Go library for the Cloudflare API.
The official API documentation mentions this cloudflare_branding: boolean field.
The same field is missing from the Go library struct.

I would consider this is a bug.


Can't make HTTP requests

I'm trying to have hubot get stock quotes from a REST API.  I'm using the following script to accomplish this:
module.exports = (robot) ->
    robot.hear /!ftnt/i, (res) ->
        res.robot.http('http://dev.markitondemand.com/Api/v2/Quote/json?symbol=ftnt')
        .get() (err, res, body) ->
            res.send "Got back #{body}"

However it always fails with the following stack:

ERROR TypeError: undefined is not a function
at /home/ian/FortiFriend/scripts/ftnt.coffee:5:8
at IncomingMessage. (/home/ian/FortiFriend/node_modules/scoped-http-client/src/index.js:95:22)
at IncomingMessage.emit (events.js:129:20)
at _stream_readable.js:908:16
at process._tickCallback (node.js:355:11)

For reference, here is /home/ian/FortiFriend/node_modules/scoped-http-client/src/index.js:95:22:
 return res.on('end', function() {
     return callback(null, res, body);
 });

I'm using nodejs v0.12.7 on Ubuntu 15.04

I think the problem is that the res in the robot.hear callback is overshadowing the res in the get() callback. Try:
module.exports = (robot) ->
  robot.hear /!ftnt/i, (res) ->
    res.http('http://dev.markitondemand.com/Api/v2/Quote/json?symbol=ftnt')
      .get() (err, httpRes, body) ->
        res.send "Got back #{body}

See https://github.com/github/hubot/blob/master/docs/scripting.md#making-http-calls for some more information. In particular, you will want to check for an err and that  httpRes.statusCode is something you expect (ie 200 to indicate success) before touching body.


Changes required PHP version to 5.4 for production as 5.5 syntax only applies to testing

Fixes #9.

:+1:

Current coverage is 92.85%

Merging #10 into develop will decrease coverage by -0.16% as of 6a67a47

@@            develop     #10   diff @@
=======================================
  Files             3       3
  Stmts           229     210    -19
  Branches          0       0
  Methods          48      48
=======================================
- Hit             213     195    -18
  Partial           0       0
+ Missed           16      15     -1

Review entire Coverage Diff as of 6a67a47

Powered by Codecov. Updated on successful CI builds.


fix(datepicker): fix today button disabled condition

Fix conditional for today button in popup

Plunker of fix in action here.
This fixes #5622.

@wesleycho, looks like the build failed.

Updated Plunker and fixed the problem

LGTM.


Updates for go weekly.2011-12-22 11100

Hey, after trying several oauth clients, I finally found one that is well written and actually works.  :)
I'm using this in yammer.go and it's working pretty well.

Good to hear! Thanks for the update! :-)


CLI should use filename as action name when one is not provided

When creating an action, we should assume the action name will match the filename if an action name is not provided. (1) For instance, if a filename of someAction.js is specified without an action name, we would create the action named someAction. (2) In the case were only one argument is passed without a file extension and a kind is specified, we can assume that the argument is a filename. (3) The same holds true if a file extension is passed that does not map to a runtime, but a kind is specified.
The action create usage would be changed to the following:
Usage:
  wsk action create [ACTION_NAME] ACTION [flags]

$ wsk action create someAction.js
ok: created action someAction

$ wsk action create someAction --kind nodejs:6
ok: created action someAction

$ wsk action create someAction.someExt --kind nodejs:6
ok: created action someAction

@mdeuser, @drcariel, @jessealva, @csantanapr, thoughts on the above?

What happens in the case of wsk action create myaction, where myaction is a file?  i guess this should fail since the runtime is not known.
Same changes for wsk action update, which can also be used to create an action ?

In that case, we would have to fail if a kind is not specified and an extension that does not map to a valid runtime is not provided.
Same changes would apply for the update command.

in scenario 2, the CLI would need to complete the file extension based on the kind flag in order to find the correct local file, right?
Im a bit confused by the 3rd scenario, would the extension need to match the kind? what happens if they do not match?

Scenario 2 and 3 are essentially the same. If a kind is not specified, we try to figure out the kind by checking the file extension. If a kind is not specified we need to error. When a kind is specified we will always use that and forego checking the file extension. We currently do this today, and it would be applied to the scenarios above.

IMO, the CLI should be careful in not inferring too much from too little in CRUD operations... and if assumptions or inferences are made, the user needs to be informed/warned of this additional behaviors.
Entity naming and Namespacing is becoming more and more important with Compositions so making these assumptions could cause unintended consequences for user/customers composing solutions with hundreds of actions.
https://swaggerhub.com/blog/api-design/api-design-best-practices/

I agree with @mrutkows , I think it's important to inform the user that these assumptions are being made in real time.

Less magic FTW and clarity. What's the point of saving the user from .js or .py it's not that many more characters and potentially confusing. Maybe I missed the rationale here.

No API changes here. This is a usability issue to allow the user to type less information. The CLI will continue to work as it does today when creating/updating actions.

To be more clear, here are some more examples.
Say the user wants to create an action that has the same name as his or her filename. Instead of the user having to type duplicate information, we only need the filename to be provided. Ex:
wsk action create myActionName.js
ok: created action myActionName

The CLI would still work as it does today allowing a filename and an action name if the user wants the action name to differ from the filename:
wsk action create myAction myActionName.js
ok: created action myActionName

I get that it's client only - I'm just not sure I like the automatic inference here. Note that the CLI change is actually going to change behavior (1 is an error today, and will work after these changes).

I can see the value in the 1st use case. Maybe the scope should be limited to this?
2  is where I see more user complications
( For example: different files with the same name but different extensions  etc..)
3 still doesn't make sense to me, but I am not sure how it work at in it's current state TBH.

I can see the value in the 1st use case. Maybe the scope should be limited to this?

yeah maybe... but also feel that if you're doing anything serious you're going to use a deployment framework like wskdeploy or you're writing your own scripts. I could be convinced to accept 1.
wsk action create foo.zip --kind nodejs:6

do you want that to work? I think that's maybe a use of 3.

Scenario two and three are just to keep in sync with the same type functionality that currently works with --kind. Currently the filename extension is disregarded if a kind is specified.
I think zip files would fall under the third scenario.
@drcariel, good point above same filenames with different extensions.
For me at least, I find that I create a lot of different actions to test the backend manually. Having to type an action name all the time just seems redundant when I name the file appropriately.

@dubeejw @rabbah as long as the CLI (API) provide the "clean contract" to the users... I agree with Rodric that the CLI should provide plain/straightforward functionality with known (contracted) results and higher-order tooling should provide the interaction/fuzzy logic.  If the CLI is performing more inference of intent then that provides less assurance to tooling built on top.  I would rather have success/fail from the CLI with clear return codes and not have any other extended logic where possible.

Just caught up.  I think I agree with @rabbah and @mrutkows .  Although the changes seem small and should not cause problems, the fact that the cli is really mostly an entry point for higher order usages it makes sense that its more reliable in its usage than providing shortcuts.
In the end, i rarely ever type things over and over, even when outside of higher level tooling I'll use the up arrow or command history to rerun the command, so don't ever need to type the command out once.  Not entirely sure why we really need this.

Good discussion. Actually found a bug in the existing code when looking into action create (https://github.com/apache/incubator-openwhisk/pull/2887).

I have started implementing the suggestions here for my own personal usage as it makes things easier for me.

Looks like we currently allow actions to be created with only the action name when the creating docker actions.
wsk -i action create dockerContainer --docker bogusihxfy6q5ssatqazy

Issue moved to apache/incubator-openwhisk-cli #124 via ZenHub


[48446] - iOS,  Questionnaire page

@fsnNC Why quiestionnaire view shows date twice? there is such thing in sketches

@nc-kano: Is by default the way it's done in iOS. I just changed the colors and borders of it and kept the functionality as in the DK app

@fsnNC Why quiestionnaire view shows date twice? there is such thing in sketches

This is because of the new way to show calender in iOS since iOS 14. Since we are building now in Xcode 12, this comes by default.

@fsnNC Why quiestionnaire view shows date twice? there is such thing in sketches

This is because of the new way to show datepicker in iOS since iOS 14. Since we are building now in Xcode 12, this comes by default.
We can however enforce wheels-style datepicker to choose the date.

@nc-kano Let's approve this design since it's the default way. Then I'll create a bug stating that we should look into a more pretty/user-friendly way of displaying the calendar on iOS


Linting, apostrophe, and comment fixes in Typography code samples

Fixes #257.

I now see that ' is not a valid HTML 4 entity!


Upstream UUID String() changes

Made a dedicated branch and reverted out other changes I had made in my master fork

Probably easier for you to just make the changes yourself instead of adding all this history but I figured I'd reconcile the pull request I made before anyway


Remove react-scroll-to-bottom dependence

Describe the bug
The component has a lot of issues when logs-react is used as a component in other projects.

:tada: This issue has been resolved in version 1.1.2 :tada:
The release is available on:

GitHub release
1.1.2
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


 In upper limit edge cases, this results in hitting max limit of 65536 characters. This causes dependabot to raise PR request with error 'Enabling auto-merge failed.
Commit message is too long'. A buffer of 1k (1024) characters is a safe limit that provides enough space to
provide supplement information without hitting the max ceiling limit. The message is trimmed at length 64512 (65536 - 1024) which leaves 1024 chars as a buffer to add additional information in Git PR commit message.

@sachin-sandhu can you add a more descriptive title

According to the error message I got earlier, the maximum length was 16,383 not 65,535. Plese see screenshot below.
Did this recently change?

Closing pull request (raised PR https://github.com/github/dependabot-api/pull/5276)


fixing issue #681

Fixes https://github.com/marmelab/ng-admin/issues/681

Please don't include build files, we'll do that after merging.

This PR is superseded by #1322, so I close this one.
Thanks for contributing! I reused your solution.


Config option: Set document-class

Accessible through the pandoc metadata option

Hey, just wondering what the progress is on this? :)

Bumping the priority of this option! Available through the metadata option documentclass, documented under Variables for LaTeX -> Layout


Will you pack the key feature as npm package?

Hi,
The repo is awesome. Thanks for your sharing.
Have you considered pack the key feature as npm package?
e.g.  https://www.npmjs.com/package/react-canvas-draw
Compared to the above package, your features are more better than that.

I'll think about it  :-)

@pengfeiw
cool !

@pengfeiw
cool !

Awesome! I'm waiting for the npm package, too.

所以有没有开发包


Docs do not mention one cannot handle PoisonPill as a regular message

When using pattern like The Reaper, it's very tempting to use PoisonPill as a stop notification (i.e. do a cleanup). Unfortunately, this does not seem to work as actors do not receive PoisonPill per se (seems to be handled by the framework). One may want to do some pre-stop cleanup on PoisonPill and the only option seems to be to enqueue some custom Stop message and do a cleanup, since there's no PreStop and one cannot "handle" PoisonPill.
Maybe it's just me, but it feels counter-intuitive that you cannot handle PoisonPill, I'd say it's worth explaining that in docs. Thoughts?

I think its explained rather well here: http://getakka.net/docs/working-with-actors/stopping-actors ?

@Danthar I'd say you're right, It's explained pretty well, but: most of the docs would say "PoisonPill is just a regular message" and this implies you can handle it If you wish, which is not true. So not that regular per se. There's only one place which suggests PoisonPill is auto handled and it would remove the ambiguity if the docs just said  PoisonPill cannot be handled by user code and clean up must be implemented by sending a custom message BEFORE sending PoisonPill. It's rather common to do some cleanup before graceful shutdown, and suggesting a way to do it in docs would be nice.


added url and IP whitelist in nginx

We often receive requests to whitelist Adyen so they can test via URI whitelist.

Run mdformat --check docs/
Error: File
"/home/runner/work/hypernode-docs-next/hypernode-docs-next/docs/hypernode-platform/nginx/basic-authentication-on-hypernode-development-plans.md"
is not formatted.

well I did:

hypernode-docs-next]$ mdformat docs/hypernode-platform/nginx/basic-authentication-on-hypernode-development-plans.md

but I guess the out of the box settings don't autoformat it correctly


Onboarding user client trigger activation

tasks

Trigger state activation when onboarding user using web app

Issue requires review and rearrangement. Close it.


Fix leaks from extHostTreeView and test

Part of #190503

An unrelated test is failing, and that test has had fixes made in the last couple hours. Let's see if pulling those changes in solves the failure.

@alexr00 still failing


Anvil parser

Subclasses DBGap parser to just change the type of element
Some test

Only change I would suggest: make the data type "AnVIL"
@alexwaldrop
updated  to AnVIL


Properly close files in mamba / boa

After building many recipes in one boa session, lsof shows many open file handles for the same json and solv cache files. We might have to properly close those in replace_channels.

I think this has been solved.


Add MDLL to DragonPHY

Preliminary MDLL code was added in PR #9.  While the MDLL has not yet been integrated into dragonphy_top, we already have an issue open for that (issue #14), so I'm closing this one.


Uncertain behavor: non-timestamped messages publish "nan" message_age

Description
message_age for non-timestamped data publishes statistics with .nan as the entry. Not sure that this is correct? Not sure what to expect.
Expected Behavior
For the message_age data not to be published.
Actual Behavior
measurement_source_name: arst
metrics_source: message_age
unit: ms
window_start:
  sec:<PHONE_NUMBER>
  nanosec: 833869000
window_stop:
  sec:<PHONE_NUMBER>
  nanosec: 835134000
statistics:
- data_type: 1
  data: .nan
- data_type: 3
  data: .nan
- data_type: 2
  data: .nan
- data_type: 5
  data: 0.0
- data_type: 4
  data: .nan

To Reproduce
Use std_msgs/String
System (please complete the following information)

OS: OSX
ROS 2 Distro: https://ci.ros2.org/view/packaging/job/packaging_osx/1865/artifact/ws/ros2-package-osx-x86_64.tar.bz2

This is expected behavior. Instead of an absence of a signal (no messages published) if a message header timestamp is not present NaNs are published.

Ok - should we leave it open until there is documentation to point to? Or is this behavior already mentioned in a docstring somewhere?

Let's leave it open until resolved by https://github.com/ros-tooling/aws-roadmap/issues/245

Opened https://github.com/ros2/ros2_documentation/pull/823 from the ROS Tooling Working Group followup to specifically note NaNs are published in the "empty" message.

Ambiguity noted in the WG meeting now resolved.


Can't Open Wordlist

I am running this on Windows, and am having an issue.
I ran
./feroxbuster -u https://mergify.com/ -w common.txt
and
./feroxbuster -u https://mergify.com/ -w Desktop/common.txt
No matter what I do, common.txt won't open and run. It's a simple wordlist text file. What's going on here? How do I fix this?

howdy! it looks like you're using cygwin or wsl. i'm not a windows user, so can't advise too much further on this setup. can you cat common.txt? it looks similar to a container issue where the insdie of the container can't see the outside filesystem.
are you able to use one of the windows builds?

doing some housekeeping, feel free to reopen


[SPARK-39057][SQL] Offset could work without Limit

What changes were proposed in this pull request?
Currently, Offset must work with Limit. The behavior not allow to use offset alone and add offset API into DataFrame.
If we use Offset alone, there are two situations:

If Offset is the last operator, collect the result to the driver and then drop/skip the first n (offset value) rows.
If Offset is the intermediate operator, shuffle all the result to one task and drop/skip the first n (offset value) rows and the the result will be passed to the downstream operator.

For example, SELECT * FROM a offset 10;  parsed to the logic plan as below:
Offset (offset = 10) // Only offset clause
|--Relation

and then the physical plan as below:
GlobalOffsetExec (offset = 10) // Skip the first 10 rows globally
|--JDBCRelation

or
CollectOffsetExec (offset = 10) // Collect the result to the driver and skip the first 10 rows
|--JDBCRelation

After this PR merged, users could input the SQL show below:
SELECT '' AS ten, unique1, unique2, stringu1
 		FROM onek
 		ORDER BY unique1 OFFSET 990;

Note: if offset value is bigger, the overhead looks not good.
Why are the changes needed?
Let Offset could work without Limit.
Does this PR introduce any user-facing change?
'No'.
New feature.
How was this patch tested?
New tests.

cc @dtenedor FYI

how about we only support Offset as the last operator? I think that's the most useful case and can unblock the dataframe API.

how about we only support Offset as the last operator? I think that's the most useful case and can unblock the dataframe API.

OK. It seems we just let users use Offset alone in test or debug, so we avoid the overhead on product environment.

@beliefer thanks for adding this! Please ping us when Wenchen's suggestion is implemented and we can take another look. Otherwise this PR LGTM.

Because https://github.com/apache/spark/pull/36417 merged


Changelog?

Is there a changelog or similar for what happened between release? I'm moving from 1.41 to 1.5.x but would like to know what changed?
Thanks!

You can always find the latest change log over here: https://backbonejs.org/#changelog.


Grid Component

CSS Grids makes it very easy to make grids. It is much simpler than using bootstrap like columns and is responsive without having to use media queries.
Proposed API
grid = Grid(columns=3, gap="10px")
for i in range(10):
    grid << i

All the css properties of the grid can be set using the .css method. This is advanced usage and once we figure out the common patterns, we'll expose them as regular api.
grid = Grid()
for i in range(10):
    grid << "Lorem ipsum dolor sit amet, consectetur adipiscing elit"

grid.css("template-template-columns", "repeat(auto-fit, minmax(300px, 1fr))")

References

https://developer.mozilla.org/en-US/docs/Learn/CSS/CSS_layout/Grids
https://travishorn.com/responsive-grid-in-2-minutes-with-css-grid-layout-4842a41420fe

I think css hack that I showed in the last example, could be abstracted as follows:
grid = Grid(gap="10px").autofit(min_width="300px")
for i in range(10):
    grid << i


Type parameter constraint clauses (where keywords) should be indented below each other

The where keywords of the type parameter constraint clauses should be indented directly below each other to read the clauses.
Instead of:
class SomeType<T1, T2> where T1 : class where T2 : class
{ }

It should be
class SomeType<T1, T2> where T1 : class
                       where T2 : class
{ }

Same should be reported for code like
class SomeType<T1, T2> where T1 : class
                   where T2 : class
{ }

or like
class SomeType<T1, T2> where T1 : class

                       where T2 : class
{ }


The fix function stack overflows

I was writing another benchmark, this time for a recursive data structure. While argonaut-codec (class-based) and codec-argonaut (value-based) worked fine, mine stack overflows.

I've found a way to get fix to work with the unidirectional codecs by using FFI to write this:
export const encodeFix = (f) => {
  const self = (a, b) => {
    return f(self)(a, b);
  }
  return self;
}

However, bidirectional codecs are a problem. After converting Codec into a newtyped record, I tried the following:
export const fix = (f) => {
  const encodeFix = (a, b) => {
    return f(
      { decoder: () => { throw new Error("fix - Cannot decode a fixpoint codec when your intent is encoding")}
      , encoder: encodeFix
      }
    ).encoder(a, b);
  }
  const decodeFix = (a, b, c, d, e) => {
    return f({
      decoder: decodeFix,
      encoder: () => { throw new Error("fix - Cannot encode a fixpoint codec when your intent is decoding")}
    }).decoder(a, b, c, d, e);
  }
  return {
    decoder: decodeFix,
    encoder: encodeFix,
  };
}

Unfortunately, I get a Stack overflow on a benchmark test of decoding 400 layers of a nested Either.

Seems like codec isn't stack-safe either as it fails with an "Internal Error: too much recursion": https://try.purescript.org/?code=LYewJgrgNgpgBAWQIYEsB2cDuALGAnGAKEJWAAcQ8AXOABQKgjCNIurgBEkqkA6AYXAwAxrwCCeAOYg0SCDSQBnOPzElylGgFEAZjpE0AFLv3CqASnVttegwKhLFAmYpCw4hqCEmXWmuAAqeACetBAEAMrCeChkCspBoeEwUTFxVv5cPOJSMnJUzgQeilQxaJIoOsGWGexZfILMwnBKKvy1NPXOTTnSsvLOwKAYrartfnXcfFooVLh4HjNz%2BIa8vOa%2BGpPZAHIwmFTBZPCGwg6KynsHRzAANHAQaJh4SGSb1pxTvPQgOo9mlA8YFIrxqwFQGAAXJC4CYDHAAKpoWaEcHoOAAXkCITCkWisQKBDQzAWGIAPGTsUk8WkCphZthBGhXO4wCBCHA4ABaLlwRQQYTCGAwMDKQw6QHAeDcOCwJQWDlwGBoYRCHYAGSQwXwygAHAAGfWKnlwHSoKBiiULKUtGhykqWTnK1XMDVanVwADMAH1DUbCM61ZrtXhlAAPTFwNmK2B292hyPSDw7LRwAAkcAASihJNgaKUIDBzHAwzHvOm%2BdgQJhZfHFGXJBWSmUKlU4IrORnDPx%2BLxA8wK6peDoUBHDAAdAjCcKKeBcgB8UZBZAeTxeK5Tg7EvZgDPwcCnM-gQ4ARiA3DAkGgNu3OZ3ayH65ycPgiJyk0hBXB9ZHP8JFR%2BX4YFiSaGJuGbqjAOgKIKxaGBgvIAIw1Gg%2ByHMccCblim7GHuCybgAQuecrXsQJIoAAbvA6AlFeQqwgAjphWiEORVFwDRPAqvAVzoTxqbeiQzJcfRERVjWm4vgQiqKOJiYgN%2BHZYPMb53kmGBgamRaRsISjwPAvxKZykHQXAqFhjQC5wKBGAANRwMhZkwBZRlZjmeYtNyi4AEQmVQqxrN5cBkousnVmZwU%2BWsvCGNmuY0EFIWVuFSCRXA3kbN5hBAA

A fix function is no longer provided here. However, given that the current design now works similar to arognaut-codec, I don't think we will get stack overflows anymore.


Android & iOS - Search - Search icon is not responsive after returning to LHN from thread

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.31-12
Reproducible in staging?: Y
Reproducible in production?: N/A
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause Internal Team
Action Performed:

Launch New Expensify app.
Go to any report.
Send a message.
Send a reply to the message in thread.
Go to Search.
Go to Chats.
Tap on the parent message sent in Step 3.
Tap on the reply count to go to thread.
Return to LHN by swiping to right twice (iOS navigation) (if on Android, use device back navigation and not app button).
Tap Search.

Expected Result:
Search icon will be responsive after returning to LHN from thread when opened from Search.
Actual Result:
Search icon is not responsive after returning to LHN from thread when opened from Search.
Workaround:
Unknown
Platforms:

[x] Android: Native
[ ] Android: mWeb Chrome
[x] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/2dc807ef-d0cf-4df7-adca-0fddcd496195

View all open jobs on GitHub

I managed to reproduce it. As opposed to any main chats that return to the search view when swiping right, a thread will go back to the LHN, maybe that's the issue here.

Most likely coming from https://github.com/Expensify/App/pull/47690

Sure, feel free to do so since you have more context!

@sakluger just missing PR review payment to C+

@luacmartins thanks for the bump.
@ishpaul777 please let me know once you've accepted the offer: https://www.upwork.com/nx/wm/offer/104347535

The contract was accepted and I paid it out. We're all done here.


Possible to run without Horovod ?

Hello,
I wanted to know if there was a solution to by-pass Horovod which is hard (impossible?) to install on windows yet.
Is there any obligatory functions from Horovod used in the scripts that I can't find anywhere else ?
I only work with 1 GPU.
Best regards.

Hi, I think you should be able to run without Horovod with minimal changes. We call hvd_init() here, which runs the functions from horovod.py, where we check of Horovod is available. I think the main change you will have to do is replace the Horovod optimizer here with the Pytorch optimizer (we should actually be checking if Horovod is available there anyway ...). Since you'll be running single GPU, you can get rid of the allreduce operations here, plus replace hvd.rank() and hvd.size() with rank() and world_size() from horovod.py.
Let me know how it goes!

It works great, I've been able to replace the hvd.functions and start the training on coco's dataset.
But since I work on private training and validation sets I've had other issues I'll may post in another topic.
Thanks for your reply !


[AutoComplete] Fix the regressions

Closes https://github.com/callemall/material-ui/issues/3854.

There is nothing to review, the fixes are really simple.

There is nothing to review

Yes there is. :smile:
This changes the signature of getStyles(). The rest of the components have getStyles(props, context, state).

Oh :scream:.
Shouldn't this be this opposite? The spec of shouldComponentUpdate is (nextProps, nextState, nextContext).

So what happened is:
Originally the components had getStyles(props, state).
When we removed the default theme, it became getStyles(props, context)
This was enough for most components, but some needed state in getMuiTheme() for other state than muiTheme, so state got appended togetMuiTheme()`s arguments only for those components.
Now that you point it out, it would make sense to follow shouldComponentUpdate() etc, but it will mean passing this.state into getStyles() many times where it isn't needed (if that matters).

Now that you point it out, it would make sense to follow shouldComponentUpdate() etc, but it will mean passing this.state into getStyles() many times where it isn't needed (if that matters).

I had this concern too. I think that consistency is more important, I have submitted a new PR to follow the code base :+1:.

In our specific case, we will be passing the context while we will not use it.

This is also true, but I believe such cases are the minority.


[WIP] First proof of concept for Nextflow logs

Aim
Aim is to support log files from Nextflow - a workflow manager / analysis pipeline builder commonly used in bioinformatics research (eg. DNA sequencing and more). Every run produces very long and verbose .nextflow.log files which are great for debugging, but extremely difficult to read. Formatting would greatly help with this.
Continuing our discussion on Discord, here are the results of my very quick and dirty play with Nextflow logs this evening.
How it looks
https://github.com/Textualize/toolong/assets/465550/7d667414-8b25-41e7-8a52-05e461242f5c
Reproducing locally
Example log file: .nextflow.log (renamed to make GitHub happy, typically called .nextflow.log, or .nextflow.log.6 etc.)
Can generate yourself by running Nextflow:
# Install Nextflow
curl -s https://get.nextflow.io | bash
# Run a pipeline with a tiny test dataset
nextflow run nf-core/rnaseq -profile test,docker --outdir results
# Check the log
tl .nextflow.log

Discussion

Things I like

Formatting of the logs, I think that they look pretty good
Native toolong functionality, such as tailing, search, navigation etc.

Things I don't like

My code
The fact that I disabled a bunch of functionality for other log types to make this work.

These logs have a few additional bits of complexity:

Log outputs can span multiple lines

Whilst looking at this during submitting the PR, I realise that anything without a timestamp is a continuation of the last log message that did. It should maintain the formatting from that log level (eg. dim for debug, red background for error etc).

There are multiple format styles

Nextflow produces output in a bunch of standardised ways. I ended up writing 5 different formatters to handle the different cases
There can also be arbitrary unstructured output from multi-line strings. These additional 4 are effectively special cases of this.

Some of the formatting rules being applied by the other formatters broke stuff / make the output look worse

I simply turn them off in this PR, but that's obviously not a real solution
This would need some kind of switch mechanism to tell toolong that we're looking at a Nextflow log file and only use that set of formatters for the rest of the session.

Going forward
This PR is just my way of asking for advice. I'm curious to see whether you think that this could / should live within the toolong codebase still, or whether it should somehow extend toolong as a separate package / plugin. Or whether it should be destroyed with fire and never be spoken of again 🔥

Thanks @ewels for being the Nextflow champion!

This would need some kind of switch mechanism to tell toolong that we're looking at a Nextflow log file and only use that set of formatters for the rest of the session.

Agreed - for the user experience pov, perhaps a $HOME/.tl.yaml file could be used for specifying the formatter level settings?
This could ideally allow the formatter specific development to proceed independently as only that particular scope with the $HOME/.tl.yaml file would be affected for turnining some features on/off.

Hi @willmcgugan!
After coming back to this, I think what would make most sense is:

Add a new --language CLI option to force parsing log files in a specific format, which can turn different highlighters on and off
Add some entry points or similar so that 3rd party tooling can add custom formatters without bloating the main toolong codebase

This would mean that the above code could move to somewhere like https://github.com/nf-core/tools and "just work" if a user has both toolong and nf-core installed (we would add toolong as a dependency to nf-core, so that would always be the case that way around). Then we can add to that custom code without bothering you and messing with the main codebase. Even the CLI flags could be handled this way if you wanted.
I can't figure out how to get the Nextflow formatters to work without disabling the other core formatters. Maybe you can, but otherwise the --language flag could force a specific set. This could also be set automatically based on filenames (these files are always called .nextflow.log*, though of course could be stdin etc).
How does this sound? If you like the idea, I can close this PR and open a different one with some entrypoints that I can start using to port the code into the nf-core codebase.
Cheers,
Phil

Closing in favour of https://github.com/Textualize/toolong/pull/47


[88] Add version script to build tools

88  - This PR adds a new command to the build tools package so you can increase the increment the project version in all files without updating them manually.
The command is build-tools version <version type> <project>
The versions currently are

Major X.0.0
Minor 1.X.0
Patch 1.1.X
major:beta X.0.0-beta.0
minor:beta 1.X.0-beta.0
patch:beta 1.0.X-beta.0
beta 1.0.0-beta.X
*X representing the value which will change when the command is applied.

The project command can be left blank which will search the whole directory or you can list projects for example theme,plugin1, plugin2 etc
This will then go through and update the files in those directories.
It currently updates the version in package.json, plugin.php and style.css
To test:
Once you have build-tools and yalc added to your test project
Run build-tools version minor  for example and this will go through all the files and folders in the directory your in and increment the version.
It will also return which files it found and inform you of the old version and the version it has been upgraded too.
Video -> http://bigbite.im/v/tf2VBx
Change Log

New version.js
Adds the semver package to increment the values.

Not able to update from project directory.
When working from a theme or plugin directory, I am not able to increment the version of that project. I should be able to increment a project version on an individual basis.
Example
cd example-site/themes/test-theme
node ../../../src/cli.js version minor

Output
No Projects Found

Short version numbers not updated.
When updating a theme or plugin, if there is an assignment of a version such as 1.0 rather than 1.0.0. As 1.0 or similar is still valid when it comes to semver standards, these should also be accepted and incremented appropriately. Here's a list of some expectations around that.

Patch → 1.0 → 1.0.1
Minor → 1.0 → 1.1.0
Major → 1.0 → 2.0.0

Example
If I change the version to 1.3 in example-site/themes/test-theme/styles.css and run the blow command.
cd example-site/themes/test-theme
node ../../../src/cli.js version minor

The blow is output, where it can be seen that it does not pick up the changed styles.css and opening the file itself will show that it remains 1.3.
searching all projects.....
Error incrementing version in client-mu-plugins/test-client-plugin/package.json: Invalid version: undefined
Error incrementing version in plugins/standards/package.json: Invalid version: undefined
#plugins/test-plugin/package.json: 1.4.0 -> 1.5.0
#plugins/test-plugin/src/plugin.php: 1.4.0 -> 1.5.0
#themes/test-theme/package.json: 1.3.0 -> 1.4.0

package-lock.json file not incremented
When updating a version, the package-lock.json also needs incrementing to keep it consistent with the package.json increase. Its also worth bearing in mind that this file may not exist, in which case it should be allowed to be skipped, so we're not forcing this file to exist even when required.

There are no options for a Release Candidate
Not in the original ticket but it would be nice to be able pass an rc sub command through to the script to create a Release Candidate version. This should work in the same way that has been implemented for Beta releases with the *:beta subcommand.
Example
build-tools version minor:rc

Output
#plugins/test-plugin/package.json: 1.8.0 -> 1.9.0-rc.1
#plugins/test-plugin/src/plugin.php: 1.8.0 -> 1.9.0-rc.1

Messaging inconsistencies
Terminal output doesn't have consistent approaches and language. The following changes should be made:

searching all projects.... → Incrementing All Projects...
finding all projects to increment.... → Incrementing List of Projects...

non-lowercase subcommands fail
Using subcommands that are not lowercase will fail, causing the same null assignment that happens without a subcommand.
Example
cd example-site/themes/test-theme
node ../../../src/cli.js version MINOR

Output
Error incrementing version in client-mu-plugins/test-client-plugin/package.json: Invalid version: undefined
Error incrementing version in plugins/standards/package.json: Invalid version: undefined
Error incrementing version in plugins/test-plugin/package.json: Invalid version: null
#themes/test-theme/package.json: 1.8.0 -> null
#themes/test-theme/style.css: 1.2.0 -> null

You have two options on how to approach this:

Force all subcommands to be treated as lowercase so they're always actionable when used, or
Check for non-valid subcommands and throw an error, preventing the process from going further and creating null versions.

Spacing inconsistencies
There's a fair amount of spacing consistencies between variable, and logical declarations. There are some indentation issues across files that would need addressing also.
const example = 'value';

if (logicCheck !== 123) {
	// ...
}

Keeping some separation will help readability overall for others.

@ampersarnie  on this comment https://github.com/bigbite/build-tools/pull/119#issuecomment-2148227042
Could you let me know what im missing here please? when I add a RC to the command it adds a rc release number http://bigbite.im/i/qcTpJF
Also I added the 1 to the prerelease to semver inc as requested however it still comes out as .0 I wondered if you know where im going wrong with this?
Thanks 😄

@jordanharris-6

Could you let me know what im missing here please? when I add a RC to the command it adds a rc release number http://bigbite.im/i/qcTpJF

When I'm looking at it, its increasing the wrong version. So if I have a current version of 10.0.0 and run a major:rc version increase, what I'm not left with is 10.0.1-rc.1. It's increasing the patch number, not the major. I would expect to get 11.0.0-rc.1.

Also I added the 1 to the prerelease to semver inc as requested however it still comes out as .0 I wondered if you know where im going wrong with this?

I changed it on your previous version to a 1 and it comes out as expected. But I'm not able to test with your latest changes as I'm getting the following error:
example-site git:(feat/88-version-increment) ✗ node ../src/cli.js version major:beta
Incrementing All Projects...
ReferenceError: path is not defined%

@ampersarnie I think I have solved this by removing an if statement and adding in another command http://bigbite.im/v/ZIup9l
You can now do build-tools version prerelease:beta  << this will change the last value in the version after -beta

@ampersarnie I've refactored this now to re-use the existing functions


AssertionError: "Editable installs cannot rebuild an absolute wheel.install-dir. Use an override to change if needed."

Given
[tool.scikit-build]
wheel.install-dir = "/data"

It is not possible to do editable installs:
$ pip install -e .

because scikit_build.build.wheel.py::_make_editable() throws.
Commenting out this check did not seem to break anything immediately with the wheel, so perhaps this check can be moved to runtime install of install-time.

You can't install editably to /data. See https://peps.python.org/pep-0660/#limitations

Though, it's possible that it could still work as long as you don't want the editable parts to be in /data, we should to verify that. The check might be too restrictive.

Though, it's possible that it could still work as long as you don't want the editable parts to be in /data, we should to verify that.

Yes, that is exactly what my build is doing. Some pieces are being installed into /data (and I'm OK that those aren't "editable" in the python sense), but other parts are being installed normally into site-packages. And those work with editable installs.

So is it possible for scikit-build-core to defer this check? It looks like the generated _<package>_editable.py::install() has a rebuild parameter that is passed to ScikitBuildRedirectingFinder(). This in turn is then used in ScikitBuildRedirectingFinder.find_spec():
# ...
if self.rebuild_flag: # A.K.A. "rebuild"
    self.rebuild()

Perhaps the the "is install-dir absolute" check can be put inside self.rebuild, that way it only fires if the feature is actually requested (and exercised).

Another approach that we should eventually support is to not run cmake --install at all and redirect the editable paths to the build directory. Would that also be a workable workflow for you? You would need to do some manipulation to make the build directory be workable, without the wheel.install-dir = "/data" assumption.

cmake --install at all and redirect the editable paths to the build directory.

That would be even better.  Copying these into the wheel takes a long time, so if scikit-build-core could skip doing that for editable builds and just leave them in the build directory that would dramatically speed up our workflow


[META] Kubernetes version upgrade

With k8s 1.26 unavailable in DigitalOcean and coming to the end of standard support in AWS, GCP and Azure, users should start upgrading their k8s versions.
There is already https://github.com/nebari-dev/nebari/pull/2276 to update the highest k8s supported version in Nebari. That should fix https://github.com/nebari-dev/nebari/issues/2285.
Here is a list of k8s versions supported by each cloud provider (might vary depending on regions):
GCP
$ gcloud container get-server-config --region us-central1
Fetching server config for us-central1
channels:
- channel: RAPID
  defaultVersion: 1.29.0-gke.1381000
  validVersions:
  - 1.29.2-gke.1060000
  - 1.29.1-gke.1589000
  - 1.29.0-gke.1381000
  - 1.28.7-gke.1026000
  - 1.28.6-gke.1456000
  - 1.28.3-gke.1286000
  - 1.27.11-gke.1062000
  - 1.27.11-gke.1018000
  - 1.26.14-gke.1044000
  - 1.26.14-gke.1006000
  - 1.25.16-gke.1570000
  - 1.25.16-gke.1537000
- channel: REGULAR
  defaultVersion: 1.27.8-gke.1067004
  validVersions:
  - 1.29.0-gke.1381000
  - 1.28.5-gke.1217000
  - 1.28.3-gke.1286000
  - 1.27.10-gke.1055000
  - 1.27.9-gke.1092000
  - 1.27.8-gke.1067004
  - 1.26.13-gke.1144000
  - 1.26.13-gke.1052000
  - 1.26.8-gke.200
  - 1.25.16-gke.1460000
  - 1.25.16-gke.1360000
- channel: STABLE
  defaultVersion: 1.27.8-gke.1067004
  validVersions:
  - 1.28.3-gke.1286000
  - 1.28.3-gke.1203001
  - 1.28.3-gke.1118000
  - 1.27.8-gke.1067004
  - 1.27.7-gke.1121002
  - 1.26.11-gke.1055000
  - 1.26.10-gke.1101000
  - 1.26.8-gke.200
  - 1.25.16-gke.1041000
  - 1.25.15-gke.1115000
defaultClusterVersion: 1.27.8-gke.1067004

DO
$ doctl kubernetes options versions
Slug            Kubernetes Version    Supported Features
1.29.1-do.0     1.29.1                cluster-autoscaler, docr-integration, ha-control-plane, token-authentication
1.28.6-do.0     1.28.6                cluster-autoscaler, docr-integration, ha-control-plane, token-authentication
1.27.10-do.0    1.27.10               cluster-autoscaler, docr-integration, ha-control-plane, token-authentication

Seems like this is done since 1.29 is supported now


Add some endpoints

Hi,
Following endpoints(methods) have been added:
https://binance-docs.github.io/apidocs/spot/en/#get-all-margin-assets-market_data
https://binance-docs.github.io/apidocs/spot/en/#get-all-cross-margin-pairs-market_data
https://binance-docs.github.io/apidocs/spot/en/#get-interest-history-user_data
https://binance-docs.github.io/apidocs/spot/en/#get-force-liquidation-record-user_data

Thanks @mfiro

Hi @mfiro could you please also help with this?
https://github.com/sammchardy/python-binance/issues/1241


Replace Docker Client with ORAS to handle interaction with OCI registries like DockerHub

Description
The recommend command is currently responsible for fetching policies from the policy-templates repository and suggesting them according to the workloads present in the cluster. It adjusts selectors and other policy attributes to tailor the recommendations. This process involves interactions with Docker Hub, pulling images, and extracting image details using the Docker client.
Proposed Change
Replace the Docker client with ORAS (OCI Registry as Storage) to handle interaction with OCI registries like DockerHub. ORAS is a tool that supports OCI-compliant registries, providing a more flexible and standard approach. ORAS offers a client library in Go that can be used: https://oras.land/docs/client_libraries/overview
Tasks

[ ] Identify all locations in code with interaction with OCI registries
[ ] Update the Scanner struct to use ORAS for registry interactions.
[ ] Replace the pullImage function to use ORAS for pulling images.
[ ] Modify the logic to save pulled images to a tar file using ORAS.

#340


Can't add User Groups

I'm from dhaka,bangladesh. So I thought of adding the user group I'm a part of to phpug, but theres always an error with the location saying "The given longitude exceeds the range of -180 to 180 degrees".

Ah, That can happen when you "rotate" the map too far. To get you set up, ping me via the contact-form or via email with the details and I'll get you set up.
Besides that I'll have to adapt the validation to also allow different values and then reduce them to appropriate values.

Hey @omranjamal - Sorry for not following up on this one! Have you tried adding the group again? Can you give me more informations on the group? I'd love to get you up and running

As there was no response, I'll close this now…


fmt hyperlinked CLI text

Is your feature request related to a problem? Please describe.
A function for creating hyperlinked text to print to CLI output (see Hyperlinks (a.k.a. HTML-like anchors) in terminal emulators).
Not sure whether this should go under fmt/colors or a new fmt/ansi. fmt/colors makes most sense to me given it already contains various formatting functionality that doesn't strictly relate to color.
Example:
console.log("\x1B]8;;https://deno.land\x07Deno dot land\x1B]8;;\x07")

^ prints in CLI console as Deno dot land.
Describe the solution you'd like
Something like this:
export function hyperlink(text: string, url: string | URL) {
    return `\x1B]8;;${url}\x07${text}\x1B]8;;\x07`
}

Describe alternatives you've considered

What should NO_COLOR output be? Maybe a markdown equivalent like [Deno dot land](https://deno.land)? Edit: or maybe not, this could easily break layouts. Probably best to just print the text without the hyperlink.
This feature isn't supported by all TTY environments — is there some way of feature detecting it? (Probably not). Per the gist linked above, all ECMA-48-compliant terminals that don't implement the feature simply fall back to showing the text with no hyperlink.

The support of that special sequence seems a bit too limited?

In spring 2017, GNOME Terminal and iTerm2 have changed this.

There seem no support on windows, and also default terminal of mac doesn't seem supporting?

Windows Terminal supports it.

👍

Not sure about macos and don't have access to a mac device to find out.

I tested with Terminal.app (the default one in mac), and it doesn't seem supporting it.

On the other hand, I found that the builtin terminal of VSCode seems supporting it.

Another note: jetbrain editor's builtin terminal supports this
Looks like this is widely adopted except Warp and Terminal.app. However it is a little concerning that Terminal.app is the default of mac os..


sigma should be a float

There is a bug when we set sigma to be an integer (FM = ConditionalFlowMatcher(sigma=0)). It modifies the type of the time tensor t.

Fixed with #72


False error about redeclaration of Final private(double-underscore) variables

Environment data

Language Server version: 2020.12.2
OS and version: macOS Catalina 10.15.7
Python version (& distribution if applicable, e.g. Anaconda): Python 3.8.5(Anaconda)

Actual behavior
from typing import Final

class Parent:
    __a: Final[int]

    def __init__(self):
        self.__a = 0
        _ = self.__a

class Child(Parent):
    # Error: "__a" cannot be redeclared because parent class "Parent" declares it as Final
    __a: Final[int]

    def __init__(self):
        self.__a = 0
        _ = self.__a

The code above is largely simplified and __init__ methods are there only to suppress "final but not assigned" errors and "unused" warnings.
Expected behavior
No error.
Reason
A class member named with a prefix of double-underscore and without a suffix of double-underscore is meant to be purely private. There should be no collisions between such members even if they share a same name.

@final private methods are OK, though there should be no needs to add @final to private methods.

The behavior is correct as per PEP 591.
As confirmation, the same error is emitted by other type checkers like mypy.

I think one could make the argument that PEP 591 should make an exception for class members with double-underscore names since they are obfuscated and therefore cannot conflict with same-named members in parent classes.
@gvanrossum, I'm interest in any advice you can offer here.

The behavior is correct as per PEP 591.
As confirmation, the same error is emitted by other type checkers like mypy.

Your are right that mypy has the same behavior.
But I'm quite confused that Final variables and @final methods are treated in different ways. As I just checked, overriding a @final method prefixed with either single-underscore or double-underscore does NOT get an error, while I believe that  at least the single-underscore case deserves one.

Revisiting this, I do think there's a bug here:

In the derived class, only the double underscore prefixed things should be error-less (as they get hidden to children and are not a part of the API). I.e., __bar has an error but should not, and _foo does not have an error but should.

Revisiting this, I do think there's a bug here:

In the derived class, only the double underscore prefixed things should be error-less (as they get hidden to children and are not a part of the API). I.e., __bar has an error but should not, and _foo does not have an error but should.

I know there's sort of a question as to whether or not __ should be exempt from Final (I'd argue it should), but the single underscore bit appears to be a bug.

I know there's sort of a question as to whether or not __ should be exempt from Final (I'd argue it should), but the single underscore bit appears to be a bug.

The __ prefix translates to name mangling (C.__foo really is a variable named _C__foo) so it really should be exempt. Agree that the single _ appears to be a bug.

The __ prefix translates to name mangling (C.__foo really is a variable named _C__foo) so it really should be exempt. Agree that the single _ appears to be a bug.

I've updated the logic in pyright to exempt private (double underscored) symbols from this check. @gvanrossum, you can decide whether to update PEP 591 to reflect this exemption. Also, it appears that mypy may need to also apply this fix.
I've also fixed the logic for single-underscore methods.

I've updated the logic in pyright to exempt private (double underscored) symbols from this check. @gvanrossum, you can decide whether to update PEP 591 to reflect this exemption. Also, it appears that mypy may need to also apply this fix.
I've also fixed the logic for single-underscore methods.

I don't see why PEP 591 should be updated -- the two __foo symbols are unrelated, by virtue of the language definition for private variable name mangling.

I don't see why PEP 591 should be updated -- the two __foo symbols are unrelated, by virtue of the language definition for private variable name mangling.

That logic makes sense, yet both pyright and mypy maintainers missed this point (or came to different conclusions) because PEP 591 didn't say anything on the topic.

That logic makes sense, yet both pyright and mypy maintainers missed this point (or came to different conclusions) because PEP 591 didn't say anything on the topic.

Okay, that's fair.

Okay, that's fair.


Fixed NeoAvatar size bug

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[x] Bugfix
[ ] Feature
[ ] Refactoring

Context

[x] Closes #8392
[ ] Requires deployment <snek/rubick/worker>

Before submitting pull request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality

Optional

[ ] I've tested it at </ksm/collection>
[ ] I've tested PR on mobile
[ ] I've written unit tests 🧪
[ ] I've found edge cases

Did your issue had any of the "$" label on it?

[x] Fill up your DOT address: Payout

Community participation

[x] Are you at KodaDot Ecosystem Telegram?

Screenshot 📸

[x] My fix has changed something on UI; a screenshot is best to understand changes for others.

Copilot Summary

🤖[deprecated] Generated by Copilot at e0f22ca
Updated the ImageMedia and NeoAvatar components to use a new prop for customizing the image component. This improves the consistency and flexibility of the image component across the UI.

🤖[deprecated] Generated by Copilot at e0f22ca

NeoAvatar rises from the ashes
With image-component-props it smashes
The old constraints of width and height
And fit as cover gives it might

Thanks!
pay 30 usd

😍 Perfect, I’ve sent the payout
💵 $30 @ 5.55 USD/DOT ~ 5.405 $DOT
🧗 16SpvdDgKNiQ3c53DxX7refnQcKUD3uRNim3Z1HBJLNGrtSP
🔗 0x6f75172463b362ea62e901de8a67d981a56bd276b15905ce6c39e9a0c013e1bb
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


Fix bytestring depwarn

bytestring -> String(copy()). This does not need a Compat min version bump. Also supersedes #55 I think.

Coverage remained the same at 35.2% when pulling 04be75d1fa59c79d50b6388488c6a793a2bf1701 on ranjanan:RA/depwarn into f175982e6469a20ed2133293f4ed4cfbdff39f0e on JuliaWeb:master.

I think the copy is not needed here...

@shashi #55 does not include the copy and doesn't pass Travis (maybe that's unrelated, not sure.) The deprecation warning suggests fixing it to  String(copy()) too. Maybe I can remove the copy and see if it passes Travis here. If not, I can revert it.

Coverage remained the same at 35.2% when pulling bb898156d31649e2e2876450d492abaf1e138183 on ranjanan:RA/depwarn into f175982e6469a20ed2133293f4ed4cfbdff39f0e on JuliaWeb:master.

Out of curiosity, why doesn't this need @compat String? I'm dealing with Compat for the first time and I would have used @compat here. Maybe I'm doing something wrong. Thanks

Also, I just notice the dep warning for bytestring(v) suggests using String(copy(v)). Why don't we need copy here? Is the suggestion wrong?

@EricForgy The reason you don't need @compat over here is because this is not a syntax rewrite. Basically Compat would internally define a constant String which is the Union{UTF8String, ASCIIString} if the Julia version is less than v0.5. import Compat.String would now simply bring it into scope. On v0.5, String would simply be Base.String

Thanks @ranjanan . I did not see the import Compat.String :sweat_smile:

Any chance this can be merged (and maybe even tagged)? I have a "dirty" copy which makes me feel... err... dirty :)

@shashi Could you merge this please?


Config issue about 'Res5ROIHeadsExtraNorm'

Hi, thanks so much for providing the detection configs.
When I run detectron2 based on "Base-RCNN-C4-BN.yaml", the process said that "Res5ROIHeadsExtraNorm" has not been registered?
And I carefully read the documents of detectron2,
it has 'Res5ROIHeads', but does not have 'Res5ROIHeadsExtraNorm'.
Does Res5ROIHeads == Res5ROIHeadsExtraNorm?

来信收到，谢谢！


Uniquely determine the representation of a data type from its definition

A custom PlutusType instance should be unsafe.
It should be such that Coercible a b also means that they have the exact same representation.

The alternative is to have type role Term phantom nominal which might be more sensible TBH

But this would break deriving via entirely.

This isn't a good idea


Blocked by AVG

Today the app was blocked by AVG (URL blacklist)

and exploit protection

I can't really do anything about overzealous antivirus software.

I tried, but the antivirus still deletes the Sam.Picker.exe. The only option is to turn off the antivirus.


Vitessce JS dependency not upgrading on Google Colab

When I install the latest version of the vitessce Python package on Google Colab, the vitessce JS version is still 1.1.0. It should be 1.1.14. It is unclear to me why this is happening.
For example:
https://colab.research.google.com/github/vitessce/vitessce-python/blob/keller-mark/colab-notebook/docs/notebooks/widget_on_colab.ipynb
Run the cells in this notebook and then open the console. It says the vitessce version is 1.1.0 and the config validation fails because the Python code is generating a config with schema version 1.0.4 but that schema did not exist under the older vitessce JS package.

Fixed by https://github.com/vitessce/vitessce-python/pull/111


Make toolchains in Groovy and Scala tasks non-optional

Depends on https://github.com/gradle/gradle/pull/22108

The Groovy part is here:

https://github.com/gradle/gradle/pull/22835


Tensor : SafeHandle

DRAFT changes to discuss whether Tensor (and all other native handles) should not be based on SafeHandle. See link for why.
https://docs.microsoft.com/en-us/dotnet/api/system.runtime.interopservices.safehandle?view=net-6.0

Fails on all builds with:
src/TorchSharp/NN/Module.cs(428,74): error CS0122: 'SafeHandle.handle' is inaccessible due to its protection level [/__w/1/s/src/TorchSharp/TorchSharp.csproj]

This is just one of several locations with the same error.

@nietras I am sorry, where's the "why" part in that link?
The rest of .NET classlib does not have common classes inherit from SafeHandle directly or indirectly. Instead, an instance of handle is usually internal to implementation (e.g. FileStream is not a SafeHandle, but contains one instead).
This makes sense also as a separation between public interface and internal implementation details.

I'm not at all sold on this. Why is this better than what we already have in place?

@nietras -- are you still of the opinion that this change should be made?


[enhancement] Add missing bower.json.

Hey, maintainer(s) of firebug/firebug-lite!
We at VersionEye are working hard to keep up the quality of the bower's registry.
We just finished our initial analysis of the quality of the Bower.io registry:
7530 - registered packages, 224 of them doesnt exists anymore;
We analysed 7306  existing packages and  1070 of them don't have bower.json on the master branch ( that's where a Bower client pulls a data ).
Sadly, your library firebug/firebug-lite is one of them.
Can you spare 15 minutes to help us to make Bower better?
Just add a new file bower.json and  change attributes.
{
  "name": "firebug/firebug-lite",
  "version": "1.0.0",
  "main": "path/to/main.css",
  "description": "please add it",
  "license": "Eclipse",
  "ignore": [
    ".jshintrc",
    "**/*.txt"
  ],
  "dependencies": {
    "<dependency_name>": "<semantic_version>",
    "<dependency_name>": "<Local_folder>",
    "<dependency_name>": "<package>"
  },
  "devDependencies": {
    "<test-framework-name>": "<version>"
  }
}

Read more about bower.json on the official spefication and nodejs semver library has great examples of proper versioning.
NB! Please validate your bower.json with jsonlint before commiting your updates.
Thank you!
Timo,
twitter: @versioneye
email<EMAIL_ADDRESS>VersionEye - no more legacy software!

@deepak1556 if the refactoring is done would you consider adding the bower.json and making us happy?
Bower is not "just" a package manager. It is "the" package manager for the front-end stuff. npm is mostly used for Node and Browserify while Bower is the most popular package manager for code that loads into browser, i.e. a good use case for Firebug lite.
I think Firebug is a really awesome package and it would be a pitty that in order to install it we have to download a tar package and extract it while the rest of our front-end packages are nicely installed using bower. I can happily help with the bower.json file (which describes the package).


add first mention of person when about=person

When an article is classified as about a person, it outputs the first sentence mentioning the person. One issue is that this might be the same as the first sentence!

It works for me.

I reckon it might be worth expanding this to list all mentions of the about:PERSON.

Another e.g. of about=person with >1 full name mention is 9697dd94-430a-11e8-803a-295c97e6fd0b

LGTM


[cognitive speech-service Long Audio API] Is it possible to retrieve words boundaries with this API ?

Is it possible to retrieve words boundaries with the "Long Audio API" like we can do with the "cognitive-services-speech-sdk-js" text-to-speech ?
If it is not possible with the "Long Audio API", is it possible with the "batch transcription service" or any Azure TTS service that does not have the Max 10min audio limit ?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 6a0f7a9a-7459-927f-743f-7fe8f649b85e
Version Independent ID: 7a99ae60-b57d-9770-3602-d993fa156db9
Content: Long Audio API - Speech service - Azure Cognitive Services
Content Source: articles/cognitive-services/Speech-Service/long-audio-api.md
Service: cognitive-services
Sub-service: speech-service
GitHub Login: @trevorbye
Microsoft Alias: trbye

@XavM we'll review your feedback and get back to you shortly, thanks.

@wolfma61, can you please review the above inquiry? Thanks.

@XavM  Long Audio API doesn't support word boundaries. The Real Time API has the limitation of 10 mins, but with SDK, you can synthesize  long sentences to a single audio file.

Please try if this blog can help you with the word boundary for long content consumpiton:
https://techcommunity.microsoft.com/t5/azure-ai/enable-read-aloud-for-your-application-with-azure-neural-tts/ba-p/2301422

What's the scenario you want to use? It can help me understand better on what solution would be good for this case.


[SPARK-18843][Core]Fix timeout in awaitResultInForkJoinSafely (branch 2.1, 2.0)

What changes were proposed in this pull request?
This PR fixes the timeout value in awaitResultInForkJoinSafely for 2.1 and 2.0. Master has been fixed by https://github.com/apache/spark/pull/16230.
How was this patch tested?
Jenkins

Test build #70091 has started for PR 16268 at commit a664cd5.

I see, so there is no point in applying this to master, because it's as if it were already superseded by a bigger fix? but we 'back port' this stop-gap to earlier branches.

Yep. I want to backport this obvious fix without the entire patch for master.

Test build #70091 has finished for PR 16268 at commit a664cd5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70091/
Test PASSed.

LGTM

Thanks! Merging to 2.1 and 2.0.


fix initialization of 'has_imu' property in depthai_descriptions model

fix flag has_imu in xacro macro depthai_camera to also support the OAK-D PRO and POE models.
Overview
Author: Jennifer Buehler
The flag has_imu in xacro macro depthai_camera is only set for the OAK-D model. With the other cameras which have an IMU it doesn't work.
It should  instead be
<xacro:if value="${model in ['OAK-D', 'OAK-D-PRO', 'OAK-D-POE']}">
  <xacro:property name="has_imu" value="true" />
</xacro:if>

Issue
Issue link (if present): n/a
Issue description: the flag has_imu in the depthai_camera macro is only set to true for the 'OAK-D' model, although PRO and POE also have IMUs.
Changes
ROS distro: humble, kinetic, probably all
List of changes: fix depthai_camera initialization of property has_imu
Testing
Hardware used: Oak-D-Pro W
Depthai library version: current (2023-09-23) ROS kinetic package depthai-ros
Visuals from testing
n/a, a screenshot of a non-existent IMU in RViz could be made but that's not really helpful

Hi, unfortunately kinetic distribution is not officially supported as it reached EOL in 2021, currently only officially supported ROS1 distribution is noetic.

Hi, ahrg I meant noetic of course, wasn't thinking ;)

Hi, yes, fix will be merged in https://github.com/luxonis/depthai-ros/pull/424


Broken link in README

The nile in 3 mins link is going to 404, assume it should point to...
https://www.thenile.dev/docs/getting-started/whatisnile

I just fixed it a few mins back. Could you try again?


Broken kotlin source compatibility

I think this is hidden because KotlinSourceCompatibilityTest is in the same package
Was: import okhttp3.CipherSuite.forJavaName
Now import okhttp3.CipherSuite.Companion.forJavaName

Fixes a proven by KotlinSourceModernTest

Documented! https://github.com/square/okhttp/pull/5176


jsx-curly-brace-presence with a single empty string child

The jsx-curly-brace-presence rule wants to convert this:
<MyComponent>{''}</MyComponent>;

to
<MyComponent></MyComponent>;

but they are semantically different. In the first example, MyComponent is passed a child of an empty string, whereas in the second example MyComponent is passed a child of null.
I'd expect jsx-curly-brace-presence to leave empty string single children alone.
cc @yungsters

can you elaborate on what MyComponent is doing that would legitimately distinguish between these?


Add Fog module

There is no Fog module. It would be useful to have one so that we don't have to fiddle with the native scene properties
Version:

[x] v2.x.x
[ ] v1.x.x

Issue type:

[ ] Bug
[x] Proposal/Enhancement
[ ] Question

 Tested on:
Desktop

[ ] Chrome
[ ] Chrome Canary
[ ] Chrome dev-channel
[ ] Firefox
[ ] Opera
[ ] Microsoft IE
[ ] Microsoft Edge

Android

[ ] Chrome
[ ] Firefox
[ ] Opera

IOS

[ ] Chrome
[ ] Firefox
[ ] Opera

done.


Ref: Array::some instead of reduceBool()

It's a day of insights today :)

coverage: 100.0%. remained the same
when pulling 73cf76b2f16da121f686c4c6539d7119d47c45cb on no-reduce-bool
into 1873a3d45d6fde30bfd01d07d51b07fe6fb2f299 on master.


Packager isn't detecting changes on node_modules

Description
Making changes on files on node_modules does not trigger a change in the bundler, therefore the new changes are not bundled into the app unless the packager is restarted.
Reproduction
Change a file at the node_modules folder and check that the bundler does not detect the file change (https://github.com/facebook/react-native/blob/ede04abf8f4463155c196c6cfb8ea47684e8c6b8/packager/react-packager/src/Server/index.js#L396 is never called). Older versions of react-native would detect this and mark the file as outdated.
Solution
Reenable the watching of node files or allow a way to customise this via config/env variables
Additional Information

React Native version: 0.39.2
Platform: iOS
Operating System: MacOS

Adding the package name here seems to fix the problem. Is there a way to pass this parameter via command line or env variable?

Same here. Installing new modules with npm install are not picked by app. I need to stop the packager and restart it completely.

duplicate https://github.com/facebook/react-native/issues/11301

Close in favor of https://github.com/facebook/react-native/issues/11301


VisualShader Varyings Appearing Incorrectly In Inspector

Godot version
v4.1.1.stable.official [bd6af8e0e]
System information
Windows 10
Issue description
When a varying is created in a VisualShader, it seems to always appear as a Vector2, and is unable to be changed.
Steps to reproduce
Create a visual shader and assign it to a material/mesh. Click "Manage Varyings" and create a new varying. Go to the shader's inspector window and try to change the value of the varying.
Minimal reproduction project
VaryingsIssue.zip

Duplicate of https://github.com/godotengine/godot/issues/79109 (fixed on master)


Running with 2M PSRAM ? (OCDOE-12)

Hi,
Thanks for the great project,  and I tested on my ESP32S3 eval board, all functions work as a charm.
Meanwhile, I am working my hardware version for this project, I would like to choose 2M PSRAM chip to build this.
I just print out the SPI PSRAM usage, I found the maximum usage is about 1.2MB memory size, which indicates that the project could be run on 2M PSRAM.
I just want to know that,

Have you try run the project on the ESP32S3 with 2M PSRAM?
Is it possible to run this project smoothly with 2M PSRAM?
What‘s the minimum PSRAM requirement for this project?

Hi @solosky it is nice to hear you have managed to work it without an issue.
Yes, I tested basic debugging with a 2MB PSRAM devkit. It was fine. Unfortunately, I didn't check how much PSRAM memory was used during runtime. So I can't say the exact number now. But 1.2MB looks reasonable.
When you design your PSRAM hardware, please check the IDF support of course. For the octal type, afaik, min 4MB is required.
Keep in mind that with the addition of (UI) LVGL, PSRAM usage may also increase.


[Translatable] After Bundle Upgrade: ReflectionException Property Namespace\MyBundle\Entity\CategoryTranslation::$id does not exist

All worked as expected but after the upgrade:

Removing knplabs/doctrine-behaviors (dev-master b7083e9)
Installing knplabs/doctrine-behaviors (dev-master 2ff5031)

I get the error in the title.
What do I need to write in composer.json to do the downgrade?
PS: I use Standard Symfony 2.5.5 with upgraded components. What do I need to show to get you able to reproduce that error?
Thanks!

issue still present in dev-master...

Do you still experience this issue in the last versions, or was it fixed by #158 ?

Sorry @Einenlum for the late reply but I was away for some time. The project from which I started this issue is finished long time ago (and now it's followed by others), but soon I should start another one using Symfony4.

I'm going through the old issues/PRs and closing most of them so we can focus on now.
This one is missing any feedback or reasoning so I cannot deduce anything from it.
Thanks for you though, but closing for reasons above.
If the bug still remains, please send failing test case to verify it. Only with test we'll be albe to help.


Add SPM support

Wondering if this is on the roadmap?

We want to make this as soon as possible but there is no ETA.


Wrong reference address in .xml and .rel files.

I struggled for days understanding why I would not manage to get it working.
This post refers to the wrong addresses, it should be "http://" and NOT "https://". I figured out when valudating the XML.
This is true for the customUI.xml file, as well as the .rels file.
customUI.xml should be:
<customUI xmlns="http://schemas.microsoft.com/office/2006/01/customui">
<ribbon>
<tabs>
<tab id="CustomTab" label="My Tab">
<group id="SampleGroup" label="Sample Group">
<button id="Button" label="Insert Company Name" size="large" onAction="ThisDocument.InsertCompanyName" />
</group >
</tab>
</tabs>
</ribbon>
</customUI>
.rel should be:
<Relationship Id="someID" Type="http://schemas.microsoft.com/office/2006/relationships/ui/extensibility" target="customUI/customUI.xml" />

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: f45df1b9-16f4-4c5e-269c-49d0d58fa2b1
Version Independent ID: 9d77130d-483c-20b3-119c-592e6b51dfbd
Content: Customize the Office Fluent Ribbon by Using an Open XML Formats File
Content Source: Library-Reference/Concepts/customize-the-office-fluent-ribbon-by-using-an-open-xml-formats-file.md
Product: office
GitHub Login: @o365devx
Microsoft Alias: o365devx

@AzmL
Thanks for the feedback. I changed these references to http://, and also updated other schemas links accordingly. Let me know if you have any questions.