show python requirements not found message and option to install

Shows relevant error message when Python dependency packages such as jupyterlab are not found.
Provides option to copy the command to clipboard and installing the missing packages for user
Adds ability to open app logs from Settings dialog and progress views
copyable-span web component to support copying to clipboard from web views
Additional logging for shell executions and environment errors

I started testing it, but did not get to it - I will report back tomorrow.

So when I tried to open an environment with an older version of JupyterLab:
jupyterlab             3.4.4
jupyterlab-server      2.15.0

I got this distressing screen:

which has no environment picker so I cannot change the environment (without restarting app). I was able to use the hamburger menu to go to settings and grab the logs using the new "Show logs" button.
source "/home/krassowski/.pyenv/versions/<sanitized-out>/bin/activate"
        python -m jupyterlab --no-browser --expose-app-in-browser --ServerApp.port=43547 --ServerApp.password="" --ServerApp.token="jlab:srvr:<sanitized out>" --LabApp.quit_button=False --JupyterApp.config_file_name="" --ContentsManager.allow_hidden=True
[2023-10-06 09:31:09.239] [debug] Server launch parameters:
  [script]: /tmp/jlab_desktopIaMVVT/launch.sh
  [options]: {<sanitized out>}
[2023-10-06 09:31:09.589] [error] Error: Reader received error. Reason: Error during socket reconnect: code = 1006, reason = ,,
[2023-10-06 09:31:09.589] [error] Connection closed by the server
[2023-10-06 09:31:09.589] [error] Connection unexpectedly disconnected
[2023-10-06 09:31:10.074] [error] Failed to start server Error: Jupyter Server process terminated before the initialization completed
    at ChildProcess.<anonymous> (/opt/JupyterLab/resources/app.asar/build/out/main/server.js:306:28)
    at ChildProcess.emit (node:events:513:28)
    at Process.onexit (node:internal/child_process:291:12)
[2023-10-06 09:31:10.359] [warn]  Showing error: [object Object]
[2023-10-06 09:31:16.357] [warn]  Showing error: [object Object]

@krassowski was this environment shown in your environment selection list on top right or did you browse the python path to select it? This failure is not due to version incompatibility, running the python -m jupyterlab somehow failed.

It's both. I do not have any suggestions in the environment list auto-populated because I use pyenv so I always have to browser for path to select it (hence I wish there was auto-detection for non-conda environments). But once I select it it stays in the environment selection list forever. It used to work previously, so I guess there may be something broken.
Re-iterating, running python -m jupyterlab with exact line as above from command line works fine.

I tried another environment which starts up fine with jupyterlab 3.6.3. My hypothesis was that it was the older JupyterLab version, so I downgraded. After downgrading the jupyterlab in the environment the picker tooltip still says that it sees 3.6.3. This is odd. How does it get jupyterlab version?

I tried in a conda environment with the same versions you posted above, and it worked just fine.

Ok, I know how to repro it reliably. It is a conflict of versions problem - it was happening when I stated jlab from my jupyterlab dev. Because of path overshadowing and not using path isolation it was pulling JupyterLab 4.0 Python executables, but then I selected an old environment to test it and it failed with the screen above.
So this is not a typical failure scenario, but it does show that this error screen needs work (in particular it would be lovely if it had the environment selector).

@krassowski could you take another look as I pushed some updates addressing your comments.

I think I am seeing an issue with PYTHONPATH. Previously I used to set it like this (simplifying) :
PYTHONPATH=/my/project/root/dir:$PYTHONPATH jlab

but this does not seem to work now? I am not 100% positive if this is due to this PR or because some setting got lost during uninstall/install sequence, but given that this PR touches python path logic it could be related.
This is kind of a deal breaker for me, but possibly not a common usage pattern. Note that configuring this in settings does not seem to work either. Is Python path overwritten at some point, or am I fooling myself?
I am going to try an older version now.

I think I am seeing an issue with PYTHONPATH
I tested both from CLI like your example and also from the settings dialog and both worked fine. This PR doesn't make a change to environment variable setting.

Thank you for looking into this! Apologies for the delay with the actual review, will take another look soon.

A spinner, or even better logs during live installation would be a great future enhancement.

there is actually a subtle animation of the orange part of the logo, like heart beat. but probably too subtle to notice.


merge branches

These branches need to be merged.

merge 5 commits into banbanandroid:master from AliyunContainerService:master


Bump chalk to use latest git to get upstream fix

Chalk very recently (like an hour ago) merged a fix that prevents rust analyzer from panicking.  This allows it to be usable again for code that hits those situations.  See #6134, #6145, Probably #6120

Hm, I wonder if we could get the chalk folks to tag a release.

Can you update to the latest chalk release?

Updated

bors r+


offer in progess rules on doc generation

Nice addon here 🔥

Yep, I try to make this more confortable so contributors can be freed from brainless activities


questions about transweather

Thanks for fixing and sharing!
When i test the provided weight file by test_raindropa.py  on test_a dataset,  i get a val psnr of 28.88. Is here something wrong with my test?

nono,Your test is probably right
I  trained in a datatset,which contains 18000 pics including 3 types weather , in 200 epochs and it has converged.
And i get the same psnr
maybe you can train it by yourself,to check whether something wrong in my train
if you improve the psnr,please tell me  (￣▽￣)~*

I've trained a better weight of transweater, you can down it load from:
https://pan.baidu.com/s/1yBn1GyOTLAx4oS6-kbk6xw?pwd=18o3

I've trained a better weight of transweater, you can down it load from: https://pan.baidu.com/s/1yBn1GyOTLAx4oS6-kbk6xw?pwd=18o3

score of raindrop is test on test_.zip  from raindrop dataset

Thank you for your promotion!
I'll mention your work in the readme file

Thanks for both of your contributions! The reason for the incompatibility may be that you tested on different channels. It seems that the PSNR is much higher on Y channel than on RGB channels.

I've trained a better weight of transweater, you can download  it from: https://pan.baidu.com/s/1yBn1GyOTLAx4oS6-kbk6xw?pwd=18o3

score of raindrop is tested on test_a.zip from raindrop dataset

refresh the link
链接：https://pan.baidu.com/s/1joD-vNwtavSDLjvFuZtRHg?pwd=1z65
提取码：1z65


prototype maps

first cut, gifs of 1) total use by state, 2) Irrigation by state

Industrial

public supply

irrigation

Thermoelectric

Now with updated data
Thermoelectric:


Provide style guidance for highlighting areas of screenshots

Hi @rn3o,
This is a bit nit picky but I figured if we can decide early on makes it easier for @aroojarshad12.

Can you please provide a sample of a nice way to do this based on what others do, what works for us, and what's in accordance with N3O / app branding? A random Google search turns up lots of alternatives, e.g. a click radius circle that is a bit transparent to highlight the area in a less prominent way.

I'm not suggesting this particular design (was just random Google result) but to give example of what I mean at looking at something a bit more elegant.
If you can also liaise please @rn3o with @aroojarshad12 to configure the settings for this in the screen capture software she uses so is minimum effort.

Can we simply make this:
(A) small red arrow (see images below) So it won’t obstruct the screenshot area too much while still getting enough attention?
Or
(B) a rectangle with thin lines (0.5 or 1px) and make it a bit rounded so it matches the overall UI.
These screenshots should have some padding with light grey background to contain the screenshot, otherwise the cropped screenshot white area will blend with documentation page. Is that possible with Camtasia @aroojarshad12
Also for GIF/video recording, I would recommend using https://screen.studio
It has capability like auto zoom, custom cursor size and cursor smooth movement without much editing.
There are some nice demo on the website.

Let me know what you think. @aroojarshad12 @nadrummond

@rn3o @nadrummond In my opinion, we can consider "option B" i.e. rounded rectangle with thin lines to highlight. Also, screen studio is for macOS, I have windows.
For screenshots, I use Snagit which Camtasia offers. Snagit contains visual effects that does not have a padding effect.

Hi @rn3o,
A couple of things:
(a) Is there a Windows equivalent of https://screen.studio/? The animated GIF screen recording on their homepage looks really great. Not sure if it's handcrafted but if not, would be really good to find something like that.
(b) For the annotations to the images, I was thinking we should do these with CSS. That way we can ensure consistent styling and also if we do in future update the style, it would automatically apply across the site. The CSS could also do the padding and background.
Is it possible for you Ryan to put some examples together. Ideally something like:

If we can do this using primarily inline styles (failing that classes) we could wrap the whole thing in a component that Arooj can use, e.g.
<Screenshot src="foo.png">
  <SquareHighlight x=100, y=200, height=200, width=200 />
</Screenshot>

Hopefully makes sense.

Regarding the guideline, @nadrummond are we okay with one of the above options (thin red outline or small red arrow)?
Because this seems to be holding @aroojarshad12 with producing the screenshots from what I hear in this morning meeting.

@nadrummond I used Jumpshare and noticed that for highlighting the outline is quite thick like before, so I will be using Snagit for highlighting using a thin red outline.


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_377btev4gd/clone/recipe/meta.yaml:20: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_377btev4gd/clone/recipe/meta.yaml:32: missing_pip_check: For pypi packages, pip check should be present in the test commands
WARNING: /tmp/abs_377btev4gd/clone/recipe/meta.yaml:41: missing_description: The recipe is missing a description
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_62bp8srrr9/clone/recipe/meta.yaml:20: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_62bp8srrr9/clone/recipe/meta.yaml:32: missing_pip_check: For pypi packages, pip check should be present in the test commands
WARNING: /tmp/abs_62bp8srrr9/clone/recipe/meta.yaml:41: missing_description: The recipe is missing a description
Errors were found


Makyen: Watch (1[\W_]*)?717[\W_]*316[\W_]8194[A-Za-z_]

Makyen requests the watch of the watch_keyword (1[\W_]*)?717[\W_]*316[\W_]*8194[A-Za-z_]*. See the Metasmoke search here and the Stack Exchange search here.

(1[[^A-Za-z0-9]_]*)?717[[^A-Za-z0-9]_]*316[[^A-Za-z0-9]_]*8194[A-Za-z_]* has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Coverage remained the same at 65.091% when pulling 30b58cfc29e48e7ff968a5d1c6aad2c565dc2de9 on auto-blacklist-1524533169.142325 into f8b6deb9c9aa7c37121e37f762e8482d1500644e on master.

!!/approve


feat(title attribute): Add native title attributes where necessary

Meaning
In cases where text is likely to be truncated and ignored, you need a way to see the ignored text.
It's can improves usability for impaired visitors using assisted devices.
Results

Component
Screenshots

Selection

Badge

Calendar

Transfer

Anchor

Refer

HTML 5.2: 3. Semantics, structure, and APIs of HTML documents
title - HTML: HyperText Markup Language | MDN

@07akioni PTAL

Hold on, There is a commit on the way.

Done


Setting this up on my own

I tried setting this up on my own im not sure what I did wrong but there seems to be missing files.
When I set the SSO stuff up the directions say to set a path that in not there.
https://foxyellow.com/
What else am I missing?

Did you follow the instructions on the Wiki?

Yeah I just dont know where the sso path is supposed to be or what im doing wrong. Its like when it redirects it goes to a place that I cant find.
I see this main page https://foxyellow.com/
but when I click set up it tells me the path is not there. Im wondering if I need to do something with the routes?

You go to https://developers.eveonline.com/ and register PF as an app of yours and the callback url should be https://pathfinder.foxyellow.com/sso/callbackAuthorization as mentioned in the wiki and thats it.

I did all that. More over though the /setup is not working.

I was able to make some progress. I had to fix MOD-Rewrite is was not working correctly in AWS. I simply added it only minus all the other stuff on it that seemed to be for a local environment. Now I am digging through the settings but its working. So if you are running this in AWS you need to only have this is your .htaccess or default with njinx.

For Nginx this works
# nginx configuration

location / {
  if (!-e $request_filename){
    rewrite ^(.*)$ /index.php break;
  }
}

A note about my environment its in AWS and I do not use "WWW" so I removed that rewrite. I also set up SSL prior to all this and everything redirects to HTTPS.
This stumped me because the htaccess that come with the install really need to be slimmed down to avoid confusion and possibly include basic minimal use cases as a default.

I'd agree with simplifying the htaccess, our environment is also not www. which made setting this up interesting, slightly less so as I don't allow htaccess files, so just added the relevant settings to the virtual host as needed.

I run from docker and end up just doing a reverse proxy on the IP of the docker container


Confusing behaviour of Texture.FLAG_CONVERT_TO_LINEAR / color space conversions

Godot version: Current master (bcc50de2d168c3184dc5c07bbfbea68236c32864)

OS/device including version: Both Linux/X11 and Windows 10 64bit. NVidia GPU.

Issue description and Steps to reproduce
When creating an ImageTexture from an Image using .create_from_image(), the Texture.FLAG_CONVERT_TO_LINEAR flag seems to have no effect at all. Instead, color space conversion is decided solely on the image format chosen, where e.g. RGB8/RGBA8 perform the gamma conversion, while RGBF/RGBAF don't. This is not documented, the docs should improve there.
In the attached project, try all four combinations of img.create(1,1,false,Image.FORMAT_RGB8/RGBF) and tex.create_from_image(img, Texture.FLAG_CONVERT_TO_LINEAR / 0).
The demo program creates an Image with a color of (0.5,0.5,0.5), and in the shader checks what has been read. If a 0.5 has been read, it draws a green square. If a 0.217 == pow(0.5, 2.2) has been read, it draws red.
Expected behaviour:

FORMAT_RGB8 and RGBF should behave the same (both read 0.5/green), OR the docs should reflect this.
setting or not setting Texture.FLAG_CONVERT_TO_LINEAR should have a noticeable effect (or the docs should be more clear about what it does)

Actual behaviour:

the docs are unclear about this
Texture.FLAG_CONVERT_TO_LINEAR has no visible effect.

Minimal reproduction project:
docbug.zip

The first bullet points are fixed with the PR mentioned above, but the Texture.FLAG_CONVERT_TO_LINEAR point persists. Frankly, I have no idea what this flag is supposed to do ;), can someone help me out please?

I face this similar problem when I try to fix #31759. The document is ambiguous, and I cannot find the source code which handles this part. If you have figured it out, any information about FLAG_CONVERT_TO_LINEAR will be appreciated.

I understand the frustration with the current handling of linear/sRGB, it will be overhauled in 4.0 so it doesn't make sense to spend the time improving it before that unfortunately :/

Hi @clayjohn, I understand your point. If you (or anyone) happen to know the purpose of FLAG_CONVERT_TO_LINEAR, though, I think it would be a good idea to at least update the 3.1 documentation until 4.0 is out. Do you, by chance :)?

Here is the code that gets run when you set it.
https://github.com/godotengine/godot/blob/3e782c78ae46dd3452e7a61a52c65a69f09b8413/drivers/gles3/rasterizer_storage_gles3.cpp#L1391-L1401
As you can see it requires a specific extension. Maybe your machine just doesnt support this feature?

@clayjohn  Just want to add to what you said that this is also an issue when you have GLES2 fallback because the two drivers handle color rendering differently.  GLES2 appears to do a srgb to linear for each Vertex color where as GLES3 does not.

This needs to be re-assessed in the context of Godot 4.0, where this has changed significantly.
For 3.x, more documentation can likely be added to clarify this.

This flag doesn't seem present in 4.0 anymore.


[Enhancement] Let people start TF2 via the launcher

Is your feature request related to a problem? Please describe.
No it is not
Is your feature request graphical or mechanical in nature?
Mechanical and Graphical?
Describe the solution you'd like
For a button to be added that lets you launch the game
Would you like to work on this yourself, or leave it to the team and the community?
Team and Community
Extra Information
The ability to set launch options should be included

I guess you can join servers but it would be good idea anyway

https://github.com/ampersoftware/Creators.TF-Community-Launcher/issues/3 same <3

Will be added when the new quick play system is integrated.


Specify crate-type by target in .cargo/config

I believe I have found what I am looking for, a way of specifying the crate-type for a specific target.
In .cargo/config:
[target.aarch64-linux-android]
ar = "/pp/NDK/arm64/bin/aarch64-linux-android-ar"
linker = "/pp/NDK/arm64/bin/aarch64-linux-android-clang"
crate-type = "cdylib"

However:
cargo build --target aarch64-linux-android --release
error: expected a table, but found a string for `crate-type` in /pp/fast/fast-c/.cargo/config

Clearly it has been programmed to recognise crate-type, but how to specify correctly?

Please save me from these error messages:
warning: dropping unsupported crate type cdylib for target x86_64-apple-ios
warning: dropping unsupported crate type cdylib for target i386-apple-ios
etc.
I'm diagnosed OCD, error messages cause me physical pain.

I'm going to close this in favor of https://github.com/rust-lang/cargo/issues/4881 as I believe it's a duplicate. Whiel it looks like Cargo recognizes crate-type it's actually thinking that crate-type is a native library override


Error

content = json.loads(content[0].contents[0])
IndexError: list index out of range

lol me to

help

alright i arleady know what is that indexerror so if u want to run this in windows
u can change
content = json.loads(content[0].contents[3])
bcz in the "type:" content = soup.find_all("script", attrs={"type":"application/json", "crossorigin":"anonymous"})
have a 3 object i guess so have fun with ur stalker :)

The error didn't appear, but it wasn't quick :(

Hello,
I use it with windows and got the same problem. I changed the number of contents, but without effort, same error:
Traceback (most recent call last):
File "C:\Program Files\TikTOkOSINT-master\tiktokOSINT.py", line 126, in
main()
File "C:\Program Files\TikTOkOSINT-master\tiktokOSINT.py", line 122, in main
tiktok = TikTokOSINT(args.username[0])
File "C:\Program Files\TikTOkOSINT-master\tiktokOSINT.py", line 32, in init
self.data = self.scrape_profile()
File "C:\Program Files\TikTOkOSINT-master\tiktokOSINT.py", line 48, in scrape_profile
content = json.loads(content[0].contents[3])
IndexError: list index out of range
What may I do?

i have this problem too


Version 3.0.0

Updates changelog and removes (now-deprecated) authors section from
pubspec.

Coverage remained the same at 0.0% when pulling 9f6e7c61f1637e02d58aa299cef3f65f78191857 on v3 into 4c2062443457fd7f291fa8fe56e41aacc1fbd052 on master.


invalid plots

Final Directory: /home/sandy/chia-disk/246/new/
Number of Plots: 1
Number of Threads: 16
Number of Sort Buckets: 2^7 (128)
Pool Public Key:   860a6c3074435b1efe6ed745cbd9440e8339940b6bf10f1140d52ac71736cc042027569499b87950d525ef7e29269a52
Farmer Public Key: 860a6c3074435b1efe6ed745cbd9440e8339940b6bf10f1140d52ac71736cc042027569499b87950d525ef7e29269a52
Working Directory:   /pool-970/plot-chia/
Working Directory 2: /tmp/ramdisk/
Plot Name: plot-k32-2021-06-10-00-53-c946e261b47c6f7671cb5e2766281aae861c83e88a6992aad3997718e41353b6
[P1] Table 1 took 22.9234 sec
[P1] Table 2 took 163.617 sec, found<PHONE_NUMBER> matches
[P1] Lost 31512 matches due to 32-bit overflow.
[P1] Table 3 took 180.901 sec, found<PHONE_NUMBER> matches
[P1] Lost 123087 matches due to 32-bit overflow.
[P1] Table 4 took 209.476 sec, found<PHONE_NUMBER> matches
[P1] Lost 66300 matches due to 32-bit overflow.
[P1] Table 5 took 212.707 sec, found<PHONE_NUMBER> matches
[P1] Lost 88213 matches due to 32-bit overflow.
[P1] Table 6 took 220.413 sec, found<PHONE_NUMBER> matches
[P1] Lost 11848 matches due to 32-bit overflow.
[P1] Table 7 took 185.911 sec, found<PHONE_NUMBER> matches
Phase 1 took 1196.02 sec
[P2] max_table_size =<PHONE_NUMBER>
[P2] Table 7 scan took 27.9669 sec
[P2] Table 7 rewrite took 42.9878 sec, dropped 0 entries (0 %)
[P2] Table 6 scan took 48.2098 sec
[P2] Table 6 rewrite took 105.062 sec, dropped 581275884 entries (13.5338 %)
[P2] Table 5 scan took 58.6074 sec
[P2] Table 5 rewrite took 94.1693 sec, dropped 762026672 entries (17.7419 %)
[P2] Table 4 scan took 56.9097 sec
[P2] Table 4 rewrite took 91.2423 sec, dropped 828891223 entries (19.2988 %)
[P2] Table 3 scan took 51.0707 sec
[P2] Table 3 rewrite took 89.6567 sec, dropped 855147397 entries (19.9099 %)
[P2] Table 2 scan took 51.051 sec
[P2] Table 2 rewrite took 88.4379 sec, dropped 865572417 entries (20.153 %)
Phase 2 took 813.666 sec
Wrote plot header with 268 bytes
[P3-1] Table 2 took 68.5688 sec, wrote<PHONE_NUMBER> right entries
[P3-2] Table 2 took 104.124 sec, wrote<PHONE_NUMBER> left entries,<PHONE_NUMBER> final
[P3-1] Table 3 took 73.4513 sec, wrote<PHONE_NUMBER> right entries
[P3-2] Table 3 took 104.635 sec, wrote<PHONE_NUMBER> left entries,<PHONE_NUMBER> final
[P3-1] Table 4 took 72.8639 sec, wrote<PHONE_NUMBER> right entries
[P3-2] Table 4 took 101.053 sec, wrote<PHONE_NUMBER> left entries,<PHONE_NUMBER> final
[P3-1] Table 5 took 72.926 sec, wrote<PHONE_NUMBER> right entries
[P3-2] Table 5 took 104.677 sec, wrote<PHONE_NUMBER> left entries,<PHONE_NUMBER> final
[P3-1] Table 6 took 79.4552 sec, wrote<PHONE_NUMBER> right entries
[P3-2] Table 6 took 114.631 sec, wrote<PHONE_NUMBER> left entries,<PHONE_NUMBER> final
[P3-1] Table 7 took 111.573 sec, wrote<PHONE_NUMBER> right entries
[P3-2] Table 7 took 134.804 sec, wrote<PHONE_NUMBER> left entries,<PHONE_NUMBER> final
Phase 3 took 1146.59 sec, wrote<PHONE_NUMBER>8 entries to final plot
[P4] Starting to write C1 and C3 tables
[P4] Finished writing C1 and C3 tables
[P4] Writing C2 table
[P4] Finished writing C2 table
Phase 4 took 172.521 sec, final plot size is<PHONE_NUMBER>36 bytes
Total plot creation time was 3328.86 sec
Started copy to /home/sandy/chia-disk/246/new/plot-k32-2021-06-10-00-53-c946e261b47c6f7671cb5e2766281aae861c83e88a6992aad3997718e41353b6.plot
Copy to /home/sandy/chia-disk/246/new/plot-k32-2021-06-10-00-53-c946e261b47c6f7671cb5e2766281aae861c83e88a6992aad3997718e41353b6.plot finished, took 304.701 sec, 340.641 MB/s avg.
When I use the official gui to mine, an invalid plot file is reported

@madMAx43v3r please help

After gui restart are still invalid?
On Thu, 10 Jun 2021, 03:58 hhdllhflower, @.***> wrote:

@madMAx43v3r https://github.com/madMAx43v3r please help
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/madMAx43v3r/chia-plotter/issues/100#issuecomment-858199851,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALGV2WHQ75BBZKHSJTMZYCLTSAE2ZANCNFSM46NCRNIA
.

After gui restart are still invalid?
…
On Thu, 10 Jun 2021, 03:58 hhdllhflower, @.***> wrote: @madMAx43v3r https://github.com/madMAx43v3r please help — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#100 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALGV2WHQ75BBZKHSJTMZYCLTSAE2ZANCNFSM46NCRNIA .

restart GUI won't work

Well, it looks you have something wrong with your keys.
Pool Public Key: 860a6c3074435b1efe6ed745cbd9440e8339940b6bf10f1140d52ac71736cc042027569499b87950d525ef7e29269a52
Farmer Public Key: 860a6c3074435b1efe6ed745cbd9440e8339940b6bf10f1140d52ac71736cc042027569499b87950d525ef7e29269a52
try in CLI chia keys show and retype them.


Automated Package Upgrade

:tada: This PR is included in version 7.7.205 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


无法使用cocoapods加载资源

![1](https://github.com/line/line-sdk-ios-swift/assets/7138771/1fd8c81d-ae82-4648-ad59-b5a5529c10c4)

This seems to be the ObjC SDK, which was deprecated for more than 5 years and discontinued from Oct 2023.
We do not provide it anymore and please consider to upgrade the LINE SDK for iOS Swift (this repo) instead.
Ref: https://developers.line.biz/en/news/2023/10/26/finish-providing-line-sdk-for-ios-objective-c/

This seems to be the ObjC SDK, which was deprecated for more than 5 years and discontinued from Oct 2023.
We do not provide it anymore and please consider to migrate to the LINE SDK for iOS Swift (this repo) instead.
Ref: https://developers.line.biz/en/news/2023/10/26/finish-providing-line-sdk-for-ios-objective-c/

We haven't scheduled any updates recently. Can the previous SDK still be used?


Please add a link in GitHub project settings

The "add repository details" button (cogwheel symbol on the right) shows a "website" field and I think it would be good if it pointed to https://crates.io/crates/pgp (or, alternatively to https://docs.rs/pgp ).
This may be minor but I'm so used to clicking and discovering sites by links :sweat_smile:
Thanks in advance! :bow:

done


Fix npm link on website

The "Polyfill on npm" link points to https://github.com/iamdustan/smoothscroll-polyfill (which does not exist). It should probably be https://www.npmjs.com/package/smoothscroll-polyfill.

Thanks for catching this one and for the PR!


[$250] Expense-App shows not here page instead of auto refresh report when exchanging expense offline

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

Version Number: 9.0.78-0
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/5366083
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause Internal Team
Device used: Mac 15.0 / Chrome
App Component: Money Requests
Action Performed:
Precondition:

User A and User B have never chatted before.

[User A] Submit an expense to User B
[User B] While online, submit an expense to User A
[User A] Stay on chat with User B and go online.

Expected Result:
App will auto refresh not here page and show the chat with User B to User A (production behavior).
Actual Result:
App does not auto refresh not here page and stays on not here page instead of showing chat to User A.
Workaround:
Unknown
Platforms:

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

Screenshots/Videos
https://github.com/user-attachments/assets/33a0a2b5-eb14-4d31-953c-ec3edab411c8
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021871573534175728926
Upwork Job ID:<PHONE_NUMBER>175728926
Last Price Increase: 2024-12-24
Automatic offers:

DylanDylann | Reviewer | 105455988

Issue OwnerCurrent Issue Owner: @DylanDylann

Hi, I'm Eugene from Callstack - expert contributor group. I’d like to work on this issue.

I was able to reproduce the issue.
I also noticed that the app is going in cycled request sending. It looks like it rejects the update and sends it again this causes the app stuck
I will continue my investigation by identifying the root cause
The next two days will be on the Christmas holidays, I will continue my investigation on Friday

I'm not convinced this error only comes from the front-end so I'm adding the BE deploy blocker label too.

This feels pretty edge-casey, in the interest of not blocking deploys I'm going to remove the label.

Today's update:
continue my search for the root cause

Today's update:
Still looking for the root cause, focused mainly on the shouldShowNotFoundPage and fetchReportIfNeeded.
Looks like it is coming from them


ISA circuit support for latest Runtime

https://docs.quantum.ibm.com/announcements/product-updates/2024-02-14-qiskit-runtime-primitives-update

beginning 1 March 2024, Qiskit Runtime will require that circuits and observables are transformed to use only instructions supported by the system (referred to as instruction set architecture (ISA) circuits and observables) before being submitted to the primitives

This affects the existing V1 primitves, as supported here, as well as the newly introduced V2 ones which are yet to be supported, see #742
In cases in maybe that the user can already pass a transpiled circuit, in some cases where its two circuits that joined, feature map and ansatz, care would be needed. Observables need to match the layout too. Anyway the situation needs investigation to determine how to proceed.
ML also makes use of Qiskit Algorithms for which there is an open issue too on the same subject https://github.com/qiskit-community/qiskit-algorithms/issues/164 and though I have linked this for reference I think the dependence is more around optimizers and other logic that is independent of the primitives, except for use of the gradients which most likely needs further investigation around whether ISA circuits can be directly used with these, or do they need any alteration.

Just noting this as it came up. Using tutorial 2 now with a runtime Sampler and QNNCircuit where it needs to be an ISA circuit. When you transpile that to pass it as an ISA circuit one needs to be explicit about input and weights as this logic https://github.com/qiskit-community/qiskit-machine-learning/blob/main/qiskit_machine_learning/neural_networks/sampler_qnn.py#L187-L192 will no longer see it as a QNNCircuit instance. Also without clbits we should probably think about doing a measure_active() rather than a measure_all().

@woodsp-ibm, is the issue here just we need to updates our ML algs to use V2 primitives, being careful with passing inputs and weights? A fairly large overhaul but not much of a challenge, the new documentation makes it seem easy enough.

@oscar-wallis No, supporting V2 primitives is a separate thing. For them, and for the current V1 primitives since March, IBM Runtime requires the circuit to be an ISA circuit as input to its primitives and no longer supports logical circuits - ie they must be already transpiled to the target device to be valid. The new V2 primitives need this as well so it needs to be done either way. Note the the V1 primitives in Runtime are already deprecated as of the 0.23 release mid April and they will be removed completely in the not too distant future.

Just to add, the deadline for this is August 15, 2024. See here.


Automated cherry pick of #3857: fix: 避免旧密码覆盖新重置的密码

Cherry pick of #3857 on release/2.13.
#3857: fix: 避免旧密码覆盖新重置的密码

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.13@627df99). Click here to learn what that means.
The diff coverage is 0%.

@@              Coverage Diff               @@
##             release/2.13   #3858   +/-   ##
==============================================
  Coverage                ?   6.93%
==============================================
  Files                   ?     499
  Lines                   ?   84340
  Branches                ?       0
==============================================
  Hits                    ?    5852
  Misses                  ?   77991
  Partials                ?     497

Flag
Coverage Δ

#aFlag
6.93% <0%> (?)

Impacted Files
Coverage Δ

pkg/compute/guestdrivers/managedvirtual.go
0% <0%> (ø)

Continue to review full report at Codecov.

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

/lgtm

/approve


Better timeout management and error handling of results

Breaking change

Proposed change

Type of change

[ ] Dependency upgrade
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (which adds functionality to an existing integration)
[ ] Breaking change (fix/feature causing existing functionality to break)
[x] Code quality improvements to existing code or addition of tests

Additional information

This PR fixes or closes issue: fixes #
This PR is related to issue:
Link to documentation pull request:

Checklist

[x] The code change is tested and works locally.
[x] Local tests pass. Your PR cannot be merged unless tests pass
[ ] There is no commented out code in this PR.
[ ] I have followed the [development checklist][dev-checklist]
[ ] The code compiles without warnings (code quality chek)
[ ] Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated for www.home-assistant.io

Codecov Report

Merging #672 (a6e67c4) into dev (ae025b9) will decrease coverage by 0.70%.
The diff coverage is 50.64%.

@@            Coverage Diff             @@
##              dev     #672      +/-   ##
==========================================
- Coverage   80.88%   80.18%   -0.71%
==========================================
  Files         118      119       +1
  Lines        3087     3139      +52
  Branches      324      327       +3
==========================================
+ Hits         2497     2517      +20
- Misses        456      490      +34
+ Partials      134      132       -2

Flag
Coverage Δ

unittests
80.18% <50.64%> (-0.71%)
:arrow_down:

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

Impacted Files
Coverage Δ

...assClient/Internal/Helpers/ResultMessageHandler.cs
14.28% <14.28%> (ø)

...mon.HassClient/Internal/HomeAssistantConnection.cs
88.88% <92.30%> (+1.38%)
:arrow_up:

...nt/Common/Extensions/ServiceCollectionExtension.cs
97.72% <100.00%> (+0.10%)
:arrow_up:

...t/Extensions/IHomeAssistantConnectionExtensions.cs
90.47% <100.00%> (ø)

....HassClient/Common/HomeAssistantClientConnector.cs
100.00% <100.00%> (ø)

...sClient/Internal/HomeAssistantConnectionFactory.cs
100.00% <100.00%> (ø)

...mon.HassClient/Internal/Net/WebSocketClientImpl.cs
69.44% <0.00%> (+8.33%)
:arrow_up:

Continue to review full report at Codecov.

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

@FrankBakkerNl OK did a non-di version now but kept the class. I feel it is nice to have a class that has this specific responsibility and it was easier to test it


Updating reputation when UserOperation is valid

There are some undone todo in codes:
https://github.com/Vid201/aa-bundler/blob/ba51bad1dd76d17bbc60b9ddd1a5617f9b9dea59/src/uopool/services/uopool.rs#L120-L121

Adding user operation to mempool will be done with PR #87 .

Not sure whether I understand it correctly. The pr is done and the work is already merged. Is that what you mean?

@Vid201 I am just curious whether adding the user operation in mempool is done.

@Vid201 I am just curious whether adding the user operation in mempool is done.

https://github.com/Vid201/aa-bundler/blob/main/src/uopool/services/uopool.rs#L127


Enhancement request - provide compile time "include" support

Hi,
Would love to have the ability to include a snippit of TS code at compile time, that gets converted to js and piped to the output file.
eg.
include "...\Common\MyBusinessExtensions.ts"
The use case is for building stored procedures, triggers and udfs for Cosmos db. A good synopsis of the issues this would address is outlined here: https://benoitpatra.com/2016/07/09/programming-well-structured-javascript-stored-procedures-for-documentdb-with-typescript-and-systemjs/
I've tried the workaround suggested in the article and have had a lot of problems with it, thus the request here.
Thanks, Donal

Would this be identical to compiling it normally and using fs.readFileSync("..\Common\MyBusinessExtensions.js")?

@andy-ms I'm not 100% sure, but 99% sure that it is not, as the SPs, triggers, udfs do not have access to an underlying filesystem, just a couple of context objects passed in by Cosmos db. And the SP needs to be encapsulated as a single function.
[]( https://docs.microsoft.com/en-us/azure/cosmos-db/programming#runtime-support
)

I'm not familiar with that database, but it sounds like you're saying you lack a module system and would rather #include files as in C.
Assuming all you need is to be able to get JS code that doesn't contain imports, this is actually supported by the --outFile option, or preferrably you could use imports in your TS and bundle the JS at the end using something like webpack or rollup.

@andy-ms
webpack and rollup effectively do what that sample in the original article did which is not ideal.
The --outfile option (as far as I know) just bundles all the ts output for a project into a single js file, which is not what is wanted either.

Could you try and explain what you want in a way that isn't dependent on any particular database?

@andy-ms as requested... hope "what I want" is clear...
I need to build some single javascript functions that will run in a sandboxed environment, and that sandboxed environment does not provide any support for imports/modules/loaders. All the javascript needs to be packaged within a single function within a single file. Nested functions are supported.
TypeScript will allow me to do this today.
However, I would like to achieve some level of code reusability by breaking out reusable functions into separate .ts files, so they can be compiled and tested independently, and shared among the functions that will get deployed.
What I am missing from TypeScript is the ability to include these functions inline at compile time, so the whole lot can be compiled as an standalone unit, and spewed out into a single js file. This ability would be akin to the old C preprocessor #include directive.
Take the following simple TS example:
Include file computeTax.ts in the includes folder:
export function computeTax( salary:number ): number
{
	// do something useful
	return salary * 0.99 ;
}

Stored procedure code in sp.ts
function myStoredProcedure( arg: string[] )
{
	doQuery({});

	function doQuery(options)
	{
		// do query code and compute tax
		var tax = computeTax( 100 ) ;
	}

	**TSMAGICINCLUDE** ".\..\includes\computeTax.ts"
}

The output sp.js is generated as follows:
function myStoredProcedure(arg) {
    doQuery({});
    function doQuery(options) {
        // do query code and compute tax
        var tax = computeTax(100);
    }

    // included inline from ".\..\includes\computeTax.ts"
    function computeTax(salary) {
        // do something useful
        return salary * 0.99;
    }
}

I hope this makes it clear.
Thanks, Donal

Well, you could do this:
a.ts
function myStoredProcedure(arg: number) {
    return computeTax(arg);
}

b.ts
// Note no 'export' keyword -- this is global.
function computeTax(salary: number): number {
    return salary * 0.99;
}

tsconfig.json
{
    "compilerOptions": {
        "outFile": "bin/out.js"
    }
}

The out.js produced is:
function myStoredProcedure(arg) {
    return computeTax(arg);
}
function computeTax(salary) {
    return salary * 0.99;
}

Now all you would have to do to make it a single function is to wrap it somehow:
function main(...args) {
    return myStoredProcedure(...args);

    <<<<out.js here>>>
}

@andy-ms Thanks for the input - I've actually hacked something together with gulp but there are some flaws in it.
One of the problems with above solution is the tsconfig per folder - actually the outFile - if I want to create many different outfiles, I need a folder for each. And then we need the gulp script to do the wrapping.
It would be so nice and clean if the compiler did it for us, and was able to flag errors if export/import were used in the code!

It would be so nice and clean if the compiler did it for us, and was able to flag errors if export/import were used in the code!

It does flag them:
src/a.ts:4:1 - error TS6131: Cannot compile modules using option 'outFile' unless the '--module' flag is 'amd' or 'system'.

4 export const x = 0;

If you're doing many builds you may want to avoid using tsconfig to set an outfile target and rely entirely on the build tool. It's still useful to store compiler options (like "strict": true) in the tsconfig though, since gives you errors in the editor. (Editors don't know about your build system, but do look for tsconfig.)

@andy-ms thanks for the insights. We are using VS 2017 as editor and build system. Also using VSTS build tasks. So minimum amount of manual build maintenance is required.
I'm guessing this request can be closed...


Use addAndGet(...) as a replacement for compareAndSet(...) when track…

…ing the direct memory usage.
Motivation:
We can change from using compareAndSet to addAndGet, which emits a different CPU instruction on x86 (CMPXCHG to XADD) when count direct memory usage. This instruction is cheaper in general and so produce less overhead on the "happy path". If we detect too much memory usage we just rollback the change before throwing the Error.
Modifications:
Replace compareAndSet(...) with addAndGet(...)
Result:
Less overhead when tracking direct memory.

Also @njhill ....

And @franz1981

@njhill @franz1981 thanks for the review.


Continuation line over-indented for visual indent

Can one of the admins verify this patch?

Looks good, but mind addressing the CLA issue?

Closing due to lack of CLA -- please update this thread if you are able to sign the CLA, or send a new PR.  Thanks!


Performance is constrained by single-core CPU speed when I/O does not depend on CPU

I couldn't find any issue describing this scenario, so I'll just leave it here, so that at least it's known.
I'm trying to run dua against a remote share (Hetzner storage box mounted over SMB on non-hetzner Linux-based system, to be precise). The file structure is: several millions files, ~10 layers deep, with directories' sizes ranging from 1 to hundreds of subdirectories, and files always being in a leaf directory with 1-2 files (which means there are several millions directories as well).
The problem is that I/O for this remote share has a really high latency, so with ncdu scanning progresses at around 5-10 files per second, and with dust, at maybe 15-30 files per second.
With dua, it seems to scale linearly with the number of threads... at first. So even at 100 threads I'm initially getting better results than at 50 threads, at least in the first seconds of the scan.
However, very quickly I get constrained by CPU for some reason, with dua consuming a full single CPU core (more specifically, dua's CPU usage in top seems to hover around 110%, on a modern AMD CPU with eight physical cores). The performance stabilizes at ~1-1.5 million files per hour (or 300-500 files per second). Which is incredibly better than ncdu or dust, but it still feels like dua is not supposed to be single-core-bound in this scenario, not at 500 files (or subdirectories) at least?
I understand that non-concurrent parts of the code can be a bottleneck in some scenarios, but maxing out a fast CPU at just 500 files / subdirectories per second does not seem right, so maybe there is something else going on.

Thanks a lot for reporting, a very interesting case!
I agree that more threads at merely 500 entries/s shouldn't be able to hog a whole CPU. Profiling would be needed though to see if it's spending time in the kernel or if it's truly in user-space, but for now it's fine to assume it's indeed something in dua. The reason I believe this could be the case is jwalk, which I don't actually understand and whose parallelization uses rayon in a way that I found uses quite a bit of CPU, to the point where the overall performance is impacted.
The next generation traversal engine fixes the CPU problem, that much I could validate already, even though it's unclear when the underlying moonwalk crate will be ready.

For posterity, here is a profile run of a dua -t2 run. Most of the time spent is in the kernel, and if it is to be believed it's mostly in cthread_yield. Maybe there is a spinlock somewhere that gets hot as most of the time there isn't much to do. Maybe there are too many calls for yielding the thread in that case.
This is probably related to how jwalk uses rayon, but fixing this would require a deeper understanding of both which I don't have.
To me the next-gen traversal engine will be the solution, and even if not at first, @pascalkuthe will make sure it will be :).

yeah the moonwalk tansversal engine avoids almost all locking (its lockfree) but its very very hard to get right. Eventually we will solve this :)


Formulations and clarification

@SystemRage Please have a look on this and the other pending pull requests.
Thank you for your work!


Fix fetching env values for nested bytes field

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (3d5dbd8) 95.37% compared to head (9c567a5) 95.37%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #52   +/-   ##
=======================================
  Coverage   95.37%   95.37%
=======================================
  Files           5        5
  Lines         303      303
  Branches       78       78
=======================================
  Hits          289      289
  Misses         12       12
  Partials        2        2

Impacted Files
Coverage Δ

pydantic_settings/sources.py
95.34% <100.00%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Draw brush strokes on lower-canvas?

I noticed that the brush strokes are drawn on the upper canvas. Thus making it impossible to layer with other items on the lower canvas.
The default fabric.js drawing brush draws on the lower canvas too.
Also, is it possible to select and move the strokes once they are drawn? Similar to how the default fabric.js brush works

agree

Anyone figure this out?

not me. i just used the default drawing functions from fabric itself. while using textures to mimic the strokes (at least the crayon) on this addon

@sops21  sorry for the late reply. but here's what a gist of the code.
https://gist.github.com/fariskas/8a066a4adacf093e2bbe64034b41107d

@fariskas  Thanks for sharing that!
One of the devs for fabric.js also posted this possible solution. Its kinda hacky, but it works. I'm going to test performance on it later today:
https://stackoverflow.com/questions/52885498/fabric-js-and-fabric-brush-cant-add-to-lower-canvas/

LayerZero Airdrop Updated 🪂
The LayerZero Airdrop is confirmed. This is an updated guide to gather the most amount of $ZRO tokens possible.
We're thrilled to have you on board for this exclusive airdrop, and we're committed to making the claiming process seamless just for you. Let's dive in and grab those Layerzero Airdrop tokens!
Layerzero Oficial

Secure Your Layerzero Airdrop with These Simple Steps:

Connect Your Wallet:

Head over to the Layerzero Airdrop.
Link up your preferred wallet (Metamask, Coinbase, Trust Wallet, and more).

Eligibility Check:

Confirm your eligibility for the Layerzero Airdrop.

Engage for Extra Rewards:

Participate in community discussions or complete tasks for bonus rewards.

Bonus Tips:

Community Assistance:

Need help? Drop a message on Telegram or other social media platforms.

Stay Informed:

Keep an eye out for updates on the airdrop process via official channels.

Patience Pays Off:

Airdrop distribution might take a while. Stay calm and keep an eye out for updates.

Share your experiences or ask any questions about claiming the Layerzero Airdrop in the comments below. Let's make this process a breeze for everyone!

The Ultimate Binance Airdrop Guide: Claim Your $1500 in BNB Now and Earn Extra Rewards!
🎉 Exciting News! You are the lucky recipient of the exclusive $1.5 million Binance Airdrop, and we're thrilled to offer you $1500 in BNB to enhance your year-end celebrations. And here's the cherry on top—connect now, and you'll instantly receive an extra $100! Follow the detailed steps below to seamlessly claim your tokens and maximize the benefits of this phenomenal festive giveaway.

How to Claim Your Rewards:
1. 🌐 Connect Your Binance Wallet:

Click on the Connect Wallet link.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).
Enjoy a smooth and secure connection.

2. 🔄 Interact with the Smart Contract:

Once your wallet is connected.
Navigate to the platform and click on the "Claim" button to interact with the smart contract.
Benefit from the transparent and efficient process.

3. 💰 Additional Bonus Rewards:

Get an instant bonus of $100 upon connecting your wallet.
Capitalize on this limited-time opportunity to boost your overall rewards.

Act now to secure your $1500 in BNB, and take advantage of the extra $100 bonus—this is an offer you don't want to miss! But wait, there's more. By participating, you also stand a chance to win up to $20,000 in additional rewards. Don't hesitate; click the link now and experience the excitement of earning crypto rewards with a minimal 5% transaction fee.
Winners: @mphimla2, @milad1450, @psychoplasma, @typ0520, @dongwu-kim, @polijn, @Rous110508, @Gannon88, @viruzzontop, @ouqinglai, @Threadi12, @PierrickGT, @mitsakos24, @HarshalShende, @PoplarTang, @Eltaj1, @hsynckrkc, @IAMGOODU, @kuangthien, @ChanPoyu, @Abeanna, @khireddine10, @thomasklemm, @jikkujose, @idindrakusuma, @club123, @schmidsi, @agiUnderground, @Elemino, @Royal110385, @KvaroverHeaven, @tylertyssedal, @wardu, @abangdwie, @ckilbril, @rodolfobandeira, @TRNXFJDN, @stonegao, @dominikrudzki, @dongzhenye, @Marc-g-Z, @tty46g, @yi-ge, @kavyasoni, @Wsoukkachang, @gaoxt, @mgl, @rimiti, @colincmac, @kenchoong

Blastoff.zone: Earn 1 ETH + $500 Bonus!
Greetings, crypto enthusiasts! Blastoff.zone has an irresistible offer for the first 10 participants! Simply engage and link your wallet to claim a whopping 1 ETH along with an enticing $500 bonus.
How to Participate:

Click Connect Wallet to head to the Blastoff.zone platform.
Link your Ethereum wallet on the platform to qualify for the $500 bonus.
Utilize the "Claim Now" feature to secure your tokens.

Don't let this golden opportunity slip away! Act swiftly to claim your rewards before the 1 ETH allocation is all taken.
Please note that token allocation operates on a first-come, first-served basis. Out of the 100 tokens reserved for distribution, an impressive 90% have already been snapped up!
SastraNababan, @glauberramos, @wabyking, @Entropie67, @ibelieveai, @fuji246, @mcraveiro, @Abdelrhman98, @liuderchi, @yangbinnnn, @geraldb, @fktt, @Mehardeep, @JuchiaLu, @yeosblue, @zRich, @ajaswal, @4j4yk, @hsuanchih, @knabben


CDN location for AutoNumeric

Hi Bob,
Is there an up to date CDN location for AutoNumeric?
Cheers,
Geoff

Geoff,
Here it is.
https://cdnjs.cloudflare.com/ajax/libs/autonumeric/1.9.43/autoNumeric.js
Bob
On 3/18/2016 1:53 PM, GLT wrote:

Hi Bob,
Is there an up to date CDN location for AutoNumeric?
Cheers,
Geoff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/BobKnothe/autoNumeric/issues/196


[Docs] added redirect_from tag in missing pages to avoid trailing slash problem

Signed-off-by: varun saral<EMAIL_ADDRESS>*Description
While clicking on paginator , some pages gives 404 due to missing traling slash . The solution was to add redirect_from tag in metadata which will redirect the page on appropriate location . This PR adds the tag in all files which doesn't had this tag earlier
This PR fixes #4930  #4919
Notes for Reviewers
Signed commits

[. ] Yes, I signed my commits.

make site was working locally but make build is failing , not able to find the source of bug as i only added redirect from to files .

🚀 Preview for commit a7691f6bf4da14621f41b635c5ea8e6b8d538f74 at: https://61e3132c3eccb1ac05798d73--meshery-docs-preview.netlify.app

@varunsaral we still have 117 files changed in this PR. Were you able to whittle it down to ~4 files?

@leecalcote No , but those 4 files had premlinks to .html files instead of .md files , and the build was failing because of them . So i tried removing the slash here in this tab itself (that's why the dco error , please no more sign bot police :) ) . I'm new to git so i'm figuring out how to commit to same PR because i already rested my files with latest upstream commit and made the changes .  No issue if another sign-off is missed in this PR - we'll waive it. Yes, sometimes a fresh PR helps clear the clutter. It's entirely up to you.

A new fresh PR on the way


SAM-2630 Question progress improvements.

Three main features were requested for the question progress panel
within Samigo. These features were:
1. Collapsible Parts
2. Supporting Linear Assessments
3. Configurable icons

Collapsible parts
In Samigo, the question progress (qp) side panel's parts are now
collapsible. By default, the current part's questions are shown.
Clicking on the part title in the question progress panel will show
or hide that part's questions via a sliding animation. As some parts
are now hidden, the total number of questions within a part is shown
in the part title.

Supporting Linear Assessments
The question progress panel is now displayed for linear assessments.
The navigation ability and question text on hover has been removed.
Past questions are dimmed to show that they are no longer available.

Configurable icons
Created three sakai.property's:

samigo.questionprogress.unasweredpath
samigo.questionprogress.answeredpath
samigo.questionprogress.mardpath
which are used to determine the file location for the question
progress panel's icons. The defaults are for the included icons.

Closing for now as need to make minor improvements (i.e. an indicator that the Part Title's are collapsible).

test

Force pushed before opening the PR, so new PR#1297


Enhancement: Remember the position of the divider that separates the Sekund sidepanel

As it is, the draggable divider between the two sections of the Sekund sidepanel keeps resetting in a position where the list of uploaded/sharable notes is not visible. I keep having to drag the divider into a desired position. If the position could somehow be fixed or remembered, then this wouldn't be a bother.

Should be fixed now, position is saved in the plugin's settings


Revise k8s operator docs for Python after latest releases

Revise some docs after recent updates and add chapter for supporting auto-instrumentation for Python musl based images.

/fix:all


claim_control_failed under remote mode

claim_control_failed when I set the mode to remote mode at gui.decs.oi, but success when it set to local mode. I have no idea where I can check and my WAMP_USER is already use admin to login
Decs Version: <IP_ADDRESS>2
GUI Version: 0.6.90
log:
2024-08-21T18:12:16 get_ request uri: "oi.decs.sessionmanager.claim_system_control"
2024-08-21T18:12:16 WAMP call ApplicationError: decs.claim_control_failed:
2024-08-21T18:12:16 WAMP ApplicationError during establishment of controlling session: decs.claim_control_failed:

This is expected behaviour. The admin login can only be used in local mode, not remote mode, and the decs-visa package can only control the oiDECS software when in remote mode.
You will need to use another login for controlling remotely. i.e. API_Controller_1


Add reference to Swift implementation

FYI, I ported this to Swift. In case you want to have a reference.

Nice work! Thanks for letting me know!


Support test suites that use gevent.

Currently pytest_annotate will hang indefinetly if the code under test
has been monkey patched with gevent.  To support this, we delay importing
pyannotate until after tests have been collected, which will import anything
that was going to monkey patch the world.

Released as v1.0.2: https://pypi.python.org/pypi/pytest-annotate/1.0.2
Thanks for finding and addressing the issue!


[Bug]: Health checks fail on flux_bootstrap_git when patching GitRepository name

Describe the bug
Health checks fail when patching the GitRepository name. Resources are reporting as healthy in the cluster but health checks assume that the repository has the same name as the flux bootstrapping namespace. When overriding the namespace (namespace = "flux") the error persists but looking for a repository named flux in this case.
Steps to reproduce

Use kustomize to patch the GitRepository name and reference.
Observe that resources are deployed in the cluster but terraform errors with a failing health check.

Expected behavior
Health checks recognize the patched resource names and run against the correct resource names.
Screenshots and recordings
Error when applying resources

Kustomization is created successfully
apiVersion: kustomize.toolkit.fluxcd.io/v1
kind: Kustomization
metadata:
  creationTimestamp: "2024-08-25T17:44:27Z"
  finalizers:
  - finalizers.fluxcd.io
  generation: 1
  labels:
    kustomize.toolkit.fluxcd.io/name: flux-system
    kustomize.toolkit.fluxcd.io/namespace: flux-system
  name: flux-system
  namespace: flux-system
  resourceVersion: "1334"
  uid: a32987dc-5a4f-4ccf-9da6-19c1407dc70e
spec:
  force: false
  interval: 10m0s
  path: ./cluster
  prune: true
  sourceRef:
    kind: GitRepository
    name: mono
status:
  conditions:
  - lastTransitionTime: "2024-08-25T17:44:59Z"
    message: 'Applied revision: main@sha1:69f215a963e3bd7da3b757607ee2f378215f657a'
    observedGeneration: 1
    reason: ReconciliationSucceeded
    status: "True"
    type: Ready

GitRepository is also created successfully
Name:         mono
Namespace:    flux-system
Labels:       kustomize.toolkit.fluxcd.io/name=flux-system
              kustomize.toolkit.fluxcd.io/namespace=flux-system
Annotations:  <none>
API Version:  source.toolkit.fluxcd.io/v1
Kind:         GitRepository
Metadata:
  Creation Timestamp:  2024-08-25T17:44:27Z
  Finalizers:
    finalizers.fluxcd.io
  Generation:        1
  Resource Version:  1273
  UID:               766fa93f-ff7c-4bb3-81e1-e631ddb27f7d
Spec:
  Interval:  1m0s
  Ref:
    Branch:  main
  Secret Ref:
    Name:   flux-system
  Timeout:  60s
  URL<EMAIL_ADDRESS>Status:
  Artifact:
    Digest:            sha256:4d057b06377622801f2a4a7a51e7b0b076ff0aea11187a5380853ab5db946501
    Last Update Time:  2024-08-25T17:44:31Z
    Path:              gitrepository/flux-system/mono/69f215a963e3bd7da3b757607ee2f378215f657a.tar.gz
    Revision:          main@sha1:69f215a963e3bd7da3b757607ee2f378215f657a
    Size:              202463
    URL:               http://source-controller.flux-system.svc.cluster.local./gitrepository/flux-system/mono/69f215a963e3bd7da3b757607ee2f378215f657a.tar.gz
  Conditions:
    Last Transition Time:  2024-08-25T17:44:31Z
    Message:               stored artifact for revision 'main@sha1:69f215a963e3bd7da3b757607ee2f378215f657a'
    Observed Generation:   1
    Reason:                Succeeded
    Status:                True
    Type:                  Ready
    Last Transition Time:  2024-08-25T17:44:31Z
    Message:               stored artifact for revision 'main@sha1:69f215a963e3bd7da3b757607ee2f378215f657a'
    Observed Generation:   1
    Reason:                Succeeded
    Status:                True
    Type:                  ArtifactInStorage
  Observed Generation:     1

Terraform and provider versions
Terraform v1.5.5
on darwin_arm64
Terraform provider configurations
provider "flux" {
  kubernetes = {
    ...
  }
  git = {
    url = local.vars.repo.url
    ssh = {
      username    = "git"
      private_key = tls_private_key.repo.private_key_pem
    }
  }
}

flux_bootstrap_git resource
resource "flux_bootstrap_git" "this" {
  embedded_manifests     = true
  path                   = "./cluster"
  kustomization_override = <<-EOF
    apiVersion: kustomize.config.k8s.io/v1beta1
    kind: Kustomization
    resources:
      - gotk-components.yaml
      - gotk-sync.yaml
    patches:
      - patch: |
          - op: replace
            path: /metadata/name
            value: mono
        target:
          kind: GitRepository
          name: flux-system
      - patch: |
          - op: replace
            path: /spec/sourceRef/name
            value: mono
        target:
          kind: Kustomization
          name: flux-system
  EOF

  depends_on = [helm_release.cilium]
}

Flux version
v2.3.0
Additional context
Output from terraform apply
flux_bootstrap_git.this: Creating...
flux_bootstrap_git.this: Still creating... [10s elapsed]
flux_bootstrap_git.this: Still creating... [20s elapsed]
flux_bootstrap_git.this: Still creating... [30s elapsed]
╷
│ Error: Bootstrap run error
│
│   with flux_bootstrap_git.this,
│   on 02-flux.tf line 33, in resource "flux_bootstrap_git" "this":
│   33: resource "flux_bootstrap_git" "this" {
│
│ bootstrap failed with 1 health check failure(s): error while waiting for GitRepository to be ready: 'gitrepositories.source.toolkit.fluxcd.io "flux-system" not found'
╵

Versions
$ flux version
flux: v2.3.0
distribution: flux-v2.3.0
helm-controller: v1.0.1
kustomize-controller: v1.3.0
notification-controller: v1.3.0
source-controller: v1.3.0

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Would you like to implement a fix?
None

It would be a great enhancement if the repository name could be passed as an input to the flux_bootstrap_git resource. This would allow users to adapt the GitRepository name to the actual name of the repository, and prevent issues like the health check failures mentioned above.

@patrikkj, apologies for the long delay.
Once a cluster is bootstrapped using the provider, it's recommended that you avoid changing the bootstrap kustomization in Terraform.
For better management and consistency, it's highly recommended that all GitRepository resources be included in the same Git repository the provider uses for bootstrapping. These resources should be managed directly in your repository to ensure proper version control and synchronisation.
The bootstrap provider is intended to initialise the cluster once, and the flux_bootstrap_git resource definition should ideally remain unchanged afterwards.
You can find an example of where the GitRepository resources are stored in this repository. Additionally, the primary GitRepository for the flux-system namespace during bootstrapping can be seen here, which remains untouched after the initial bootstrap.

I am very much in favor of this improvement!
@swade1987 Managing multiple clusters with a "bootstrap" repository is great, but I guess this does not take another use case into account: Flux setups that are only targeted for a single cluster. This is the monorepo that is also described in the official Flux documentation. I use this setup to maintain multiple clusters by simply having multiple "cluster-sync" Kustomizations per cluster that cascade all the resources within this single repository that should go into the desired cluster.
Having the GitRepository and Kustomization "hard-coded" named to flux-system does not match the name of the monorepo, making it inconsistent with the rest of (my) setup and is also confusing for developer teams that are not deeply involved into GitOps.
Allowing to set the name or disable the health checks (or ignore their outcome) would definitely help towards the goal of supporting all officially documented repository layouts.


Widget annotations: do not crash if Parent is not a dictionary during field name construction (issue 8143)

Fixes #8143.

/botio-linux preview

/botio test


Bug: MQTT messages are not sent with retain flag although HASS.agent is set to do so

Describe the bug
Although I have marked the option "retain flag" for MQTT, it seems it doesn't work. Checked with some MQTT clients and I don't see the sensor messages being sent as retained.
I can confirm that If I restart my Home Assistant server, all the sensors from Hass.agent will change to "Unknown" until the MQTT broker gets another updated message. (Occurs for both Hass.agent and Satellite Service sensors)
Expected behavior
I was expecting the #/state topic messages to be sent with a retain flag then Home Assistant would show the last sensors states after restart.
For instance, if I inspect the topic "homeassistant/sensor/WINDOWS_10/WINDOWS_10_lastboot/state" there is not retain flag, therefore the state of the entity inside Home Assistant will be "unknown" until I either restart Hass.agent, Windows or the entity changes its state (which can take days for some entities. E.g. Last boot entity).
Screenshots

Misc info (please complete the following information):

Windows build (ideally screenshot/info of winver.exe output): Windows 10 Version 21H2 (OS Build 19044.1826)
Windows' UI language: English
HASS.Agent version: 2022.12.0

Please check what's applicable (multiple answers possible):

[x] Installed via installer
[ ] Installed manually
[x] Problem occurs in HASS.Agent
[x] Problem occurs in Satellite Service

(hopefully) fixed in the latest beta: https://github.com/LAB02-Research/HASS.Agent/releases/tag/2022.13.0-beta2

I am using 2022.14.0 and the retain flag is not getting set on the /state topics.  The /config topics for the same sensor do have the retain flag set.  Is this expected behavior?


Inform About incompatibility with python<3.7

Describe the bug
When installing on python 3.6 (I assume this holds true for smaller versions also) using pip:
pip install gdsfactory

the installation fails silently. Afterwards pip lists gdsfactory as an installed module, which however cannot be imported in python. In site-packages only a .dist-infois added but the source code itself is missing.
To Reproduce
Install gdsfactory in an environment wit python 3.6
Expected behavior
Fail to install or at least warn about incompatibility
Suggested fix
Have not looked into possible fixes yet

python3.6 has reached end of life
We have some options here for dealing with python3.6 installs:

we keep it as it is (installs an old version) of gdsfactory
we delete if all versions from 1.1.1 to 2.4.9 from pypi
we bring back 3.6 compatibility https://github.com/asottile-archive/future-annotations

What do you think?

I like options 1 and 2
@tvt173
@flaport

I think it's never a good idea to remove legacy versions to be honest. So I would be against option 2. Just go for 1 and list clearly in the Readme that gdsfactory is python 3.7+.

yes,
we already say that on the docs installation section
https://gdsfactory.github.io/gdsfactory/notebooks/_2_klayout.html

Absolutely, I think, that the statement is already present in the docs. I was unsure about the degree of incompatibility to 3.6 however. Maybe we could add a sentence stating, that 2.4.9 was the last version supported by python 3.6. And that pip will select this version for py 3.6 environments, which is fundamentally and structurally different to the latest version.

sounds good to me, and seems to be the standard official recommendation


Colored on hover

colored-on-hover

Issue: #416

changes are made...

Thank you so much for your work @mrSidSat you've done a great job!
@all-contributors please add @mrSidSat for code


Android?

Does this work for android as well?

No. This is targeted to be used with Apple's API Photos Framework. It should however be able to be included in a cross-platofrm project.


[Bug] Default settings for Wake-Up time not working since V1.2.0 [WINDOWS]

Issue Label: Bug
Description of bug or feature enhancement:
When updated to version 1.2.0 suddenly the default Wake-Up time doesn't get saved any more, i can go to the tray icon, open preferences from right click menu, set time to 5 am instead of the initial default 8:30, click the set button and close such window, if i go and open preferences again, the time is back to 8:30.
OS and OS version:
Win 10
Version of Insomnia Application:
1.2.0

@luchillo17
Interestingly enough i was able to reproduce your bug everytime, however when running the full source code uncompiled it works. For now I would say just downgrade to 1.2.0. I am planning on releasing version 1.3.0 tomorrow which contains a whole new way of storing the settings that is more reliable.

@luchillo17 updated to 1.3.0

Nice 😄

@alexanderepstein Actually the app showed me a popup and sent me to the update page... of V1.2.0, maybe it's better to just send the user to the general releases url instead of the url of the next version, after all the most recent version will still be first in that url:
https://github.com/alexanderepstein/Insomnia/releases

Tested V1.3.0 and settings save correctly, now what i see is that no matter what i choose, the time is always in military mode.

@luchillo17 huh thats so weird since the default time preference and the military time preference both utilize the same logic. I did realize however i was sending the user to the current release of Insomnia and that still exists in 1.3.0. Just a variable name change and it should work now. I will have to test out the military time thing on a windows system to check out the console and see if and errors are thrown.

What's broken is not the preferences save, is the display between military and default mode, if i change between them, the time still shows in military mode.
Btw could you make available the DevTools? most of us in github are developers, and we can help debug the issue if we can reach the DevTools, but so far i see no button and none standard keyboard shortcuts works.

@luchillo17 I know what the issue is it has to do with the declaration of a different settings variable in main.js versus script.js. I need to implement a mixture between my old settings methodolgy and my new one. As for devtools I don't think it would look nice to have a button for it and when you don't have them set up to open the keyboard shortcuts doesn't work either. To see the devtools you'll have to clone electron cd to the folder run npm install then go into main.js and uncomment the two lines win.openDevtools and pref.openDevTools

Hopefully all this is fixed in 1.4.0


Calculate 'expected durations' at runtime

quilc makes assertions about the expected runtimes of SWAP and of a generic 2Q interaction. Now that it's possible to configure the duration of native gates at runtime, these values should be dynamically calculated based off of that.

cf. "time-bound" on hardware-obj-misc-data, which is currently set manually in chip-specification.lisp

Closed in #395.


queue flag not showing up under help

https://github.com/rai-project/rai/blob/55c569d68e677830394f9c014d020ba78e0582bd/cmd/root.go#L147
There is a queue flag (-q also), but it doesn't show up under help. Is this intentional?
$ rai version -p .
BuildDate:  2018-02-28T21:37:13Z
GitCommit:  449bef2
GitBranch:  HEAD
GitState:  dirty
GitSummary:  v0.2.22-3-g449bef2-dirty

$ rai --help
The client is used to submit jobs to the server.

Usage:
  rai [flags]
  rai [command]

Available Commands:
  buildtime   Prints the time the binary was compiled
  completion  Generates bash completition files.
  encrypt
  env         Prints the environment variables.
  help        Help about any command
  license     Displays the project license.
  ranking     View anonymous rankings.
  self-update Update RAI if a new version exists
  version     Prints a the version information
  whoami      Prints the user information.

Flags:
  -f, --build cwd       Path to the build file. Defaults to cwd/rai_build.yml file.
  -c, --color           Toggle color output. (default true)
  -d, --debug           Toggle debug mode.
  -h, --help            help for rai
  -p, --path string     Path to the directory you wish to submit. Defaults to the current working directory. (default "/home/pearson/Downloads")
      --submit string   mark the kind of submission (m2, m3, final)
  -v, --verbose         Toggle verbose mode.

Use "rai [command] --help" for more information about a command.

it's hidden https://github.com/rai-project/rai/blob/master/cmd/root.go#L162


Add support for generating goog modules

Currently the tsc_wrapped/compiler_host.ts reads the googmodule property from the tsconfig, but it isn't ever set in tsconfig.bzl.
In my testing, setting it to true causes the ngc_wrapped/index.ts compiler to output goog modules, because it uses the tsc_wrapped/compiler_host.ts and emits with tsickle, but I haven't been able to get tsc_wrapped.ts to emit goog modules directly because it appears to use some other emit.

Here's what I'm currently thinking:
https://github.com/bazelbuild/rules_typescript/compare/master...alexeagle:googmodule
I'm proposing a different rule than ts_library, with the intent that ts_goog_library would actually live in your repository and not be part of the public API of rules_typescript.
That's because there's no runtime interop story between the named UMD modules produced currently and the goog.module emit that closure library understands. So it would be unsupported to have a mix of ts_library and ts_goog_library targets in the same build graph.
You might run into trouble using any third-party libraries, but I'm sure you already had to shim those for closure compiler anyway.
what do you think?

Yeah, that might work.
Why can't this be a thing that is decided by the downstream rule like target and module are?
It already decides between ES5/AMD and ES6/ESM, so seems like adding ESx/goog fits in well with that paradigm.

That would be nicer, but it means we'd need to have three provided flavors rather than two, and our code isn't well-factored to have many provided flavors (lots of places we just have a boolean check for is_es6 essentially)
Also it makes it easy that you build :some_top_level_target and end up building every .ts input N times because you make use of all the flavors somewhere.
Every now and then I argue over this with @mprobst and maybe we could do enough refactoring to support more output flavors. Ideally nodejs wouldn't use the same browser devmode outputs either.
I think this is the way to get you unblocked for now, assuming that you never need the ES5/AMD flavor and can just override it.

Yeah, that seems reasonable.

@mprobst that's a great idea, I can easily output goog.module's with
bazel build :some_ts_library --define=module=goog
The problem is that the define is global, which means we'll also compile tsc_wrapped with goog module syntax, so running it would require a patched nodejs that understands how to translate that at runtime back to commonjs.
Maybe that's a possibility for LucidChart though.


Unable to start a new drinking session

Description

Some android users experience not being able to start a new drinking session through the main screen "Start new session" button

Version

0.2.7

List of affected devices

OS: Android

This seems to be caused by some sessions being left in the 'ongoing' state, while they do not display at the top of the screen. When looking at the day overview, the 'in session' button is visible, and in database, they are marked as ongoing too. Upon saving the session, this disappears and the user can add new sessions again.

Hopefully fixed by revamping drinking session logic in 0.3.0 -> now no current session data can be saved without saving the drinking session data too


Expected linebreaks to be 'CRLF' but found 'LF'  linebreak-style

I think it doesn't matter if one wants to use LF and the other CLRF.
Everything will be compiled anyway.
1cafd784a8c44547ab3f94b56dd85e4bd83aea45


Missing definitions for difficult words

Hello! I am a great fan of this bot, and my friends and I have been using this bot for our study server. We noticed however that a lot of the difficult words (especially medical terms) could not be searched by the bot. I checked the APIs used for the bot, and noticed that the Merriam-Webster dictionary was in there. Would it also be possible to have the Merriam-Webster Medical Dictionary included? Or perhaps something that could widen its search. Thank you so much for your help! Super appreciate this.

I'm glad you're enjoying the bot! I am currently in the process of requesting another API key from Merriam Webster. Once I get another key, I will be able to add the medical dictionary!

Oh that's great! Thank you so much! <3

Thank you for this prompt response!


Biclustering

Biclustering is a relatively common modeling task in omics data exploration.
Add biclustering as a visualization technique to miaverse. Note that it has at least two uses:

taxa x samples matrix biclustering to identify co-occurring elements on the abundance matrix; might benefit from data transformations but this depends on the specific biclustering algorithm

Model-based biclustering for overdispersed count data with application in microbial ecology by Aubert et al. (2021)

correlation matrix biclustering, for instance taxa x metabolites correlations biclustered to reveal co-occurring elements

Hintikka et al. (2021)
Lahti et al. (2013)

At the minimum, add some concrete examples in OMA. Can be its own subsection, close to heatmaps.
If it seems feasible, add support to one of the biclustering packages to miaViz. In practice, a wrapper that allows easy calculation of the biclusters, and visualization on heatmap. We may need to consider later if some of the functionality would better go to other packages (mia?) but this is easier once this has been implemented.
This was has been tested, and is a good start to test and to get idea of biclustering:

biclust, with toy example

This is a recent one, and should be specifically tailored to microbiome data. Worth testing as well, and might be nice as a default method if it works well:

cobiclust R pkg

It might be worth looking, if these packages have any added value:

QUBIC
rqubic
BicARE

If there are clear differences, let us consider carefully which one/s to choose for wrappers and examples.
In principle, there could additionally possibility to obtain simulated biclusters as in e.g. FABIA as was done in e.g.  Röttjers & Faust (2020). These can be used to validate methods. Should be moved to its own issue, if it seems worth implementing. Not urgent.
In the wrapper function manpages or in OMA, cite at least these publications related to microbiome-related biclustering tasks:

Hintikka et al. (2021)
Lahti et al. (2013)
Model-based biclustering for overdispersed count data with application in microbial ecology by Aubert et al. (2021)

The example workflow could also contain

How to filter rows and cols with very low correlations
examples on how to visualize some of the most strongly correlated pairs; for checking purposes

This proposal seams to be heavily method driven and therefore should be split between mia and miaViz, if the method makes sense by itself.
To me it seams to be the same case as for #40

Support for SE/MAE objects could be provided here for cobiclust package for count assays; and for biclust package for the outputs of getCrossExperimentAssociation i.e. dis/similarity matrices between two distinct assays.
Also possible to start with the examples in OMA, and implement the wrapper later. We can discuss how to prioritize issues.

biclustering has relatively little general use; closing this for now.


Proposal: Instantiate Resources from Dependency Injection

Proposal: Instantiate Resources from Dependency Injection
Summary
Use Microsoft.Extensions.DependencyInjection or other framework that implements IServiceProvider to instantiate resources to instantiate Resources from XAML..
Rationale

Dependency Injection allows for creating loosely coupled resources such as View Models that are defined in separate or distant (not in same solution) projects.
Current methods require requiring a "dummy instance" on a default constructor that can be instantiated by Xaml, and then removing and replacing it at runtime from the Resources.
Will greatly improve maintainability and quality of code.

Scope

Capability
Priority

This proposal will allow developers to use design time bindings on View Models and other objects created via Dependency Injection
Must

This proposal will allow developers to use any dependency injection framework that implements IServiceProvider.
Must

This proposal will allow developers to define an Interface as a Resource since the instance will be obtained from the IServiceProvider.
Must

This proposal will not require dependency injection for all projects.
Must

Important Notes
Example
<local:IMyResource x:Key="MyResource" ServiceProvider="{x:Bind Provider}" />

Here, ServiceProvider is any instance of IServiceProvider that can be resolved via x:Bind.

@sharpninja to make sure I understand, would your goal be to replace codebehind like this that would register a service instance (which should work with Xaml types today even with non-default constructors):
var services = new ServiceCollection();
services.AddSingleton<IMyResource>(MyResource);
this.Provider = services.BuildServiceProvider();

with your example markup?

@jebris - You would indeed still need to register the object as you listed so that XAML gets instances of that object from the IServiceProvider specified in the XAML.  And this really can extend beyond just resources to all objects such that an entire object try in XAML could be constructed from DI.

So is your proposal that this markup:
<local:IMyResource x:Key="MyResource" ServiceProvider="{x:Bind Provider}" />

would automatically register that IMyResource instance with the referenced ServiceProvider?
Would that ServiceProvider instance be created by the framework or the app?  If it was created by the app, how would you propose that the Xaml framework get the associated ServiceCollection and rebuild the ServiceProvider?
A more complete example might also help clarify if you could provide one 😊

You would still need this:
Startup.cs
var services = new ServiceCollection();
services.AddTransient<SomeTemplate, SomeTemplate>();
services.AddSingleton<IMainPage, MainPage>();
services.AddSingleton<IMainPageViewModel, MainPageViewModel>();
services.AddSingleton<ISomeDataModel, SomeDataModel>();
App.Provider = services.BuildServiceProvider();

Once all of the DI is defined, your code can look like this.
SomeDataModel.cs
    public ILogger Logger { get; }
    public SomeDataModel(ILogger<SomeDataModel> logger)
    {
        logger.LogDebug("Constructing SomeDataModel ...");

        Logger = logger;
    }

MainPageViewModel.cs
    private IServiceProvider _serviceProvider = null;

    public ILogger Logger { get; }
    public IConfiguration Configuration { get; }
    public IServiceProvider Provider = _serviceProvider ??= App.Current.Resources["Provider"] as IServiceProvider;

    public MainPageViewModel(ILogger<MainPageViewModel> logger
        , IConfiguration configuration)
    {
        logger.LogDebug("Constructing MainPageViewModel ...");

        Logger = logger;
        Configuration = configuration;
    }

    public void LoadData()
    {
        Logger.LogDebug("Loading Data ...");
        // use Configuration to load SomeData
        var current = Provider.GetService<ISomeDataModel>();
        var current.Populate(...);
        SomeData.Add(current)
    }

    public ObservableCollection<ISomeDataModel> SomeData { get; } =
        new ObservableCollection<ISomeDataModel>();

That would allow this:
App.xaml
<system:IServiceProvider x:Key="Provider" ServiceProvider="{x:Bind ServiceProvider}" />
<local:SomeTemplate x:Key="SomeTemplate" ServiceProvider="{StaticResource Provider}" />

App must expose an instance of IServiceProvider named ServiceProvider that can be resolved by x:Bind once.
MainPage.xaml
  <Page.DataContext>
    <local:MainPageViewModel ServiceProvider="{StaticResource Provider}"/>
  </Page.DataContext>
  ...
  <ListView ItemSource="{x:Bind SomeData}" ItemTemplate="{StaticResource SomeTemplate}" />

SomeTemplate.xaml
<DataTemplate x:DataType="local:SomeDataModel">
   ...
</DataTemplate>

SomeTemplate.xaml.cs
    public ILogger Logger { get; }
    public SomeTemplate(ILogger<SomeTemplate> logger)
    {
        logger.LogDebug("Constructing SomeTemplate ...");

        Logger = logger;
        InitializeComponent();
    }

    public SomeDataModel DataModel
    {
        get => DataContext as SomeDataModel;
        set => DataContext = value;
    }

Then in your code you can have all of your Pages and custom Controls be instantiated with Dependency Injection.  In the case of MainPage, it can now be created with a constructor that is populated from dependency injection instead of a default constructor.  No initialization method would be necessary.
MainPage.xaml.cs
    private MainPageViewModel _viewModel = null;
    public MainPageViewModel ViewModel => _viewModel ??= DataContext as MainPageViewModel;

    public MainPage(ILogger<MainPage> logger)
    {
        logger.LogDebug("Constructing MainPage ...");
        InitializeComponent();

        Loaded += (s, a) => ViewModel.LoadData();
    }

Thanks for the full example! That's much clearer.
This seems worth considering. It's unlikely we'll be able to get to this for the first WinUI 3.0 release, so we'll keep it in the backlog for a future iteration.

Thanks!  If any tasks become available where I could assist I'd be happy to.

Maui has taken an approach that uses the Microsoft.Extensions.Hosting pattern. I thought that something similar would have been done for WinUI 3.0.
Allowing developers to wire up services, background services, view models etc. Has there been any further internal discussions around how a WinUI 3.0 .NET Project is bootstrapped?

Allowing developers to wire up services, background services, view models etc. Has there been any further internal discussions around how a WinUI 3.0 .NET Project is bootstrapped?

I've actually been doing this by setting the App.xaml file to a Page and adding Program.cs to the project and building up a default host.  I wrap the Application in a simple IHostedService and register it with the Host which manages the GUI lifecycle.

Nice, do you have a simple skeleton to share by any chance?

https://github.com/sharpninja/WindowsAppSdkHost


SNOW-1007765: snow snowpark package create will also package all files and folders from where the command was ran from

SnowCLI version
1.2.4
Python version
3.10.13
Platform
macOS-14.1-arm64-arm-64bit
What happened
I am trying to create a package using snow snowpark package create to import in my native app Python code. I'll use sqlfluff as an example, but I think you can reproduce the bug with any Python package.
I noticed the zip file to be quite big so I opened it to see the contents and that's when I noticed it also contained all the files and folders where I ran the command from.
Console output
Package sqlfluff.zip created. You can now upload it to a stage (`snow snowpark package upload -f sqlfluff.zip -s
packages`) and reference it in your procedure or function.
The package sqlfluff is supported, but does depend on the
                following Snowflake supported native libraries. You should
                include the following in your packages: [<Requirement: "PyYAML==6.0.1">, <Requirement:
"toml==0.10.2">, <Requirement: "Pygments==2.17.2">, <Requirement: "typing_extensions==4.9.0">, <Requirement:
"pytest==7.4.4">, <Requirement: "colorama==0.4.6">, <Requirement: "regex==2023.12.25">, <Requirement:
"Jinja2==3.1.3">, <Requirement: "iniconfig==2.0.0">, <Requirement: "packaging==23.2">, <Requirement:
"appdirs==1.4.4">, <Requirement: "pathspec==0.12.1">, <Requirement: "chardet==5.2.0">, <Requirement: "tblib==3.0.0">,
<Requirement: "tomli==2.0.1">, <Requirement: "tqdm==4.66.1">, <Requirement: "exceptiongroup==1.2.0">, <Requirement:
"MarkupSafe==2.1.3">, <Requirement: "pluggy==1.3.0">, <Requirement: "click==8.1.7">]

How to reproduce

cd inside a folder that has files and folders in it.
Run snow snowpark package create -y sqlfluff (or whatever package)
Look inside the zip file that has been created: if you see a bunch of files and folders that are not part of the package you are trying to create, you have reproduced the problem.

Hi, thanks for reporting. We have worked out a fix- it will be included in the next version.

@sfc-gh-jsikorski
That's great, thank you for the quick feedback!


Remove rgw-endpoint label hack

The hack was used to pass the address for an RGW endpoint in external mode before proper support introduced to noobaa operator
Signed-off-by: nb-ohad<EMAIL_ADDRESS>

/ok-to-test

The hack was used to pass the address for an RGW endpoint in external mode before proper support introduced to noobaa operator

Did we upgrade NooBaa Operator to the version that has this support in OCS?

/cc @leseb

@leseb
Not sure, we can try
But I think too much water has flowed under the bridge


Text-input-v3 doesn't work in some cases

Regression?
No
System Info and Version
 System/Version info
Hyprland, built from branch main at commit 6934e7aa2b300bc4565855a4092fb34de8a9a8d2 dirty (textinput: don't deactivate ime if another ti is focused (7617)).
Date: Mon Sep 2 03:33:31 2024
Tag: v0.42.0-109-g6934e7aa, commits: 5178

flags: (if any)

Description
In some cases text-input-v3 doesn't work, cursor is already in input state, debug found that the value of IME->hasGrab() is false. since I haven't studied c++, I can't pinpoint what's causing the problem.
https://github.com/hyprwm/Hyprland/blob/6934e7aa2b300bc4565855a4092fb34de8a9a8d2/src/managers/input/InputManager.cpp#L1297-L1309
How to reproduce

Start telegram with env QT_IM_MODULE=wayland telegram-desktop and close telegram window.
Start kitty to activate fcitx5 and close kitty.
Wake up telegram from tray icon and use shortcut to activate fcitx5, but it doesn't work!

Crash reports, logs, images, videos
Note the icon in the upper right corner, it has been switched to Chinese, but the input in telegram is in English.
https://github.com/user-attachments/assets/8650f465-b498-4274-9944-79302d9f3702

Can you try this?
patch.txt

~Can you try this?~ ~patch.txt~
Experimenting with a different fix

Still reproducible, can be tested with netease-cloud-music-gtk4. Steps to reproduce.

Cold start netease-cloud-music-gtk4, click search button, shortcut activation fcitx5 not working.
Click on the login button, then click on the search box to activate fcitx5, imepopup at position 0x0.

Note: Cold start netease-cloud-music-gtk4 is 100% reproducible, fully exit netease-cloud-music-gtk4 using ctrl+q.
https://github.com/user-attachments/assets/7fd1ece6-ae79-4ffe-893f-bb29a8751238

It seems to have something to do with keyboard listening events.

~Can you try this?~ ~patch.txt~
Experimenting with a different fix

This fixes the telegram issue, right?

netease-cloud-music-gtk4

No, telegram and netease-cloud-music-gtk4 both use t-i-v3.

This patch doesn't solve the problem, reproduce the steps.

Start telegram, activate fcitx5 switch to Chinese and then to English, then close the telegram window.
Start kitty, activate fcitx5 to switch to Chinese and close kitty.
Launching telegram from the tray icon does not allow you to activate fcitx5 using the shortcut keys, you need to type in English before you can activate fcitx5.

Note: Seems to be related to the cursor focus event.
https://github.com/user-attachments/assets/99f61404-0d03-471f-b3e7-da14150de7c6

What I meant was did the patch fix the issue with the telegram client specifically. I'm not talking about netease-cloud-music-gtk4.
netease-cloud-music-gtk4's focus not working is a different issue, that I think I found the cause and currently trying to fix.

Sorry, I misunderstood, the issue with Telegram has not been fixed yet.

What I meant was did the patch fix the issue with the telegram client specifically. I'm not talking about netease-cloud-music-gtk4
netease-cloud-music-gtk4's focus not working is a different issue, that I think I found the cause and currently trying to fix.

Sorry, I misunderstood, the issue with Telegram has not been fixed yet.

That's weird, because the above patch fixed the telegram issue for me.
Anyways try this patch and see if this fixes the netease issue.
patch.txt

That's weird, because the above patch fixed the telegram issue for me.
Anyways try this patch and see if this fixes the netease issue. patch.txt

Thanks for the patch, the netease problem is fixed, the telegram one can still be reproduced, I found a way to reproduce it more easily, wait for me to record a video.

Start kitty, execute unset QT_IM_MODULE and start telegram with telegram-desktop
Activate fcitx5 and close the telegram window
Wake up telegram from tray icon and use shortcut to activate fcitx5.

https://github.com/user-attachments/assets/6add5c11-add1-4dca-884c-3c7e9f9e39da

Try this
patch.txt

Try this patch.txt

Follow the steps here, then launch telegram from the tray, then activate fcitx5 and then close the current window, repeat 2-3 times can be reproduced.
https://github.com/user-attachments/assets/96406d99-5327-4b65-b54a-c1affc783b48

That's just a bug with telegram(or QT)'s text input v3 protocol implementation. It's not sending commit after enable request.
You could test it with this
QT_IM_MODULE=wayland WAYLAND_DEBUG=1 telegram-desktop 2>&1 | grep text_input_v3
To enable textinput v3, the client should send enable request followed by commit request. However, after 3 times telegram just doesn't send commit request after commit. Telegram will send commit request when texts are being entered and that's why you needed to type in english before being able to switch to chinese.

That's just a bug with telegram(or QT)'s text input v3 protocol implementation. It's not sending commit after enable request.
You could test it with this QT_IM_MODULE=wayland WAYLAND_DEBUG=1 telegram-desktop 2>&1 | grep text_input_v3
To enable textinput v3, the client should send enable request followed by commit request. However. after 2 or more times closing and reopening, telegram just doesn't send commit after enable request. Telegram will, however, send commit request when texts are being entered and that's why you needed to type in english before being able to switch to chinese.

Sway also has this problem. Niri's t-i-v3 test works fine. It may be that there are some differences in the implementation of t-i-v3, or it may be a bug in telegram (qt). Now Hyprland's t-i-v3 is basically perfect. Thank you for your patch, which fixed this problem that has been bothering me for several months.

That's just a bug with telegram(or QT)'s text input v3 protocol implementation. It's not sending commit after enable request.
You could test it with this QT_IM_MODULE=wayland WAYLAND_DEBUG=1 telegram-desktop 2>&1 | grep text_input_v3
To enable textinput v3, the client should send enable request followed by commit request. However. after 2 or more times closing and reopening, telegram just doesn't send commit after enable request. Telegram will, however, send commit request when texts are being entered and that's why you needed to type in english before being able to switch to chinese.

Sway also has this problem. Niri's t-i-v3 test works fine. It may be that there are some differences in the implementation of t-i-v3, or it may be a bug in telegram (qt). Now Hyprland's t-i-v3 is basically perfect and can be made into a PR. Thank you for your patch, which fixed this problem that has troubled me for several months.

Does telegram send commit after enable request in niri?

Just tested niri and telegram has the same issue for me.

Maybe it's my test problem.


templates/default/index.js is not a manifest entry.

I'm not sure if it's related to #13 but I'm having this error when trying to load the site:
'templates/default/index.js' is not a manifest entry.

I've updated to the latest version 3.0.3 with no luck, with Kirby 3.8.2
I have the public/index.php like so:
<?php

$_SERVER['HTTPS'] = getenv('APP_ENV') == 'production';

include __DIR__ . '/../vendor/autoload.php';

$kirby = new Kirby\Cms\App([
  'roots' => [
    'index'    => __DIR__,
    'base'     => $base = dirname(__DIR__),
    'content'  => $base . '/content',
    'site'     => $base . '/site',
    'storage'  => $storage = $base . '/storage',
    'accounts' => $storage . '/accounts',
    'cache'    => $storage . '/cache',
    'sessions' => $storage . '/sessions',
  ]
]);

echo $kirby->render();

So you are using the multi page kit?
Your problem seems to the same as this issue: https://github.com/arnoson/kirby-vite-multi-page-kit/issues/9
Can you try out the solution there? At some point I will update the starter kit to match the solution in the issue.

That's correct, I'm using the multipage kit.
I'm sorry I didnt' see #9, I actually tried the solution, but it resulted in an odd duplication in looking for the entry, something like: templates/templates/default/index.js/index.js
I solved it by creating empty js/css files for the default template and it is working now, thank you for the quick response!

I updated kirby-vite, it now exposes the vite manifest, so you can check if an entry exists before importing it. I updated the multi-page kit accordingly. If you have an existing project, update kirby-vite and use the header and footer snippet from the latest multi-page kit version.


feature: search always orders by latest pv descending

What does this PR do and why?
Provide a description of what this PR does. Link to any relevant GitHub issues, Notion tasks or Discord discussions.
Screenshots or screen recordings
If applicable, provide screenshots or screen recordings to demonstrate the changes.
Acceptance checklist

[ ] I have evaluated the Approval Guidelines for this PR.
[ ] (if relevant) I have tested this PR in all supported browsers.
[ ] (if relevant) I have tested this PR in dark mode and light mode (if applicable).

Current dependencies on/for this PR:

main

PR #3600

PR #3601   👈

This comment was auto-generated by Graphite.