Add an open-source LICENSE

A project needs an open-source license to be classified as open-source and to allow people to download, run, contribute, etc, without specifically requesting permission.

Has been added!


Get a non-Sybil model from the OP data team to use in the Trusted User model

We should include an address model that filters out Sybils, prepared by the OP data team

See draft spec here. This is likely more ambitious than what's possible for RF4, but a great vision of the types of user fields we should work towards including!


Partial issues with metamask while running tests with "chromeWebSecurity: false"

It seems that metamask doesn't work well with Synpress if it's run with "chromeWebSecurity": false flag.
There seems to be some issues with connection between metamask and browser in this case.
In case of 1inch, I wasn't able to connect my wallet:

In case of kwenta, I was able to connect my wallet, but dapp couldn't read any balances from it.

Everything works fine with "chromeWebSecurity": true.
https://docs.cypress.io/guides/guides/web-security.html
Couldn't find anything helpful in this topic, only issues without answers, like this one:
https://github.com/MetaMask/metamask-extension/issues/6817

It seems that metamask doesn't work well with Synpress if it's run with "chromeWebSecurity": false flag. There seems to be some issues with connection between metamask and browser in this case.
In case of 1inch, I wasn't able to connect my wallet:
In case of kwenta, I was able to connect my wallet, but dapp couldn't read any balances from it.
Everything works fine with "chromeWebSecurity": true. https://docs.cypress.io/guides/guides/web-security.html
Couldn't find anything helpful in this topic, only issues without answers, like this one: MetaMask/metamask-extension#6817


the per-rationale accuracy is more higher than the per-problem accuracy

But why is the per-rationale accuracy higher than the per-problem accuracy?
When I ran your project and generated rationales using LLMs, my per-rationale accuracy was similar to my per-problem accuracy. Additionally, when I ran preprocess_cot.py, I obtained fewer correct chains of thought (CoTs) compared to your results.
Could you please explain how you generated the rationales using LLMs?Thank you.

Thank you for your follow-up question! One possible cause could be that we use google/flan-t5-large for rationale filtering.
For rationale generation with LLMs, we apply the prompt provided in Table 12 of the Appendix.
I hope this helps clarify your question!


Support dict[str,str] parameters (support for arbitrary dict is not going to be there)

For the record, this is strictly required for the new version - not because of dict[str,str], but because of the input_specs and output_specs of the napari_workflows_wrapper task in fractal-tasks-core.

Re-opening, to keep track of:

[ ] Add feature for removing a key-value element


[vcpkg baseline][liblzf] Move header lzf.h to include/liblzf

Fix
Install port liblzf failed with conflict include/lzf.h installed by h5py-lzf.
Change
Update liblzf installation header lzf.h from include to include/liblzf.
Checklist

[x] Changes comply with the maintainer guide.
[x] SHA512s are updated for each updated download.
[ ] ~The "supports" clause reflects platforms that may be fixed by this new version.~
[ ] ~Any fixed CI baseline entries are removed from that file.~
[x] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

Test
Port usage tests pass with following triplets:

x64-windows

Maybe it is better to change h5py

Maybe it is better to change h5py-lzf. AFAIU

lzf there is an outdate vendored copy.
lzf there is not the primary (public) interface.

So if lzf cannot be devendored (to the new port) there, maybe the header can be moved to a subdir there.

Thanks for your suggestion, this is more appropriate. I will change the installation path of h5py-lzf to include/h5py

Originally I suggested to only move the conflicting header lzf.h, not the main header lzf_filter.h. But I start to realize that h5py-lzf is a pure vcpkg artifact. Upstream's aim is to provide an extension to HDF5, not liblzf. AFAIU register_lzf(void) is the only public interface of h5py-lzf. So the port probably shouldn't install lzf.h at all. Users who want the liblzf API must use the new liblzf port.

Update liblzf installation headers to include/{PORT}, and it was a workaround to avoid conflict.
Update h5py-lzf installation headers block with local test failed, I will fix this later.

See #37332.

Update liblzf installation headers to include/{PORT}, and it was a workaround to avoid conflict.
Update h5py-lzf installation headers block with local test failed, I will fix this later.

Thanks @dg0yt  for #37332, the incorrect dependencies of h5py-lzf has been fixed and local test pass.
So, closing this in favor of #37332


CB-10355: Add Python installation instructions to Readme.md file

Modified the Readme.md file to include instructions for Python
installation. This is done because Python is required for a successful
build of the cordova-docs project

@dblotsky @riknoll Can you please review these changes and accept the pull request?

LGTM but I'll default to @dblotsky on this since he knows more about the docs dev environment than I do.

Aside from the above comment,  LGTM.

@dblotsky Can you check and merge when you have time?

I just setup my machine and these instructions are accurate for windows


Loss increases during pretraining

Hi @alcinos, @ashkamath, @nguyeho7,
I hope you are doing good.
I was trying to pretrain MDETR using the provided instructions. What I noticed is that loss started increasing during the 20th epoch. It kept decreasing to around 39 till the 19th epoch and jumped to around 77 after the 20th epoch. What could be the reason for this? Note that I am using the EfficientNetB5 backbone. The log.txt is attached.
Thanks
log.txt

Hi @mmaaz60
Thank you for your interest in MDETR.
It looks like you training diverged. Can I ask how many gpus you used?

Hi @mmaaz60
Thank you for your interest in MDETR.
It looks like you training diverged. Can I ask how many gpus you used?

Thank You @alcinos,
I used 32 GPUs with batch_size of 2 per GPU.

Hum that’s quite surprising then. Nothing fishy happened, like the job getting preempted then restarted?
Are you sure you have the correct transformers version?
Otherwise mb try with a slightly smaller lr?

Thank You

Hum that’s quite surprising then. Nothing fishy happened, like the job getting preempted then restarted?

Nothing such happened during training

Are you sure you have the correct transformers version?

I am using transformers version 4.5.1

Otherwise mb try with a slightly smaller lr?

I actually stopped and then resumed the training from the 19th epoch and now it reaches to 25th epoch and seems to be converging. Not sure what went wrong previously as I didn't change anything when resuming.


Fix network handling

Resolves #297.
The current flow of network handling creates the awkward issue of an unresolved query in perpetuity when the node.json exists but the network key does not. This PR, therefore, proposes to add goerli2 and integration gateways to the node.json in the nile node command. This means that the goerli2 and integration gateways will be accessible irrespective of the node.json file's existence.

Agreed on centralizing file writes and with that, a refactor on get_gateways. Should we keep this patch for now or go for the refactor in this PR?

if you think it's small enough, let's squeeze it here. otherwise the patch + tracking the issue is enough.

if you think it's small enough, let's squeeze it here.

On it!

this PR is not solving the linked issue imo. Even when the error message differs from the last merges, the expected behavior, for me at least, is goerli2 transactions working by default (there are no docs telling users they need to add this manually to node.json afaik).

The patch does solve that^. The user doesn't have to add anything manually. The flow was that if a node.json exists, localhost and DEFAULT_NETWORKS would already be written in the json. If the node.json doesn't exist, it would be created with all three networks ready to go. Please correct me if I'm misunderstanding the comment.
This PR now (since after that review) doesn't write the DEFAULT_NETWORKS to the node.json. Instead, it follows the suggestion of returning the contents of node.json (which consists only of localhost) with the DEFAULT_GATEWAYS dictionary from common.py.

Issue resolution that I think could be solved by returning the DEFAULT_GATEWAYS in the get_gateways method even when they are not in the node.json file.
Replace node.json with nile.ini file with a networks config.

The latest pushed changes follow #1. The one minor issue with this flow is that a node.json will be created (if it doesn't already exist) when querying DEFAULT_NETWORKS.
If we want to go for #2, we should adjust the issue size or create a new issue :)

The patch does solve that^. The user doesn't have to add anything manually. The flow was that if a node.json exists, localhost and DEFAULT_NETWORKS would already be written in the json. If the node.json doesn't exist, it would be created with all three networks ready to go. Please correct me if I misunderstood the comment.

This is assuming users would follow the flow, but my concern was also with upgrades, like what happened to me: Dev already have a nile project, already having a node.json file, they try to use goerli2, cryptic error not documented.

If we want to go for #2, we should adjust the issue size or create a new issue :)

Totally agree. I think we should prioritize this as possible (not necessarily my proposed solution, but something to improve the network handling). cc @martriay

this is my middle ground suggestion:
let's not write any node.json. it's confusing pollution to the user, and it's information nile already knows. no more writing conflicts nor retrocompatibility issues.
let's still read node.json. if there's any user custom network, let's add it (or overwrite any default). this is the stem for an actual nile config file, but retrocompatible and it buys us time to design properly, while fixing this.
thoughts?

Just confirming. You mean not writing to the node.json file after creating it, but keeping all network configurations and reading from it, right? I'd keep all the networks in a network config, to allow changing the RPC providers (gateway_urls) if we have more than one at some point for the same network (I assume we'll have).

Just confirming. You mean not writing to the node.json file after creating it, but keeping all network configurations and reading from it, right? I'd keep all the networks in a network config, to allow changing the RPC providers (gateway_urls) if we have more than one at some point for the same network (I assume we'll have).

I mean not write/create any node.json files anymore, but take it into account if there's one already (e.g. with a custom network). this would be retrocompatible and fix the UX issue you point out, without us introducing any weird logic to overwrite/fix existing node.json files, and also provide a way for users to add their custom ones.
node.json would be the network file you suggest, my approach only simplifies it so we can ship a fix asap. this node.json file can eventually evolve into a full config file #73, but i'm leaving it for later to avoid the design + refactor time it entails.

Here's a few notes on the latest changes aiming to address the conversations regarding this PR @martriay @ericnordelo:

The default localhost now belongs in DEFAULT_GATEWAYS.
node.json will only be written when nile node receives a different host or port i.e. nile node --port 5001.

In the event that the localhost key points to a different value (this key/value entry would only exist in the node.json), the network will favor the node.json value.
When the node.json receives new entries, the previous entries will not be deleted. Should the node.json receive a duplicate key, however, the new value will overwrite the old.


[1.11.x] Fixed #27895 -- Fixed test client response.json crash with non-ASCII characters on Python 2.

Based on Aniruddha Maru's patch (ticket #27895).

Closing per ticket: the issue doesn't merit a fix on stable/1.11.x per our supported versions policy.


operators hive-operator (1.2.3716-d6fe109)

Update Hive community operator channel(s) [['alpha']] to 1.2.3716-d6fe109

/merge possible

/merge possible


Remote Desktop Web Client Windows button causes freeze

Hey all!
Basically if I am inside of a remote app while using the RD Web client, if I hit the windows button it opens my local start menu, and after about 5 seconds the RD client's start menu opens.  At this point, everything is frozen and to unfreeze I have to hit the windows button again.

Hi @JacobT14 thank you for your feedback. This looks like product feedback, not feedback on a topic on docs.microsoft.com. Am I interpreting your comment correctly? If so, you have two better options to communicate your feedback and to ask for help. The first is the "product feedback" button right next to the "documentation feedback" button at the bottom of  every topic. The second is the product forum where experts and product team members can help.
I'm closing this doc feedback but feel free to reply back if I am missing your point.
Best,
Dawn


Feature/sql

Summary by CodeRabbit

New Features

Introduced a new IRepository interface to define a consistent contract for repository operations, enhancing data manipulation capabilities.
Added various TypeScript types and enumerations to support robust query building and data manipulation.

Chores

Configured testing framework with TypeScript support to streamline testing.
Updated dependencies to include necessary packages for testing and TypeScript improvements.
Modified TypeScript configuration to include the test directory for improved test recognition.

@coderabbitai fixed all, please review again

@coderabbitai I tried to fix the comment, please review it again

@coderabbitai could you review the injection risk first

@coderabbitai i fix and refactor the code, please review

@coderabbitai i can't re-request you review, if you think the code is fine, could you approve this pr


Find creating prompts a bit confusing

There are two aspects to this that I've noticed.

From what I've done and seen, there's a disconnect between writing the prompts themselves and the screens on which they will appear. When folks (including me despite my using it several times) write the field prompts, it's hard to keep the context in mind - that they will be grouped on screens that will have a title and explanatory text.

To describe my personal experience, I find it a bit confusing and frustrating. A simplified example is writing the 'prompt' for a first name as 'What is your first name?' instead of just 'First name'.
An idea that I think is a strong candidate for a solution is to switch the order in which the form is constructed - first ask for a screen and it's title text, allowing the user to select which variables will be set on that screen, then, showing one screen at a time, have users set the prompts and data types of each variable. I think this could have a lot of impact to put what a user is doing into context.

A problem I've seen with first-time users is that they're not sure what 'prompts' are going to look like. For that, I think we could show an image of an existing page on screen that circles the 'prompt'. Maybe it's worth just making a diagram of the parts in general. Something like the below:

This exact image would be extremely helpful when filling out the field information, just to keep track of what the interview would actually look like when writing it.

This exact image would be extremely helpful when filling out the field information, just to keep track of what the interview would actually look like when writing it.

To restate:

First, create screens, and select the variables that appear on that screen.
Second: give prompts to the variables on each screen and choose the data types.

To restate:

First, create screens, and select the variables that appear on that screen.
Second: give prompts to the variables on each screen and choose the data types.

I wonder how feasible it would be to add prompts at the same time that you select fields. This could be too much going on on one screen though.
A quick fix for the next cohort would be some more context and prompts at the labeling stage, and reducing the number of options on the labeling screen will also help (combining datatype/input type in issue #117 )

I wonder how feasible it would be to add prompts at the same time that you select fields. This could be too much going on on one screen though.
A quick fix for the next cohort would be some more context and prompts at the labeling stage, and reducing the number of options on the labeling screen will also help (combining datatype/input type in issue #117 )

I am working on issue #320, and found the term "on-screen prompt" confusing. Why can't we simply call it "input field label" as this seems to be all about the input fields labels, right?

I think we just need language that does not get mixed up with the PDF fields. That is why I think we used "on screen" and prompt--since we call the process of prepping the PDF labeling it.

In that case would "on screen" be sufficient to distinguish the two? Because "prompt" would seem to mean more than a plain label.
Anyway, this is what I am going to use for that screen's question:
Define on-screen prompt for your question variables (i.e. labels for input fields on your question screens)

Yes, I think "on screen label" on its own would probably be good

I think this is resolved for now, but good to track our evolving thinking.


issue#30 corrected the docker image name in readme

See adop-nginx-issue#30

LGTM, thanks @RobertNorthard!


Increment versions for eventgrid/azure-resourcemanager-eventgrid releases

Increment package versions for eventgrid/azure-resourcemanager-eventgrid releases

/check-enforcer override


Should pop up "overwrite or new" when Save

I see this feature is remove from new version.
When Ctrl+S to save, i think it's better if we pop up dialog to select action overwrite or save as new.
It's not a big problem, but it makes user avoid miss take when creating new from an existed rest.
Expected outcome
pop up dialog to select action overwrite or save as new
Actual outcome
Only save
Versions
App: <IP_ADDRESS>6-stable
Chrome: 53.0.2785.143
Platform: Linux (-)
Steps to reproduce

create new rest
edit rest above to make new rest similar a bit
if ctrl+s instead of ctrl+shift+s => overwrite old rest. i lost it.

Hi,
If you press ctrl+s on unsaved request then the popup bill be displayed by default. However, if the request is restored then the default action for ctrl+s is immediate save instead of showing the popup. You have to use ctrl+shift+s keys combination to display the popup in this case.
This is due to have consistent UX with other application. This is how the ctrl+s and ctrl+shift+s works with most applications.


When using this on client side then a worker would the perfect place for it?

When using this on client side then a worker would the perfect place for it?
Agree?
This way any query etc would happen in its own thread, and not really block any other stuff

Closing this as isomorphism has taken a whole new level of importance. I'm not discarding the use of workers in the future, especially if something like that becomes available on node.


[mgmt] cosmosdb release

https://github.com/Azure/sdk-release-request/issues/4831

API change check
APIView has identified API level changes in this PR and created following API reviews.
@azure/arm-cosmosdb


Improve error message for imported EKS cluster having zero nodes

Setup

Rancher version: v2.8-d0a177f1745fe14a0b3de7483aa3ef895d1ab967-head
Rancher UI Extensions: None
Browser type & version: Chrome latest

Describe the bug
Improve error message for imported EKS cluster having zero nodes
To Reproduce
Import EKS cluster having no nodegroups and no self-managed nodes
Expected Result
In https://github.com/rancher/eks-operator/issues/301, support is added for self-managed EKS nodes
So the error message needs to changed from "Cluster must have at least one managed nodegroup" to "Cluster must have at least one managed nodegroup or one self-managed node"
Screenshots

cc: @mjura @salasberryfin

A couple of PRs to address this:

release/v2.9: https://github.com/rancher/rancher/pull/44240
release/v2.8: https://github.com/rancher/rancher/pull/44238

Validated as fixed on builds:
v2.8-4716667b53c4b76e82fe34207899da46470c5e0d-head
v2.9-ed248439462a81464f287ac8de6ebcb2b430087a-head


Update the description for the mimic note module on the frontpage

This updates the description for the mimic note module on the frontpage of the website.

For me "acquired from either the hospital wide electronic health record or the ICU information system" sounds like we aren't sure which! Could this be tweak to "from the ... and ..."? Or perhaps we could say "from the hospital information systems" (given that the sources may expand over time).


Tsig is not being set in client request messages.

The client type function exchange doesn't set the message Tsig even if the client type TsigSecret var is set properly. TsigGenerate does not get called from client-side at all--shouldn't it? Thanks.

By my read Exchange should sign the message if the message contains a TSIG record. Are you placing a TSIG record in the message you pass to Exchange?

An example using the SIDN workbench:
package main

import (
        "time"

        "github.com/miekg/dns"
)

func main() {
        qm := new(dns.Msg)
        qm.SetQuestion("types.wb.sidnlabs.nl.", dns.TypeNS)
        qm.SetTsig("wb_sha256.", dns.HmacSHA256, 300, time.Now().Unix())
        c := new(dns.Client)
        c.TsigSecret = map[string]string{"wb_sha256.": "npfrIJjt/MJOjGJoBNZtsjftKMhkSpIYMv2RzRZt1f8="}
        am, _, err := c.Exchange(qm, "bind9.sidnlabs.nl:53")
        if err != nil {
                panic(err)
        }
        println(am.String())
}

That did the trick! Thank you for the prompt and thorough response.
I wasn't setting the tsig properly, via the dns.Msg.SetTsig call. This works perfectly.


Not working on Gigabyte GA-Z170N-WiFi (ALC1150)

It's not working on Gigabyte GA-Z170N-WiFi. This motherboard has ALC1150 audio and I see library is loaded without any problems, but I see nothing in audio devices.
I have another build with MSI Z97A Gaming 7, which also has ALC1150 and there all is okay.
May be I can provide some additional info? Logs, dumps etc.

Definitely the problem is on your side. AppleALC works pretty well with this board... Provide full  DarwinDumper report, IORegistry report and your EFI folder...

Also show me output of this command
codesign --verify --no-strict /System/Library/Extensions/*.kext/

And
codesign --verify --no-strict /Library/Extensions/*.kext/

EFI.zip
DarwinDumper_3.0.3_12.04_23.22.50_iMac17,1_AMI_X64_4049_Sierra_16E195_zinp.zip
IORegistryReport.zip
codesign.zip

OK. Delete your EFI folder and replace it with this EFI. Don't change anything in it. After login provide IOReg
EFI.zip

It didn't loaded. It breaking on step of Intel HD 530 initialisation. But with one difference. Usually on this step I see Mac logo teared to parts horizontally. Here is happening the same but colours was changed to purple

So, You suppose that the problem in graphics card?

Replace config.plist and try again. Only config.plist in my EFI folder!!!
config.plist.zip

The same result

What next we can do?

May be the problem is caused not only by intel graphics and by something else? I tried to put MSI GTX960 4GB and install web drivers. Of course Intel HD 530 I disabled in bios. And sound devices didn't appeared. I tried just half of possible layers, but I guess that will not solve my problem.

And as I see in 'sudo kextstat' AppleAlc didn't loaded...

Try this config
config.plist.zip

And that one is working :-)
What You changed?

Compare your previous config.plist with mine... Audio works? I can close issue?

Yeah, sure. Thanks!

You are welcome


[SIEMINT-69] DDS: Cisco Secure Endpoint: Crawler Integration Without Assets

What does this PR do?
PR includes skeleton of Cisco Secure Endpoint integration without any assets like dashboards, image, README content, etc.
Motivation
This is beta release of crawler integration and is intended for internal testing before going live. As per @nathanandersen's suggestion, we are planning to raise separate PR with assets and all necessary information later.
Additional Notes
Only crawler-based integration related changes are included in this PR.
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Changelog entries must be created for modifications to shipped code
[ ] Add the qa/skip-qa label if the PR doesn't need to be tested during QA.

looks good - fix failing tests and you should be good to merge

Everything seems to be passing now, can this be merged?

we must add this disclaimer at the bottom of the overview tab @ankitarajput-crest
"Your use of this integration, which may collect data that includes personal information, is subject to your agreements with Datadog. Cisco is not responsible for the privacy, security or integrity of any end-user information, including personal data, transmitted through your use of the integration"
(cc: @jnhunsberger )


it says failed to open ARC file and i don't know why

i don't know how or what is happening but it's driving me crazy please help (i've already searched on google and found nothing)
here is the log
log.txt

Make sure your file is dumped correctly and isn't corrupted by checking the hash.
tutorial: https://gamebanana.com/tuts/12827
MD5 Hashes: https://gist.github.com/ThatNintendoNerd/2eff671ecbafd2043e9e48758a6e61bd


ParticleEditor Add ability to change X and Y scale or aspect ratio

Could you add a possibility to change x and y scale of the particle as a two separate curves?
Or better to be able to change x/y ratio?
This would help bring some animations to the particles.

We'd appreciate PRs, i don't believe anyone on the team will implement this.


Avoid URL requirement name collisions

Fixes #1145.
This avoids cache collisions when using URLs which both end with the same filename where a later dependency depends on an earlier one, e.g.
file:///tmp/mydep1/master.zip
file:///tmp/mydep2/master.zip
(where mydep2 depends on mydep1)
Changelog-friendly one-liner: Avoid name collisions with URL based requirements.
Contributor checklist

[X] Provided the tests for the changes.
[X] Gave a clear one-line description in the PR (that the maintainers can add to CHANGELOG.md on release).
[ ] Assign the PR to an existing or new milestone for the target version (following Semantic Versioning).

Re: Cache directory, good point. I've factored out the logic to a _get_download_path function and used it in both places it appears to be needed.
I've not introduced a specific test for this as I believe it constitutes internal implementation detail rather than external behaviour that should be asserted upon, but let me know if a test is required.

Can we get a nice explanation of the problem being solved, as the docstring for the test?

@AndydeCleyre Sure, I've just added that- hopefully it's not too verbose?

Not sure why CI is failing at the moment, will take a look when it deigns to give me logs.

@geokala

Not sure why CI is failing at the moment, will take a look when it deigns to give me logs.

I've fixed it in https://github.com/geokala/pip-tools/pull/1. You could merge this if it works.

@geokala
FYI, also fixed test test_name_collision due to changes in https://github.com/geokala/pip-tools/pull/1/commits/2c4c5719bc2be19ce8fa61a39df6e78ab3191ce8.

@atugushev Awesome, thanks for that- makes sense, so I've merged it into my PR.

Looks like CI is failing on 2.7 pip latest (I think?):
File "/tmp/easy_install-P_Otzu/jaraco.windows-5.0.0/temp/easy_install-B_XSyo/path.py-12.4.0/temp/easy_install-kjgCqJ/more-itertools-8.3.0/more_itertools/init.py", line 1, in
File "/tmp/easy_install-P_Otzu/jaraco.windows-5.0.0/temp/easy_install-B_XSyo/path.py-12.4.0/temp/easy_install-kjgCqJ/more-itertools-8.3.0/more_itertools/more.py", line 482
yield from iterable
^
SyntaxError: invalid syntax
=================================== log end ====================================
ERROR: FAIL could not package project - v = InvocationError(u'/opt/hostedtoolcache/Python/2.7.18/x64/bin/python setup.py sdist --formats=zip --dist-dir /home/runner/work/pip-tools/pip-tools/.tox/dist', 1)
Has pip already stopped supporting 2.7 for make_sdist or something?

A local run of tox --notest -p auto --parallel-live doesn't give me that error, though of course my environment doesn't match travis.

So yeah, looks like more-itertools is being used by easy_install but doesn't support python2.7 any more- I'm guessing I already had an older version of easy_install installed so didn't run into this problem.
Presumably this means that something needs pinning- easy_install or more-itertools- and presumably it wants pinning in tox??

Unfortunately, recent setuptools-scm-4.0.0 broke our CI. Working on a fix.

@geokala
Please rebase onto master it should be fixed now.

Rebased. Fingers crossed.

@geokala
Great first-time contribution 🚀 Thanks!

No worries, and thanks for your help!


Call to undefined method yii\helpers\ArrayHelper::recursiveSort()

What steps will reproduce the problem?
I save ActiveRecord model with jsonb postgres column type
What is the expected result?
Save/update model without error
What do you get instead?
Additional info
I corrected the error by copying functions "isIndexed", "recursiveSort" from file BaseArrayHelper to file ArrayHelper

Q
A

Yii version
<IP_ADDRESS>?

PHP version
8.2

Operating system
Linux 4.15.0-213-generic Ubuntu

IIRC it was fixed with 2.0.49, could you check it?

I will check, thanks

After update to <IP_ADDRESS> error still exist:
"name": "Exception",
"message": "Call to undefined method yii\\helpers\\ArrayHelper::recursiveSort()",
"code": 0,
"type": "Error",
"file": "private/vendor/yiisoft/yii2/db/BaseActiveRecord.php",
"line": 1784,
"stack-trace": [
    "#0 /private/vendor/yiisoft/yii2/db/BaseActiveRecord.php(653): yii\\db\\BaseActiveRecord->isValueDifferent()",
    "#1 private/vendor/yiisoft/yii2/db/BaseActiveRecord.php(806): yii\\db\\BaseActiveRecord->getDirtyAttributes()",
    "#2 private/vendor/yiisoft/yii2/db/ActiveRecord.php(680): yii\\db\\BaseActiveRecord->updateInternal()",
    "#3 private/vendor/yiisoft/yii2/db/BaseActiveRecord.php(690): yii\\db\\ActiveRecord->update()",
    "#4 private/controllers/DocumentsController.php(170): yii\\db\\BaseActiveRecord->save()",

Have you altered ArrayHelper or BaseArrayHelper in any way? This method is defined, so this error does not make any sense:
https://github.com/yiisoft/yii2/blob/01f39832275527bf9a8247df41a3d287e093a571/framework/helpers/BaseArrayHelper.php#L1026

i have error in project
BaseArrayHelper was replaced by other definition
the error has been resolved
sorry


Allow freezing the MTCNN PNet, RNet and ONet tensorflow models

Make possible to freeze and safe the MTCNN PNet, RNet and ONet, after they have been loaded with the pre-trained weights.

what is Pnet, Rnet, Onet??I am new to using this models.


Change YARA grammar

Description
This PR changes the grammar used for YARA to https://github.com/infosec-intern/vscode-yara, which is more currently maintained than the previous one and has new support like 4.1 keywords
Checklist:

[x] I am changing the source of a syntax highlighting grammar

License: MIT
Preview: Before | After

FWIW, some issues I encountered in linguist when making this change:

MIT license was inaccurately classified as other

error running script/list-grammars on Ubuntu 20.04 :

$ script/list-grammars
/home/user/Documents/linguist/vendor/gems/ruby/3.0.0/gems/charlock_holmes-0.7.7/lib/charlock_holmes.rb:1:in `require': /snap/core18/current/lib/x86_64-linux-gnu/libm.so.6:
version `GLIBC_2.29' not found (required by /lib/x86_64-linux-gnu/libicuuc.so.66) - /home/user/Documents/linguist/vendor/gems/ruby/3.0.0/gems/charlock_holmes-0.7.7/lib/charlock_holmes/charlock_holmes.so (LoadError)
	from /home/user/Documents/linguist/vendor/gems/ruby/3.0.0/gems/charlock_holmes-0.7.7/lib/charlock_holmes.rb:1:in `<top (required)>'
	from /home/user/Documents/linguist/lib/linguist/blob_helper.rb:2:in `require'
	from /home/user/Documents/linguist/lib/linguist/blob_helper.rb:2:in `<top (required)>'
	from /home/user/Documents/linguist/lib/linguist.rb:1:in `require'
	from /home/user/Documents/linguist/lib/linguist.rb:1:in `<top (required)>'
	from script/list-grammars:4:in `require'
	from script/list-grammars:4:in `<main>'

I followed install instructions from https://github.com/github/linguist/blob/master/CONTRIBUTING.md#dependencies
except used ruby snap to install latest ruby and bundler


[PyTorch pin update] 951c21d6 - Inductor tests are starting to fail after pybind11 update to 2.13.6

PyTorch commit id: 951c21d6790334d57862e94a3f582ac724147a53
https://github.com/intel/intel-xpu-backend-for-triton/actions/runs/10879914823/job/30185587514
_________ KernelTests.test_layout_constraint_needs_fixed_stride_order __________
Traceback (most recent call last):
  File "/runner/_work/intel-xpu-backend-for-triton/intel-xpu-backend-for-triton/pytorch/test/inductor/test_triton_kernels.py", line 1072, in test_layout_constraint_needs_fixed_stride_order
    eager_out = f(x)
  File "/runner/_work/intel-xpu-backend-for-triton/intel-xpu-backend-for-triton/pytorch/test/inductor/test_triton_kernels.py", line 1068, in f
    arange_out(x, y)
  File "/runner/_work/intel-xpu-backend-for-triton/intel-xpu-backend-for-triton/pytorch/test/inductor/test_triton_kernels.py", line 1059, in arange_out
    kernel[grid](x, out, n_elements, BLOCK_SIZE=4)
  File "/opt/hostedtoolcache/Python/3.9.20/x64/lib/python3.9/site-packages/triton/runtime/jit.py", line 330, in <lambda>
    return lambda *args, **kwargs: self.run(grid=grid, warmup=False, *args, **kwargs)
  File "/opt/hostedtoolcache/Python/3.9.20/x64/lib/python3.9/site-packages/triton/runtime/jit.py", line 657, in run
    kernel = self.compile(
  File "/opt/hostedtoolcache/Python/3.9.20/x64/lib/python3.9/site-packages/triton/compiler/compiler.py", line 315, in compile
    metadata_group[metadata_filename] = fn_cache_manager.put(json.dumps(metadata, default=vars), metadata_filename,
  File "/opt/hostedtoolcache/Python/3.9.20/x64/lib/python3.9/json/__init__.py", line 234, in dumps
    return cls(
  File "/opt/hostedtoolcache/Python/3.9.20/x64/lib/python3.9/json/encoder.py", line 199, in encode
    chunks = self.iterencode(o, _one_shot=True)
  File "/opt/hostedtoolcache/Python/3.9.20/x64/lib/python3.9/json/encoder.py", line 257, in iterencode
    return _iterencode(o, 0)
TypeError: vars() argument must have __dict__ attribute

CC @alexbaden

Reducing the number of potential problems:

db393fb95e5b057ca49472828bb6dba2db4f859e (works)

Do you think we can temporary pin pybind11 to the last working version?

Blocked by:

https://github.com/pytorch/pytorch/pull/136280

pybind fix was merged last night/this morning, so this should also be resolved.

Removed in https://github.com/intel/intel-xpu-backend-for-triton/pull/2351


fix: AzureRM version 4 support

Description
Support for AzureRM version 4. Fix for #10

Type of Change

[ ] Non-module change (e.g. CI/CD, documentation, etc.)
[x] Azure Verified Module updates:

[x] Bugfix containing backwards compatible bug fixes

[x] Someone has opened a bug report issue, and I have included "Closes #{bug_report_issue_number}" in the PR description.
[ ] The bug was found by the module author, and no one has opened an issue to report it yet.

[ ] Feature update backwards compatible feature updates.
[ ] Breaking changes.
[x] Update to documentation

Checklist

[x] I'm sure there are no other open Pull Requests for the same update/change
[ ] My corresponding pipelines / checks run clean and green without any errors or warnings
[x] I did run all  pre-commit checks

@microsoft-github-policy-service agree


SAML2-JS dependency

The SAML2-JS dependency.
The SAML2 package should not be installed by default but only if required.
v3.0.1 of the package had some vulnerabilities which required us to fork the repository into https://github.com/GEOLYTIX/saml2 where these dependency issues were addressed.
The original maintainer has since addressed the issues in the v4.0.1 release.
https://github.com/Clever/saml2

https://github.com/GEOLYTIX/xyz/pull/582


Update to flow v0.66

Fixes #36
Closes #37

Coverage remained the same at 100.0% when pulling a3c230f23458a6a8c872b0129e1eed498d8e270b on flow-66 into dfef1576294fea2786709927624ef7c0279bbfbe on master.


Add Cpf digit calculator

Codecov Report

Merging #15 into master will increase coverage by 0.15%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #15      +/-   ##
==========================================
+ Coverage   98.19%   98.34%   +0.15%
==========================================
  Files           7        8       +1
  Lines         221      241      +20
==========================================
+ Hits          217      237      +20
  Misses          3        3
  Partials        1        1

Impacted Files
Coverage Δ

cpf.go
100% <100%> (ø)

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 8cc02e9...ac8a4aa. Read the comment docs.


no-document-write: Error is triggered by call to "write" method on object other than document

See the following example:
export class MyClass {
    public myMethod = () => {
        let writer = new jsts.io.GeoJSONWriter(),
            geom: jsts.geom.Geometry;

        writer.write(geom);
    }
}

When the no-document-write rule is true, the following error is thrown:
test.ts[6, 9]: An error occurred visiting a node.
Walker: NoDocumentWriteWalker
Node:

        writer.write(geom)
TypeError: Cannot read property 'members' of undefined

This example requires that the ambient declaration for the jsts library be available.: https://github.com/DefinitelyTyped/DefinitelyTyped/tree/master/jsts.

Hi, in my project (https://github.com/rovale/micro-mqtt) I get a similar error if I enable the no-document-write rule:
C:\src\micro-mqtt> npm run-script tslint

><EMAIL_ADDRESS>tslint C:\src\micro-mqtt
> tslint ./src/**/*.ts

src/module/micro-mqtt.ts[107, 9]: An error occurred visiting a node.
Walker: NoDocumentWriteWalker
Node:

        this.networkSocket.write(MqttProtocol.createConnectPacket(this.options))
TypeError: Cannot read property 'flags' of undefined
src/module/micro-mqtt.ts[133, 17]: An error occurred visiting a node.
Walker: NoDocumentWriteWalker

I disabled the rule because I will never write in any document, but I just wanted to let you know :-)

It seems like the typescript typechecker just doesn't work at all from within tslint. There are so many errors like this that basically everything is an 'any' type.

Hi guys, I fixed this issue today by simplifying the rule. You will now receive a violation for:
document.write('...');
document.writeln('...');
window.document.write('...');
window.document.writeln('...');

But you will not receive a violation if document is reassigned to a variable:
const doc = document;
doc.write('...'); // this no longer causes a violation because it often simply caused an error


Wrongly lint tsconfig.json when using eslint

Current behavior
When using eslint, tsconfig.json is wrong included and cause lots of error
Steps to reproduce the issue
passing { eslint: true } to fork-ts-checker-webpack-plugin
Issue reproduction repository
https://github.com/Austaras/boilerplate
Environment

fork-ts-checker-webpack-plugin: 1.4.0
typescript: 3.5.3
webpack: 4.35.3

after some digging I found out that also passing { useTypescriptIncrementalApi: false } will solve this problem.

after some digging I found out that also passing { useTypescriptIncrementalApi: false } will solve this problem.

Well, after some more digging, seems like when enable useTypescriptIncrementalApi, this pulgin will get tsconfig.json at first compile. TSLint will ignore any file that isn't js or ts, while ESLint doesn't and that is the cause of the problem.

Hi @austaras
Thanks for reporting. That's super curious and I'd like to understand why it's happening. I'm hoping you can help me. If you take a look at this minimal example here:
https://github.com/TypeStrong/ts-loader/tree/master/examples/fork-ts-checker-webpack-plugin
The issue does not present itself and it is using useTypescriptIncrementalApi. I'd like to understand what's different between your example and this one. Would you be able to take that example and see what triggers the different behaviour please?
Incidentally, we can easily work around this by adding an extra case to the if statement here to check if the file is a tsconfig.json and excluding if so. I'd be happy to receive a PR that did that if you wanted to submit one.
But I'd love to understand what causes the issues you're seeing anyway. Also, what are the errors you're being presented with? You don't mention in the issue...

@johnnyreilly the main difference is I use json as config file so I simply pass "./tsconfig.json" to project while your example pass path.resolve(__dirname, './tsconfig.json'). Change that to './tsconfig.json' in your example will generate same error.
It's true that a dirty and quick fix is possible but I believe in a wide used package like this one such fix is bad for long term maintenance. However I don't know what's the difference between the two value and even if it's possible to use path in json due to my lack of knowledge on ESLint. I hope you will find a better fix.
BTW I got some quite generic error like expect an expression and should use single quotes

Thanks!
Given ESLint shouldn't parse JSON anyway, I think my suggestion is probably fine. I'm on my phone right now, but if you'd like to submit a PR with my suggested fix I'll happily merge it and get this out there.
https://github.com/eslint/eslint/issues/5076

Actually I've got a screen and put together a prospective fix: https://github.com/TypeStrong/fork-ts-checker-webpack-plugin/pull/313
Could you give this PR a try locally and confirm if this resolves your issue please?  Thanks!

It works fine on my project.


Build process for Angular application showing .NET Framework dependency

Our build platform is a Linux container using .NET 6.0 SDK.  I am unable to build the solution created by this process in that platform due to a .NET Framework dependency in the Angular project--which I don't believe I added.  I see the scaffolding is still in Preview, so not sure if the .NET Framework dependency is still a WIP, or not.   Or am I doing something wrong, which is very possible.

This is the error message I receive from running
dotnet publish -c Release --sc -r linux-musl-x64 .
MSBuild version 17.3.2+561848881 for .NET
Determining projects to restore...
Restored /home/rlangel/_source/conaf/webapi/webapi.csproj (in 321 ms).
/usr/share/dotnet/sdk/6.0.403/Microsoft.Common.CurrentVersion.targets(1220,5): error MSB3644: The reference assemblies for .NETFramework,Version=v4.7.2 were not found. To resolve this, install the Developer Pack (SDK/Targeting Pack) for this framework version or retarget your application. You can download .NET Framework Developer Packs at https://aka.ms/msbuild/developerpacks [/home/rlangel/_source/conaf/angularapp/angularapp.esproj]
/usr/share/dotnet/sdk/6.0.403/Microsoft.Common.CurrentVersion.targets(1220,5): error MSB3644: The reference assemblies for .NETFramework,Version=v4.7.2 were not found. To resolve this, install the Developer Pack (SDK/Targeting Pack) for this framework version or retarget your application. You can download .NET Framework Developer Packs at https://aka.ms/msbuild/developerpacks [/home/rlangel/_source/conaf/angularapp/angularapp.esproj]

for clarity -- if needed, this is in reference to
https://learn.microsoft.com/en-us/visualstudio/javascript/tutorial-asp-net-core-with-angular?view=vs-2022

@ParallaxOW -- Russ, thank you for your detailed feedback.  This issue appears to be implementation-related, rather than documentation-related.  Please consider these resources:

Ask a question or find an answer on the Microsoft Q&A page for Visual Studio.
Check the Visual Studio MSDN Forum.
Stack Overflow
Reddit

cc @Mikejo5000 for awareness
I am closing this issue now.  You are welcome to @ mention me for any followup.  We hope to hear from you again.

@ParallaxOW @name Thanks for your feedback; we appreciate it. To make sure that this is logged in the right place, please use the Report a Problem tool to report the issue. When you do this, your report becomes part of the work items that are listed in the in the Visual Studio Developer Community, which allows you to engage directly with the engineering team while they address and fix the problem.


Add a file opener

I'm putting this here because it will be growing in scope and might be reusable outside of a plugin.

Looks good to me.

:+1:


Unable to resolve firebase-ios-sdk package, add anyway?

Step 1: Describe your environment

Xcode version: 13.3
Firebase SDK version: 8.0.0
Installation method: Swift Package Manager
Firebase Component: (Auth, Analytics, Firestore, Storage,)
Target platform(s): | iOS |

Step 2: Describe the problem
After clicking "Add package" it starts to resolve and then shows the attached alert, asking whether to add anyway.
This is before we get to choose Auth/Storage etc
I've closed xcode and re-opened.
cache has been cleared About > Storage > Manage > Developer > delete caches & indexes.
If select add anyway it fails.
internet connection is OK.
Any thoughts would be appreciated, thanks.
David

Sorry for the trouble @Dibit144 - we haven't seen this before and I can't see any other references online to it (or this error in the SwiftPM source code). Does this happen on the same computer on a different network, or if possible can you try a different computer on the same network? Also a reboot of your machine just in case.
Just trying to narrow down if it's your device, your internet connection, or something else. Thanks!

Thanks Ryan, got it working this morning.
after "add anyway" it would try and load but many firebase errors for missing parts as didn't do all the downloads properly.
After deleting the apple frameworks recommendation on the left tab when searching for packages in Xcode, was then able to load firebase properly and it pulled all sources correctly, select relevant packages and download and use (obviously may not have had anything to do with it though).
will close it out. thanks


Minimize loading time of the images

Can you please assign this to me?

@parthib2004 Sure. Do star ⭐ the repo and follow me to start.


change import statement

needed in order to build the project properly as a framework.

This isn't the only fix the project needs to fix the framework problems.  After making the above change I now get
CocoaAsyncSocket/GCDAsyncUdpSocket.h file not found

Thanks for your contribution. I used your code and it is solved


Will you add that the program closes itself?

Will you add that the program closes itself, just a little reminder and also if you want more popularity why not
make a video on youtube for that.
Wish you the greatest of luck

Thanks for submitting an issue!
Frankly, I think it's pretty apparent that the script closes itself when it is finished, so I don't really feel a need to explicitly say this. But I'm opening to changing my mind if it comes up or appears to be a point of confusion


x/gamm: Charge gas for computation in swaps instead of just I/O

Swaps currently only charge gas for I/O, regardless of the amount of computation they take up. One minor but useful change would be to make it so that gas is also charged for computation in swaps.

What exactly do you have in mind @AlpinYukseloglu? Some fixed cost per specific operation?

Yes, some fixed cost for now with a TODO to turn it into a param later (since making it a param now would require migration code and it's too small of a change to be worth it for that)

Agreed, lets add a fixed cost for now.

@AlpinYukseloglu is anyone working on this right now? If not, happy to take this!

@AlpinYukseloglu is anyone working on this right now? If not, happy to take this!

Nope, feel free to take it!


Add PACK, MATH, UNPACK kernels information to METALIUM_GUIDE.md

Unlike Data Movement kernels, Metalium compile pipeline actually generates three kernels for a Compute kernel, i.e., PACK, MATH, and UNPACK kernels. A Compute kernel developer can see only one kernel i.e., the Compute kernel itself. It is nice to provide the abstraction, but for advanced developers who have to understand the exact details of how Unpack and Pack for tiles work, the information about the three kernels will help them a lot. This commit adds the information to METALIUM_GUIDE.md.
Ticket
N/A
Problem description
When I debugged my simple kernel, I met the situation that pack_tile(..) does not work. I was not able to debug it by reading known APIs and examples. I wanted to see how the Compute kernel is actually compiled. While investigating on it, I got some useful information, and I believe sharing this with other developers can help them a lot.
What's changed
I added PACK, MATH, UNPACK kernels information to METALIUM_GUIDE.md.
Checklist

[ ] Post commit CI passes
[ ] Model regression CI testing passes (if applicable)
[ ] New/Existing tests provide coverage for changes

I want to link it to one of examples: https://github.com/tenstorrent/tt-metal/tree/main/docs/source/tt-metalium/tt_metal/examples, but not sure what the good example is for this explanation.

@ayerofieiev-tt  @SeanNijjar Could you please take a look at this PR?

thank you @jaebaek , this is really good.
I left couple of comments regarding the use of new tile_regs_ APIs for math->pack sync

I want to link it to one of examples: https://github.com/tenstorrent/tt-metal/tree/main/docs/source/tt-metalium/tt_metal/examples, but not sure what the good example is for this explanation.

your standalone example is good as a very basic example of what is done on each of unpack, math, pack

Thanks! I added some description about tile_regs_..() functions. I hope it is clear. Please let me know!

Great!


Terms of use and onedrive

If terms of use work perfectly when i first logon in the portal or in Teams, I never see the terms of use if i first logon in OneDrive.
I tried on a Windows client, Android, and Iphone.
Is this behaviour normal ?

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

ID: 618ca851-56c8-5ba1-35de-a7b8e7af8aa2
Version Independent ID: 5facc663-2e29-65ad-f208-982150351b22
Content: Azure Active Directory Terms of use
Content Source: articles/active-directory/active-directory-tou.md
Service: active-directory
GitHub Login: @rolyon
Microsoft Alias: rolyon

Thanks for your feedback! We will investigate and update as appropriate.

@Najib314 No it works for OneDrive as well, I am able to see the Terms Of Use for OneDrive as well.  Please make sure that you are trying to accept the Terms of Use for the first time through OneDrive and haven't accepted it using portal or application earlier.

@Najib314 We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Increased data availability & submission

Aspect
Description

Challenges
Greater need for standardised and well structured data solutions that can accommodate a potential wider range of data types. Will likely require more information requirements from originators (data context, lineage etc.) to minimise inferred validation/QC.

Mitigating strategies and controls
DIG to Monitor. Discussions around dynamic submission of data versus quality control already in progress

legacy entry

Likelihood: There is at the tieme being a medium likelihood for increased data availablity with new data platforms and sensors.
Impact: Increased and diversified data availability will lead to a bigger need for standardized data management that will impact both the scientific and the data managment communities.


WIP: the configurable version

This idea wasn't good, why create a dedicated artifact, for a developer it will be also easy to change the artifactId, as it is easy to add the config> keep.my.own.editorconfig


alternative syntax

from discussion in #10
The current implementation is the same as python's doctest.
{-|
    >>> indexOfLineWithEquals
    ...      <| Array.fromList
    ...      <| String.lines "a\nb\nc = d\nf = d\n"
    2
-}

vote 👍
{-|
    > indexOfLineWithEquals
    #      <| Array.fromList
    #      <| String.lines "a\nb\nc = d\nf = d\n"
    2
-}

vote 😄
{-|
    > indexOfLineWithEquals
    ~      <| Array.fromList
    ~      <| String.lines "a\nb\nc = d\nf = d\n"
    2
-}

vote 🎉
{-|
    > indexOfLineWithEquals
    \      <| Array.fromList
    \      <| String.lines "a\nb\nc = d\nf = d\n"
    2
-}

vote ❤
{-|
    > indexOfLineWithEquals
    |      <| Array.fromList
    |      <| String.lines "a\nb\nc = d\nf = d\n"
    2
-}

vote : 😕 (sorry for the emoji, there aren't many options)

I would vote for :tada: if we decide on going with an alternative solution. I still think that using one character can be confusing and can be mistaken for part of the code, where ... and >>> is more obvious.

btw. haskell doctest also uses >>>

Why do you try to invent new syntax when elm-repl has already established one?
> (\a -> a)
| [ 2
| , 3
| , 5
| ]
[2,3,5] : List number

I vote for non of the above but for the elm-repl one!

@AnthonyJacob good idea to consider the elm repl syntax as well, although the tone of your suggestion can be improved.
I'd also vote for the elm repl syntax.

I included repl-style in the poll.
The reason why I don't like the syntax with only one char is, it can easily be confused to be a typo. >>> is more obvious.
The reason why I am against repl-style is, because:

it implies that elm-doc-test uses the repl to run your tests
you need \ to start a new line in repl and you don't have to type > or |
>>> is used in a doctest for other languages (more familiar to newcomers)
>>> is easier to search

I am sorry for that tone, I was going trough some packages-extra made by haskellers and it wasn't pleasant experience :)

you need \ to start a new line in repl
I hope that in the near feature this won't be the case. ( shift+enter is much more convenient )
in repl you don't have to type > or |
Once there will be good IDE support for documentation, you also won't need it. It might feel exactly like the repl actually.
...
All the other things you pointed out are can be solved by one bold line in documentation.

On the other hand, if the change would get propagated into repl, I would go with the first one.

Why the hell are you trying to invent new syntax when elm-repl has already established one?

This tone is not okay. This is an open source project that we have spent time on. We've also asked for people's feedback in multiple places on the syntax.

I vote for non of the above but for the elm-repl one!

The reason why the prompt does not matter is because 1) >>> is not going seem much different from > to beginners 2) doc tests are not completely equivalent to the repl and 3) clear, defining syntax is important in doctest parsing.
Haskell's doctests do not prefix things with Prelude>. In languages like Python, where doctests came from, there is a lot of confusion for beginners when doctests do not behave like the REPL. An example is how strings are handled. Very confusing. Even more confusing considering that multiple languages have different prompts and syntax handling entirely. I think this is a solution to a problem that nobody will have. However, if you can present somebody who's confused by the use of >>> over >, I'm happy to change my opinion.

I am sorry for that tone, I was going trough some packages-extra made by haskellers and it wasn't pleasant experience :)

If you are having a bad day, talk it out with someone else - don't take it out on people who are giving up their free time to work on projects.

I hope that in the near feature this won't be the case. ( shift+enter is much more convenient )
Once there will be good IDE support for documentation, you also won't need it. It might feel exactly like the repl actually.

Wishful thinking. Maybe if we planned this whole library around Elm having reflection then we wouldn't even need to generate elm-tests! Except that's not how things are right now, and to plan on some non-existent tooling changes at some point in the future is not a good way to get things done.

Maybe if we planned this whole library around Elm having reflection then we wouldn't even need to generate elm-tests!

So it is easier to write generator instead of making a pull request that would bring reflection into elm?

Adding reflection is a huge design decision, it's not as simple as just adding a pr. So yes.

Is / Was there some discussion on this topic (reflection) ? I wanted to make elm-repl better at exploration of the language / libraries / documentation etc. which would very likely require some amount of reflection ( even if only repl would have permission to use it ).

Is / Was there some discussion on this topic (reflection) ? I wanted to make elm-repl better at exploration of the language / libraries / documentation etc. which would very likely require some amount of reflection ( even if only repl would have permission to use it ).

closed in favour of #22


interrupt_controller: rv32m1: fix intmux driver initialization priority

Use the RV32M1 SoC intmux driver initialization priority set by
Kconfig.
Signed-off-by: Henrik Brix Andersen<EMAIL_ADDRESS>

@andrewboie any comments on this?


fix: Label position gets deselected on selecting already selected option

Description
Fixes the issue pointed out in this Loom recording where the label position gets deselected when an already selected item is selected.
This was happening since these controls didn't have a defaultValue associated to it.

Working fine

I'm working on adding migration so that the existing applications won't break on enabling dynamic height.


https://material.money not working anymore

https://material.money/ is not reachable at the moment, probably another certificate error (ERR_CONNECTION_CLOSED)? Unfortunately, you can't take the PWA to show it.

Seems to be working fine for me. Tried in several browsers.


List of icons

Notice: this is not the same as #6!
I have a https://pub.dev/packages/markdown_viewer plugin that draws icons using a markdown extension: [i:font_package:font_name:codepoint:color].
I was wondering if it would be possible to generate those information in a map, such as:
Map<String, IconData> materialSymbolsMap = {
  'ten_k': IconDataIconData(0xe951, fontFamily: _family_outlined, fontPackage: _package),
  ...
};

This way, I cound simplify the markdown extension to [i:ten_k] to show the ten_k icon on a Text widget.
I think this would not mess with the treeshaking process.

Hello @JCKodel  -
I am not completely understanding what you are trying to accomplish.  I took a look at your package and it looks nice.
Could you explain more about what you are trying to accomplish?
The symbols_map,dart file looks like this:
Map<String, IconData> materialSymbolsMap = {
  'ten_k': Symbols.ten_k,
  'ten_k_rounded': Symbols.ten_k_rounded,
  'ten_k_sharp': Symbols.ten_k_sharp,

...

It is basically a map like you describe, but containing all 3 of the versions of each icon (outlined, rounded and sharp).
I have been thinking of moving the symbols_map.dart file to the main package so that it would be available to reference for users that need a map of all icons.  If it wasn't included it would not effect tree shaking, but if it was included it would trigger the including of every icon in the fonts.

Basically, at run-time, I need to get the package name, font name and
unicode code of a string, such as "add_circle".
The Markdown extension will then render a Text using those info to actually
render the icon (it doesn't need nor should know what is Symbols).
In other words: given a String, I need to use a simple Text widget that
is able to render it (for example, an Icon(Icons.add) will actually
render to Text(String.fromCharCode(icon.codePoint), style: TextStyle(package: icon.fontPackage, fontFamily: icon.fontFamily))). The
Icon widget never knows what IconData package it receives (the same
would be true for the markdown extension, except the markdown will contain
a text, such as ten_k that I would need to render as the Text above,
without knowing about Symbols consts (introspection is soooo needed for
Dart :( ))
For this particular font package, the ideal would be a Map<String, IconData> with all icons (since IconData is const, it will not duplicate
the package and font names and also will not create any issues with Icon
treeshaking).
On Sun, Dec 17, 2023 at 12:05 AM Tim Maffett @.***>
wrote:

Hello @JCKodel https://github.com/JCKodel -
I am not completely understanding what you are trying to accomplish. I
took a look at your package and it looks nice.
Could you explain more about what you are trying to accomplish?
The symbols_map,dart file looks like this:
Map<String, IconData> materialSymbolsMap = {
'ten_k': Symbols.ten_k,
'ten_k_rounded': Symbols.ten_k_rounded,
'ten_k_sharp': Symbols.ten_k_sharp,
...
It is basically a map like you describe, but containing all 3 of the
versions of each icon (outlined, rounded and sharp).
I have been thinking of moving the symbols_map.dart file to the main
package so that it would be available to reference for users that need a
map of all icons. If it wasn't included it would not effect tree shaking,
but if it was included it would trigger the including of every icon in the
fonts.
—
Reply to this email directly, view it on GitHub
https://github.com/timmaffett/material_symbols_icons/issues/16#issuecomment-1859022924,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAC4TS2V3WZAEAZHFO4TFTDYJZOQTAVCNFSM6AAAAABAVRLAACVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNJZGAZDEOJSGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi @JCKodel  - it seems like there may be another solution that will certainly be safe (as far as not preventing tree shaking), and that is to provide something like this:
const Map<String,int> iconCodePoints = {
   'add' : 0xff33,
...
};
const String iconFontPacklage='....';
const String iconFontFamily='....';

This would create the minimum static data and the iconCodePoints map would be identical for all 3 symbols fonts (outline,rounded and sharp)..
Would something like that work ?  You are just starting with the icon's name string correct ?

@timmaffett That would work perfectly!

OK, version 4.2716.0 is being released today and I now generate a file lib\iconname_to_unicode_map.dart which has a map that looks like this:
Map<String, int> materialSymbolsIconNameToUnicodeMap = {
  'ten_k' : 0xe951,
  'ten_mp' : 0xe952,
  'eleven_mp' : 0xe953,
...
};

You could be able to include the file with
import 'package:material_symbols_icons/iconname_to_unicode_map.dart';
I did not include the package or font name as these can be found in the package itself and did not seem to be a useful addition.
@JCKodel - Let me know how it works!

Just perfect! Thank you!
Now I'll be able to
[i:ten_k]


[Bug] Program attempts to write to protected memory

When attempting to compare over 100k pictures I have been getting this errror:
See the end of this message for details on invoking
just-in-time (JIT) debugging instead of this dialog box.
************** Exception Text **************
System.AccessViolationException: Attempted to read or write protected memory. This is often an indication that other memory is corrupt.
at System.Runtime.InteropServices.Marshal.PtrToStructureHelper(IntPtr ptr, Object structure, Boolean allowValueClasses)
at System.Runtime.InteropServices.Marshal.PtrToStructure(IntPtr ptr, Type structureType)
at AntiDupl.NET.CoreLib.StatusGet(ThreadType threadType, Int32 threadId)
at AntiDupl.NET.SearchExecuterForm.EstimateSearchProgress()
at AntiDupl.NET.SearchExecuterForm.TimerCallback(Object obj, EventArgs eventArgs)
at System.Windows.Forms.Timer.OnTick(EventArgs e)
at System.Windows.Forms.Timer.TimerNativeWindow.WndProc(Message& m)
at System.Windows.Forms.NativeWindow.Callback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)
************** Loaded Assemblies **************
mscorlib
Assembly Version: <IP_ADDRESS>
Win32 Version: 2.0.50727.9148 (WinRelRS6.050727-9100)
CodeBase: file:///C:/Windows/Microsoft.NET/Framework64/v2.0.50727/mscorlib.dll
AntiDupl.NET
Assembly Version: <IP_ADDRESS>
Win32 Version: 2.3.10
CodeBase: file:///C:/Users/kulgi/Downloads/AntiDupl.NET-2.3.10d/AntiDupl.NET-2.3.10/AntiDupl.NET.exe
System.Windows.Forms
Assembly Version: <IP_ADDRESS>
Win32 Version: 2.0.50727.9136 (WinRelRS6.050727-9100)
CodeBase: file:///C:/WINDOWS/assembly/GAC_MSIL/System.Windows.Forms/<IP_ADDRESS>__b77a5c561934e089/System.Windows.Forms.dll
System
Assembly Version: <IP_ADDRESS>
Win32 Version: 2.0.50727.9147 (WinRelRS6.050727-9100)
CodeBase: file:///C:/WINDOWS/assembly/GAC_MSIL/System/<IP_ADDRESS>__b77a5c561934e089/System.dll
System.Drawing
Assembly Version: <IP_ADDRESS>
Win32 Version: 2.0.50727.9136 (WinRelRS6.050727-9100)
CodeBase: file:///C:/WINDOWS/assembly/GAC_MSIL/System.Drawing/<IP_ADDRESS>__b03f5f7f11d50a3a/System.Drawing.dll
System.Xml
Assembly Version: <IP_ADDRESS>
Win32 Version: 2.0.50727.9136 (WinRelRS6.050727-9100)
CodeBase: file:///C:/WINDOWS/assembly/GAC_MSIL/System.Xml/<IP_ADDRESS>__b77a5c561934e089/System.Xml.dll
System.Configuration
Assembly Version: <IP_ADDRESS>
Win32 Version: 2.0.50727.9136 (WinRelRS6.050727-9100)
CodeBase: file:///C:/WINDOWS/assembly/GAC_MSIL/System.Configuration/<IP_ADDRESS>__b03f5f7f11d50a3a/System.Configuration.dll
prcmftg7
Assembly Version: <IP_ADDRESS>
Win32 Version: 2.0.50727.9147 (WinRelRS6.050727-9100)
CodeBase: file:///C:/WINDOWS/assembly/GAC_MSIL/System/<IP_ADDRESS>__b77a5c561934e089/System.dll
s9a6xu9l
Assembly Version: <IP_ADDRESS>
Win32 Version: 2.0.50727.9147 (WinRelRS6.050727-9100)
CodeBase: file:///C:/WINDOWS/assembly/GAC_MSIL/System/<IP_ADDRESS>__b77a5c561934e089/System.dll
gngwky6u
Assembly Version: <IP_ADDRESS>
Win32 Version: 2.0.50727.9147 (WinRelRS6.050727-9100)
CodeBase: file:///C:/WINDOWS/assembly/GAC_MSIL/System/<IP_ADDRESS>__b77a5c561934e089/System.dll
************** JIT Debugging **************
To enable just-in-time (JIT) debugging, the .config file for this
application or computer (machine.config) must have the
jitDebugging value set in the system.windows.forms section.
The application must also be compiled with debugging
enabled.
For example:

When JIT debugging is enabled, any unhandled exception
will be sent to the JIT debugger registered on the computer
rather than be handled by this dialog box.

This is now possibly fixed in #70
Please retry with this version.

@LucasErG What are the testresults? Is it fixed?

I just scanned the folder that was consistently giving the error with version 2.3.10d and the bug didn't happen. I'm thinking there may have been a corrupted picture that may have been triggering the issue. I will continue to test other folders that gave me problems though.

for my amusement, I attempted to scan my entire pic hoard, 1,200,000+ pics. loading pictures was successful, but the comparing portion was not. The program closed itself with no error message or log at least that I know of anyway.

I can reproduce this or a very similar crash on my machine with a sufficiently large image collection pretty much every single time. It's due to CoreLib.cs failing to correctly use Marshal.UnsafeAddrOfPinnedArrayElement(). The docs for that method indicate that it MUST NOT BE CALLED on an array unless that array has been pinned or fixed. The current method does not pin or fix the array.
There is a commit https://github.com/ermig1979/AntiDupl/commit/e27f25841e86567d8b125121859164c96e11830b that claims to fix the issue by wrapping those unsafe calls in a try {} catch {} block. That doesn't fix the issue - it merely mutates it into a different one (the program silently exits, or something else doesn't work right).
If I get my act together I'll try to put up a git pull request to actually fix the issue, but I don't use git much so it may take me a while. In the meantime, if someone wants to take code from here instead, here's the modified version of StatusGet() I wrote that fixes the issue:
public CoreStatus StatusGet(CoreDll.ThreadType threadType, int threadId)         {             object statusObject = new CoreDll.adStatusW();             byte[] status = new byte[Marshal.SizeOf(statusObject)];             // MUST NOT CALL UnsafeAddrOfPinnedArrayElement() on an unpinned/unfixed array!             unsafe             {                 fixed (byte* buff = status)                 {                     IntPtr pStatus = new IntPtr(buff);                     if (m_dll.adStatusGetW(m_handle, threadType, new IntPtr(threadId), pStatus) == CoreDll.Error.Ok)                     {                         CoreDll.adStatusW statusW = (CoreDll.adStatusW)Marshal.PtrToStructure(pStatus, statusObject.GetType());                         try                         {                             return new CoreStatus(ref statusW);                         }                         catch (Exception)                         {                             return null;                         }                     }                 }             }             return null;         }

Also note that there are other calls in CoreLib.cs to Marshal.UnsafeAddrOfPinnedArrayElement() on arrays that have not been pinned or fixed. They don't seem to cause crashes anywhere near as often as the one in StatusGet() does, but they are nevertheless bugs that should be fixed.

I dealt with C# very time ago. Could you check my solution of this bug:
    public CoreStatus StatusGet(CoreDll.ThreadType threadType, int threadId)
    {
        try
        {
            object statusObject = new CoreDll.adStatusW();
            byte[] statusArray = new byte[Marshal.SizeOf(statusObject)];
            GCHandle statusHandle = GCHandle.Alloc(statusArray, GCHandleType.Pinned);
            if (m_dll.adStatusGetW(m_handle, threadType, new IntPtr(threadId), statusHandle.AddrOfPinnedObject()) == CoreDll.Error.Ok)
            {
                CoreDll.adStatusW statusW = (CoreDll.adStatusW)Marshal.PtrToStructure(statusHandle.AddrOfPinnedObject(), statusObject.GetType());
                return new CoreStatus(ref statusW);
            }
            statusHandle.Free();
        }
        catch(Exception)
        {
        }
        return null;
    }

Are there any troubles?

Hi ermig1979, I don't do much native interop so it takes me a while too. That solution has a couple of known issues:

It will leak a pinned GCHandle each time an Exception is thrown, which will make future memory allocations more fragmented. That normally won't matter much as we don't expect many exceptions.

It doesn't free the GCHandle if the DLL call succeeds, so it consumes large amounts of memory (and usually fragments the heap making the problem much worse). It hit 12GB of active private working set half way through a scenario that normally uses just over 1GB by the end.

Both issues can be fixed via a try {...} finally {...} block:
public CoreStatus StatusGet(CoreDll.ThreadType threadType, int threadId)
{
    try
    {
        object statusObject = new CoreDll.adStatusW();
        byte[] statusArray = new byte[Marshal.SizeOf(statusObject)];
        GCHandle statusHandle = GCHandle.Alloc(statusArray, GCHandleType.Pinned);
        try
        {
            if (m_dll.adStatusGetW(m_handle, threadType, new IntPtr(threadId), statusHandle.AddrOfPinnedObject()) == CoreDll.Error.Ok)
            {
                CoreDll.adStatusW statusW = (CoreDll.adStatusW)Marshal.PtrToStructure(statusHandle.AddrOfPinnedObject(), statusObject.GetType());
                return new CoreStatus(ref statusW);
            }
        }
        finally
        {
            statusHandle.Free();
        }
    }
    catch (Exception ex)
    {
        // Debug.WriteLine(ex.Message);
    }
    return null;
}

And when I test that memory usage is about typical, and it hasn't crashed yet. The old code would normally have crashed three or four times in the same amount of testing.
So that looks like it works. However it might be easier to use the newer language features since about C# 7.3 rather than messing around with allocating and freeing a GCHandle. The fixed(byte * buff = status) {} statement in my code snippet pins status in memory for the duration of that block and allows safe access via the buff pointer (see docs). However it must appear inside an unsafe context, so it's no good if you have a strict requirement to avoid unsafe.
You could also use stackalloc to allocate a local variable on the stack where its location is fixed for the duration of the method and hence doesn't need to be explicitly pinned (see docs), but I think that may also require unsafe.
And the `Span' classes are also useful abstractions (but probably not here seeing the interop has already been written).
So this comment from a .NET Platform team member with a recommendation is worth reading as a general principle, even though it was about a scenario using spans:

The preferred pattern is to use unmanaged pointer in the PInvoke signature and use fixed block to call it ... This pattern best describes the intent and avoids corner case bugs that the other patterns suffer from.

While we're on the topic, all other calls to Marshal.UnsafeAddrOfPinnedArrayElement() are almost certainly bugs because they are called on variables that are not pinned, so the variable may be moved by the GC before the DLL method tries to write to it.
I haven't seen any of those produce a crash yet, but I see bogus comparison results in the UI (e.g. pairs of image files that are identical byte-for-byte are reported to have significant differences) that might result from such a bug.
Some of those calls can probably be simplified if you dig into the PInvoke and marshalling rules and tweak the PInvoke declarations (e.g. in some circumstances blittable variables passed by reference are pinned by the PInvoke marshaller without you having to explicitly do it), but it's probably simpler to pick an implementation pattern and use it everywhere.


openapi-react-query: can't change queryKey/functionKey default behaviour OR set on use

Description
The opinionated queryKey/functionKey is [method, path, params]. This means that if a PATCH/PUT is done to book/5, no GET request to book/5 are invalidated which is very unhelpful for anyone designing a REST API and defeats a lot of the functionality of react-query. Combine the fact that I cannot set the queryKey generator function on the api on create AND that i cannot set queryKey in useQuery/useMutation and this makes this package unusable for me unless I plan on wrapping every single function, lift up the queryClient and do manual invalidation.
Proposal
I see two ways this could be soled
1. Keep the opinionated design but expose overrides

Allow the user to override the default logic for generating the queryKey.
Allow the user to set the queryKey with the options on useQuery/useMutation

2. Use a less opinionated design
Another option would be to stop being so opinionated about how the client is wrapped. The main purpose of this library is type safety and not having to repeat our definitions over and over. We could just let the user get full access to the native options for each hook and prefix if with a object used to create the queryFn/mutationFn:
const { mutate } = useMutation({ method: "GET", path: "/book/:id", params: {id: 62}, body: {title:"New title"}, tsQueryOptions... },{queryKey:["book", 62], reactQueryOptions...})
And optionally also allow data to be injected when invoking
mutate({body: {title:"Even newer title"}, ...extraTsQueryOptions}, ExtraReactQueryOptions)
Checklist

[ ] I’m willing to open a PR for this (see CONTRIBUTING.md)

For context; the app in working on uses a predictable REST api and the end goal is that the default behaviour of our api client is to invalidate the queries related to resources when they are mutated.
I've somewhat solved the end issue I'm having with automatic invalidation by hooking into the mutationCache. I'm not sure if this is useful to someone but this might better explain why I'm keen on a more react query native api. A Typescript wizard can probably solve this in a type safe way
`import { StrictMode } from 'react'
import { createRoot } from 'react-dom/client'
import { MutationCache, QueryClient, QueryClientProvider } from '@tanstack/react-query'
import App from './App.tsx'
const queryClient = new QueryClient({
mutationCache: new MutationCache({
onSuccess: (_data, _variables, _context, mutation) => {
if (!Array.isArray(mutation.options.mutationKey) || mutation.options.mutationKey.length < 2) {
return
}
queryClient.invalidateQueries({ queryKey: ["get", ...mutation.options.mutationKey.slice(1)] })
},
}),
})
createRoot(document.getElementById('root')!).render(

,
)
`

I recognise that this is a niched use case and since I've gotten it working for me, albeit a bit hacky, I'm going to close the issue.


Topicos Maio 2024

Issue para sugestao de topicos pro encontro de Maio.
Por favor incluam suas sugestoes no seguinte formato para facilitar a vida dos editores:

zkSnacks descontinuando o servico de coordenacao de Coinjoin
https://blog.wasabiwallet.io/zksnacks-is-discontinuing-its-coinjoin-coordination-service-1st-of-june

ECash: Rascunho de NUT para distribuição de recompensa de pools de mineração usando blind sigs
https://github.com/plebemineira/nuts/blob/nut-pool/X.md

BOLT12 Available in LDK: https://lightningdevkit.org/blog/bolt12-has-arrived/

libsecp256k1 v0.5
https://github.com/bitcoin-core/secp256k1/blob/master/CHANGELOG.md#050---2024-05-06

Emesbee - Coinjoins with no coordinator
https://stacker.news/items/529905

Mining Development Kit is heading to Beta
https://www.mining.build/mdk-beta/

[bitcoindev] Signing a Bitcoin Transaction with Lamport Signatures (no changes needed)
https://bitcoinops.org/en/newsletters/2024/05/08/#consensus-enforced-lamport-signatures-on-top-of-ecdsa-signatures
<EMAIL_ADDRESS>

Silent Payments BIP merged
https://twitter.com/reardencode/status/1788577217884364861

First mainnet multi-hop asset payment w/ Taproot Asset channels
https://twitter.com/roasbeef/status/1788624974728790471

Bitcoin Keeper new release with Inheritance Planning, Assisted Keys, Canary Wallets, interoperability and several other key features
https://twitter.com/bitcoinKeeper_/status/1788599293152301281

Cashu.me 0.1 – Release
https://primal.net/e/note150ywd3rmpw086uwus22kyf45gvqk78r6dl8qm4rv65jem49hglts3um34a

bitcoin++ ecash - berlin - Oct 10 - 12, 2024
https://primal.net/e/note1msmj6unvdqnzeesafq28933nydv5wr6mfznslu0zh3gcu50td5wqxtj3tt

Introducing Boardwalk Cash - the first dollar-based Cashu wallet built on top of #bitcoin and connected to #nostr.
https://primal.net/e/note178du87hqdtfyuflpfznq5m3lmgc3sae0ur29qda7v2fyjlzwh0nskfazq7

Beta release of utreexod
<EMAIL_ADDRESS>

Silentium - Silent payments enabled wallet
https://twitter.com/TheSingerLouis/status/1790824126472667227

BitScrow announces Developer Playground
https://twitter.com/BitEscrow_app/status/1791086192089329785

Securing Git: Addressing 5 new vulnerabilities
Git is releasing several new versions to address five CVEs. Upgrading to the latest Git version is essential to protect against these vulnerabilities.
https://github.blog/2024-05-14-securing-git-addressing-5-new-vulnerabilities/#details

Ecash TIDES using Cashu and Stratum v2
https://delvingbitcoin.org/t/ecash-tides-using-cashu-and-stratum-v2/870/1

Blinded Authentication Tokens in Mutiny
https://blog.mutinywallet.com/blinded-authentication/

Silent Payments: Light Client Protocol
https://delvingbitcoin.org/t/silent-payments-light-client-protocol/891
https://github.com/setavenger/BIP0352-light-client-specification

BIP de Miniscript por A. Chow
<EMAIL_ADDRESS>

Shouldn't we close this @lorenzolfm, @lucasdcf, @jaonoctus?


diozero description is incorrect

limited to JDK 11. Almost 600 Java files compared to 33 for Java Periphery. Root access required in some situations.

diozero works with Java 8 onwards. The ~600 classes in not a fair comparison - the difference is related to completely different goals between the two libraries. As well as providing a lot of device classes, diozero includes its own portability layer with built-in default implementation and can work on Firmata devices like Arduino. java-periphery delegates to c-periphery and provides just low-level interfaces.
Also, diozero itself has no specific requirement to run as root - it purely depends on the O/S config itself and can usually be corrected by additional config, e.g. device tree permissions at boot.

Totally fair comparison Java file count wise. Something with 600 files is inherently more complex and harder to test. As far as project focus my focus is clearly userspace IO (which is clearly stated). Again a fair comparison for someone looking for a simple IO solution. I make no claims otherwise.
So you regression tested everything with JDK17? I've done several large conversions from JDK 8 to 11 and all the transitive dependencies broke things. I see nowhere on your site where you support JDK 17. Saying you support Java 8 forward is disingenuous if you haven't tested "forward" specifically.
Also, can you show an implementation without root access for all IO services? Again I'll correct anything inaccurate.

All I/O in diozero is userspace other than the memory mapped GPIO control which is the same as for java-periphery.
Yes, I do test with Java 8, 11 as well as 17 - no issues whatsoever.
The comparison on the number of classes would benefit from clarification - that diozero provides a full portability layer as well as high-level abstractions for I/O and includes support for a number of common categories of devices (ADCs, GPIO expanders, LEDs, OLEDs, motors, ...). Of course this does add complexity but that is what unit tests are for.

The project comparison is very clear I'm only comparing IO (not a device library which can be easily ported from other projects). I need no portability layer as that's what the kernel provides. Also, I took a totally different approach with MMIO by using property files. No one off code needed for each supported board. I also provide the tooling to build these files without having to hand code from a datasheet. I didn't put that in the comparison. I removed the JDK limitation from the readme. If you provide configuration showing no root access required I'll remove that too.
Are you using mocks for your tests?

I provide an I/O abstraction layer as I want full portability - java-periphery uses c-periphery as the I/O abstraction layer. Both of these use the exact same kernel interfaces. For this reason the two libraries have the exact same root access requirements.
Yes, I use some mocks - because diozero has its own I/O implementation it was fairly trivial to mock GPIO, I2C, SPI, serial device communication. I will always need additional unit tests.

OK, so mocks are worthless in my opinion as they test nothing other than pre canned data unless you are using emulators on all the devices. Can you articulate what full portability is (you keep saying this, but userspace IO is already fully portable)? Yes, java-periphery uses c-periphery as you have stated several times. This is not a negative. It's a well built library. No use creating the wheel again. Do you write everything from scratch or so you leverage libraries and frameworks? I also generate all the JNI code as where you chose to hand code it. Does that make your code any better? I'm just leveraging these technologies to my benefit and it produces better quality code in my opinion.

This isn't a discussion as to which is the best approach and I'm not stating anything negative about c-periphery or java-periphery, just correcting statements regarding diozero on your project page. c-periphery is clearly a fantastic library that has enabled a number of language bindings - that has not been my goal at all.
We have quite different goals, but fundamentally use the exact same kernel userspace APIs. Yes, the diozero approach has meant that I have had to implement some of what c-periphery implements but I wasn't aware of that library at the time.
Regarding "full portability", I'm referring to the diozero pluggable architecture that also allows communication with Arduino / Pico devices via Firmata as well as over the wire protocols such as gRPC.

I think you are missing the big picture. I'm only comparing IO features and I made changes to the comparison already (thanks for reminding me about the MMIO stuff). The protocol layers are just unnecessary complexity as far as I'm concerned if you are looking for a Java IO layer (a lot of developers just want a cross platform wiringPi). As you stated we have different goals, but you can see where the projects intersect, right? I'm just contrasting the differences and made adjustments based on this discussion.

OK, list the libraries that have the same level of SBC support (yours and mine maybe a couple more?) I'd say most do not have the same level of support is a true statement. diozero was in the same boat as the rest of the libraries until you adopted userspace method later in the game (I started this with my Userspace IO project in the beginning). Take a peek at your commits. Sure you can adopt any approach for MMIO, but I'm comparing to today, not the future. In any event I've addressed all your concerns that are material to my comparison which I'll let stand since it's the truth. Why not make your own comparison, so people can contrast your project against others. It's helpful if they are looking for specific features.
Like I said a lot of people could care less about Firmata. A lot of devs are using the SBC to do everything. It all depends on the application. If I wanted to support Firmata honestly I'd just get a good C library and generate the JNI wrappers. No need to code protocol from scratch.

diozero has adopted userspace kernel interfaces from the very start. Let's end this discussion as I don't think you're going to update the text further and are missing my points.

I'm not missing any points. Just go back and look at your "providers" in commit history (libgpiod only came out in kernel 4.8). Like I said I changed what you proved was incorrect, you should be happy with that instead of rambling on. As far as I'm concerned it's 100% accurate now. 600 classes vs 33 true, hand coded MMIO vs configuration driven true. Sorry you don't like those facts, but that's all they are.

Prior to that I used sysfs like already stated.

Prior to that I used sysfs as already stated. Anyway...

sysfs was deprecated  3 years ago :) Anyways all good, the comparison is up to date now.


Update diagnostic settings configuration

Change screenshot enable-diagnostic-logs.png to enable-diagnostic-logs_2022.png to update with the news design of Azure
Delete  "Specify the number of days for which the data must be retained. Retention is only applicable if you are using Azure storage account to archive log data." because it's not anymore available in 2022

@normesta

Can you review this PR?
If you approve the changes, please note that the file name for the newly added jpg must conform to the guidance (no underscores) and that the screenshot will need code that adds the gray border.

IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Looks good! Thank you!
#sign-off

@normesta @EtienneSIG  - we can't merge the image into the repo because the filename contains and underscore. Can you update this PR by changing the filename to articles/data-lake-store/media/data-lake-store-diagnostic-logs/enable-diagnostic-logs-2022.png ? Then update the article to match?
Thanks.
#hold-off

Sure. @EtienneSIG I created a pull request into your branch for the image reference that is located in the doc. I don't have permissions to upload a replacement image to your branch. If you can rename that image & merge my PR, we can complete this PR. Thanks!

Hi @EtienneSIG, I added another pull request to change the image name. No need to rename the file. If you merge my pull requests into your branch, we can merge this PR. Thx!

I went ahead and updated the image on our side. Closing this PR as dup. Thank you for raising!
#please-close


enh(gcode): rewrote language for moden gcode support

Complete rework of the gcode language to allow for extended uses cases.
Not all scope rules will apply to all implementations of g-code, but many applications of gcode have added a lot on top of the original spec.
This language implementation aims to be more extensive but still flexible.
My research has used the following documentations:

LinuxCNC
RepRap
ISO 6983

And countless code examples extracted from GitHub's search:

GitHub Seach For GCode
GitHub Seach For NGC

Changes

More keywords
Stricter numbers matching (c-style numbers are too broad)
Differentiation between functions (G-Codes, M-Codes, …), Axes (ABC, UVW, XYZ), Parameters (P, Q, R, S, …)
Semicolon comments
Quoted strings
Word commands

Question about code
I have used this pattern to re-use complex regexes:
const NUMBER = /[+-]?((\.\d+)|(\d+)(\.\d*)?)/;
const match = new RegExp(`(?<![A-Z])[FHIJKPQRS]\\s*${NUMBER.source}`)

I would like to use this for certain duplicated parts (e.g. (?<![A-Z])) as well. Is this acceptable?
The performance impact is surely negligible since this only happens during initialisation.
It greatly increases readability and IDE support in my opinion.
Screenshots

Name
Before
After

Default

Extended

Checklist

[x] Added markup tests
[x] Updated the changelog at CHANGES.md

@joshgoebel thank you for your patience.
My latest commit added disableAutodetect: true to the language, as well as the aforementioned on:begin filter.
I added the filter as a variant to full matches with \b.
This means that readable gcode with sane spacing will rarely, if ever, run into the callback filter.

I've tested both implementations with the existing markup test, as well as some 100lines of spaceless gcode.
The results are always within 5% of each other, always in favor of the v12 language with lookbehinds.

Running Benchmark & Results
Shell commands to download v12 and install benny
wget https://raw.githubusercontent.com/barthy-koeln/highlight.js/2c55db96e8a0523fdbdd6d4069c7007f75d5288b/src/languages/gcode.js -O src/languages/gcode-v12.js
npm run build gcode gcode-v12
cd build
npm install benny

# create benchmark script here e.g. bench.mjs

node bench.mjs

Benchmark script
import b from 'benny'
import fs from 'fs'
import hljs from 'highlight.js'

const code = fs.readFileSync(import.meta.dirname + '/../test/markup/gcode/extended.txt').toString('utf-8')

b.suite(
  'gcode bench',

  b.add('v11', () => {
    hljs.highlight(code, { language: 'gcode' })
  }),

  b.add('v12', () => {
    hljs.highlight(code, { language: 'gcode-v12' })
  }),

  b.cycle(),
  b.complete(),
  b.save({ file: 'gcode', version: '1.0.0' }),
  b.save({ file: 'gcode', format: 'chart.html' })
)

Results

Hey @joshgoebel, this is just a careful ping to see if I can or should do anything else here?
I need this fix for a freelance project, but can use my fork for the initial release.


Implement tessel init

Need example files so I can load them into the root for auto init but this should work now. :)

Fixes #28


account setting language provider need to support Arabic

https://github.com/wagtail/wagtail/blob/9afcd1a8be0d1538dfdcfdf5af277c50d57d7044/wagtail/admin/utils.py#L27

support Arabic language

WAGTAILADMIN_PROVIDED_LANGUAGES = WAGTAILADMIN_PROVIDED_LANGUAGES+ [('ar', ugettext_lazy('Arabic'))]

pull request


[Bug] It does not work inside properties

As the example below shows, it is possible to copy and paste emojis inside a property. However, the autocomplete feature does not work, and a short code does not turn into an emoji while the same properties are being edited. I think the short code detection system has a blind spot when it comes to a note's properties.
It's strange because Obsidian notes are just plain text files, even the properties. The fancy interface is only there to match the current theme.

sorry, but the properties core plugin only uses a <div contenteditable></div> instead of the proper editor like the rest of the file.
It's not possible to create a suggestion windows there.
if you want to use the emoji autocomplete in the properties, use source mode (or set up a hotkey to toggle between source mode & live preview).
You could also use the system emoji picker or copy & paste the emoji in.


Add Fluentbit installation + configuration

We should be able to install and configure Fluentbit with tdp-collection-prerequisites in order to use the sample Fluentbit configuration files to tail the logs of the TDP services we will implement in tdp-collection with https://github.com/TOSIT-IO/tdp-collection/issues/613

Closing in favor of https://github.com/TOSIT-IO/tdp-observability/pull/24


more small cleanup tasks

addressing everything in #150, including URLs for cincho, location bar queue fixes, removal of the favorability quote re: shen snakes, and other stuff.

... oh god i never actually merged these did i? god i hope this doesn't create awful collisions with the LoL mega-PR


TypeError: Cannot read property '0' of undefined

I'm having an issue running the Gulp plugin for CSScomb that I can't seem to figure out for the life of me, and was hoping someone else could take a look and maybe shed some light. I posted this issue on the plugin's GitHub, but there's more of an audience here and was hoping to get more eyes on it. Thank you in advance!
events.js:72
        throw er; // Unhandled 'error' event
              ^
TypeError: Cannot read property '0' of undefined
    at module.exports.process (/Users/raltvater/Sites/www-static/node_modules/gulp-csscomb/node_modules/csscomb/lib/options/eof-newline.js:16:26)
    at /Users/raltvater/Sites/www-static/node_modules/gulp-csscomb/node_modules/csscomb/node_modules/csscomb-core/lib/core.js:206:21
    at Array.forEach (native)
    at processNode (/Users/raltvater/Sites/www-static/node_modules/gulp-csscomb/node_modules/csscomb/node_modules/csscomb-core/lib/core.js:202:14)
    at /Users/raltvater/Sites/www-static/node_modules/gulp-csscomb/node_modules/csscomb/node_modules/csscomb-core/lib/core.js:189:13
    at Array.forEach (native)
    at processTree (/Users/raltvater/Sites/www-static/node_modules/gulp-csscomb/node_modules/csscomb/node_modules/csscomb-core/lib/core.js:188:19)
    at processString (/Users/raltvater/Sites/www-static/node_modules/gulp-csscomb/node_modules/csscomb/node_modules/csscomb-core/lib/core.js:415:16)
    at DestroyableTransform._transform (/Users/raltvater/Sites/www-static/node_modules/gulp-csscomb/index.js:65:27)
    at DestroyableTransform.Transform._read (/Users/raltvater/Sites/www-static/node_modules/gulp-csscomb/node_modules/through2/node_modules/readable-stream/lib/_stream_transform.js:184:10)

I has same error
» cat test.sass
.page-sidebar
  background #ccc

» csscomb -v  -l test.sass
stack:  TypeError: Cannot read property '0' of undefined
    at module.exports.process (/usr/local/lib/node_modules/csscomb/lib/options/eof-newline.js:16:26)
    at /usr/local/lib/node_modules/csscomb/node_modules/csscomb-core/lib/core.js:206:21
    at Array.forEach (native)
    at processNode (/usr/local/lib/node_modules/csscomb/node_modules/csscomb-core/lib/core.js:202:14)
    at /usr/local/lib/node_modules/csscomb/node_modules/csscomb-core/lib/core.js:189:13

@ryanaltvater @lightningtgc, this should be fixed in dev branch.
@leonko, that's a parsing error actually, I've opened an issue here: https://github.com/tonyganch/gonzales-pe/issues/62


Math Blocks justified in live preview

Seems related to other bugs #4 and #5. Math blocks are justified in live preview, however this behaviour is not seen in inline math.

Is this still the case? I can't seem to reproduce it in a sandbox:


Wrong import of Kerning groups

Sometimes the "Kering Classes" from the UFO files end up as "Opentype Classes" in the exported Glyph file. These is wrong.
See attached image: Empty kerning groups in the columns, and kerning classes wrongly imported as opentype classes

Instead, they should end up being "Kerning Groups", as displayed in the next image:
Note that the groups columns are now filled, and the opentype classes where removed.

Can you send me the ufo?
Georg

Am 18.08.2016 um 15:59 schrieb Pablo Impallari<EMAIL_ADDRESS>Sometimes the "Kering Classes" from the UFO files end up as "Opentype Classes" in the exported Glyph file. These is wrong.
See attached image: Empty kerning groups in the columns, and kerning classes wrongly imported as opentype classes
https://cloud.githubusercontent.com/assets/3036973/17793459/d14ae092-657d-11e6-9492-2d2422c1f13d.png
Instead, they should end up being "Kerning Groups", as displayed in the next image:
Note that the groups columns are now filled, and the opentype classes where removed.
https://cloud.githubusercontent.com/assets/3036973/17793507/352071ea-657e-11e6-9f57-770472764014.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/googlei18n/glyphsLib/issues/71, or mute the thread https://github.com/notifications/unsubscribe-auth/AAKqRBlBEPqCHdkoTnRVTTHvLc3EhBUrks5qhOPigaJpZM4Jn_3U.

The source and converted files are here: https://github.com/m4rc1e/Raleway

The .ufos in the Raleway/old/version-3.000/src/ folder contain FontLab style kerning. That is not supported in Glyphs and probably nowhere else, too.

From a coarse glance it looks like this bug can be closed. If anyone disagrees, feel free to re-open it.