tf.reduce_min([inf, nan]).eval() == inf, should be nan

Describe the problem
tf.reduce_min and tf.reduce_max do not propagate nans correctly.  E.g.,
>>> tf.reduce_min([inf, nan]).eval()
inf

but the correct answer (the one most useful for debugging) is nan.  This is presumably due to a backwards comparison, similar to the old tf.nn.relu(nan) == 0 bug that @alexalemi found.
Source code / logs
Here's a colab illustrating the problem with TensorFlow 1.6.0-rc1: https://drive.google.com/file/d/1nDA0Q48PveBlx_D5Zurchbw8l7eczRSB/view?usp=sharing

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
TensorFlow version
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce

OS Platform and distribution: colab
TensorFlow installed from: intrinsic to colab
TensorFlow version: 1.6.0-rc1
Bazel version: n/a
CUDA/cuDNN version: none
GPU model and memory: none
Exact command to reproduce: tf.reduce_min([inf, nan]).eval() or run the colab

Oops, looks like the colab was only shared within OpenAI.  Should be fixed.

Related issue: https://github.com/tensorflow/tensorflow/issues/12659

@mrry Is the underlying issue a dupe of #12659?

It sounds like it. Closing as a duplicate of #12659.

@mrry A dupe means that fixing #12659 will necessarily fix this one, which isn't obvious to me.  Similar causes doesn't mean duplicate.

Assigning to @benoitsteiner, since he's got #12569, and is best placed to figure out if there is a common solution.

Nagging Assignee @benoitsteiner: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @benoitsteiner: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @benoitsteiner: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @benoitsteiner: It has been 15 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @benoitsteiner: It has been 78 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @benoitsteiner: It has been 95 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @benoitsteiner: It has been 110 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @benoitsteiner: It has been 125 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @benoitsteiner: It has been 140 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @benoitsteiner: It has been 155 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @benoitsteiner: It has been 170 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Gah, this is a bug in fmin itself: https://en.cppreference.com/w/c/numeric/math/fmin :(

Julia thread which similarly concluded that the C standard is broken and fixed Julia to not follow the broken C standard: https://github.com/JuliaLang/julia/issues/7866

Nagging Assignee @benoitsteiner: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

We depend on  Eigen::internal::MinReducer<float>() for the implementation of tf.reduce_min(), so I'm assigning this to @rmlarsen, who's most qualified to comment on the Eigen behavior.

Is it related to #12659 ?

Nagging Assignee @rmlarsen: It has been 15 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @rmlarsen: It has been 30 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @rmlarsen: It has been 45 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @rmlarsen: It has been 61 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @rmlarsen: It has been 76 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @rmlarsen: It has been 91 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Hi @girving ! This issue is not replicating in TF 2.7 .Closing as it is resolved in latest version. Thanks!

Are you satisfied with the resolution of your issue?
Yes
No


Additional #1085 feedback

If you choose to leave the first line of the Important things to note as it is, then the words immediately & other should be changed to bold.

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

ID: 2e31ad37-068e-3007-2e46-7afc15f6b73b
Version Independent ID: cd16bd61-cd0d-be6b-0af7-c1e9d81c81f5
Content: Using Designer - Business Central
Content Source: dev-itpro/developer/devenv-inclient-designer.md
Service: dynamics365-business-central
GitHub Login: @SusanneWindfeldPedersen
Microsoft Alias: solsen

Thanks for the feedback. This will be further emphasized with the next doc refresh. Thanks.


[google_sign_web] error compiling for iOS

Steps to Reproduce

add the plugin google_sign_in_web
try to compile for ios

Target Platform: IOS
Target OS version/browser: MacOS 10.14.6
Devices: iOS Simulator 8
Logs

Hi @arctouch-felipefontes
are you using both google_sign_in  and google_sign_in_web in your project?

Remember that for web plugins you need to depend both on the "native" version that provides the Dart interface that you'll use in your app), and the "web" version, that provides the implementation of the plugin for the web platform.

could you please provide your updated  flutter doctor -v
and your flutter build --verbose ?
Also, to better address the issue, would be helpful
if you could post a minimal code sample to reproduce the problem
Thank you

Yes i'm using both plugins. i'am able to login in web so everything looks ok, i have a project that started in android & iOS, but i'm trying to have them as web app to using the bloc that I already have only changing the UI.
I didn't put the code sample because i just remove the package  google_sign_in_web: ^0.8.0 from the pubspeck and ios can build fine.
Here is a project that i'm using to test it. i remove the google key to be able to share it.
https://drive.google.com/file/d/1L4OfPfZJYo8nzS-xjp47NyDjf-ZY2Ug2/view?usp=sharing
Here is the flutter doctor -v.
[✓] Flutter (Channel dev, v1.12.12, on Mac OS X 10.14.6 18G103, locale en-BR)
• Flutter version 1.12.12 at /users/felipefontes/development/flutter
• Framework revision bae92c32e5 (28 hours ago), 2019-11-24 22:29:28 -0800
• Engine revision b6b54fd606
• Dart version 2.7.0
[!] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
• Android SDK at /Users/felipefontes/Library/Android/sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-28, build-tools 28.0.3
• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
! Some Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses
[✓] Xcode - develop for iOS and macOS (Xcode 11.2)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 11.2, Build version 11B52
• CocoaPods version 1.8.4
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 3.5)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin version 41.0.2
• Dart plugin version 191.8593
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
[✓] VS Code (version 1.40.1)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.6.0
[✓] Connected device (2 available)
• Chrome     • chrome     • web-javascript • Google Chrome 78.0.3904.108
• Web Server • web-server • web-javascript • Flutter Tools
! Doctor found issues in 1 category.
flutter build ios --verbose
[  +21 ms] executing: pod --version
[ +910 ms] 1.8.4
[   +4 ms] Running pod install...
[ +703 ms] Running pod install... (completed in 0.7s)
[   +1 ms] CocoaPods' output:
↳
[        ]       Preparing
           Analyzing dependencies

           Inspecting targets to integrate
             Using `ARCHS` setting to build architectures of target `Pods-Runner`: (``)

           Fetching external sources
           -> Fetching podspec for `Flutter` from `Flutter`
           -> Fetching podspec for `google_sign_in` from `.symlinks/plugins/google_sign_in/ios`
           -> Fetching podspec for `google_sign_in_web` from `.symlinks/plugins/google_sign_in_web/ios`
           [!] No podspec found for `google_sign_in_web` in `.symlinks/plugins/google_sign_in_web/ios`

           /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/lib/cocoapods/external_sources/path_source.rb:14:in `block in fetch'
           /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/lib/cocoapods/user_interface.rb:64:in `section'
           /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/lib/cocoapods/external_sources/path_source.rb:11:in `fetch'
           /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/lib/cocoapods/installer/analyzer.rb:901:in `fetch_external_source'
           /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/lib/cocoapods/installer/analyzer.rb:880:in `block (2 levels) in fetch_external_sources'
           /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/lib/cocoapods/installer/analyzer.rb:879:in `each'
           /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/lib/cocoapods/installer/analyzer.rb:879:in `block in fetch_external_sources'
           /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/lib/cocoapods/user_interface.rb:64:in `section'
           /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/lib/cocoapods/installer/analyzer.rb:878:in `fetch_external_sources'
           /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/lib/cocoapods/installer/analyzer.rb:117:in `analyze'
           /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/lib/cocoapods/installer.rb:410:in `analyze'
           /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/lib/cocoapods/installer.rb:234:in `block in resolve_dependencies'
           /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/lib/cocoapods/user_interface.rb:64:in `section'
           /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/lib/cocoapods/installer.rb:233:in `resolve_dependencies'
           /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/lib/cocoapods/installer.rb:156:in `install!'
           /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/lib/cocoapods/command/install.rb:52:in `run'
           /Library/Ruby/Gems/2.3.0/gems/claide-1.0.3/lib/claide/command.rb:334:in `run'
           /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/lib/cocoapods/command.rb:52:in `run'
           /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/bin/pod:55:in `<top (required)>'
           /usr/local/bin/pod:22:in `load'
           /usr/local/bin/pod:22:in `<main>'

[   +8 ms] Error output from CocoaPods:
↳
[        ]
[!] Automatically assigning platform iOS with version 8.0 on target Runner because no platform was specified. Please specify a platform for this target in your Podfile. See
https://guides.cocoapods.org/syntax/podfile.html#platform.
[  +14 ms] "flutter ios" took 14,376ms.
Error running pod install
#0      throwToolExit (package:flutter_tools/src/base/common.dart:28:3)
#1      CocoaPods._runPodInstall (package:flutter_tools/src/macos/cocoapods.dart:326:7)

#2      CocoaPods.processPods (package:flutter_tools/src/macos/cocoapods.dart:145:15)

#3      processPodsIfNeeded (package:flutter_tools/src/macos/cocoapod_utils.dart:36:46)

#4      buildXcodeProject (package:flutter_tools/src/ios/mac.dart:357:9)

#5      BuildIOSCommand.runCommand (package:flutter_tools/src/commands/build_ios.dart:80:43)

#6      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:615:18)
#7      _asyncThenWrapperHelper. (dart:async-patch/async_patch.dart:73:64)
#8      _rootRunUnary (dart:async/zone.dart:1134:38)
#9      _CustomZone.runUnary (dart:async/zone.dart:1031:19)
#10     _FutureListener.handleValue (dart:async/future_impl.dart:139:18)
#11     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:680:45)
#12     Future._propagateToListeners (dart:async/future_impl.dart:709:32)
#13     Future._completeWithValue (dart:async/future_impl.dart:524:5)
#14     Future._asyncComplete. (dart:async/future_impl.dart:554:7)
#15     _rootRun (dart:async/zone.dart:1126:13)
#16     _CustomZone.run (dart:async/zone.dart:1023:19)
#17     _CustomZone.runGuarded (dart:async/zone.dart:925:7)
#18     _CustomZone.bindCallbackGuarded. (dart:async/zone.dart:965:23)
#19     _microtaskLoop (dart:async/schedule_microtask.dart:43:21)
#20     _startMicrotaskLoop (dart:async/schedule_microtask.dart:52:5)
#21     _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:118:13)
#22     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:175:5)

thank you for providing those information @arctouch-felipefontes

I was trying to add labels, but this is my first report in an issue. i don't know if i don't have permissions or I just don't know how to do it.
Anyway thanks for helping me. let me know if you need anything.

Hey @ditman and @mklim can you guys help me?. i downloaded the google_sign_in example and did this
1- flutter config --enable-web
2- added the plugin google_sign_in_web
2- flutter create .
3- flutter build web (build ok)
4- flutter build ios and happens the same as above.
[!] No podspec found for `google_sign_in_web` in `.symlinks/plugins/google_sign_in_web/ios`
Building io.flutter.plugins.googleSignInExample for device (ios-release)...
Warning: Missing build name (CFBundleShortVersionString).
Warning: Missing build number (CFBundleVersion).
Action Required: You must set a build name and number in the pubspec.yaml file version field before submitting to the App Store.
Signing iOS app for device deployment using developer identity: "iPhone Developer: Felipe Santos (BNDJ94Z7BP)"
Running pod install...                                              0.8s
CocoaPods' output:
↳
      Preparing

    Analyzing dependencies

    Inspecting targets to integrate
      Using `ARCHS` setting to build architectures of target `Pods-Runner`: (``)

    Finding Podfile changes
      A google_sign_in_web
      - Flutter
      - google_sign_in

    Fetching external sources
    -> Fetching podspec for `Flutter` from `Flutter`
    -> Fetching podspec for `google_sign_in` from `.symlinks/plugins/google_sign_in/ios`
    -> Fetching podspec for `google_sign_in_web` from `.symlinks/plugins/google_sign_in_web/ios`
    [!] No podspec found for `google_sign_in_web` in `.symlinks/plugins/google_sign_in_web/ios`

    /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/lib/cocoapods/external_sources/path_source.rb:14:in `block in fetch'
    /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/lib/cocoapods/user_interface.rb:64:in `section'
    /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/lib/cocoapods/external_sources/path_source.rb:11:in `fetch'
    /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/lib/cocoapods/installer/analyzer.rb:901:in `fetch_external_source'
    /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/lib/cocoapods/installer/analyzer.rb:880:in `block (2 levels) in fetch_external_sources'
    /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/lib/cocoapods/installer/analyzer.rb:879:in `each'
    /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/lib/cocoapods/installer/analyzer.rb:879:in `block in fetch_external_sources'
    /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/lib/cocoapods/user_interface.rb:64:in `section'
    /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/lib/cocoapods/installer/analyzer.rb:878:in `fetch_external_sources'
    /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/lib/cocoapods/installer/analyzer.rb:117:in `analyze'
    /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/lib/cocoapods/installer.rb:410:in `analyze'
    /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/lib/cocoapods/installer.rb:234:in `block in resolve_dependencies'
    /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/lib/cocoapods/user_interface.rb:64:in `section'
    /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/lib/cocoapods/installer.rb:233:in `resolve_dependencies'
    /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/lib/cocoapods/installer.rb:156:in `install!'
    /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/lib/cocoapods/command/install.rb:52:in `run'
    /Library/Ruby/Gems/2.3.0/gems/claide-1.0.3/lib/claide/command.rb:334:in `run'
    /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/lib/cocoapods/command.rb:52:in `run'
    /Library/Ruby/Gems/2.3.0/gems/cocoapods-1.8.4/bin/pod:55:in `<top (required)>'
    /usr/local/bin/pod:22:in `load'
    /usr/local/bin/pod:22:in `<main>'

Error output from CocoaPods:
↳

    [!] Automatically assigning platform `iOS` with version `8.0` on target `Runner` because no platform was specified. Please specify a platform for this target in your Podfile. See
    `https://guides.cocoapods.org/syntax/podfile.html#platform`.

Error running pod install```

Hey @arctouch-felipefontes, I've just seen this notification!
Thanks for the No podspec found for google_sign_in_web in .symlinks/plugins/google_sign_in_web/ios error message, I'll take a look!

I'm looking at our other web plugin, and indeed it contains a podspec file. I'll add this to the google_sign_in plugin to try to unblock this ASAP! (Currently I'm not setup to build iOS :/)

This got auto-closed when I landed the patch, sorry!
@arctouch-felipefontes please let me know if the above helps!

It' worked thanks man. saved my life.


Minor update to french translations

Good evening and thanks for the reminder. Here is a minor contribution with update/addition to the french translation.

Coverage remained the same at 58.511% when pulling 7f8b43b48de58abbfc30b04fb37f321f3d251070 on tkerdonc:french_translations_2020 into 80fd835915c84d29e61d85c88e3af5a474572e01 on newsboat:master.

Thanks a lot for the der-lyse and Minoru, it would indeed seem that I shot myself in the foot by not editing with poedit, lesson learnt. My apologies for things that I should have seen while re reading (I was indeed a bit trigger happy with c formats ^^)
This new commit should address your comments, let me know if there is more.
I was unable to add myself to the translator team using poedit though, (version too old maybe ?) so I made the edit by hand using the syntax from uk.po.

Thanks for the remarks, tried to address them in this new version. Went the final file and also offered translation for the remaining entries.

Thanks a lot for the help and accurate reviews, I'll try to keep a closer eye on this po in the future in order to make updates more straightforward.


[DI] Add python customizations

Include customizations on:

Rename property "Items" in model "DocumentFieldSchema" to "ItemsSchema".
Rename parameter "AuthorizeCopyRequest" in operation "authorizeModelCopy" to "body".

API change check
APIView has identified API level changes in this PR and created following API reviews.
Microsoft.Authorization


get_tree().set_debug_collisions_hint(true) doesn't work in tool mode

When using the get_tree().set_debug_collisions_hint(true) function in tool mode, nothing happens.
Example:
tool
export (bool) var Enabled = false setget initialize
func initialize(value):
	Enabled = value
	if get_tree().is_editor_hint():
		if value == true:
			get_tree().set_debug_collisions_hint(true)

Enable the exposed variable from above in the editor,
then check debug options in the editor - the collision hint is not enabled

Closing due to lack of response (see Akien's comment above). Please comment if you can still reproduce this issue.


Test against auth0 vulnerabilities

Just saw this: https://auth0.com/blog/2015/03/31/critical-vulnerabilities-in-json-web-token-libraries/ .
You might want to test against those cases just to be sure.

I've submitted a Pull Request to the node-jwt-simple repo, although it doesn't seem that active. I might publish my fork of the project or switch libraries, keep you posted on this issue

Alright, great.
Btw passport-local-sequelize is going to get a nice PR merged soon. That will allow me to finish that Sequelize demo for DarkLord.


Speeding up training stage

I am new to gaussian splatting and taichi as well. Is there quick way to speed up training process without loosing quality. Is it possible to do batch processing?
P.S. The training process takes more than 1 hour for 20k iterations on Tesla T4.

I think so. It's in previous TODO list. But I don't have that much time to implement it... It shall not be difficult, just need to add a new dim in sorting key(tile uv to duv), and also add the dim in rasterization/backward. As we only use less than 4GB GPU memory now, I think it shall help.

@wanmeihuali do you still have plans for batch processing?

@Alexma3312 @Alexma3312 @yanzhoupan Do you have time to take a look?

Will start working on batch training.


Support switching to binary protocol with HTTP upgrade

The server now follows HTTP/1.1 upgrade protocol to promote a single
frontend connection from HTTP-based protocol to EdgeDB binary protocol.
The protocol is named edgedb-binary and it has to be exactly the same
in the Upgrade header, e.g. Upgrade: edgedb-binary.
Previous mechanism to detect if a new connection is HTTP or binary by
looking at the first byte still remains, so old clients are still
supported. But new clients are suggested to use HTTP upgrade on
connect in order to switch to the binary protocol (or remain in HTTP).

Closed for #2610.


InputTextIntl: initialValue should be localized

I have encountered this as well when working with complex components like Table and Modal, we should find a way to localize several props at the same time.
For this reason I think we should consider to rewrite the withIntl HOC component and rethink its logic.
I won't say that it has a high priority because host projects can localize values and set them into Mosaic components, but definitively it has some priority.
Since we are starting to collect different issues on withIntl HOC, I'm also creating a issue that collects all related issues, because the idea would be to solve them all in one PR.

This can be closed with release of v1.1.x
It is not possible to localize automatically the value of a Input, the reason is that it is not a static text (as labels or others).
Value of the input is subject to changes over time, and this would require to have potentially infinite translations available.
What the user can do is to pass-in as initialValue something that is already localized on utilizing project.


Firefox OIT fix

Fixes https://github.com/AnalyticalGraphicsInc/cesium/issues/4658
There seems to be an issue where a texture that is not initialized with a source and is bound to an FBO color attachment (besides the 0th) cannot be rendered to correctly.
So instead of passing null to pixels I pass in an empty array buffer.
I consider this just a workaround until the actual bug is fixed. I have a small test case that is displaying the same issue so I'm going to file a Firefox bug.

Very nice, thanks @lilleyse!

@lilleyse is there an issue that links to the Firefox bug for this so we can remove the workaround when the browser is fixed?

No there isn't, thanks for the reminder.


uvx doesn't upgrade when existing tool is installed

This is a confusing workflow:
❯ uv tool install ruff==0.0.60
Resolved 1 package in 5ms
Installed 1 package in 3ms
 + ruff==0.0.60
Installed 1 executable: ruff

❯ uvx ruff --version
ruff 0.0.60

❯ uvx --upgrade ruff --version
ruff 0.0.60

Not 100% sure what the semantics should be here, it's actually pretty tricky. Even if we respected --upgrade here properly, we wouldn't upgrade the installed tool past 0.0.60, since the receipt has ruff==0.0.60 in it. So, like, even if you did uvx --upgrade ruff and we gave you latest Ruff, running uvx ruff again would... give you 0.0.60.
I wonder if we should avoid using installed tools ever in uvx now that we have cached environments.

\cc @zanieb

I think using the installed tool wasn't intended as a performance thing, rather it's part of the experience to use the version you've chosen to install by default. This becomes more important when we consider project-level tools where uvx would invoke the project-specific version by default. We could reconsider that experience.
The --isolated flag was intended to ignore the installed version. Is there an equivalent for that now?
A --latest flag was discussed in the design, should we add that?
I don't think --upgrade is even intended to be available in uvx since it's an ephemeral environment? If resolver options are passed, we should probably just ignore the receipt entirely?

Yeah --isolated already exists.

I guess we can ignore the installed tool if you pass --upgrade or --reinstall?

@zanieb -- I might expect ruff@latest to work


split examples into separate algorithms https://github.com/marrlab/DomainLab/blob/master/scripts/ci_run_examples.sh

https://github.com/marrlab/DomainLab/blob/master/scripts/ci_run_examples.sh

files=("file1.txt" "file2.txt" "file3.txt")
for file in "${files[@]}"
do
echo "Processing $file"
# Add your commands to process each file here
done

Name
Latest commit
smilesun
smilesun
Merge pull request https://github.com/marrlab/DomainLab/pull/876 from marrlab/doc_example
588fff9
·
2 minutes ago
History


no schemes handler support on osx

Hyper does not open on typical terminal handler schemes, as default Terminal.app or iTerm does.
problem:
with Docker-Kitematic apps starting an ssh session. As well as about any other tool starting a commandline (also the bash and shell scheme).
from the ref (but leaving out the bash:// and shell:// scheme):
These would de ssh, telnet and x-man-page URL schemes.
For completeness the file endings .term (sessions), .terminal (settings), .tool and .command (scripts).
reference
implementation in osx apps and info: https://developer.apple.com/documentation/uikit/core_app/allowing_apps_and_websites_to_link_to_your_content/defining_a_custom_url_scheme_for_your_app
required handlers:
https://superuser.com/a/379354

Was it added?

We register for the ssh scheme now, if that is not enough, please make a new issue :)

.. great. bash and shell are also defaults ... creating the same issue again seems strange to me.


Infinite loop

The problem is reproduced on Vite.
Create base fetcher and store.
I see it in the console:
10297 deduped [my key]
5883 already runs [my key]

Example on StackBlitz

@artem-shkurenko I just ran this with 0.0.4 and it ran.

There seems to be a problem connected to Vite in Stackblitz itself, not nanoquery. Here, same example, but on codesandbox.
The link works (and worked) good on my machine (MBP on Apple Silicon), but it doesn't (and didn't) work on my colleagues' machines running on Linux/Windows. It does work locally everywhere though, whatever processor arch/OS you have.

I filed an issue in SB's repo, so far no answer.
I'll close this as it isn't really nanoquery-specific and I can't do anything with it at all.

I also have a problem with local development.
Mac: Intel Core i7
Mac OS: 13.0.1 (22A400)
Vite: 4.1.1


Tekton Upgrade

Hi,
I want to upgrade my current tekton pipeline? Could anyone please let me know whether I need to uninstall the existing tekton and install the latest version? Or, can I simply run 'kubectl apply <latest-version-install.yaml> file?
Want to make sure as need to upgrade tekton. Thanks!

As long as you are upgrading from a 0.11+ version you should only need to apply the latest version. Since the beta release we are being very careful about breaking changes but before that things could be a bit wild. Also always take a look at the release notes to see what has changed in a release and if there are any deprecation warnings: https://github.com/tektoncd/pipeline/releases
We also have this open issue to provide better guidance on how to upgrade: https://github.com/tektoncd/pipeline/issues/1222

This is also one of the goal of tektoncd/operator, to embedding any knowledge required for upgrading (like migration of CRD version from one version to another, etc…)
/cc @sthaha @nikhil-thomas

@bobcatfish - Thanks !
@vdemeester  - Thanks! Will look at tektoncd/operator as I end up manually deleting the old version pods/deployments. operator would be useful.


Touch event on a link issue

It's in AngularJS 1.2.3.
This is only happening on touch devices or in Google Chrome with touch event emulation enabled.
This link works (redirects to the url):
search
And this link doesn't (no redirect occurs on tap):
search
Click handler invoked and after that something strange happens inside of
element.on('touchend', function(event) (angular-touch.js:385).
Somehow it decides to prevent event by calling preventGhostClick(x, y);
It's really sad because it blocked our big mobile app to use AngularJS 1.2.3 in production. We're still on 1.0.8.

+1 this bug still exist....

default behaviour of link is broken, +1 to fix this bug

+1

+1
Another workaround is to add the following into your ng-click handler.
//
$scope.hideMenu = function (event) {
    // ...
    var href = angular.element(event.target).attr('href');
    if (event.type === 'touchend' && href) {
        href = href.substring(1); // remove initial #
        $location.path(href);
    }
};

It'd be nice if we can get a response on wether this is a bug or a feature...I think it's a bug, but I'm new to Angular...
Is it recommended to not use ngHref when also using ngClick? If it is, I'll deal with it I guess, otherwise, it'd be nice to have this confirmed as a bug, and we can expect a bug fix sometime in the future.

+1

+1 Same bug here.

+1

+1

+1

+1

+1

Please stop with the :+1:ing. It's not adding anything to the discussion of the actual issue and only emails people hwo are watching this for actual progress unnecessarily.

Better, is to wait on Angular 2 and hope that doesn't have this bug. Just unsubscribe because there will probably never be any progress on this bug in version 1.

There seems to be some code that ensures the tap did not "end" too far away from origin, called clickbuster.
This may be related to the CLICKBUSTER_THRESHOLD, which is set to 25px (too small for chubby thumbs?).

still exists, I think

The touch override from ngTouch has been disabled by default since 1.5.0, so I doubt this issue still exists in this version. It may be an issue with 3rd party touch handlers. There's also a point to be made that having a click handler on a link is bad for accessibility.

Dne 27. 10. 2016 8:12 napsal uživatel "David-Tsui"<EMAIL_ADDRESS>
:
still exists, I think
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I am going to lock this issue, as I don't think we can lead a constructive discussion here. Please open a new issue with a clear description and a runnable example if you have problems with this in the current Angular version (1.5.8).


Update cgf.13182

postprint for Bögl et al., CyclePlot

FYI I am going to update the icon on the supplemental material from paper to other. This is more consistent with the existing papers that link to supplemental material.


Lucide-Icons out of date

Current Behavior
The @tamagui/lucide-icons is out of date and needs an update. Not all icons are exported and many are missing for example

ellipsis-vertical

Expected Behavior
should have all icons and can be imported
Tamagui Version
1.100.0

Platform (Web, iOS, Android)
web, ios, android

Reproduction
try to import in your project EllipsisVertical

works from original import { EllipsisVertical } from 'lucide-react';

but not from Tamagui
import { EllipsisVertical } from '@tamagui/lucide-icons'

System Info
No response

This seems like a good first issue, something I could look into as a new contributor?

Ellipsis was the very first icon I tried to import. Frustrated that it's not included. Could you please support all commonly used Lucide Icons?

Please someone do send a PR, not very hard to do.


Where do you list the dependencies needed for it to compile?

I am trying to compile it for more than half an hour, and every time i have an error that a package is missing.
Why didnt you include a list of dependencies.
Also, now i am stuck on this one:
 --- stderr
  `"pkg-config" "--libs" "--cflags" "gdk-3.0" "gdk-3.0 >= 3.22"` did not exit successfully: exit status: 1
  --- stderr
  Package gdk-3.0 was not found in the pkg-config search path.
  Perhaps you should add the directory containing `gdk-3.0.pc'
  to the PKG_CONFIG_PATH environment variable
  No package 'gdk-3.0' found
  Package gdk-3.0 was not found in the pkg-config search path.
  Perhaps you should add the directory containing `gdk-3.0.pc'
  to the PKG_CONFIG_PATH environment variable
  No package 'gdk-3.0' found

And i simply can't find ANY package named gdk-3.0 or similar. I am on Void Linux and there is literally nothing about it on the repos. What should i do?

Also check for gtk-packages (with a "t").
gdk and gtk are both part of the Gnome GUI toolkit and on my Debian both libgtk and libgdk are packed in the libgtk package.
It may be similar in your distribution.

Hi @hrqmonteiro
I'm on void too and I solved the gdk3.0 not found error by installing this package
gtk+3-devel
But then I got another issue
could not compile simplexpr
Checked why the error was coming, and it said something like the feature can only be used in nightly version. Went to eww docs and found that they recommend nightly version
Rather than with your system package manager, I recommend installing it using rustup, as this makes it easy to use the nightly toolchain necessary to build eww.
So I uninstalled rustup, cargo and rust (which I had downloaded from void packages repo)
And then I installed rustup from here
https://rustup.rs/
And I chose advanced installation/custom installation(can't remember exactly), and specified that I wanted the nightly version
and after rustup was downloaded, I did cargo build --release and this time the build was successful!


Reuse the friend group to associate new MeaningChallenges

Same as we do with AzkarChallenges. If the challenge is between only to friends, so associate with their group instead of creating a new group, so as to decrease the number of groups created.

Hasn't proved to be super beneficial yet, I would actually suggest the opposite for readability.


Rename kind (used as alias for type)

Describe the feature you'd like
I find kind as alias for type, which conflicts with Rust's type keyword, confusing
and propose to change it to type
and to use type_ as alias where type conflicts with the keyword.
I'm of course open for other suggestions how to prevent the conflict,
it's just that kind seems too different to make a good candidate.
Is your feature request related to a problem? Please describe. (Optional)
See the discussion in
https://github.com/yewstack/yew/pull/1298
It's not obvious that kind stands for type and it would need to be well documented, probably on all locations where it appears in public API.
In private APIs it might still confuse contributors.
Describe alternatives you've considered (Optional)
Improve documentation
https://github.com/yewstack/yew/pull/1298#issuecomment-643712228
Additional context (Optional)
Add any other context or screenshots about the feature request here.
Questionnaire

[x] I'm interested in implementing this myself but don't know where to start
[ ] I would like to add this feature
[ ] I don't have time to add this right now, but maybe later

I'd try to fix this after a consensus is found what should be used instead of type where it causes a syntax error because of the conflict with the type keyword.

@zoechi how about input_type?

Discoverability is probably not great if it does not start with type

@zoechi I don't think discoverability is an issue, can you elaborate which API you are concerned about?

Right, we could still use VTag::set_type() and have the struct field named input_type.
#1322 should make this field obsolete anyway.

True, what do you think about pursuing #1322 instead?

@jstarry see https://github.com/yewstack/yew/issues/1322#issuecomment-647017110
I need to improve my Rust-Fu to better understand Yew code first, but otherwise I'm interested.

I'm closing this issue because this has been fixed :) I don't know when but kind as an alias for type is not used by Yew anymore 🎉


Use a canvas pattern in the vector-layer example

Update the example to demonstrate how to use a canvas pattern as a fill color.
based on http://jsfiddle.net/61b7szjn/

+1

Thanks for the review

Sorry for commenting post-merge. This would have been better suited for a separate, new example.


Add tslint.json

Codecov Report

Merging #7 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master       #7   +/-   ##
=======================================
  Coverage   97.26%   97.26%
=======================================
  Files           1        1
  Lines          73       73
  Branches       16       16
=======================================
  Hits           71       71
  Misses          2        2

Impacted Files
Coverage Δ

src/DebounceLink.ts
97.26% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


toHaveNoViolations not detecting accessible name using label htmlFor

Hello!
I have this structure here:

    date picker label

And tried to run this axe test for a11y:
    const { container } = render(<DatePickerMC label="date picker label" />);
    const results = await axe(container);
    expect(results).toHaveNoViolations();

However, the test is failing because DatePickerMc supposedly doesn't have an accessible name. But it does.
    expect(received).toHaveNoViolations(expected)

    Expected the HTML found at $('#Datetime1-label') to have no violations:

    Received:

    "ARIA input fields must have an accessible name (aria-input-field-name)"

    Fix any of the following:
      aria-label attribute does not exist or is empty
      aria-labelledby attribute does not exist, references elements that do not exist or references elements that are empty
      Element has no title attribute

It looks like jest-axe is not detecting the label provided with . Is this a bug or am I doing something wrong?

Just published 7.0.0 please try your issue in 7.0.0 and re-open if it's an issue, thanks a lot.


[Table] Introduce padding prop and deprecate previous props

Introduce TableCell padding property.
Deprecate checkbox, compact, disabledPadding and add warnings.
Introduces dependency on prop-types-extra (MIT License)
Remove unused code

I originally wanted to minimise changes but have also renamed compact -> dense across the class. Removed deprecations etc. Do you want api doc updates as part of PRs or do you do them seperately?

@eyn Thanks


Change graph rendering parameters to optimize it

Done.
We decided to replace Vis javascript library with Cytoscape. The new library performed better when tested with large schemas.


Webhooks for New Accounts

Adds webhooks on new account creation request. Currently the server must be polled for new account creation requests. Auto-accepting all account creation requests still not possible (yet!).

Rebased after the ZMQ-PUB merge. New accounts creation requests are now notified over optional ZMQ PUB/SUB socket.


How to split chair legs based on connectivity

Hi thanks for the great work. I am wondering how you split chair legs based on connectivity? Is it possible for you to provide code for this? Thanks

If you are working on voxels, you can use the function skimage.morphology.lable to find connected components, hence splitting chair legs.

thanks for the help, I also noticed from mesh-voxelization that the first step is to scale the raw meshes to lie in [0, H] x [0, W] x [0, D] corresponding to the chosen resolution H x D x W, will this affect the calculation of bounding box of each part in rescale_part_vox.py?

The the calculation of bounding box of each part in rescale_part_vox.py is independent. The function find_bounding_box computes axis-aligned bounding box on any voxel input.
I don't see any difficulties about computing bounding box parameters on meshes or voxels. You simply need to find the max and the min points. Translations and size can be in the range of (0, 1), if you are transforming original meshes.

If you are working on voxels, you can use the function skimage.morphology.lable to find connected components, hence splitting chair legs.

May I ask if there are any tools that can implement the split on mesh instead of voxels？

@lyxhope You can try this trimesh api: https://trimsh.org/trimesh.html#trimesh.Trimesh.split.

@lyxhope You can try this trimesh api: https://trimsh.org/trimesh.html#trimesh.Trimesh.split.

Thanks for your help. I have a further question. In your paper, you said that your split based on the second layer of partnet. So did you specifically split the chair legs based on connectivity? Or you split all parts based on connectivity?

In your paper, you said that your split based on the second layer of partnet. So did you specifically split the chair legs based on connectivity? Or you split all parts based on connectivity?

We specifically split the chair legs based on connectivity. Other parts are not further split.

In your paper, you said that your split based on the second layer of partnet. So did you specifically split the chair legs based on connectivity? Or you split all parts based on connectivity?

We specifically split the chair legs based on connectivity. Other parts are not further split.

Thanks a lot！

thanks for the help, I also noticed from mesh-voxelization that the first step is to scale the raw meshes to lie in [0, H] x [0, W] x [0, D] corresponding to the chosen resolution H x D x W, will this affect the calculation of bounding box of each part in rescale_part_vox.py?
I want to resample the part from PartNet dataset but if I directly voxelize parts of an object, it returns all-zero voxels. If I translate and scale the parts then how should I compute the box parameters? Do the translations and size need to be in the range of (0,64), can they be in the range of (0,1) for custom dataset?

Hi @qiminchen,
Do you recall how did you resolve the all zero-voxels issue? Did you use the code provided in mesh-voxelization to scale the mesh first? I was using the scale_off.py without fill_occupancy.py but there are gaps in the final outputted part.
Thank you


feat(connector-polkadot): polkadot connector

This PR adds a long-term effort to adding preliminary support to Substrate-based chains. Most work wonderfully done by @CatarinaPedreira
"The Polkadot Connector provides functionality that enables any permissioned blockchain (as long as it is supported by Hyperledger Cactus) to connect to the Polkadot network and perform monetary transactions to the latter. Besides this, the connector provides methods for these blockchains to deploy and interact with smart contracts in the network.
Our connector in Hyperledger Cactus uses three different API’s to communicate with the Polkadot network: ”@polkadot/api” (which has the base API functionality), ”@polkadot/api-contracts” (which contains the API specific for smart contract interaction) and ”@polkadot/types” (which encompasses specific
Polkadot types)."
Features:
-transact on the network
-read smart contracts
-deploy smart contracts
-read from smart contracts
-endpoints for the features
Future work/limitations

The integration tests are commented/skipped because we need to add the source code of the smart contracts (metadata, etc) to the repository, such that they can be executed.
Some of the features could be more extensible/flexible, but again, this is the first version of the connector
Some endpoints are not fully implemented

Thanks to @petermetz for helping develop the test ledger.
Co-authored-by: CatarinaPedreira<EMAIL_ADDRESS>Signed-off-by: Rafael Belchior<EMAIL_ADDRESS>

@jagpreetsinghsasan @AnmolBansalDEV I'm closing this for now just to keep our backlog of PRs tidy(er) but please re-open as soon as it is ready for review with the new changes incorporated from the LF mentorship program!
cc: @RafaelAPB


Added python type annotation.

Trac ticket number

ticket-XXXXX
Branch description
Provide a concise overview of the issue or rationale behind the proposed changes.

python type annotation added, might be useful for documentation.

Checklist

[ ] This PR targets the main branch.
[ ] The commit message is written in past tense, mentions the ticket number, and ends with a period.
[ ] I have checked the "Has patch" ticket flag in the Trac system.
[ ] I have added or updated relevant tests.
[ ] I have added or updated relevant docs, including release notes if applicable.
[ ] I have attached screenshots in both light and dark modes for any UI changes.

We currently don't accept type annotations in Django. Please see django-stubs instead.


Bug report - v2.6.0, Configure Fragment Page, cannot slide to GATT CONFIGURATOR page, the App crashed.

as the title.

This is reported for R&D

Hi @RadioOperator, the issue has been fixed in 2.6.1. You can expect the release on Wednesday Feb 01.

Thanks


ProcessPoolExecutor error in writting features

An error occurred when I ran the following codes (egs/librispeech/asr/simple_v1/prepare.py:45) on Xiaomi's server (the one with the IP address 10.xx.x.72):
with LilcomFilesWriter(f'{output_dir}/feats_{partition}') as storage, ProcessPoolExecutor(num_jobs, mp_context=multiprocessing.get_context("spawn")) as ex:
        cut_set = CutSet.from_manifests(
            recordings=manifests['recordings'],
            supervisions=manifests['supervisions'])
        cut_set = cut_set.compute_and_store_features(
            extractor=Fbank(),
            storage=storage,
            executor=ex)

The error message:
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/usr/lib/python3.8/multiprocessing/spawn.py", line 116, in spawn_main
    exitcode = _main(fd, parent_sentinel)
  File "/usr/lib/python3.8/multiprocessing/spawn.py", line 125, in _main
    prepare(preparation_data)
  File "/usr/lib/python3.8/multiprocessing/spawn.py", line 236, in prepare
    _fixup_main_from_path(data['init_main_from_path'])
  ...
  File "/usr/lib/python3.8/multiprocessing/spawn.py", line 154, in get_preparation_data
    _check_not_importing_main()
  File "/usr/lib/python3.8/multiprocessing/spawn.py", line 134, in _check_not_importing_main
    raise RuntimeError('''
RuntimeError:
        An attempt has been made to start a new process before the
        current process has finished its bootstrapping phase.

        This probably means that you are not using fork to start your
        child processes and you have forgotten to use the proper idiom
        in the main module:

            if __name__ == '__main__':
                freeze_support()
                ...

        The "freeze_support()" line can be omitted if the program
        is not going to be frozen to produce an executable.

Traceback (most recent call last):
  File "prepare.py", line 45, in <module>
    cut_set = cut_set.compute_and_store_features(
  File "lhotse/lhotse/cut.py", line 1328, in compute_and_store_features
    cut_set = CutSet.from_cuts(f.result() for f in futures)
  File "lhotse/lhotse/cut.py", line 989, in from_cuts
    return CutSet({cut.id: cut for cut in cuts})
  File "lhotse/lhotse/cut.py", line 989, in <dictcomp>
    return CutSet({cut.id: cut for cut in cuts})
  File "lhotse/lhotse/cut.py", line 1328, in <genexpr>
    cut_set = CutSet.from_cuts(f.result() for f in futures)
  File "/usr/lib/python3.8/concurrent/futures/_base.py", line 439, in result
    return self.__get_result()
  File "/usr/lib/python3.8/concurrent/futures/_base.py", line 388, in __get_result
    raise self._exception
concurrent.futures.process.BrokenProcessPool: A process in the process pool was terminated abruptly while the future was running or pending.

I'm not have experience with ProcessPoolExecutor so I haven't found the reason of this error.

Try the solution that I suggested here https://github.com/lhotse-speech/lhotse/issues/126#issuecomment-727269748

Thanks @pzelasko , it worked. I should search the closed issues before adding a new one :flushed:


add to deck.gl showcase

Preferred solution
you should submit this project to the deck.gl showcase section !

The submission process is just to file an issue, like this one on vote suppression in 2016 or this from Propublica


Add dinamic icon for color scheme preference

I added a way to automatically change the icon when the user is using either dark mode or light mode. The reason is that when you are in the app with the dark mode activated, the icon has no contrast with the background and is difficult to visualize.
Before:

On the app:

Out the app:

Now:

On the app dark mode:

Out the app dark mode:

On the app light mode:

Out the app light mode:

The icons are dynamically changed based on the user's preference. The custom hook receives the URLs for both the light and dark mode icons, allowing for flexibility and customization.

WOW THIS IS BRUTAL!!! Thanks for this contribution!

I need the PR to not have conflicts for approval

I already fixed the conflicts 😄

This functionality can be moved out of React and converted from a hook to a normal function, there is a flash where it initially loads the default icon and then determines the icon according to the match, this because it depends on the execution of React
Also, since it doesn't need React directly, you can remove the useEffect and let main.tsx execute the function

@juanpablo-is  You are right, this shouldn't be a hook since i'm using react unnecesary, but the icon flash will always happen because the plain html use the "default icon" in the link href and then js changes it given the preference, I don't think that there is a way to fix this little flash of the icon.


Modify WInSeparator color

I would like to make the WinSeparator color a little lighter than c.bg0 . Is there any other way than rewriting this value directly in my local?
https://github.com/Tsuzat/NeoSolarized.nvim/blob/c28b7f9bdc48a9017480da41529ef63ab6ccb885/lua/NeoSolarized/theme.lua#L79

Sorry for creating a new issue even though it was discussed in a past issue.
https://github.com/Tsuzat/NeoSolarized.nvim/issues/5
I was able to achieve what I wanted to do with the following settings. Thank you.
require("NeoSolarized").setup({
  on_highlights = function(highlights, colors)
    highlights.WinSeparator.fg = "#abcdef"
  end,
})
vim.cmd([[colorscheme NeoSolarized]])

Not a problem. Happy Coding ❤


Javascript code run before Update Request

would be great to create validations to allow or not before Update/Delete/Create Request actions

@melquelima What kind of validations you are looking for?
We are not going to add custom JavaScript code like Business Forms in the Table panel to avoid security risks.

like, using current row match a few informations in a custom javascript code, before editing, if not i can show an error popup, things like that.

@melquelima Thank you for providing details, we will look into this.


ConversationSummaryMemory does not support for multi-languages.

https://github.com/hwchase17/langchain/blob/499e76b1996787f714a020917a58a4be0d2896ac/langchain/chains/conversation/prompt.py#L36-L59
Every time the openai call summary request is just added to the prompt head message, it will lead llm giving a response with the same language, and then, when you ask it do something again, it will make a mistake.

Hey FavorMylikes, did you find a solution to customizing the ConversationSummaryMemory summarization prompt? I'm looking forward to using summarization in my Brazilian Portuguese chatbot, and right now it seems I'd have to implement a new Memory class.


Cmd/Ctrl+A in Dropdown Input doesn't select text

Describe the bug
When typing in the dropdown input, I'm unable to use cmd+a or ctrl+a to select all text typed into the input.
To Reproduce

Go to https://tom-select.js.org/plugins/
Click on the Dropdown Input example
Start typing
With the dropdown input still focused, hit cmd+a or ctrl+a

Expected behavior
I would expect this input to behave just like a normal text input: cmd+a or ctrl+a should select the text in the input.
Additional context
Add any other context about the problem here.

OS: Windows
Browser: Chrome
Version 95.0.4638.69
Device: Desktop

Should be all set with the above commit

Thanks @oyejorge ! I've been loving this library.


Specifying withCopyFileToContainer should copy the files to all members before start

Is your feature request related to a problem? Please describe.
To configure SSL, the trusted.keystore needs to be available on the locator and all servers before they are started.
Describe the solution you'd like
Make sure than withCopyFileToContainer copies files before starting the member and that it copies to all containers, not just the locator as it does now.
Describe alternatives you've considered
No response
Additional context
No response

WIth the latest 2.0.0 version, files can be copied with:
withPreStart(ALL_GLOB, x -> x.copyFileToContainer(Transferable.of(keystoreBytes), "/keystore.jks"))


param-amplitude-factor key is not supported by the connected device at rs2_load_json

Camera Model: D415
Firmware Version: <IP_ADDRESS>
Operating System & Version: Windows 10 18963
SDK Version: 2.25
Language: C#
I was having an issue where I couldn't use any presets with my camera.  A new Windows 10 build came out on Friday and it now allows me to select a preset in the RealSense Viewer.  The new issue is I can't load up a preset in my own code.
I can load my json file in the realsense viewer, turn on stereo module and rgb camera and everything looks as expected.  I can then save the preset over the existing file.  When I try to
ctx = new Context();
var devices = ctx.QueryDevices();
if (devices.Count > 0)
{
Device dev = devices[0];
AdvancedDevice ad = AdvancedDevice.FromDevice(dev);
ad.JsonConfiguration = File.ReadAllText("GanderD415-4.json");
}
I get an error of "param-amplitude-factor key is not supported by the connected device at rs2_load_json".  If I edit the json and get rid of the param-amplitude-factor key, I get a "hr returned: HResult 0x8007001f: "A device attached to the system is not functioning at rs2_load_json".
If I return to RealSense Viewer, load up and resave out the json, it puts back the param-amplitude-factor key.
If I go to https://github.com/IntelRealSense/librealsense/wiki/D400-Series-Visual-Presets and attempt to load ShortRangePreset.json I get the same  I get a "hr returned: HResult 0x8007001f: "A device attached to the system is not functioning at rs2_load_json".  If I use ShortRangePreset.json in RealSense Viewer, it loads fine, if I overwrite the json from Viewer, I, of course, get the param-amplitude-factor error.
If I don't set JsonConfiguration, it loads fine(but of course is unusable as I don't have my settings)
Since I'm trying to do extreme close up scans, I'd really like Subpixel Linearity(depth linearity, disparity modulation, amplitude-factor, whatever you want to call it, it's painful to try and find something when it's named so many different ways depending on where you look).  At this point I'd like to at least be able to load any preset file.
Here's my json file(even though it shouldn't matter since it doesn't work with the files downloaded from https://github.com/IntelRealSense/librealsense/wiki/D400-Series-Visual-Presets )
{
"aux-param-autoexposure-setpoint": "200",
"aux-param-colorcorrection1": "-0.878906",
"aux-param-colorcorrection10": "-0.342773",
"aux-param-colorcorrection11": "-0.342773",
"aux-param-colorcorrection12": "0.0537109",
"aux-param-colorcorrection2": "1.49707",
"aux-param-colorcorrection3": "1.49707",
"aux-param-colorcorrection4": "-1.71777",
"aux-param-colorcorrection5": "-0.166992",
"aux-param-colorcorrection6": "-0.0888672",
"aux-param-colorcorrection7": "-0.0888672",
"aux-param-colorcorrection8": "0.344727",
"aux-param-colorcorrection9": "0.633789",
"aux-param-depthclampmax": "3000",
"aux-param-depthclampmin": "0",
"aux-param-disparityshift": "200",
"controls-autoexposure-auto": "True",
"controls-autoexposure-manual": "33000",
"controls-color-autoexposure-auto": "True",
"controls-color-autoexposure-manual": "156",
"controls-color-backlight-compensation": "0",
"controls-color-brightness": "0",
"controls-color-contrast": "50",
"controls-color-gain": "64",
"controls-color-gamma": "300",
"controls-color-hue": "0",
"controls-color-power-line-frequency": "3",
"controls-color-saturation": "64",
"controls-color-sharpness": "50",
"controls-color-white-balance-auto": "True",
"controls-color-white-balance-manual": "4600",
"controls-depth-gain": "16",
"controls-depth-white-balance-auto": "False",
"controls-laserpower": "360",
"controls-laserstate": "on",
"ignoreSAD": "0",
"param-amplitude-factor": "0.08",
"param-autoexposure-setpoint": "200",
"param-censusenablereg-udiameter": "9",
"param-censusenablereg-vdiameter": "9",
"param-censususize": "9",
"param-censusvsize": "9",
"param-depthclampmax": "3000",
"param-depthclampmin": "0",
"param-depthunits": "100",
"param-disableraucolor": "0",
"param-disablesadcolor": "0",
"param-disablesadnormalize": "0",
"param-disablesloleftcolor": "0",
"param-disableslorightcolor": "0",
"param-disparitymode": "0",
"param-disparityshift": "200",
"param-lambdaad": "800",
"param-lambdacensus": "26",
"param-leftrightthreshold": "10",
"param-maxscorethreshb": "2048",
"param-medianthreshold": "0",
"param-minscorethresha": "10",
"param-neighborthresh": "0",
"param-raumine": "0",
"param-rauminn": "0",
"param-rauminnssum": "0",
"param-raumins": "0",
"param-rauminw": "0",
"param-rauminwesum": "0",
"param-regioncolorthresholdb": "0.000978474",
"param-regioncolorthresholdg": "0.000978474",
"param-regioncolorthresholdr": "0.000978474",
"param-regionshrinku": "0",
"param-regionshrinkv": "0",
"param-robbinsmonrodecrement": "0",
"param-robbinsmonroincrement": "0",
"param-rsmdiffthreshold": "4",
"param-rsmrauslodiffthreshold": "1",
"param-rsmremovethreshold": "0",
"param-scanlineedgetaub": "72",
"param-scanlineedgetaug": "72",
"param-scanlineedgetaur": "72",
"param-scanlinep1": "60",
"param-scanlinep1onediscon": "105",
"param-scanlinep1twodiscon": "70",
"param-scanlinep2": "342",
"param-scanlinep2onediscon": "190",
"param-scanlinep2twodiscon": "130",
"param-secondpeakdelta": "0",
"param-texturecountthresh": "0",
"param-texturedifferencethresh": "0",
"param-usersm": "1",
"param-zunits": "100",
"stream-depth-format": "Z16",
"stream-fps": "30",
"stream-height": "720",
"stream-width": "1280"
}

@NXTwoThou hello,
I cannot immediately comment on the Insider's preview build that you're using, but I've tried the preset that you shared on a stable Win10/ Redstone3 (build 16299) using D415 with identical v<IP_ADDRESS> FW: The preset could be loaded, executed and exported properly both in USB2 and USB3 modes, including the A-factor flag.
Can you check whether running the code on Win10 Redstone 3/4 builds would mitigates this?

Testing with the realsense viewer unfortunately doesn't help as the realsense viewer(as of Friday) works again on this machine.  Using IntelRealsense.dll v<IP_ADDRESS> in a C# app crashes on JsonConfiguration=

@NXTwoThou, please clarify -  are you able to load this preset correctly using the realsense-viewer but not the C# wrapper on that specific PC ?
Or is it the case that the preset cannot be loaded with neither C# nor realsense-viewer ?

realsense-viewer works fine
C# wrapper does not
Saving preset from realsense-viewer generates "param-amplitude-factor key is not supported by the connected device at rs2_load_json" error in C# wrapper
Removing param-amplitude-factor key from json preset file allows C# wrapper to get a little further by giving a "hr returned: HResult 0x8007001f: "A device attached to the system is not functioning at rs2_load_json"

Unfortunately same issue with v2.26.0.

RealSense Viewer works fine without loading a preset
RealSense Viewer works fine loading my custom preset file.
Using the C# wrapper gives an error about an invalid parameter when trying to set JsonConfiguration.
If I remove the parameter from the json file, setting JsonConfiguration says "a device attached to the system is not functioning"
If I remove trying to set JsonConfiguration, it runs fine(only without the presets I require)

Here's some images since my original description might not have been understood clearly...

RealSense Viewer with "Default"(no) preset:

RealSense Viewer with "GanderD415-4.json" preset:(contains param-amplitude-factor)

My code first error

Detailed error view

System.ArgumentException
  HResult=0x80070057
  Message=param-amplitude-factor key is not supported by the connected device!
  Source=Intel.RealSense
  StackTrace:
   at Intel.RealSense.ErrorMarshaler.MarshalNativeToManaged(IntPtr pNativeData) in C:\jenkins_sys_rsbuild\workspace\LRS_windows_compile_pipeline\wrappers\csharp\Intel.RealSense\Helpers\ErrorMarshaler.cs:line 72
   at System.StubHelpers.MngdRefCustomMarshaler.ConvertContentsToManaged(IntPtr pMarshalState, Object& pManagedHome, IntPtr pNativeHome)
   at Intel.RealSense.NativeMethods.rs2_load_json(IntPtr dev, String json_content, UInt32 content_size, Object& error)
   at Intel.RealSense.AdvancedDevice.set_JsonConfiguration(String value) in C:\jenkins_sys_rsbuild\workspace\LRS_windows_compile_pipeline\wrappers\csharp\Intel.RealSense\Devices\AdvancedDevice.cs:line 84
Inner Exception 1:
ExternalException: rs2_load_json(dev:000002851D9D99F0, json_content:000002851DABAEC0, content_size:3615)

My code after removing param-amplitude-factor from json file

Detailed error view

System.Exception
  HResult=0x80131500
  Message=hr returned: HResult 0x8007001f: "A device attached to the system is not functioning."
  Source=Intel.RealSense
  StackTrace:
   at Intel.RealSense.ErrorMarshaler.MarshalNativeToManaged(IntPtr pNativeData) in C:\jenkins_sys_rsbuild\workspace\LRS_windows_compile_pipeline\wrappers\csharp\Intel.RealSense\Helpers\ErrorMarshaler.cs:line 78
   at System.StubHelpers.MngdRefCustomMarshaler.ConvertContentsToManaged(IntPtr pMarshalState, Object& pManagedHome, IntPtr pNativeHome)
   at Intel.RealSense.NativeMethods.rs2_load_json(IntPtr dev, String json_content, UInt32 content_size, Object& error)
   at Intel.RealSense.AdvancedDevice.set_JsonConfiguration(String value) in C:\jenkins_sys_rsbuild\workspace\LRS_windows_compile_pipeline\wrappers\csharp\Intel.RealSense\Devices\AdvancedDevice.cs:line 84
   at Gander.RealSense.RealSense_Load(Object sender, EventArgs e) in Inner Exception 1:
ExternalException: rs2_load_json(dev:000001D37DD269E0, json_content:000001D37DDCE5B0, content_size:3576)

My code after commending out JsonConfiguration line

Same error with v2.27.0

Same issue with build 18970

Same issue with 2.28.0

Hit a pausing point in my primary project so I could take a real look at this.  I went ahead and downloaded the latest source so I could try and debug what was going on.
When I first created my project I had added a reference to C:\Program Files (x86)\Intel RealSense SDK 2.0\bin\x64\Intel.Realsense.dll to my project.  When I compile, it of course copies the latest Intel.Realsense.dll to my folder.  When I first tried to run my project, I would have gotten an error that realsense2.dll isn't found.   I then copied realsense2.dll from that folder to my output folder.  Looks like I started this back in April as that was the file date of realsense2.dll
Visual Studio will copy the most recent Intel.Realsense.dll on every compile to my output folder.  Unfortunately it doesn't check it's dependencies.  I've gone ahead and added it to my solution as a build action of "Content" and Copy to Output Directory of "Copy if newer".     I suggest adding this to the example projects.


[SPARK-34794][SQL] Fix nested transform issue

What changes were proposed in this pull request?

Increment a global counter and use the current value to name LambdaVariables created by higher order functions.
Why are the changes needed?

This moves away from the current hard-coded variable names which break on nested function calls. There is currently a bug where nested transforms in particular fail (the inner variable shadows the outer variable)
For this query:
val df = Seq(
    (Seq(1,2,3), Seq("a", "b", "c"))
).toDF("numbers", "letters")

df.select(
    f.flatten(
        f.transform(
            $"numbers",
            (number: Column) => { f.transform(
                $"letters",
                (letter: Column) => { f.struct(
                    number.as("number"),
                    letter.as("letter")
                ) }
            ) }
        )
    ).as("zipped")
).show(10, false)

This is the current (incorrect) output:
+------------------------------------------------------------------------+
|zipped                                                                  |
+------------------------------------------------------------------------+
|[{a, a}, {b, b}, {c, c}, {a, a}, {b, b}, {c, c}, {a, a}, {b, b}, {c, c}]|
+------------------------------------------------------------------------+

And this is the correct output after fix:
+------------------------------------------------------------------------+
|zipped                                                                  |
+------------------------------------------------------------------------+
|[{1, a}, {1, b}, {1, c}, {2, a}, {2, b}, {2, c}, {3, a}, {3, b}, {3, c}]|
+------------------------------------------------------------------------+

Does this PR introduce any user-facing change?

No
How was this patch tested?

I'm not sure how to test this because the current tests don't run the functions.transform directly
@nvander1 @RussellSpitzer

Can one of the admins verify this patch?

Maybe you can make a test with an example of the shadowing to demonstrate how it works correctly now.

I think your example from slack would be prefect as a test. Especially because of the divergent behavior between Spark SQL and the programmatic api

@nvander1 test added

pinging @maropu, @HyukjinKwon
This correctness related PR has gone unnoticed

ok to test

branch-3.0 has the same issue? Also, could you describe more in the PR description? It should be self-contained for commit logs, I think (Its okay to just copy&paste the example query in the jira)

Test build #136924 has started for PR 31887 at commit 8c2ad61.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41501/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41501/

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

Test build #136924 has finished for PR 31887 at commit 8c2ad61.

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

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

Kindly ping.

@maropu I won't be able to work on this until June 1, other duties are getting in the way.

cc @ueshin too FYI

@maropu I won't be able to work on this until June 1, other duties are getting in the way.

Ah, I see. Is it okay that I take this over?

@maropu I won't be able to work on this until June 1, other duties are getting in the way.

Ah, I see. Is it okay that I take this over?

Go ahead.


Change Spell Damage and Heal

Hello, hope you're doing well fzzy :). I was wondering if it's possible to actually change the damage or effects that some scepters do? For example is there a way to be able to increase the damage of a spell like Soul Missle (since I find most of the fury spells to be underwhelming in damage, especially late game)?  When I go over to the SoulMissileAugment_v2.jsonfile it shows me:
"id": "amethyst_imbuement:soul_missile",
"enabled": true,
"pvpMode": false,
"cooldownBase": 0.5,
"coolDownPerLvl": 0,
"manaCost": 0.5,
"minLvl": 1,
"castXp": 1
but no way of actually editing the damage and increasing it.
Is there also a way to edit the way a spell works? As another example, Mass Fortify fully upgraded and with the path like enduring bard basically gives me Res 4.75 (Res 5 is 100% damage resistance) and with enough duration that I can almost infinitely have it which I find to be a bit too op lol. Is there any way to change this on my side?
Thanks for the help and have a great day!

Oh I forgot to add, I changed some stuff already in the soul missile which is why it looks a bit different lol.

Hi, I am doing pretty well, thanks for asking 💯. I hope you are also.
First thing I'll note is that cooldown and mana cost are probably causing issues in the config. That cooldown is 0.5 ticks, which isn't a valid unit of time in MC, and the mana cost is one half of a mana, which also isn't valid. Mana is measuring in whole numbers (because it's just durability with some paint over top).
To your actual request... I am working on a total rebuild of the spell system, which will

Change up the configuration system
Add a huge new spell feature that will alleviate some concerns with spells late game.
I have to be careful with how powerful I can make spells in general. There should be some trade offs considering the extreme flexibility of scepters, the "infinite" ammunition, the near-infinite range and accuracy (of some spells), etc.

I will consider including the "stat" values in the updated configuration (damage, range, duration, amplifier), but it may not happen, or it may be a compromise like a single "factor" instead of every lever and knob (there are a lot of internal levers and knobs).
I also want to point out that Soul Missile isn't a particularly powerful spell. You can dish out a lot more DPS with tier 2 and 3 spells.


drivers: ramdisk: Add support for embedded disk images as backing store of RAM disks

Add an optional devicetree binding, and corresponding support in the RAM disk driver, to allow using a disk image - embedded as a binary blob in the application image - directly as the backing store of a RAM disk instance.
This significantly improves usability of deploying a RAM disk in scenarios where it has to be populated with some initial content provided at build time, especially where that content is provided as a disk image created during the overall system-level build steps.
Moreover, it will help ease migration for users moving to Zephyr from some legacy RTOS (such as Nucleus) that supported creating RAM disks from existing disk images, directly using the disk image as a RAM disk afterwards. Zephyr's existing RAM disk implementation can also support this usecase with this small update.
Note that this implementation uses .incbin directive which is technically GNU-specific but is widely supported by other mainstream toolchains as well (for compatibility with GCC). C23 #embed directive can be a more portable option but toolchains supporting it are just coming out. We can consider updating this implementation when sufficient toolchain support of #embed becomes available. The .incbin based implementation can be downgraded to a fallback at that time.
Update:
The description above reflects the initial PR state. Since then, as a result of discussion during the review of this PR, the implementation has been updated to transparently account for all toolchains, both with and without GNU extensions. For toolchains, where GNU extensions are available, .incbin will be leveraged for build-time efficiency. For other toolchains, a generic fallback will be used that uses CMake build system and scripts (inspired by existing generate_inc_file CMake function and its associated scripts) to include binary data as an array, initialized with binary data converted into initializer list in hex character from.

@hIAhmad I've approve the devicetree. The quotation marks that need to be added and the blank lines at the end of the file have been completed. I apologize for the misunderstanding :(
The first PR went well, which is a good start.

@decsny I just pushed a small update to zephyr,ram-disk.yaml in response to a comment from @jfischer-no and re-requested a review from him. But then noticed that your review had gotten dismissed and along with it the approval. I am not sure if re-requesting the review (that was not from you though) did it or my update to the file you had commented on.
My apologies for the inconvenience. Kindly have another look.

So we already have generate_inc_file, which can generate a header file from binary (which can then be included in a build). You can make the header file based on the ramdisk nodelabel's name, and then include that header file within the ramdisk driver to initialize the ramdisk array.
Using generate_inc_file here would be a lot more portable than incbin, and follows patterns we already use in Zephyr.
Some examples of the usage:
https://github.com/zephyrproject-rtos/zephyr/blob/2d60d248e846102867e4aca7a9e3d3e6ae591f06/subsys/net/l2/wifi/CMakeLists.txt#L54
https://github.com/zephyrproject-rtos/zephyr/blob/2d60d248e846102867e4aca7a9e3d3e6ae591f06/subsys/net/l2/wifi/wifi_shell.c#L53
@hIAhmad any thoughts on this approach?

@danieldegrasse Thanks for your helpful suggestion. I am looking into leveraging this mechanism.

@danieldegrasse I have revised the implementation to take advantage of the CMake build system and scripts.
generate_inc_file (and variants) are certainly useful (and I have leveraged them) but there are two caveats here:

Unlike the straightforward case in the existing example you pointed out in the comment above, in the present use case the data to be converted is per-device-instance. So it requires an extra level of indirection (both at build configuration as well as at compile time) as converted data cannot be included directly in the related macros in ramdisk.c (as #include cannot be used inside macro definitions).
Compared to .incbin directive, including binary data as array data is definitely generic (after conversion step is out of the way) but it is highly inefficient. Even with a disk image of ~1MiB size, the build momentarily slows when array data including the converted binary data is being compiled. The build time and host memory usage can increase almost exponentially as larger sized binary data would be embedded with this approach (see the tables 3.2.1 and 3.2.2 in https://thephd.dev/_vendor/future_cxx/papers/C - embed.html for reference). For small data files, however, such as cert files, this is perfectly suitable. For larger files less so, especially as the size increases. Given this concern, I have used this method as a generic fallback, to be used when GNU extensions are not supported by the toolchain in use or user has explicitly turned them off. When GNU extensions are available - which is default - .incbin directive will still be used.

As a side benefit, the updated implementation now handles build dependencies (on disk image files and DTS) more precisely than before.

Compared to .incbin directive, including binary data as array data is definitely generic (after conversion step is out of the way) but it is highly inefficient. Even with a disk image of ~1MiB size, the build momentarily slows when array data including the converted binary data is being compiled. The build time and host memory usage can increase almost exponentially as larger sized binary data would be embedded with this approach (see the tables 3.2.1 and 3.2.2 in https://thephd.dev/_vendor/future_cxx/papers/C - embed.html for reference). For small data files, however, such as cert files, this is perfectly suitable. For larger files less so, especially as the size increases. Given this concern, I have used this method as a generic fallback, to be used when GNU extensions are not supported by the toolchain in use or user has explicitly turned them off. When GNU extensions are available - which is default - .incbin directive will still be used.

Thanks for linking this- interesting to see. Given the concrete benefit of using .incbin here, I'm ok with supporting generate_inc_file as a fallback method only.


TreeView is not working on IE11.

I'm developing on IE11 using the Blazor server side. But, this package is not working on IE11. Is it right?

Hi @Hyung-Kim,
I cannot recommend using IE11 at all with Blazor. It is only supported with .NET 3.1 and Blazor server-side, and it is not supported with .NET 5 or later.
I just had a quick look at this and it seems that both CSS is broken for IE11 and events are also not propagated to the server. Given that this library doesn't do anything overly complex in terms of events and they're not working, I'm not inclined to fix this for a defacto abandoned browser.
If this is a critical issue to you, I can only recommend forking the repo and modifying it for this explicit purpose.
BR
Stefan

@stefanloerwald  Ok I modified CSS code. so It is working well. Thank you.


License Snippet Dropdown update

This extends the work done in PR 3090 to solve #1976
I reorganized things a little bit to use sub-menus and files per licensing organization. It makes it a bit easier to work around in though it's still awkward with the Creative Commons
To Do:
Creative Commons "deeds"
Creative Commons image snippets
layout tweaks ( sub-lists after rows with STRONG, in particular )
Maybe To Do:
Various Publisher community licenses, such as the AGE Creators
Alliance, Super-Powered by M&M, various Drive Through RPG Content licenses, etc.

Altering PR location to preferred process.


Make std.digest.hmac @safe and runnable

When I first encountered the version(StdDdoc) unittest in std.digest.hmac, I tried to go "the easy way" and ignore versioned unittest in the tests_extractor (-> https://github.com/dlang/tools/pull/249).
However @CyberShadow correctly reminded me that it makes more sense to fix the outlier and I even discovered another problem about the version(StdDdoc) approach - it's not part of the testsuite and apparently at some point in the time std.digest.hmac was @safe (it's not anymore). I fixed that as well and as a consequence std.digest.hmac is now completely @safe :tada:
So I am really happy about @CyberShadow's remark to avoid working around the outlier with the tool!
CC @jpf91

#5013 was merged and as far as I understand, this can be changed to: import std.digest : toHexString

Yes - 🎉


Trdm schema migration

Migrated columns for LOA and TAC causing parsing issues. These new column mods from int(4) to varchar(20) are in line with TRDM expectations. These migrations will convert existing integers to strings without data loss.
Tests have been adjusted per integer to string conversion.
Additionally, this turns back on the midnight cron jobs for TRDM data.

Warnings

:warning:
Please add the JIRA issue key to the PR title (e.g. MB-123)

  Generated by :no_entry_sign: dangerJS against 1089e469ac4a143e8ac4ebe336ab6b37ce83b98d


Automatically activate Roslyn analyzers in NuGet packages in Unity

Is your feature request related to a problem? Please describe.
Unity supports running Roslyn analyzers which can be installed through NuGet. The main downside in Unity is that they recommend grabbing the DLL from the NuGet package and dropping it in your project, which is unfortunate since it doesn't allow automatic updating of the package, and forces you to include packages in version control.
Describe the solution you'd like
When a package containing Roslyn analyzers is installed, they are automatically registered with Unity as roslyn analyzers.
Describe alternatives you've considered
Installing them manually the Unity way or making a script to install them (which is what I'm doing for the moment).
Additional context
Examples of such packages include Roslynator, ErrorProne, and Microsoft Unity Analyzers.
Automating this comes down to the following:

Download and extract the NuGet package (already happens).
Search for analyzer DLLs in the package, copy them to the Assets/Scripts/Foo folder (or another folder that is picked up by Unity).
Either generate or update a matching meta file for each DLL that has the RoslynAnalyzer label set (or Unity will not recognize the DLL as an analyzer).

I'm not sure how to recognize DLLs that are analyzers as opposed to regular assemblies for step 2, but looking at the above examples, they all use subfolders analyzers/dotnet/cs to store these DLLs in.
If users do not want to run these analyzers automatically (as their errors block play mode), they can simply disable Roslyn analyzer support in the Unity project settings, so it should be safe to always do this.
Finally, an added bonus to this is that Unity (2020 and up) will also put these analyzers in C# project files generated for e.g. VSCode, so OmniSharp will also be able to use them.

Good feature request.
It is difficult to recognize exactly the analyzer since the official NuGet package structure did not include it. As you say, we may assume that the analyzer is the one in a folder such as analyzers/dotnet/cs.
I can implement the automatic addition of the analyzer by doing something like CsprojModifier, but it's not very pretty, so I'll think about it some more.

Sorry for the very late reply...
I implemented the installation of Roslyn Analyzer.
.dlls under analyzers/dotnet/cs are installed as Roslyn Analyzer.
I have not tested Rider, so I don't know, but it worked fine in VS and VSCode.


Sign in with security key not working if email is cached

What is happening
If you try to sign in but don't complete MFA and visit the initial sign in page again, it caches the email and only shows password and security key sign up options, even if security key is not set up. If the security key is not set up it's not putting an error message, it just keep redirecting back to the same page.
What should happen
First of all it shouldn't obscure the email address that is cached, it's bad UX practice as the user might be trying to sign in to a different account, and you're making them guess which email has been cached. Display the email even thought it's been cached. Secondly, either display all methods of sign in or only supported ones, cause now you're doing neither.
Preferred flow

Flow in screens

User visits resource that is accessible via Login UI. UI displays all of possible sign in methods

User signs in with email and password. Login UI asks for MFA. User closes the tab

User visits resource that is accessible via Login UI. UI displays cached email with ability to edit email, and all sign in methods that this email is able to use (in this example, email has security key and google, but not Okta set up for this identity.)

Compromise option
Same flow, but display all of the possible sign in option, regardless of what options are possible.

I don't really like the approach we take here:

the button that says Change e-mail, seems weird/counter-intuitive to me as a user.
The logic for resetting aal1 feels very hacky and I think will be difficult for someone who does not have our context to extend.
Even though we ask the user to provide their password, this is not needed. The session is cached in the cookies (we can just redirect them to the 2fa page.

We should have a look at what major providers do and copy them. For example on github (not the best example, I know) if you press back on the authorizer screen, you will be asked to enter your username/password again and if you do browser forward, you will again will be asked for totp.
I think for us the best flow would one of the following:

show the user a message that says something along the lines Please validate that this is you: {email} and underneath 2 buttons (one for accepting and one for declining). If they decline, then we delete the user session and we show the login screen as well. If they accept, then we set the aal param to aal=aal2 in the query param. This will show the user the totp page again.
Do something similar to github and automatically reset the session and clear both fields, this would need some more work as we would want to keep the forward button working as well.

What do you think? cc @lukasSerelis @edlerd
Originally posted by @nsklikas in https://github.com/canonical/identity-platform-login-ui/pull/292#pullrequestreview-2337635863

I don't really like the approach we take here:

the button that says Change e-mail, seems weird/counter-intuitive to me as a user.
The logic for resetting aal1 feels very hacky and I think will be difficult for someone who does not have our context to extend.
Even though we ask the user to provide their password, this is not needed. The session is cached in the cookies (we can just redirect them to the 2fa page.

We should have a look at what major providers do and copy them. For example on github (not the best example, I know) if you press back on the authorizer screen, you will be asked to enter your username/password again and if you do browser forward, you will again will be asked for totp.
I think for us the best flow would one of the following:

show the user a message that says something along the lines Please validate that this is you: {email} and underneath 2 buttons (one for accepting and one for declining). If they decline, then we delete the user session and we show the login screen as well. If they accept, then we set the aal param to aal=aal2 in the query param. This will show the user the totp page again.
Do something similar to github and automatically reset the session and clear both fields, this would need some more work as we would want to keep the forward button working as well.

What do you think? cc @lukasSerelis @edlerd
Originally posted by @nsklikas in #292 (review)

The github option is probably less fussy. Even if we don't have the browser forward action working it is a less convoluted flow. The change email and caching email is a thing a lot of SSOs do as well, so this isn't necessarily a unique flow. But I agree that it'd be simpler to just clear all fields. More frustrating in some cases, but less complex, and I'll always lean to less complexity. I'll update the flows to match this and then we can go forward with the github style of not saving either field imho.

@nsklikas @edlerd
Flow that I agree with Nikos is much more simplified. Nothing is cached. The only thing is a question of implementation if we can make it work but would be nice is the going forward in browser after going back to see working MFA again...

Flow that I agree with Nikos is much more simplified. Nothing is cached. The only thing is a question of implementation if we can make it work but would be nice is the going forward in browser after going back to see working MFA again...

Kratos is controlling which flow will be rendered. For a user coming with a kratos session, kratos initiates the "Enter password" with immutable email flow. I have no idea how to change this behaviour other than removing the cookie. Doing that automatically in the frontend -- without user interaction, as the flows suggest -- seems very hacky to me. Happy for other suggestions, maybe you can configure kratos accordingly?

I don't know about the possible kratos configurations off the top of my head (probably others are more suitable in answering that). But worst case possible we could handle this in the backend. There are 2 options:

Enhancing the createLoginFlow endpoint, with a new parameter that would clear the kratos cookies. In this case the UI will just have to send the refresh parameter to the backend
Create a new endpoint that would call the kratos endpoint that clears a session, if the frontend sees that the user has an active session with aal1 and they are at the password page, then the frontend will just call the backend to clear the session.

Both of these should be possible, but I will have to validate it.

@nsklikas I think this has been fixed with #301
Can this be closed?

I think so, but I would leave it up to @lukasSerelis to decide

Closing after talking to Lukas and he agreed.


Fix a crash when parsing HTML with XPath.

some html data declared in utf8,but it contains some unicode char.
hpple's XPathQuery.m:31,40,44 will lead to crash(set nil object to dictionary).

http://www.qiushibaike.com/text   the data from this url sometimes contains unicode char in utf-8 content.

what content did you parsed?

如图，一些字符变成了乱码


SIG_MTG: receptor signaling proteins

I would like to do a clean up (obsolete/rename/redefine several terms) in the 'receptor signaling protein' bit of GO function.
Most of the terms in this node are confusing, undefined and combine subcellular location and molecular function info. In most cases, it's not clear whether it's the receptor or the signaling molecule that has the kinase activity. It contains molecules upstream of the receptor, receptors themselves (in one case), proteins that bind to the receptor to convey the signal, and proteins that lie further downstream of the receptor (eg MAPKs).
Whether the signaling molecule is downstream of a tyrosine kinase or ser/thr kinase receptor should be conveyed by existing GO PROCESS terms.
I think it's still important to capture that the protein is conveying the signal by phosphorylating/dephosphorylating downstream targets, so I'd like to keep some of the terms, but rename them to (hopefully) make it clearer.
1/ Redefine 'signal transducer activity ; GO:0004871'
From:
Mediates the transfer of a signal from the outside to the inside of a cell by means other than the introduction of the signal molecule itself into the cell.
(this is an old def taken straight from The Dictionary of Biochemistry and Molecular Biology)
to:
Converts a signal into a form where it can ultimately trigger a change in the state or activity of a cell. A signal is a physical entity or change in state that is used to transfer information in order to trigger a response.
(people were previously uncomfortable with 'converting a signal from one form to another' for cases where the signal was conveyed by transferring a phosphate group on one molecule to a phosphate group on another, since the form of the signal is the same, so I've tried to avoid that text string.)

2/ Define: receptor signaling protein activity ; GO:0005057
Conveys a signal from an upstream receptor or intracellular signal transducer, converting the signal into a form where it can ultimately trigger a change in the state or activity of a cell.
Comment: This term is intended for annotation of signal transducers that act directly or indirectly downstream of a receptor. For receptors themselves, consider annotating to the term 'receptor activity ; GO:0004872'. Also consider annotating to the term 'intracellular signal transduction ; GO:0035556'.
NB: I was originally thinking of obsoleting this term, and suggesting annotating to the term: 'intracellular signal transduction ; GO:0035556' , to distinguish the downstream molecules from the receptors themselves but looking at the annotations, I think people want a term to distinguish the receptors from the downstream signaling molecules.

3/ Up for obsoletion we have:
receptor signaling protein serine/threonine kinase signaling protein activity ; GO:0005076 (undefined) (0 annotations)
receptor signaling protein tyrosine kinase signaling protein activity ; GO:0008268 (undefined) (3 direct annotations)
receptor signaling protein serine/threonine phosphatase signaling protein activity ; GO:0005081 (undefined) (0 annotations)
receptor signaling protein tyrosine phosphatase signaling protein activity ; GO:0005082 (undefined) (2 annotations)
transmembrane receptor protein serine/threonine kinase signaling protein activity ; GO:0005071 (undefined) (2 direct annotations)
transmembrane receptor protein tyrosine kinase signaling protein activity ; GO:0005066 (undefined) (13 direct annotations)
I'm not even sure what these mean… there's too many instances of 'signaling protein' in one term name.
Annotations can be transferred to'signal transducer activity ; GO:0004871' or 'receptor signaling protein activity ; GO:0005057'

4/ Up for renaming, we have:
receptor signaling protein tyrosine kinase activity ; GO:0004716
>
signal transducing protein tyrosine kinase activity
Acts directly or indirectly downstream of a receptor to convert a signal into a form where it can ultimately trigger a change in the state or activity of a cell, by catalysis of the reaction: ATP + protein tyrosine = ADP + protein tyrosine phosphate.
receptor signaling protein tyrosine phosphatase activity ; GO:0004728
>
signal transducing protein tyrosine phosphatase activity
receptor signaling protein serine/threonine kinase activity ; GO:0004702 (undefined)
>
signal transducing protein serine/threonine kinase activity
receptor signaling protein serine/threonine phosphatase activity ; GO:0009400
>
signal transducing protein serine/threonine phosphatase activity
(NB: I know Amelia has plans to split the ser/thr terms into separate functions, but this can be done en-masse once the parent terms are in place).
5/
I will move:
transmembrane receptor protein serine/threonine kinase activity ; GO:0004675 from under GO:0004702 as it's a receptor itself.
I will move:
receptor regulator activity ; GO:0030545' out from under 'signal transducer activity', as it creates TPVs for ligands, which are not signal transducers according to GO:0004871 comment.
Thanks,
Becky
Reported by: rebeccafoulger
Original Ticket: geneontology/ontology-requests/7908

Receptor signaling protein terms renamed. See.  https://github.com/geneontology/go-ontology/issues/12677  Also fixed issues with incorrect inference of receptors under these terms. Decided not to touch TM receptor terms.  I think it is useful to make clear which activities involve transduction across membranes and which do not.


feat: add new file picker component

What was done
Added new component (filepicker) for uploading both images and pdf:s in the same component
How was it done
Added the new component type as other components are added in formbuilder.
Why was it done in this way
This is how its done
How was it tested
Changed a form that uses the new component and tested it in the application

Task linked: CU-2qa1knt Skapa ny komponent i formbuildern för att kunna ladda upp filer


Dark Mode

Thanks to @calandryll 's work, Fermentrack can now have a "dark mode" theme selected from the configuration page. This effectively adds the feature @jnr-nat requested in #507 (even though that issue was closed)
Thanks again @calandryll -- this is awesome!

Awesome, Thanks, guys. Now to get it to display dark on my from my laptop..... Selected Nord Theme, and still white

Yeah, saved reloaded, then even restarted the Pi, same result

I think I know what the problem is.  I forgot to add in the settings.py file in my commit.  I'll do another pull request and that should hopefully fix it.

I think I know what the problem is. I forgot to add in the settings.py file in my commit. I'll do another pull request and that should hopefully fix it.

I thought I added the settings.py changes on my end

I think I know what the problem is. I forgot to add in the settings.py file in my commit. I'll do another pull request and that should hopefully fix it.

I thought I added the settings.py changes on my end

Yep you did.  Hmm, not sure why it worked for me on my dev setup under a Ubuntu VM.

"Works here."

I’m not seeing it yet.  Upgraded, set the option, saved.  It shows set to dark mode.
\Greg
From: Lee @.>
Sent: Wednesday, May 12, 2021 7:01 PM
To: @.>
Cc: @.***>
Subject: Re: [thorrak/fermentrack] Dark Mode (#634)
"Works here."
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fthorrak%2Ffermentrack%2Fpull%2F634%23issuecomment-840147672&data=04|01||53125076bda44884441308d91599ef8a|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637564573148849384|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=IhScHsZRPJZbd4TQtW8RjcL2HEyO9K6%2F1TsP3EtkAgo%3D&reserved=0, or unsubscribehttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAGEO6Z46DFVYX62REFVK6D3TNMCGBANCNFSM44XTOHYQ&data=04|01||53125076bda44884441308d91599ef8a|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637564573148849384|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=akyiHJ9%2B6ZtVgh45XY%2FrGK3STNVWukza1J1CvRe9Iqs%3D&reserved=0.

I’m not seeing it yet. Upgraded, set the option, saved. It shows set to dark mode.

HAve you used a different browser, your phone, run it in privacy mode?


ResultType

Fixes #6336

If arrays and scalars are mixed, only arrays are considered.
If float dtypes and other dtypes are mixed, only float dtypes are considered.
If unsigned and signed integers are mixed, and the widest one is unsigned, a signed dtype with one level wider is chosen.

Jenkins, test this please

Jenkins CI test (for commit 9ad7539ab9186c97936e3176b8775d103382cb6a, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

jenkins, test this please.

Jenkins CI test (for commit 70b5857e5df0f6d6cd994ff44186bb3f0e0604c1, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

Jenkins and Travis failures seem unrelated.

LGTM(us @asi1024 )!

Jenkins CI test (for commit 70b5857e5df0f6d6cd994ff44186bb3f0e0604c1, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)


How to use npm in laravel sail

I had tried ../../vendor/bin/sail npm install this just install node modules in base_path

It is not supported by laravel sail