Driver error

i got the error when i convert yolov3.onnx to .trt:
Traceback (most recent call last):
File "tools/deployment/onnx2tensorrt.py", line 253, in
verbose=args.verbose)
File "tools/deployment/onnx2tensorrt.py", line 44, in onnx2tensorrt
max_workspace_size=max_workspace_size)
File "/root/bcl/downloads/mmcv/mmcv/tensorrt/tensorrt_utils.py", line 84, in onnx2trt
engine = builder.build_engine(network, config)
RuntimeError: Driver error:
Pytorch==1.6.0, onnx==1.7.0 and TensorRT-<IP_ADDRESS>.Ubuntu-18.04.x86_64-gnu.cuda-10.2.cudnn8.0. The tool doesn't support ubuntu18.04?

@BChunlei Hi, could you provide more information as below:
Reproduction

What command or script did you run?

A placeholder for the command.

Did you make any modifications on the code or config? Did you understand what you have modified?
What dataset did you use?

Environment

Please run python mmdet/utils/collect_env.py to collect necessary environment information and paste it here.
You may add addition that may be helpful for locating the problem, such as

How you installed PyTorch [e.g., pip, conda, source]
Other environment variables that may be related (such as $PATH, $LD_LIBRARY_PATH, $PYTHONPATH, etc.)

Error traceback
If applicable, paste the error trackback here.
A placeholder for trackback.

Hi @RunningLeon , i run the following command:
python tools/deployment/onnx2tensorrt.py configs/yolo/yolov3_d53_mstrain-416_273e_coco.py my_onnx/yolov3.onnx --trt-file my_trt/yolov3.trt --shape 416 416
I didn't change the config file or code.
The environment is:
sys.platform: linux
Python: 3.7.10 (default, Jun  4 2021, 14:48:32) [GCC 7.5.0]
CUDA available: True
GPU 0,1: GeForce GTX 1080
CUDA_HOME: /usr/local/cuda-10.2
NVCC: Cuda compilation tools, release 10.2, V10.2.89
GCC: gcc (Ubuntu 7.5.0-3ubuntu1~18.04) 7.5.0
PyTorch: 1.6.0
PyTorch compiling details: PyTorch built with:

GCC 7.3
C++ Version: 201402
Intel(R) oneAPI Math Kernel Library Version 2021.2-Product Build 20210312 for Intel(R) 64 architecture applications
Intel(R) MKL-DNN v1.5.0 (Git Hash e2ac1fac44c5078ca927cb9b90e1b3066a0b2ed0)
OpenMP 201511 (a.k.a. OpenMP 4.5)
NNPACK is enabled
CPU capability usage: AVX2
CUDA Runtime 10.2
NVCC architecture flags: -gencode;arch=compute_37,code=sm_37;-gencode;arch=compute_50,code=sm_50;-gencode;arch=compute_60,code=sm_60;-gencode;arch=compute_61,code=sm_61;-gencode;arch=compute_70,code=sm_70;-gencode;arch=compute_75,code=sm_75;-gencode;arch=compute_37,code=compute_37
CuDNN 7.6.5
Magma 2.5.2
Build settings: BLAS=MKL, BUILD_TYPE=Release, CXX_FLAGS= -Wno-deprecated -fvisibility-inlines-hidden -DUSE_PTHREADPOOL -fopenmp -DNDEBUG -DUSE_FBGEMM -DUSE_QNNPACK -DUSE_PYTORCH_QNNPACK -DUSE_XNNPACK -DUSE_VULKAN_WRAPPER -O2 -fPIC -Wno-narrowing -Wall -Wextra -Werror=return-type -Wno-missing-field-initializers -Wno-type-limits -Wno-array-bounds -Wno-unknown-pragmas -Wno-sign-compare -Wno-unused-parameter -Wno-unused-variable -Wno-unused-function -Wno-unused-result -Wno-unused-local-typedefs -Wno-strict-overflow -Wno-strict-aliasing -Wno-error=deprecated-declarations -Wno-stringop-overflow -Wno-error=pedantic -Wno-error=redundant-decls -Wno-error=old-style-cast -fdiagnostics-color=always -faligned-new -Wno-unused-but-set-variable -Wno-maybe-uninitialized -fno-math-errno -fno-trapping-math -Werror=format -Wno-stringop-overflow, PERF_WITH_AVX=1, PERF_WITH_AVX2=1, PERF_WITH_AVX512=1, USE_CUDA=ON, USE_EXCEPTION_PTR=1, USE_GFLAGS=OFF, USE_GLOG=OFF, USE_MKL=ON, USE_MKLDNN=ON, USE_MPI=OFF, USE_NCCL=ON, USE_NNPACK=ON, USE_OPENMP=ON, USE_STATIC_DISPATCH=OFF,

TorchVision: 0.7.0
OpenCV: 4.5.2
MMCV: 1.3.8
MMCV Compiler: GCC 7.5
MMCV CUDA Compiler: 10.2
MMDetection: 2.13.0+11f3ca2
I use conda to install pytorch.
$PATH:
/usr/local/cuda-10.2/bin:/root/anaconda3/envs/torch160trt7216/bin:/root/anaconda3/condabin:/usr/local/cuda-10.2/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin: No such file or directory
$LD_LIBRARY_PATH:
/root/bcl/downloads/TensorRT-<IP_ADDRESS>/lib:/usr/local/cuda-10.2/lib64: No such file or directory
Error traceback:
Traceback (most recent call last):
File "tools/deployment/onnx2tensorrt.py", line 253, in
verbose=args.verbose)
File "tools/deployment/onnx2tensorrt.py", line 44, in onnx2tensorrt
max_workspace_size=max_workspace_size)
File "/root/bcl/downloads/mmcv/mmcv/tensorrt/tensorrt_utils.py", line 84, in onnx2trt
engine = builder.build_engine(network, config)
RuntimeError: Driver error:

@BChunlei Hi, couldn't find the cause. Maybe there's something wrong with Ubuntu18.04. It works fine with Ubuntu16.04 though. BTW, you could add --verbose to onnx2tensorrt.py to print more detailed messages.

VERBOSE.txt
Thanks. I have attached the verbose, could you give me some advice?

@BChunlei Maybe you could provide config file and onnx file and we could try on our machine to reproduce.

The config file is 'configs/yolo/yolov3_d53_mstrain-416_273e_coco.py'
The onnx file is in : https://pan.baidu.com/s/1sO1dcLmI4_UTQhcCapdHUQ , code：7s70
Thanks.

@BChunlei It works successfully with your onnx file on my machine.
My envs:
sys.platform: linux
Python: 3.7.0 | packaged by conda-forge | (default, Nov 12 2018, 20:15:55) [GCC 7.3.0]
CUDA available: True
GPU 0: NVIDIA GeForce RTX 2080
CUDA_HOME: /usr/local/cuda-10.2
NVCC: Cuda compilation tools, release 10.2, V10.2.89
GCC: gcc (Ubuntu 5.4.0-6ubuntu1~16.04.12) 5.4.0 20160609
PyTorch: 1.6.0
PyTorch compiling details: PyTorch built with:
  - GCC 7.3
  - C++ Version: 201402
  - Intel(R) Math Kernel Library Version 2020.0.2 Product Build 20200624 for Intel(R) 64 architecture applications
  - Intel(R) MKL-DNN v1.5.0 (Git Hash e2ac1fac44c5078ca927cb9b90e1b3066a0b2ed0)
  - OpenMP 201511 (a.k.a. OpenMP 4.5)
  - NNPACK is enabled
  - CPU capability usage: AVX2
  - CUDA Runtime 10.1
  - NVCC architecture flags: -gencode;arch=compute_37,code=sm_37;-gencode;arch=compute_50,code=sm_50;-gencode;arch=compute_60,code=sm_60;-gencode;arch=compute_61,code=sm_61;-gencode;arch=compute_70,code=sm_70;-gencode;arch=compute_75,code=sm_75;-gencode;arch=compute_37,code=compute_37
  - CuDNN 7.6.3
  - Magma 2.5.2
  - Build settings: BLAS=MKL, BUILD_TYPE=Release, CXX_FLAGS= -Wno-deprecated -fvisibility-inlines-hidden -DUSE_PTHREADPOOL -fopenmp -DNDEBUG -DUSE_FBGEMM -DUSE_QNNPACK -DUSE_PYTORCH_QNNPACK -DUSE_XNNPACK -DUSE_VULKAN_WRAPPER -O2 -fPIC -Wno-narrowing -Wall -Wextra -Werror=return-type -Wno-missing-field-initializers -Wno-type-limits -Wno-array-bounds -Wno-unknown-pragmas -Wno-sign-compare -Wno-unused-parameter -Wno-unused-variable -Wno-unused-function -Wno-unused-result -Wno-unused-local-typedefs -Wno-strict-overflow -Wno-strict-aliasing -Wno-error=deprecated-declarations -Wno-stringop-overflow -Wno-error=pedantic -Wno-error=redundant-decls -Wno-error=old-style-cast -fdiagnostics-color=always -faligned-new -Wno-unused-but-set-variable -Wno-maybe-uninitialized -fno-math-errno -fno-trapping-math -Werror=format -Wno-stringop-overflow, PERF_WITH_AVX=1, PERF_WITH_AVX2=1, PERF_WITH_AVX512=1, USE_CUDA=ON, USE_EXCEPTION_PTR=1, USE_GFLAGS=OFF, USE_GLOG=OFF, USE_MKL=ON, USE_MKLDNN=ON, USE_MPI=OFF, USE_NCCL=ON, USE_NNPACK=ON, USE_OPENMP=ON, USE_STATIC_DISPATCH=OFF,

TorchVision: 0.7.0
OpenCV: 4.5.2
MMCV: 1.3.8
MMCV Compiler: GCC 5.4
MMCV CUDA Compiler: 10.2
MMDetection: 2.14.0+8f1eb16

@RunningLeon ,thank you. I have solved the issue by updating the driver version to 460.32.03. my card is GTX 1080.


Speed-up bootstrap and injection

[x] Start pivoting all DI to needing to be declarative (compile-time, const, etc)
[x] Optional generation of the injector used for bootstrap (instead of at runtime)
[x] Support the same generated injector pattern for deferred loading and testing
[x] Rename ReflectiveInjector to SlowInjector, accessible via a deprecated.dart

With all of these changes made it will technically be possible to run your application without a single initReflector or @Injectable() annotation needed anywhere, assuming you entire transitive application doesn't use DynamicComponentLoader or ReflectiveInjector.

We are skipping the last step for 5x.


Create a behavior to send out auto notifications

As you know keeping controllers clean and implementing the data related logic into model level is a better approach for CakePHP in a  few different ways.
So, since the notifications are created with templates, I often find myself using variables from my entity just after saving it. For example; I want to notify all users when I add a new record to my posts. So I suggest thinking about something like below:
    $this->addBehavior('bakkerij/notifier.notifier', [
                'template' => 'someTemplateName',
                'keywords' => [
                    'keyword1' => 'columnNameOne',
                    'keyword2' => 'columnNameTwo',
                ],
                'to' => 'recipentListName?'
    ]);

Sorry for the delay @suhaboncukcu. What would be the sense of a behavior?
If you want to keep your controllers clean (I agree with that :)), you can still add your code to send notifications within events in your model right?

As I can code my own notifications plugin :) I thought it would be something nice working out of the box.

Would be kind of 'double' right? ;)
Can you show me an example how it would work better in your situation? I am curious!

I have a system that people joins to some channels and they get mails each time something added to their channels.
I installed the notifications plugin to set up internal notifications (or with adapters may be, to be able to send to different places).
After configuring the plugin, I should run the component each time I need a notification sent, which could be at different endpoints. So I really don't want to do that since we are working as several people on the project and it could lead some conflicts. So, I should create en event listener and fire an event to handle this notification sending which I would use the utility class since I'm on model level.
If I had a behavior instead; I could just call that behavior within my model class. I can -of course- create a behavior with the utility class. Only if I don't have a behavior class waiting ready for me. :)

I don't have a behavior class waiting for you unfortunately. Can you set up a PR? I think you're right that the behavior should use the Utility class heavily...

surely, as soon as I have time. I will send a PR. Which I think would describe my case better as well.


Update Keyboard or AlphabeticKeyboard

Hi,
How to Update Keyboard or AlphabeticKeyboard after func handleTap() since text if the text equals "test1" ?
Thank you

Hi! I’m not sure I follow. Can you explain in more details?

J'ai deux type de clavier azerty et qwerty et dans autocomplete et ajouter un array ['azerty', ´qwerty '] et je sais que la fonction handleTap qui gère le truc alors ma question comment je peux mettre le clavier a jour après le  clique sur handleTap ça veut dire changer les nouveaux caractères vers azerty ou qwerty

Aha, so you want to change the keyboard type?
If you create a custom action handler (you can look at DemoKeyboardActionHandlerBase in the demos), you can just override handle(_ gesture: KeyboardGesture, on action: KeyboardAction, sender: Any?), call super then do your custom things, whatever that may be.
To actually replace the keyboard, it works differently if you use UIKit or SwiftUI.
In UIKit, you can just do a keyboardStackView.removeAllArrangedSubviews() and then recreate and insert a new keyboard. Have a look at KeyboardViewController+Keyboard in the UIKit demo for details. So in UIKit, the custom action handler can do stuff with the views directly.
In SwiftUI, things work differently. Here, you can instead listen to the observed context and just use a different view whenever the keyboard type changes. In your case, since you're switching between two alphabetic keyboards, you probably need another observable property as well, besides keyboard type.

Thank you for your reply,
I use UIKit
I tried to override func handleTap() in your DemoKeyboardViewController but I can't
and especially reload the keyboard after some conditions on handleTap ()

Do you also call super.handleTap()?

Do you also call super.handleTap()?

can you provide me a piece of code like in function or extension to add on DemoKeyboardViewController so that I can override handleTap and add code that reloads the keyboard ?
we are still talking about switching the keyboard to AZERTY after handleTap

can you provide me a piece of code like in function or extension to add on DemoKeyboardViewController so that I can override handleTap and add code that reloads the keyboard ?
we are still talking about switching the keyboard to AZERTY after handleTap


Lots of P1 timeouts

The connection to the p1 reader seems to time out quite freequently. Probably need to make some more tests here, maybe try some telnet lib and see how that behaves.

hmmm broke out the p1 stuff into its own project and thus far is seems to work well enough....

Been testing this for a few hours now and it seems to work ok in the firmware also. Maybe this was a temporary thing... will continue the testing for a while more.

This has been running all night now without issues. The likely cause was either some temporary network stuff or computer hibernation issues. I.e. I turned off the suspend in 20 min of inactivity thingy on my test computer.


Header Block pattern

Design goal: draft a header block pattern by EOS. Likely overlaps into next sprint. Also re-align the figma component to remain as a pattern stickersheet, rather than it's own component in:

[x] Salt Next component library
[x] Salt Light component library (current)
[x] Salt Dark component library (current)
[ ] Figma guide

Galao Goal: design review by end of wk1
We will bring in remaining work as and when we start working on them

Latte Goal: page needs publishing, ready to go, review during sprint, maybe bumped to Lungo

No work to happen beyond pushing dialog and overlay header to labs in Latte. Updates to header block pattern page to be updated for core in Lungo sprint.

Lungo Goal: no dev capacity

Moved to Macchiato sprint. No capacity in Lungo. @navkaur76 to resume work after holiday.

Macc Goal: Kickoff planned to align the remaining work

Pattern page


Android application size is big when building on Windows host

Steps to reproduce
Create a simple flutter application.
run the application using the command flutter run
Actual results
-The size of the Flutter runner (debug app) is exceptionally high (480Mb) for a very simple application that just displays text on the screen. This substantial runner size significantly impacts my build time, especially during the "installing / sync files to device" phase, which takes 3-4 minutes. (wich i think is normal due to the app size)
-i'm using flutter run to run the app in debug mode
-Additionally, the project size is abnormally high at 1.2GB.
Logs

Logs

<!-- Paste your logs here -->

Flutter Doctor output
[√] Flutter (Channel stable, 3.16.8, on Microsoft Windows [Version 10.0.22000.1936], locale fr-FR)
• Flutter version 3.16.8 on channel stable at C:\Users\Administrator\flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision https://github.com/flutter/flutter/commit/67457e669f79e9f8d13d7a68fe09775fefbb79f4 (10 days ago), 2024-01-16 16:22:29 -0800
• Engine revision 6e2ea58a5c
• Dart version 3.2.5
• DevTools version 2.28.5
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
• Android SDK at C:\Users\Administrator\AppData\Local\Android\sdk
• Platform android-34, build-tools 33.0.1
• Java binary at: C:\Program Files\Android\Android Studio\jbr\bin\java
• Java version OpenJDK Runtime Environment (build 17.0.7+0-b2043.56-10550314)
• All Android licenses accepted.
[√] Chrome - develop for the web
• Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe
[X] Visual Studio - develop Windows apps
X Visual Studio not installed; this is necessary to develop Windows apps.
Download at https://visualstudio.microsoft.com/downloads/.
Please install the "Desktop development with C++" workload, including all of its default components
[√] Android Studio (version 2023.1)
• Android Studio at C:\Program Files\Android\Android Studio
• Flutter plugin can be installed from:
https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 17.0.7+0-b2043.56-10550314)
[√] VS Code (version 1.73.1)
• VS Code at C:\Users\Administrator\AppData\Local\Programs\Microsoft VS Code
• Flutter extension version 3.60.0
[√] Connected device (4 available)
• BE2013 (mobile) • <IP_ADDRESS>:5555 • android-arm64 • Android 11 (API 30)
• Windows (desktop) • windows • windows-x64 • Microsoft Windows [Version 10.0.22000.1936]
• Chrome (web) • chrome • web-javascript • Google Chrome 120.0.6099.225
• Edge (web) • edge • web-javascript • Microsoft Edge 118.0.2088.46
[√] Network resources
• All expected network resources are available.
! Doctor found issues in 1 category.

Hi @nassim3952
Are you targeting Window app? Do you add any packages/plugins to your project? Please try this on a newly created Flutter project (counter app) and confirm if the issue persists. Thanks!

Hi @nassim3952 Are you targeting Window app? Do you add any packages/plugins to your project? Please try this on a newly created Flutter project (counter app) and confirm if the issue persists. Thanks!

Hi, @huycozy
I'm targeting android, and I haven't tested it on an iOS device yet however I've encountered the same issue on emulator and another android device
for a newly created Flutter project (the counter app) here are my findings:
-after running flutter clean the project size is only 870KB
-When building the application using flutter run --release, the application size is 22MB (acceptable, but I can't benefit from hot
reload in release mode) and the project size is 88MB
-Running flutter run in debug mode increases the application size to 487MB, and the project size becomes 1.25GB

-Running flutter run in debug mode increases the application size to 487MB, and the project size becomes 1.25GB

This is too big for an app from the newly created Flutter project. On my end, it's only ~68MB:

Can you check which AGP and Gradle versions the project is using? You can find them in:

AGP: /android/settings.gradle (since Flutter 3.16) or android/build.gradle (before Flutter 3.16)
Gradle: android/gradle/wrapper/gradle-wrapper.properties

-Running flutter run in debug mode increases the application size to 487MB, and the project size becomes 1.25GB

This is too big for an app from the newly created Flutter project. On my end, it's only ~68MB:

Can you also try analyzing the output APK using Android Studio and share the result here?
Also, can you check which AGP and Gradle versions the project is using? You can find them in:

AGP: /android/settings.gradle (since Flutter 3.16) or android/build.gradle (before Flutter 3.16)
Gradle: android/gradle/wrapper/gradle-wrapper.properties

This time, I don't know why the issue has improved a bit:
In debug mode, the application size is 48MB. However, on my device or emulator it's showing 187MB :

In release mode, the application size is 18MB.
-The project size is now 660MB.

AGP version: 7.3.0
Gradle version: 7.5

As I can see, it's normal sizes. Please observe if you can replicate it consistently and see if it has any pattern. If you are unable to see the original issue again, feel free to close it and write a comment when you have clear steps to reproduce. Thanks!

As I can see, it's normal sizes. Please observe if you can replicate it consistently and see if it has any pattern. If you are unable to see the original issue again, feel free to close it and write a comment when you have clear steps to reproduce. Thanks!

I retried the same process of creating a new project (the counter app) before i build the project everything seems normal the project size is only 524KB however when run the application on an android device in debug mode with the command flutter run (without even making any changes in the project) here are my results:

the project size explodes and now becomes 1.23 GB:

the application size on my device is 506MB:

the APK size is 126MB:

Is it normal for the build folder and the app to take up so much space for such a simple flutter project?

the application size on my device is 506MB:

This is big. On my end, it's only around 170MB (Realme 6, Android 11) and 150MB (Pixel 7, Android 14).
Could you share which device you are working on? (OS, model). Could you also check this on a new Android native project?

Could you share which device you are working on? (OS, model). Could you also check this on a new Android native project?

I don't think the issue is coming from my android device because I have already tried the same thing with two smartphones and even an emulator and I have more or less the same problem:

on Oneplus Nord N100 (Android 11) the app size is between 400 and 500MB (I don't know why the app size can vary by 100MB with the same code )
on redmi Note 10 (Android 11) the size of the app is approximately 300MB
on BlueStacks 59.300 (Android 9 I think ) the app is also approximately 300MB

I don't have this issue on native

Thanks for your response. I just tried building Flutter Android app on Windows host (same as your host OS) and this issue seems to happen there. The application size is around 450MB (which was built from the default Flutter counter project) on my a same device above (Realme 6, Android 11).
This issue doesn't appear if building Flutter Android app on macOS host as my result above.

flutter doctor -v (stable & master)
[√] Flutter (Channel stable, 3.16.9, on Microsoft Windows [Version 10.0.19045.3930], locale en-US)
    • Flutter version 3.16.9 on channel stable at C:\WIP\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 41456452f2 (11 days ago), 2024-01-25 10:06:23 -0800
    • Engine revision f40e976bed
    • Dart version 3.2.6
    • DevTools version 2.28.5

[√] Windows Version (Installed version of Windows is version 10 or higher)

[√] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at C:\AndroidSDK
    • Platform android-34, build-tools 34.0.0
    • Java binary at: C:\Program Files\Android\Android Studio Hedgehog\jbr\bin\java
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-b2043.56-10550314)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.4.4)
    • Visual Studio at D:\DOWNLOADWORK\VS2022
    • Visual Studio Community 2022 version 17.4.33213.308
    • Windows 10 SDK version 10.0.19041.0

[√] Android Studio (version 2022.3)
    • Android Studio at C:\Program Files\Android\Android StudioGiraffe
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-10027231)

[√] Android Studio (version 2023.1)
    • Android Studio at C:\Program Files\Android\Android Studio Hedgehog
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-b2043.56-10550314)

[√] VS Code (version 1.85.2)
    • VS Code at C:\Users\ADMIN\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.80.0

[√] Connected device (4 available)
    • RMX2001 (mobile)  • EUYTFEUSQSRGDA6D • android-arm64  • Android 11 (API 30)
    • Windows (desktop) • windows          • windows-x64    • Microsoft Windows [Version 10.0.19045.3930]
    • Chrome (web)      • chrome           • web-javascript • Google Chrome 121.0.6167.140
    • Edge (web)        • edge             • web-javascript • Microsoft Edge 121.0.2277.83

[√] Network resources
    • All expected network resources are available.

• No issues found!

[!] Flutter (Channel master, 3.20.0-2.0.pre.17, on Microsoft Windows [Version 10.0.19045.3930], locale en-US)
    • Flutter version 3.20.0-2.0.pre.17 on channel master at C:\WIP\flutter_master
    ! Warning: `flutter` on your path resolves to C:\WIP\flutter\bin\flutter, which is not inside your current Flutter SDK checkout at C:\WIP\flutter_master.
      Consider adding C:\WIP\flutter_master\bin to the front of your path.
    ! Warning: `dart` on your path resolves to C:\WIP\flutter\bin\dart, which is not inside your current Flutter SDK checkout at C:\WIP\flutter_master.
      Consider adding C:\WIP\flutter_master\bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 6322fef7e0 (3 hours ago), 2024-02-05 21:53:12 -0500
    • Engine revision 808886312e
    • Dart version 3.4.0 (build 3.4.0-104.0.dev)
    • DevTools version 2.32.0
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[√] Windows Version (Installed version of Windows is version 10 or higher)

[√] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at C:\AndroidSDK
    • Platform android-34, build-tools 34.0.0
    • Java binary at: C:\Program Files\Android\Android Studio Hedgehog\jbr\bin\java
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-b2043.56-10550314)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.4.4)
    • Visual Studio at D:\DOWNLOADWORK\VS2022
    • Visual Studio Community 2022 version 17.4.33213.308
    • Windows 10 SDK version 10.0.19041.0

[√] Android Studio (version 2022.3)
    • Android Studio at C:\Program Files\Android\Android StudioGiraffe
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-10027231)

[√] Android Studio (version 2023.1)
    • Android Studio at C:\Program Files\Android\Android Studio Hedgehog
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-b2043.56-10550314)

[√] VS Code (version 1.86.0)
    • VS Code at C:\Users\ADMIN\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.80.0

[√] Connected device (4 available)
    • RMX2001 (mobile)  • EUYTFEUSQSRGDA6D • android-arm64  • Android 11 (API 30)
    • Windows (desktop) • windows          • windows-x64    • Microsoft Windows [Version 10.0.19045.3930]
    • Chrome (web)      • chrome           • web-javascript • Google Chrome 121.0.6167.140
    • Edge (web)        • edge             • web-javascript • Microsoft Edge 121.0.2277.83

[√] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

I'll be able to test this tomorrow on my personal desktop (windows) vs my work laptop (mac)

Additional checks: if I run project in release mode, the app size is normal (~20MB). So, this issue seems to be in debug mode only.
in release mode, the size of my application is only 27MB, so the issue probably only occurs in debug mode


Convert to using dep instead of glide

dep seems to be the way that the community is headed, so apig should switch too.
Additionally, glide segfaults for me when I try to build apig. Not sure why, but switching to dep fixed it.

Hi, sorry for delayed response!
This package is now maintained at https://github.com/shimastripe/apig
Please reopen in the new repo if you are still interested in.
Thanks!


Dropbear a SSH/SCP server allowing for remote access via WiFi.

Provides the ability to SSH/SCP into goggles via WiFi.  Could be used for both debugging and transferring files remotely via SCP/WinSCP.
Eventually this service will be relocated to the OS distribution once we have the ability to update the OS distribution.  For now we will distribute this service as part of the application binary archive.
Note: Host key file will be automatically generated upon the initial installation and each user will have their own private host key.

Frankly, i am still skeptical about this approach. Even for a interims solution it seems like a bad idea to me. One particular concern is that it will make the transition to a rootfs-shipped variant harder, as port 22 will already be in use.

If we ship a rootfs variant which has openssh built in it will create a conflict, but we can avoid this by specifying an alternate port when starting up dropbear. 10022 for example.  However, I believe we can avoid this all together by deprecating dropbear once a new rootfs-version is ready for release which would contain support for openssh.
Which leads me to the followup question, if we release a new rootfs variant, will we require a minimum version of the goggle application compatibility?  If so, then the Goggle Application will have deprecated dropbear and we can avoide incompatibility issues that would have occurred.
Hence, rootfs would require a minimum version of X.Y.Z for the goggle application which would not include dropbear.
The alternative solution is we don't use the goggle application to install dropbear, but this would require HDZ to officially update the HDZG_OS.bin to include this functionality instead and any potential maintenance or future updates can be isolated to the OS distribution vs having a convoluted application distribution which updates the installed OS.

Closing out this PR, Proposing an alternative method which will allow us to support various 3rd party services.


SOLR-15834: Films example readme needs updating and including useParams mention

https://issues.apache.org/jira/browse/SOLR-15834
Description
Ref Guide suggests useParams is documented in the films example, but it isn't.  Now introducing that.
Solution
Updating the README and the CLI.
Tests
Manual
Checklist
Please review the following and check all that apply:

[X ] I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
[ X] I have created a Jira issue and added the issue ID to my pull request title.
[ X] I have given Solr maintainers access to contribute to my PR branch. (optional but recommended)
[ X] I have developed this patch against the main branch.
[ X] I have run ./gradlew check.
[ ] I have added tests for my changes.
[X ] I have added documentation for the Reference Guide

@HoustonPutman love any thoughts on how I've tweaked the demo...  I haven't ripped out the use of the schemaless guessing stuff, but could if you think I should.


Peer Disconnect Reconnect Replication Issue

Tested versions

Reproducible 4.2.1, 4.3-dev4

System information
Ubuntu 22.04 - Godot 4.2.1
Issue description
If a MultiplayerSynchronizer's SceneReplicationConfig is modified via script and client disconnects and reconnects. Previous multiplayer unique id replication is still cached and throughs errors. Newly connected Clients  are no longer replicated.
If replication_config is not touched with script there is no issue.
Steps to reproduce
Peer Connects
Create "player" with MultiplayerSynchronizer

add new property to replication_config
Disconnect player
Reconnect player

Minimal reproduction project (MRP)
client_reconnect_issue.zip
Project has interface to reproduce issue.
Read short instructions watch output.
Example output
Server started putting api on node /root/main/Server
Client started putting api on node /root/main/Client
Client : peer connected 1
Server : peer connected<PHONE_NUMBER>
Client: _exit_tree()
Server : peer disconnected<PHONE_NUMBER>
2076042518: removing peer
Client started putting api on node /root/main/Client
Client : peer connected 1
Server : peer connected 601296107

E 0:00:09:0138   multiplayer_api.gd:42 @ _on_peer_connected(): Node '/root/main/Server/2076042518:dynamic_sync' not found.
  <C++ Error>    Condition "!node || !node->has_node(p_path)" is true. Returning: nullptr
  <C++ Source>   modules/multiplayer/multiplayer_synchronizer.cpp:41 @ _get_prop_target()
  <Stack Trace>  multiplayer_api.gd:42 @ _on_peer_connected()

E 0:00:09:0138   multiplayer_api.gd:42 @ _on_peer_connected(): Parameter "obj" is null.
  <C++ Source>   modules/multiplayer/multiplayer_synchronizer.cpp:164 @ get_state()
  <Stack Trace>  multiplayer_api.gd:42 @ _on_peer_connected()

E 0:00:09:0138   multiplayer_api.gd:42 @ _on_peer_connected(): Unable to retrieve spawn state.
  <C++ Error>    Condition "err != OK" is true. Returning: err
  <C++ Source>   modules/multiplayer/scene_replication_interface.cpp:516 @ _make_spawn_packet()
  <Stack Trace>  multiplayer_api.gd:42 @ _on_peer_connected()

E 0:00:09:0138   multiplayer_api.gd:42 @ _on_peer_connected(): Condition "!p_buffer || p_size < 1" is true. Returning: ERR_INVALID_PARAMETER
  <C++ Source>   modules/multiplayer/scene_replication_interface.cpp:462 @ _send_raw()
  <Stack Trace>  multiplayer_api.gd:42 @ _on_peer_connected()

 endless repeat of errors...

E 0:00:11:0154   _get_prop_target: Node '/root/main/Server/2076042518:dynamic_sync' not found.
  <C++ Error>    Condition "!node || !node->has_node(p_path)" is true. Returning: nullptr
  <C++ Source>   modules/multiplayer/multiplayer_synchronizer.cpp:41 @ _get_prop_target()
E 0:00:11:0154   _watch_changes: Node not found for property '/root/main/Server/2076042518:dynamic_sync'.
  <C++ Error>    Condition "!obj" is true. Continuing.
  <C++ Source>   modules/multiplayer/multiplayer_synchronizer.cpp:389 @ _watch_changes()

I've made some progress
When adding  a property to the config here are some recommendations:

should use relative NodePaths and not absolute, or relative, Strings.
should create new SceneReplicationConfig (SRC) if using dynamic properties.

it is unknown if a fresh new SRC should be used every time the config is updated

or is it a consequence of the SRC resource that is part of the .tscn part of the issue?

If I use relative NodePaths over Strings and create a new config the issue goes away. There is one consequence in doing this, the old editor SRC is overwritten. Which makes sense since I'm using a new SRC in its place.

Using those recommendations above, minus making a new config and using the scene generated one. I think the issue is resolved.
I have made version 2
client_reconnect_issue_v2.zip
the last thing that keeps happening is every time I add a property to the SRC it throws an error.
E 0:00:08:0680   syncer.gd:20 @ _ready(): Condition "properties.find(p_path)" is true.
  <C++ Source>   modules/multiplayer/scene_replication_config.cpp:115 @ add_property()
  <Stack Trace>  syncer.gd:20 @ _ready()

Happens once on first connect, twice on second connect

I did a little more debugging. It seems like the replication_config is cached somewhere for the MultiplayerSynchronizer even if its deleted from the scene tree.
I got it down to one error relating to "Condition 'properties.find(p_path)' is true." if I only allowed the server side to add the new parameter when the client first connects. This makes sense if the only the server creates it, I assume it gets replicated in some fashion to the client.  (I tried switching the order of client server on who added the new property and it seemed the error only happened on whoever was the second actor to add the property. It turns out the client can add the property determine what is replicated! 😮)
Then, if the client disconnects and the syncer is deleted, upon returning when the server creates a new syncer, the added property already exists in the replication config, throwing the error message.
This then explains the original issue relating to the old player node path still existing when using an absolute path, after a new client reconnects. The old property still exists with the old absolute path!

I have also noticed that the MultiplayerSynchronizer is being cached, and even after it's deleted, errors continue to appear related to a MultiplayerSynchronizer that no longer exists. Additionally, when the MultiplayerSynchronizer was still in place but I removed one of its fields, I kept encountering the same error, likely because it remains cached somewhere. I should be able to debug this issue.
E 0:00:16:0955   MultiplayerSynchronizer::_watch_changes: Node not found for property 'Camera3D/Flashlight:visible'.
<C++ Error>    Condition "!obj" is true. Continuing.
<C++ Source>   modules\multiplayer\multiplayer_synchronizer.cpp:390 @ MultiplayerSynchronizer::_watch_changes()


PPH database migration, updated with current branch

Description:
Added DB models for the PPH Salesforce integration, seed scripts for those tables and updated seeding routines. Re-added the original Wondertix models to ensure API stability.
Per Johnny:
"invite me, Eben, Ben Williams, Justin, Henry, Jaeger and Adam."
Risks:
Some of the seeding routines will compare the data given to it, and if no relation can be made, it will skip over that instance of data. Johnny approved this step, as one-sided relations broke the seeding process- changing the fields to optional or not requiring a relation didn't fix this.
Because of this, dataset validation and integrity is key to this working appropriately. At present, a lot of the DEV seed code isn't linked appropriately to the other tables' relations, which leads to a lot of seeded data being skipped over. Production tables have a better success rate.
Validation:
I've had to beef up the seeding scripts to sanitize data as it came in. Salesforce had a lot of mixed data types in any given field (I suspect all of their data is sanitized to strings), or dangling nodes of data that were probably assigned to a catch-all account.
Over several days I added changes to the seeding scripts to make them more robust. I checked it with the deployment tables, the dev tables, and instances of custom data to ensure they worked.
Operating System:
Mac, M2-series CPU

Hey all, Johnny wanted me to re-push with a more updated version of Main.
Made the changes, confirms it works, it's here for review!

Hey all, Johnny wanted me to re-push with a more updated version of Main. Made the changes, confirms it works, it's here for review!

Thanks for getting the latest version of main in this PR!

At present, a lot of the DEV seed code isn't linked appropriately to the other tables' relations, which leads to a lot of seeded data being skipped over. Production tables have a better success rate.

Does this mean that you've tried with real patron manager data? I'd be curious what the success rate was. To your point about data integrity/validation, it sounds like we might have to manually add ID values to foreign key vales in some instances so that some items don't get skipped?

At present, a lot of the DEV seed code isn't linked appropriately to the other tables' relations, which leads to a lot of seeded data being skipped over. Production tables have a better success rate.

Does this mean that you've tried with real patron manager data? I'd be curious what the success rate was. To your point about data integrity/validation, it sounds like we might have to manually add ID values to foreign key vales in some instances so that some items don't get skipped?

Hey Ben,
I did try with the actual dev data, which had a higher success rate. I'll look into getting you a %.
I think there's some dataset relation errors in the dev data, but it's only a few tables that have this issue.


Module resolution - four issues (specific and simple to general)

I know there have been plenty of questions around module resolution, I read some of them (SO and here). I still have these issues: The most general one last, the most specific one first.
1 Fix documentation
From page https://www.typescriptlang.org/docs/handbook/module-resolution.html

Classic
This used to be TypeScript’s default resolution strategy. Nowadays, this strategy is mainly present for backward compatibility.

But "Classic" still is the default, no?
This sounds like it should not be the default.
2 Don't ascend above project directory
(At least not by default, or provide an option to prevent it.)
Why is TS allowed, by default, to ascend up out of the project directory?
I have a folder Projects/ with various projects, some included by others, but via packages and not directly from those directories but they have the same name locally. What happens is that when I run tsc in one of those projects it ends up including definitions from another project instead of from node_modules/, which makes no sense to me. I mean I get it if that was an optional behavior, but by default going into folders that don't belong to the project?
How can I prevent tsc from looking for anything outside the current project's directory?
3 Include node_modules in module resolution
What I mean is that it should always be used, or at least make it configurable but not as it is now where it depends on module resolution "node". The use of node_modules is not node.js specific at this time.
I'm also confused because TS does find things <PROJECT_ROOT>/node_modules, it only finds @types imports. I bundle .d.ts files in my own library, which since it's internal and private has no way to get its .d.ts published via DefintelyTyped. When I import from "myLib/lib/someFile", for which a .js and a .d.ts and a .d.ts.map file exists, it only finds it in "node" module resolution mode.
It may as well include that folder in its search even in "Classic" mode, that IMHO makes more sense than the search outside the project directory.
4 Module resolution depends on build target
First I checked if this still applies, and it does.
In reference to:

https://github.com/Microsoft/TypeScript/issues/8189#issuecomment-212082894
https://github.com/Microsoft/TypeScript/issues/18865#issuecomment-333412703 ("An issue here is that the module resolution strategy we use is determined by your compiler options... ")

I don't see that mentioned in the docs (page linked above)
I don't understand the purpose

At least for me, the build and development are orthogonal. I build for browser, react-native, node.js and experimentally even for two embedded platforms (Moddable, low.js) and I don't see why my code structure used for development should depend on the build target! After transpiling (here using Babel, but that would be the same if we used TS for that) module resolution on the various platforms is vastly different. From webpack-packages to a "flat" module namespace (Moddable, many platforms use their own very different module handling. Whether Webpack or Moddable find a module has nothing whatsoever to do with TypeScript used while writing the code and how it type-checks and finds modules.
IMHO development time module resolution should not depend on the build target.
My own build target is "esnext" (I also have "noEmit": true because I only use TS at development time, if there is any transpilation it is done by Babel, usually that's just removing the type annotations and maybe a — flexible — module system target), but even node supports ES2015 modules these days, so I don't think there is the direct connection between "commonjs" and "node_modules" any more, if ever there was.
Especially since the support of the Babel option the distinction that already and always existed between development with type checks (when the type checker needs to be able to find a) included modules, but b) also just type info like .d.ts files which have no runtime equivalent and there location surely should not depend on what the build target is), transpiling and packaging, and running the code, probably deserves another look to remove unnecessary cross-concern bindings like module resolution (used for type checking) depending on transpilation (which does not even need to know about where to find modules since it is in between type checking and runtime).
On a more general note, I think TS has too tight a coupling because those concerns, which despite all efforts I can see even here in the "Issues" in reaction to feature requests where anything that would make TS different from JS is routinely rejected, leads to a lot of confusion esp. among new(er) developers (to the ecosystem, not necessarily by age) that TS is a different language. Especially since Babel workflows are officially supported too now the more clear separation of development time (type checking) and transpilation might be beneficial. It's not even a code change, maybe a few tsconfig layout changes at most.

As an aside, there is even more of a disconnect between development and runtime. I, for example, use TSypeScript only for type checking. It's a setup I could also run with Flow, same for anyone that uses Babel for transpilation (many people don't seem to know, all the Babel-TS plugin does is remove the types, because then TypeScript code is just "ESnext" (if one doesn't use namespaces, enums, import=/export=).
That means I use tsc only from the IDE as language service and for type checks. It (tsc) only needs to understand the layout of the code during development. It does not need to concern itself with either transpilation or where modules might be found during runtime — especially the latter, in the days of Webpack & Co., is something tsc does not need to know. So finding files and types (.d.ts) during development should be separate from finding code modules during runtime, and TS should focus on the former (the latter is the domain of various build systems anyway, which have their very own ideas).
That means that, for example, I can use TS-only features like namespaces for my types (inside a .d.ts file) even if I would not or can't use it in my code. In my particular case it helps with type checks (declaration merging between a private lib an apps using it, just my particular private case and example).
So in my — I would claim not unusual — setup, 1) type checking (by tsc), 2) transpiling (by Babel), 3) packaging (Webpack, Moddable compiler, React Native, and others) and then 4) the actual runtime all are disconnected. This is in line with the architecture of TS I think, the merging of concerns is, I think, a product of user-facing components. I don't think tying those components together, for example by binding module resolution (used during development for type checks) on a build target (used only when actually building, and even that does not necessarily need to be done with tsc since there is the Babel option) is not good.

@RyanCavanaugh @DanielRosenwasser Whats the action needed here? I am not sure what we can do here?

@lll000111 there's a lot to unpack here and this seems like a mix of reactional feedback (something didn't work the way you expected it to - this is fair to note, but generally unactionable on our side), possible bugs (the docs should be accurate if they're not), design change proposals (e.g. decoupling target/module resolution), and feature proposals.
It'd be useful on our end if you could be more clear about what you expect to be different, and ideally in separate issues.

But "Classic" still is the default, no?

It is unless you use --moduleResolution node. It is correct that there's a doc update needed, but we're working on new docs.
There's also an open issue about this: https://github.com/Microsoft/TypeScript/issues/11434

Don't ascend above project directory

You can control this with --rootDir, and you'll get an error if you resolve outside of the rootDir.

What I mean is that it should always be used, or at least make it configurable but not as it is now where it depends on module resolution "node". The use of node_modules is not node.js specific at this time.
I'm also confused because TS does find things <PROJECT_ROOT>/node_modules, it only finds @types imports.

This is because of the typeRoots option walking up the directory chain for node_modules/@types directories. You can explicitly include node_modules as well as node_modules/@types for your typeRoots. See more here.


Question_2a by Nikhil

Accepting the solution but below changes are suggested

Unit test cases should be written for all functions.
NULL check is missing in below function implementations for all parameters passed.

error_t open_file(char* address, char* mode, FILE **ptr);
free_struct(info_t *temp);
readLine(FILE **ptr, char* data);
void allocAndStore(char **data, char *token);

Can avoid the usage of double pointers.
Can add doxygen based comments in header file.
Can add Github actions for build, unit test, analysis in README.md file of your solution file.
Function open_file is not required as it is only doing fopen (it returns NULL if fails).
Use strncpy instead of strcpy (unsafe if string is not null terminated).
Use single naming convention file_data (snake case) or allocAndStore (camel case).


SCIM2 Roles Patch - Replace operation not working for users and groups

Describe the issue:
If we invoke a PATCH call for /t/<tenant_name>/scim2/v2/Roles/<role_id> or /t/<tenant_name>/o/scim2/v2/Roles/<role_id> with replace operations, the response returns 200 OK but the users and groups are not updated.
Eg: The users that are assigned to the role with id 128e2f13-9472-4673-959c-a7fc61aa0d9b before and after the below call are the same.
curl --location --request PATCH 'https://localhost:9443/t/carbon.super/o/scim2/v2/Roles/128e2f13-9472-4673-959c-a7fc61aa0d9b' \
--header 'Content-Type: application/json' \
--header 'Authorization: Bearer <token>' \
--data '{
    "Operations": [
        {
            "op": "replace",
            "value": {
                "users": []
            }
        }
    ],
     "schemas": [
        "urn:ietf:params:scim:api:messages:2.0:PatchOp"
    ]
}'

How to reproduce:

Expected behavior:

Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: [e.g., IS 5.10.0, IS 5.9.0]
OS: [e.g., Windows, Linux, Mac]
Database: [e.g., MySQL, H2]
Userstore: [e.g., LDAP, JDBC]

Optional Fields
Related issues:

Suggested labels:

replacing empty list / replacing another set of values doesn't work properly

completed with

https://github.com/wso2/carbon-identity-framework/pull/5153
https://github.com/wso2-extensions/identity-inbound-provisioning-scim2/pull/506


Mention removal of inline case in 0.16 upgrade docs?

The upgrade docs don't currently mention that the syntax
f = (\x -> case x of { Just x -> x ; Nothing -> 0 })

was abolished.
Was omitting this mention accidental, or was this syntax change simply not considered major, since inline case was rarely used anyway?

Nevermind.


waymo training weight

According to the official rule of Waymo dataset, we can not provide the trained weight of Waymo. You can just use the script to train from scratch.


Display footer email via feature flag

https://www.pivotaltracker.com/n/projects/2481183/stories/184034247

https://github.com/ideacrew/quoting_tool/pull/28


<EMAIL_ADDRESS>@callstack/repack/commands error

Environment
react-native.config.js
commands: require('@callstack/repack/commands'),

[!] Invalid `Podfile` file: 859: unexpected token at 'TypeError: Cannot read properties of undefined (reading 'description')

    at Object.<anonymous> (/xxxxx/node_modules/@callstack/repack/commands.js:62:32)

Description

Reproducible Demo

Hello! Did you find any fixes yet? IMO it will take time to shift everything to KT.

Fix merged, new version of Re.Pack should be available later today 🚀


Add support for creating web workers from web extensions served from VS Code Server

Does this issue occur when all extensions are disabled?: No

VS Code Version: VS Code Server 20220909.27
OS version: Ubuntu 22.04.1 LTS

Steps to Reproduce:

Install VS Code Server on a local machine
Install devsense.phptools-vscode extension
Fails with Creating workers from remote extensions is currently not supported.
Any idea what is the difference in vscode.dev that allows this extension to run, but not in VS Code Server?

Thank you

@bilogic Looking at where we throw the error: https://github.com/microsoft/vscode/blob/c6df3c99687de544fcda33483763afeeb64cd060/src/vs/workbench/api/worker/extensionHostWorker.ts#L100-L103
From the web clients point of view, this situation can be encountered when a web extension is served from the vscode server and the web extension tries to create a new WebWorker. Web extensions which are served from the vscode server have the URI vscode-remote:. This URI cannot be synchronously resolved by the web worker, because it needs to travel to the main thread to be resolved.
On vscode.dev we avoid this situation by serving web extensions from vscode's unpkg service which makes it that the web extensions have the URI https:.
What are exact steps that you use to reproduce this? My understanding is that if you use the web ui to install a web extension, it will be "installed" in the web browser and will use the unpkg service and https:?
Related: #144513

@alexdima
My exact steps as follows based on https://code.visualstudio.com/blogs/2022/07/07/vscode-server:

Installed vscode-server using wget -O- https://aka.ms/install-vscode-server/setup.sh | sh
Ran it with code-server serve-local --without-connection-token --port 8001
And I accessed VSCode from Chrome thru a reverse proxy from https://domain-name.com
Installed the plugin via VSCode's Install from VSIX...

May I know what is this web ui that you refer to? If I have a VSIX file, how do I get it installed in the web browser? Thank you!

@alexdima

To give a bit more context, in my code-server, I see Install Locally, but at https://vscode.dev, I see "Install"
Install Locally fails, which made me go for Install from VSIX... (which succeeds but can't create new webworkers)
https://vscode.dev's Install will succeed, this is what I need to make happen on my own code-server
Perhaps the question now becomes, how do I make code-server install extensions in the web browser?

Thank you!

Is there more info required from me?

@alexdima there is a info-needed label, but I'm not sure what info is required from me. Can let me know? Thak you.

Hey @alexdima, this issue might need further attention.
@bilogic, you can help us out by closing this issue if the problem no longer exists, or adding more information.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


504 and 502 Error Message are not rendered correctly

 Issue by do4gr
Thursday Sep 07, 2017 at 16:51 GMT
Originally opened as https://github.com/graphcool/api-bugs/issues/263

The messages are turned into a json.

@do4gr Is this in the CLI?

Yes. In case of a long running deploy there is a timeout message returned from cloudfront. This is then not rendered correctly.

This issue was moved to graphcool/graphcool-cli#176


fix: Disable duplicate management events for Lambda & S3 Cloudtrail event selectors

Issue #, if available: #449
Description of changes:
Prevents duplicate management event selectors being created in the organisation Cloud Trail when Lambda & S3 event selectors are enabled via the global logging config.
Resulting CloudTrail event selector configuration:
aws cloudtrail get-event-selectors --trail-name AWSAccelerator-Organizations-CloudTrail
{
    "TrailARN": "arn:aws:cloudtrail:eu-west-2:account-id:trail/AWSAccelerator-Organizations-CloudTrail",
    "EventSelectors": [
        {
            "ReadWriteType": "All",
            "IncludeManagementEvents": false,
            "DataResources": [
                {
                    "Type": "AWS::Lambda::Function",
                    "Values": [
                        "arn:aws:lambda"
                    ]
                }
            ],
            "ExcludeManagementEventSources": []
        },
        {
            "ReadWriteType": "All",
            "IncludeManagementEvents": false,
            "DataResources": [
                {
                    "Type": "AWS::S3::Object",
                    "Values": [
                        "arn:aws:s3:::"
                    ]
                }
            ],
            "ExcludeManagementEventSources": []
        },
        {
            "ReadWriteType": "All",
            "IncludeManagementEvents": true,
            "DataResources": [],
            "ExcludeManagementEventSources": []
        }
    ]
}

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Updated for compatibility with 1.8.0 release. Issue still persists.

Hi @rjbaker , thank you for your contribution to the Landing Zone Accelerator!
I'm pleased to say this work has been integrated into the code base and will be available shortly in the next release. I'll close this PR in the meantime and update the associated issue once that release is publicly available.
Thank you for your support of the LZA!

Good news :tada: . @nagmesh is there a roadmap or any future release dates you're aiming for?

@mbevc1  - dare to say end of year
https://github.com/awslabs/landing-zone-accelerator-on-aws/issues/320#issuecomment-2433175786

Thanks! :crossed_fingers:

Would be nice if you could open a roadmap and move development to this repo at some point :wink:


Include assembly files in auto-generated CMakeLists

This is a feature enhancement, to handle assembly files. It looks like they are currently not included by default.

This was fixed by f02b99277fca0161559af00387cbbeee68a4ccab

:+1:


Improved texture creation helpers

Also, fixing this error with the previous helper:

thread 'main' panicked at 'called Result::unwrap() on an Err value: Texture(Invalid data size provided to update the texture, expected size 0)', /home/rustbuild/src/rust-buildbot/slave/nightly-dist-rustc-linux/build/src/libcore/result.rs:750

Merging since it is fairly straightforward, and my work depends on it now.


Change --other to --others in ls-files

Per an email report, the cheat sheets currently show the use of --other instead of --others as an option for the ls-files command. The online documentation shows that --others is the correct option. This fixes the .md files, but leaves the PDFs untouched for now. This will also need to be fixed in #403.
Will :ship: with one approved review!
/cc @github/services-training FYI

Thank you for reviewing @brianamarie :ship:ing!


where is the route command

https://github.com/fabiolb/fabio/wiki/Routing#manual-overrides
what's a route command ? where is the route binary ?
How can I execute it .

It's not a command.  That is just config syntax.  The urlprefix- tags are actually converted into fabio route syntax when processed.  The actual route syntax can be used within the config file or placed in the Consul K/V to provide static routing information.

so I need execute it as
consul kv put "fabio/config"  'route add srv1 / http://domain.com opts "host=dst" '

is it right ?

@suconghou you should not need to, no. Fabio will create routes for you, based on the tags you've added to your services (registered in consul). Let's say you run a couple services on systemd and a couple on nomad. For each of those services, you make sure you define one or more tags for fabio, that tell fabio how to route requests for that service. If you use nomad, you can have it add those tags as part of the service registration with consul. If you use systemd, you are registering the service yourself, and will need to include the tag(s) you want for each service. The tag(s) provide fabio the info it needs to create the routes.
See here from the main page:

When you register a service in Consul all you need to add is a tag that announces the paths the upstream service accepts, e.g. urlprefix-/user or urlprefix-/order and fabio will do the rest.

@ketzacoatl It is awesome , but It needs all my service  modify code to adapt to consul, It is inbreaking .
can Fabio work without consul and use http rest api to create route ?

It’s just as easy to register the application as a service in consul.  Consul has a REST API as well.

@suconghou yea, I'm not sure what you are referring to - what about your services need to change? For this service registration, you can use Consul's HTTP API, or even easier, if you are running the consul agent on the nodes where the services run, you can add a JSON file to the config and register services and health checks that way. Consul is very flexible in this regard, and you have a lot of leeway to make it work for you in your unique constraints. I've yet to be totally stuck.


Build out videos object #19

Needs some work with melos, for example some of the text is too small and the text in the button isn't centred.

Not sure why this is closed?


JAMES-3680 Make JMAP authentication strategies modular and configurable

I find it really weird to use FQDN for the strategy name. I sounds a lot like a Spring re-implementation. Can't we leverage SPI to discover auth implementation and call them by their friendly names?

Hello @mbaechler

I find it really weird to use FQDN for the strategy name.

That is what we used in all places within James.
We can likely do something like mailets by having an implicit FQDN prefix, just like mailets.
Would it solve your concerns?

Rebased + squash up fixup to avoid more boilerplate commits. 3 lastest fixup comment for fixing Benoit's comment.

@quantranhong1999 a rebase is needed...


Update schema flow for recent changes

relevant api PR is here: https://github.com/docknetwork/dock-api/pull/152 - we can merge this providing those changes are accepted otherwise more are needed here


Volt::ReactiveArray#[] does not return nil for out of bounds index

The current code fails if index is out of bounds, unlike ruby array which will return nil.
Suggestion:
    def [](index)
      # Handle a negative index, depend on size
      index = @array.size + index if index < 0

     # NEED THIS
     return nil unless index < @array.size

      # Get or create the dependency
      dep   = (@array_deps[index] ||= Dependency.new)

      # Track the dependency
      dep.depend

      # Return the index
      @array[index]
    end

Won't @array return nil if it's out of bounds on that?  I'm on my phone, so I can check later.

but it fails on @array_deps[index] ||= Dependency.new

Ah, I'll fix :-)

Ok, I still don't understand why this would fail.  Its doing ||=, so it will create one at the index.  Do you have some example code that is failing?

@ryanstout well, setting an element out of bounds does work in both MRI and Opal, but you may or may not want this:
$ pry
[1] pry(main)> a = [1,2,3]
=> [1, 2, 3]
[2] pry(main)> a[42] ||= 4
=> 4
[3] pry(main)> a
=> [1,
 2,
 3,
 nil,
 nil,
 nil,
 nil,
 nil,
 nil,
 nil,
 nil,
 nil,
 nil,
 nil,
 nil,
 nil,
 nil,
 nil,
 nil,
 nil,
 nil,
 nil,
 nil,
 nil,
 nil,
 nil,
 nil,
 nil,
 nil,
 nil,
 nil,
 nil,
 nil,
 nil,
 nil,
 nil,
 nil,
 nil,
 nil,
 nil,
 nil,
 nil,
 4]
[4] pry(main)>

Also, returning nil if index < 0 is probably a good idea - and there's already a PR for that: https://github.com/voltrb/volt/pull/259/files

@vais Ruby arrays behave the same way.  Or am I missing something?

@ryanstout yes, they behave the same way in Ruby and in Opal. My point is that you may not want this behavior in this particular case. I.e. you will end up with a bunch of nils filling the gap.

@ryanstout you learn something every day :smile: I had no idea ruby padded an array out with nils if you set an element beyond its size.
I got a failure client side - so may be it's the Opal implementation. I was calling #last on empty array. Definitely failed at the ||=. Can reproduce if you need.
I think @vais has a point though - do you want @array_deps array to lose one-to-one correspondence to @array?

@balmoral Can you give me more info so I can duplicate?  (code would be good)
Also for @vais    I think maybe I haven't explained the way its written as it is now.  I don't want to just return nil if its out of bounds, because we want the missing value to be reactive.  So if the value was out of bounds, but becomes in bounds (because someone sets it to a value), we want the out of bounds nil to reactively update inside of a watch!  So to make that work, we need to keep track of the deps even for out of bound indexes.
Perhaps however I should switch @array_deps to a hash, so it could be sparse.  Thoughts?

Also, sorry it took me a while looking at this to figure out what you were talking about :-)

@ryanstout sorry my suggested fix above was wrong, but still an issue:
It's a simple failure which will happen any time you access array.last or array[-n] on an empty ReactiveArray or ArrayMode (or when size - n is less than 0).
My code is page._charts.last and I'm expecting it to return nil if the _charts array is empty.
Error on console is:
Uncaught IndexError: index -1 too small for array; minimum 0

and it occurs on the line
      dep   = (@array_deps[index] ||= Dependency.new)

This happens because index remains negative after the preceding code
     # Handle a negative index, depend on size
      index = @array.size + index if index < 0

and fails because it's trying to set a value at a negative index.
So maybe the patch should be
   return nil if index < 0

and this shouldn't interfere with the reactive logic you're handling.

@balmoral I just fixed this on master actually :-)  Please reopen if your still seeing issues.

@ryansout my timezone keeps me a bit out of sync. As always, I appreciate the great service :smile:

@balmoral no problem.  Thanks for letting me know about the issue.  Should have a pre-release gem out soon.


Remove left-hand table of contents

The structure/content of the existing document includes navigation (world image-map, clickable links, table of countries, table of classes).
So, the left-hand TofC becomes a waste of screen space.
We should remove it. Hopefully we can do that via some property in the scenario.
(left-hand table of contents removed)

If we don't have the left-hand table of contents, and all navigation is via in-page links, then I don't think we need to maintain the ditamap of content within the regions folder.  The home page just need to include a reference for regions.dita - and navigation happens from there.
I've just checked, and the WebHelp process is indexing files that aren't in the ditamap (such as french power units).  So, it looks-like we don't have a strong reason to maintain the region/country ditamaps.

Done.


about the GPU memory

I am trying to train the model with the code:
python3 contrastive_train.py --use-cuda --use-intensity --segment-contrast --checkpoint segcontrast

but I found that GPUs have different loads,just like

this prevents me from setting bigger batch_size to speed up my training, is there any way to sovle this?
I am looking forward to your reply!Thank you !

I also noticed the same behavior in mine, this is because of pytorch lightning accelerator for the parallel processing. You can maybe try using a different accelerator here

I also noticed the same behavior in mine, this is because of pytorch lightning accelerator for the parallel processing. You can maybe try using a different accelerator here

hello, I found that, if I change the code
        self.model_q = model(in_channels=4 if args.use_intensity else 3, out_channels=latent_features[args.sparse_model]).type(dtype)
         self.head_q = model_head(in_channels=latent_features[args.sparse_model], out_channels=args.feature_size).type(dtype)

        self.model_k = model(in_channels=4 if args.use_intensity else 3, out_channels=latent_features[args.sparse_model]).type(dtype)
        self.head_k = model_head(in_channels=latent_features[args.sparse_model], out_channels=args.feature_size).type(dtype)

as
        self.model_q = model(in_channels=4 if args.use_intensity else 3, out_channels=latent_features[args.sparse_model])
        self.head_q = model_head(in_channels=latent_features[args.sparse_model], out_channels=args.feature_size)
        self.model_k = model(in_channels=4 if args.use_intensity else 3, out_channels=latent_features[args.sparse_model])
        self.head_k = model_head(in_channels=latent_features[args.sparse_model], out_channels=args.feature_size)

the GPU load is balance, just like

se_model], out_chan


SAA-1502 adding logging to the WebClientConfiguration to monitor usage/instance creation in particular for @RequestScope beans.

Adding to configuration logging for monitoring purposes to see if we need the config the way it is or whether it can/should be changed.

Codecov Report
Attention: 4 lines in your changes are missing coverage. Please review.

Comparison is base (2dd5957) 86.97% compared to head (c7b5221) 86.96%.

Files
Patch %
Lines

...tiesmanagementapi/config/WebClientConfiguration.kt
81.81%
4 Missing :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #759      +/-   ##
============================================
- Coverage     86.97%   86.96%   -0.01%
- Complexity     3435     3436       +1
============================================
  Files           358      358
  Lines         11936    11945       +9
  Branches        999      999
============================================
+ Hits          10381    10388       +7
- Misses         1091     1093       +2
  Partials        464      464

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Test creation: rabbit_callback

NorthboundControllerAbstraction needs better coverage for testing.
add testing for rabbit_callback to test_poseidonMonitor.py

I'll take that


update content/ja/docs/reference/glossary/kube-apiserver.md

fixes #21298

@chez-shanpu Thank you for contribution!
Please sign CLA following the instruction above. Note that you need to sign 2 types of CLA.

I signed the CLA.

/assign @cstoku

/approve


Update windowsservercore versions

@mcy94w,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


chore: force interface validation

Closes #644
Forces interface validation even if we didn't call it yet. For already implemented (or sufficiently implemented) providers this would not make much of a difference, but it helps a lot when you are developing a new provider, and did not reach the parts where structs are actually being assigned to comply with interfaces. With that said it makes sense to have this at the top of every provider so people implementing new ones can look and know that this is a thing

/merge

/merge

lol @knelasevero at the same time

Double simultaneous merge 🔥💥

Lol Paul got mad


Feedbacks import problem #2

Gorse version
last version
Hi, I still have the same issue, even if I put auto_insert_item = true in the configuration, feedbacks cannot be imported. Can you help me please ?
Describe the bug
I imported my feedbacks in the Import feedbacks window on Gorse. However, in the Gorse Dashboard Overview I do not have any feedback displayed. Moreover, then, I exported the feedbacks from Gorse and there is literally nothing in there. It is as if Gorse fakes to import the feedbacks, but in reality it does not. I don't think it's a TimeStamp Type issue (but maybe it is), as I tried with many different types and it still doesn't work.
Even when I try with a csv file of 5 lines like this one it does not work :
favori,UserId,ItemId,Timestamp
favori,3423,186758,2022-06-24 21:20:11
favori,3423,186758,2022-06-24 21:20:13
favori,2823,186085,2022-06-24 21:20:19
favori,2140,185965,2022-06-24 21:20:20

Duplicate of #517


subgroup_functions_ext: fix missing flag initialization

Add the initialization of test_params.use_core_subgroups flag to the value passed by the function call. This fixes a segmentation fault caused by the call of an undefined clGetKernelSubGroupInfo function for OpenCL versions <= 2.0.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Add Payhawk.com VDP

Summary
Adding payhawk.com/security/disclosure program.


Native Color Temperature not working

Describe the bug
Under display settings, the “Use Native Color Temperature” no longer works. It used to change the screen color when I toggled it on and off.
Manually adjusting the color is not functioning either.
What did you expect to happen?
Color temperature should change when toggling it on or off.
Output of rpm-ostree status
No response
Hardware
Legion Go
Extra information or context
No response

I also heard that this isn’t working on a GPD 4


parameter includes colon when matched in the middle of a url part

Version
5.1.2
Test Case
https://codesandbox.io/s/react-router-forked-gb036
Steps to reproduce
if attempting to match a path like /path/foo:id the parameter returned when using /path/foo:bar is prefixed by a colon
Expected Behavior
there should be no colon prefixing the parameter value
Actual Behavior
there is a colon prefixing the parameter value

This is expected behavior. The colon is part of the parameter definition. So you have essentially /path/foo{id}, therefore the colon is a part of that parameter value in the URL /path/foo:bar


Update link to GPU texture compression intro

Unfortunately the domain on the original link has expired — I've replaced the link with one from the Wayback Machine.

Oops, duplicate of https://github.com/BinomialLLC/basis_universal/pull/357! :)


Logic in NativeXr.cpp and XR.cpp should be split into multiple files

Looking into NativeXr.cpp and XR.cpp, a lot of classes are currently defined in a single file. This does not make for great legibility. For example, function definitions in XR.cpp (OpenXR) for System::Sesssion, System and System::Session::Frame are mixed in order throughout. I believe it would improve legibility if we started splitting this logic into separate files.

Reopen if we find this to be annoying and want to split.


Java doc for remote netty4

What is the purpose of the change
add some comments for dubbo-remoting-netty4
Brief changelog
XXXXX
Verifying this change
XXXXX
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Run mvn clean install -DskipTests=false & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[ ] If this contribution is large, please follow the Software Donation Guide.

i have merged the latest code into your branch but still check fail.

And the problem happen second time at the same UT position

Codecov Report

Merging #4180 into master will decrease coverage by <.01%.
The diff coverage is 80%.

@@             Coverage Diff             @@
##             master   #4180      +/-   ##
===========================================
- Coverage     62.91%   62.9%   -0.01%
  Complexity      505     505
===========================================
  Files           769     769
  Lines         33060   33062       +2
  Branches       5218    5222       +4
===========================================
- Hits          20801   20799       -2
- Misses         9862    9866       +4
  Partials       2397    2397

Impacted Files
Coverage Δ
Complexity Δ

...main/java/org/apache/dubbo/remoting/Constants.java
100% <ø> (ø)
0 <0> (ø)
:arrow_down:

...bo/remoting/transport/netty4/NettyTransporter.java
100% <ø> (ø)
0 <0> (ø)
:arrow_down:

.../remoting/transport/netty4/NettyClientHandler.java
68.42% <ø> (ø)
0 <0> (ø)
:arrow_down:

...e/dubbo/remoting/transport/netty4/NettyServer.java
76.56% <ø> (ø)
0 <0> (ø)
:arrow_down:

...n/java/org/apache/dubbo/common/utils/UrlUtils.java
78.04% <ø> (ø)
0 <0> (ø)
:arrow_down:

...java/org/apache/dubbo/remoting/utils/UrlUtils.java
57.14% <ø> (ø)
0 <0> (ø)
:arrow_down:

...e/dubbo/remoting/transport/netty4/NettyClient.java
70.58% <100%> (ø)
0 <0> (ø)
:arrow_down:

.../remoting/transport/netty4/NettyServerHandler.java
71.42% <100%> (ø)
0 <0> (ø)
:arrow_down:

.../dubbo/remoting/transport/netty4/NettyChannel.java
65.88% <50%> (ø)
0 <0> (ø)
:arrow_down:

...ng/exchange/support/header/HeartbeatTimerTask.java
73.68% <0%> (-5.27%)
0% <0%> (ø)

... and 5 more

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 28bea9c...963c63b. Read the comment docs.


Fix dbtransfer and add unittests

I have implemented several tests for dbtransfer tool and made several fixes in QueryDatabase: add passing result_types and typeids to result object of MaterializedQueryResult type.

Thanks!

@hannesmuehleisen You are welcome! It's interesting for me to help this opensource project, even, still, with small changes.

Also PRs that make small changes are very welcome!


OperationNotSupported exception when Stream is not writable.

I get a OperationNotSupported exception when the stream passed to ExcelDataReader does not support writing.
Is this by design?
Thank you.

There is no code in ExcelDataReader that writes to the input stream. What does the stacktrace say?

I'm sorry for the confusion, it appears the actual problem is that the stream is not seekable. This is related to https://github.com/ExcelDataReader/ExcelDataReader/issues/431.
I don't understand the decision behind not making a ZipArchiveEntry not seekable when the ZipArchiveMode is set to ZipArchiveMode.Read.


Confirm breadcrumbs support

Descriptive summary
It appears that Hyrax bundles in some support for breadcrumbs via render_breadcrumbs
A more complete listing of breadcrumbs in Hyrax
Please:

Look into what this feature provides natively
Post screenshots demonstrating the scope of what "breadcrumbs" means in this context

Rationale
"Breadcrumbs" are a desired feature to bring into the new system that was not re-created in DAMS4.

@mcritchlow
The hyrax project use gem breadcrumbs_on_rails,
https://github.com/projecthydra-labs/hyrax/blob/1e504c200fd9c39120f514ac33cd42cd843de9fa/app/builders/hyrax/bootstrap_breadcrumbs_builder.rb
which in your controller, call add_breadcrumb to push a new element on the breadcrumb stack. add_breadcrumb requires two arguments: the name of the breadcrumb and the target path.
There is a class breadcrumbs.rb created in hyrax to handle it:
https://github.com/projecthydra-labs/hyrax/blob/master/app/controllers/concerns/hyrax/breadcrumbs.rb#L5
Currently the default_trail is like this, the label name is read from config/locales:
https://github.com/projecthydra-labs/hyrax/blob/master/app/controllers/concerns/hyrax/breadcrumbs.rb#L14
And there is breadcrumbs_for_collections.rb to define default collection breadcrumb behavior, and breadcrumbs_for_works.rb to define default object breadcrumb behavior:
https://github.com/projecthydra-labs/hyrax/blob/master/app/controllers/concerns/hyrax/breadcrumbs_for_collections.rb
https://github.com/projecthydra-labs/hyrax/blob/master/app/controllers/concerns/hyrax/breadcrumbs_for_works.rb
Attached screenshots show the default breadcrumbs behaviors for collection and objects.

After log in, the object and collection view breadcrumbs:

After log in, the collection and object edit action breadcrumb:

The public view for collection and object breadcrumb:

@hweng - thanks for posting these. If a user is not authenticated, will they see any breadcrumbs? Or are these breadcrumbs only shown for authenticated users and specific to their content?
I suspect all we'd want is essentially (at the object level) breadcrumbs that say:
Animal Collection / Bunny 1, for example.

@mcritchlow If a user is not authenticated, they can still see the breadcrumb, but the trail is different from the logged in user, see the above public viewer vs. logged in viewer screenshots:
For example:
Logged in: My Dashboard/My Objects/ Bunny
Not logged in:  My Objects/Bunny
For your 2nd question: No, the current hyrax behavior for object breadcrumb is that it doesn't include collection name in the trail:
For example:
The Bunny object which belongs to the Animal collection, it's trails is:
My Dashboard/My Objects/ Bunny
And the animal collection trail is:
My Dashboard/My Collections/Animal Collection
https://github.com/projecthydra-labs/hyrax/blob/master/app/controllers/concerns/hyrax/breadcrumbs_for_works.rb#L26

@hweng - I see, thanks for clarifying. Based on what you've learned so far, do you think it would be difficult to adjust/override so that, for example, the Object includes a breadcrumb back to the Collection?
And, to clarify, I think everything you've already provided covers the scope of this ticket. Any actual customization can be done in a new ticket, if we decide to go that route.

@mcritchlow I think it is possible to override the hyrax breadcrumbs behavior, but i'm not sure how difficult it is. I will look into it and figure out if we decide to go that route.

@hweng - Sounds like a plan. I think you can mark this Closed for now then, and we can chat with @gamontoya about when/if we may want to do any modifications via a separate ticket.

@mcritchlow I think we want to see if the modifications (to pull in the collection) can be done and creating a new ticket sounds perfect.

@gamontoya sounds great, it's created as #83


Move Connection.isHealthy() checks to where reuse happens

One behavior change here is we won't fire connectionAcquired()
followed by connectionReleased() if we take an connection from the
pool only to immediately find it's unhealthy.
I'm working towards moving the loop over connect attempts and
fails from RetryAndFollowUpInterceptor and into ExchangeFinder.
(We'll still need some recovery in RetryAndFollowUpInterceptor,
but not connection recovery.)

Behavior Change
We no longer recover if there’s a single route and it fails with a refused_stream immediately. I think this is an improvement.

One behavior change here is we won't fire connectionAcquired()
followed by connectionReleased()

Was this ever clear why it was released? I'm not concerned about the events not being seen for a call in event listener, but overall with connection pool behaviour and soon happy eyeballs, it is maybe worth considering how we make this behavior visible.

Working towards https://github.com/square/okhttp/issues/506


Show non-aliased ingredient names in ingredient list?

Right now as I'm experimenting, it looks like if aliases are used, the alias is used everywhere in the human and web output, and the real name is unused. I'm not sure what the primary use case for aliases is, but I've been using it to be specific in the ingredient list but have a short name. For example, @small onion|onion{1*}(chopped). I want the step to show onion, but the ingredient list to show small onion.
Locally I was able to quickly emulate this behavior in the CLI by using .name instead of display_name() at the relevant point in the code, but I'm not sure if this is behavior that is useful to anyone else. Perhaps having a configuration setting or extension might be helpful? I'm more than happy to code something up if this is something you'd like.

There is already an issue about this in the parser repo https://github.com/cooklang/cooklang-rs/issues/18
There I explain the design reasoning for why it works this way. I'll leave this open here as well to maybe, if more people ask, add a configuration setting to chef to change the behavior, but for now I'm hesitant.


Implementing with Kraken configuration files

Hello,
First of all, thank you for this awesome module. While I am new to Node.js, coming from the PHP world, this module has really been a pleasure to work with.
I am attempting to use it with Kraken and specifically am having issues passing configuration data to createPool in such a way that I can continue using the database connection as an exported module.
Currently, my exported db module looks like:
var mysql = require('mysql');

var db = mysql.createPool({
    connectionLimit: 10,
    host: 'localhost',
    user: 'root',
    password: '',
    database: 'test'
});

// Test the connection
db.getConnection(function (err, connection) {
    if (err) {
        console.error('error connecting: ' + err.stack);
    }

    console.log('connected as id ', connection.threadId);

    connection.release();
});

module.exports = db;

This works fine as I can reuse the connection elsewhere. I'd like to leverage use of Kraken's config file such as they do at https://github.com/krakenjs/kraken-example-with-shoppingcart/blob/master/lib/database.js. I've been trying to implement this, but come across issues such as mysql.getConnection is not a function. My attempt was as follows.
var mysql = require('mysql');

var db = function () {
    return {
        config: function (conf) {

            var db = mysql.createPool({
                connectionLimit: 10,
                host: conf.host,
                user: 'root',
                password: '',
                database: 'test'
            });

            // Test the connection
            db.getConnection(function (err, connection) {
                if (err) {
                    console.error('error connecting: ' + err.stack);
                }

                console.log('connected as id ', connection.threadId);

                connection.release();
            });
        }
    };
};

module.exports = db();

Is there a way to configure the MySQL connection via a function? Any suggestions?

That is not how this works, I don't know exactly, how it works with mongoDB from the example, but all your code does is exporting a function "conf", that takes a config-literal, opens an encapsulated pool connection in local var db, test it and exits.
When the callback function is executed and is done the local var db will be garbage collected since it is not accessible any longer.
I guess you code looks something like this:
// main app-file
[...]
var mysql = require('lib/database.js');
[...]

The only method on mysql there is conf, which does nothing but locally connecting and testing.
What you want to do is something like this:
In "database.js":
'use strict';
var daPool = require('mysql');
function conf(conf) {
    return daPool.createPool({
        connectionLimit: conf.connectionLimit,
        host: conf.host,
        user: conf.user,
        password: conf.password,
        database: conf.database
    });
};
module.exports = conf;

in the part of your app where you load the modules:
// [...] other modules loaded before
var mysql = require('lib/database.js');
var pool = mysql.conf(dbConfigurationLiteral);

/* Test the connection. The execution of the test is asynchronous!!
   So it will execute later, when the app is running */
pool.getConnection(function testCallback(err, connection) {
    if (err) {
        console.error('error connecting: ' + err.stack);
    }
    console.log('connected as id ', connection.threadId);
    connection.release();
});
// [...] more modules loaded and app magic happens

// App starts listens to http-rquests.
// Now you might get a console message from ```testCallback```

Please close the issue, if this does answer your question.

@kai-koch thanks for the response. My apologies for the delay in responding, I will take a further look at this as soon as I can.

Hi @betweenbrain, I'm going to close this issue unless you think there is an issue with this module itself. The discussion is welcome to continue, and all parties will receive email alerts of new comments :)


autohealsettings parameter is not available with 'az functionapp config set' cli command

Related command

Is your feature request related to a problem? Please describe.

Describe the solution you'd like

Describe alternatives you've considered

Additional context

Is there any update ?

Please share an update on this.

Hello @thechaoticbuddha This is a feature request that we have added to our backlog. In the meantime, you can still enable auto heal using the az webapp config set --auto-heal-enabled true command. Please let us know if this temporary solution would unblock you for now. Thank you.


As an operator, I want to be able to interact with a processing job that is being executed on a U-SPS cluster

Once a job is submitted, the operator needs to be able to monitor the status of the job (looking at rich metadata), possibly canceling the job, inspecting possible errors, etc.

Removed because we decided that jobs and workflows are the same thing for the FY23 scope - so this functionality is covered by the workflow Epics.


Processing button feedback for reporting and disputing pages

Processing button state design: https://www.figma.com/file/bZh7tOiLm6Ls9Hl4RL6qMC5X/Augur-Redesign?node-id=8223%3A4544
Needed in these places
Get Participation Tokens and Redeem Past Participation Tokens

Update button in Designated and Open reporting - remove dot dot dot and add spinner to be consistent across all processing buttons

Report button on market page

Dispute/support button on market page

See this ticket for dispute cards https://app.zenhub.com/workspaces/augur-5bfed3424b5806bc2bf8d173/issues/augurproject/augur/6461

closing - dupe of https://github.com/AugurProject/augur/issues/6469


Network trained on GPU gives different results in CPU

I trained a relatively standard convolutional neural network with Keras using Theano with a K40m GPU. The training goes well and I'm happy with the results. I then save the network topology with model_to_json and the weights with save_weights and read it from another computer without a GPU to do some applications for the network using model.predict. The results with the CPU are completely off and different from what I should get. I run the CPU version with the environment variables device=cpu,floatX=float32 to make sure that the CPU is using 32 bits for the floating point numbers. My version of Keras is 1.0.8. Any indication of how to proceed to solve this issue?

if you are using cudaNN some of the convolution functions are non-deterministic

I finally realized that the Keras version in the computer with the GPU was 1.0.7, while I was doing the prediction with the CPU in Keras 1.0.8. Apparently, there was a change in the saving/loading weights routines between the two versions, that made it incompatible. Anyway, thanks for pointing out the issue that cudaNN is non-deterministic.


Add index documentation page

Codecov Report
Patch and project coverage have no change.

Comparison is base (9cbbca5) 100.00% compared to head (52b5d4c) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #95   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           57        57
  Lines         2057      2057
  Branches       143       143
=========================================
  Hits          2057      2057

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Preparing features takes too long

Hi,
Preparing filter banks for large datasets (~6000h) takes too long. Even time of preparing manifest is approximately 10 hours. Do you have any suggestion to make this process fast?
I am trying to modify Dataset part as on-the-fly computing (if feature is not computed then compute&write else read). So I can observe early batches earlier.

Please have a look at https://github.com/lhotse-speech/lhotse/pull/422
Feature extraction for 100 hours of audio takes about 3 minutes.

I missed this merge, thank you :)

@EmreOzkose
Could you share with us some statistics, e.g, the time taken for feature extraction about 6000 hours of audio using CUDA?

As for preparing manifests, you can usually get a good speed-up with multiprocessing, see e.g. the librispeech recipe in Lhotse. Unfortunately every corpus requires different code structure for manifest preparation so I can’t give a more specific advice.
One thing that also might be useful with large data is incremental writing of the manifest — it allows you to stop and resume from it left, see here for examples: https://lhotse.readthedocs.io/en/latest/api.html#lhotse.audio.RecordingSet.open_writer

@csukuangfj I started compute_fbank() with num_jobs=15. After ~30hours, there wasn't any progress (using cpu). However I will try  lhotse-speech/lhotse#422 today and report here.
@pzelasko I am also checking https://lhotse.readthedocs.io/en/latest/api.html#lhotse.audio.RecordingSet.open_writer, thank you.

What does top look like: is it using up lots of CPU, or waiting on
something, e.g. disk?
If your disk access is very slow for some reason, any method is going to be
slow.
On Wed, Nov 3, 2021 at 1:04 PM Yunusemre @.***> wrote:

@csukuangfj https://github.com/csukuangfj I started compute_fbank()
with num_jobs=15. After ~30hours, there wasn't any progress (using cpu).
However I will try lhotse-speech/lhotse#422
https://github.com/lhotse-speech/lhotse/pull/422 today and report here.
@pzelasko https://github.com/pzelasko I am also checking
https://lhotse.readthedocs.io/en/latest/api.html#lhotse.audio.RecordingSet.open_writer,
thank you.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/k2-fsa/icefall/issues/101#issuecomment-958658153, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZFLOZW7SUA2RNRU6F5TSTUKC7FRANCNFSM5HFUIYUA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Sorry for late reply, there was a glitch on my machine.
@danpovey I checked this issue, but disk access seems good.
With the help of @csukuangfj and @pzelasko, I made 2 changes in this script:
compute_and_store_features to compute_and_store_features_batch
Fbank(FbankConfig(num_mel_bins=num_mel_bins)) to KaldifeatFbank(KaldifeatFbankConfig(device='cuda:0'))
Now, estimated progress time is ~4.5 hours with utilization of GPU. Thank you.

@EmreOzkose Thanks for sharing.

Preparing filter banks for large datasets (~6000h) takes too long.

Now, estimated progress time is ~4.5 hours with utilization of GPU.

Did you mean it takes about 4.5 hours to extract features for 6000 hours of audio on a single GPU?

Yes, for single GPU.


Add disable letsencrypt

This PR adds the setup-option NETBIRD_DISABLE_LETSENCRYPT in setup.env.example to allow disabling the internal letsencrypt-setup.
e.g. for use with a reverse-proxy or to ship own letsencrypt-certs from an external-system.
For this i also had to add some env-setup-variables.
I also reordered the compose aswell as the setup.env file to make it more readable.
When running configure with NETBIRD_DISABLE_LETSENCRYPT=true the user is informed about actions he must take to get a working setup.
I have also prepared doc-changes to include more extensive documentation about using netbird behind a reverse-proxy and shipped an example-file for traefik.
I plan to submit that PR when this one is accepted.
But I am not 100% happy with how much the user has to edit to archive such a setup, but i think the example-files will help alot.
(Maybe combine the domains and ports, so only 1 thing has to be changed?)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

I have also marked the doc-changes as ready: https://github.com/netbirdio/docs/pull/34


Return existing signups from campaign sign-up endpoint

If an API client (such as Northstar) tries to use the API to create a signup, and the given user is already signed up for the campaign, the API returns a relatively unhelpful [ false ] response.
It would be nice if it returned the existing signup, with either an "already exists" message or a timestamp representing when the original signup was made. In this case, it should probably return 200 Okay rather than 201 Created.
References DoSomething/Northstar#95.
/cc @jonuy

It'll also be a good idea with this one to standardize the response from this endpoint. Whether it's a success or failure, the response should come back as an object. And in addition to the sid should also come back with the rest of that resource data - at least uid, nid, source, and timestamp.

doi, replaced with #6118


Implement frame based backmapping protocol

Extend the gen_coords processor to allow for more backmapping protocols. As a first example, a frame-based protocol will be implemented, which places molecule templates on generated or user-provided orientation frames.
The main aim of this backmapping protocol is currently to allow for the construction of double-stranded DNA molecules.

this is outdated and succeeded by the new approach based on frames per base not base pair