Fix associated constants of InputWeightPrediction

These constants had an error that they had script_size set to 0 which
was incorrect because it's not length of the script but serialized size.
Rather than just bumping the value this uses the from_slice method
which is less error-prone.
Closes https://github.com/rust-bitcoin/rust-bitcoin/issues/1834

Yeah, as I predicted, the test is useless. A better test would use an actual transaction. Anyone wants to help find some? I don't think I will today.

Here's one for you.

    #[test]
    fn predict_weight_of_transaction() {
        // Taken from https://en.bitcoin.it/wiki/Weight_units
        const TX: &str = "0100000000010115e180dc28a2327e687facc33f10f2a20da717e5548406f7ae8b4c811072f85603000000171600141d7cd6c75c2e86f4cbf98eaed221b30bd9a0b928ffffffff019caef505000000001976a9141d7cd6c75c2e86f4cbf98eaed221b30bd9a0b92888ac02483045022100f764287d3e99b1474da9bec7f7ed236d6c81e793b20c4b5aa1f3051b9a7daa63022016a198031d5554dbb855bdbe8534776a4be6958bd8d530dc001c32b828f6f0ab0121038262a6c6cec93c2d3ecd6c6072efea86d02ff8e3328bbd0242b20af3425990ac00000000";
        let raw_tx = Vec::from_hex(TX).unwrap();
        let tx: Transaction = deserialize(&raw_tx).unwrap();

        let weight = tx.check_weight();
        assert_eq!(weight, Weight::from_wu(512)); // From wiki example.
    }

Have I missed something, we have the same logic in scaled_size as in predict_weight/predict_weight_internal, what's the plan here?

And here is two more if they are useful

    #[test]
    fn predict_weight_of_non_segwit_transaction() {
        // Taken from https://bitcoin.stackexchange.com/questions/92689/how-is-the-size-of-a-bitcoin-transaction-calculated
        const TX: &str = "0200000001d2eed35b0d55763981c635cab7788c28e3683af3d329947a17d7f6005390e6ef010000006a47304402200ffaac8f12e56f4af66109220812b76c7d6bb0e5906cf2de235b79496b3e080b0220037304894b648fa0b50e0d82ef9f58b537233ee282ab31fc04fb3d169d2c97cc0121030120a287eb98922752a89b39df64f7f3314036f6f096341c9189b0cb3c692aaeffffffff02404ff200000000001976a9140b818b11f9624e6a2d5b757a3e8fe45db3f6647788ace0b4b401000000001976a914a1e5e47fce1c5c0868107dba3851eb696c2ead5388ac00000000";

        let raw_tx = Vec::from_hex(TX).unwrap();
        let tx: Transaction = deserialize(&raw_tx).unwrap();

        let weight = tx.check_weight();
        assert_eq!(weight, Weight::from_wu(900)); // From link above.
    }

    #[test]
    fn predict_weight_of_segwit_transaction() {
        // Taken from https://bitcoin.stackexchange.com/questions/92689/how-is-the-size-of-a-bitcoin-transaction-calculated
        const TX: &str = "02000000000101caba4ccb61cca412fe29ec553d286134a02335c04355a9cc1e056fe2403692cf1400000000fdffffff026406010000000000160014093c864a10516154d18d2accd61c6b2920a2040f60361e000000000017a9140195e8dd3d1527038a0a77e0b0e4515d6c8ab195870247304402200760efedbcee3bbd913661fb40c364caed622b331ee3b5bc70ac25df2d3763d5022020efdb41ef325089d146eb48948b6aaec85c086fa38a3df4d2a51ee729645ebc012102be338e0362fb01101a873a7502012cb82f7d9d303b4943fa0702829a763e49762b550900";

        let raw_tx = Vec::from_hex(TX).unwrap();
        let tx: Transaction = deserialize(&raw_tx).unwrap();

        let weight = tx.check_weight();
        assert_eq!(weight, Weight::from_wu(565)); // From link above.
    }

cough macro

Is there some way we can just setup fuzzing for predict_weight to catch errors?  For example randomly create transactions with different script types and cross check the created transaction weight is what predict_weight predicted?

Sounds like a kani test could do that.

@tcharding that would be awesome, but I suspect Kani won't be able to handle it because transactions have variable numbers of inputs and outputs, and Kani tends to choke on Vecs.
A fuzztest would be a great idea. @yancyribbens are you willing to give it a shot?

@apoelstra for sure, Ill give it a go.  I feel like it will be rather involved to fuzz every type of script and every possible variation however.

@yancyribbens if it seems like you need to produce speciifc scripts and satisfactions, it would be better to do this in rust-miniscript. My hope was that we could just parse arbitrary Scripts or TxIns or whatever, try to use the weight prediction methods, and then construct a transaction and compare the resulting weight.

@tcharding those examples do not test constants though.
Regarding predict_weight vs weight I think the logic should be unified and we should have only one (basically Transaction::weight calling predict_weight). I originally didn't want to mess with existing stuff too much because I wanted to know whether it even works, whether the API is good... But I think we can start moving in that direction.

I think the difference between weight and predict_weight is very confusing.  If we can move towards unifying the two it would simplify the library a ton imo.

You mean internally? (not API)

I mean the difference between predict_weight(..) and Transaction::weight().  Those two methods should use the same internals, right?

Yes, weight should just call into predict_weight.

Decided to delete the test because I see no point in it:

The constants are now defined in terms of from_slice which is tested separately
We could test against real-world transaction but once we unify the logic test would become tautology

And honestly, I think tests of constant's aren't very valuable. These should never change anyway.

It's unrelated to this PR, but heads up that InputWeightPrediction::from_slice has an "only available in 1.46.0" doccomment that should now be dropped.


PR: Fix errors when Projects plugin is disabled

Description of Changes
Fix two issues when Projects plugin is disabled:

Fix re-opening of files from previous session when Projects plugin is disabled. Added test.
Don’t add empty Projects menu when Projects plugin is disabled

Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.
I certify the above statement is true and correct:
rear1019

Rebased and applied suggestions.


using vendored_openssl with bundled ducdkb-rs

I am trying to build duckdb-rs for AWS Lambda Runtime with vendored_openssl approach.
Is there a way to pass 'vendored' feature information from libducksys crate to openssl module within the libducksys crate.

@Maxxen Would it be feasible to include  https://github.com/duckdb/duckdb-rs/pull/321https://github.com/duckdb/duckdb-rs/pull/321 in coming release if the changes are acceptable?

Sure, ill have a look


ASE on VNet is not supported?

My customer tried to select (pull) a registered image in ACR on VNet from App Service Environment, but "Loading..." status appears on the Image dropdown listbox on the portal and registered images are not listed. I think that's because of the 2nd bullet limitation in the Preview Limitations, correct? And I would like to know any plans of supporting for other Azure services.  Thanks.

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

ID: 5bb583eb-86b2-5383-350c-985f66de2dd6
Version Independent ID: d08f5439-4db8-f9d0-eeb4-c1f4656fe0fa
Content: Deploy an Azure container registry into a virtual network
Content Source: articles/container-registry/container-registry-vnet.md
Service: container-registry
GitHub Login: @dlepow
Microsoft Alias: danlep

@kazshi, Thanks for the Comment. We are actively investigating and will get back to you soon.

@kazshi I reached out to the internal teams. I will post an update here once i have some information.

@jakaruna-MSFT Thanks for your update. Here's my questions:

ASE on VNet is not supported due the limitation and there's no way to deploy container for the environment (the current UI shows "loading..." status is very confusing).
The limitations would be cleared once the feature is in GA?

@kazshi I reached out to the internal teams. I will update you once i have the information

Got the update. Support for Linux App service in VNET is on the pipeline. Once that is in preview we can try accessing the ACR through service endpoint.
Right now the support is not there.

@jakaruna-MSFT Thank you for the clarification. Closing.

@jakaruna-MSFT Sorry, reopening since I forgot to confirm about the 2nd question.
2. The limitations would be cleared once the feature is in GA? I'd appreciate if you could provide any ETA of GA as well.  Thanks.

@kazshi Product team has plans to add support for deploying the Linux app service inside the VNET.
Once that is there, We can add a service endpoint to the subnet where the Linux app service is running.
Then we can add that vnet in the ACR firewall rules also.
Theoretically this way will work once we have the Linux App service in the Vnet. We still need to verify once the Linux App service in Vnet comes out in preview.
I dont have any timelime for the preview or GA date of Linux App service. So mostly this problem will be solved once the Preview release comes out. But we need to verify.

@kazshi Did you get a chance to look at the above comment.

@jakaruna-MSFT Thanks for your comment. I got it today, and provided to my customer.  Thanks.


(spike) Determine Open Source LLMs for Initial Evaluations

Description
As part of the deliverables for an MVP Evals framework, we will need a short list of LLMs we are evaluating on as part of LFAI. The models chosen should fit the following criteria (with room for potential exceptions):

Fewer than 5 total models (excess evals at the beginning is information overload, requires extra time, and may not be valuable out of the gate)
Models should be foundational (avoid hyper-specific fine-tuned models)
Models should be Apache-2.0 (or more permissible)
Models should be able to fit on 24Gb VRAM

Relevant Links
Inspiration can be found on the HuggingFace Open LLM Leaderboard

Since we implement Synthia-7B as our default model, it will be included in the list of initial evaluations.
We have also done our own quantization of a fine-tuned Mistral-7B, so this should be another model used in the initial evaluations.

Potential models for consideration:

Phi-3-small-instruct, in either the 8k or 128k context (also available for ONNX)
Mistral-NeMo (currently debating on size constraint concerns)

Quick update to balance scope and time; the following models will be used:

SynthIA-7B (default model for now)
Hermes2 (Defense Unicorns quantization)
LLama3.1-8B (pending license approval)

If not available, Phi-3-small (8k context)

GPT-4o will also be used as a point of comparison in the results.

Hi, @jalling97! Have you seen NVIDIA's distilled Llama 3.1 8B models?

https://huggingface.co/nvidia/Llama-3.1-Minitron-4B-Width-Base
https://huggingface.co/nvidia/Llama-3.1-Minitron-4B-Depth-Base

Here's the paper on model distillation, if interested: https://arxiv.org/abs/2408.11796

Hey @jxtngx! I have not, I'll be sure to take a look at these. The primary issue when it comes to using LLama3.1 is the license. We're still not sure based on the wording whether or not we're able to use it. Not being an expert in the legal side of things, I'm not sure if the NVIDIA Open Model License changes any of that, but regardless I'll be sure to mess around with these models anyways. The benchmark comparison looks promising. Thanks for sending this over!

There's also Nvidia's own Minitron 4B and 8B. Again though – the proprietary license may be a blocker.
I've added all of the models to this collection for easy reference:
https://huggingface.co/collections/jxtngx/nvidia-minitron-models-66d714aebae0e60d003a9693

@jalling97 the 24Gb vRAM requirement should be lowered to 16Gb (or 12Gb ideally), and also reflected back into our documentation. If we are considering lower-end GPUs (e.g., laptop GPUs, v100s, etc.) and/or laptop CPU RAM - think government laptop with 16Gb of RAM (13Gb free in ideal situation) - as the only offloading target in a worst case scenario, then we should reduce our minimums.
If the "example/demo" model for our repository is to fit into the minimum requirement laptops and machines, then we should focus on heavily quantized and/or lower param models. Until we activate vLLM's (still not prod ready though), or another engine's, CPU + GPU offloading, we are restrained to one or the other's compute RAM.
Other factors besides model parameter size include architecture, context size, and engine parameters we may be able to tune to lower vRAM usage at the edge. An oversimplified example of these considerations can be seen in the ADDITIONAL CONTEXT section of this WIP PR: https://github.com/defenseunicorns/leapfrogai/pull/854#issue-2438181942

Good call out! When I listed the 24Gb vRAM requirement, I don't think I properly conveyed that the intention was that the model should be fully "usable" on 24Gb vRAM (i.e not reaching OOM errors when using the model's full context). While keeping that determination of success the same, I agree that it makes sense to reduce that vRAM cap to 12Gb or 16Gb. I'll try to keep to 12Gb but may increase to 16Gb if a very promising model comes into play that requires it. Happy to discuss further if you like.

This research spike will be closed. The three models that will initially be evaluated are going to be:
In order to balance scope and time; the following models will be used:

SynthIA-7B (default model for now)

Hermes2 (Defense Unicorns quantization)

LLama3.1-8B

GPT-4o will also be used as a point of comparison in the results.


Refactored to kingpin and bugfix.

Refactored this exporter to use kingpin.
Fixed an EOF in CollectShowqFromReader.
@mjtrangoni can you confirm my changes please? :)

@BartVerc don’t forget to update the README file. This will be 2 - instead of one.


Add pre-commit hook

Can we add pre-commit hook? There is no Python markdown linter.

I'm confused about what this is asking. Are you asking that git commit hooks are added to the Python Markdown project to lint its own resources? Or are you asking for a separate project that uses Python Markdown and can be installed as a git hook to lint other Markdown content in other projects?
If it is the latter, you are free to create any project you need using Python Markdown as a dependency, I don't imagine we will be taking on an additional project to provide such functionality. If it is the former, I'm pretty sure we have everything we need to ensure proper project health and don't really need to add pre-commit hooks at this time.
Feel free to clarify if I am still a bit confused as to what is being asked.

I mean checking other projects on Markdown file correctness.

I mean checking other projects on Markdown file correctness.

That it out-of-scope for a Markdown parser. Note that a linter enforces best practices which is much more strict than a parser. Therefore, the linter does not need to make use of the same parser as you are using to parse your document.
Personally, I use and recommend DavidAnson/markdownlint as a Markdown linter.


Unable to activate claude sonnet 3.5 new

Type: Performance Issue
I have activated/enabled claude from my github copilot setting online but am unable to see them on my copilot chat on the vcsode
Extension version: 0.21.2
VS Code version: Code 1.94.2 (Universal) (384ff7382de624fb94dbaf6da11977bba1ecd427, 2024-10-09T16:08:44.566Z)
OS version: Darwin arm64 23.5.0
Modes:

System Info

Item
Value

CPUs
Apple M1 (8 x 2400)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabledwebgpu: enabledwebnn: disabled_off

Load (avg)
3, 4, 3

Memory (System)
8.00GB (0.07GB free)

Process Argv
--crash-reporter-id 9282fea8-c8cd-40a7-afe4-6e52c150d932

Screen Reader
no

VM
0%

Process Info
CPU %	Mem MB	   PID	Process
    8	   139	 30783	code main
    0	    57	 30788	   gpu-process
    0	    33	 30789	   utility-network-service
    0	   221	 30794	window [1] (import os.py — Sheldon_6.11_23.42)
    0	    82	 30807	shared-process
    0	     0	 30956	     /bin/ps -ax -o pid=,ppid=,pcpu=,pmem=,command=
    0	    90	 30850	extensionHost [1]
    0	     0	 30876	     /Users/chaya/.vscode/extensions/ms-python.python-2024.18.0-darwin-arm64/python-env-tools/bin/pet server
    0	    49	 30884	     electron-nodejs (server.js )
    0	    49	 30923	     electron-nodejs (bundle.js )
    0	    49	 30851	fileWatcher [1]
    0	    49	 30903	ptyHost
    0	     0	 30904	     /bin/zsh -i

Workspace Info
|  Window (import os.py — Sheldon_6.11_23.42)
|    Folder (Sheldon_6.11_23.42): 0 files
|      File types:
|      Conf files:;

A/B Experiments
vsliv368:30146709
vspor879:30202332
vspor708:30202333
vspor363:30204092
vscod805cf:30301675
binariesv615:30325510
vsaa593cf:30376535
py29gd2263:31024239
c4g48928:30535728
azure-dev_surveyone:30548225
vscrpc:30673769
962ge761:30959799
pythongtdpath:30769146
pythonnoceb:30805159
asynctok:30898717
pythonmypyd1:30879173
h48ei257:31000450
pythontbext0:30879054
cppperfnew:31000557
dsvsc020:30976470
pythonait:31006305
dsvsc021:30996838
01bff139:31013167
dvdeprecation:31068756
dwnewjupytercf:31046870
impr_priority:31102340
nativerepl2:31139839
refactort:31108082
pythonrstrctxt:31112756
cf971741:31144450
iacca1:31171482
notype1cf:31157160
5fd0e150:31155592
dwcopilot:31170013
g7688163:31175162

Please provide screenshots of your GH settings, VS Code, and the VS Code Copilot Chat output channel (CMD + SHIFT + U -> GitHub Copilot Chat).

PFA the screenshots as asked, let me know if you need anything else.
On Thu, Nov 7, 2024 at 10:09 PM Logan Ramos @.***>
wrote:

Please provide screenshots of your GH settings, VS Code, and the VS Code
Copilot Chat output channel (CMD + SHIFT + U -> GitHub Copilot Chat).
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode-copilot-release/issues/2380#issuecomment-2462708772,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BJAFK2I6R4C2RA3PIB5MD3DZ7OJVHAVCNFSM6AAAAABRK2OTX2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINRSG4YDQNZXGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

Emails lose attachments, can you upload them via GitHub

2024-11-08 07:45:22.439 [info] Using the Electron fetcher.
2024-11-08 07:45:22.439 [info] Initializing Git extension service.
2024-11-08 07:45:22.500 [info] Logged in as elvizlai
2024-11-08 07:45:22.525 [info] Successfully activated the vscode.git extension.
2024-11-08 07:45:22.525 [info] Enablement state of the vscode.git extension: true.
2024-11-08 07:45:22.525 [info] Successfully registered Git commit message provider.
2024-11-08 07:45:24.606 [info] Got Copilot token for elvizlai
2024-11-08 07:45:24.627 [info] copilot token chat_enabled: true, sku: yearly_subscriber
2024-11-08 07:45:24.627 [info] GitHub.vscode-pull-request-github extension is not yet activated.
2024-11-08 07:45:24.630 [info] Registering default platform agent...
2024-11-08 07:45:25.829 [info] Fetched model metadata in 1216ms d8648106-9425-45d1-8bb8-0f0ff86f517f
2024-11-08 07:45:26.003 [info] activationBlocker from 'languageModelAccess' took for 3387ms
Maybe I have similar issue, one of my computer has no choice for claude, but another has that with same account.

@elvizlai Anthropic models are not supported in China. Maybe that is what happening for you as I see your location is in Shanghai.

@hindunational Do you have a VPN you could try just to see if your IP is improperly being grouped as Chinese?

I don't have VPN and im not in China and I'm not at all enjoying using GPT
with copilot, it barely solves any problems. I'm using Claude Sonnet on my
subscription to fix all my error and it's rate limited. So currently I'm
wasting my copilot trial, not enjoying it and likely to not continue.
Please tell me how to get Claude Sonnet working within vscode for me. Btw,
downloaded cursor and it's working fine in their UI, why not yours.
On Fri, 8 Nov, 2024, 10:37 pm Logan Ramos, @.***> wrote:

@hindunational https://github.com/hindunational Do you have a VPN you
could try just to see if your IP is improperly being grouped as Chinese?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode-copilot-release/issues/2380#issuecomment-2465298049,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BJAFK2LFHIGK7PBIEPEIMA3Z7TVTNAVCNFSM6AAAAABRK2OTX2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINRVGI4TQMBUHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

@hindunational So are you stating that you could not enable claude or just you hit a rate limit on Copilot Chat? Just trying to understand the issue at hand.

I could not use Claude at all. I am getting rate limited on using Claude
directly on my own pro account, on vs code it's just not working. I'm
forced to use it on my rate limited pro account because it's not even
showing up as option on my vs code copilot.
On Thu, 14 Nov, 2024, 9:51 pm Logan Ramos, @.***> wrote:

@hindunational https://github.com/hindunational So are you stating that
you could not enable claude or just you hit a rate limit on Copilot Chat?
Just trying to understand the issue at hand.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode-copilot-release/issues/2380#issuecomment-2476852458,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BJAFK2N6GK64RCKNP5SNGUL2ATEY3AVCNFSM6AAAAABRK2OTX2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINZWHA2TENBVHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

@CicerBro the duplicate issue is resolved.
@hindunational Could you try again and let me know?

No, still the same. Only gpt 4o, o1 preview and o1 mini are visible as
options.
On Fri, 15 Nov, 2024, 6:51 pm David Jensenius, @.***>
wrote:

@CicerBro https://github.com/CicerBro the duplicate issue is resolved.
@hindunational https://github.com/hindunational Could you try again and
let me know?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode-copilot-release/issues/2380#issuecomment-2478802924,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BJAFK2J5JNKJBLCFSKV7MJ32AXYN5AVCNFSM6AAAAABRK2OTX2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINZYHAYDEOJSGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

@lramos15 I reloaded the users cached settings. How quickly does VS Code refresh the menu options? Can you tell if @hindunational is using the latest VS Code and Plugins?

@hindunational I have given you complimentary access through the end of the year for your bug report. You'll need to go through the signup flow again.

@hindunational Please ensure you update to the latest VS Code as Claude only works there as well. You will want copilot chat 0.22.2 and VS Code 1.95.3

@hindunational Please ensure you update to the latest VS Code as Claude only works there as well. You will want copilot chat 0.22.2 and VS Code 1.95.3

Thanks for the tip. Updating vs code helped!


[Help]: Where are wenet pretrained models?

Problem Overview
I want to use wenet features for svc training. But no wenet pretrain models found referred by 'pretrained/wenet/20220506_u2pp_conformer_exp/final.pt'. Where is the wenet pretrained  models 'pretrained/wenet/20220506_u2pp_conformer_exp/final.pt'? I will appreciate for any help.
Screenshots

Maybe you should download it yourself from huggingface,like how you get the whisper pretrained models.


Remove scope failed

[I'm supposed to describe steps to replicate my issue here and if I don't do that, I understand that my issue will be probably ignored.]

Brackets 1.2.0-15697 (mac), Git 2.2.1 & Brackets-Git 0.14.17
Remove scope failed
Error: git: 'ftp' is not a git command. See 'git --help'.

Did you mean this?
	fetch

call: git ftp remove-scope https://github.com/khanboy1989/WeatherWebApp.git
Error: git: 'ftp' is not a git command. See 'git --help'.

Did you mean this?
	fetch
    at Object.exports.toError (file:///Users/sk527/Library/Application%20Support/Brackets/extensions/user/zaggino.brackets-git/src/ErrorHandler.js:162:19)
    at NodeConnection.<anonymous> (file:///Users/sk527/Library/Application%20Support/Brackets/extensions/user/zaggino.brackets-git/src/Cli.js:176:44)
    at j (file:///Applications/Brackets.app/Contents/www/thirdparty/thirdparty.min.js:559:26911)
    at Object.k.fireWith (file:///Applications/Brackets.app/Contents/www/thirdparty/thirdparty.min.js:559:27724)
    at NodeConnection._receive (/utils/NodeConnection.js:492:34)

@khanboy1989 https://github.com/zaggino/brackets-git/blob/master/docs/GIT-FTP.md


Wrong colors on html-erb files using the desktop-session

Loading an html-erb file from a desktop session the colors of the themes are wrong while loading it with --no-desktop are shown correctly.
This happens with every theme.
I've done 2 screenshots:
with desktop session (wrong)

With --no-desktop (correct)

Thanks

That's because it's in MMM (multiple major-mode) mode, and desktop.el doesn't really understand it - so what's happened is that when your desktop was saved, the major mode for the file was remembered as ruby because your point was in a ruby block at the time. You can see that in the modeline. So yes, this will happen sometimes, but it should be sufficient to say M-x normal-mode to reset things. Not sure that there's a general fix for this, or at least I can't currently think of a good one, sorry.

Ah ok, thanks


The Gaffer If predicate in python fails if the predicate field is not set

This should be valid:
g.pred.If(
    condition=True,
    then=g.IsLongerThan(
        min_length=20,
        or_equal_to=True
    )
)

Merged into develop


Output seems to be MAC address instead of IP

Was outputting IP address using Docker container. However, I've installed using Go on a different device and now the outputs only seem to list MAC address.

Docker version was 1.0.0 seems to be an issue with 1.0.1

Hi,
Thanks for using an app :) I've figured out why is this happening. I've managed to reproduce it by using -O option in nmap (fingerprint OS). An example of such XML output:
<address addr="<IP_ADDRESS>" addrtype="ipv4"/>
<address addr="FF:FF:FF:FF:FF:FF" addrtype="mac" vendor="[Snippet]"/>

In this case nmap-formatter takes only the last address. Will fix it.

Fixed in v1.0.2 bugfix release
@Elevennails


Add disassembler option to show byte offset

The option is disabled by default.
The offset is printed in hex, as a comment after each instruction.

Rebased

+2


Use kite://home instead of /clientapi/sidebar/open

There's a new way to open the Copilot. Please use kite://home instead of GET /clientapi/sidebar/open.
Spec: https://kite.quip.com/HinNAHAoaVpH/Commands-in-editors#DPAACAbII8L

we may have an issue here, not sure why, but in VSCode we had some kite:// calls not properly opening the copilot so we had to make a request to /clientapi/sidebar/open to make sure the copilot is open before using the kite:// protocol

I confirm, using kite://home does not open the copilot from within VSCode, the whole kite:// protocol seems to only work from VSC when the copilot is already open. We've been stuck for a while with @dbratz1177 on this back when I was in SF and we couldn't figure why this failed to work.


Adds a warning to not start new integrations with Google sign in

WHAT
Adds a warning to not start new integrations with Google sign in
WHY
Google SignIn is deprecated
HOW
Checklist :clipboard:

[ ] Add explicit visibility modifier and explicit return types for public declarations
[ ] Run spotless check
[ ] Run tests
[ ] Update metalava's signature text files

LGTM


samples: create Pub/Sub BigQuery subscription

Rebased, though I'm not sure it changed anything.

Is the branch you are rebasing on up to date?

@amanda-tarafa Yes, it is up to date. Can you tell me more specifically what the issue is? The "Files changed" looks correct. So when it gets squashed and merged, I expect everything will be fine.

We usually do rebase and merge to respect the commits of the PR if those are meaningful. But is true that in this case we would have a single commit anyway so I can squash and merge. Doing so now...


Update HONO-TSO data

The website is giving TLS errors. Please fix.

Fixed! Website up and running (@ hono-tso.net!) - Thank you

Da: RareData @.>
Inviato: venerdì 13 gennaio 2023 21:11
A: TowoLabs/ftso-signal-providers @.>
Cc: kosamatteo @.>; Author @.>
Oggetto: Re: [TowoLabs/ftso-signal-providers] Update HONO-TSO data (PR #274)
The website is giving TLS errors. Please fix.
—
Reply to this email directly, view it on GitHubhttps://github.com/TowoLabs/ftso-signal-providers/pull/274#issuecomment-1382339733, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADSW2OCINKFBO2MQQ4T4Y3TWSGZI3ANCNFSM6AAAAAATZOH364.
You are receiving this because you authored the thread.Message ID: @.***>

Can you please update the logo to not include any whitespace around it?

I can see there's still transparency/margin between the icon and the borders. Do you mind removing that? It'll look bad in most wallet software.

Fixed in https://github.com/TowoLabs/ftso-signal-providers/commit/2b25addb2b8bdd533effb40f4071ed14b0bca9cc


Node::Advertise doesn't fail when advertising a duplicate service

I'd expect that there can only be one "responser" for each advertised service. But that's not true.
Taking code from the examples:
$ ./responser_oneway &
[1] 10281
$ sleep 5 # to be sure the first responser gets properly advertised
$ ./responser_oneway &
[2] 10313

$ ign service -s /oneway -i
Service providers [Address, Request Message Type, Response Message Type]:
  tcp://<IP_ADDRESS>:39721, ignition.msgs.StringMsg, ignition.msgs.Empty
  tcp://<IP_ADDRESS>:43201, ignition.msgs.StringMsg, ignition.msgs.Empty

$ ign service -s /oneway --reqtype ignition.msgs.StringMsg --reptype ignition.msgs.Empty --timeout 2000 --req 'data: "Hello"'
Request received: [Hello]
Service call timed out

So it seems the discovery service is perfectly fine with having multiple providers of a service, but the client gets confused (the console only has one Request received... line, which means only one of the service responders got called). The timeout is weird.
I think it's wrong that the discovery service registers the other responser (but given the distributed nature of the system, maybe it' s inevitable?). However, Node::Advertise should definitely check for duplicate services before returning true.

Hi @peci1 , this is a design decision. The discovery layer knows about all the services because in case one of them becomes unavailable, the requester node can use the next service up and running. If you have multiple instances of the same service, there's no guarantee of which one will provide the response. You can only assume that one of them will provide the response.

Hi @peci1 , this is a design decision. The discovery layer knows about all the services because in case one of them becomes unavailable, the requester node can use the next service up and running. If you have multiple instances of the same service, there's no guarantee of which one will provide the response. You can only assume that one of them will provide the response.

Ah, then the docs are lacking, because I was searching really thoroughly and haven't found anything about this design decision. I searched both tutorials and source code of the library. I think such non-trivial decision needs to be described in the docs of Node::Advertise() then...
And the timeout shows to be unrelated to the two running responsers - it also happens with just one. It is probably caused by #97.

Ah, then the docs are lacking, because I was searching really thoroughly and haven't found anything about this design decision. I searched both tutorials and source code of the library. I think such non-trivial decision needs to be described in the docs of Node::Advertise() then...
And the timeout shows to be unrelated to the two running responsers - it also happens with just one. It is probably caused by #97.


Publish to Maven Central using GitHub Actions

As per the title, I'm creating an issue to track the work of automatically publishing Hamcrest through GitHub actions. There was already much discussion on #405, but this deserves its own issue. @olibye is assigned, but if I (@tumbarumba)  can work out what to do, I can probably help too.
We may be able to copy parts of the jMock release workflow (see https://github.com/jmock-developers/jmock-library/blob/master/.github/workflows/release.yml), though jMock uses Maven, so is probably not directly transferable to a Gradle project

@olibye, you said in the #405 thread:

I need to be given permissions/credentials to the publishing account. Or have some security token that GitHub actions can use.

I published 3.0-rc1 using my own personal credentials. I'm not keen on adding those directly to GitHub. Perhaps we should look at setting up some sort of service account? How does jMock handle that issue?

After spending weeks tracking down all the bits and pieces I had to do to be able to publish, I wrote down everything I did and put it in the repo, here: https://github.com/hamcrest/JavaHamcrest/blob/master/RELEASING.md

So, you can create a publishing token, which is separate to your username and password.
Then you can add that as a GitHub secret to the hamcrest GitHub repo and I'll copy and tweak the jmock GitHub actions for hamcrest.

https://central.sonatype.org/publish/generate-token/

https://docs.github.com/en/actions/security-guides/using-secrets-in-github-actions#creating-secrets-for-a-repository

I'll try and look at this over the weekend.
I've also created #419 as a way of improving this workflow. Instead of commiting an updated version to the build every time, the version is derived from git tags. I've used this style in some other python projects I've been working on, but I haven't really seen it used in Java projects.
What do folks think of that approach?

add that as a GitHub secret

Unfortunately AFAICT there is no other option for now despite https://github.com/sigstore/sigstore-java/tree/main/sigstore-maven-plugin


handling Qubits in extract_qubits

Fixes #6

@tarballs-are-good Can you take a look at this?

Back to you @tarballs-are-good!

LGTM


"NotImplementedException" from OwnershipHandler.cs

Hi there! From Multi-user capabilities got a "NotImplementedException" #8311, filling something here as the  issue is related to the tutorial :)
As explained in the MRTK issue, I was wondering the reason behind the "throw new NotImplementedException() " inside OnInputUp(), OnOwnershipTransfered() and OnTriggerExit() as it fire an error.
Is it for a future update of the series, or is it something that we (as users) have to fullfill according to our needs ?
The script is located at MRTK.Tutorials.MultiUserCapabilities/Scripts/OwnershipHandler.cs
Additional context
Again, I might be missing an obvious point here because of my lack of knowledge on certain points because of my "weak" background in software development.

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

ID: 439d2561-655e-5168-3504-a50ed1ba9879
Version Independent ID: e3d7c1bf-82e4-2b30-df08-c5fb340a25c3
Content: Multi-user capabilities tutorials - 5. Integrating Azure Spatial Anchors into a shared experience - Mixed Reality
Content Source: mixed-reality-docs/mr-learning-sharing-05.md
Product: mixed-reality
Technology: mr-dev
GitHub Login: @jessemcculloch
Microsoft Alias: jemccull

@mr0ng @martinwilter Can you confirm?

Hi @nooxouille,
Sorry for the confusion. Those are required delegate members that are not used in the project, you can comment out the body like you already did.
@hferrone, this shouldn't prevent users from running the completed app, but I can see how it is confusing. I will update the tutorial assets later this week.
CC @mr0ng

@martinwilter Thank you for the clarification. It is indeed confusing as the error prevent the compilation and therefore, to run the samples.

@martinwilter Did this get updated?

@hferrone, I should be able to get this done today, I'll ping you here once the patch has been released.

@hferrone, the asset has been updated.
@nooxouille, just as an FYI, the reason your testing in the Unity editor was interrupted by the 'NotImplementedException' logs is because you had 'Error Pause' enabled, you can toggle it on/off by clicking the button in the Console window. Anyway, it's best without those appearing in the console logs, so thanks for pointing this out.

@nooxouille, just as an FYI, the reason your testing in the Unity editor was interrupted by the 'NotImplementedException' logs is because you had 'Error Pause' enabled, you can toggle it on/off by clicking the button in the Console window. Anyway, it's best without those appearing in the console logs, so thanks for pointing this out.

Oh! Thx for the tip! Always happy to help :)


carto: use buildNpmPackage

Description of changes
See https://github.com/NixOS/nixpkgs/issues/229475

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

cc @Luflosi who originally packaged it in https://github.com/NixOS/nixpkgs/pull/187891

Result of nixpkgs-review pr 249879 run on x86_64-linux 1

1 package marked as broken and skipped:

magnetophonDSP.ConstantDetuneChorus

6 packages failed to build:

geph.gui
imgbrd-grabber
iosevka-comfy.comfy
iosevka-comfy.comfy-duo
iosevka-comfy.comfy-motion
sitespeed-io

150 packages built:

activitywatch
ansible-language-server
antennas
antora
ariang
assemblyscript
audiobookshelf
authelia
aw-server-rust
balanceofsatoshis
bibtex-tidy
bitwarden
bitwarden-cli
carto
castnow
cdxgen
cz-cli
deltachat-desktop
djot-js
docker-compose-language-service
documenso
dot-language-server
eask
elmPackages.elm-test
emscripten
epgstation
eslint_d
faust (faust2)
faust2alqt
faust2alsa
faust2csound
faust2firefox
faust2jack
faust2jackrust
faust2jaqt
faust2ladspa
faust2lv2
faust2sc
faustPhysicalModeling
faustlive
filebrowser
flood
forgejo
forgejo.data
fx-cast-bridge
github-copilot-cli
gnomeExtensions.pop-shell
google-clasp
gtop
guitarix
haskellPackages.aeson-typescript
haskellPackages.aeson-typescript.doc
homepage-dashboard
hred
hueadm
husky
iosevka
iosevka-comfy.comfy-fixed
iosevka-comfy.comfy-motion-duo
iosevka-comfy.comfy-motion-fixed
iosevka-comfy.comfy-wide
iosevka-comfy.comfy-wide-duo
iosevka-comfy.comfy-wide-fixed
iosevka-comfy.comfy-wide-motion
iosevka-comfy.comfy-wide-motion-duo
iosevka-comfy.comfy-wide-motion-fixed
jellyfin
jellyfin-media-player
jellyfin-web
kapitonov-plugins-pack
karma-runner
kaufkauflist
kavita
lineselect
lv_img_conv
magnetophonDSP.CharacterCompressor
magnetophonDSP.CompBus
magnetophonDSP.LazyLimiter
magnetophonDSP.MBdistortion
magnetophonDSP.RhythmDelay
magnetophonDSP.VoiceOfFaust
magnetophonDSP.faustCompressors
magnetophonDSP.pluginUtils
magnetophonDSP.shelfMultiBand
maizzle
markdownlint-cli
matrix-alertmanager
memos
mermaid-filter
mirakurun
mongosh
mooSpace
moonfire-nvr
mpvScripts.webtorrent-mpv-hook
mw
nest-cli
newman
node-manta
nodehun
npm-check
ntfy-sh
open-music-kontrollers.mephisto
open-stage-control
osmtogeojson
pairdrop
paperless-ngx
photofield
photoprism
pnpm-lock-export
polaris
polaris-web
psitransfer
pufferpanel
pyCA
pyCA.dist
readability-cli
resumed
reveal-md
semantic-release
sharing
slskd
sunshine
swagger-cli
tambura
terminal-stocks
thelounge
torq
triton
twspace-crawler
typescript
uivonim
uptime-kuma
vencord
vencord-web-extension
vieb
vimPlugins.YouCompleteMe
vsce
vscode-extensions.vadimcn.vscode-lldb
vscode-langservers-extracted
webcord
webcord-vencord
webtorrent_desktop
weylus
windmill
yarn-lock-converter
ycmd
zigbee2mqtt
zincsearch
zrythm
zx


[MB-5797] Add migration to create zip3_distances table

Description
This PR adds a migration to create a zip3_distances table that will be storing zip3-to-zip3 distances (data coming in a subsequent subtask/migration).  This data will eventually be used for pricing moves.
See the design doc for this work.
Reviewer Notes

We only store unique zip3-to-zip3 pairs in one direction (since presumably the opposite direction should have the same distance and our input file only gives us one direction).  The zip3_distances_ordering constraint and zip3_distances_unique_zip3s unique index enforce that.
I put a default of NOW() on the created/updated timestamps.  We don't do that on many other tables since Pop sends it anyway, but Pop isn't involved in the data load.  Since the data load will be a large file, using the defaults helps to reduce the file size.
We haven't historically documented constraints/indexes, but these two seemed like good candidates to have further explanation.

Setup

make db_dev_reset db_dev_migrate to make sure the migration runs without errors and you see the schema changed appropriately.
make schemaspy, then open the index.html file in the generated web site (link at end of make output).  You should see the comments appear for that table.  Alternatively, use GoLand or some other database tool that shows Postgres comments.

Code Review Verification Steps

Any new migrations/schema changes:

[ ] Follow our guidelines for zero-downtime deploys (see Zero-Downtime Deploys)
[ ] Have been communicated to #g-database
[ ] Secure migrations have been tested following the instructions in our docs

[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change
Design Doc

Messages

:book:
:link: MB-5797

  Generated by :no_entry_sign: dangerJS against 14fb51a7cf360e69fd79d950468099ff1772dddf


UI Organization

Group related UI elements within wellPanel or box for better visual organization.
Use sidebarLayout with sidebarPanel and mainPanel ?

Heres an app that uses the white space we are neglecting so far. Thoughts? https://chatlize.ai/

putting the map and table on different tabs makes it a little less weird that filtering the table doesn't affect the map and clicking on the map doesn't affect the table.

yeah I don't mind that. But which tab should go first: the table or the map?

map first looks a little more visually friendly for new users

I think we're good, ready for user feedback


doc change regarding consul SSL

document change to clarify consul ssl, vs consul ssl client certificate security
https://github.com/containous/traefik/issues/1718

original #1773


Animations in wallet connection flow

Add transitions and animations for react-wallet package.

Task will need @t-z-u to add video and maybe description.

@creed-victor could we add this to the sprint board?

@creed-victor could we add this to the sprint board?

Maybe for next sprint, this one already is fulll. Unless you need it to be available as dependency?

no animations are def nice to have


Safely end training even if trackers weren't initialized

What does this PR do?

If you create an instance of Accelerator() without logging to a tracker, and then call end_training (which would be common in development/debugging), the accelerator shouldn't crash if there aren't trackers. Since self.trackers isn't initialized in the __init__ of the class, this is possible
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[X] Did you read the contributor guideline,
Pull Request section?
[ ] Was this discussed/approved via a Github issue or the forum? Please add a link
to it if that's the case.
[ ] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines, and
here are tips on formatting docstrings.
[ ] Did you write any new necessary tests?

Tested by running the following
>>> a = Accelerator()
>>> a.end_training()

Made sure this wasn't thrown
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File ".venv/lib/python3.10/site-packages/accelerate/accelerator.py", line 615, in _inner
    return PartialState().on_main_process(function)(*args, **kwargs)
  File ".venv/lib/python3.10/site-packages/accelerate/accelerator.py", line 2499, in end_training
    for tracker in self.trackers:
AttributeError: 'Accelerator' object has no attribute 'trackers'

Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.

I wonder if the better solution wouldn't be to set self.trackers = [] inside of Accelerator.__init__ instead of init_trackers. Within init_trackers, we could call self.trackers.clear() for consistency.

I wonder if the better solution wouldn't be to set self.trackers = [] inside of Accelerator.__init__ instead of init_trackers. Within init_trackers, we could call self.trackers.clear() for consistency.

@BenjaminBossan i totally agree, and in fact wrote that first, but I thought there was likely a good reason why that wasn't done in the first place. It's uncommon not to initialize all member variables on __init__ (in fact i'm pretty sure mypy will complain), so I assumed it was purposeful.
Do you think I should go ahead with that?

You make a good point that there could be a non-obvious reason. If it's okay to you, we could wait for Zach to comment on this (which may take a week or so).

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Hi @muellerzr updated!

Hey thanks @BenjaminBossan  --  the first (removing self.trackers in init_trackers) was already removed, but i've cleaned up the getattr and just referenced self.trackers


ReferenceError: regeneratorRuntime is not defined

[Show/hide message details.] ReferenceError: regeneratorRuntime is not defined

app.js line 9278 > eval:1678:7
mutate
vue-apollo.esm.js:1667
mutate self-hosted:981:17 submit
PasswordReset.vue:65
invoker
vue.runtime.esm.js:2023
withMacroTask/fn._withTask
vue.runtime.esm.js:1822

@archSeer do you tried beta-16?

Beta 16 is broken for me because of #295.

Tried on beta 17, still broken, I'm just using a basic vue-cli 3 setup.

Do you have beta-17 in package-lock?

I'm also seeing this while using onLogin to set an auth token:

https://github.com/babel/babel/issues/5085

Fixed for me by doing:
yarn add --dev @babel/plugin-transform-runtime
yarn add --dev @babel/core

and then in babel.config.js:
module.exports = {
  presets: ['@vue/app'],
  plugins: [
    "@babel/transform-runtime"
  ]
}

Solved
$ npm install --save-dev babel-plugin-transform-runtime
then In .babelrc, add the runtime plugin
{
"plugins": [
["transform-runtime", {
"regenerator": true
}]
]
}
thanks me later

Another solution for those stumbling to this error too
npm i -S core-js@latest regenerator-runtime@latest

Then in your Vue main.js file
// Import these two at the top
import 'core-js/stable'
import 'regenerator-runtime/runtime'
...


support file inputs

Supports file inputs, in a way I think it makes most sense:
On single file uploads:

returns null if input is empty
returns File if a file was selected

On multiple file uploads:

returns [] if input is empty
returns File[] if one or more files were selected

Thank you#


Add ucx version specifier

This PR adds a version specifier for ucx.
This particular version specifier will ensure that ucx packages from the rapidsaiai and rapidsai-nightly channels are not used.
Instead, the ucx packages from conda-forge will be used.
See the links below for details on the versions that exist in each organization:

https://anaconda.org/rapidsai-nightly/ucx/files
https://anaconda.org/rapidsai/ucx/files
https://anaconda.org/conda-forge/ucx/files

/merge

/merge

Thanks @jjacobelli ! You read my mind, was just typing to request an approval! 😄


[Find] checked out from albaloo master

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

If this PR is for a new extension or change to an existing extension, use the following to try out the changes in this PR:
docker run -it microsoft/azure-cli:latest
export EXT=<NAME>
pip install --upgrade --target ~/.azure/cliextensions/$EXT "git+https://github.com/williexu/azure-cli-extensions.git@albaloo_master#subdirectory=src/$EXT&egg=$EXT"


Create Home page in Meshery UI

Description
Users need a landing page that both introduces them to the app and provides an overview of the status of their environment and the configuration of Meshery.

Screen design doc - https://docs.google.com/document/d/1XOQFIyHsY_s_LgOBPDdg__dmgE3GpBK3e1Gir4Me1Kg/edit#


cppclient: add unit test of getNodeInfo()

Imitate the unit test of getBalances(), and there is another unit test way which is imitated from iota.lib.cpp.

@jserv  I fixed it


Fix unpacking update request

This should fix https://github.com/pmorie/osb-broker-lib/issues/19

LGTM

Should we add test cases to verify that the correct data is getting passed through?

@shawn-hurley Agreed! Adding them now :)

@LiliC tells me she's time constrained and won't be able to finish the tests for a couple days. Merging now since I trust her to follow through :)


Tweak the property hint ranges of caret blink and line length guideline

This allows for more precise adjustments of the caret blink speed and the line length guideline.

Cherry picked into 3.0.3


Case sensitivity fix

Adds most keywords into the match statement for allowing case insensitivity.
This is related to #49
I was hoping that the regex expressions would allow for regex groups for some sort of wildcard match, but oh well.
It does not handle the "!OSRE" keyword case.

It does not handle the "!OSRE" keyword case.

What's wrong with adding a rule to fix this?
  r"(?i)osre" => "osre",

Does that break something?

It did not seem to recognize it. I will have another go at it tomorrow if I have time. Maybe I just misconfigured something.

! and osre are separate tokens. Perhaps you tried to handle !osre as a single token?

Ok, so it turns out when I was initial trying to implement it, I just wrote a bad jmp instruction, and it was that simple.


500 errors

I'm getting a 500 error when I try to go to "Instruments" on UMIL...

I'll look into and fix this.

I found the reason of this issue. We were using lowercases in urls, e.g., "https://vim.simssa.ca/instruments/?language=french"
But after we updated the language list, the en_label for each language are uppercases. I'll fix it to keep the case consistent.

This is not fixed for many languages, including Japanese: https://vim.simssa.ca/instruments/?language=Japanese.


Update readme

Should be more detailed about using the package, what each does, what the return object looks like how to use it, etc.

Readme should also outline that all comparison are case insensitive and suffix is puncuation insensitive


able to resolve dot-com, but not dot-org names, what?

This is very bizarre behavior.  Actually I seem to have at least these couple of issues at play:

all the .org names I try to resolve with host or ping, yield:  Host perlmonks.org not found: 2(SERVFAIL), but same message and error regardless of which dot-org name I try to resolve.

I get Host puppet.mydomain.com not found: 3(NXDOMAIN) for anything listed in my own domain, which is displayed by weave status with these results:

        Service: dns
         Domain: mydomain.com.
       Upstream: $ip.for.providers.ns1, $ip.for.providers.ns2
            TTL: 1
        Entries: 1

I get this:
# weave status dns
nfs          <IP_ADDRESS>       20f6c15aa4b6 7a:2b:e2:33:29:74

but neither mtr (Failed to resolve host: Name or service not known), ping (unknown host), nor host (not found: 3(NXDOMAIN)) give me any joy finding containers on my network with either host names or fqdn's.
What is working:  containers hosted on distinct docker hosts are able to ping one another by my assigned IPs.  I am able to resolve dot-com addresses in DNS using the tools mentioned above.
Anyone here have any idea what I am looking at?  Is this related to my weave usage or my docker usage?
-- Hugh

Hi @hesco,
There is nothing in the Weave DNS code to prefer .com over .org; I have to guess that these replies are coming from your upstream DNS service.

Thanks.  Except that my host can resolve the .org names just fine.  Very strange.

You could try weave launch --log-level=debug to see more detail about the DNS requests and responses.

Closing.  If you get any information about Weave Net's involvement please re-open.


Remove exposing windows.h header from our public headers within storage.

Follow up to https://github.com/Azure/azure-sdk-for-cpp/pull/2188, fixes https://github.com/Azure/azure-sdk-for-cpp/issues/2182

@Jinming-Hu I am waiting for your review before merging this. Should be fairly quick :)

This is a .cpp file, why do we need to remove windows.h from .cpp file?

We don't, fixed the PR to update the header file instead. Good catch.


[Integration] write Apache Avro data format extension

do it in a separate bagri-extensions project;
get message content in Avro format, pass to the server side;
translate message in Avro format directly to List<Data>; what if message contains several documents!?

can be used to process Google Protobuff messages as well?

implemented, has to be tested.
investigate: Protobuff support

Good enough for the first release. Support for Protobuff can be added on the next one


What is the mechanical method of connection for the switches?

For other Amoeba variants, soldering or hotswap sockets are used to keep the mechanical switches in place. Do the optical switches fit tightly enough into the optical amoeba pcbs to keep them from slipping off if they were in something like a dactyl manuform?

No, there is some play in the hole where switch sits. Making that hole tighter will interfere with the functioning, and it will not hold the switch anyway (hole is too small compared to size of the switch).
If you are 3D printing the case, you can make the square hole a little smaller than 14x14 so you can have a really snug fit.
Even better would be to design the whole pcb, have it made with a semi-flexible pcb material (0.6mm), have smd components soldered (all by jlcpcb for instance). You just have to solder a MCU on to it.
See:
https://girishji.github.io/opticalkb/2022/08/17/optical-keyboard-design.html
https://github.com/girishji/optical-keyboard
https://github.com/girishji/optical-keyboard-mx
https://girishji.github.io/kicad/2022/08/17/kicad-python-footprints-curved-tracks-edge-cuts.html

Thanks! This is exactly what i was looking for!


Modal is also dismissed when pressing the ESC key to close a combobox's dropdown panel.

Describe the bug
I have a combobox inside a modal but when you try to press the ESC key to close the it's dropdown panel, the modal is also get dismissed.

How to reproduce
https://stackblitz.com/edit/clarity-v4-light-theme-jz6if2?file=src/app/app.component.ts
Steps to reproduce the behavior:

Toggle the dropdown panel of the combo box.
Press ESC key.

Expected behavior
It should only dismiss/close the items panel/dropdown panel of the combo box and not the modal.
Versions
4.0.2
App

Angular: 10
Node: 12
Clarity: 4.0.2

Device:

Type: [e.g. MacBook]
OS: [e.g. iOS]
Browser: Chrome/Edge Chromium
Version [e.g. 22]

Additional notes
Also, you will notice that there's a console error when pressing the ESC key.

I know that this is a collision between the combobox/popover code and the modal code that dismisses on escape. But I can't find where that collision is happening. It would require a deeper dive than I have timeboxed. I will put this in the backlog.
I know why it's happening but cannot say where it is happening. Any attempt I made to correct it introduced EHCAIWC errors. 🤷♂

I had a research session based on this scenario last week.
Clarity can not know if a component should or should not stop event propagation. But this can (and should) be handled on the user side. Here is a demo, illustrating how this can be achieved for a modal and dropdown:
https://stackblitz.com/edit/pattern-modal-smart-escape-demo?file=src%2Fapp%2Fsmart-escape.directive.ts
It may currently not work with combobox, because the combobox is emitting an extra "open" event, which is a combobox bug and will be logged separately.

The combobox event bug mentioned above:
https://github.com/vmware/clarity/issues/5553

@Jinnie Could you elaborate a bit more conceptually? Why can't a component decide to stop event propagation? In the case of a popped open dropdown, it seams reasonable to assume that event propagation of ESC can be stopped?
ps: Just trying to get a better grasp of Clarity concepts here :-)

@Jinnie Could you elaborate a bit more conceptually? Why can't a component decide to stop event propagation? In the case of a popped open dropdown, it seams reasonable to assume that event propagation of ESC can be stopped?
ps: Just trying to get a better grasp of Clarity concepts here :-)

As a library we cannot stop event bubbling. We need to make that guarantee so that Clarity users have access to events generated inside a Clarity component when they need it. We don't know when they may be listening. If we top propagation we can prevent the application from having access to user generated events that they may depend on.
In this case, what @Jinnie is recommending is that the application handle listening for the escape event on the application side. This leaves the application in control of when event.stopPropagation() is used. If Clarity does it, a different app that is listening for the esc event will never hear it.

Thanks @hippee-lee - that's exactly the case - the user should decide (and has the means to do it, naturally, without using a hackish solution) if the event propagation must be stopped.
The supplied stackblitz is an example, it's not necessary to be implemented exactly this way. Some user-side logic is needed to determine when to stop and when to pass the event. This logic may vary, I tried to make a Directive that can probably work in most cases.

Thanks for elaborating!

@mathi123 - in case it is not clear ... in this case the user Ivan is referring to is the user of Clarity aka the application 😉.

Update: after 4.0.12 and 5.0.2 the solution from the stackblitz example should work with Combobox too. The erroneous event has been fixed.


fix(curriculum): adding explanation for indexOf in music player project

Summary of changes
Added a new step for introducing the indexOf method since we removed the use of the indexOf method in the RPG. This will be first time campers will learn about this method so it needs an introduction to it.
Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #52926

I just marked the other PR as blocked since I think this one should go in first.
Since we removed the explanation for indexOf in the RPG project, technically there is no lesson to teach it right now. This PR fixes that.

Hey Jessica 👋🏽
The songs in the last step (Now 85) should be by Quincy, not Rafa:

The songs in step 84 (the last step before this PR) are Quincy's:

HI @Ksound22 !
I am having trouble reproducing what you are seeing on your end.
On my end the song order is displaying correctly
Here is what I am seeing for step 84

Here is my step 85

Here are the meta.json files
step 85
step 84

HI @Ksound22 !
I am having trouble reproducing what you are seeing on your end.
On my end the song order is displaying correctly
Here is what I am seeing for step 84
Here is my step 85
Here are the meta.json files
step 85
step 84

I thought of it as an issue on my end and decided to clear my browser ccokies and cache. Everything is now as it is on your end.


What is transaction in react?

They are just a pattern the React team has adopted to make the framework more robust and easier to maintain.


Mysql and BLOB column

Setup
Versions

Rust: rustc 1.19.0
Diesel: 0.15.2
Database: Mysql
Operating System OSX

Feature Flags

diesel: mysql
diesel_codegen: mysql

Problem Description
When using infer_schema! on a table that includes a BLOB column compilation does not complete successfully.
What are you trying to accomplish?
Using a struct of the following definition:
#[derive(Queryable)]
struct RSAKeyPairData {
    pub id: i64,
    pub private_key_pem: Vec<u8>,
    pub certificate_pem: Vec<u8>,
    pub published: bool,
    pub active: bool,
}

What is the expected output?
Valid compilation.
What is the actual output?
error[E0412]: cannot find type `Blob` in this scope
 --> src/schema.rs:5:1
  |
5 | infer_schema!("dotenv:DATABASE_URL");
  | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ not found in this scope
  |
  = help: there is an enum variant `diesel::mysql::MysqlType::Blob`, did you mean to use `diesel::mysql::MysqlType`?
  = note: this error originates in a macro outside of the current crate

Are you seeing any additional errors?
No.
Steps to reproduce
CREATE TABLE rsakeypairs (
  id SERIAL PRIMARY KEY,
  private_key_pem BLOB NOT NULL,
  certificate_pem BLOB NOT NULL,
  published BOOLEAN NOT NULL DEFAULT 0,
  active BOOLEAN NOT NULL DEFAULT 0
)

Use infer_schema! to trigger.
n.b. diesel print-schema outputs:
table! {
    rsakeypairs (id) {
        id -> Bigint,
        private_key_pem -> Blob,
        certificate_pem -> Blob,
        published -> Bool,
        active -> Bool,
    }
}

Manually supplying the following table! definition as opposed to using infer_schema! removes the compilation fault and all appears to function correctly, I am not sure if Binary is correct here (though from docs it seems to be):
table! {
    rsakeypairs (id) {
        id -> Bigint,
        private_key_pem -> Binary,
        certificate_pem -> Binary,
        published -> Bool,
        active -> Bool,
    }
}

Checklist

[x] I have already looked over the issue tracker for similar issues.

Thanks for the report! Seems like this is just a small missing piece. And @Eijebong already opened #1090 to fix it :)

Can't argue with a response as quick as that!
Thanks folks :).


Implement multi-kernel XCLBIN

This issue is an amd-aie backend counterpart of upstream issue https://github.com/iree-org/iree/issues/7824
We will initially follow the SPIR-V/vulkan commit sequence below

https://github.com/iree-org/iree/pull/15785
https://github.com/iree-org/iree/pull/15788 (bug fix vibes will merge with 1)
https://github.com/iree-org/iree/pull/15789
https://github.com/iree-org/iree/pull/15802

Once this is done the state will be that we will have a XCLBIN archive that contains multiple XCLBINs per executable.
Then in the last stage we will use the utility provided here, https://github.com/Xilinx/mlir-aie/pull/1508
to merge the XCLBIN's and achieve the desired multi-kernel XCLBIN

I did some profiling on what we get by having a XCLBIN with multiple PDIs. My test example had three kernels and I measured the loading time in the following setups

Make a new xclbin for each kernel and load the xclbin and then the corresponding kernel from it.
Make one xclbin for all three kernels but load it three times, once for each kernel.
Make one xclbin, load it only onces and get the three kerenels from it.

Here are the times I saw for small GEMM shapes (M,N,K = 32,32,32) and large GEMM shapes (M,N.K = 1024,1024, 32)

Setup
Small
Large

1
19.82 ms
21.66 ms

2
24.11 ms
22.37 ms

3
19.88 ms
21.53 ms

I think it can be concluded that the main bottleneck is getting the kernel from the xclbin itself and merging to one xcdlbin isnt changing the time taken to load the kernels much.

@kumardeepakamd please see the findings above. Looks like we need to do some different pdi loads to actually benefit from xclbin merging?


fix: start charge service

fix start charge service currently sending stop charge to car

Thanks a lot, looks good to me


None of the configured keyfiles exists: ~/.ssh/id_rsa, ~/.ssh/id_dsa

Default key locations specified in SshLogin.scala are ~/.ssh/id_rsa, ~/.ssh/id_dsa but the tilde is a shell specific expansion and not applied, resulting in the following exception when using the default key locations:
java.lang.RuntimeException: None of the configured keyfiles exists: ~/.ssh/id_rsa, ~/.ssh/id_dsa
```

Replacing the tilde with System.getProperty("user.home") would be better.

The tilde is already expanded, see here:
https://github.com/sirthias/scala-ssh/blob/master/src/main/scala/com/decodified/scalassh/HostConfig.scala#L121

If anyone is having this error, for me it was because these files under macOS were encrypted. I had to use the AgentLogin instead (which depends on jsch.agentproxy.sshj and jsch.agentproxy.connector-factory). Then it worked well.


[INTEGRATIONS-0.5] Integrate LLMs of Snowflake as Model

https://docs.snowflake.com/en/developer-guide/snowpark-ml/reference/1.5.0/api/model/snowflake.cortex.Complete
Deliverable

Open source plugin with a Model wrapper for the LLM.

Important links for reference
https://docs.snowflake.com/en/user-guide/snowflake-cortex/llm-functions#using-snowflake-cortex-llm-functions-with-python
https://docs.snowflake.com/en/developer-guide/snowflake-ml/snowpark-ml#label-snowpark-ml-get-started


feat: remove more instaceof checks

[x] Added or updated tests
[ ] Documented user facing changes

Changes
This PR gets rid of more instanceof checks thanks to the new methods in PHPStan 1.10
Fixes #1567
Fixes #1557
Breaking changes
N/A

What should I do to kill last instanceof in src/Rules/ModelRuleHelper.php?

isGenericObjectType method needs to be implemented in PHPStan for that.


[BUG] DeltaTableBuilder.execute() sometimes throws "Could not find active SparkSession"

Bug
Describe the problem
Steps to reproduce
import io.delta.tables.DeltaTable
import org.apache.spark.sql.{Encoders, SparkSession}

import java.nio.file.{Files, Path, Paths}

val tempDir: Path = Files.createTempDirectory("scratch")
val warehouseDir: Path = Paths.get(tempDir.toString, "warehouse")
val tableDir: Path = Paths.get(tempDir.toString, "table")

val spark = SparkSession.builder()
  .appName("scratch")
  .master("local[*]")
  .config("spark.local.ip", "<IP_ADDRESS>")
  .config("spark.driver.host", "<IP_ADDRESS>")
  .config("spark.driver.bindAddress", "<IP_ADDRESS>")
  .config("spark.sql.warehouse.dir", warehouseDir.toString)
  .config("spark.sql.extensions", "io.delta.sql.DeltaSparkSessionExtension")
  .config("spark.sql.catalog.spark_catalog", "org.apache.spark.sql.delta.catalog.DeltaCatalog")
  .getOrCreate()

case class Table(id: Long, text: String)

val schema = Encoders.product[Table].schema

val tableBuilder = DeltaTable.create(spark)
  .tableName("Table")
  .addColumns(schema)
  .location(tableDir.toString)

// sometimes throws org.apache.spark.sql.delta.DeltaIllegalArgumentException: Could not find active SparkSession
// even though a session has been passed into the DeltaTable.create(spark) statement
tableBuilder.execute()

Observed results
We are running some integration tests of our Spark jobs against some Delta tables which are created locally, on the fly, in the integration tests. The tables are stored on disk. The code above shows a (simplified) snippet of how we setup the Delta tables on disk for testing.
While running those test we sometimes (not always) observe that calling DeltaTableBuilder.execute() for the Delta table that we create in the tests sometimes throws an exception: org.apache.spark.sql.delta.DeltaIllegalArgumentException: Could not find active SparkSession.
This happens despite us passing the SparkSession into the DeltaTable.create(spark) method.
Expected results
If a SparkSession is passed into DeltaTable.create(spark), the exception I described above should never be thrown, since an active session exists and has been passed into the builder.
Further details
Stack trace:
org.apache.spark.sql.delta.DeltaIllegalArgumentException: Could not find active SparkSession
at org.apache.spark.sql.delta.DeltaErrorsBase.activeSparkSessionNotFound(DeltaErrors.scala:2038)
at org.apache.spark.sql.delta.DeltaErrorsBase.activeSparkSessionNotFound$(DeltaErrors.scala:2037)
at org.apache.spark.sql.delta.DeltaErrors$.activeSparkSessionNotFound(DeltaErrors.scala:2293)
at io.delta.tables.DeltaTable$.$anonfun$forName$1(DeltaTable.scala:680)
at scala.Option.getOrElse(Option.scala:189)
at io.delta.tables.DeltaTable$.forName(DeltaTable.scala:680)
at io.delta.tables.DeltaTableBuilder.execute(DeltaTableBuilder.scala:369)

From the stack trace we can see that the problem is here: https://github.com/delta-io/delta/blob/2041c3b7138b9614529540613195d4afd39fde71/core/src/main/scala/io/delta/tables/DeltaTableBuilder.scala#L367-L371
A possible solution could be to always forward the SparkSession object that is present in the DeltaTableBuilder to DeltaTable.forPath() and DeltaTable.forName() - they both have an optional parameter that we could use. Maybe it could be changed to something like this:
if (DeltaTableUtils.isValidPath(tableId)) {
  DeltaTable.forPath(spark, location.get)
} else {
  DeltaTable.forName(spark, this.identifier)
}

Environment information

Delta Lake version: 2.1.1
Spark version: 3.3.0
Scala version: 2.12.14

Willingness to contribute
The Delta Lake Community encourages bug fix contributions. Would you or another member of your organization be willing to contribute a fix for this bug to the Delta Lake code base?

[X] Yes. I can contribute a fix for this bug independently.
[ ] Yes. I would be willing to contribute a fix for this bug with guidance from the Delta Lake community.
[ ] No. I cannot contribute a bug fix at this time.

Good catch. Yep we should just use the SparkSession object in DeltaTableBuilder to create DeltaTable. Feel free to submit a PR to fix that.
By the way, do you know why active SparkSession is not available in your tests?

By the way, do you know why active SparkSession is not available in your tests?

No, I don't. It's very weird, when we are running the integration tests locally on our computers the tests work just fine. However, in CI (we're using Azure DevOps), the tests fail a lot, but sometimes succeed. I feel like it could be some race condition somewhere since SparkSession.getActiveSession() returns the active session per thread? Not sure.

@zsxwing I published a draft PR here: #1476. It was easy to fix but maybe someone can provide me some guidance on the testing, is there any test I could add anywhere? It's my first PR here

Same issue during tests. Looking forward to having this fixed!

@pgrandjean fyi Delta 2.2.0 was released which includes the fix. I upgraded the dependency and the tests now work in CI 🥳

awesome! thank you 👍


Add Mastodon link

The rel="me" allows for the "verified" checkmark on Mastodon.
This breaks the nice markdown formatting though. Not sure how much you care about that. If you want, I can try to find a more Markdown-idiomatic way of doing rel="me" if you don't know one off hand.
The Mastodon account is set up for now, so there's at least something to see when you click on this new link. I don't want to add the link back from the Mastodon account to bostonopen.dev until this link is live on bostonopen.dev, so I can confirm right away that the "verified" checkmark works. (Otherwise I don't know how long it'll take for the Mastodon server to check again for the rel="me" tag)
https://floss.social/@bostonopen

Ha! This looks great! Thanks, @orblivion!

This breaks the nice markdown formatting though.

Visually, it looks the same, which is great:

I'm fine with a little HTML in my Markdown. 😄


bump cinder deployments to v0.2.0

bump cinder deployments to v0.2.0

Approved, but I think this will be removed since cinder work is now handled by the openstack community in their repo.


disable orientation for hmd tracking controller

Hi,
My controller just got broken and showing same direction, it seems only direction works but no gravity, and just shaking... so pretty useless as controller but still works as hmd tracking. The problem is that even tough it points to one direction, it is shaking continuously which translates to shaking of view.  Therefore I would like to introduce parameter to totally ignore its orientation so I will avoid shaking. Perhaps as another type of orientation filter?
Please let me know if you agree.
Anyway its interesting. In one day, one controller got gravity broken (orientation works). Other controller I need to re-calibrate orientation each time. If I re-run config tools it is rotated 45 degrees left. It was working almost from the start in past. I have not done any upgrades recently...
Anyway I think that even if I have working controller I can still benefit from disabling orientation on hmd controller.

I think you might be able to get the effect you want with the "PassThru" filter. That filter just takes the raw optical orientation and returns it back out without any filtering. Since the PSMoveController has no optical orientation it should just return the identity orientation constantly. If that doesn't work I'm fine with adding "disabled" filter.

it sound like what I just described :) otherwise what could it be that PassThru is doing based on ur description :)
i should check first what's available. thanks.

yups. PassThru does the work...


💡 [REQUEST] - deepseek-coder-6.7b-base

Model introduction
deepseek-coder-6.7b-base is a 6.7B parameter model with Multi-Head Attention trained on 2 trillion tokens.
Model URL
https://huggingface.co/deepseek-ai/deepseek-coder-6.7b-base
Data type
float16
Additional instructions (Optional)
No response
Author
Yes
Security

[X] I confirm that the model is safe to run which does not contain any malicious code or content.

Integrity

[X] I confirm that the model comes from unique and original work and does not contain any plagiarism.

@guoday Thanks for the request! Please find our reproduced results below.

Note that we use bf16 and 512 max new tokens (similar to how we evaluate other models listed in our leaderboard (https://evalplus.github.io/leaderboard.html). Only 3 out of 164 produced programs have unfixable syntactical errors such as infinite enumeration shown blow:

Let us know if you have further questions. Thanks!

Thanks @ganler for evaluation. Could update the results in the leaderboard? Thanks.

I have just updated it on the leaderboard. :)


Control reaches end of non-void block

(NSURLSessionDownloadTask *)downloadTaskWithRequest:(NSURLRequest *)request
progress:(void (^)(NSProgress *downloadProgress)) downloadProgressBlock
destination:(NSURL * (^)(NSURL *targetPath, NSURLResponse *response))destination
completionHandler:(void (^)(NSURLResponse *response, NSURL *filePath, NSError *error))completionHandler

@Gaozn  can you please provide some information ??
It will help us in better understanding this problem...

sorry，忘记return个NSURL *


Remove spurious log that adds no value and make ITs fail sometimes because of an additional line

sometimes fail because of an additional line.
Signed-off-by: David Gageot<EMAIL_ADDRESS>

LGTM


MNT: Remove Travis CI

I see Azure already running, so no reason to run Travis CI anymore?
Fix #103

Codecov Report

Merging #108 (56d9540) into master (0fd2dc3) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #108   +/-   ##
=======================================
  Coverage   83.22%   83.22%
=======================================
  Files          17       17
  Lines         930      930
=======================================
  Hits          774      774
  Misses        156      156

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 0fd2dc3...56d9540. Read the comment docs.

I'm out of the loop here, but I suppose this is all good.


HttpHandler is inferred to be generic when in a different file

Starting from the dotnet Giraffe template, if I try to define any HttpHandler outside of the Program.fs file, the compiler complains that my HttpHandler functions are generic. Which is weird because code lens shows the correct signature.

If I annotate every thing to be HttpHandler it compiles fine, but I'm confused why I am required to do that here.

This is weird indeed. I don't have a good explanation for it, but I will annotate all http handlers in the Giraffe library, which should fix this issue I think. It won't hurt to explicitly annotate all existing http handlers.

I tried with the updated version and seeing the same behavior on curried HttpHandlers.... I'm playing around with it from the source and still stumped. Thankfully there are easy workarounds

I moved the auth related functions of the SampleApp into a separate file, and it seems to only be a problem with HttpHandlers that have another handler as an argument

Cool.. thanks for drilling down to the bottom of this issue!


Contact surface is too thin when draging on the slider

I want to make a slider as spotify and that can be easily draggable on phones or other devices but here i tried to increase the touchable area of the slider without increasing the size of the slider itself by using the panHitSlop property but it doesn't increase it as the slider on spotify and others. Do you have an idea ?
This is my component and he is rendered in a view
    <Slider
      theme={{
        maximumTrackTintColor: "#A0A0A0",
        minimumTrackTintColor: "#fff",
      }}
      containerStyle={{ borderRadius: 5, height: 4 }}
      style={styles.slider}
      progress={progress}
      minimumValue={min}
      maximumValue={max}
      onValueChange={onSliderValueChange}
      thumbTintColor="#1DB954"
      panHitSlop={{ top: 20000, bottom: 20000, left: 20000, right: 20000 }}
      renderBubble={() => null} // Return null to avoid rendering the bubble
      setBubbleText={() => {}} // No-op function to avoid setting text in the bubble
    />

Do you have any touchable views around your slider? Be sure that doesn't happen around that.

It works for me, so I will close it.
https://github.com/user-attachments/assets/1bdcb227-2c36-4386-947a-f31fa7714e24


sync thrift to final crypto spec

Codecov Report

Merging #4022 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4022      +/-   ##
==========================================
+ Coverage   87.83%   87.83%   +<.01%
==========================================
  Files         734      734
  Lines       90433    90433
  Branches     1252     1252
==========================================
+ Hits        79429    79432       +3
+ Misses      10883    10880       -3
  Partials      121      121

Impacted Files
Coverage Δ

cpp/src/arrow/io/readahead.cc
95.95% <0%> (-1.02%)
:arrow_down:

cpp/src/arrow/util/thread-pool-test.cc
97.66% <0%> (-0.94%)
:arrow_down:

cpp/src/plasma/thirdparty/ae/ae.c
71.69% <0%> (+0.94%)
:arrow_up:

js/src/ipc/metadata/json.ts
96.73% <0%> (+4.34%)
:arrow_up:

Continue to review full report at Codecov.

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

Can you create a JIRA issue for this?


advice for efficient selection change tracking

Suppose you have several immutable items with unique IDs and I would like to efficiently track a single selection scenario of them without requiring to add an isSelected property to each item (since the items are immutable). The items are displayed as cards in a React UI. By efficient, I mean, only the card of the selected or unselected item should be updated, not all others.
I first tried something like this:
const selectedItem = mobs.obervable('');

const setSelectedItem = (id = '') => { selectedItem(id); }
const isSelectedItem = (id) => { return (selectedItem() === id); }

and then used the isSelectedItem(id) within a mobs.observer decorated React component for each of the items.
The problem is, that the isSelectedItem(id) fires on each change of selectedItem for all item cards, selected or not.
I would like to have a mechanism to turn the isSelectedItem(id) observable to only fire if there is an actual change for the returned value, not each time the selectedItem changes.
Maybe there is already a simple mechanism for this, I haven't found so far?

Actually, what I am asking for is more or less the same optimization you do when assigning the same value to an observable that you don't trigger an update event. The same should be true for derived values, IMHO: if the derived value doesn't change the derived value shouldn't trigger an update event either...

to answer your first question, the solution to that is to generate observables on the fly, for example:
render() {
   var isSelected = observable(() => selectedItem === this.props.item)
   if (isSelected()) {
     // do something
   }
}

You will notice if you add a debugger statement, that changing selectedItem will cause the isSelected observable to re-evaluate, but the render (which is the expensive one) will no longer be evaluated.
Since instantiating and inspecting observables on the fly is quite a common pattern, there is a convenience method for it, expr(function) (sugar for observable(function)()) so that you can write the above as
render() {
   if (expr(() => selectedItem === this.props.item)) {
      // item of this component is selected
   }

I hope that answers your question! Otherwise just let me know

OK, thanks for the quick help.
Yes, I can confirm that adding mobs.expr() like this in the example above produces the desired output:
    const track = (len) => mobs.expr(() => (v().length === len));

Your lib is incredible!


Create dual_core-vodka_redbull.md

Cannot tell if my submissions are making it in.

On Feb 5, 2017, at 3:24 PM, int0x80<EMAIL_ADDRESS>wrote:
You can view, comment on, or merge this pull request online at:
https://github.com/theDevilsVoice/1337-Noms-The-Hacker-Cookbook/pull/12 https://github.com/theDevilsVoice/1337-Noms-The-Hacker-Cookbook/pull/12
Commit Summary
Create dual_core-vodka_redbull.md
File Changes
A drinks/Dual_Core-Vodka_Redbull/dual_core-vodka_redbull.md https://github.com/theDevilsVoice/1337-Noms-The-Hacker-Cookbook/pull/12/files#diff-0 (12)
Patch Links:
https://github.com/theDevilsVoice/1337-Noms-The-Hacker-Cookbook/pull/12.patch https://github.com/theDevilsVoice/1337-Noms-The-Hacker-Cookbook/pull/12.patch
https://github.com/theDevilsVoice/1337-Noms-The-Hacker-Cookbook/pull/12.diff https://github.com/theDevilsVoice/1337-Noms-The-Hacker-Cookbook/pull/12.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/theDevilsVoice/1337-Noms-The-Hacker-Cookbook/pull/12, or mute the thread https://github.com/notifications/unsubscribe-auth/AHpLH30Wuc2k8PR1Fg3PbrOaj2HZwk32ks5rZi__gaJpZM4L3nRD.


BUG - register open generic pipeline

Hey!
I have pulled the newest 12.00 MediatR version into my modular monolith application.
Previously, I registered my validation pipeline as the code:
    public static IServiceCollection AddPipelinesModule(this IServiceCollection serviceCollection) =>
        serviceCollection.AddScoped(typeof(IPipelineBehavior<,>), typeof(ValidationPipeline<,>));

After migration, the validation doesn't work correctly. I have checked the lib methods and transformed the upper piece of code into
builder.Services
    .AddMediatR(cfg =>
    {
        cfg.AddOpenBehavior(
            typeof(ValidationPipeline<,>));
        cfg.RegisterServicesFromAssembly(typeof(Program).Assembly);
    });

But still without success.
Btw. when I have skipped RegisterServicesFromAssembly(typeof(Program).Assembly); the library throws an exception - no assemblies to scan. It's problematic because a program is only a shared entry point to my app.
My app looks like this:
WebApi - Controllers, Entrypoint,
ModuleA - Logic for A, Command handlers,
ModuleB - logic for B, Bommand handlers,
etc.

I have the same problem in my application.

Have a repro? The tests look fine here.

Looks like it happens when I have IRequestHandler implementation without returning anything.

Ok, I Found it.
It happens when Pipeline implementation looks like
public class ValidationPipeline<TRequest, TResponse> :
    IPipelineBehavior<TRequest, TResponse>
    where TRequest : IRequest<TResponse>

When I changed into
public class ValidationPipeline<TRequest, TResponse> :
    IPipelineBehavior<TRequest, TResponse>
    where TRequest : IRequest

Void pipelines works correctly, but....
When handler return something, in the upper case not working.
So, currently to hack it, I must have 2 pipelines instead of 1.

Did you follow the upgrade guide?

I added a big bold message to also correct your behaviors:
https://github.com/jbogard/MediatR/wiki/Migration-Guide-11.x-to-12.0
That will fix your issue.


stdenv: Store one package per line in nix-support/propagated-*

Motivation for this change
This makes those files a bit easier to read. Also, for what it's worth, it brings us one tiny step closer to handling spaces in store paths.
Also, I optimized handling of many transitive deps with read. Probably, not very beneficial, but nice to enforce the pkg-per-line structure. Doing so let me find much dubious code and fix it.
Two misc notes:

propagated-user-env-packages also needed to be adjusted as sometimes it is copied to/from the propagated input files.

local fd should ensure that file descriptors aren't clobbered during recursion.

Things done
Built stdenv on Darwin and Linux

Spaces in store paths can never possibly work since you can't escape a space in a #! line (and probably one zillion similar places).

@dezgeg That's true. I guess take this PR as a matter of style.

OK, this is tested.

Seems fine to me, though I don't see a huge advantage - these files are not intended for human consumption so the readability is not really important, and as @dezgeg mentioned, whitespace is unlikely to ever work well (also this solution wouldn't support newlines in filenames).

@edolstra Thanks for the verdict. TBH this was basically me getting side-tracked when working on that previous PR, but I do occasionally read those files by hand---at least when debugging.

printLines broke package propagation. Please, revert.
See #27427 for discussion.


iwd: 0.20 -> 0.22

Motivation for this change
This bumps iwd to 0.22, which finally supports certificate domain matching.
Switched my systems' NetworkManager wifi backend to iwd via
networking.networkmanager.wifi.backend = "iwd";

Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @NixOS/backports for backporting

@flokli Turns out that changelog is not out of date, just a mixup because of ell.

iwd changes
ver 0.22:
	Fix issue with hotplug and device capability detection.
	Add support for IPv6 network configuration handling.
	Add support for embedding certificates in profiles.

ver 0.21:
	Fix issue with cleartext ACK and tunneled EAP-Success.
	Fix issue with setting bits in extended capabilities.
	Fix issue with WPA-1 information element parsing.
	Add support for handling QoS Map IE and frames.
	Add support for STATE_DIRECTORY environment.
	Add support for CONFIGURATION_DIRECTORY environment.

ell changes
ver 0.24:
	Add support for extended groups in settings files.

ver 0.23:
	Add support for checking if uintset is empty.

ver 0.22:
	Fix issue with DHCP and T1 timeout handling.
	Fix issue with escape characters in peer identity string.
	Add support for setting peer certificate's subject CN mask.

Backporting doesn't really match the policy of security fixes and bug fixes, but it's pretty much experimental. I know with each release I notice it being less broken for people.

@GrahamcOfBorg build ell, iwd

backported to 19.09 in 76da71f5eb6, e1607586a99.

@flokli module update should be fine too :+1:

you mean https://github.com/NixOS/nixpkgs/pull/68147?


Matching engine order status management

#763 exposed a weakness in current order status management via matching engine.
Looks like some statuses are mixing two different things: order status in matching engine and order status based on the executed volume. As such matching engine should not manage status for executed volume anymore, it's sufficient to have updated volume to operate.
Removal of volume based statuses allows us to implement order reactivation required by demand pause/resume in the correct way.

This issue has been already fixed.


Some images are rotated in site build vs. how they are on filesystem

What happened?
 On my filesystem they are all in the correct orientation.
I've tried rotating them and rebuilding the site but they remain in the same orientation. It's also a mix of portrait and landscape images.
Github repo for that post.
Any suggestions? Thanks.
Theme version
v2.4.2
Hugo version
v.0.108.0+extended
What browsers are you seeing the problem on?
Firefox, Chrome, Safari
Relevant Hugo log output
INFO 2022/12/19 22:21:51 syncing static files to /
LOG:   2022/12/19 22:21:51 Built in 112 ms
LOG:   2022/12/19 22:21:51 Watching for config changes in /home/alex/dev/alexjj.com/config/_default, /home/alex/dev/alexjj.com/config/_default/resources, /home/alex/dev/alexjj.com/config/_default/resources/_gen, /home/alex/dev/alexjj.com/config/_default/resources/_gen/assets, /home/alex/dev/alexjj.com/config/_default/resources/_gen/images, /home/alex/dev/alexjj.com/go.mod

I'm not sure what would be causing this. The theme only uses standard Hugo image processing commands and so it shouldn't be rotating the images. I suspect this might be some sort of metadata issue in the images that is telling them to change orientation when Hugo is processing them.


Fix rendering in table with empty cells

This is for the 3.0 branch.
find_emph_char returns 0 if the char specified is not found in the current line, but this is also what happens when there's an empty cell. This patch adds logic to work around this problem.
Tests added for general table syntax validation. The last example in Table.text addresses the problem mentioned.
See uranusjr/macdown#321.

Thanks!


[Https][Tooling] Add digital signature to the KeyUsage of the HTTPS dev-cert

No, they will have to remove the very from the store and then recreate it.
We don’t want this to affect people with existing certs that don’t need it.
Specially if they trusted the cert already (it would require them to trust it again)
If we see that this becomes a problem then we can think of auto-upgrading

My concern is that if you're running Windows 10, you'll reboot one day and your ASP.NET Core app will be broken, even if you're up-to-date on the latest SDK. That's a bad experience.

@anurse The counter point is that if we upgrade it automatically, then you install a new SDK in your box and it breaks the trust relationship with your cert. (And we can't remove things from the trust store without prompting, which is also annoying).
The certs lasts for 1 year and they get automatically renewed after that, so when its time you'll get an updated cert.
Lets see if this becomes a problem and then we can decide what to do.

Fair points, I'm fine getting customer feedback on this :)


[Perf] Micro Benchmarking as part of the perf framework

Originally reported by @mikeharder
Context - https://github.com/Azure/azure-sdk-for-js/pull/20283 adds a perf improvement to the core-rest-pipeline, along with a perf test to measure the improvement.
Performance improvements such as https://github.com/Azure/azure-sdk-for-js/pull/20283 can be tested with some sort of micro-benchmarking tools/functionality support instead of having to add a new perf test.
Mike also mentions - In .NET, for our end-to-end perf tests we use the home-grown perf framework, but for micro-benchmarks we use a different third-party tool called Benchmark.NET since it's better suited for micro-benchmarks.
This issue is to track the work to study the existing micro-benchmark tools in JS/node or to think of approaches to add more micro-benchmarking features to the JS perf framework.

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Azure.Core:0.08831879,Network - Virtual Network:0.073076226,Docs:0.05010764'

If we can find an actively-maintained project in JS/TS similar to Benchmark.NET, we should compare it to our internal perf framework.  Otherwise, I think we should use our internal framework for both end-to-end and micro benchmarks, and add features to our internal framework for micro-benchmarks as needed.


Discussion on Choice of License

MIT License
We considered different package licenses and agreed that the MIT license is the most appropriate for both Python and R versions of the snapedautility package. There are a few reasons why the MIT license was selected.
First and most importantly, we want to ensure that the package can be used, distributed, and published. In short, we promote this package to be open-source. The purpose of this package is not to generate profits, but its purpose is solely to help the users to make efficient use of their time when conducting exploratory data analysis.
Second, we want to allow users to extend this package in a larger software. Under the MIT license, users can copy and modify this package to make it appropriate for their use cases. Users are also allowed to sell their products even after using or extending the snapedautility package.
Third, including the MIT license ensures that we are free from legal matters. As we aren't looking to generate profit off this package, we want to avoid being liable for any consequences that might come with potential errors. Under the MIT license, we should be protected from being legally responsible for any problems that users' software may encounter.
Although there are other similar licenses such as "Boost Software License 1.0" and "Apache License 2.0", these licenses have some constraints that may restrict users from using this package at its full potential. For example, the Apache license offers the same permissions as the MIT license except for one circumstance. The Apache license requires developers to disclose any major changes made to the source code, whereas the MIT license does not require any disclosures.
Since the MIT license is the most unrestricted license that protects us from legal matters, we believe that the MIT license is the most suitable license for the snapedautility package.
Reference: https://choosealicense.com/licenses/

Good!


Vaadin 22 compatibility

As Vaadin 22 introduced a lot of changes on how a textfield is composed the add-on does not seem to be working correctly anymore.
https://vaadin.com/docs/latest/ds/upgrading/#changes-in-vaadin-22
When you insert numbers they are not automatically formatted anymore. It just looks like a normal textfield.
However loading seems to be working fine. The number is rendered correctly.
Example: Left -> loaded, Right -> typed in

Yes, not working with Vaadin 22

Experienced the same issue when testing with V23 pre-release. My error is:
Uncaught TypeError: Cannot read properties of null (reading 'length') (webpack-internal:///../node_modules/cleave.js/dist/cleave-esm.js:1074:26)

V22 should be fixed in 5.4.0

@johanneshayry  Thanks for fixing (and maintaining) this awesome Vaadin extension. Already deployed the new version. Working like intended again.


Updated tutorials and attacks

I have updated the tutorials that use keras, and the attacks to use the new abstraction. Further, as discussed in the group, I have added support for the older model by setting a new attribute called get_probs(x).

Thanks. The changes look good so far. Let's see what the decision on call would be. Then if needed we can update the tests as well.

I am actually going to close this, and use a class Wrap instead of setattr as discussed, instead of adding extra commits.


JPG read error on some photos, which can be viewed by normal tools with no problem

use jimp read function like below, but got error.
jimp.read(targetjpg, function (err, demo) {
if (err) throw err;
if ((demo.bitmap.width==640 && demo.bitmap.height==480)
........

some JPG photo (photo taken by different devices, android camera, micro controller controlled camera)
,which can be view by normal picture tool, but can not be read by Jimp, the error message is "if (err) throw err;  ReferenceError: width is not defined"
Not all photo can not be read, but only a small portion
Failure Information (for bugs)
ReferenceError: width is not defined

Steps to Reproduce

Context

Jimp Version:
Operating System: centos 7
Node version: v8.9.4

Failure Logs

@KangarooBee what jimp version are you on?

@hipstersmoothie Thanks for the reply!
I rechecked the code, there is some variable mismatch in my program. Due to unfamiliar with the call back method, thought it is a Jimp problem.  As you said, Jimp works fine.
My bad, for this issue reporting. I am closing it. Or you can delete it.
Thank you again for your time and contribution!


allow (DTS-HD Master Audio) to be valid

mkvmerge was giving "Track ID 1: audio (DTS-HD Master Audio)" which is OK, but the script was requiring "(DTS)" in the string. Removing the closing paren allows it to continue. The audio is fine in the output file.

OK, should have checked other PRs first :) I'll leave this one here with the others.

Yeah, when we were actively maintaining this we would have to change the string fairly often as the underlying tools would change their format and our matching string would stop matching.  Coincidentally we originally had it as DTS, then (DTS and had to go to (DTS) for matching reasons at that time.  Now it's changed back to needing a (DTS* concept but hopefully it won't match odd corner cases which won't work.