Calling VirtualDisplayController::resize() just before VirtualDisplayController::dispose() causes a crash

Steps to reproduce
In PlatformViewsController.java, there is a channelHandler object that implements the PlatformViewsChannel.PlatformViewsHandler interface. This interface includes two methods: resize() and dispose(). If resize() is called immediately before dispose(), it may lead to a crash.

When resize() is called with viewId = 0, it triggers a call to vdController.resize(). This call includes a Runnable object (lambda) as its third parameter, which is added to the message queue using View::postDelayed().

Then we call the dispose() function, which invokes vdController.dispose() and removes vdController from the HashMap.

After calling dispose(), the lambda previously added to the message queue using View::postDelayed() begins execution. Here, we attempt to retrieve the dimensions of the render target: vdController.getRenderTargetWidth(), vdController.getRenderTargetHeight().

In VirtualDisplayController::getRenderTargetWidth() we call renderTarget.getWidth()

Inside the SurfaceProducerPlatformViewRenderTarget, the producer object was released when vdController.dispose() was called, resulting in producer being null.

Expected results
Calling VirtualDisplayController::resize() just before VirtualDisplayController::dispose() shouldn't cause a crash
Actual results
The app is crashed.
Code sample
Code sample
No code. This issue is obvious from the attached screenshots of Flutter code from the steps to reproduce section.

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs

Crash log

08-02 15:25:22.167 18044 18044 E AndroidRuntime: FATAL EXCEPTION: main
08-02 15:25:22.167 18044 18044 E AndroidRuntime: java.lang.NullPointerException: Attempt to invoke interface method 'int io.flutter.view.TextureRegistry$SurfaceProducer.getWidth()' on a null object reference
08-02 15:25:22.167 18044 18044 E AndroidRuntime: 	at io.flutter.plugin.platform.SurfaceProducerPlatformViewRenderTarget.getWidth(SurfaceProducerPlatformViewRenderTarget.java:25)
08-02 15:25:22.167 18044 18044 E AndroidRuntime: 	at io.flutter.plugin.platform.VirtualDisplayController.getRenderTargetWidth(VirtualDisplayController.java:132)
08-02 15:25:22.167 18044 18044 E AndroidRuntime: 	at io.flutter.plugin.platform.PlatformViewsController$1.lambda$resize$0$io-flutter-plugin-platform-PlatformViewsController$1(PlatformViewsController.java:352)
08-02 15:25:22.167 18044 18044 E AndroidRuntime: 	at io.flutter.plugin.platform.PlatformViewsController$1$$ExternalSyntheticLambda0.run(Unknown Source:8)
08-02 15:25:22.167 18044 18044 E AndroidRuntime: 	at android.os.Handler.handleCallback(Handler.java:958)
08-02 15:25:22.167 18044 18044 E AndroidRuntime: 	at android.os.Handler.dispatchMessage(Handler.java:99)
08-02 15:25:22.167 18044 18044 E AndroidRuntime: 	at android.os.Looper.loopOnce(Looper.java:230)
08-02 15:25:22.167 18044 18044 E AndroidRuntime: 	at android.os.Looper.loop(Looper.java:319)
08-02 15:25:22.167 18044 18044 E AndroidRuntime: 	at android.app.ActivityThread.main(ActivityThread.java:8919)
08-02 15:25:22.167 18044 18044 E AndroidRuntime: 	at java.lang.reflect.Method.invoke(Native Method)
08-02 15:25:22.167 18044 18044 E AndroidRuntime: 	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:578)
08-02 15:25:22.167 18044 18044 E AndroidRuntime: 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1103)

Flutter Doctor output
Doctor output
[Paste your output here]

@andriiudovenko
Can you provide flutter doctor -v so as to know in which version is this crash occuring ?

[✓] Flutter (Channel stable, 3.22.3, on Ubuntu 22.04.4 LTS 6.8.0-40-generic, locale en_US.UTF-8)
• Flutter version 3.22.3 on channel stable at /usr/bin/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision b0850beeb2 (7 weeks ago), 2024-07-16 21:43:41 -0700
• Engine revision 235db911ba
• Dart version 3.4.4
• DevTools version 2.34.3
[!] Android toolchain - develop for Android devices (Android SDK version 35.0.0)
• Android SDK at /home/audovenko/Android/Sdk/
✗ cmdline-tools component is missing
Run path/to/sdkmanager --install "cmdline-tools;latest"
See https://developer.android.com/studio/command-line for more details.
✗ Android license status unknown.
Run flutter doctor --android-licenses to accept the SDK licenses.
See https://flutter.dev/docs/get-started/install/linux#android-setup for more details.
[✓] Chrome - develop for the web
• Chrome at google-chrome
[✗] Linux toolchain - develop for Linux desktop
✗ clang++ is required for Linux development.
It is likely available from your distribution (e.g.: apt install clang), or can be downloaded from https://releases.llvm.org/
• cmake version 3.23.1
• ninja version 1.10.1
• pkg-config version 0.29.2
✗ GTK 3.0 development libraries are required for Linux development.
They are likely available from your distribution (e.g.: apt install libgtk-3-dev)
[✓] Android Studio (version 2023.3)
• Android Studio at /home/audovenko/android-studio-2<IP_ADDRESS>-linux/android-studio
• Flutter plugin version 80.0.1
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 17.0.10+0-17.0.10b1087.21-11572160)
[✓] Connected device (3 available)
• sdk gphone64 x86 64 (mobile) • emulator-5554 • android-x64    • Android 14 (API 34) (emulator)
• Linux (desktop)              • linux         • linux-x64      • Ubuntu 22.04.4 LTS 6.8.0-40-generic
• Chrome (web)                 • chrome        • web-javascript • Google Chrome 126.0.6478.126

Thanks for the update @andriiudovenko
Although you mentioned code sample is not required, but in order to properly address the issue, it would be helpful to have a  reproducible code sample that we can also verify at our end and to analyze further.

Thanks for the update. Keeping the issue open and labeling for team's tracking.

@johnmccutchan FYI, on a first pass does the suggested fix look like what we would want to implement or would you want to fix the race condition another way.

My app also affected by this issue from November 12. Because I have updated flutter from 2.10.5 on the date. Then I found this issue in play console. Affected by many type of android versions.
Crash Log:
Exception java.lang.NullPointerException: Attempt to invoke interface method 'int io.flutter.view.TextureRegistry$SurfaceProducer.getWidth()' on a null object reference   at io.flutter.plugin.platform.SurfaceProducerPlatformViewRenderTarget.getWidth (SurfaceProducerPlatformViewRenderTarget.java:25)   at io.flutter.plugin.platform.VirtualDisplayController.getRenderTargetWidth (VirtualDisplayController.java:132)   at io.flutter.plugin.platform.PlatformViewsController$1.lambda$resize$0$io-flutter-plugin-platform-PlatformViewsController$1 (PlatformViewsController.java:352)   at io.flutter.plugin.platform.PlatformViewsController$1$$ExternalSyntheticLambda0.run (Unknown Source:8)   at android.os.Handler.handleCallback (Handler.java:942)   at android.os.Handler.dispatchMessage (Handler.java:99)   at android.os.Looper.loopOnce (Looper.java:204)   at android.os.Looper.loop (Looper.java:291)   at android.app.ActivityThread.main (ActivityThread.java:8134)   at java.lang.reflect.Method.invoke   at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run (RuntimeInit.java:588)   at com.android.internal.os.ZygoteInit.main (ZygoteInit.java:1019)

@darshankawar  We are also facing same issue . Please find below log.
java.lang.NullPointerException: Attempt to invoke interface method 'int io.flutter.view.TextureRegistry$SurfaceProducer.getWidth()' on a null object reference                                                                                at io.flutter.plugin.platform.D.getWidth(Unknown Source:2)                                                                                at io.flutter.plugin.platform.F.f(Unknown Source:4)                                                                                at io.flutter.plugin.platform.w$a.k(Unknown Source:24)                                                                                at io.flutter.plugin.platform.w$a.j(Unknown Source:0)                                                                                at io.flutter.plugin.platform.v.run(Unknown Source:8)                                                                                at android.os.Handler.handleCallback(Handler.java:958)                                                                                at android.os.Handler.dispatchMessage(Handler.java:99)                                                                                at android.os.Looper.loopOnce(Looper.java:230)                                                                                at android.os.Looper.loop(Looper.java:319)                                                                                at android.app.ActivityThread.main(ActivityThread.java:8919)                                                                                at java.lang.reflect.Method.invoke(Native Method)                                                                                at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:578)                                                                                at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1103)


Create Documentation: how to add media to Zooniverse projects via III…

Please review my first pass at documentation for this new feature! :D

Looks good but Eleventy ignores it during the build because it doesn't have a .md extension. I'm going to add a docs directory and rename it to give it a shorter URL.

Thanks! I've published it as https://zooniverse.github.io/iiif-annotations/docs/Add-a-manifest/ It should be live shortly.

I should probably add some default styling to the Eleventy build. It's raw HTML at the moment.


Add Environment Unit Tests

Please use the following example at as a template:
https://github.com/DefectDojo/django-DefectDojo/blob/dev/tests/Product_unit_test_example.py
Implement the following tests:

Add
Edit

Master List and Overview: https://docs.google.com/document/d/1dSS7tWuQukAX3VWtzPAlOfFNoqLQh-mBHBfmsu2WlU0/edit?usp=sharing

@Maffooch, please, really have a look at https://github.com/DefectDojo/django-DefectDojo/issues/1131#issuecomment-493279138 before using https://github.com/DefectDojo/django-DefectDojo/blob/dev/tests/Product_unit_test_example.py for all the functional tests. :pray:

@Maffooch working on this!


Fix linux driver for kernel 5.19+

fix #2875
Use "\n" instead of spaces

Tested and passed in the following versions

linux-5.4.214
linux-5.15.69
linux-5.16.10
linux-5.18.19
linux-5.19.1
linux-6.0-rc6

Thanks! 👍


How to add Log when create ent.NewClient

I would like to create ent.NewClient(ent.Driver(drv)) and add log here?
But how to use Log(fn func(...interface{})) here. Please give me example. Many thanks
https://sourcegraph.com/github.com/facebookincubator/ent/-/blob/entc/integration/json/ent/config.go#L23:2&tab=references

What are you trying to do? Passing a custom log function, or log queries?

What are you trying to do? Pass a custom log function, or log queries?

custom log

Here's an example that replaces the default Log function with log.Println.

import (
     "log"

     "<pkg-path>/ent"
)

func main() {
	client, err := ent.Open("<dialect>", "<conn-string>", ent.Log(log.Println))
	if err != nil {
		log.Fatalf("failed to open client: %v", err)
	}
	defer client.Close()
        // ...
}

I hope it helps.
Closing, but feel free to open if you still have questions.
Thanks!


Add fixed ID to bounding box in Cvat Xml

Hello,
We are trying to show annotators how we would like our objects annotated by sending them a .pdf report. This report referrences every frame, every bounding box and how we would like for things to be different in it.
The problem is that every time a box is added, all the ids of all boxes of all frames change as well.
Even within a frame, the id order changes.
What I think should be done in the code is:

Create a counter which increases every time a bounding box is created.
Assign the current value of the counter to every new bounding box created
When loading/dumping an annotation, use the attribute "id" inside  as id.

The changes in the .xml should be:

Create a tag under  called <id_counter> holding the next id value
Create an attribute under  called "id"

What do you think, @nmanovic ? Does that make sense? Would that break something in the interpolation mode or something?
Thank you very much,
Eric

@egrinstein,
It is a complex topic. There are many problems here. Previously we had a client id but got a lot of problems.

Which code should assign ids? Client? What if several clients edit the same task?
CVAT server cannot trust clients to assign ids by default. Need to check/validate and it is performance issues.
If we find an issue with assigned ids on the server what should we do? Reject the request? Indeed a user can open a couple of tabs on the same machine and it will give us a problem with saving annotations. We had the pain.
If we assign all ids on the server it will require constant synchronization between the server and its clients. I'm not sure that it is good. Need to think more about the option.

P.S. I don't want to say that it is impossible to implement. It is not so trivial as it looks.

I will close the issue for now. Don't hesitate to reopen it if you have any ideas how to implement the feature.

The main problem here that the ID can be changed after some time. For example, let's say you update a shape with server id #X. Sometimes on server it is more efficient to delete the object and recreate it from scratch (with another server id #Y). Thus the previous dump will not be valid even the shape is mostly the same. Then we start to think about sync ids between client and server it becomes big pain. Every time when you create an object on client you have to go to the server and allocate an ID (it is a performance issue). Previously the client generated an ID but it led to many problems especially when people tried to change the same job in parallel.

@nmanovic Can you provide some insight into what operations might cause the object to be deleted and recreated resulting in the server id changing? (Assuming this behavior still exists since the time you wrote this)
Consistency is a fairly important property for object IDs. In my case, I am uploading annotations and trying to keep track of which objects have been modified/added/deleted which requires a mapping between my custom object id and the CVAT object's server id. The fact that the server id can change would break this mapping and result in unwanted behavior. (Any other ideas of how to link CVAT objects with external object ids in a stable way would be welcome as well!)
A possible solution for your case could be to switch to using UUIDs instead of integer ids as this would allow clients to create ids without the concern of any ids overlapping. I assume this would require a significant code overhaul, though...

@ehofesmann , it is better to look at code to get the answer. But probably any update operation will lead to the behavior. Probably the feature which you have mentioned is very important. Don't hesitate to create a new issue and describe your needs. We will see how to support. Also we are happy to accept any contribution in the direction. There is bulk_update method in Django. Probably we can use it to avoid "deleting" objects.


Dark mode setting not saving

Description
Dark mode setting is not saving when set to "off". Upon next entry to app it reverts back to default "Auto" selection.
Steps to reproduce the behavior

Go to 'Settings-->Dark Mode'
Click 'Off'
Log out, log back in
See setting reverted to 'Auto(Detect)' default

Expected behavior
Setting to be saved
Smartphone

Device: Samsung S8+ and Pixel 3a
OS: Android 9 and GrapheneOS Android 11
Version 1.4.0

I can reproduce this with my Pixel 3 XL using Graphene OS 11.  The dark mode setting remains off if you exit the app without logging out. If you log out and log back in the setting reverts to auto.

Hello @SecretestSquirrel,
Thanks for reporting. We have work on this issue and notify you when the fix will be released.
Kind regards.

@SecretestSquirrel Now DarkMode setting will be saved in your account settings and repaired when you re-login. This feature will be available in the next release.
Additional info: Now if you have turned off the dark theme on the Web Client, the theme setting in the app will only change if it was dark before. If the dark theme in the app was in an "off" or "auto" state then this value will remain. Thus you can turn off the dark theme on the Web Client but remain it in the app if by Android system defaults dark mode is enabled. Or you can forcibly disable the dark theme in the app settings if you want to remain dark theme only in the Android system.


Cannot get it to load on P73

I'm unable to successfully load the program on my P73 running Win10 Pro 1909. If it's of any value, I see the following two errors in the event viewer:
Application Error
Faulting application name: Auto Backlight for ThinkPad.exe, version: 1.0.7428.6847, time stamp: 0x5eae7740
Faulting module name: KERNELBASE.dll, version: 10.0.18362.815, time stamp: 0xb89efff3
Exception code: 0xe0434352
Fault offset: 0x000000000003a799
Faulting process id: 0x1f1c
Faulting application start time: 0x01d62e84674ab296
Faulting application path: C:\Program Files\Auto Backlight for ThinkPad\Auto Backlight for ThinkPad.exe
Faulting module path: C:\Windows\System32\KERNELBASE.dll
Report Id: 9b14eb7f-349d-402a-9bc7-888ccfd414c8
Faulting package full name:
Faulting package-relative application ID:
.NET Runtime
Application: Auto Backlight for ThinkPad.exe
Framework Version: v4.0.30319
Description: The process was terminated due to an unhandled exception.
Exception Info: System.Exception
at Auto_Backlight_for_ThinkPad.Capture..ctor(Auto_Backlight_for_ThinkPad.RawInput.RawInput)
at Auto_Backlight_for_ThinkPad.AutoKeyboardBacklightController..ctor(Auto_Backlight_for_ThinkPad.PowerManagement.PowerManagement, Auto_Backlight_for_ThinkPad.RawInput.RawInput)
at Auto_Backlight_for_ThinkPad.App..ctor()
at Auto_Backlight_for_ThinkPad.App.Main()
at Auto_Backlight_for_ThinkPad.Program.Main()

It appears most of the other issues others have submitted may be similar to mine where the program simply isn't loading. I hope that the information in my original post is helpful. This app seems to be exactly what I'm looking for, so I'd love to contribute anything to help improve it.


Update 004 - Median-of-two-sorted-array

Added Solution of lc problem 004 - Median of two sorted array using python
Related
Fix: #722
[Cite any related issue(s) this pull request addresses. If none, simply state "None”]
Description
Added Solution of lc problem 004 - Median of two sorted array using python
[Please include a brief description of the changes or features added]
Type of PR

[ ] Bug fix
[ ] Feature enhancement
[x] Documentation update
[ ] Other (specify): _______________

Screenshots / Videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes]
Checklist

[x] I have performed a self-review of my code.
[x] I have read and followed the Contribution Guidelines.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.

Additional Context
[Include any additional information or context that might be helpful for reviewers.]
Resources for Guidance
Here are some resources that may be helpful as you contribute to CodeHarborHub:

Code Harbor Hub Community Features
Docusaurus Documentation
React.js Documentation
Markdown Guide
MDX Documentation
Mermaid Documentation

Resources for DSA Features on Our Site:

Tabs: Switch Languages
Line Numbering
Math Equations

/assign
merged?

Hello @Ajay-Dhangar , please merge it

Hey @Ajay-Dhangar , can you tell me if it is showing closed then can i also get points of this PR?


Got error Kibana server is not ready yet.

i get the error Kibana server is not ready yet. I'm using v8, how do I fix it?
[2022-02-16T03:02:58.666+00:00][INFO ][plugins.screenshotting.config] Chromium sandbox provides an additional layer of protection, and is supported for Linux Ubuntu 20.04 OS. Automatically enabling Chromium sandbox. [2022-02-16T03:03:00.190+00:00][ERROR][elasticsearch-service] Unable to retrieve version information from Elasticsearch nodes. connect ECONNREFUSED <IP_ADDRESS>:9200 [2022-02-16T03:03:01.090+00:00][INFO ][plugins.screenshotting.chromium] Browser executable: /usr/share/kibana/x-pack/plugins/screenshotting/chromium/headless_shell-linux_x64/headless_shell [2022-02-16T03:03:04.149+00:00][ERROR][elasticsearch-service] Unable to retrieve version information from Elasticsearch nodes. security_exception: [security_exception] Reason: unable to authenticate user [kibana_system] for REST request [/_nodes?filter_path=nodes.*.version%2Cnodes.*.http.publish_address%2Cnodes.*.ip]

I am getting the same error

@agungsaputro @joyshmitz there are important initial steps that need to be performed since v8. They are documented in the README.
I am currently working on automating these steps in #671, but for the time being you need to generate a password for Kibana manually.
Related to #666

Hi, I was running 7.17.0 with data in a persistent volume before i tried to upgrade to version 8.0.0 and now I got stuck with the same error.
What is did is:

docker-compose down
edited .env from 7.17.0 to 8.0.0
docker-compose build
docker-compose up -d
notices the error with docker-compose logs (rered the README)
docker-compose down
changed in kibana.yml the "elasticsearch.username" from elastic to kibana_system
docker-comose up -d elasticsearch
docker-compose exec -T elasticsearch bin/elasticsearch-reset-password --batch --user elastic

now I get the error message:
WARNING: Owner of file [/usr/share/elasticsearch/config/users] used to be [root], but now is [elasticsearch] WARNING: Owner of file [/usr/share/elasticsearch/config/users_roles] used to be [root], but now is [elasticsearch] ERROR: Failed to determine the health of the cluster.
To be honest, now i dont know what to do :)

Thank You


AMBARI-25060. Install wizard fails on step 8

What changes were proposed in this pull request?
Install wizard fail on step 8
Please review Ambari Contributing Guide before opening a pull request.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/4775/
Test PASSed.


Add tips for download workflow

Codecov Report

Merging #2831 into dev-1.3.0 will increase coverage by 0.15%.
The diff coverage is 24.13%.

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


AnimatedSampler issue

Please let me start off by saying: wow! Amazing tool!
I can do most of what I'd like, but taking the (fully working! wow!) AnimatedSampler pixelation example, and I try to add another uniform, it crashes with RangeError (index): Index out of range: index should be less than n: n. This happens fairly often with a number of different changes to my .frag code (even if it doesn't relate to how many uniforms I expect in the .frag code).
This happens whether I use shader.setFloatUniforms, or if I set uniforms individually... Part of me suspects this error is on the end of the Flutter team, who I also love.
I've attached screen caps of the error that I can least understand. I think it's related to the issue above, since it's still an Index out of range error (only lines 22, 23, 24, and 26 are commented out - everything else is the same).
Is this a common issue? Please let me know if more information would be helpful.

Trying to make metaballs based on circular containers with a radial gradient... 🤔

First of all, please don't ever file bugs with screenshots of code. Just put the code in the issue so it can be searched/copied.
Your problem is that the uSize uniform is unused, so it is being optimized out. See https://github.com/flutter/flutter/issues/116900

The way I read that felt a bit negative, but that was very helpful... Might just be my tendency of how to read tones. 🤷♂
Do you have anything like a Patreon for people like me to show support?

You did read that correctly, it was a bit negative! 😉

How frustrating.


flash attention leads to error

I try to use ggml_flash_attn to accelerate the process, so I replace ggml_mul_mat in cross-attention in UNET in stable-diffusion.cpp:
...
#if 1
                struct ggml_tensor * kqv = ggml_flash_attn(ctx, q, k, v, true);
#else
                struct ggml_tensor* kq = ggml_mul_mat(ctx, k, q);  // [N * n_head, h * w, h * w]
                // kq = ggml_diag_mask_inf_inplace(ctx, kq, 0);
                kq = ggml_soft_max_inplace(ctx, kq);

                struct ggml_tensor* kqv = ggml_mul_mat(ctx, v, kq);  // [N * n_head, h * w, d_head]
#endif
...

But it leads to an error. Looks like the max_position = 2, N = 64, and const int64_t P = nek1 - N; which is less than 0. Can someone help me? Great thx!

Did you observe any speed improvement?


[REF] remove unnecessary commands for new build step

Summary of changes
Removed unnecessary call to getaddons.py and removed the copy of the old config file

@ruiztulio ready for review


Some Youtube videos cannot be casted

https://www.youtube.com/watch?v=5s7_WbiR79E
for example

Hello,
Could you provide the log, located at /home/pi/RaspberryCast/RaspberryCast.log ?

06-06 11:23:49 - DEBUG - Parsing source url for https://www.youtube.com/watch?v=5s7_WbiR79E with subs :False
06-06 11:23:50 - DEBUG - [youtube] 5s7_WbiR79E: Downloading webpage
06-06 11:23:50 - DEBUG - [youtube] 5s7_WbiR79E: Downloading video info webpage
06-06 11:23:51 - DEBUG - [youtube] 5s7_WbiR79E: Extracting video information
06-06 11:23:51 - DEBUG - [youtube] 5s7_WbiR79E: Downloading MPD manifest
06-06 11:23:57 - DEBUG - CASTING: Youtube link detected, extracting url in maximal quality.
06-06 11:23:57 - DEBUG - Full video URL fetched.
06-06 11:23:57 - INFO - Starting OMXPlayer now.
06-06 11:24:02 - INFO - Starting next video in playlist.
06-06 11:24:02 - INFO - Starting OMXPlayer now.
06-06 11:24:06 - INFO - Playlist empty, skipping.

continued, trying to play another one that didn't cast
06-06 11:23:57 - INFO - Starting OMXPlayer now.
06-06 11:24:02 - INFO - Starting next video in playlist.
06-06 11:24:02 - INFO - Starting OMXPlayer now.
06-06 11:24:06 - INFO - Playlist empty, skipping.
06-06 11:27:53 - INFO - No video currently playing, playing url : https://www.youtube.com/watch?v=Hrv25UTR_co
06-06 11:28:07 - INFO - Extracting source video URL...
06-06 11:28:07 - DEBUG - Parsing source url for https://www.youtube.com/watch?v=Hrv25UTR_co with subs :False
06-06 11:28:08 - DEBUG - [youtube] Hrv25UTR_co: Downloading webpage
06-06 11:28:12 - DEBUG - [youtube] Hrv25UTR_co: Downloading video info webpage
06-06 11:28:13 - DEBUG - [youtube] Hrv25UTR_co: Extracting video information
06-06 11:28:13 - DEBUG - [youtube] Hrv25UTR_co: Downloading MPD manifest
06-06 11:28:20 - DEBUG - CASTING: Youtube link detected, extracting url in maximal quality.
06-06 11:28:20 - DEBUG - Full video URL fetched.
06-06 11:28:20 - INFO - Starting OMXPlayer now.
06-06 11:28:51 - INFO - Playlist empty, skipping.

Does it works with most of other videos ?
Or no Youtube videos at all works ?

It's not a severe issue, I just wanted to point out the inconvenience. I'm new to the rpi and github.
I've come across at least 20 youtube videos that didn't cast. Others work fine.
Regards


🚀 Feature: Fetch GraphQL Schema under the authentication with kind: API.

🔖 Feature description
Support for

authentication for schema pulling with the specs

🎤 Context
building an api catalog with the graphql endpoints
✌ Possible Implementation
No response
👀 Have you spent some time to check if this feature request has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
None

👋  hey! Think I need a little more information here - is this when you're processing the Entity and loading in the .graphql schema and need authentication server side, or is this when you want to do introspection and need the authentication on the frontend in the plugin?

I ment the first one
processing the Entity and loading in the .graphql schema from a authenticated endpoint

Message which throws like this
{"error":{"name":"InputError","message":"InputError: Processor PlaceholderProcessor threw an error while preprocessing; caused by Error: Placeholder $text could not read location https://company.com/api/graphql, Error: could not read https://company.com/api/graphql, 400 Bad Request","stack":"InputError: InputError: Processor PlaceholderProcessor threw an error while preprocessing; caused by Error: Placeholder $text could not read location https://company.com/api/graphql, Error: could not read https://company.com/api/graphql, 400 Bad Request\n at DefaultLocationService.processEntities (/Users/roommen/Work/dx-backstage/node_modules/@backstage/plugin-catalog-backend/dist/index.cjs.js:2188:15)\n at processTicksAndRejections (node:internal/process/task_queues:96:5)\n at async DefaultLocationService.dryRunCreateLocation (/Users/roommen/Work/dx-backstage/node_modules/@backstage/plugin-catalog-backend/dist/index.cjs.js:2219:22)\n at async /Users/roommen/Work/dx-backstage/node_modules/@backstage/plugin-catalog-backend/dist/index.cjs.js:3512:22"},"request":{"method":"POST","url":"/locations?dryRun=true"},"response":{"statusCode":400}}

Graphql endpoint has the authentication

Hey :wave: sorry this kind of slipped through the net a little bit here.
So what I think is the best option here is to create your own URLReader implementation which gets passed to the createRouter method of the catalog in packages/backend/plugins/catalog.ts which then would be used in the YAMLPlaceholderProcessor
I think that you will need to implement readUrl which would get called with your graphql endpoint, and you are then responsible for returning the data from that URL, and you can add your own authentication there and then delegate back to the original implementation when the URL does not match your GraphQL endpoint.

Thanks, i will try this out. But I request to provide this feature to the API Kind in a native way because in terms of graphql I see most of the schemas are behind the authentication.
With this suggestion I like to know any active example which I can refer for the custom URLReader implementation and some resources which I can look to

There's not really any examples of a custom ´UrlReader´ I think. What you can do is to look at what the default does, which is here. Note that there's a create static method that you can use to pick the exact ones that should be added, instead of the standard list. And then you can study the individual factories such as the GitHub one for example.

There's not really any examples of a custom ´UrlReader´ I think. What you can do is to look at what the default does, which is here. Note that there's a create static method that you can use to pick the exact ones that should be added, instead of the standard list. And then you can study the individual factories such as the GitHub one for example.

I have tried this not working as expected

Any update on this issue to provide introspection support to authenticated endpoint

I have tried this not working as expected

What do you mean by this? What happened when you tried to do this? What was your approach?
Using a custom processor or URLReader for this should help you get access to the protected endpoint.


Stuck after showing "We couldn't get the ipsw curl. we will proceed with -k option with curl"

Device: iPhone 6s
ipsw: 14.0.1/14.0
The Phone iOS version was 14.6 and I try to downgrade to 14.0/14.0.1, and stuck after prompting "We couldn't get the ipsw curl. we will proceed with -k option with curl", please help, thanks.
➜  downr1n git:(main) ✗ ./downr1n.sh --downgrade 14.0.1
[*] Command ran: ./downr1n.sh --downgrade 14.0.1
downr1n | Version 3.0
Created by edwin, thanks palera1, and all people creator of path file boot

[*] Waiting for devices
[*] Detected DFU mode device
[*] Waiting for devices
[*] Detected DFU mode device
[*] Getting device info...
Detected cpid, your cpid is 0x8003
Detected model, your model is n71map
Detected deviceid, your deviceid is iPhone8,1
./downr1n.sh: line 705: [: ipsw/14.6.ipsw: binary operator expected
[-] we found ipsw/14.0.1.ipsw, do you want to use it ? please write, yes or no
yes
ipsw/14.0.1.ipsw
[*] Checking if the ipsw is for your device
[*] Checking ipsw version
[*] Extracting ipsw, hang on please ...
[*] Got extract the IPSW successfully
[*] Creating ramdisk
[-] Ramdisk is already created so SKIPPING ...
[*] Booting ramdisk
[/] We couldn't get the ipsw curl. we will proceed with -k option with curl

I close this issue because after few times retry, it works.


Add song name support

This depends on the Vorbis tags being set correctly, which most of the default songs have set, otherwise you'll end up with this:

Resolves https://github.com/space-wizards/space-station-14/issues/15947
Requires https://github.com/space-wizards/RobustToolbox/pull/4207
Supported by https://github.com/space-wizards/space-station-14/pull/18432

You could use the RGA metadata as a fallback.

That sounds like a good idea. Is there an API to access the metadata? I did a couple quick greps for RobustGeneric, GenericAttrib, RGA, but I turned up nothing.

It's just a yaml file in the same dir as the sound file, so you'd probably need to add the boilerplate for that.


The chart-releaser GitHub Action is failing

See: https://github.com/IBM/cp4waiops-gitops/runs/7036836464 and https://github.com/IBM/cp4waiops-gitops/actions/runs/2553948296 as example

This issue may be also related to https://github.com/IBM/cp4waiops-gitops/issues/136.
I removed all the tags and releases that are created unexpectedly, w/ following TODOs:

Since we only officially support GitOps install started from v3.4, all the releases and tags we created previously can be dropped.
The chart-releaser config can also be updated to only pick up v3.4 given the same reason as above.
For the AIManager and EventManager release name, it was previously in the form of: <eventmanager|aimanager><release version>-<chart version>, e.g.: eventmanager33-0.0.1, aimanager32-0.0.1. How about we change it to something such as eventmanager-3.4.0, aimanager-3.4.0, where 3.4 represents the product release version, and the last number represents the helm chart revision number. Originally I thought it could be conflicted w/ the product fix pack release version, e.g. after CP4WAIOps 3.4.0, there will be FP release 3.4.1 coming soon. However, given that we do not have too much helm chart (i.e. GitOps configuration) change per each FP release, we can simplify it by only sticking to the product major release number, and use the last number as the helm chart revision to reflect the changes we made to the GitOps configuration itself.

issue fixed by merging PR: https://github.com/IBM/cp4waiops-gitops/pull/194
I'm closing this issue.


Prototype integration of openweb-ui

Is your enhancement related to a problem? Please describe
Seems openweb-ui can be integrated through a container, would be good to prototype
Describe the solution you'd like
Replace current playground UI will openweb-ui
Describe alternatives you've considered
No response
Additional context
No response

Shouldn't we have something like an abstract layer, and be able to support multiple playground ui ? Just like the inference servers

This is a prototyping issue - let's not over-engineer this.
Today, our playground UI is not abstracted to support multiple playground ui. We may need it in the future or may never need it. Right now, that's not "the problem" this issue should be focusing on.


Fix outdated link to "Do I have to put all my state into Redux?"

name: :memo: Documentation Fix
about: Fixing a problem in an existing docs page
Checklist

[ ] Is there an existing issue for this PR?

link issue here

[ ] Have the files been linted and formatted?

What is the problem?

A change made to the section header in https://github.com/reduxjs/redux/pull/4434 was not updated in other locations in the docs

What changes does this PR make to fix the problem?

Updates the links to the new header

this.setState({ thanks: true })


Presence re-entry requirement change for 1.1

See https://github.com/ably/docs/pull/711.
After confirming that the lib satisfies the new RTP17c, please update the spec spreadsheet accordingly
┆Issue is synchronized with this Jira Bug by Unito

@paddybyers I'm reviewing this and I think it's updated but maybe I'm missing something. So, on every SYNC message (RTP17c1) the library will do a sync and if the message is the last serial then an automatic re-entry is performed.
Now, the library does a sync when the ATTACHED message has false in HAS_PRESENCE flag (RTP17c2) but it checks some extra checks:
https://github.com/ably/ably-cocoa/blob/f3017852ec5a9a34ef3da72cf7b8fe5429903d89/Source/ARTRealtimeChannel.m#L698-L708
Is this still valid?
Or the library should do an automatic re-entry every time the HAS_PRESENCE is false?

@paddybyers bump ^

@paddybyers Hey Paddy, can you have a look at Ricardo's comment please

As far as I understand it we shouldn't do any additional checks like here:
https://github.com/ably/ably-cocoa/blob/48cb1c6f103fb5a7192c1463b1c327ecfafe3642/Source/ARTRealtimeChannel.m#L735-L737
according to RTP17f
The RealtimePresence object should perform automatic re-entry whenever a channel moves into the ATTACHED state. (It does not need to do so for RTL12 ATTACHED events received on already-ATTACHED channels).

Am I correct @paddybyers?

@maratal is currently working on #1494, which updates the library to use protocol version 2 and specification version 2.0.0. This specification version simplifies the criteria for performing an automatic re-entry, replacing RTP17c with RTP17f. I would suggest that we park the current bug and close it once we've implemented RTP17f as part of #1494.

Hey @maratal, would you mind confirming whether you agree with my above assessment, that #1494 will resolve this issue? I'll assign this one to you, to close after #1494. If you disagree with my assessment feel free to assign back to me.

@lawrence-forooghian Correct, RTP17c was replaced with RTP17f and implemented in https://github.com/ably/ably-cocoa/issues/1494


Command to deploy macOS/iOS universal binaries

This is a very rough version of a command to deploy dependencies and run lipo to create universal binaries.  This could certainly use feedback and testing on iOS builds.

See #58 for a different way to accomplish this.  If you like that approach better, we can close this PR.

 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.


Error generating initial cache, almost everything is marked as deleted

I just decided to try plexdrive out.
I am using an ubuntu 16.04 x86_64 and latest release version.
Something seem to go very wrong when I started building the initial cache
Here is part of the log-file (with --log-level  4)
The command I am running ./plexdrive--config /home/qnorsten/.config/.plexdrive -log-level 4 -fuse-options allow_other  /home/qnorsten/.plexdrive/
2017-05-08 15:24] DEBUG  : log-level            : TRACE
[PLEXDRIVE] [2017-05-08 15:24] DEBUG  : config               : /home/qnorsten/.config/.plexdrive
[PLEXDRIVE] [2017-05-08 15:24] DEBUG  : temp                 : /tmp
[PLEXDRIVE] [2017-05-08 15:24] DEBUG  : chunk-size           : 5242880
[PLEXDRIVE] [2017-05-08 15:24] DEBUG  : refresh-interval     : 5m0s
[PLEXDRIVE] [2017-05-08 15:24] DEBUG  : clear-chunk-interval : 1m0s
[PLEXDRIVE] [2017-05-08 15:24] DEBUG  : fuse-options         : allow_other
[PLEXDRIVE] [2017-05-08 15:24] DEBUG  : Opening cache connection
[PLEXDRIVE] [2017-05-08 15:24] DEBUG  : Migrating cache schema
[PLEXDRIVE] [2017-05-08 15:24] DEBUG  : Authorizing against Google Drive API
[PLEXDRIVE] [2017-05-08 15:24] DEBUG  : Loading token from cache
[PLEXDRIVE] [2017-05-08 15:24] TRACE  : Got token from cache {{0 0001-01-01 00:00:00 +0000 UTC 0001-01-01 00:00:00 +0000 UTC <nil>}  0001-01-01 00:00:00 +0000 UTC  }
[PLEXDRIVE] [2017-05-08 15:24] DEBUG  : Token could not be found, fetching new one
Go to the following link in your browser:
Paste the authorization code: [PLEXDRIVE] [2017-05-08 15:24] DEBUG  : Storing token to cache
[PLEXDRIVE] [2017-05-08 15:24] INFO   : Mounting path /home/qnorsten/.plexdrivetest/
[PLEXDRIVE] [2017-05-08 15:24] DEBUG  : Checking for changes
[PLEXDRIVE] [2017-05-08 15:24] DEBUG  : Getting change id from cache
[PLEXDRIVE] [2017-05-08 15:24] TRACE  : Got change id 0
[PLEXDRIVE] [2017-05-08 15:24] INFO   : First cache build process started...
[PLEXDRIVE] [2017-05-08 15:24] DEBUG  : Getting root from API
[PLEXDRIVE] [2017-05-08 15:24] TRACE  : Change &{true <nil> 0B2nuI1QwLYgmVjRkRXNWYWhTQXc 233891 drive#change 2017-01-23T19:39:57.625Z https://www.googleapis.com/drive/v2/changes/233891 <nil>  file {0 map[]} [] []}
[PLEXDRIVE] [2017-05-08 15:24] TRACE  : Change &{true <nil> 0B2nuI1QwLYgmdmxQTE5obUNhUms 266478 drive#change 2017-01-23T21:12:46.913Z https://www.googleapis.com/drive/v2/changes/266478 <nil>  file {0 map[]} [] []}
[PLEXDRIVE] [2017-05-08 15:24] DEBUG  : Deleting object 0B2nuI1QwLYgmVjRkRXNWYWhTQXc
[PLEXDRIVE] [2017-05-08 15:24] DEBUG  : Deleting object 0B2nuI1QwLYgmdmxQTE5obUNhUms
[And so on]
[Followed by ]
[PLEXDRIVE] [2017-05-08 15:25] INFO   : Processed 5060 items / deleted 5060 items / updated 0 items

Rest of log is similar until it say cache is built and no files can be seen in mount folder.

Please check if your files are really 'deleted' ones.
You can visit the Google Drive API documentation (https://developers.google.com/drive/v2/reference/changes/list). On the right sight you will see the "try this api" field. Just click execute and post the results here (please strip the filenames etc. out).

I think it fixed itself (I was also getting some 5xx errors too at the time). It has been running for over 10 hours now, but seem to progress. So guess I will close this for now.


Browserlibrary Docker container does not work on Apple Mac M1

Bug Description
When executing
docker container run --rm -v /path_to_your_working_folder/:/test --ipc=host --security-opt seccomp=/path_to_your_working_folder/seccomp_profile.json --user pwuser marketsquare/robotframework-browser bash -c "robot --outputdir /test /test"
command on Apple MacBook M1 following error is received:
WARNING: The requested image's platform (linux/amd64) does not match the detected host platform (linux/arm64/v8) and no specific platform was requested
Test
Test.Bugreport
Open Web Page And Crawl Links :: Basic test for crawl links on web... [ WARN ] Keyword 'Take Screenshot  fail-screenshot-{index}' could not be run on failure:
Error: Tried to take screenshot, but no page was open.
| FAIL |
Error: browser.newContext: Browser closed.
==================== Browser output: ====================
 /home/pwuser/.local/lib/python3.8/site-packages/Browser/wrapper/node_modules/playwright-core/.local-browsers/chromium-1005/chrome-linux/chrome --disable-field-trial-config --disable-background-networking --enable-features=NetworkService,NetworkServiceInProcess --disable-background-timer-throttling --disable-backgrounding-occluded-windows --disable-breakpad --disable-client-side-phishing-detection --disable-component-extensions-with-background-pages --disable-default-apps --disable-dev-shm-usage --disable-extensions --disable-features=ImprovedCookieControls,LazyFrameLoading,GlobalMediaControls,DestroyProfileOnBrowserClose,MediaRouter,DialMediaRouteProvider,AcceptCHFrame,AutoExpandDetailsElement,CertificateTransparencyComponentUpdater,AvoidUnnecessaryBeforeUnloadCheckSync --allow-pre-commit-input --disable-hang-monitor --disable-ipc-flooding-protection --disable-popup-blocking --disable-prompt-on-repost --disable-renderer-backgrounding --disable-sync --force-color-profile=srgb --metrics-recording-only --no-first-run --enable-automation --password-store=basic --use-mock-keychain --no-service-autorun --export-tagged-pdf --headless --hide-scrollbars --mute-audio --blink-settings=primaryHoverType=2,availableHoverTypes=2,primaryPointerType=4,availablePointerTypes=4 --no-sandbox --user-data-dir=/tmp/playwright_chromiumdev_profile-7preHm --remote-debugging-pipe --no-startup-w...
[ Message content over the limit has been removed. ]
[pid=245][err] [0524/082320.446736:ERROR:file_path_watcher_inotify.cc(329)] inotify_init() failed: Function not implemented (38)
[pid=245][err] [0524/082320.548537:WARNING:bluez_dbus_manager.cc(248)] Floss manager not present, cannot set Floss enable/disable.
[pid=245][err] [0524/082320.553055:ERROR:gpu_process_host.cc(967)] GPU process launch failed: error_code=1002
[pid=245][err] [0524/082320.553151:WARNING:gpu_process_host.cc(1273)] The GPU process has crashed 1 time(s)
[pid=245][err] [0524/082320.573965:ERROR:gpu_process_host.cc(967)] GPU process launch failed: error_code=1002
[pid=245][err] [0524/082320.574043:WARNING:gpu_process_host.cc(1273)] The GPU process has crashed 2 time(s)
[pid=245][err] [0524/082320.586672:ERROR:gpu_process_host.cc(967)] GPU process launch failed: error_code=1002
[pid=245][err] [0524/082320.586744:WARNING:gpu_process_host.cc(1273)] The GPU process has crashed 3 time(s)
[pid=245][err] [0524/082320.593181:ERROR:gpu_process_host.cc(967)] GPU process launch failed: error_code=1002
[pid=245][err] [0524/082320.593239:WARNING:gpu_process_host.cc(1273)] The GPU process
Test.Bugreport                                                        | FAIL |
1 test, 0 passed, 1 failed
Test                                                                  | FAIL |
1 test, 0 passed, 1 failed
Output:  /test/output.xml
Log:     /test/log.html
Report:  /test/report.html
According to https://docs.docker.com/desktop/mac/apple-silicon/ the problem is related to processor architecture:
In summary, running Intel-based containers on Arm-based machines should be regarded as “best effort” only. We recommend running arm64 containers on Apple silicon machines whenever possible, and encouraging container authors to produce arm64, or multi-arch, versions of their containers. We expect this issue to become less common over time, as more and more images are rebuilt supporting multiple architectures.
To Reproduce
Steps to reproduce the behavior:
NOTE!: Development environment with Docker and marketsquare/robotframework-browser image must be already installed!

Use MacBook WITH M1 processor
Unzip attached files to your working directory
Execute following command:
docker container run --rm -v /path_to_your_working_directory:/test --ipc=host --security-opt seccomp=/path_to_your_working_directory/seccomp_profile.json --user pwuser marketsquare/robotframework-browser bash -c "robot --outputdir /test /test"
See error

Expected behavior
Expected behavior is that test is passed without error when Docker image is used.
NOTE:

This test works on Mac M1 when using firefox instead of chromium and also when the test is executed without container using chromium.
This same error message is received also another M1 chip based Mac so it seems to be general
Probable solution: Marketsquare builds ARM64 based image with Chromium which supports ARM64 architecture

Desktop (please complete the following information):
macOS Monterey Version 12.3.1
MacBook Pro (14 inch, 2021)
Chip: Apple M1 Pro
Memory: 32 GB
Docker version: 20.10.12, build e91ed5707e
Additional context
Files what have been used to demonstrate the problem.
bugreport.robot.zip
seccomp_profile.json.zip

@xylix this belongs to your are of expertise, could you look at some point.

@allcontributors please add @SeppoTuura for bugs

Hi! Any updates on this issue?

@xylix Any news related to this?

Need M1 support for docker images, if you can please help @xylix

that should be solved now.

Closing due inactivity.


Fixes and Additions for DH2E Advanced

Changes / Comments
Fleshed out Gear tab.  Added new sections for Gear organization: Backpack/Vest & Stored Gear(weight not applied)
Fixed calculations that weren't running previously and added new ones in Gear tab
Fixed calculations that weren't running in Advancements tab
Updated a few Translation sheets with new and changed data-i18n labels.
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[ ] Does the pull request title have the sheet name(s)? Include each sheet name.  Yes, Dark Heresy 2nd Edition Advanced
[ ] Is this a bug fix?  Yes, as well as some small QoL additions
[ ] Does this add functional enhancements (new features or extending existing features) ?  Yes, in the Gear Tab
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?   No
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?  The Attribute changes and additions were to simplify the game for players.
[ ] If this is a new sheet, did you follow [Building Character Sheets standards]  N/A
(https://wiki.roll20.net/Building_Character_Sheets#Roll20_Character_Sheets_Repository) ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.

I accidentally deleted a line in the translation that is used in other locations:
From line 382 "current-u":"Current: ",
I replaced it with a different name in the Gear tab before realizing it was used in other locations.  Only in translation.json, I'm not able to attach it below, "We don't support that file type."


update xeus 5

It looks like cpp-tabulate exports the CXX standards publiclly, and is built with CXX 11 on conda-forge. This overwrites whatever CXX standard we can set on xeus-sql targets on Windows.


Await result in readFile

I updated to the latest API changes and found that the FileReader is not awaited in the readFile method, so in my application the file was always deleted before it could be read.
(By the way you were right, the changes were not as hard to deal with as I thought they would be.)

thanks, good catch.. I wonder why I didn't stumble on that 🤔
maybe readAsBytes synchronously creates a file handle on that file (ie. open), and only reads asynchronously?
did you do some other async action in the callback before reading?

did you do some other async action in the callback before reading?

Yes, the part in my FileReader that actually reads the file is a separate async function because I need to detect the encoding if the file is not UTF-8.

Okay that makes perfect sense, at least now i don't have to rush another bug fix release for AuthPass 😅 should still be stable as long as I've got a file handle synchronously


Cannot specify dependencies between requests in batch messages

Sometimes it is useful to specify dependencies between requests in the batch messages so that they are executed in the correct order. Facebook offers some properties to help one do just that such as name and depends_on. But this is not implemented in this library as far as I can see.
At the moment, the only way I can see to add the properties is to tack it onto the object returned by sendRequest.
What do you think?
Here's the relevant documentation
https://developers.facebook.com/docs/graph-api/making-multiple-requests#operations

Hi @ayoisaiah, thanks for the information of name and depends_on options.
For now, we use https://github.com/Yoctol/messenger-batch to keep the order of our requests.
We will look into name and depends_on for the future underlying implementation.

Hi @ayoisaiah, We already add the support to specify dependencies between operations using  name and depends_on. See #501.
We will release this commit in v1.0 version in early October.


How to support snippets on code actions expansions.

If I have the following code:
pub struct Foo;

impl std::fmt::Display for Foo {
}

fn main() {
}

and I use the code action Implement missing members, the generated code is:
pub struct Foo;

impl std::fmt::Display for Foo {
    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
        ${0:todo!()} // I want my cursor to move here.
    }
}

fn main() {
}

I am using vim-vsnip for regular snippets.
I don't know if this is in the scope of rust-tools or not.

It makes sense. Should this issue be closed? I'll keep an eye on the vim-snip issue.
Thanks!

Let's keep it open for now

I think rust-tools.nvim should get the snippet expansion function as a configuration item. (nvim-cmp does it).
And You can override the vim.lsp.util.apply_text_edits function. (I don't like this solution but I have no another idea...

I would expect it to work like this where you can configure how the snippet gets expanded: https://github.com/hrsh7th/nvim-cmp/blob/main/README.md?plain=1#L70-L81

I don't know if this is related, but when using rust_analyzer with lspconfig without rust-tools I get this expansion on "fill match arms":
match c {
    Test::A(_) => todo!(),
    Test::B(_) => todo!(),
    Test::C { a, b, c } => todo!(),
}

But with rust-tools this happens:
match c {
    Test::A(${0:_}) => todo!(),
    Test::B(_) => todo!(),
    Test::C { a, b, c } => todo!(),
}

Maybe it is possible to have the lspconfig expansion for now, until the cursor move is implemented, as that code is at least compilable?

I agree for now I can't really use rust-tools for that :s
Can we have an option to switch from one another ?

Fixed with https://github.com/simrat39/rust-tools.nvim/commit/01904961647c44a4111a856e5afd5aa83ce19621


Update to qscout tutorial notebooks (Jaqal program)

As part of https://github.com/Infleqtion/server-superstaq/issues/2285 explaining the jaqal_program capability in Superstaq for the user.
One circuit output change in the css notebook when base_entangling_gate=zz is due to https://github.com/Infleqtion/server-superstaq/issues/2659.
More explanation of the jaqal program to be added based on need/discussion.

Looks great! Approved, just need to address the small nits above.
This is a completely optional suggestion, but since this is a tutorial, you could add a pointer to the quantum volume paper.


Example yaml files apiVersion usage clarification is not enough

This is a...

[ ] Feature Request
[ ] Bug Report

Problem:
Some of the example yaml (e.g. https://raw.githubusercontent.com/kubernetes/website/master/docs/tasks/run-application/mysql-deployment.yaml) has a clarification that use apiVersion apps/v1beta2 before Kubernetes release 1.9.0. However, the suggested version won't work on earlier versions like 1.7 or 1.6 where correct version is apps/v1beta1. Using apps/v1beta2 on a 1.7.5 gives following error:
Error from server (BadRequest): error when creating "nginx.yaml": Deployment in version "v1beta2" cannot be handled as a Deployment: no kind "Deployment" is registered for version "apps/v1beta2"

Version 1.7 seems still used by many and only a two release older version. We should provide a better clarification.
Proposed Solution:
Provide a better clarification.
Page to Update:
http://kubernetes.io/...

Current solution is to have docs and sample yaml files maintained on a per-version basis. So in v1.9 doc, we are about to document apps/v1. To check docs for v1.8, you will switch to that version from website, there you will see apps/v1beta2 documented. There could still be defects though.


Sudden jumps in position due to GPS device error

Implement filtering of invalid data. More info:
https://www.traccar.org/forums/topic/sudden-jumps-in-position-due-to-gps-device-error-its-effect-to-traccar-server/

hi, is there and update for this issue? same problem reported for tk103 devices

It is already implemented in 17a023a62366ed4b85fd8f0f9cb41d591169c940.


Include model name in NLU parse_data

Description of Problem:

The policy and confidence are reported in the parse_data dict, but the model name is missing.
Overview of the Solution:

Add a new key model to UserMessage.parse_data

@RasaHQ/production-squad I'd argue that this should be done soonish as we are currently assigning model names based on a best guess base within Rasa X. So I'd move it from future to backlog

@wochinge Isn't the model lifecycle part of Enable squad? Or what aspect of this would put it under Practice CDD?

My reasoning was that the practice CDD squad needs that information to relate messages to used models. The lifecycle of the model is in my opinion separated from the conversation events this is creating. Total edge case though 👍🏻 Happy with having it in either squad.


Problem with entry files

I used broccoli-fast-browserify earlier and all was fine. Currently it does not work and I suspect it is related to the way how it traverse entry points. I have simple definition:
jsFiles = fastBrowserify(jsFiles, {
    bundles: {
        'index.js': {
            entryPoints: ['**/index.js']
        }
    }
});

But, for unknown reason broccoli-fast-browserify is now trying to load different files:
Error: EMFILE, open '/Users/hipertracker/tmp/react-es7/node_modules/react/package.json'
So, I updated my config to use more specific entry point:
jsFiles = fastBrowserify(jsFiles, {
    bundles: {
        'index.js': {
            entryPoints: ['src/index.js']
        }
    }
});

Now  I get another error:
Bundle specified by " index.js " does not have any entry files

I'll check this out.

I tried to build your project with broccoli serve and I get the error:
Built with error:
Error: Cannot find module 'react-semantify' from '/Users/caleb/Desktop/react-es7/tmp/babel-tmp_dest_dir-lwUJY1t7.tmp/app/components'
    at /Users/caleb/Desktop/react-es7/node_modules/broccoli-fast-browserify/node_modules/browserify/node_modules/browser-resolve/node_modules/resolve/lib/async.js:46:17
    at process (/Users/caleb/Desktop/react-es7/node_modules/broccoli-fast-browserify/node_modules/browserify/node_modules/browser-resolve/node_modules/resolve/lib/async.js:173:43)
    at ondir (/Users/caleb/Desktop/react-es7/node_modules/broccoli-fast-browserify/node_modules/browserify/node_modules/browser-resolve/node_modules/resolve/lib/async.js:188:17)
    at load (/Users/caleb/Desktop/react-es7/node_modules/broccoli-fast-browserify/node_modules/browserify/node_modules/browser-resolve/node_modules/resolve/lib/async.js:69:43)
    at onex (/Users/caleb/Desktop/react-es7/node_modules/broccoli-fast-browserify/node_modules/browserify/node_modules/browser-resolve/node_modules/resolve/lib/async.js:92:31)
    at /Users/caleb/Desktop/react-es7/node_modules/broccoli-fast-browserify/node_modules/browserify/node_modules/browser-resolve/node_modules/resolve/lib/async.js:22:47
    at FSReqWrap.oncomplete (fs.js:77:15)

I think this might be related to the react-semantify package's package.json file's main declaration:
{
...
"main": "lib/index.js",
...
}

The react-semantify package doesn't have a lib/index.js file in it's npm package. It doesn't have a lib/index.js file even in it's source repository.
It doesn't look like that's your problem, but this is the problem I ran into when trying to build it. Is this something you've run into?

@hipertracker: I removed references to react-semantify to test the rest of the build and everything worked correctly with the '**/index.js' entryPoint.
As for why using src/index.js as an entry point doesn't work, it's because your broccoli tree jsFiles uses Funnel to select files from src, but the destination directory is the root of the broccoli tree. So if you wanted to hardcode the entryPoint you would use:
entryPoints: ['index.j']

Or you can tell Funnel to output the files to a different directory you can use the destDir option:
ar jsFiles = new Funnel('src', {
  exclude: [new RegExp(/__tests__/)],
  include: [new RegExp(/\.js$/)],
  destDir: 'src'
});

@caleb thanks, it works. I've upgraded my project.


Characters

What is your question?
Bro, how do I add Characters to the Port? bone Al fnf psych engine

yes I also don't know how to add characters in the character editor in fnf

the mod folder is not working


install on debian arm64 fails (tries to use aarch64 instead of arm64)

running on debian on arm-m1
helm plugin install https://github.com/nikhilsbhat/helm-drift

errors out with
Installing helm-drift...
download url set to https://github.com/nikhilsbhat/helm-drift/releases/download/v0.0.7/helm-drift_0.0.7_Linux_aarch64.tar.gz

artifact name with path /root/.local/share/helm/plugins/helm-drift/helm-drift.tar.gz

downloading https://github.com/nikhilsbhat/helm-drift/releases/download/v0.0.7/helm-drift_0.0.7_Linux_aarch64.tar.gz to /root/.local/share/helm/plugins/helm-drift

  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
curl: (22) The requested URL returned error: 404
failed while downloading helm archive

Failed to install plugin helm drift
Error: plugin install hook for "drift" exited with error

Since the files listed in releases have arm64 instead of aarch64, this is prob the cause.

Closing this, since this commit should fixes the issue.


Tooltip over cursor

Is there an option to have the tooltip popup where the cursor is like the title attribute does?

As v-tooltip is based on Popper.js, that would probably be something more Popper.js-related in my opinion.
Actually there has been done already some work on their side, but because of some issues (for example when scrolling on the page) they closed it for now (that was back in 2017 already): https://github.com/atomiks/tippyjs/issues/32

This could work. Below is a first quick draft, on how it's possible with Popper.js and v-tooltip. Adding a few config options and fine-tuning to limit it's movement to the selected component/element.

Progress: Now configurable for each component individually. Also plays along nicely with pre-defined Popper.js placement options (top, bottom,...)

#143 the PR is up, anyone who's interested or has suggestions is more than welcome to provide feedback! I'd love to hear your opinion

there is also a codepen showing popper following the mouse. This could be also play well with other targets to follow e.g. a range input knob or progress bar.

Thanks @joernroeder, that is actually the demo I took as a base for the PR

Would love this. Could be very nicely implemented like clubhouse has. By restricting it to the X-axis. And maybe the ability to restrict it to the Y-axis as well.
(My mouse is apparently not screen-captured)


og:title & url fixed

og:title & og:url is now dynamic.

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

Comparison is base (34830a7) 80.33% compared to head (0510af6) 80.32%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1232      +/-   ##
==========================================
- Coverage   80.33%   80.32%   -0.02%
==========================================
  Files          95       95
  Lines        6602     6601       -1
==========================================
- Hits         5304     5302       -2
- Misses       1298     1299       +1

Files
Coverage Δ

superduperdb/base/logger.py
86.95% <80.00%> (-3.05%)
:arrow_down:

... and 5 files with indirect coverage changes

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


Ability to use a different model for tab completion

Validations

[X] I believe this is a way to improve. I'll try to join the Continue Discord for questions
[X] I'm not able to find an open issue that requests the same enhancement

Problem
For question answering, RAG, editing, code generation, etc. I want to use the biggest, slowest model I can fit on my machine, as that will have the highest accuracy. When I'm asking questions or using /edit, I expect that to take some time. A good model for this might be DeepSeek33B. This should be an instruction-tuned model.
However, for tab completion, I want it to be fast, even at the cost of accuracy like DeepSeek1B. Furthermore, I don't want it to be instruction tuned, as the next-token pretraining objective is perfect for tab completion.
Solution
In config.json, we should be able to specify which model we want for tab completion. Furthermore, the codebase should be able to handle sending tab completion requests to one model and all other requests to another model.

@jbohnslav this is already possible! Check out the docs here for setting up a custom tab autocomplete model. And here for setting up a chat/quick edit model
Let me know if I can help at all with setting up : )

@sestinj: Right now it is only possible with Ollama? Tried withLMStudio, but not able to suceed. Maybe you can help with the setup if LMStudio is supported?
What i configured in config.json and tried with different endpoints is (below models sections):
"tabAutocompleteModel": {
    "title": "Tab Autocomplete Model",
    "provider": "lmstudio",
    "model": "Phi2",
    "apiBase": "http://localhost:1234/v1/models"
  },

@JosefLaumer No need to change the API Base, but if you wanted to it should be http://localhost:1234/v1 (we default to this). I believe this would solve your problem:
"tabAutocompleteModel": {
    "title": "Tab Autocomplete Model",
    "provider": "lmstudio",
    "model": "Phi2"
  },

I have a similar problem,
I run text gen webui API and it just tells me that openai doesnt have Mistral-7B as a model.
When I add text-gen-webui as provider it tells me its unknown

I tried using second-state/StarCoder2-15B-GGUF/starcoder2-15b-Q4_K_S.gguf with lmstudio, but it didn't generate any usable output, whereas it was working just fine with ollama.
I think there are missing some crucial setup instruction in order to get this working with lmstudio.

I'm using a remote machine to run Ollama Models, trying to use the same model for chat and auto tab completion.
"models": [
    {
      "title": "Ollama Remote",
      "model": "codestral:22b",
      "completionOptions": {
        "keepAlive": 3000000,
      },
      "apiBase": "http://<IP_ADDRESS>:11434",
      "provider": "ollama"
    }
  ],
"tabAutocompleteModel": {
    "title": "Ollama Remote",
    "provider": "ollama",
    "apiBase": "http://<IP_ADDRESS>:11434",
    "model": "codestral:22b",
    "completionOptions": {
      "keepAlive": 3000000,
    }
  },

When opening chat, the model is loaded, but when i hop in the editor then tab auto complete model is loaded and previous one is unloaded. this takes huge time,
how can i use the same model in Chat and Autocomplete without reloading the model in ollama?

@craftpip given your config here it doesn't seem like the unloading/reloading should happen, but if anything it might be that the different values of the keepAlive parameter are causing this. It's just the first thing that comes to mind, but I would try adding the same keepAlive value to your chat model

This is now possible (in the latest VS Code Pre-release) by using an array for tabAutocompleteModel, and then clicking on the "Continue" button in the status bar


Update textfield.py

Now, when you set the text, hint_text goes up (text=self.set_text)

This has already been fixed in the master branch - https://github.com/kivymd/KivyMD/commit/8d3cd3279047223a92c71884bcab221a46bf5661#diff-a5f4aff9627246307e242d5d49a08ca67d44d38d2ea330d2836c48722b3c87c0


fix: Simplified level assignment logic in BrowserConsole

Description
I’ve updated the logic in the BrowserConsole function that handles the level property. Previously, there was a redundant check for opts in the conditional expression. Since we are already checking for the existence of opts?.level, it's unnecessary to verify opts again.
I’ve simplified the assignment to use the nullish coalescing operator (??), which directly assigns the value of opts.level if it exists, or defaults to "info" if it doesn't ;)

 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.


When listing workflows show result of a query in a column

It is very useful to be able to see a list of workflows with the correspondent statuses.
It can be done by emitting a query to a workflow execution for each member of the list. As listing is paginated it shouldn't increase latency too much. If it does then the status should be loaded asynchronously after the list is already rendered.

Another option is to show results of a DescribeWorkflowExecution call. From its result determine what to show in the column. It might require some server side changes if some info is missing.

You're talking about the Workflow list screen currently, correct? The status of the workflows is already an input parameter. Did you mean some other piece of information per workflow?
The workflows list is 100 per page, so firing 100 ajax request that most browsers process 6 at a time per domain, it's not terribly fast, but it can be asynchronous.

With the new memo feature (https://github.com/uber/cadence/pull/1663) we don't need to execute a query per workflow in the list. This issue should be repurposed to show the memos in the list.

Closing this, might be revisited on the new iteration of cadence-web.


Instead of Getting data from file. How to get it from Neo4j server

Hi,
I would like to get data from Neo4J server directly when i click on each node. how can i do that.

if you have sample, can you send me pls

Example
Not tested.

How to get data from Neo4j server initially. is there any way we can do without chnanging the code like using options.neo4jDataUrl.

Actually by implementing options.onNodeClick you're not changing the code but adding a custom functionality through the options, the core remains the same.
Alternatively, you can use options.neo4jDataUrl = 'http://example.com/data.json' to get your data from the server initially. In any case, if you want to get new data when a node is clicked, you need to implement options.onNodeClick.
If you still have doubts, share some code to help understand what you need exactly.

We want to display all relations first and when we click on relation it has to all the nodes. All this info we should read it from neo4j server.

I'm not sure if you can display a relationship without both startNode and endNode, but I think it's not possible... What you can do instead is to get all the nodes the first time and then use css to hide some of them. In this way, you'll be able to unhide them again when clicking the relationship easily.


Calling isOptional() on a coerce.bigint throws an error

Try the following code:
const bigintSchema = z.coerce.bigint();
const isOptional = bigintSchema.isOptional();

That code should be able to run IMO, but now it throws an error, TypeError: Cannot convert undefined to a BigInt

Kind of related to the following:
https://github.com/colinhacks/zod/discussions/1897
https://github.com/colinhacks/zod/discussions/1856
I can't really see the benefit of why Zod shouldn't catch these errors and handle them in the same way as you use to with most of the other errors that can occur. What are the drawbacks of having Zod a bit smarter and having this as the default implementation of coerce.bigint?
        try {
            return BigInt( value )
        } catch ( error ) {
            ctx.addIssue( {
                code: 'invalid_type',
                expected: 'unknown',
                received: value,
                message: `Can't be parsed to BigInt`,
            } )
        }

@marcus13371337
Was my answer satisfactory? or would you like to leave this issue open as an enhancement request?

Yes, I was able to get around the problem by creating a custom bigint-schema. But I don't see why zod, shouldn't be able to reply of a schema is optional without crashing (which happens to be a coerce bigint), to me that's clearly a bug

same about isNullable
const f = z.coerce.bigint();
f.isNullable();


MAgent2 integration

Hi! I'm trying to integrate BenchMARL with the MAgent2 environment, but I've encountered some problems. The main issue is that when I execute experiment.run(), I get the following error:
File [~/miniconda3/envs/marl/lib/python3.11/site-packages/benchmarl/experiment/logger.py:134] in Logger.log_collection(self, batch, task, total_frames, step)
    132 to_log.update(task.log_info(batch))
    133 # print(json_metrics.items())
--> 134 mean_group_return = torch.stack(
    135     [value for key, value in json_metrics.items()], dim=0
    136 ).mean(0)
    137 if mean_group_return.numel() > 0:
    138     to_log.update(
    139         {
    140             "collection[/reward/episode_reward_min](/reward/episode_reward_min)": mean_group_return.min().item(),
   (...)
    143         }
    144     )

RuntimeError: stack expects each tensor to be equal size, but got [1] at entry 0 and [16] at entry 1

While I was trying to fix it, I found that the error I'm encountering is likely due to BenchMARL's expectation of equal "done" signals for both adversary groups. This means BenchMARL doesn't currently support scenarios where the number of agents that die in an epoch differs between groups.
I'd like to know if there is a solution or wrapper that can cover this case, or if there is a way to add it.

This issue may be related to feature two in #94 (support for variable number of agents).

Hey! Thanks for opening this and I think we can definitely fix this issue. The code in the loggers can defineitely be more flexible and it is ablout time I seriously imporved it.
To understand the issue better I need more context. In particular:

Do you have a variable number of agents (dying agents)?

Benchmarl does not currently support this, but it supports some agent being done before others. In this case of some agents are done before others, the rollout countiunes until the global done is set (in pettingZoo this can be computed with any or all on the agent dones).  If some agents are done and the rollout continues these agents will be required to continue acting but you can either ignore thier action in the env and give a reward of 0 or mask their actions such that only the no-op action remains available (like in SMAC).
If you could provide more details on your env, I'll be able to help.
In general, I will already start making a PR that makes the code snippet you got stuck on more flexible as I think there is a lot i can already improve there.

In this environment, some agents finish before others (when they lose all their health points). Once an agent is done, a truncation or termination flag is set and returned every iteration until the episode ends. This effectively ignores actions from finished agents, as there is no no-op action available.
This causes a problem when logging. The logger sets the done flag using experiment.logger._get_done(group, batch), and then calculates the mean episode reward per group with episode_reward.mean(-2)[done.any(-2)]. Since some groups have more agents that finished earlier, the tensors end up with different sizes, causing an error because torch.stack requires tensors to have the same shape.

Very clear got it. I will fix this in #136

If you want to try the PR as of now it should work well in collection logging. (you can try without evaluation)
I still have to fix evaluation logging.

Thanks for bearing with me.
PR is ready and the description gives an overview of how things are computed now

Btw if once you finish you would like to contribute MAgent2 to BenchMARL we would love that!
Otherwise, could you share your implementat? I might consider adding its wrapper in the future

Sure, my intention is definitely to contribute. However, I still need to investigate some performance issues within MAgent2. Once I've finished training some models, I'd be happy to share my contributions.


CentOS installer doesn't support kernel-lt/kernel-ml

As mentioned in #320, the current implementation of the CentOS installer doesn't support the use of kernel-lt or kernel-ml from the Community Enterprise Linux Repo (ELRepo), as it assumes the guest is running the standard kernel package. These kernels also have their own matching support packages (kernel-lt-{devel,doc,headers,tools} and kernel-ml-{devel,doc,headers,tools}). To install these packages, the elrepo-kernel yum repo must be enabled.
An important note: older versions of these packages are NOT left available for download. So if the guest isn't running the most recent version of their kernel, something like yum install -y kernel-lt-devel-`uname -r`  will fail.

Yeah, is there any way to downgrade? 0.17.2 totally broken my company's processes (which rely on kernel-ml-*)

You don’t need to downgrade. Just put the “installer” config in your Vagrantfile set to the RedHat installer class.


Updating code example for logging on rotation

'rotating-file-stream' api has changed and most people will install the latest. So the example should adapt to that.
Also the previous code example was incomplete and left many people wondering about details of implementation.
My example is complete and short. Anyone can copy-paste it and get it immediately working. No need to go to the 'rotating-file-stream' library and read its documentation!

Also the previous code example was incomplete and left many people wondering about details of implementation.

Can you elaborate on that? What details were missing? Just reading the changes here, I'm still not sure what exactly is going on in that regard.

My example is complete and short. Anyone can copy-paste it and get it immediately working. No need to go to the 'rotating-file-stream' library and read its documentation!

I'm confused, as I believe the current documentation is already that way? At least, I copied out the example and it worked for me without modification. Can you elaborate on this point?

Thank you for the feedback. That all sound good, but sounds like feedback to make to the rotating-file-stream about the default options. I would rather keep the example simple, and let users expand it as they need. The example is just to get it to work.
On your rotation note, replacing the same file is still rotating the file; it sounds like your need it to keep the older data, which is fine, but not everyone needs that.
The goal of the example is to use as little non-morgan code as possible to show how to use an external stream with morgan.


Feature Request: Option to disable sync while on battery

Just as SyncTrayzor handles metered connections to save data, I think it may be a good idea to provide similar mechanism to save battery life.
While working on laptops (on battery) and frequently saving small edits of files, ST becomes a major battery killer (probably rightfully so due to encryption). It would usually cut battery time to half.
So it may be nice to add "power source awareness" to the wrapper do be able to (optionally) disable sync while on battery, and re-enable it when back on AC.

Yep, fair enough request.

Please consider this a +1 from me.

Additionally, it would be nice to have an option to automatically Pause if the system is in Battery Saver mode.

Yes please. Or an option to right-click on the tray icon and pause would also work.
And thanks for this app...


Image explains itself

There's something wrong about code, I don't understand. Can someone suggest a solution?
(it's in the newest commit in VS branch)

Issue  closed
Trivial Solution


wip: admin-panel

Tasks:

[ ] Add Contributions Mutation
[ ] Add UserForm Mutation
[ ] Add Media UI + Mutation
[ ] Add Docs Support with permission handling

Additional Context
Related to admin panel

@Shurtu-gal @120EE0692 please make an umbrella issue (like this) for this and track all related issues in it. Either convert this into it or close this and make a new one.
Also, let's make this a level 1 priority right now.


Swagger API review - Enable review gen for all branches

Swagger API review is generated automatically from swagger PR only if target branch is one of the release branches. We should enable A{PI review gen for swagger for all target branches.

Done.


What is the effective SEW when considering EDIV and widening?

Suppose there is a hardware which supports the maximum configuration like SEW=32b, FLEN=32b, EDIV=8.
there is a sentence mentioned in the chapter 14. Vector Floating-Point Instructions

If the current SEW does not correspond to a supported IEEE floating-point type, an illegal instruction exception is raised.

I have a question about the current SEW?  Would it be the effective SEW?
A code sequence like below
vsetvli x0, x0, e64, m1, d4
vfwadd.vv v3, v2, v1  # Widening FP add

the effective SEW for vfwadd would be 64*2/4 = 32 and treat it as a legal instruction?
and this case
vsetvli x0, x0, e32, m1, d2
vfwadd.vv v3, v2, v1  # Widening FP add

the effective SEW should be 32*2/2 = 32 and still a legal instruction?

Yes, that's the intent.
EDIV is an optional extension that will need more work in definition.


Parallax effects with custom Layer heights and not using top-level Parallax for scrolling

We have a site that has some specific needs for it's sizing of sections. A few aren't compatible with Parallax. Namely we have both min-height and max-height requirements.
How hard would it be to have Parallax let you do the following:

Pass in a Layer height
Allow it hooking into your own custom scroll layer

Number two is important because we want to also keep scroll bounce / have the body scroll.

@natew From what i remember it simply goes 100% of the parent container, so it should respect the height if its either relative or absolute. The parallax in the main page demo for instance sits in a little grid-box.
If you mean changing the height of a layer in general, then you can do this with the factor prop, it defaults to 1 (100%), 2 would be 200%, 1.5 150%, etc.

You can also copy this file: https://github.com/drcmda/react-spring/blob/master/src/Parallax.js and change styles - or perhaps make a PR to make it more flexible. If you're going for a local fork, all the imports there you can get directly from react-spring.

@drcmda I should have specified better my specific use case. I'll investigate and comment back if this already works but I thought it didn't.
Basically: I have sections that are:
height: window.innerHeight;
max-height: 1200px;
min-height: 600px;

The max and min to me looked impossible as Parallax was assuming effects based on window.innerHeight. So I think the real thing would be to be able to define a "viewport height" of your own and have parallax use that. But of course I can fork and implement.

It shouldn't have anything to do with innerHeight/Width - it takes the height of the parent container - if its either relative or absolute. Like this: http://react-spring.surge.sh/ (upper left, upper middle). Wouldn't this fulfill it?

for clarity of the ongoing issues i'll close and re-open again when new info comes up

Hi, I've played a bit with the code and made a version that works with "dynamic" heights.
It also plays well with responsiveness. Though, it's still not tested properly (I just tweaked it for my needs, I cannot guarantee it works properly for others) and some props are not being used anymore (like the factor prop).
However, here you go: https://github.com/0m15/react-spring


Round-tripping consecutive markdown cells from notebooks

We need to round-trip notebooks via jmd back to Notebooks.
This works very well except that currently (as far as I can tell) there is no way of preserving the distinction between consecutive Markdown cells in a notebook, since they just become one big text blob in the jmd.
Would it be possible to separate consecutive markdown cells with some kind of marker, eg ----?

It would also be useful to start new doc chunks at every heading (## etc)

I have added support for comments in markdown parser so it would now be possible to use comments to separate cells in markdown without breaking output in other formats.
We could use something like:
<!-- cell -->

as the separator.
If either of you would have time to make pull request you need to change writers.jl to split doc chunks at these delimiters around here:
https://github.com/mpastell/Weave.jl/blob/edd558de0e7028be9975910ee9c9c0049d22341f/src/writers.jl#L93


TS Jest unsupported version warnings when running tests

When running tests in the v2 branch the output below is noticed.
ts-jest[versions] (WARN) Version 4.0.5 of typescript installed has not been tested with ts-jest. If you're experiencing issues, consider using a supported version (>=2.7.0 <4.0.0). Please do not report issues in ts-jest if you are using unsupported versions.

This is due to a recent upgrade of the Typescript version without bumping the version for TS Jest. The following changes are recommended:

Bump ts-jest from 25.2.1 to 26.4.4
Bump jest from 25.1.0 to 26.6.3
Bump @types/jest from 25.1.4 to 26.0.16
Change tsConfig key in jest.config.json to tsconfig (former is now deprecated)

Upgraded and fixed in beta 26!


Update SkiaSharp to 2.88.7 Fixes jpeg creation bugs Version 4.1.2

Depends on: https://github.com/Mapsui/Mapsui/pull/2413

This PR has all the changes in it from this PR. That is just fine. It is better to put it in small steps working with PR dependencies. The other PR should be reviewed first. Then this one should be updated.
I wrote something about PR dependencies in the guidelines:
https://github.com/Mapsui/Mapsui/pull/2420
There is just one this to improve, and that is to mention this dependency in the description (the first comment). ... I will add it right now.

I merged the previous branch and updates this one. Now I have only one changed file to review. This works well.


Bug: Test Case Result UI. Graph does not extend to negative values on vertical axis

Affected module
It impacts the UI
Describe the bug
The Graph in Test Case Result UI does not extend to negative values
To Reproduce
Create a test with min column value a negative number.
Screenshot showing the issue:

Expected behavior
The vertical axis of the graph should be extended to negative values
Version:

OS: run on docker on ubuntu 23.04 machine
OpenMetadata version: 1.3.1
OpenMetadata Ingestion package version: 1.3.1

Additional context
It is a visualization only issue

This has already been taken care of with #15368, and will be part of 1.3.2


feat: island sub-directories

This change adds support for sub-directories in the /island folder and simplifies the path collection logic for routes and islands.
Closes #1004.
Supercedes #1030.

I suspect the current CI errors on Windows relate to paths. Any help from someone with a Windows machine is welcome.

PTAL, @lucacasonato


All method serialized .

Hello Mr. @jrief: Thank you very much for your response. I'm working on finding how to integrate django-shop with three-part applications, do API or api access that exist with the framework REST.
I can found that you and your team are making it easy with the rest framework, like views and rest framework serializer class or method for django-shop.
Problem:
We need to access others methods like add, delete, update of the product, ¿are these ready like views are ready of productcatalog of django-shop?: info say "15.2. Every URL is a REST endpoint"
¿how we can post a Product polimorphy way REST framework?, like others function pre estableshed like AddSmartPhoneToCartSerializer for example but for publicate a new product.
Now only can see: rest framework onlye Allow GET, HEAD, OPTIONS. (?format=api)
With REST framework How can get or show (serialized and json) all cms pages (id of page and name) to assign when I post a product (by REST) and set all or some of these with the api way REST?.
We need to accessl all funtionality with REST Framework: like "Post_office" and add, delete, update products for example.
Thanks again.

could you please be more specific. What exactly are you trying to "add", "delete" and "update"?

is this question still relevant to you? Otherwise please close.

Closing for inactivity.


connectivity: Add more tests

Description
Couple of more connectivity tests, please refer to individual commit for more details.
I will remote temp commit after first round of review.

All the tests are passed, waiting for reviews then I will remove temp commit.

Just to help the protocol, I've added dont-merge/preview-only while there is a temporary commit. This way, even if you had all reviews :+1: and all tests passing, it's still communicating that this PR should not be merged in the current state. Feel free to remove the label when you have dropped the temporary commit.

GKE was passed previously after timeout increase, all comments are incorporated, temp commit was removed. Marking this ready to review again.


[Bug] small pic appearance in menu

Still after someone fix this issue for other pages in appearance page small logo is there.
Also too less padding given in search button

Please assign me this issue and label gssoc-ext, hacktoberfest

please assign me with this task @aslams2020

Hi @vedant3337  You can start working on it.
I got my account back today. Sorry for the inconvinience.


Integration: Add image build workflow

This change adds a new workflow that builds the volume test images and pushes them to a remote registry. This action needs to be manually triggered and can be configured to push to any remote repository to which the google cloud credentials configured as secrets have access to push.
For Windows images, a single image builder helper is used. Hyper-V isolation is leveraged to build images for all supported targets.
The easiest way to use this workflow is to fork the repo then configure your own secrets, run a build, test the images, then mirror the images to gcr.io.
Depends-on: https://github.com/containerd/containerd/pull/6274
Signed-off-by: Gabriel Adrian Samfira<EMAIL_ADDRESS>

/cc @estesp
/cc @kevpar
/cc @dmcgowan
/cc @mikebrow

Do we need to trigger this now? Windows CI is failing seemingly due to https://console.cloud.google.com/gcr/images/k8s-cri-containerd/GLOBAL/volume-copy-up@sha256:62399bfb9fc29c28412a53f208cf2045c88286ba6ca97be8e1abf6686b7ee450/details?tag=2.1&tab=pull not having windows images in the index

Do we need to trigger this now? Windows CI is failing seemingly due to https://console.cloud.google.com/gcr/images/k8s-cri-containerd/GLOBAL/volume-copy-up@sha256:62399bfb9fc29c28412a53f208cf2045c88286ba6ca97be8e1abf6686b7ee450/details?tag=2.1&tab=pull not having windows images in the index

Yes. This will need to be run, and images will need to be pushed to gcr.io/k8s-cri-containerd.
This is only a partial fix. There are 2 more parts needed in order for tests to pass:

The following PR will need to merge: https://github.com/containerd/continuity/pull/188
After images are pushed, tags will need to be updated in order to make use of the updated images, here: https://github.com/containerd/containerd/blob/main/integration/common.go#L54-L55

After which tests should pass, and Windows containers will preserve file ownership and security information present in images. It will also allow us to enable this test: https://github.com/containerd/containerd/pull/6275
The root cause of volume tests failing was two-part:

Images were being created on Linux. Security info would not get attached to files.
copyFileInfo() from continuity/fs was largely a noop on Windows

@gabriel-samfira the latest failure on the main branch seems to be that this repository is no longer being created with Windows images (https://console.cloud.google.com/gcr/images/k8s-cri-containerd/global/volume-copy-up?tab=pull). I am not sure where there is even being built today

Looks like Windows variants (3 of them) are being built, but the manifest (index) is only populated with Linux entries. The new workflow added, along with the Makefile and associated content in integration/images looks like it should work, but given the images have only Linux components I wonder if the REMOTE_DOCKER_URL setting is not working properly?
https://github.com/containerd/containerd/blob/main/integration/images/volume-copy-up/Makefile#L47-L52

There also seems to be a configuration missing. I tried to run this workflow but it was missing some authentication.

Looks like Windows variants (3 of them) are being built, but the manifest (index) is only populated with Linux entries. The new workflow added, along with the Makefile and associated content in integration/images looks like it should work, but given the images have only Linux components I wonder if the REMOTE_DOCKER_URL setting is not working properly?
https://github.com/containerd/containerd/blob/main/integration/images/volume-copy-up/Makefile#L47-L52

The workflow makes sure to pass REMOTE_DOCKER_URL as an argument when building the manifest and pushing the images, which ensures that the windows versions are added to the list:
https://github.com/containerd/containerd/blob/main/.github/workflows/build-test-images.yml#L152-L153
The result should be that the remote manifest includes all supported images. Here is a test run in my fork:
https://github.com/gabriel-samfira/containerd/runs/4321636494?check_suite_focus=true
and the resulting images are pushed to an artifact repo I have access to (public pull is allowed):
europe-west3-docker.pkg.dev/k8s-personal-222212/containerd
Here is the output from docker manifest inspect:
https://paste.samfira.com/public/p/iLuMvDcBeKUU8Fkn0kWbCGrn <-- volume-copy-up
https://paste.samfira.com/public/p/H3NPlQiAUNmAJ16v9bxY9SDM <-- volume-ownership


[Menu] Search is not working for the desktop from the main search button in the mega menu

This is for the search menu on the magnifying glass. It works if you hit enter, but not on when clicking the search button

Addressed in https://github.com/hlxsites/moleculardevices/pull/662


DBN pretraining for NN - Regression

I am using the toolbox to perform regression and using DBN pre-training before NN fine-tuning.
I use linear output units and have added a GBRBM layer as input-layer by replacing the hid-to-vis transition with a normal-distribution sampling. I have also changed the nntest.m to output RMSE with respect to the actual values to infer and the inferred values.
My issue in short:
No matter the size of the training set or the number of epochs, the network converges, after only 2 or 3 epochs to an almost steady state. The output vector ends up being identical no matter which test vector is applied.
Any ideas as to what may go wrong? The DBN-part of the training seems to work as intended.
Do I have to make changes in the backpropagation and gradient code?
Any input would be highly appreciated!

have solved this problem?

Yes, thank you for reminding me. I had completely forgotten to close the question.

Can you tell me how to solve it ? I have met the same problem you refered above.Thank you!

There wasn't really an issue with the code for my part, but rather the way the layers were initialized. The initial hidden biases were set to too large values which lead to the sigmoids producing more or less the same outputs due to the fact that the sum of the activations was smaller than the bias by several magnitudes. Also, the initialization of the regression layer was done with the pseudo-inverse method, but this is a very unstable approach and due to the nature of my training set, it gave values of size (10^8, 10^10). Initializing this to small random values and setting a lower initial hidden bias solved the problem for me.

Thank you very much!


Pre-forking state refactor

Usage related changes

Needed for #195
Temporarily disable --state-archive-capacity full
Improve performance: close #308

Development related changes

Ignore tests that rely on full state archive (9 tests)
Unify declaring and storing of class in StarknetState
Unify state diff calculation and state clearing

Dirty vs clean state terminology replaced by relying on CachedState commitment diff
No more explicit applying of state diff
Class diff done via new utility struct CommittedClassStorage (property of StarknetState)

Introduce DictState

Copy of blockifier's DictStateReader testing utility with added State (writing) functionality
Used for predeclaration and predeployment

Prevents invalid state diff (since state diff is now calculated using CachedState diff

Currently CachedState always takes DictState as its underlying state, but in a future PR this will be made generic to support a ForkedState that will be the same as DictState, but will also an origin

Setting initial balance is now a part of account predeployment
Remove StateChanger: where applicable, replaced with blockifier::State
Remove StateExtractor: where applicable, replaced with blockifier::StateReader
Introduce CustomState and CustomStateReader

Seems redundant as no function parameters use impl CustomStateReader, the methods of these traits could be implemented directly under StarknetState

Since we are more relying on CachedState now and many of its methods require &mut state

This mutability requirement propagates to other parts of our code
Lock acquiring changed in some places from state.read() to state.write() accordingly

Proposals regarding things not changed by this PR (perhaps these need separate GitHub issues):

Rename trait Deployed to Predeployed and its methods accordingly (predeploy, predeclare)
Block context needs to be stored together with state for correct old state calls
Do something about StateDiff:

In production it is only used to be converted to ThinStateDiff
Other use is only in testing

Checklist:

[x] Applied formatting - ./scripts/format.sh
[x] No linter errors - ./scripts/clippy_check.sh
[x] No unused dependencies - ./scripts/check_unused_deps.sh
[x] Performed code self-review
[x] Rebased to the last commit of the target branch (or merged it into my branch)
[x] Documented the changes
[x] Linked the issues which this PR resolves
[x] Checked the TODO section in README.md if this PR resolves it
[x] Updated the tests
[x] All tests are passing - cargo test

While working on #360, I spotted a bug in state diff generation. Converting to draft until fixed.


Java.lang.NoClassDefFoundError: scala/collection/GenTraversableOnce

Tried several different scenarios but still getting this error. Using spark 2.1 with scala 2.11.7 and play-json 2.11.

I am gonna close it until some more context information is provided or steps to reproduce.


Fix typos + better readability

This PR fixes a couple of typos and introduces spaces between emojis for better readability.

Thanks for the fix!