Release v0.12.0

@kelseyhightower Hi there,
Any plan on releasing the v0.12.0 soon, or at least a new alpha or beta ?

Ping

@carmstrong @gabrtv @bacongobbler @kelseyhightower Anyone ?

The only person who can authorize a new release is kelsey, as per https://github.com/kelseyhightower/confd/blob/master/CONTRIBUTING.md#bdfl. He's been busy with other work so sorry about the long wait... We're all maintaining this project on our own free time.
Since it's been a long time I might take a weekend and create a fork with an "unofficial" 0.12 release and maintain the fork, but any official release on the official confd repo has to be signed off by him.

It would be good to have 0.12 release as it follows go best practice and is thus much easier to package.

Hi @kelseyhightower - Is there anything I can do to help get v0.12 out? I know you're busy so if there's any code / documentation I can help write let me know.

I've gone ahead and made a v0.12.1 release on my fork that includes many of the PRs here. See the v0.12.0 and v0.12.1 release notes for more information on the changelog.
Assuming this fork is "dead", I'll be happy to continue maintaining confd there.

@kelseyhightower any chance of an official 0.12 release ?

0.12.0 release is already here:
https://github.com/kelseyhightower/confd/releases/tag/v0.12.0


Maybe we can have a tree command?

It seems to be possible via Spigot.
https://hub.spigotmc.org/javadocs/spigot/org/bukkit/World.html#generateTree-org.bukkit.Location-org.bukkit.TreeType-
https://hub.spigotmc.org/javadocs/spigot/org/bukkit/TreeType.html
EDIT: Er, I want a command that can generate a tree haha

Added in my latest PR


Added PUMA 4mm model

Pull Request
Description:
Added the new model for PUMA 4mm isometric model
Changes Made:

Made the necessary changes to resources and constants to add a new model
Updated version number

Related Issues:
None
Testing Done:
Tested on one CT
Screenshots or GIFs (if applicable):

Reviewer Checklist:

[x] Code is clean and follows the team's coding standards.
[x] Tests have been added or updated to cover the changes.
[x] Documentation has been updated or added if necessary.
[x] The code builds and runs without errors.
[x] The changes have been tested on relevant environments or browsers.
[x] Any relevant dependencies have been updated.
[x] The changes have been reviewed for security implications.

@LalithShiyam, I think we should do that as a postprocessing measure.
Although I'm unsure if we should smooth all the segmentations or each label individually, @mprires, could you try that or let me know if I should do it?

@Keyn34 This postprocessing shouldn't be in moosez, we should move it to pumaz. I would just use c3d, since we already have it downloaded for the package.


[Build] bundle OpenSearch Dashboards

Description
Produce an OpenSearch Dashboards bundle with plugins provided by a manifest.
This requires the manifest to be in order with plugins first and then Dashboards
being last. The plugins will need to pulled down and placed in the plugins
folder. Then we need to bootstrap to generate the modules so that we can actually
install the plugins to the release build.
What is missing?

Environment setup, this requires nvm and yarn to be already set up.
We will need to set that up.
Signing of the artifact
Diff between release and snapshot
Default script for plugins, this will need more information on how we want this to
look like.
All the other plugins included into the manifest.

Signed-off-by: Kawika Avilla<EMAIL_ADDRESS>Issues Resolved
https://github.com/opensearch-project/opensearch-build/issues/158
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@a0f2614). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #435   +/-   ##
=======================================
  Coverage        ?   60.36%
=======================================
  Files           ?       39
  Lines           ?     1143
  Branches        ?        0
=======================================
  Hits            ?      690
  Misses          ?      453
  Partials        ?        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 a0f2614...c2606e1. Read the comment docs.


Does Not Work Well With Generic Types

Considers this example:
use impls::impls;
use std::fs::File;

fn impls_clone<T>() -> bool {
    impls!(T: Clone)
}

fn main() {
    // false (wrong!)
    println!("{:?}", impls_clone::<String>());
    // false (right)
    println!("{:?}", impls_clone::<File>());
}

It says that String does not implements Clone, which is wrong (I mean, String does implement Clone). I am not sure if this possible to be fixed though.

This limitation is mentioned in "Generic Types". Without plugging into the compiler, this unfortunately is not possible.


Automate cabling process

Discussed with @KylerBurke and @jinaloo7, we could have a script to automate the creation of cables. The pattern that we agree on is all the 1-2 ports go to efr1, and 3-4 goes to efr2.

We should make sure this script only runs in dev - in production we'll want to have the network techs actually validate and mark cabling


Add info about extended cron syntax

Add a note about using the Fugit::Cron parsing for cron strings.
https://buildkite-docs-review-pr-571.herokuapp.com/docs/pipelines/scheduled-builds
@ticky I saw that you wanted to add tests for Fugit features, did you want me to hold off on merging this docs note until after that's set up?

Honestly I feel like Fugit’s docs are pretty garbage, so this is one case where I’d rather we document it more exhaustively ourselves (and if we ever replace it, hold ourselves to that contract)

Added some more words @harrietgrace, needs further verification!

Epic, thanks @ticky!

needs further verification!

Verification on the code side of things?
Also, 'crontext' 😂👌🏻

Verification on the code side of things?

Yep! I just want to double check my examples are sound. Will get on that today, hopefully!

OK I’ve verified that the examples all work, and added support for the extra heading level this doc is using. If we want to not use that, I’m not sure what the right approach is, but it seems not entirely controversial to cover a further 16% of the heading levels HTML defines!

Oh right! Tests!

Tests added!

@harrietgrace I can't request a review from you because you opened this PR, but could you give this a once-over? I'm happy to ship it if you're happy, and then I'll get the tests through too.

That's fantastic @ticky! I added two little copyedits, and I'm happy to ship 🌈


mutt: 1.9.4 -> 1.9.5

Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg built mutt

Success on x86_64-linux (full log)
Attempted: mutt
Partial log (click to expand)
shrinking /nix/store/r3cz4l47p3zki1a147k2fhm377fc89pz-mutt-1.9.5/bin/pgpring
shrinking /nix/store/r3cz4l47p3zki1a147k2fhm377fc89pz-mutt-1.9.5/bin/pgpewrap
gzipping man pages under /nix/store/r3cz4l47p3zki1a147k2fhm377fc89pz-mutt-1.9.5/share/man/
strip is /nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/r3cz4l47p3zki1a147k2fhm377fc89pz-mutt-1.9.5/bin
patching script interpreter paths in /nix/store/r3cz4l47p3zki1a147k2fhm377fc89pz-mutt-1.9.5
/nix/store/r3cz4l47p3zki1a147k2fhm377fc89pz-mutt-1.9.5/bin/muttbug: interpreter directive changed from "/bin/sh" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/sh"
/nix/store/r3cz4l47p3zki1a147k2fhm377fc89pz-mutt-1.9.5/bin/flea: interpreter directive changed from "/bin/sh" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/sh"
/nix/store/r3cz4l47p3zki1a147k2fhm377fc89pz-mutt-1.9.5/bin/smime_keys: interpreter directive changed from " /usr/bin/perl -w" to "/nix/store/cxdmh98g0lvl1dyq304c1lq7f90dh01f-perl-5.24.3/bin/perl -w"
checking for references to /build in /nix/store/r3cz4l47p3zki1a147k2fhm377fc89pz-mutt-1.9.5...

Success on x86_64-darwin (full log)
Attempted: mutt
Partial log (click to expand)
glibPreFixupPhase
post-installation fixup
gzipping man pages under /nix/store/7l2xn0y8y5f9z7di5qqhs56jhnps1gm0-mutt-1.9.5/share/man/
strip is /nix/store/kdff2gim6417493yha769kh00n63lnrw-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/7l2xn0y8y5f9z7di5qqhs56jhnps1gm0-mutt-1.9.5/bin
patching script interpreter paths in /nix/store/7l2xn0y8y5f9z7di5qqhs56jhnps1gm0-mutt-1.9.5
/nix/store/7l2xn0y8y5f9z7di5qqhs56jhnps1gm0-mutt-1.9.5/bin/flea: interpreter directive changed from "/bin/sh" to "/nix/store/r8bx3qf1bpncb14i9gzma4vr089pc3pv-bash-4.4-p19/bin/sh"
/nix/store/7l2xn0y8y5f9z7di5qqhs56jhnps1gm0-mutt-1.9.5/bin/smime_keys: interpreter directive changed from " /usr/bin/perl -w" to "/nix/store/lizm7q36j5sf5524bw9gsykflv9i2l7z-perl-5.24.3/bin/perl -w"
/nix/store/7l2xn0y8y5f9z7di5qqhs56jhnps1gm0-mutt-1.9.5/bin/muttbug: interpreter directive changed from "/bin/sh" to "/nix/store/r8bx3qf1bpncb14i9gzma4vr089pc3pv-bash-4.4-p19/bin/sh"
/nix/store/7l2xn0y8y5f9z7di5qqhs56jhnps1gm0-mutt-1.9.5

Success on aarch64-linux (full log)
Attempted: mutt
Partial log (click to expand)
shrinking /nix/store/524wj2fqy93hvvi1574zb4xg04w4h16p-mutt-1.9.5/bin/mutt
gzipping man pages under /nix/store/524wj2fqy93hvvi1574zb4xg04w4h16p-mutt-1.9.5/share/man/
strip is /nix/store/j7d4mr0ikv974ig7yzhknpsq288js4bs-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/524wj2fqy93hvvi1574zb4xg04w4h16p-mutt-1.9.5/bin
patching script interpreter paths in /nix/store/524wj2fqy93hvvi1574zb4xg04w4h16p-mutt-1.9.5
/nix/store/524wj2fqy93hvvi1574zb4xg04w4h16p-mutt-1.9.5/bin/smime_keys: interpreter directive changed from " /usr/bin/perl -w" to "/nix/store/3qhzffjvl8gbiabikiqdnz4jhh9v84nz-perl-5.24.3/bin/perl -w"
/nix/store/524wj2fqy93hvvi1574zb4xg04w4h16p-mutt-1.9.5/bin/flea: interpreter directive changed from "/bin/sh" to "/nix/store/adw9jx59wnrh5659wz43nbjya3m4b3gl-bash-4.4-p19/bin/sh"
/nix/store/524wj2fqy93hvvi1574zb4xg04w4h16p-mutt-1.9.5/bin/muttbug: interpreter directive changed from "/bin/sh" to "/nix/store/adw9jx59wnrh5659wz43nbjya3m4b3gl-bash-4.4-p19/bin/sh"
checking for references to /build in /nix/store/524wj2fqy93hvvi1574zb4xg04w4h16p-mutt-1.9.5...
/nix/store/524wj2fqy93hvvi1574zb4xg04w4h16p-mutt-1.9.5

Success on x86_64-darwin (full log)
Attempted: mutt
Partial log (click to expand)
/nix/store/7l2xn0y8y5f9z7di5qqhs56jhnps1gm0-mutt-1.9.5

Success on aarch64-linux (full log)
Attempted: mutt
Partial log (click to expand)
/nix/store/524wj2fqy93hvvi1574zb4xg04w4h16p-mutt-1.9.5

Success on x86_64-linux (full log)
Attempted: mutt
Partial log (click to expand)
/nix/store/r3cz4l47p3zki1a147k2fhm377fc89pz-mutt-1.9.5


event playing even tho EventJSon doesnt exist.

Describe your problem here.
so I made a song in which the character changes once. I put in the change character event with the value one as DAD and the second as the character. of course, it didn't work. then I fixed it by changing the value one to 1. then it worked. I saved it. but each time I restart the game the old event JSON plays. this happened like. one and a half weeks ago. I am trying to fix it since then. I tried everything. redoing the whole data folder, changing the event JSON manually. I tried literary everything, and the event even plays when no event JSON exists. it also appears in the chart editor, but when I press load events the new one pops up which does work. the old event even appeared after I ported it to version 0.5. XD I'm laughing at this point. it's haunting me. the file is no longer on my computer. I don't get it. (it happens both on 64x and 32x)
ever happened to you before? please help--
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No response

events can also play from the song chart

events can also play from the song chart

I'll make sure to check that as soon as I get access to my PC again asap

Events are loaded from two files:
mods/data/your-song-name/events.json
mods/data/your-song-name/your-song-name.json (or whatever difficulty you're in)
Check those two files.


Raspberry pi 4 model B support

Can Nuttx support raspberry 4 model B?
Or anyone work on this board?

@TaiJuWu the initial support to Raspberry Pi board (the first board) was removed by Greg some years ago because nobody was interested to still working in the port: https://bitbucket.org/patacongo/obsoleted/src/master/Patches/Remove-arch-bcm2708-2019-01-02.patch
If you want to work on it, you can look that initial support and use it as reference.

@TaiJuWu the initial support to Raspberry Pi board (the first board) was removed by Greg some years ago because nobody was interested to still working in the port: https://bitbucket.org/patacongo/obsoleted/src/master/Patches/Remove-arch-bcm2708-2019-01-02.patch
If you want to work on it, you can look that initial support and use it as reference.

Thanks for your information.

Checking if this is in development at all? I am interested in contributing to get support for this board.

Checking if this is in development at all? I am interested in contributing to get support for this board.

No, I am not working on it.

Could this issue be assigned to me? I have begun work on it.

Could this issue be assigned to me? I have begun work on it.

Done, thanks.

So I have an initial implementation for this board (the 4GB variant is all I have access to for testing right now) which boots successfully and has NSH running with a UART console. It passes the ostest as well.
Obviously I plan to implement some drivers for the board so it can actually be used for programming/development (I2C, remaining UARTs, GPIO, etc.). I am wondering, however, at what point is a port to a new board considered "good enough" to be merged? I imagine that full support for everything on the board (like HDMI ports, Bluetooth, etc.) does not have to be implemented to be merged, but is there a guideline on how complete it must be at a minimum to become part of the main source tree?
I ask only because I think that at some point in the development process I'd like to be able to call it "complete enough" to allow for it to be merged and more accessible to others to make changes/add features.

it's enough to implement uart and timer driver to ensure the port enter nsh correctly.

Any progress on this issue?

Any progress on this issue?

@csukuangfj I have managed to get NuttX booting into NSH on the 4GB RAM model, haven't been able to test on other models. I have a partially made I2C driver which is quite buggy and is only able to do read operations on the bus. I was starting on SPI aand got quite busy with my coursework. What I'll likely do is open a PR with what I've got working so far and mark features as incomplete on the board docs. Then hopefully the community can pitch in and we can get more features done quicker!


Realsense L515 camera overheating issue and camera node streaming error

Required Info

Camera Model
L515

Firmware Version
<IP_ADDRESS>

Operating System & Version
Ubuntu 22.4

Kernel Version (Linux Only)
5.19.0-41-generic

Platform
PC

SDK Version
2.53.1

Language
Ros2

Segment
{Robot/Smartphone/VR/AR/others }

Issue Description
I have been experiencing two issues with my Realsense L515 camera and camera node:

Overheating issue: The camera gets "feverish" and heats up after running for a while. I suspect this is an issue with the hardware and would appreciate any suggestions on how to deal with this.

Streaming error: When running the realsense2_camera_node, I get the following error messages:

[realsense2_camera_node-1] [INFO] [1683896366.588692908] [camera.camera]: RealSense Node Is Up!
[realsense2_camera_node-1] [WARN] [1683896366.878453617] [camera.camera]:
[realsense2_camera_node-1] [WARN] [1683896366.878490560] [camera.camera]: frame's time domain is HARDWARE_CLOCK. Timestamps may reset periodically.
[realsense2_camera_node-1] [WARN] [1683896369.280320953] [camera.camera]: XXX Hardware Notification:Error that may be overcome in few sec.
[realsense2_camera_node-1] Stream stoped. May be recoverable,1.6839e+12,Error,Hardware Error
[realsense2_camera_node-1] [WARN] [1683896369.280363680] [camera.camera]: Hardware Notification:Error that may be overcome in few sec.
[realsense2_camera_node-1] Stream stoped. May be recoverable,1.6839e+12,Error,Hardware Error

I have no problem to run my code when the stream is not stopped on the start of the day. Additionally, I have written a simple code to display the image from the camera, but the image window is showing black.
Thank you for your assistance.

Hi @iyeszin  Do you have an update about this case that you can provide, please?  Thanks!

Hi @MartyG-RealSense, I had added initial_reset:=true when launching. I had the RealSense Viewer tool. For your information, the camera is able to stream normally without starting the ros node at the same time, however, it show black screen when I run with ros node. I'm currently working on with another camera, I will come back to check the camera when I have time.

Thanks very much, @iyeszin - I look forward to your next report about the L515.  Good luck!

Hi @iyeszin  Do you require further assistance with this case, please?  Thanks!

Case closed due to no further comments received.


[MWPW-133249] Multiple Column Toggle Bar

Please always provide the JIRA issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix MWPW-133249
Description:
A new toggle bar built for homepage v3x
This toggle bar features swappable icons for active & non-active state, a subtle status effect, and remembers the users choice on page reload.
Test URLs:

Before: https://main--express--adobecom.hlx.page/drafts/qiyundai/homepage-v3/toggle-bar?lighthouse=on
After: https://mwpw-133249--express--adobecom.hlx.page/drafts/qiyundai/homepage-v3/toggle-bar?lighthouse=on

Testing note: Because the block surrounding the toggle bar are not currently available in main, to visually test this block, it's recommended to visit https://homepage-v3x-sandbox--express--adobecom.hlx.page/drafts/qiyundai/homepage-v3/sandbox instead of the after link above.

the header is on holiday and hiding behind the marquee :) -
and can we get some unit tests for this toggle-bar as well? At least 50% test coverage would be huge

@vhargrave Sorry, I fell behind on updating the alt text on the individual draft pages. That said, it's probably better to observe the visual aspect of the block on https://homepage-v3x-sandbox--express--adobecom.hlx.page/drafts/qiyundai/homepage-v3/sandbox page as the new/updated surrounding blocks' code is only available on that branch.
Also -- Yes! I'll work on unit tests for all my homepage v3 block next. We are going to aim for > 90%.

the header is on holiday and hiding behind the marquee :) -
and can we get some unit tests for this toggle-bar as well? At least 50% test coverage would be huge

@vhargrave Sorry, I fell behind on updating the alt text on the individual draft pages. That said, it's probably better to observe the visual aspect of the block on https://homepage-v3x-sandbox--express--adobecom.hlx.page/drafts/qiyundai/homepage-v3/sandbox page as the new/updated surrounding blocks' code is only available on that branch.
Also -- Yes! I'll work on unit tests for all my homepage v3 blocks next. We are going to aim for > 90%.

@qiyundai ah ok.. can we close this PR then? Code should only be merged once it's done

the header is on holiday and hiding behind the marquee :) -
and can we get some unit tests for this toggle-bar as well? At least 50% test coverage would be huge

@vhargrave Sorry, I fell behind on updating the alt text on the individual draft pages. That said, it's probably better to observe the visual aspect of the block on https://homepage-v3x-sandbox--express--adobecom.hlx.page/drafts/qiyundai/homepage-v3/sandbox page as the new/updated surrounding blocks' code is only available on that branch.
Also -- Yes! I'll work on unit tests for all my homepage v3 blocks next. We are going to aim for > 90%.

@qiyundai ah ok.. can we close this PR then? Code should only be merged once it's done

circling back to this -- Abhinav wants these ready for authors tomorrow. Do you think it' appropriate to raise separate PRs for unit tests for the homepage v3 blocks just to unblock authors or we need to hold fast on that >50% test coverage requirement?

@qiyundai yes that's ok, but apparently this branch doesn't have the latest version of stuff. The homepage-v3x-sandbox branch does. We probably want to merge that branch instead then right?

@

the header is on holiday and hiding behind the marquee :) -
and can we get some unit tests for this toggle-bar as well? At least 50% test coverage would be huge

@vhargrave Sorry, I fell behind on updating the alt text on the individual draft pages. That said, it's probably better to observe the visual aspect of the block on https://homepage-v3x-sandbox--express--adobecom.hlx.page/drafts/qiyundai/homepage-v3/sandbox page as the new/updated surrounding blocks' code is only available on that branch.
Also -- Yes! I'll work on unit tests for all my homepage v3 blocks next. We are going to aim for > 90%.

@qiyundai ah ok.. can we close this PR then? Code should only be merged once it's done

circling back to this -- Abhinav wants these ready for authors tomorrow. Do you think it' appropriate to raise separate PRs for unit tests for the homepage v3 blocks just to unblock authors or we need to hold fast on that >50% test coverage requirement?

@qiyundai yes that's ok, but apparently this branch doesn't have the latest version of stuff. The homepage-v3x-sandbox branch does. We probably want to merge that branch instead then right?

@vhargrave Oh I see the confusion I've caused -- this branch has the newest code for the toggle bar, but doesn't have the code of the other homepage v3x blocks. Sandbox has everything but with a very messy commit history. I've basically been updating feature branches, and merging things onto sandbox branch to have the full page preview.

@qiyundai ah ok.. got it. Yeah..let's get this merged then!


Translations

Hi Everyone.
I have added a translation template, so we can share Jexcel translations with other users.
If you wish to collaborate, please fork, add a new file to the lang folder and do the PR.
Example
Thanks.

https://github.com/paulhodel/jexcel/pull/984
Hi Paul, here is a translation of Chinese(simplified) version. Thanks for your great project.

Hi, here file for French of France
#987


Downloading dataset from OneDrive

Hello!
Thanks for great work! I am currently trying to download a dataset from OneDrive (can not use Baidu as far as I'm not living in China). I have tried to generate link from network using method provided here to download dataset directly to my remote server. But loading always interrupts, and I get only 15-17 GBs from 180. Manual downloading is challenging too, because a little network issue ruins all loading.
You provided a script for downloading the dataset from AWS here but later you deprecated it. If there is some link to it, could you, please, share it?
Could you, please, provide some script/method for automatic downloading from OneDrive or maybe you have some S3 link?

Sorry, we no longer maintain AWS due to cost issues.
For the convenience of downloading, we have divided the dataset into fragments of about 1G.
You can consider downloading in batches one by one, without worrying about network fluctuations causing all re-downloads.


Query on Dependency Resolution and Code Suggestions/Completion

I'm currently using the Groovy Language Server in a project and am quite new to the entire concept of Language Server Protocols. I want to understand more about its features and capabilities, especially around dependency resolution and code suggestions/completion.
For my requirement, I need the language server to provide code suggestions and completion from dependencies that could be from other Java/Groovy projects/JARs. From my understanding so far, I couldn't figure out if the Groovy Language Server picks up these dependencies or how to make it do so.
Could you please provide some guidance on whether this is currently possible with the Groovy Language Server? If not, might there be any plans to include such feature in the future? This feature would definitely enhance my development experience as it allows the server to cater to more complex projects with external dependencies.
Thank you for your assistance

There is a groovy.classpath setting that you may use to add jar files, or directories containing jar files.

Thank you for the prompt response.
I have successfully integrated the Groovy Language Server with the Monaco Editor using monaco-languageclient. However, I'm struggling to figure out how to configure the groovy.classpath setting through Monaco Editor and monaco-languageclient.
Can you provide some guidance on how I can pass this configuration to the Groovy Language Server via Monaco Editor? This would greatly assist in making full use of the features provided by the language server within my editor setup.
Thank you once again for your invaluable assistance.

I have never directly used the Monaco Editor or monaco-languageclient before, so unfortunately, I cannot provide guidance for how to use it.
In language server terminology, settings are sometimes called "configuration options". Perhaps that will help a bit.

Thank you for your prompt response. I appreciate your clarification regarding language server terminology and the suggestion to consider "configuration options" in the context of the Monaco Editor integration.
I understand that you haven't directly used the Monaco Editor or monaco-languageclient before. I'll take your advice and delve deeper into the documentation and resources available for both tools. I'm sure I'll find valuable insights that will help me in integrating the Monaco Editor with the Groovy LSP server.
If I encounter any specific challenges or have further questions, I'll make sure to reach out.


chore: upgrade to vite 5

I’m putting this up to see if we can migrate to Vite 5 already. At a first glance, this seems possible so I now want to see whether the CI interactions work, too.
Changes
Upgrades vite from v4 to v5 (see https://vitejs.dev/guide/migration.html#migration-from-v4).
Sets pkg.type to module.
Changes the extension of the files dev-utilities/deprecatedUtilityClasses.js, .eslintrc.js, and stylelint.config.js to .cjs as they’re CommonJS modules and can’t be converted to ESM.
Changes the module format of the file docgen.config.js to ESM.
Removes vite-plugin-rewrite-all as it’s no longer needed.
Signed-off-by: Philipp Rudloff<EMAIL_ADDRESS>Resolves #1791.
Notes

During yarn run build, we’re seeing some deprecation warnings through vite-plugin-html (see "Blockers" in #1791).

@kleinfreund could you let me know what this upgrade is giving us, apart from it being the new major

Seeing as Vite is our primary tool for all things development, we should really try and stick to its latest versions. Over time, we’ll be more and more locked out of fixes and security updates that we want or need.

Right, but seeing as its only on 5.0.2 and the public site doesn't yet have docs for v5, it could also give us unforeseen breakages and security bugs, plus hard to find docs and probably some other things.
I'm not saying lets not ever upgrade, but I'm just wondering jumping on the latest thing might cause more issues just for lack of being a little more patient.
I'm not totally sure about vites support, but I would also guess in the immediate future if anything was to be amiss with the far more stable and "battle tested" v4 which we are currently on, it would be released as a bugfix for v4. Again not saying we should stay on on v4 forever.

So, could you let me know what this upgrade is giving us, apart from it being the new major?

and the public site doesn't yet have docs for v5,

What makes you say that? The website seems to reflect v5 for me.

😆 there's no v5 in the Version menu

So, could you let me know what this upgrade is giving us, apart from it being the new major?

So here's how I would have answered my own question:

So, could you let me know what this upgrade is giving us, apart from it being the new major?

Yeah sure:

I mentioned in the description that the upgrade would mean we can remove vite-plugin-rewrite-all , Vite fixed the issue they had with . characters in URL paths which means we won't need the code here https://github.com/kumahq/kuma-gui/blob/6b453886e8eec507c171134429f5654b01b6865d/vite.config.production.ts#L30-L35
...
...
...

Just so we don't go back there.

I agree that updating software means the software you are using doesn't eventually become outdated. If you can explain to me what else this upgrade is giving us there might be more we can agree on.
Yes, I saw that the site has the version 5 docs by default, I guess must be stupid and immediately looked in the Version menu to see what versions were in there, I found it funny 🤷 . Thanks for pointing that out.

So before this starts sounding like a backwards Monty Python sketch again:

So, could you let me know what this upgrade is giving us, apart from it being the new major?

Let's reconsider merging this in January


[RFR] not is an operator, not a function

Tiny minor change. Cheers. :)

I think this was a holdover from an older version of Libsass, but I'm guessing the issue was resolved.


Add files via upload

@ChrisMi1 uniwa.txt I am sorry, I have to close the request without merging, because you made a mistake in a request and did not specify any filepath. Please ensure that you are familiar with this repository rules located at the bottom of this page and make a new correct request. Thank you for your understanding!


Blog Post: Eighth Wave of Nostr Grants

Build Preview:

/blog/8th-wave-nostr-grants

Social Preview:

@OpenSats/volunteers team, would some one mind giving it one more quick read as a sanity check for any last minute adjustments? 🙏

It says 4 projects but 5 are listed. Is that meant to be?

It says 4 projects but 5 are listed. Is that meant to be?

Fixed 🙏

@dergigi ready for a final review, and then merge. 🫡

@Arvin21M can you do a search for "four" and replace it with "five"? You use it a couple of other times in the blog!

@Arvin21M can you do a search for "four" and replace it with "five"? You use it a couple of other times in the blog!

Fixed... again 😆
Feel free to make suggested edits directly into the code file. You should be able to from your end next time, and I will just press a button to accept the change on my side. Here is a helpful how-to on this.
🙏

Looks great .. let that puppy bark out load!

Thanks everyone for writing/editing/reviewing this one! Appreciate it 🙏🧡
Live now:

https://opensats.org/blog/8th-wave-nostr-grants
nostr post


chore(nuget-sign): Use osslsigncode for now

We cannot use mono's signcode tool until a release is available that
supports SHA256 signatures, so we'll be using osslsigncode in interim.
A patch has been submitted to mono to add support for SHA256 signatures
should be making it to a release "soon".
Related: mono/mono#11999

A previous iteration of this code was already see in awslabs/cdk-ops@844181b86893436661f75c4be12baf312d03a32a

This was tested by running nom test update and ensuring that the signature log outputs looked sane.

Please run the delivlib integration test and make sure publishing with signing actually works.

@eladb
$ osslsigncode verify /Users/rmuller/Downloads/eladb.jsiisample.1.5.0.nupkg/lib/netstandard2.0/Eladb.JsiiSample.dll
Current PE checksum   : 000105A7
Calculated PE checksum: 000105A7

Message digest algorithm  : SHA256
Current message digest    : 79AA8FCB5AAD4B4526354EAE3F6C42DDCDA903FA0ADC1594729ADA465BAE0F68
Calculated message digest : 79AA8FCB5AAD4B4526354EAE3F6C42DDCDA903FA0ADC1594729ADA465BAE0F68

Signature verification: ok

Number of signers: 1
	Signer #0:
		Subject: /CN=delivlib-test/C=IL/ST=Ztate/L=Zity/O=Amazon<EMAIL_ADDRESS>		Issuer : /CN=delivlib-test/C=IL/ST=Ztate/L=Zity/O=Amazon<EMAIL_ADDRESS>
Number of certificates: 3
	Cert #0:
		Subject: /CN=delivlib-test/C=IL/ST=Ztate/L=Zity/O=Amazon<EMAIL_ADDRESS>		Issuer : /CN=delivlib-test/C=IL/ST=Ztate/L=Zity/O=Amazon<EMAIL_ADDRESS>	Cert #1:
		Subject: /C=US/O=DigiCert Inc/OU=www.digicert.com/CN=DigiCert Assured ID CA-1
		Issuer : /C=US/O=DigiCert Inc/OU=www.digicert.com/CN=DigiCert Assured ID Root CA
	Cert #2:
		Subject: /C=US/O=DigiCert/CN=DigiCert Timestamp Responder
		Issuer : /C=US/O=DigiCert Inc/OU=www.digicert.com/CN=DigiCert Assured ID CA-1

Succeeded


[QUESTION] Use especific word to initialize commands?

It is possible?
Define one word to initialize commands.
var word_init_comma = "orange"; // ok... sorry my poor example :P

This word possible uses: app name, website name, ...
artyom.addCommands([
{
    indexes: [word_init_comma + " open contacts"],
    action: function(i){
        console.log("function for open contacts html fragment!");
    }
},
{
    indexes: [word_init_comma + " send message"],
    action: function(i){
        console.log("function to send message");
    }
}
]);

// or use in source property (if possible add personality command constructor)
artyom.initialize({
  lang:"en-US",
  continuous:true,
  debug:false, // Show what recognizes in the Console
  listen:true, // Start listening after this
  speed:0.9, // Talk a little bit slow
  //mode:"normal", // This parameter is not required as it will be normal by default
  word_init_comma: "orange"
});

// usage (by voice command) like:  orange open contacts

Technically, this seems to work in the way you've described. You just need to declare a phrase at the beginning of every command, as you've done. I tested that by naming my system "Winston." Commands without my "word_init_comma" didn't work. Commands with it did.

I've been rewriting Artyom.js in TypeScript again, because the current model forces me to work with 2 files namely the VanillaJS and the TypeScript version. In this case it will be worked on a single TypeScript file and it can be transpiled to a version that works in the Browser (window) and with modules (require, import etc).
Your proposal works indeed, however it makes your code less dynamic. The solution that i'm providing is the new property name at the initialization object. If this name was provided, then the commands will be executed only if the name is spoken too.
// Now artyom can be initialized with different instances
var artyom = new Artyom();

artyom.addCommands([
    {
        // Triggered only when you say "Jarvis Good Morning"
        indexes: ["Good Morning"],
        action: () => {
            MyAssistant.say("Hello, nice to hear you !");
        }
    }
]);

artyom.initialize({
    // Initialize with some default setup
    lang: "en-GB",
    debug: true,
    listen: true,
    continuous: true,
    // Give a name to your assistant !
    name: "Jarvis"
});

In this case, we have a "Good Morning" command. As the name "Jarvis" was providen at the initialization, the user would need to trigger the "Good Morning" command by saying "Jarvis Good Morning".
This feature will be available with the new version 1.0.6 that should be published in this week (maybe).

Yes, this seems to be more succinct using in the constructor.
Waiting for a new release

Well, the usage of the constructor isn't meant to provide the initialization object but just for create multiple instances of Artyom. This can be really useful when you work with bilingual apps e.g:
// Note: obviously only 1 instance of speech recognition can run at time
// otherwise the browser may crash.

var JarvisSpanish = new Artyom();

JarvisSpanish.addCommands({
    indexes: ["Hola"],
    action: () => {
        JarvisSpanish.say("Hola, todo bien?");
    }
});

JarvisSpanish.initialize({
    listen: true,
    debug: true,
    lang: "es-ES",
    continuous:true
});

// And english instance

var JarvisEnglish = new Artyom();

JarvisEnglish.addCommands({
    indexes: ["Hello"],
    action: () => {
        JarvisEnglish.say("Hello, everything in order?");
    }
});

JarvisEnglish.initialize({
    listen:true,
    debug:true,
    lang: "en-GB",
    continuous:true
});

The new version 1.0.6 is now available 🤓

Thanks man, its is beathfull!


RegexMatcherError

Hey, the 'getThrottlingFunctionCode(String js) function in Cipher class throwing an RegexMatcherError exception, can you fix it?

Hi, I sent the correction, the changes are available here: https://github.com/felipeucelli/JavaTube/commit/1fbe8065f535592fafbc1489b03436eb72cf3e4a. Thanks for reporting.

thank you very much for your support.

;a.D&&(b="nn".?&&([abc]=(?[a-zA-Z$]{3})[(?\d{1})].?a.set(b.[abc]))}};
Shouldn't change the "}}" to "}}" at the end of this regular expression?
Your code works without issues, but on Android, the following error occurs.
"java.util.regex.PatternSyntaxException: Syntax error in regexp pattern near index 89
;a.D&&(b="nn".?&&([abc]=(?[a-zA-Z$]{3})[(?\d{1})].?a.set(b.[abc]))}};"

You are right, some versions of the android jdk need this escape character, I added the fix https://github.com/felipeucelli/JavaTube/commit/60c0be65370dab5f3fb6cbbefe5f85041be8b837, also available on jitpack https://jitpack.io/#felipeucelli/javatube


Can I override just one table header and not all?

Hi,
Can I override just one table header and not all? I want a sortable table with a custom first column (a fancy checkbox first column). Right now all the logic in the sorting is in tbody > tr. I do not have thead > th and this is how my code looks like:

And now my code looks like this

Name

I did so because I followed this example: http://bazalt-cms.com/ng-table/example/18
Is there any way I can have a simple default sortable column (like I used to have in my first example) and have custom stuff only in the first one? I have lots of columns and the boilerplate sortable stuff complicates my code a lot and it is not DRY.
A workaround is to make my own custom directive for the header but I was hoping there is a simpler solution.
Thank you!

Yes you can.
Create a simple custom template like so:

Then define your special column like this:
...

... instead of this:
...

Thank you @Jeremy-Walton! I did not made a full implementation yet, just a small example and it seems to work :P I will take a look more at how ng-table is using it https://github.com/esvit/ng-table/search?utf8=✓&q=type%3D"text%2Fng-template"

How can I pass a variable inside? I have something like this:

Anything you put in the header will be executed in the same scope as the rest of your table, outside of the body loop. See https://github.com/esvit/ng-table/blob/master/src/ng-table/header.html#L16 and https://docs.angularjs.org/api/ng/directive/ngInclude.
Instead of using this.whatever, you should use whatever.

Thank you!
For posterity :P  This worked for me:

Yes you can.
Create a simple custom template like so:

Then define your special column like this:
...

... instead of this:
...

Wonderful. Still useful!!!


LibWeb: Fix handling of SSEs split across chunks

Should fix #950

Can we add a test for this. Also a bit more explicit in the commit description about what this actually fixes would be nice

Can we add a test for this

We unfortunately do not have the infrastructure to test SSE in our unit tests. They require running an HTTP server in the background.


Make Delete take data, not an id.

delete: function(id, cb) {
    return table.findOneAndDelete({id: id}, cb);
}

Could this be changed to the following?
delete: function(data, cb) {
    return table.findOneAndDelete(data, cb);
}

That way we can use it in the following manner:
controller.storage.users.delete({ something: { $lt: 30 } }).catch((error) => {
    console.log(error);
});

Thanks for all the great work!

Done. PR #33


JBPM-5530 - Review package name and GWT modules name for human-tasks…

… module
Renamed packages from org.jbpm.console.ng to org.jbpm.workbench
Updated GWT module name

Jenkins please retest this.

Jenkins please retest this.


sql.NullString is used when field is filtered with is not null

Version
1.21.0
What happened?
I'm unsure if this is a bug or maybe a feature request.
When crafting a query, filtering with a WHERE <field> IS NOT NULL, I'd (maybe ?) expect this to be returning string not an sql.NullString.
I've used v1.20.0 in the playground as v1.21.0 is returning an HTTP 500 error.
Relevant log output
No response
Database schema
No response
SQL queries
No response
Configuration
No response
Playground URL
https://play.sqlc.dev/p/616d691ebd54faab288c774e1b41a588cc2d4328ebe9b84b880dbecb72fa1223
What operating system are you using?
Linux
What database engines are you using?
SQLite
What type of code are you generating?
Go

This is a feature request

Thanks for the feature request! This proposal reminds me of narrowing from TypeScript. Sadly, this isn't something we can easily support right now in sqlc, as our type inference is primitive. We'd need to add a significant amount of code to ensure that the inference worked for many different types of expressions.


[BUG] Popup Menu offset is wrong in a 21:9 monitor

Version: v3.6.0
Godot: 4.1.1

Hello.
Well... This is new.
I try to check this out by myself and see if that's the case still or not at the first free time that I can find.
Thank you for making an issue when there is an odd behavior.

Hello again.
My friend, it seems fine here.
I can not replicate the bug in my own Laptop so...
If you can fix it up or at least tell me how to generate the bug, that would be helpful.

One year has passed, I'm just gonna close this.


Productattributes

Tasks:

[x] done
[ ] todo
[ ] got feedback

Q
A

Bug fix?
no

New feature?
yes

Tests pass?
yes

Fixed tickets
https://github.com/massiveart/POOL-ALPIN/issues/800

Doc
none

Please check travis-error log:
.................PHP Fatal error:  Cannot use Sulu\Bundle\ProductBundle\Entity\AttributeType as AttributeType because the name is already in use in /home/travis/build/sulu-io/SuluProductBundle/Api/ProductAttribute.php on line 19


Add gRPC Server Reflection support

Is your feature request related to a problem? Please describe.
gRPC and connect-go support a server-side reflection API that serves protobuf schema information to clients. As of now, connect-node does not support this feature.
This feature would be useful as various gRPC debugging utilities can use it to grab schema information on-demand.
Describe the solution you'd like
Ideally, connect-node would have an optional implementation of serverside reflection, including the necessary code generation in protobuf-es to support this (as needed.)
Additional context
gRPC Server Reflection Protocol
connect-go grpcreflect implementation

+1 on this one, let's hope we get some more feature parity between the Node and Go ConnectRPC libraries. Still very happy with the current Node implementation, this would just make it more complete. Thank you to the contributors and the ConnectRPC team as a whole.

+1 this would greatly improve integration into things like postman etc... which many of our dev use on their current REST apis.

Any updates on this?

+1

+1

https://github.com/protoflow-labs/connect-grpcreflect-es


I can't install it in mac

Thank you for your docker
but I am getting error message after I am running (on Mac)
docker-compose up android-studio-stable
Error Message:
{create .: "." includes invalid characters for a local volume name, only "[a-zA-Z0-9][a-zA-Z0-9_.-]" are allowed
ERROR: Encountered errors while bringing up the project.
}

Which docker-compose version you used?

Thank you for your reply
Docker version 1.12.1, build 6f9534c

To check your compose version by the following command: docker-compose -v

I think mac is not supported $XAUTHORITY and $DISPLAY variables. Need to find out a way for mac.

Thank you, I will let you know if I find anything


Proposal: sortable manifests (templates)

This is a proposal to solve issues such as #2994. I wanted to open this in a separate issue so that this specific implementation could be discussed and potentially declined separately from the problem report.
The proposal is to:

Add a helm.sh/order-weight annotation to the first object in a manifest (a.k.a template) file. The annotation would sort the files within a chart.
An alternative here would be a comment header at the top of the file (e.g. # weight: 123)
Add a weight attribute to each dependency in requirements.yaml. The attribute would sort the charts as whole units.
An alternative here would be a sort: true in the top level of requirements.yaml, which would enable sorting the dependencies in the listed order.

In the event that a subchart is required by multiple charts, the highest level chart's requirements sets the weight of that chart.
Sorting precedence would then become:
(kind sorting) -> (annotation sorting) -> (subchart sorting) -> (chart sorting)
Weight would be a signed integer, with a default of 0. Thus all existing charts would be sorted the exact same way since kind sorting would not be overridden.

I'd be willing to work on this as this is currently a major blocker for us since we're using object types that helm is not aware of, and thus it applies them in the wrong order.
However I wanted to get approval that this would be accepted before working on it because the amount of work involved would be moderately sized. Right now manifests are sorted without regard to the chart they came from, and that context is not available at sort time. So the code would have to be restructured a bit.

This is definitely an interesting proposal. I'd be interested in @technosophos' or @adamreese's thoughts on this. We could discuss this in one of the dev calls as well. This next week's call is focused on Helm 3, but we could slot it in after that as well

One more clarification question here, is this purely to solve the CRD use case or is there another use case it would solve for you?

One more clarification question here, is this purely to solve the CRD use case or is there another use case it would solve for you?

Our use case is actually around openshift. Since helm doesn't recognize any of the openshift resource types, it treats them all as unknown and puts them last. Same thing as the CRD though.
I think there's also another more advanced use case that I ran into the other day with ordering between the charts (a resource in one chart depended on a resource in another chart), but I can't recall what that was.

Hello. Has there been any progress with this proposal or is there an alternative approach. We are particularly interested in the weighting/ordering of the charts in the requirements.yaml.

The short answer is that the core maintainers are not likely to implement this feature ourselves because we are working on Helm 3. But if someone does a PR with this feature (and it doesn't break compatibility), it is likely to get merged.

closing as inactive.


FIX: May-24 update in coinbase auth

Before submitting this PR, please make sure:

[x] Your code builds clean without any errors or warnings
[x] You are using approved title ("feat/", "fix/", "docs/", "refactor/")

A description of the changes proposed in the pull request:
On May 24, Coinbase updated their authentication SDK 1.4.1:
https://github.com/coinbase/coinbase-advanced-py/commit/ca1f934bdf002f527d81b927a394c1582e526153#diff-06572a96a58dc510037d5efa622f9bec8519bc1beab13c9f251e97e657a9d4ed
Note, this is not yet posted on their official ChangeLog page:
https://docs.cdp.coinbase.com/advanced-trade/docs/changelog/
Tests performed by the developer:
Tips for QA testing:

hey @MementoRC this PR is the one that solves the auth issue in coinbase?

Another user found that updating the JWT pypi package solved the issue. My opinion is that this PR should be the one that fixes the issue because it requests the installation of the latest coinbase-advanced-py conda package (which should require the correct jwt to be installed). But that was not the observation of @ajjcop or @rapcmia, possibly because pypi uninstalled the conda correct version in the installation, just a speculation
So the pypi update was PR'd and already merged

Obsolete. This proposed to remove the internal JWT (copy of SDK but using the time-sync), which was not the issue with the auth failures.


Error in building docker

I tried to follow the instructions here: http://invenio.readthedocs.org/en/latest/installation/installation-quick.html
And it had an error when I was running:
docker build -t invenio:2.0 .
Here's the error:
---> Running in<PHONE_NUMBER>bd
Command "/usr/local/bin/python2 -c "import setuptools, tokenize;file='/tmp/pip-build-ociO2Y/Pillow/setup.py';exec(compile(getattr(tokenize, 'open', open)(file).read().replace('\r\n', '\n'), file, 'exec'))" install --record /tmp/pip-MncOSU-record/install-record.txt --single-version-externally-managed --compile" failed with error code 1 in /tmp/pip-build-ociO2Y/Pillow
The command '/bin/sh -c python requirements.py --extras=$REXTRAS --level=min > requirements.py.lowest.txt &&     python requirements.py --extras=$REXTRAS --level=pypi > requirements.py.release.txt &&     python requirements.py --extras=$REXTRAS --level=dev > requirements.py.devel.txt &&     pip install -r requirements.py.$REQUIREMENTS.txt --allow-all-external --quiet' returned a non-zero code: 1
Thanks for your help.

error code 1 in /tmp/pip-build-ociO2Y/Pillow

Are you well on the latest maint-2.0 branch?  I.e. do you have the following patch?

50373f9a78f53456a90a528628340d8bd5c6f966

(With this patch, I verified about a week ago that the build process succeeds.)
BTW this kind of problems may also appear due to network connection troubles, such as connected to docker process loosing connection due to laptop's suspend/resume, for example.  You may debug the problem by editing your Dockerfile and removing the --quiet mode. This will print the detailed information on what went wrong.

Yes, that patch wasn't applied to my copy of maint-2.0.  Had to do git pull to make it work.  It's working now. Thanks.


installation problem

Hi,
I am trying to install SeqPrep in my Dell server. But It is giving error as
gcc  -c -Wall -Werror -O0 -g SeqPrep.c -o SeqPrep.o
SeqPrep.c: In function ‘main’:
SeqPrep.c:136:8: error: variable ‘print_overhang’ set but not used [-Werror=unused-but-set-variable]
cc1: all warnings being treated as errors
make: *** [SeqPrep.o] Error 1
Any suggestion to overcome this situaion.

but ... how about this one
cc  -c -Wall -O0 -g -std=c99 SeqPrep.c -o SeqPrep.o
In file included from SeqPrep.c:10:0:
utils.h:7:18: fatal error: zlib.h: No such file or directory
#include <zlib.h>
^
compilation terminated.
make: *** [SeqPrep.o] Error 1

Looks like you need to install libz on your system.
On Sun, Aug 27, 2017 at 1:20 AM SuperPz<EMAIL_ADDRESS>wrote:

but ... how about this one
cc -c -Wall -O0 -g -std=c99 SeqPrep.c -o SeqPrep.o
In file included from SeqPrep.c:10:0:
utils.h:7:18: fatal error: zlib.h: No such file or directory
#include <zlib.h>
^
compilation terminated.
make: *** [SeqPrep.o] Error 1
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jstjohn/SeqPrep/issues/10#issuecomment-325184232, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAcBBgZwu3ZQFIffAP7PP7wI_-MoZIxtks5scSbpgaJpZM4AmnnW
.

Hi!
Just wondering if I can just ask the same problem like them above in this thread.
How about this -->
cc  -c -Wall -O0 -g -std=c99 SeqPrep.c -o SeqPrep.o
process_begin: CreateProcess(NULL, cc -c -Wall -O0 -g -std=c99 SeqPrep.c -o SeqPrep.o, ...) failed.
make (e=2): The system cannot find the file specified.
make: *** [SeqPrep.o] Error 2
And does Error 1 and Error 2 differ?
Thank you for your time!


AI breaks the rules

AI often seems to be able to make impossible moves.  I'm new to the game but I'm pretty sure it can't make that "knight-move" shape from the cards available.

I've also had it claim a win when I moved through one of its pieces.

@PeekabooSteam

AI often seems to be able to make impossible moves. I'm new to the game but I'm pretty sure it can't make that "knight-move" shape from the cards available.

The card it used to move is the one now to the side of the board. We can't see what card it used in your screenshot so we can't be sure if the move was illegal or not. What were the other cards in the game? Did any allow for that move?

I've also had it claim a win when I moved through one of its pieces.

Pieces in this game do not slide like in Chess. They simply move from their origin square to the target square on the move card they chose. Did it capture your Master after your move or move its own Master into your Shrine?
You can find the complete rules in this PDF.


[DISCUSS] How do we reorganize the homepage for easier feature discovery

right now it's not obvious what is available, especially lots of the code are in scripts. as a starter, here are some ideas:

rename scripts to Toolkits, and list them in the home page
split the scripts into their individual pages instead of one unified page

"Toolkit" may be confusing with Gluon nlp toolkit. But sure it is better to have a hierarchical structure.

Let's keep the "Scripts" name then. We definitely need nicer page layouts for them.


wasm js bindings generated by wasm-pack is not compatible with raw browser import

This is a duplicate of the following issue on dojo.js repo https://github.com/dojoengine/dojo.js/issues/268
But I thought it might be best to post it here too as the dojo.c readme also indicate the use of wasm-pack for generating the wasm and js binding.
Issue
To re-iterate the issue is that  wasm-pack generate a js file that load wasm via es import import * as wasm from "./<lib>.wasm" which is not natively supported by browsers.
Some bundler will magically transform such import so browser can import it, some other (like vite) requires a community plugin to make it work. But ideally dojoengine should work without such external tool and as such it would be great if the tool used would generate  js boilerplate compatible with browser (without requiring any other toool)
It seems using the wasm-bindgen cli to generate the js binding generate the correct js so it can be loaded in browser without any extra bundler
Reproduction Steps
generate import map for torii-client and its dependencies
set them up in the index.html and add a script that load the torii-client and you ll'see the following error:
Failed to load module script: Expected a JavaScript module script but the server responded with a MIME type of "application/wasm". Strict MIME type checking is enforced for module scripts per HTML spec.

see reproduction repo : https://github.com/bug-reproduction/torii-wasm-in-browser
see codepen : https://codepen.io/wighawag/pen/RwzddBM

That issue is dojo.js specific. If you want to use dojo wasm bindings natively within the browser, you can use the script in scripts/build_wasm.sh. This will compile the wasm with the "no-modules" target which doesn't depend on a bundler

no-modules is less than ideal, es module should work in modern browser.. I ll closed this anyway as this is indeed a dojo.js issue and I link your answer here : https://github.com/dojoengine/dojo.js/issues/268#issuecomment-2352859743
(though, you should probably update the README to make note of this or update with a README with instruction for direct browser import)

no-modules is less than ideal, es module should work in modern browser.. I ll closed this anyway as this is indeed a dojo.js issue and I link your answer here : dojoengine/dojo.js#268 (comment)
(though, you should probably update the README to make note of this or update with a README with instruction for direct browser import)

You can take a look at this page that lists all of the targets that wasm-pack supports: https://rustwasm.github.io/docs/wasm-pack/commands/build.html#target
Web / no-modules should both work. In my case I suggested no-modules because it is the most barebones target and will work in almost any environment. This is for eg. what we do for Unity, where we load and initialize our WASM directly within the template HTML.

I got you, I know of the options for wasm-pack, I reported the issue there too by the way : https://github.com/rustwasm/wasm-pack/issues/1432
But for my setup I need es module so we can make use of library that import torii-wasm  that way.
Actually the library I import is torii-client which uses es import to import torii-wasm and as such will not work with the js generateds by no-modules


Coder.Coder version 0.23.6

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

🤖 cc: @deansheather @matifali

This can be closed, see https://github.com/microsoft/winget-pkgs/pull/108048


Issue using sound samples from memory that are 32-bits per sample

Details: https://github.com/openfl/lime/issues/307
We may also need to work out code to try and figure out the audio type without headers?

there are also none of parameters except bytes are used. Such as sampleRate, stereo, etc. How can you detect from PCM data if it's 44100 Hz 16 bit stereo or 44100 Hz 32 bit mono?


Fix intelli-sense enter not working b/c insert next is undefined

Fixes: https://github.com/microsoft/pxt-minecraft/issues/1711

Im assuming fixes https://github.com/microsoft/pxt-minecraft/issues/1705 too? Looks like same issue

Im assuming fixes microsoft/pxt-minecraft#1705 too? Looks like same issue

Yes, my issue was dupe
Fixes: https://github.com/microsoft/pxt-minecraft/issues/1705


Variable Transaction Tracking

Describe the bug
I faced this bug when trying to navigate to the Detach section where the plugin would freeze right before opening the Detach modal. I tried to reproduce it but this time with the session recording enabled and the bug was present. Crash report analysis from @SorsOps:
Unable to find a variable with a specific key prior to the detach action.
•	The missing variable is causing a freeze.
•	Observation of a long transaction record.
•	Currently tracking update tokens, but not the detach action.
•	The actual DOM transactions complete quickly.
•	Lack of tracking for figma variable logic.
•	Gradual increase in memory usage during the freeze.
To Reproduce
Steps to reproduce the behavior:

Go to 'Manage Themes'
Click on Arrow to 'Detach styles & variables'
Click on 'Detach styles & variables' tab
See error - plugin freezes and you can't click on anything for a couple of minutes

Screenshots or Screencasts
Observation of a long transaction record:

Gradual increase in memory usage during the freeze.

Unable to reproduce, closing - let's find a reproducable example


Print version when library is loaded

We should print the version when the library is loaded, so that we can trace versions all the way back.
As noted #530.

Done in #543.


Add the ability to flag channel as private when opening via lnurl channel

Add the ability to flag channel as private when opening via lnurl channel
Description
When opening a channel there is no option to flag the channel as private so the channel opens as public
Context
Our users are using LND running on mobile devices which do not have public static ips, so all channels need to be private so route hints can be added. When using a service like bitrefills thor it is not possible to open the channel as private.

Possible Implementation
A check box to flag as private when viewing the open channel confirmation screen, or an option in settings to make all channels open as private

Just wondered if this request has been seen and/or considered? thanks!

Was doing a lot of other PRs lately. This will be one of the next, thanks for bringing up the issue.

shipped with v0.4.0-beta


Migrate to Gradle Version Catalog.

@donbeave thanks a lot for the PR.

Thanks again for the contribution @donbeave!
Superseded by https://github.com/micronaut-projects/micronaut-acme/pull/233


exception

hi。
When I insert and remove the USB flash drive, I occasionally encounter the following error: W/system.err: java.io. IOException: Could not write to device,result == -1 errno 71 Protocol Error？

This means that libaums/Android would have liked to communicate with the device but you removed it to soon.


Installing from template is broken

What version of starlight are you using?
0.14.0
What version of astro are you using?
4.0.2
What package manager are you using?
npm
What operating system are you using?
Mac
What browser are you using?
Chrome
Describe the Bug
The command npm create astro@latest -- --template starlight is not working.
Steps to reproduce:

Create a new project using starlight template as instructed on the Getting Started tutorial.
Go through the steps
Observe it fail

Here's how it looks like:

If you go to the directory created (successfully) and just run npm install it gives this error:

Here's the package.json file:
 {
  "name": "confraria.tech",
  "type": "module",
  "version": "0.0.1",
  "scripts": {
    "dev": "astro dev",
    "start": "astro dev",
    "build": "astro build",
    "preview": "astro preview",
    "astro": "astro"
  },
  "dependencies": {
    "@astrojs/starlight": "^0.14.0",
    "astro": "^4.0.1",
    "sharp": "^0.32.5"
  }
}

This is likely caused by the fact the template is trying to fetch the latest release.
Link to Minimal Reproducible Example
https://stackblitz.com/github/withastro/starlight/tree/main/examples/basics?file=README.md
Participation

[ ] I am willing to submit a pull request for this issue.

Just merged our release PR (#1240) so Starlight v0.15 with Astro 4 support should be released any second now to fix this! Thanks again for reporting and apologies for the roadbump.


No data available error

When running the command to generate. I am getting a "Error! No data available" message.

issue fixed, had the coordinates in the wrong order, grabbed from the URL instead of the bbox box coords in the selection ui


Some urls don't work.

This library is working quite good in many cases. It's snappy and doesn't seem to consume many resources. It works good on recyclerviews as well.
But, some urls, even popular ones such as www.google.com does not work. The return results are all 'null' and the error logs  says that
'Unable to resolve host "xn--()-3y0jv63bvsjymf": No address associated with hostname'
Is there a workaround for this?

Hello @June6-android
Thanks for the observation
Some URLs do not follow the protocol, and the response that you get can be due to the referrer or the agent that is used in the OpenGraphProtocol.
This will be looked into shortly, meanwhile, any contribution from your side is also welcome.


chore: add node 20 test

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add support for 65C02 opcodes.

This adds support for the base set of 65C02 opcodes, fixing #1261.
The documentation for the various opcodes used to be at https://wdc65xx.com/Programming-Manual/ but the newest version was removed from the website and it is now not available for free anymore.  However, there is a copy at http://archive.6502.org/datasheets/wdc_65816_programming_manual.pdf which is an older version and (probably) free to distribute and/or link to.  The manual page numbers are based on the archive.6502.org's version.
~Newer revisions of the 65C02 and its clones adopted opcodes from Rockwell's variant (namely BBS/BBR/RMB/SMB) for which there is no known long-form document, thus they would not have an entry in the manual index.  If that is not a problem, I can add those to this patch rather than submitting a new PR.~ (Edit: those have been added later.)  However, for proper display of opcodes, #1409 should get a bit more attention first - things can work fine without, but it'd look fairly ugly if you ask me...
This PR can be merged independently of #1533, #3095, and #3096.

Wow, I'm going to try this out now. Thanks so much for this - I'm working on a R-E project on a 65C02 based platform, this is going to save me a lot of work!

@agatti, do you have an updated version of your #1409 PR (basemodifier branch)? It seems to have some merge conflicts, but it sounds from this PR like you've used them together?

@philpem Right now it is not needed, as the extra opcodes mentioned in the PR comment haven't been added for now, since (as far as I know) there's no reference for them to be put in the manual index.
If it's ok for the manual being incomplete I can just copy them from an earlier version of this PR, and then #1409 could be brought back to life if there's any feedback from the Ghidra team (although things can still work without).

@agatti I've just given this a try in a build which includes PRs #3170, #3159, #3096 and #3095. It seems to work okay until Ghidra hits an SMB instruction (opcode D7) - which is reasonable enough, as this isn't implemented in the SLEIGH code (it looks like #1409 is required to implement it).
As far as the manual goes - I'm not a Ghidra maintainer, but a lot of the manual links were broken the last time I was working with SLEIGH. To me, an archive.org link is certainly no worse than what's already there. You could alternatively link to the CPU datasheet, if linking to the book is frowned upon.
Re. 1409, if it's a lot of work to implement, you could use 65tass's alternative syntax for the extended bit instructions:
RMB 7, 0x12
SMB 3, 0x34

It'd probably still display as SMB 0x3, 0x34, but it's a bit cleaner than SMB0x3.

@philpem for SMB, try copying the contents of https://github.com/agatti/ghidra/blob/new6502/Ghidra/Processors/6502/data/languages/extra_bitops.sinc into 65C02.slaspec.  It will display the opcode using an alternate syntax that doesn't rely on #1409.

@agatti That did it! This is incredible. The code I've been struggling to make sense of, finally makes some kind of sense!
It looks like there's some kind of arithmetic bug in the SBC instruction, though I don't think that's related to your changes.
LDA #0xC1
SEC
SBC #0x0

Virtual 6502 reports that the accumulator should contain 0xC1 but Ghidra miscalculates 0xC0. This seems to be down to this line in the SLEIGH spec, in the definition of the SBC instruction:
	local result = A - op1 - C;

The result should be ACC, minus op1, minus ~C. That is to say with carry set, you get result = A - op1, and with it clear you get result = A - op1 - 1.

@philpem I just realised that the manual pages for the extra opcodes are in fact present in an appendix of the document I used as a source - I'll bring them in later.
Regarding SBC, the definition is shared with plain 6502 and is not touched by my PR.  It'd be great if you could submit a PR of your own just for that - it'd be effectively independent of this one and would fix 6502 too.

I've filed #3189 for the the SBC carry-in bug, and submitted PR #3190 to fix it.

Speaking of which, I noticed I do have an SBC instruction definition as well for an extra addressing mode provided by the 65c02...  Going to fix that too.
I'll probably have read up on pcodeTests and how to integrate those on the 65(C)02 targets...


[Gally]: master <- dev

Automatically created by Git-Ally

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Error from server (ServiceUnavailable): the server is currently unable to handle the request

Focus on the bold parts
[root@k8s-master ~]# kubectl get --raw "/apis/metrics.k8s.io/v1beta1"
Error from server (ServiceUnavailable): the server is currently unable to handle the request
[root@k8s-master ~]# kubectl get apiservice
v1beta1.metrics.k8s.io monitoring/prometheus-adapter False (FailedDiscoveryCheck) 44s
[root@k8s-master ~]# kubectl describe apiservice v1beta1.metrics.k8s.io
Name: v1beta1.metrics.k8s.io
Namespace:
Labels:
Annotations: kubectl.kubernetes.io/last-applied-configuration:
{“apiVersion”:“apiregistration.k8s.io/v1”,“kind”:“APIService”,“metadata”:{“annotations”:{},“name”:“v1beta1.metrics.k8s.io”},“spec”:{"group…
API Version: apiregistration.k8s.io/v1
Kind: APIService
Metadata:
Creation Timestamp: 2020-04-21T09:30:16Z
Resource Version: 18821900
Self Link: /apis/apiregistration.k8s.io/v1/apiservices/v1beta1.metrics.k8s.io
UID: d0ae5bbf-e63d-4a77-ad7d-7ab45266b1f9
Spec:
Group: metrics.k8s.io
Group Priority Minimum: 100
Insecure Skip TLS Verify: true
Service:
Name: prometheus-adapter
Namespace: monitoring
Port: 443
Version: v1beta1
Version Priority: 100
Status:
Conditions:
Last Transition Time: 2020-04-21T09:30:16Z
Message: failing or missing response from https://<IP_ADDRESS>:6443/apis/metrics.k8s.io/v1beta1: Get https://<IP_ADDRESS>:6443/apis/metrics.k8s.io/v1beta1: net/http: request canceled while waiting for connection (Client.Timeout exceeded while awaiting headers)
Reason: FailedDiscoveryCheck
Status: False
Type: Available
<IP_ADDRESS>:6443 is prometheus-adapter endpoints
So,certificate is issus?
https://github.com/coreos/kube-prometheus/blob/master/manifests/prometheus-adapter-deployment.yaml
There is a:
volumeMounts:
- mountPath: /tmp
name: tmpfs
readOnly: false
- mountPath: /var/run/serving-cert
name: volume-serving-cert
readOnly: false
It appears to be an automatically generated certificate
spec:
containers:
- args:
- --cert-dir=/var/run/serving-cert
- --config=/etc/adapter/config.yaml
- --logtostderr=true
- --metrics-relist-interval=1m
- --prometheus-url=http://prometheus-k8s.monitoring.svc:9090/
- --secure-port=6443
image: quay.io/coreos/k8s-prometheus-adapter-amd64:v0.5.0
…...omit..…
volumes:
- emptyDir: {}
name: tmpfs
- emptyDir: {}
name: volume-serving-cert
But the log seems fine, right
[root@k8s-master ~]# kubectl logs prometheus-adapter-5cd5798d96-59zxf -n monitoring
I0421 09:26:19.002932       1 adapter.go:93] successfully using in-cluster auth
I0421 09:26:19.661459       1 serving.go:273] Generated self-signed cert (/var/run/serving-cert/apiserver.crt, /var/run/serving-cert/apiserver.key)
I0421 09:26:20.349449       1 serve.go:96] Serving securely on [::]:6443
So what do we do?

The problem has been solved

@w182757427 how to solve? I also met this problem

@w182757427 how to solve? I also met this problem

The custom API cannot request an Apiserver, which is resolved by deploying Kubelet on the master and joining the cluster

Thank you very much, I also encountered this problem in Alibaba Cloud environment, and I found that prometheus-adapter needs to be deployed to all master nodes. The following is my modification:

Thank you very much, I also encountered this problem in Alibaba Cloud environment, and I found that prometheus-adapter needs to be deployed to all master nodes. The following is my modification:
apiVersion: apps/v1
kind: Deployment
metadata:
name: prometheus-adapter
namespace: monitoring
spec:
replicas: 3
selector:
matchLabels:
name: prometheus-adapter
strategy:
rollingUpdate:
maxSurge: 1
maxUnavailable: 0
template:
metadata:
labels:
name: prometheus-adapter
spec:
tolerations:
- key: node-role.kubernetes.io/master
effect: NoSchedule
affinity:
nodeAffinity:
requiredDuringSchedulingIgnoredDuringExecution:
nodeSelectorTerms:
- matchExpressions:
- key: node-role.kubernetes.io/master
operator: Exists
containers:
......


Documentation improvement suggestions

Reported by Telegram users: Thanks!

TonyT908
Lemii

FIX:

[x] The /api/accounts/delegates endpoint does not exist anymore
[x] Make clearer on how to format the each env variable in docker-compose.yml. Some need "", some not!
[x] Document GNY_LOG_LEVEL
[x] remove wrong gny-js references in HTTP API document

Fixed by amongst other things PR: #43


panic upon committing a zero-sized grant

Sometimes zero-sized grant points beyond the buffer size (grant_ptr=buffer_ptr+buffer_len), with such grant commit panics on assert!(self.is_our_grant(&grant.buf));.

Repro:
use bbqueue::*;

fn main() {
    let bbq = bbq!(1000).unwrap();

    let size = 1000;
    let grant = bbq.grant(size).unwrap();
    bbq.commit(size, grant);

    let grant = bbq.grant(0).unwrap();
    bbq.commit(0, grant);
}

I'd be open for a PR that fixes the check in is_our_grant!


⚠ Pihole-KA has degraded performance

In 8b1b43e, Pihole-KA ($URL_PIHOLE_MERZ_NIMBUS) experienced degraded performance:

HTTP code: 200
Response time: 11951 ms

Resolved: Pihole-KA performance has improved in 86c4156.


proc_macro tokenstream dependency broken

error[E0599]: no function or associated item named `empty` found for type `proc_macro::TokenStream` in the current scope.

 |             TokenStream::Nightly(proc_macro::TokenStream::empty())
 |                                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ function or associated item not found in `proc_macro::TokenStream`

Vaguely recall seeing this a while ago and if I remember correctly I just needed to update to the latest nightly.

Indeed updating dependencies should do the trick here! If it doesn't though please let us know!

I believe I'm running the latest nightly: rustc 1.28.0-nightly (e3bf634e0 2018-06-28).

@David-OConnor have you tried cargo update -p proc-macro?


Update seq2seq

This PR brings updates to the seq2seq examples, removing all of the with ops.device("/cpu:0"):.
There are many ops implemented with cuda kernels recently, so maybe it's time to remove some cpu device specifications from the translate.py examples.
I don't have profile numbers for the translate.py example, but according to a similar experiment, this will boost the speed 5~6 times on a NVIDIA Titan GPU, especially when there are more than one independent runs going together.
Side note: when training with multiple cards (independent runs), the frequent data transfer between gpu and cpu is particularly harmful in my case.

Can one of the admins verify this patch?

ping ...
Any feedbacks are appreciated.

@tensorflow-jenkins test this please

@vrv Thanks for your attention. Updated. Do you know why the test failed on the matmul?

no idea, let me try again.
@tensorflow-jenkins test this please


first idea on pattern loops

As an alternative to our current header rules:
key: 'heading'
containsLoops: true
loopData: {
  level: ['6','5','4','3','2','1']
}
patterns: [
  # Empty headings
  {
    match: '^(?: {0,3})((?:#{1,6})\\s*(?:#*)\\s*)$'
    name: 'empty.heading.md'
    captures:
      1:
        patterns: [
          {
            patternLoop: true,
            template: {
              match: '^(#{<%= level %>})\\s*(#*)\\s*$'
              name: 'heading-<%= level %>.md'
            }
            fixed: {
              captures: {
                1: { name: 'punctuation.md' }
                2: { name: 'punctuation.md' }
              }
            }
          }
        ]
  }
  {
    # NOTE
    # Does it make sense to first do a general check to determine if there is a
    # match for a heading? It seems more logical to test against one match, than
    # to go through all six of them every iteration.
    match: '^((?: {0,3})(?:#{1,6})(?: +)(?:.+))$'
    name: 'heading.md'
    captures:
      1:
        patterns: [
          {
            patternLoop: true,
            template: {
              begin: '^(?: {0,3})(#{<%= level %>})(?: +)'
              end: '(?: ){1}(#*)(?: *)$'
              name: 'heading-<%= level %>.md'
            }
            fixed: {
              captures:
                1: { name: 'punctuation.md' }
                2: patterns: [{ include: '#inline' }]
                3: { name: 'punctuation.md' }
            }
          }
        ]
  }
]

I did not even know that was a possibility. Where do you find these things?
I was working on a different solution. In the first-mate package, there are two custom commands that can be used in the result of the match. One of them (downcase) is used in language-todo:
{
  'match': '(?<!\\w)@?(TODO|FIXME|CHANGED|XXX|IDEA|HACK|NOTE|REVIEW|NB)\\b'
  'name': 'storage.type.class.${1:/downcase}'
}

I've tried adding a length command, but because first-mate is a npm package, it works a little different from others, and I just can't get it to work in the way https://atom.io/docs/latest/behind-atom-developing-node-modules says it should.
My ideal solution would then look something like this:
key: 'heading'
patterns: [
  {
    match: '^(?: {0,3})((?:#{1,6})\\s*(?:#*)\\s*)$'
    name: 'empty.heading.md'
    captures:
      1:
        patterns: [
          {
            match: '^(#{1,6})\\s*(#*)\\s*$'
            name: 'heading-${1:/length}.md'
            captures: {
              1: { name: 'punctuation.md' }
              2: { name: 'punctuation.md' }
            }
        ]
  }
]

That is the beauty of it. It doesn't need to be possible.
We can make it possible, we have our own preprocessor(lib/main) which loads the rules, so there we can create custom stuff like looping.
We can do any addition to the rules (like disable, todo, loops with templates, debugging, etc).

So I do not keep finding these, I think of them ;)

Yeah, I already thought something along those lines. But would you mind trying to push that change into the first-mate package? I prefer to keep things 'basic' so others will understand things without any extra knowledge. But you can give it up after three sighs. I've sighed enough over that package.

@leipert Haven't heard from you in a long time, how are things?
I'm closing this issue/idea to keep things tidied up. I still think it is a good idea, but it might be better to implement these things as simple scripts (in a similar fashion that the fenced-code-blocks are generated) with a cson output, than as options that complicate the straightforwardedness of otherwise simple cson repositories. For a similar reason, I've also closed #10, although I think that actually concerns something that is missing from cson-parse or one of the other packages included.


Make IIS tests only use the netcoreapp2.1 as the client.

Currently tests run with netcoreapp2.0, 2.1 and net461. However, theses tests are only for the client, while the deployed result is always the same. We should just make all tests use netcoreapp2.1 and remove duplication of test coverage. The same thing was done by @Tratcher in server tests: https://github.com/aspnet/ServerTests/commit/53564567662a09adb3c8d4c5ecbd5b4cfada1351

Done


When parsing launch.json we need to account for invalid json (unnecessary commas)

For #332
Parsing launch.json fails when there are extra commas in array elements.
Unfortunately this is invalid json, hence JSON.parse bombs out and we display messages to the user (misleading messages).

#Prescribed solution:

Use jsonc-parser (used in extension today)

Breaks "Debug Test"

I was able to verify that on the last release "Debug Test" (via test explorer) fails if I have a trailing comma; and that it works if I use the most recent insiders .vsix.


Upgrade Docker Image to fix CI

Not sure if this is enough to fix CI.

I am not paying almost any attention to CI checks for a rather long time, so thanks for an update!


Assertion failed in fgMorphCall

In CoreCLR, canTailCall is hardcoded to return false in Ready To Run compilation - we never attempt to do tail calls. The CoreRT compiler doesn't put such restrictions and we're hitting the following assert when compiling Hello world with CHK RyuJIT (the combination of R2R and tail calls allowed triggers this):
#ifdef DEBUG
        // Tail call needs to be in one of the following IR forms
        //    Either a call stmt or
        //    GT_RETURN(GT_CALL(..)) or
        //    var = call
        noway_assert((stmtExpr->gtOper == GT_CALL && stmtExpr == call) ||
                     (stmtExpr->gtOper == GT_RETURN &&
                      (stmtExpr->gtOp.gtOp1 == call || stmtExpr->gtOp.gtOp1->gtOp.gtOp1 == call)) ||
                     (stmtExpr->gtOper == GT_ASG && stmtExpr->gtOp.gtOp2 == call));
#endif

To repro, undo the workaround from #2074.
@jkotas scoped the problem down to an issue around FEATURE_TAILCALL_OPT_SHARED_RETURN

@sandreenko We have a workaround that unblocks CoreRT for now, but we might hit this in other code.

Ok, I will try to fix it before tomorrow.

(I was trying to imply it's not urgent and there's no need to drop what you're working on right now. Just put it close to the top of the queue :).)

@vijayrkn can you take a look please.
Is it correct gtTree for tail call?
               [000029] --CXG-------             /--*  cast      int <- bool <- void
               [000026] --CXG-------             |  \--*  call ind  void
               [000023] ------------ arg0        |     +--*  lclVar    ref    V05 loc2
               [000024] ------------ arg1        |     +--*  lclVar    ref    V04 loc1
               [000025] ------------ calli tgt   |     \--*  lclVar    long   V03 loc0
               [000031] -A--G-------             *  =         int
               [000030] D----+-N----             \--*  lclVar    int    V07 tmp0

Call returns bool, but we cast it to int for asg.
Should we delete cast or do not consider such calls as tail calls?

@sivarv please take a look.
               [000029] --CXG-------             /--*  cast      int <- bool <- void
               [000026] --CXG-------             |  \--*  call ind  void
               [000023] ------------ arg0        |     +--*  lclVar    ref    V05 loc2
               [000024] ------------ arg1        |     +--*  lclVar    ref    V04 loc1
               [000025] ------------ calli tgt   |     \--*  lclVar    long   V03 loc0
               [000031] -A--G-------             *  =         int
               [000030] D----+-N----             \--*  lclVar    int    V07 tmp0

Is it correct tail call tree or not?

@sandreenko - It doesn't look like a valid tail call tree that RyuJIt supports as per fgMorphCall() assert.
What is even more suspicious is that call returns void and we have a cast from void to int!  I think that will point to the actual issue here.

It should be cast from int to bool to int, but we force call return type to void for all tail calls during morph phase.
        // As we will actually call CORINFO_HELP_TAILCALL, set the callTyp to TYP_VOID.
        // to avoid doing any extra work for the return value.
        call->gtType = TYP_VOID;

Please send me the full jitdump of the method by e-mail.


Typo?

p.13 “valid if it is formally valid” for “valid iff it is formally valid”

It already says "if" I think?

Sorry I meant the opposite. “iff” instead of “if”, or maybe “only if” instead of “if.”

fixed in https://github.com/rzach/forallx-yyc/commit/034ff3de5f79708339a389beb32bd8cd1604052b; thanks!


Work around a bug in libxml2

This commit works around a bug in libxml2 where parsing schemas can
result in dangling pointers which can lead to a segv.
Upstream bug is here:  https://gitlab.gnome.org/GNOME/libxml2/issues/148
Fixes #1985

This was released in v1.10.9


ft(NX-3071): Add see details link on initial offer message (inbox)

The type of this PR is: Feature
NX-3071
Description
Because users will be pointed to the inbox after making an offer (see NX-3041), we need to make it clear for them where to see the details of the offer.

Create conversation details route.
Allow opening offer order details from the initial offer message.

Twin PR on Force: https://github.com/artsy/force/pull/9292

Figma file for reference.
PR Checklist (tick all before merging)

[X] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[X] I have tested my changes on iOS and Android.
[X] I have added tests/stories for my changes, or my changes don't require testing/stories, or I have included a link to a separate Jira ticket covering the tests.
[X] I have added a feature flag, or my changes don't require a feature flag. (How do I add one?)
[X] I have documented any follow-up work that this PR will require, or it does not require any.
[X] I have added an app state migration, or my changes do not require one. (What are migrations?)
[X] I have added a changelog entry below or my changes do not require one.

To the reviewers 👀

[X] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

Add see details link on initial offer message (inbox) - Leandro L

iOS user-facing changes

Android user-facing changes

Dev changes

This PR contains the following changes:

Cross-platform user-facing changes (Add see details link on initial offer message (inbox) - Leandro L)

  Generated by :no_entry_sign: dangerJS against 6a9811d12d8b59e6fd19aa979bf76a4e352141a9

I just added a test to check the user is redirected to the order details when the link is pressed.


Decide on contract source elements approach for next iteration of prototype

Following from the conversation on #25, we hopefully have room to discuss our approach to dealing with contracts and contract templates. I think a good starting point would be to break down conceptually and interrogate what's necessary to generate a final contract artifact, which I assume is a PDF document with zero or more signatures rendered into it. To render such a contract, we need:

The text of the contract itself, structured in a way that formatting can be applied to it. For this, given that we want the contract to be configurable, we need:

A contract template with placeholders for the configurable parts.
The values that need to be injected into, or override, the configurable parts. These values could be specified directly or indirectly (i.e. they may need to be computed).

Signatures for zero or more signatories (only when all signatories have signed do we consider a contract to be "complete").
Formatting configuration to control how the contract is rendered (fonts, font sizes, letterhead, footers, etc.).
(Optional) Images that make up part of the contract (it's not clear whether this is important right now though).

I'll call these components the contract source elements for now.

UX Hypotheses
I have some UX hypotheses that I'd like to lay out here which we could confirm/refute over the coming weeks where we could devise small experiments and test them out on users and/or ask them. These should help shape the approach we take to structuring our contracts and/or contract templates.

[x] 1. Users want to be able to compose contracts by taking the "best"/most relevant parts of other existing (tried and tested) contracts and stringing them together as needed. (This seems self-evident due to our desire for contract templates in the first place.)
[ ] ~2. Users are more comfortable composing contracts in a templating language (e.g. Markdown/Jinja2) than in a programming language (e.g. Dhall).~ See 3 below
[ ] ~3. Users are more comfortable composing contracts in a templating language (e.g. Markdown/Jinja2) than in a configuration language (e.g. TOML). (The latter being our current approach with Themis Contract.)~ It appears as though we're fine with using a configuration language as an entrypoint into the contract.
[ ] 4. Users don't mind having formatting configuration be defined somewhere other than within/alongside the contract (i.e. they don't mind having it as part of their profile on their local machine).
[ ] 5. Users want to be able to customize contract formatting on a per-contract basis.
[ ] ~6. Users care whether contract source elements are organized within a single file as opposed to multiple files (this is my skeuomorphic hypothesis). The multi-file approach means contract source elements must be organized into folders if they are to be cleanly separated from one another, whereas the single-file approach means that multiple contracts can be stored in a single folder alongside each other.~ Users are fine so far with the multi-file approach.
[ ] ~7. Users would prefer to edit a full copy of the parameterized contract template (e.g. in Markdown/LaTeX) than use a templating, configuration or programming language to define their contracts. (Diffs here could easily be obtained by (1) including a reference to the original template in the contract, and (2) just doing a simple diff to see what changed from the template to the current contract.)~ This is effectively taken care of by including a local copy of the original template as per #47.

If there are other UX hypotheses anyone wants to add, please comment below and I'll add them to the list.

Implemented in v0.2 (currently on branch prototype/v2, soon to be merged to master).


getChats Function do not return value via tdJson.so

Hello!
I have a problem. I built a library and linked to libtdjson.so. (QtCreator, gcc 4.9).
I successfully called setTdlibParameters, setDatabaseEncryptionKey, setAuthenticationPhoneNumber, checkAuthenticationCode functions and connected to Telegram network.
I'm trying to get a list of chats via getChats to get list of ids and expect to get results via td_json_client_receive(). But I see output only via inner log and resulted request do not returned through td_json_client_receive(). I see desired output generated by this line.
What am I doing wrong?  Or td_json_client_receive method is designed to work only with UpdateObjects?

Here is the code where I am trying to call and process tdlib requests\responses

Everything works fine. Because of timer  I didn't see the output of response. Thanks for your Time. Closing issue.


Create BaseModel for all models

create a BaseModel that has these fields:
created_at field where it will be automatically generated when the model object is created
updated_at field which is updated whenever the model objects is modified
Then change all models in the project to have this BaseModel as their parent (with exception of User model which needs both AbstractBaseUser and BaseModel as it's parent at the same time)

completed in #110


FluxCD addon should set service account name when possible

Background
FluxCD has supported impersonating into a service account when installing or upgrading a helm release, since 0.16

https://github.com/fluxcd/helm-controller/blob/main/CHANGELOG.md#0160
https://github.com/fluxcd/helm-controller/pull/406

Kubevela has supported impersonating into a user since 1.4

https://github.com/kubevela/kubevela/pull/3713
https://github.com/kubevela/kubevela/pull/3434

Problem
By default, helm-controller uses it's own service account to install charts into cluster, which is bound to not only the essential flux-system:sa-helm-controller cluster role, but also a super user cluster-admin.
This lead to a problem that an application may escalate it's own privilege by installing a helm chart.
Proposal
Allow user to set the service account or let helm controller use the annotations set by kubevela mutating webhook

If user set the serviceAccountName in parameter, use the parameter
If kubevela set the username in annotation and it's a service account in the helm release's namespace, strip the prefix and use the service account name
or else it fallbacks to default

So that user can specify a service account, or the platform can restrict the action of helm chart.
Demo Implementation
Talk is cheap, let's see the code. This is a simple modification on fluxcd addon v2.1.1 3e7001cb55ff1897a7a7d6439dc3569a01db7222. tested on kubevela 1.5.10
diff --git a/addons/fluxcd/definitions/helm-release-def.cue b/addons/fluxcd/definitions/helm-release-def.cue
index e474fbe..72c8784 100644
--- a/addons/fluxcd/definitions/helm-release-def.cue
+++ b/addons/fluxcd/definitions/helm-release-def.cue
@@ -1,3 +1,4 @@
+import ("strings")
 helm: {
        attributes: {
                workload: type: "autodetects.core.oam.dev"
@@ -140,6 +141,12 @@ template: {
                        if parameter.values != _|_ {
                                values: parameter.values
                        }
+                       if parameter.serviceAccountName != _|_ {
+                               serviceAccountName: parameter.serviceAccountName
+                       }
+                       if parameter.serviceAccountName == _|_ && strings.HasPrefix(_serviceAccountIntermediate.username, _serviceAccountIntermediate.prefix) {
+                               serviceAccountName: strings.TrimPrefix(_serviceAccountIntermediate.username, _serviceAccountIntermediate.prefix)
+                       }
                        install: {
                                remediation: {
                                        retries: parameter.retries
@@ -170,6 +177,11 @@ template: {
                }
        }

+       _serviceAccountIntermediate: {
+               username: context.appAnnotations["app.oam.dev/username"]
+               prefix: "system:serviceaccount:\(context.namespace):"
+       }
+
        parameter: {
                repoType: *"helm" | "git" | "oss" | "oci"
                // +usage=The interval at which to check for repository/bucket and release updates, default to 5m
@@ -208,6 +220,8 @@ template: {
                version: *"*" | string
                // +usage=The namespace for helm chart, optional
                targetNamespace?: string
+               // +usage=The service account used to install or upgrade helm release
+               serviceAccountName?: string
                // +usage=The release name
                releaseName?: string
                // +usage=Retry times when install/upgrade fail.

It can be ported to master and kustomize with almost no effort, but I known little about kustomize

Good catch! ping @Somefive @wangyikewxgm


Break up repo?

As a long time scalatest user (thanks for this library!), I just cloned the source to try to poke around it more, and I'm finding it to be quite unwieldy; 568k lines of scala in what appear to be a dozen or so distinct modules.
I attempted to sbt test-compile after setting the JDK8 SBT_OPTS from the README, and gave up after 10s of minutes of what look like GC stall; I'm on a late-2013 macbook pro, 4 cores, 8GB RAM.
Intellij also had considerable difficulty understanding/navigating the project, to the point of being unusable.
This is a drive-by comment but it seems like there could be benefits to putting some of the subprojects here in their own repos.

Yes, it is awfully big, but that's mostly because we have soooo many tests. For some reason we seem to like testing. Part of that too is redundancy, which we haven't found the time to remove as yet.
What I do is
sbt compile
sbt test:compile
I do that in separate steps. Or if I'm just working on Scalactic at the time, I may just do:
sbt compile // compiles Scalactic and ScalaTest
sbt "project scalactic-test" test:compile // compiles just Scalactic's tests
But that's not all of the story, because to really compile everything you need to compile the whole thing under 2.10 and 2.11 (under Java 6) and 2.12 (under Java 8), as well as Scala.js on 2.10, 2.11, and 2.12. And we're wanting to cross build to Dotty too before too long.
Upshot is: there's no way we can separate these into different projects. We need them together so we can do all these builds in one place.

Thanks for that context. Some inlines:

mostly because we have soooo many tests. For some reason we seem to like testing

😀  fwiw here are the top 10 3-level-deep directories in the project by LoC of Scala:
$ gfns scala | x wc -l | pcs :3 | sbk | t11
  2574	project/GenInspectorsShorthands.scala
  2586	common-test/src/main
  2710	project/GenGen.scala
  3127	project/GenFactories.scala
  8008	scalactic-macro/src/main
 14263	scalactic/src/main
 14581	examples/src/test
 20449	scalactic-test/src/test
150085	scalatest/src/main
338601	scalatest-test/src/test
568228	total

Indeed it is mostly tests.

What I do is
sbt compile
sbt test:compile
I do that in separate steps.

I actually had done that too in my attempts yesterday, and still had the issues described.

Upshot is: there's no way we can separate these into different projects. We need them together so we can do all these builds in one place.

I don't think that this follows; surely there are logical components in this project (the top-level subdirs would seem to imply this) that could be factored out and depended on here (presumably as JARs on e.g. Maven Central) by the remaining pure-scalatest code.
Of course, "this would be nice but is a big project that no one has time for" is a fine answer, but "it has to (or should) be this way" does not seem very true or reasonable to me!


DiskLruCache and MediaPlayer

Hi,
I'd like to use DiskLruCache to store media files that I would play in a MediaPlayer later.
My problem is that the possible inputs to set the data source of a MediaPlayer are:

An Uri
A FileDescriptor
A path

The DiskLruCache.Snapshot only returns an InputStream so the only solution for me is to write the stream to a temp File and create a new FileInputStream. That's not really something I want to do :smile:
After a quick look under the hood, I saw that you are actually using FileInputStream so I suppose I could just cast the stream but it seems like a bad solution since the API doesn't specify it ==> it could break at any moment.
If we add a getFileInputStream or a getUri method to the DiskLruCacheSnapshot class, would it be a bad thing? I prefer to discuss this with you before working on a pull request.
Thanks for your help!

I don't think we want to add File here, particularly because the underlying files aren't stable. Anything that receives a File is naturally racy because of eviction.
You'll have to fork DiskLruProject to add this API!

Ok thanks for your answer, that's what I was afraid of :wink:
Will fork then!


As a user, I can view several instances of each model on the application

-at least 9 instances per model
Story points estimation: 8

@clairebingham Would you mind taking this task? We need to get more data 😄

@elmdecoste sure! 😄