evtxdump UniDecodeError with MS DNS Server log file

Firstly, thanks for writing this library. I'm having an issue with converting binary evtx files that are generated using Windows DNS server. I'm assuming that the error is being caused by a field containing maybe a DNS request that has characters that cannot be decoded to ascii. This would be pretty normal when dealing with DNS queries. See output below....
Traceback (most recent call last):
  File "evtxdump.py", line 53, in <module>
    main()
  File "evtxdump.py", line 48, in main
    for xml, record in evtx_file_xml_view(fh):
  File "build/bdist.macosx-10.12-intel/egg/Evtx/Views.py", line 221, in evtx_file_xml_view
  File "build/bdist.macosx-10.12-intel/egg/Evtx/Views.py", line 172, in _build_record_xml
  File "build/bdist.macosx-10.12-intel/egg/Evtx/Views.py", line 163, in rec
  File "build/bdist.macosx-10.12-intel/egg/Evtx/Views.py", line 156, in rec
  File "build/bdist.macosx-10.12-intel/egg/Evtx/BinaryParser.py", line 64, in __call__
  File "build/bdist.macosx-10.12-intel/egg/Evtx/Nodes.py", line 945, in fast_substitutions
  File "build/bdist.macosx-10.12-intel/egg/Evtx/BinaryParser.py", line 475, in unpack_string
UnicodeDecodeError: 'ascii' codec can't decode byte 0x96 in position 16: ordinal not in range(128)

thanks for reporting this issue! as you presumed, there's an issue parsing some string data. from what i can gather from the stacktrace, there's data declared as either ascii or utf-8 that doesn't get extracted correctly. i'm not sure if its bad data, or valid utf-8 data that's being shoveled into the ascii decoder.
unfortunately, due to the evtx structures, without an example file, i'll have a hard time preparing a fix. are you able to provide a test log file that i can inspect? if not, perhaps we can try to find an alternative, though it might take a bit longer.

Hi, thank you for the reply. I understand it will be difficult to track down the problem without a sample log file. However, its a bit tricky for me to send you the log file I'm working on as it contains info on DNS lookups from with our organisation along with source IP etc. I might be able to look at the files using MS DNS server log file viewer in order to isolate the offending entry. If I can do this I could at least send on a smaller sample. I will see if I can work on this during the week and send on what I can.
I stumbled across your library because I need to analyse DNS log files and there is a bug in MS DNS server 2012/2016 that causes the exporting of evtx log fles as CSV or XML to fail. Our techs logged a call to MS and they said they are working on a fix but no timeline. I do most of my analysis using Python so your library is a great fit for my problem.
Regards
Stephen

ok, i completely understand. i certainly wouldn't want my logs shipped off to some guy on the internet ( ;-) ), so i don't expect you to. let me brainstorm a procedure for triaging this. if its affecting you, its probably affecting other people, so i'd really like to get this fixed up.

Hi,
I added a try block to my own code to see could I figure out whereabouts in the log file was causing the encoding error. I found the point where the xml conversion crashed and then used MS Event Viewer to seek to that point in order to look at the event contents. It looks like a bug in the code that generates the log file in the first place as the MS Event viewer is unable to read the problematic event, see attached screen grab of the Event Viewer.
I wonder is there a nice way to handle XML events that are NOT "well formed"?
I have isolated a small section of the evtx log file that I can share with you if you like?

huh, this is tricky. there is a routine we can used to strip out the invalid string data; however, typically i avoid using it cause it tends to hide my bugs when dealing with ascii/unicode. here, it seems fair to use that function.
i'll implement and test a fix today.
if you're able, can you send me a fragment of that example file that contains the malformed string shown in the screenshot above? perhaps use a hex editor to extract from a few bytes (16? 32?) before the string, until a bunch of bytes afterwards (another 32 bytes?). feel free to redact sensitive bytes by replacing them with A or X.

Hi, Sorry for the delay.
I extracted the offending records into a separate evtx file.
Tried to edit in hex editor to anonymise  but for some reason was corrupting file, tried to load back into MS event viewer and was getting corrupt file error.
So eventually gave up and exported offending sample as plain XML, anonymised and have attached.
NOTE: I have renamed to .txt in order to attach
dns-problem-log-xml.txt
You can see the offending characters in the QNAME field. I am not entirely sure why strange characters are appearing, code be a bug in the DNS server log generator or code well be strange encoding error.
Maybe you code revert attached file back to evtx format?
Regards
Stephen

i've just finished a substantial amount of work to refactor the encoding and handling of xml data in this library. i suspect this issue should be fixed, but without the source data, I cannot confirm. please try again using v0.6.0 onwards, and reopen the issue if anything crops up!
closed in 4b4ba9569c2dc1e7aa3d98d76ba92ce08d8ee0d7


⚠ Bofrost.es has degraded performance

In 0106bea, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1458 ms

Resolved: Bofrost.es performance has improved in 31e2381 after 6 minutes.


[Question] Speed of GPTQ transformer vs MLC ?

Hi,
Thanks for this great library!
I have one question since Transformers has enabled inference of GPTQ models: why inference of quantized (GPTQ) model with MLC faster than quantized (GPTQ) model with Transformers library?
Is this because of the C++ compilation or something else?

BTW which specific repo are you referring to?

https://huggingface.co/lmsys/vicuna-7b-v1.5 for the non quantized model.
And https://huggingface.co/TheBloke/vicuna-7B-v1.5-GPTQ for the GPTQ one.

As the underlying compiler that AutoGPT uses, Triton isn't actually performant in most of the cases I've seen. It's programming interface is really nice but probably isn't good for squeezing performance.


Create tutorial "Make your Kubernetes cluster - and the apps running on it - obseravable"

Proposed in https://github.com/cncf/tag-observability/issues/16#issue-626401107
Note sure if it should be part of the whitepaper. It feels like a separate blog/turorial would be adequate.
As a response to https://github.com/cncf/tag-observability/issues/16#issuecomment-1633867650

Hi, can you assign this to me? i would love to work on it

Hi @bwplotka  If the issue has not been worked on I'd like to work on it

@bwplotka do you envision this similar to the TAG App Delivery Potato Head tutorials? https://github.com/podtato-head/podtato-head
Maybe a selection of tutorials could be based on the potato head app and address different observability goals/ adding and using different types of telemetry?

To add, what do you think of working with Tag App Delivery to add a repository to the potato-head GitHub organisation on observability?

To reduce duplicating efforts
Have a separate repository at a place where people can find the tutorials instead of adding the tutorials to the TAG Observability Repo https://github.com/cncf/tag-observability

A tutorial such as this may well fit under the Observe k8s WG, which is part of TAG Observability


Docker build failed at cloning sources.

I'm trying to build docker image, but it failed at cloning sources.
$ docker build -t nis-cli .
[+] Building 2.7s (7/10)
 => [internal] load build definition from Dockerfile                                                                        0.0s
 => => transferring dockerfile: 913B                                                                                0.0s
 => [internal] load .dockerignore                                                                                0.0s
 => => transferring context: 2B                                                                                0.0s
 => [internal] load metadata for docker.io/library/ubuntu:20.04                                                             0.0s
 => [1/7] FROM docker.io/library/ubuntu:20.04                                                                               0.0s
 => CACHED [2/7] RUN apt-get update   && apt-get install -y tzdata   && apt-get install -y openjdk-8-jdk-headless git libs  0.0s
 => CACHED [3/7] RUN mkdir -p ~/.ssh && ssh-keyscan -t rsa github.com >> ~/.ssh/known_hosts                                 0.0s
 => ERROR [4/7] RUN --mount=type=ssh mkdir -p /build   && cd /build   && git clone git@github.com:NemProject/nis-client.gi  2.5s
------
 > [4/7] RUN --mount=type=ssh mkdir -p /build   && cd /build   && git clone git@github.com:NemProject/nis-client.git   && cd nis-client   && git submodule update --init --recursive:
#7 0.252 Cloning into 'nis-client'...
#7 1.875 Warning: Permanently added the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts.
#7 2.457<EMAIL_ADDRESS>Permission denied (publickey).
#7 2.458 fatal: Could not read from remote repository.
#7 2.458
#7 2.458 Please make sure you have the correct access rights
#7 2.458 and the repository exists.
------
executor failed running [/bin/sh -c mkdir -p /build   && cd /build   && git clone git@github.com:NemProject/nis-client.git   && cd nis-client   && git submodule update --init --recursive]: exit code: 128

Fixed in #3 (b869a347a440e0df5a11ef29b31e680b0137ea2b).


protobuf_25: 25.2 -> 25.3

Description of changes
Changelog: https://github.com/protocolbuffers/protobuf/releases/tag/v25.3
cc @jonringer

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Yes, they are caused by python3Packages.protobuf not building with this version.
It was already the case with 25.2, hence not a regression.
I am working on a fix before updating the default protobuf to protobuf_25.

protobuf_25 and python3Packages.protobuf builds on master, where is the regression coming from ?

python3Packages.protobuf builds fine on master because it relies on the sources of pkgs.protobuf, which is protobuf_24.
What doesn't build is the python binding of protobuf_25 (which cannot be directly built, you can however set protobuf = protobuf_25; in all-packages.nix).

thank you for clarification, what is the merge plan for this ? merging this breaks protobuf_25.tests

thank you for clarification, what is the merge plan for this ? merging this breaks protobuf_25.tests

I would encourage a merge of this PR as it doesn't bring any regression.
protobuf_25.tests are also broken on master for the same reason (i.e. python3Packages.protobuf's tests fail).

kirillrdy thanks for keeping a keen eye on this.
I agree with Gaetan that merging this does not break protobuf_25.tests. But also, doesn't fix protobuf_25.tests. It just upgrades the Go package. Which was my initial reasoning on this too. (I had been using protobuf in Go, but not Python.)
To show master is already broken:

So I think, merging this makes sense. As it only affects protobuf_25. And does not cause breakage of other things. And if Python part is fixed. It should be fixed for the new version. So it is an advancement still and desirable to have this PR merged.

Maybe because we know that python test is specifically broken, we should mark it broken?

Maybe because we know that python test is specifically broken, we should mark it broken?

The C++ library (pkgs.protobuf_25) itself is not broken. Also, the only version of the python bindings that exist in nixpkgs (i.e. python3Packages.protobuf) is built against the default protobuf (24.4 currently) and is not broken.
So no package is actually broken currently.

I meant the test that is broken. Not the package.

Something like this could work:

But there may exist a better way to do this.

@superherointj I directly marked the python package as broken for protobuf newer than 25. Does that work for you ?
The test is automatically marked as broken for protobuf_25.

@superherointj I directly marked the python package as broken for protobuf newer than 25. Does that work for you ? The test is automatically marked as broken for protobuf_25.

Yes. That is a much better idea indeed.

Whenever marking a package broken it is usual to add a comment for:

date
error
tracking issue
That is still missing.

Whenever marking a package broken it is usual to add a comment for:
* date

* error

* tracking issue

Sure ! Done


Transaction pool rejects transaction which sets and immediately uses the verification key

It should be possible to send a zkapp deployment account update (i.e., setting the verification key) and another account update which already uses proof authorization, in the same transaction.
The tx logic which is embedded in snarkyjs also allows such behaviour.
However, a user reports that the graphQL endpoint returns an error:
Couldn't send zkApp command: (Verification_failed "Missing_verification_key: [B62qmjXoXSKzrMGSncSoAnh9i7LhTRuuTWdDbdAe8Nxbo5DH8X7zdWq]")

This seems to indicate to me that the graphQL endpoint doesn't use the verification key updated in the same transaction.
Here is the failing transaction:
https://app.zenhub.com/files/114581326/913f6a1c-f468-496b-9674-86e5ab64bd96/download

Unfortunately, this is not possible in the current version of the protocol. If you just set the verification key then you need to wait for that account update to be included in the transition frontier and the staged ledger being updated, then you can use that verification key in your account update.

I’m fixing that here:
https://github.com/MinaProtocol/mina/pull/12398

waiting on #12380


Crontab MAILTO variable

Once the project is finalized, do you think we need to add others to this variable? I know that many of the other projects send emails to the larger group email list and Fil gets updates.
 That said, I would recommend adding at least one other person's email from the developers' team.
Also, I will add myself to keep an eye on this while I am still around.

closed via #42


nanoScroller can't be destroyed multiple times

I'm using nanoScroller on a responsive site where under 700px page width nanoScroller isn't required. I've set things up so that if nanoScroller has been initialised and the viewport width changes to below 700px it gets destroyed. It seems that if it is once again initialised, it then cannot be destroyed again.
I've set up a JS fiddle: http://jsfiddle.net/qrmv2Lqr/2/
If you play with the width of the pane, you see that after one 'destroy' it starts to act up.

Thanks for reporting!

@freakyleaf I had a look at your demo, but I wasn't seeing anything weird. Which browser are you using to reproduce the bug?

using his fiddle I was able to reproduce on chrome 42 on linux,
Simply drag the viewport so its size gets below 500px, then drag it back out. Nano Scroller should now have been destroyed once. Now drag the viewport again so its below 500px and see that nano scroller was not destroyed again.

@nordfjord Thanks, I'll try to do that


Autocreate and assign Groups from IDP

Describe the problem
I wanna assign netbird users the same groups they have in authentik, so that i can manage them central. But when i tryed it, it didn´t work. It´s an fresh installed netbird on an vps with public IP and an authentik behind traefik. But it should configured right, because other OpenID/OAuth2 Apps work perfectly.
To Reproduce
Steps to reproduce the behavior:

Install Netbird with authentik like described here
After Installation you should be able to Login successful.
Now go to Settings -> Groups
Enable: user group propagation & JWT group sync
As JWT claim chose "groups" and save.
I also added groups as "scope" unter: "NETBIRD_AUTH_SUPPORTED_SCOPES" in setup.env

Expected behavior
I expected that when a new user logs in, his Authentik Groups will be created and assigned in Netbird.
Screenshots

Additional context
Iam using Authentik with LDAP Backand and other OPENID Applications can get the groups with the groups scope.

Any Updates on that topic.

@NiclasPe have you configured any Scope Mapping in authentik to ensure the groups are being sent as part of the JWT claims?

@mlsmaycon That was the clue!
Although I entered “groups” in each scope field, I forgot to set the option framed in the photo. Now it works as expected!
Thanks alot!

Thanks for the feedback. Would you share the steps here that will help us update the documentation for Authentik?

Of course:
You have to do everything as described here: https://docs.netbird.io/selfhosted/identity-providers#authentik
The only Thinks to change are:

Set the Option "Include claims in id_token" in the Provider Configuration for Netbox in Authnentik (photo above)

When you set the  properties in the setup.env file, you need to add groups to the NETBIRD_AUTH_SUPPORTED_SCOPES. It should look like this:
NETBIRD_AUTH_SUPPORTED_SCOPES="openid profile email offline_access api groups"

After the first successful login into Netbird, go to the Settings tab. Under "Groups" activate: "Enable user group propagation" and "Enable JWT group sync". The JWT claim must be set to "groups".

If you perform an relogin, the groups will be taken over from Authentik

Thank you @NiclasPe; appreciate it.

Hi, I have followed the above steps to setup, but when I setup the jwt claim it says "please check the fields and try again".

What should I do?

@HiFallMaple, please update your dashboard image with the commands below; there was an issue fixed yesterday.
docker compose pull
docker compose up -d --force-recreate dashboard

Awesome!!
thanks for the feedback @HiFallMaple

Is it possible to configure these settings using the API? Or using any other automated-method?

Is it possible to configure these settings using the API? Or using any other automated-method?

Good Question, i think its not at the moment, why did you need that? Its an one time Operation per Instance. Do you have these many Instances?

@MohammedNoureldin you can use the accounts API: https://docs.netbird.io/api/resources/accounts#update-an-account

Hi @mlsmaycon, thank you, yes I figured it out and tested it. I posted that also on Slack. IMO, it is kind of unconventional API design, that is why I was a bit unsure in the beginning. I mean to edit an array with a single element that will then be applied for all other users/accounts.

@NiclasPe @HiFallMaple
Hello, faced with issue when deleting user from group in IdP (Zitadel) doesn`t affect netbird groups, user in netbird still member of group. Does in reproduce in your setup too or yours IdP sync groups with netbird correctly?

@1ndef1n1te
I use Netbird in Combination with Authentik for testing and there everything works fine.
After a Refresh or Re-Login the Groups will be updated.
Witch Netbird Version do you use? Anything in the Logs?

@NiclasPe
Now I am using the following setup:
netbirdio/dashboard:v2.4.1
netbirdio/management:0.28.6
netbirdio/signal:0.28.6
coturn/coturn:latest

The problem is very annoying because even after re-login user still in the group in Netbird dashboard while in IdP no

As mentioned, I don´t know / understood Zitadel well. The Netbird versions you use are up to date an the same as mine. I would suggest to check your Configuration against the Documentation and check your log files on both sides.
Otherwise we can only guess what the problem could be.
I also understood that that Problem is maybe anoying, but please keep in mind that this is a opensource Projec. So if you need priority, fast or individual Support feel free to contact the developing Company.
Please also check if you checked the first option in Netbird here:


Prettify Changes JSON Data Types From Float To Int

Hi
I'm running the latest Bruno release and I've come across an issue with running prettify converting float values to ints.
Before running prettify:

After running prettify:

This completely changes the datatype if there are trailing .0's which is raising validation errors by my service.
I've not tested any other datatype conversions so I'm unsure the extent of this issue.
For reference Postman's Beautify behaves how I would expect a formatter too:

Thanks

Probably because the "prettifying" is done using JSON.stringify/JSON.parse, we need to start using the lossless-json everywhere.

is there a ticket for the lossless-json work? i agree this is an important change

I've implemented the lossless-json changes in the PR #2006

Thanks for the PR @DJSaunders1997 !
It has been merged.
This will go out in the 1.13.0 release. Please subscribe to Discord announcement channel to stay notified on the releases.


Fix for heap access violation

Description of changes:
Fixes a heap access violation found when implementing AppVerifier into the C++ V2 SDK CI. AppVerifier finds the issue at the end of running a sample and, according to the logs provided by AppVerifier, it is caused by an access violation caused by calling CertFreeCertificateContext in windows/secure_channel_tls_handler.c in the s_secure_channel_ctx_destroy function.
The issue seems to be that CryptDestroyKey frees the private key certificate context and so then when CertFreeCertificateContext is called to free the private key certificate context further down, it leads to the access violation since it is already free. Freeing the private certificate key store first, before the private key, causes the same issue when trying to free the private key.
The fix implemented is to check if the private key was loaded from the private certificate store and, if it was, not delete the certificate store on shutdown.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Thanks! Merging into main...


Broken Link

Broken link
A description of the issue:
The link for !thisisfine in COMMANDS.md is broken. https://giphy.com/gifs/comic-adult-swim-this-is-fine-3o6UBpHgaXFDNAuttm
Proposed Solution:
Change it.

This was fixed, thanks for letting us know.


[Bugfix] Fix batchify bug

Description
The previous batchify.Stack() has not kept the input dtypes.
Checklist
Essentials

[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage
[x] Code is well-documented

Changes

[x] Set the output of the stack batchify function to have the same dtype as the input.

Job PR-198/3 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-198/3/index.html


Lots of "note" has been modified externally, merging changes automatically when update-time-on-edit enabled.

I don't know if this is something that's happening with the insider version of Obsidian (1.4.2)  or the update-time-on-edit plugin (version 2.0.2), but I've been getting lots of the following messages.
"Note title" has been modified externally, merging changes automatically.

Sometimes the messages come one right after another. Sometimes, it will start immediately upon typing in a note, and other times, it doesn't for a brief period of time. I noticed the plugin no longer has the setting in the Obsidian interface to change the time between updating and is updating almost immediately after and during typing in a note.  There is a setting in the data.json file, and it's set to one minute, but the plugin doesn't wait one minute to update the notes.
  "minMinutesBetweenSaves": 1,

When I disable the plugin, these messages stop. I have a number of other plugins enabled. I could disable them all and try again, but none of them update notes like this one does, and I really like/want what this one does.
https://github.com/beaussan/update-time-on-edit-obsidian/assets/4204038/a24077ce-f3dc-499e-b684-d36a460f2719

I am on Obsidian 1.37 and noticed a similar issue with 2.02.  I never had a issue previously with Update Time on Edit.  I noticed the issue when some files that I was editing would suddenly erase keypresses that I just had made a few seconds earlier.  I noticed a lot of sync activity going on with the last few tiles I edited.  I also noticed that the time kept incrementing every 10 seconds in front matter and it would of course sync that change.   The file just kepg going around and around.
I disabled the plug and the syncing resolved itself.  Hope this helps.

I, first of all, thanks for the issue !

I noticed the plugin no longer has the setting in the Obsidian interface to change the time between updating and is updating almost immediately after and during typing in a note. There is a setting in the data.json file, and it's set to one minute, but the plugin doesn't wait one minute to update the notes.

Yes, I removed this option with the new v2 approach, but I am going to rollback this, while changing the way it behaved. This should also temorrary fix the issue about the sync

I also noticed that the time kept incrementing every 10 seconds in front matter and it would of course sync that change. The file just kept going around and around. I use Obsidian Sync and was testing with 3 computers in front of me.

This is a limitation currently of Obsidian api that should be fixed soon (according to this github issue) ! TLDR: this plugin relies on mtime (modification time) from filesystem & obsidian, however, the current api don't allow to update such time to the previous one while editing the frontmatter, and thus, causing a sync with a new ctime; causing the second computer to update the header again, etc.

Thanks for the response!

v2.1.0 added back the minMinutesBetweenSaves setting in a new way, this should address the sync issue.
Here is the TLDR of how it works :
graph TD
I{File changed} --> A
A{"Update header already present?"} -->|No| U{Update header with mtime}
A --> |Yes| C{Read update value. Is it enough passed ?}
C --> |Yes| U
C --> |No| E{Do nothing}
U

In the meanwhile we have the proper api from Obsidian to ensure mtime is not modified, you can increase the delay between updates ! :tada:

I can confirm that this has fixed the issues with syncing and merging issues that I was having.  Sync logs are now short as there is no repetitive activities.  Thank-you..

Thank you!

This should be way better with the newer version, https://github.com/beaussan/update-time-on-edit-obsidian/releases/tag/2.3.0 ! This plugin no longer edit the file's creation & modification time

Two things were done to improve this futher :

this plugin will now not longer edit file ctime & mtime, thanks to a new Obsidian api
using this experimental flag, you can now ensure the header is not updated, even if the time was, when the file was not modified (like with a thrid party sync) https://github.com/beaussan/update-time-on-edit-obsidian/releases/tag/2.4.0


[cardlink]: (react wrapper) disabled does not change heading/copy color

Description
When CardLink is disabled heading/copy color do not change

Component(s) impacted
cardLink
Browser
Chrome
Carbon for IBM.com version
1.31.0-rc.1
Severity
Severity 4 = The problem is not visible to or noticeable to an average user. Affects minor functionality, no workaround needed.
Application/website
carbon for ibm.com
Package
@carbon/ibmdotcom-web-components
CodeSandbox example
https://carbon-design-system.github.io/carbon-for-ibm-dotcom/next/web-components-react/?path=/story/components-card-link--default
Steps to reproduce the issue (if applicable)

go to this story: https://carbon-design-system.github.io/carbon-for-ibm-dotcom/next/web-components-react/?path=/story/components-card-link--default
disable card link and view heading/copy

Release date (if applicable)
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate issues

Also seeing in Firefox:

Seems to be fixed


Simple Ratelimit doesnt work with envoy frontproxy

Simple Ratelimit doesnt work with envoy frontproxy
envoy front proxy config:
`static_resources:
listeners:

address:
socket_address:
address: <IP_ADDRESS>
port_value: 80
filter_chains:

filters:

name: envoy.http_connection_manager
config:
codec_type: auto
use_remote_address: true
stat_prefix: ingress_http
access_log:
- name: envoy.file_access_log
config:
path: /dev/stdout
route_config:
name: local_route
virtual_hosts:
- name: backend
domains: ["*"]
routes:
- match:
prefix: "/"
route:
# Send request to an endpoint in the Google cluster
cluster: google
host_rewrite: www.google.com
rate_limits:
- actions:
- remote_address: {}
http_filters:

name: envoy.rate_limit
config:
domain: rate_per_ip
rate_limit_service:
grpc_service:
envoy_grpc:
cluster_name: rate_limit_cluster
timeout: 0.25s
name: envoy.router
typed_config: {}
clusters:

name: google
connect_timeout: 1s
type: logical_dns
dns_lookup_family: V4_ONLY
lb_policy: round_robin
load_assignment:
hosts: [{ socket_address: { address: google.com, port_value: 443 }}]
tls_context: { sni: www.google.com }

name: rate_limit_cluster
type: STATIC
connect_timeout: 1s
lb_policy: ROUND_ROBIN
http2_protocol_options: {}
hosts: [{ socket_address: { address: <IP_ADDRESS> , port_value: 8081 }}]
admin:
access_log_path: "/dev/null"
address:
socket_address:
address: <IP_ADDRESS>
port_value: 8001`

ratelimit config:
`domain: rate_per_ip
descriptors:

key: database
value: users
rate_limit:
unit: minute
requests_per_unit: 5
key: remote_address
rate_limit:
unit: minute
requests_per_unit: 5`

I can see both ratelimit and envoy working, and can access them, but envoy doesnt hit ratelimit service at all. Please help me.

hello

in the configuration above the rate_limits field is not indented correctly. The field should be inside of the route field.

Sir, you are such a savior that worked, been struggling with for a week now. very thankful for your help.


Avoid overriding the TE in Timer when selecting from AutoComplete

📒 Description
This PR will fix the issue, which describes in the following Problem section
Problem
As soon as we stop the TE (The TimeEntry could be new or existed in the list), the 📚 Library will notify bunch of kDisplayTimerState notifications (4-5 times) to TimerEditViewController (https://github.com/toggl/toggldesktop/blob/c0e7870da78abe19e33bfb4e5050a41ffec7d397/src/ui/osx/TogglDesktop/TimerEditViewController.m#L204).
If we quickly click by mouse (enter not work) the TimeEntry in autocomplete in Timer. One of above notifications will clear it out => Result in the disappearance of the TE.
Solution

In order to prevent it, we make the Description label to become first responder, so the if-statement https://github.com/toggl/toggldesktop/blob/c0e7870da78abe19e33bfb4e5050a41ffec7d397/src/ui/osx/TogglDesktop/TimerEditViewController.m#L210-L213
will be true and return immediately.
Secondly, we are able to edit on the selected TE as well (see the GIF)

Library issue
By throwing bunch of notifications after stopping the TE, we have to find alternative solution to make sure:

1 Notifications

🕶 Types of changes
Bug fix (non-breaking change which fixes an issue)
🤯 List of changes

[x] Make Description Editable and become first responder -> Prevent the clear
[x] Able to edit the selected TE

👫 Relationships
Close #3240
🔎 Review hints

Start any TE
Stop
Quickly type something in Timer -> select any TE by mouse
If the TE is still there and we're able to edit it => 💯

GIF

Library ticket: https://github.com/toggl/toggldesktop/issues/3242

The bug is fixed 👍 and I like the editable timer description. But with this, I, as a user, also kinda want more control over the entry. At least to be able to clear the project, like in the Windows app.

Up to you, guys, to decide if you want to add it.

I created a separate issue for the last comment #3250


Single API URL for a chain of libraries.

Feature request
Feature description
Single API URL for a chain of libraries.

The same feature is implemented in Google fonts as it is given below:
<link href="https://fonts.googleapis.com/css2?family=Recursive:wght@400;700&family=Roboto:wght@300;400&display=swap" rel="stylesheet">

This can be a similar format for cdnjs API.
<link href="https://api.cdnjs.com/libraries/css?load=twitter-bootstrap@4.5.0:bootstrap.min.css;photoswipe@4.1.3:photoswipe.min.css,default-skin/default-skin.min.css;" rel="preload" as="style" onload="this.onload=null;this.rel='stylesheet'">
<script src="https://api.cdnjs.com/libraries/js?load=jquery@3.5.1:jquery.min.js;photoswipe@4.1.3:photoswipe.min.js,photoswipe-ui-default.min.js;" defer type="text/javascript">
When this request is received by the API server, query is parsed to create a JSON object. Using this object, server will retrieve all the specific libraries requested and concatenate all of them in the same order as it is requested.
The only restriction is the CSS & JS can not be mixed up in same request as it can not be loaded by the same HTML tag. Also, Images cannot be requested in the same way. But map files can be requested in a similar way in future versions if really needed.

How the feature is useful
This could decrease the request count. When all libraries needed at loaded in a single request, there will not be any blocking requests and it increases the web performance of the website as part of optimization.

This should not be a feature of API server.


Fix: Responsive screen on mobile and tablet screen sizes

Before:

After:

Before:

After:

@mairin @jennydaman


Reporting of failures of nested "suites" drops class name

When a nested "suite" reports an error before any of its children are started, it is currently reported as a standalone test class in the default package instead of belonging to the test class it is declared in.
Expected Behavior
The failing "suite" method is reported as a child of its test class.
build/reports/tests/test/classes/org.example.LibraryTest.html

build/test-results/test/TEST-org.example.LibraryTest.xml
<?xml version="1.0" encoding="UTF-8"?>
<testsuite name="org.example.LibraryTest" tests="1" skipped="0" failures="1" errors="0" timestamp="2021-10-18T12:45:14" hostname="Marcs-MBP-2.fritz.box" time="0.41">
  <properties/>
  <testcase name="nested suite" classname="org.example.LibraryTest" time="0.41">
    <failure message="java.lang.RuntimeException: boom" type="java.lang.RuntimeException">java.lang.RuntimeException: boom
	at org.example.LibraryTest.params(LibraryTest.groovy:24)
	at org.example.LibraryTest.nested suite(LibraryTest.groovy:20)
</failure>
  </testcase>
  <system-out><![CDATA[]]></system-out>
  <system-err><![CDATA[]]></system-err>
</testsuite>

Current Behavior
The failing "suite" method is reported as a standalone test class in the default package.
build/reports/tests/test/classes/nested#20suite.html

build/test-results/test/TEST-nested#20suite.xml
<?xml version="1.0" encoding="UTF-8"?>
<testsuite name="nested suite" tests="1" skipped="0" failures="1" errors="0" timestamp="2021-10-18T12:45:14" hostname="Marcs-MBP-2.fritz.box" time="0.049">
  <properties/>
  <testcase name="failed to execute tests" classname="nested suite" time="0.049">
    <failure message="java.lang.RuntimeException: boom" type="java.lang.RuntimeException">java.lang.RuntimeException: boom
	at org.example.LibraryTest.params(LibraryTest.groovy:24)
	at org.example.LibraryTest.nested suite(LibraryTest.groovy:20)
</failure>
  </testcase>
  <system-out><![CDATA[]]></system-out>
  <system-err><![CDATA[]]></system-err>
</testsuite>

Context
That neither XML nor HTML report point to the test class is confusing when investigating why the test task failed.
Steps to Reproduce

clone https://github.com/marcphilipp/gradle-sandbox
cd nested-suite-reporting
./gradlew test

Your Environment
Build scan URL: https://scans.gradle.com/s/vv6gjegf2pc5g

Still relevant.


getAllDomainsForAccount() Hangs

I have a hedera_main resolver that successfully can get address from domain via resolver.resolveSLD. On the inverse of getAllDomainsForAccount(), it hangs forever.

Hi,
Thanks for reaching out, is possible to get the inputs you are attempting?
Thanks!

Hi @jwtong, we have patched the service to include some error handling within the method, which you can then handle within a try/catch.
Please find the update package here and a link to the updated docs here.
If you have any issues, please create them in the new repository. Thank you.


Tools/SandboxTest.ps1 fails when you don't already have dependencies downloaded

When running a Tools/SandboxTest.ps1 I get:
Test-FileChecksum: c:\winget-pkgs\tools\SandboxTest.ps1:444
Line |
 444 |      if (!(Test-FileChecksum -ExpectedChecksum $dependency.Checksum -P …
     |            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | The property 'Hash' cannot be found on this object. Verify that the property exists.

This was due to dependency files not being present locally on the drive, but due to this failure, they wouldn't download either.

Fixed in #196640


Add support for sharded PubSub

Refactor PubSub
fix #2354 - Add support for sharded PubSub
fix #1968 - PUBSUB SHARDCHANNELS command
fix #2345 ref #2321 - PING in PubSub mode
fix #2302 - error listener note in the main README
Add minimizeConnections configuration for clusters
renamed branch from #2344

Codecov Report
Base: 95.85% // Head: 94.92% // Decreases project coverage by -0.93% :warning:

Coverage data is based on head (fa52217) compared to base (c5b6f77).
Patch coverage: 72.85% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2373      +/-   ##
==========================================
- Coverage   95.85%   94.92%   -0.94%
==========================================
  Files         449      452       +3
  Lines        4224     4431     +207
  Branches      471      497      +26
==========================================
+ Hits         4049     4206     +157
- Misses        107      148      +41
- Partials       68       77       +9

Impacted Files
Coverage Δ

packages/client/lib/commands/CLUSTER_SLOTS.ts
100.00% <ø> (ø)

packages/client/lib/client/socket.ts
72.80% <30.76%> (-4.98%)
:arrow_down:

packages/client/lib/cluster/index.ts
70.58% <56.25%> (-7.03%)
:arrow_down:

packages/client/lib/cluster/cluster-slots.ts
64.89% <62.94%> (+5.43%)
:arrow_up:

packages/client/lib/client/pub-sub.ts
82.73% <82.73%> (ø)

packages/client/lib/client/commands-queue.ts
87.36% <87.80%> (+4.47%)
:arrow_up:

packages/client/lib/client/index.ts
91.37% <89.47%> (+0.24%)
:arrow_up:

packages/client/lib/client/commands.ts
100.00% <100.00%> (ø)

packages/client/lib/cluster/commands.ts
100.00% <100.00%> (ø)

packages/client/lib/commands/PING.ts
100.00% <100.00%> (ø)

... and 6 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Code refactoring for 2023 summer classes.

Describe problem
All code modifications related to classes conducted during the summer semester of 2023.

#327 fixes it.

Fixed. Created Release https://github.com/mpostol/TP/releases/tag/4.3


Multiple DbContexts not synchronizing data

Entity Framework: 6.1.3
Database-first approach
Repro:
EntityConnection connection = Effort.EntityConnectionFactory.CreateTransient(connectionString);
var dbContext = new MyDbContext(connection);
dbContext.Users.Add(new User() { UserName= "John" });
dbContext.SaveChanges();

var dbContext2 = new MyDbContext(connection);
var user = dbContext2.Users.FirstOrDefault();

user.UserName= "Mike";
dbContext2.SaveChanges();

// This fails
Assert.AreEqual(dbContext.Users.FirstOrDefault().UserName,
                                       dbContext2.Users.FirstOrDefault().UserName);

The workaround is that I have to re-initialize "dbContext" before the Assert statement:
dbContext = new MyDbContext(connection);
Is this by design? I've even tried the persistent connection but it's the same behavior.

You are using the CreateTransient(connectionString) method to create a connection. If you use the CreatePersistent(connectionString) method, the same internal database should be used.

I tried that, but I still have the same issue:
EntityConnection connection = Effort.EntityConnectionFactory.CreatePersistent("test", connectionString);
var dbContext = new MyDbContext(connection);
dbContext.Users.Add(new User() { UserName = "John" });
dbContext.SaveChanges();

var dbContext2 = new MyDbContext(connection);
var user = dbContext2.Users.FirstOrDefault();

user.UserName = "Mike";
dbContext2.SaveChanges();

// This fails
Assert.AreEqual(dbContext.Users.FirstOrDefault().FullName,
                         dbContext2.Users.FirstOrDefault().FullName);

The same workaround still solves the issue but it is annoying to have to re-initialize the DB context every time I do an update in my code so that it reflects in the unit tests.

Hi
Yes, the underlying database instance should be shared across the data contexts.
I believe what you see is a by-design EF behaviour.
The problem is that the result of the dbContext query is an entity that is already attached to the context: the one that you just created. In case you detach that entity before, the query is going to give you the expected result.
EntityConnection connection = Effort.EntityConnectionFactory.CreateTransient(connectionString);
var dbContext = new MyDbContext(connection);
var user1 = new User() { UserName= "John" };
dbContext.Users.Add(user1);
dbContext.SaveChanges();

var dbContext2 = new MyDbContext(connection);
var user2 = dbContext2.Users.FirstOrDefault();

user2.UserName= "Mike";
dbContext2.SaveChanges();

// !!!!
dbContext.Entry(user1).State = EntityState.Detached;

// This fails
Assert.AreEqual(dbContext.Users.FirstOrDefault().UserName,
                                       dbContext2.Users.FirstOrDefault().UserName);


setting::phase(phase).right()

this way it works with things like sine_hz
is this correct?

Yes, good catch!


fix: /projects seems to have incorrect watchers total

looks like badgen.net has the same values for watchers vs stars


does it can used in llama2.c?

the llama2.c can be trained in different size ,not usually 7B 13B 65B .
does it can be used in 15M mini llama2 models ?

if it can be used in , how can i do it?


Makes the doc clear

Have added some more info so that user can install it easily.

Thank you for the PR, could you please add the suggestions to FAQs.. Let's keep README as minimal and informative as possible.


A bug my transactions are going to other people's account

Expected Behavior

Current Behavior

Possible Solution

Steps to Reproduce (for bugs)

Context

Your Environment

Version used:
Browser Name and version:
Operating System and version (desktop or mobile):
Link to your project:

Hi @rsam036 could you please provide steps to replicate? Additionally, please reach out to<EMAIL_ADDRESS>for further assistance

Account appears to be spamming multiple repos with empty issues and PRs. Closing issue


[Masonry] 'MuiMasonry' does not exist in type 'Components'

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
I get a compilation error
Type '{ MuiMasonry: { styleOverrides: { root: { margin: number; }; }; }; }' is not assignable to type 'Components'.
  Object literal may only specify known properties, and 'MuiMasonry' does not exist in type 'Components'.

Expected behavior 🤔
It should compile and override Masonry root margin in app.
Steps to reproduce 🕹
Create a react app in TypeScript, write in App.tsx
const theme = createTheme({
  components: {
    MuiMasonry: {
      styleOverrides: {
        root: {
          margin: 0,
        },
      },
    },
  },
});

Context 🔦
I am trying to use theming API to override Masonry component style
Your environment 🌎

`npx @mui/envinfo`
  System:
    OS: Linux 5.10 NixOS 21.11 (Porcupine) 21.11 (Porcupine)
  Binaries:
    Node: 14.17.6 - ~/.nix-profile/bin/node
    Yarn: 1.22.11 - ~/.nix-profile/bin/yarn
    npm: 6.14.15 - ~/.nix-profile/bin/npm
  Browsers:
    Chrome: Not Found
    Firefox: Not Found
  npmPackages:
    @emotion/react: 11.4.1 => 11.4.1
    @emotion/styled: 11.3.0 => 11.3.0
    @mui/base:  5.0.0-alpha.55
    @mui/icons-material: 5.1.1 => 5.1.1
    @mui/lab: 5.0.0-alpha.55 => 5.0.0-alpha.55
    @mui/material: 5.1.1 => 5.1.1
    @mui/private-theming:  5.1.1
    @mui/styled-engine:  5.1.1
    @mui/system:  5.1.1
    @mui/types:  7.1.0
    @mui/utils:  5.1.1
    @types/react: 17.0.0 => 17.0.0
    react: 17.0.2 => 17.0.2
    react-dom: 17.0.2 => 17.0.2
    typescript: 4.1.2 => 4.1.2

Hi. Thanks for the report. Your createTheme is imported from @mui/material while Masonry lives in @mui/lab package. In fact, if you inspect the component, you can see that your theme style is actually taking effect. Check out https://codesandbox.io/s/thirsty-christian-o4y4b?file=/src/Demo.tsx. However, the type error still exists. I'd use // @ts-ignore until Masonry gets merged into @mui/material package. I hope that helps.


Input of some characters result in HTML code instead

Expected behavior
For example, entering <  > [  ] should print those characters
Actual behavior
It is printing: "<=]["
(it doesn't happen if its the very first character)
Steps to reproduce
Type those characters
Browser and OS
Ubuntu 16.04 64bits
Chrome 61.0.3163.102
Firefox 56.0

The reason seems to be in line 1689:
function draw_cursor_line(string, position) {
                var len = $.terminal.length(string);
                if (position === len) {
                    before.html(string);   //<-------- replaced: format(string) for string
                    cursor.html(' ');
                    after.html('');
                } else if (position === 0) {

Now seems to be working fine. I'm not sure it it could have side effects.

This is already fixed in devel branch, can you test https://codepen.io/jcubic/pen/LxgJQG


<EMAIL_ADDRESS> Path error, unable to install

Closes #12262

[x ] I have read the Contributing Guide.

/verify

/verify

/verify

Cherry-picking the one and only commit of this PR worked for me:

Thanks for the fix :)
>git cherry-pick aee9f005bc20c960982842cae16cdfdbb0c498e8
[master 3fec31f01] fix: Calibre normal path error
 Author: mo_n<EMAIL_ADDRESS> Date: Sat Nov 18 11:25:11 2023 +0800
 1 file changed, 1 insertion(+), 1 deletion(-)

> git log --oneline -3
3fec31f01 (HEAD -> master) fix: Calibre normal path error
7f83941c4 (origin/master, origin/HEAD) notesnook: Update to version 2.6.9
d7d91e4a9 elisa: Update to version 23.08.3-1687

@mo-n thank you for your contribution!


Inference results on my own dataset

I have trained a new model on my own dataset with normlization applied.
Then I checked the inference result, but found bed results.
the input partial point cloud is as followed:

the complete point cloud is as followed:

the predicted result (with out merging with input partial):

the merged result:

I wonder why and looking forward to your suggestion!

Our method is designed for object. Adopting to scene level completion does lead to a lot of problems. This is aligned with my expectation. The way we create patch needs knn and it is likely the reason. See a recent paper [1] from ECCV 2024 shows similar problem of our method in Fig 3. Our method is not good at producing large planes. It is prone to create spherical clusters.
let me know if you have further concerns
[1] https://arxiv.org/pdf/2409.15939


Process Extended Segment Address records

They are currently passing through, without taking in consideration the memory address change as we do with the Extended Linear Address records.
So, until this is implemented, hex files with Extended Segment Address records will not work.

This has been implemented and they are converted into Extended Linear Address records.


Runtime error when passing record with missing default fields to encoder

I can write a function that takes a record with some default fields and directly encodes it, without providing default values. This compiles, but then results in a runtime failure.
This is a minimal example using roc-json, but I think the error is the same regardless of what encoder you use. Run the file with roc test to get the error.
app [main] {
    cli: platform "https://github.com/roc-lang/basic-cli/releases/download/0.15.0/SlwdbJ-3GR7uBWQo6zlmYWNYOxnvo8r6YABXD-45UOw.tar.br",
    json: "https://github.com/lukewilliamboswell/roc-json/releases/download/0.10.2/FH4N0Sw-JSFXJfG3j54VEDPtXOoN-6I9v_IA8S18IGk.tar.br",
}

import json.Json

main = Task.ok {}

expect
    encode : { hi ? Str } -> List U8
    encode = \arg -> Encode.toBytes arg Json.utf8
    encode {} == ['{', '}']

Here's the error:
thread 'main' panicked at crates/compiler/mono/src/borrow.rs:396:34:
internal error: entered unreachable code:
	No borrow signature for LambdaName { name: `12.IdentId(26)`, niche: Niche(Captures([])) } layout.

	Tip 1: This can happen when you call a function with less arguments than it expects.
	Like `Arg.list!` instead of `Arg.list! {}`.
	Tip 2: `roc check yourfile.roc` can sometimes give you a helpful error.

note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

I don't know what the fix should be. Probably a compile time error rather than allowing the behavior at runtime? The reason I was trying this was to create a cute DSL for writing Html:
Html.ul { class: "list-of-things } [
    Html.li {} [ Html.text "an item!" ],
]

# --- Html.roc
ul : { class ? Str, id ? Str, ............ }, List Html -> Html
li : { class ? Str, id ? Str, ............ }, List Html -> Html

I realize that's not what these default field types are made for though 😅

Sorry, I meant to create this on the roc compiler repo. Closing this here!


Fix flaky cartservice build

testing to see if updating builder fixes cartservice build failures on GHA

This is ready for review - couple of notes on the process:
My goal with this PR is to resolve the following build error that cropped up after the updates to .net7:
 > [linux/arm64 builder 5/5] RUN   RUNTIME_IDENTIIFER=linux-musl-x64;   if [ "$(uname -m)" = "aarch64" ]; then RUNTIME_IDENTIIFER=linux-musl-arm64; fi;   dotnet restore ./src/cartservice.csproj -r $RUNTIME_IDENTIIFER &&   dotnet publish ./src/cartservice.csproj -p:PublishSingleFile=true -r $RUNTIME_IDENTIIFER --self-contained true -p:PublishTrimmed=False -p:TrimMode=Link -c Release -o /cartservice --no-restore:
21.25   Determining projects to restore...
56.39   Restored /usr/src/app/src/cartservice.csproj (in 30.06 sec).
63.50 MSBuild version 17.6.8+c70978d4d for .NET
157.2   cartservice -> /usr/src/app/src/bin/Release/net7.0/linux-musl-arm64/cartservice.dll
160.5 /usr/share/dotnet/sdk/7.0.306/Sdks/Microsoft.NET.Sdk.Razor/targets/Microsoft.NET.Sdk.Razor.StaticWebAssets.targets(658,7): error MSB4018: The "ReadStaticWebAssetsManifestFile" task failed unexpectedly. [/usr/src/app/src/cartservice.csproj]
160.5 /usr/share/dotnet/sdk/7.0.306/Sdks/Microsoft.NET.Sdk.Razor/targets/Microsoft.NET.Sdk.Razor.StaticWebAssets.targets(658,7): error MSB4018: System.NullReferenceException: Object reference not set to an instance of an object. [/usr/src/app/src/cartservice.csproj]
160.5 /usr/share/dotnet/sdk/7.0.306/Sdks/Microsoft.NET.Sdk.Razor/targets/Microsoft.NET.Sdk.Razor.StaticWebAssets.targets(658,7): error MSB4018:    at InvokeStub_ReadStaticWebAssetsManifestFile.set_ManifestPath(Object, Object, IntPtr*) [/usr/src/app/src/cartservice.csproj]
160.5 /usr/share/dotnet/sdk/7.0.306/Sdks/Microsoft.NET.Sdk.Razor/targets/Microsoft.NET.Sdk.Razor.StaticWebAssets.targets(658,7): error MSB4018:    at System.Reflection.MethodInvoker.Invoke(Object obj, IntPtr* args, BindingFlags invokeAttr) [/usr/src/app/src/cartservice.csproj]
160.5 /usr/share/dotnet/sdk/7.0.306/Sdks/Microsoft.NET.Sdk.Razor/targets/Microsoft.NET.Sdk.Razor.StaticWebAssets.targets(658,40): error MSB4026: The "ManifestPath=$(StaticWebAssetBuildManifestPath)" parameter for the "ReadStaticWebAssetsManifestFile" task is invalid. [/usr/src/app/src/cartservice.csproj]
160.5 /usr/share/dotnet/sdk/7.0.306/Sdks/Microsoft.NET.Sdk.Razor/targets/Microsoft.NET.Sdk.Razor.StaticWebAssets.targets(658,7): error MSB4063: The "ReadStaticWebAssetsManifestFile" task could not be initialized with its input parameters.  [/usr/src/app/src/cartservice.csproj]```

This error appeared only during `amd64` builds, and it wasn't easily reproducible - nor is it an error I can find any evidence of anyone else running into. I surmise that there is a particular bug in the .NET SDK in this particular build environment under QEMU (this would certainly not be the first time we've seen strange issues with QEMU).

This PR makes the following changes to alleviate the issue:
1. Update QEMU by pulling main from the QEMU action
2. Take the most recent .NET security updates
3. Disable Static Web Assets and Reference Assemblies

Static web assets aren't necessary for cartservice (because there aren't any, it's an API-only project) and reference assemblies also aren't necessary (because they're included in the runner image).

lmao welp

I think I might have found something useful though - https://github.com/dotnet/sdk/issues/27190

I'm looking through this issue (https://github.com/dotnet/dotnet-docker/issues/4388) which seems to indicate that this should be working on .NET 7.0.3xx (which we're on), but I'm not entirely sure if there's something we need to do in the Dockerfile to get it to reliably build.

Ok, this fix works more than it fails, so I'm gonna go ahead and merge this with the increased debug info left on so we can try to diagnose it better when it happens again.


BLD: Fix Bento build of source distribution

Several files required to build Scipy using Bento are not included
in source distributions created using python setup.py sdist.

Yep. Probably also needs to go in 0.10.1, as the bento stuff doesn't seem to be included in the tarballs.

Just a heads up that I get some test failures related to undefined Fortran symbols after building and installing with Bento.
As it stands this change only lets the build proceed without errors..

Can you open a ticket for the test failures, or report them on the list?

If one of you has any time today to try my suggested change above, that would be great. I would like to get 0.10.1rc2 out the door tonight.

I'll report the build failures on the list and have a go at improving BENTO_BUILD.txt, but don't have time to do that today.

Thanks Scott. I've merged this in 759eda9 and removed the .dat files in 785a822. I'll backport the first commit to 0.10.x.

I think the more detailed instructions and better testing with Bento will have to wait till 0.11.

The issue seems to be that _cephes.so should be linked with the Fortran compiler, even though it has no Fortran source files. Unfortunately, I have no idea how to tell the waf/bento combination to do that...

No idea either. I don't see anything to that effect in setup.py or SConscript though. What am I missing?

Ok, the problem is a link flag order issue; '-lgfortran' should come after '-lsc_*', not before.


[Help needed] energyplus: init at 9.0.1

Motivation for this change
Add EnergyPlus https://energyplus.net/
This is work in progress, I opened this pull request already now so that I could perhaps get some help.
I can build the package like this:
nix-build '<nixpkgs>' -A energyplus

But when I try to run the compiled binary, I get:
$ result/energyplus
result/energyplus: symbol lookup error: /nix/store/vdj3wf599ry7ls6ghlh1mfss56vv31pm-energyplus/libenergyplusapi.so.9.0.1: undefined symbol: XXX

I have no idea what is this symbol XXX, why it's missing and how to make it available.. Any ideas how to fix this?
Here are the official build instructions: https://github.com/NREL/EnergyPlus/wiki/BuildingEnergyPlus
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

ping (triage)

I'd say it would be appropriate to close this PR. @jluttine If you're still interested in having this packaged
I'd recommend opening an issue as a request with the expression you have.


kmir parse failsure in the globally installed kmir

The command works under DIR mir-semantics/kmir but fails outside this DIR.
For example, it failes when tests under mir-semantics.
On demo branch, command to reproduce:
kmir parse --definition-dir $(kbuild which llvm) demo/assert-true.mir

Error report:
Traceback (most recent call last):
  File "~/.local/bin/kbuild", line 8, in <module>
    sys.exit(main())
  File "~/.local/lib/python3.10/site-packages/pyk/kbuild/__main__.py", line 21, in main
    return do_which(**args)
  File "~/.local/lib/python3.10/site-packages/pyk/kbuild/__main__.py", line 74, in do_which
    project_file = _project_file(start_dir)
  File "~/.local/lib/python3.10/site-packages/pyk/kbuild/__main__.py", line 60, in _project_file
    return find_file_upwards(PROJECT_FILE_NAME, start_dir)
  File "~/.local/lib/python3.10/site-packages/pyk/kbuild/utils.py", line 107, in find_file_upwards
    raise FileNotFoundError(f'{file_name} not found')
FileNotFoundError: kbuild.toml not found
usage: kmir parse [-h] [--verbose] [--debug] [--definition-dir DEFINITION_DIR] [--input {program,binary,json,kast,kore}]
                  [--output {pretty,program,json,kore,kast,none}]
                  input_file
kmir parse: error: argument --definition-dir: invalid dir_path value: 'demo/assert-true.mir'

An update: the reason to the problem is the given --definition-dir is not valid. A working solution is proposed by @dkcumming.
However, I am still wondering if there be something implemented at the installation stage. @tothtamas28

How is kmir installed? With nix or pip?

How is kmir installed? With nix or pip?
pip

@tothtamas28 would give some clue in this error? Where can I locate the problem? Thank you.

It doesn't solve the error if you do which to use kbuild explicitly, but it should give you the same functionality through a different avenue. You can use the environment variable script set_env.sh and omit --definition-dir entirely:
daniel@daniel-MS-7E06 kmir$ make build
//-snip-
/home/daniel/Applications/mir-semantics/kmir/kdist/v6.1.24/llvm
poetry run kbuild kompile haskell
/home/daniel/Applications/mir-semantics/kmir/kdist/v6.1.24/haskell
poetry build
Building kmir (0.1.0)
  - Building sdist
  - Built kmir-0.1.0.tar.gz
  - Building wheel
  - Built kmir-0.1.0-py3-none-any.whl
daniel@daniel-MS-7E06 kmir$ source set_env.sh
daniel@daniel-MS-7E06 kmir$ pip install
//-snip-
Successfully installed kmir-0.1.0 pyk-0.1.509
daniel@daniel-MS-7E06 kmir$ which kmir
/home/daniel/.local/bin/kmir
daniel@daniel-MS-7E06 kmir$ cd src/tests/integration/test-data/handwritten-rust/integers/
daniel@daniel-MS-7E06 integers$ kmir parse --output pretty const-arithm-simple.mir
fn test :: .FunctionPath ( _1 : usize  , _2 : usize  , .ParameterList ) -> bool  { debug x => _1 ;  debug y => _2 ;  .DebugList let mut _0 : bool  ;  .BindingList .ScopeList bb0  : { _0 = Gt ( _1 , _2 ) ;  .Statements return ; }  .BasicBlockList }  fn main :: .FunctionPath ( .ParameterList ) -> ( ) { .DebugList let mut _0 : ( ) ;  let  _1 : usize  ;  let mut _4 : bool  ;  let mut _5 : bool  ;  let mut _6 : ! ;  .BindingList scope 1 { debug x => _1 ;  .DebugList let  _2 : usize  ;  .BindingList scope 2 { debug y => _2 ;  .DebugList let  _3 : bool  ;  .BindingList scope 3 { debug z => _3 ;  .DebugList .BindingList .ScopeList }  .ScopeList }  .ScopeList }  .ScopeList bb0  : { _1 = const 42_usize ;  _2 = const 0_usize ;  .Statements _3 = test :: .ExpressionPathList ( _1 , _2 , .OperandList ) -> bb1  ; }  bb1  : { _5 = _3 ;  _4 = Not ( move _5 ) ;  .Statements switchInt ( move _4 ) -> [ 0 : bb3  , .SwitchTargets , otherwise : bb2  ] ; }  bb2  : { .Statements _6 = core :: panicking :: panic :: .ExpressionPathList ( const "assertion failed: z" , .OperandList ) ; }  bb3  : { .Statements return ; }  .BasicBlockList }  .Mir

Using this method, if you update the version of k and build kmir again you will need to set the environment variables the same way as last time (source set_env.sh). You can check where the environment variables are set to with:
daniel@daniel-MS-7E06 ~$ env | grep LLVM
KMIR_LLVM_DIR=/home/daniel/Applications/mir-semantics/kmir/kdist/v6.1.24/llvm
daniel@daniel-MS-7E06 ~$ env | grep HASKELL
KMIR_HASKELL_DIR=/home/daniel/Applications/mir-semantics/kmir/kdist/v6.1.24/haskell

Looks like this line is not true

Yes, that's the existing manual, I am updating it with a set environment step.
Though, I hope pip install could solve it (set the env after every installation).

kmir parse: error: argument --definition-dir: invalid dir_path value: 'demo/assert-true.mir'

This indicates that demo/assert-true.mir either does not exist, or is not a directory.


BUG: dd.concat cant handle Series

This makes dd.concat to work with Series.
import dask.dataframe as dd
s1 = pd.Series([1, 2, 3, 4, 6, 7], index=[1, 2, 3, 4, 6, 7])
ds1 = dd.from_pandas(s1, 2)
s2 = pd.Series([1, 2, 3, 4, 6, 7], index=[8, 9, 10, 11, 12, 13])
ds2 = dd.from_pandas(s2, 2)

# pandas, OK
pd.concat([s1, s2])
# outputs concatenated Series

# dask, NG
dd.concat([ds1, ds2])
# AttributeError: 'Series' object has no attribute 'columns'

+1


Migrate away from LangChain

LangChain has several security vulnerabilities. We should be referring to a different solution: perhaps manually calling the GPT api in a wrapping class.

https://github.com/advisories/GHSA-7gfq-f96f-g85j


Issue with dead sd card

My pt202 escam that was already hacked for 1 year ended up losing the card. I'm trying to hack again with two new sd card's and nothing. It just doesn't work anymore, I tried to reset the camera several times with the hack and nothing. When I remove the sd card from the camera, the Factory.Done folder appears and the hack doesn't work. I already tested removing the hack with a file in this part (https://github.com/roleoroleo/yi-hack-Allwinner-v2/wiki/Remove-the-hack-(deprecated)) and reinstalling the hack and it didn't work . The last process I need to do is "unbrick the cam" but I think the process is very complicated, I just don't know how to work with linux and I don't have any ttl-rs232/usb adapter. So both processes are out of the question. What more options do I have? Is there anything I can put on an sd card and unlock it, just like it is for installing the hack? I don't know what else to do, I've tried all the options at my disposal... My old sd card with the hack died...

New unbrick procedure doesn't require serial connection.
Try it.

It's weird my sd card became corrupted after I deployed to my outdoor cam, it has 1GB from 8GB, and tried a lot of ways including different OS also no luck getting the 6.5GB back. Hope it doesn't happen to my other SD card again and this is only once...

Even the new process to unbrick for me is quite complicated. I've tried but I just don't understand the instructions, it says putting it on a linux machine... But I don't understand how, if I had an image of how to do it I think it would be easier. The instructions are very superficial and for those who don't understand anything it's confusing.

The old method of putting a file on the sd card and waiting for the camera to recognize it is much easier. How was the process remove the hack which is now deprecated. It would be much easier for me.

@carlosvelosa
I can't simplify this part very much because I can't prepare a file for each cam. The procedure must use your backup file.
Send me your backup and I will create the file for you.

https://www14.zippyshare.com/v/junY1xiS/file.html
@roleoroleo Thank you, sorry for my lack of knowledge in linux. And thank you very much.

https://www.sendspace.com/file/95cmnn

Here it is:
b091qp_unbrick.tgz
I think that backup partition should be enough to restore your cam.
But I created home and rootfs also, if you need them.
There are 2 different backup partitions:

"factory" is the original partition
"hacked" is like the original but it's ready for the hack (some files changed)

@roleoroleo How do I run this file? Just put it on the sd card and that's it? Or do I have to change some name?

Just copy the file backup_091qp into your sd card and switch on the cam.

Just copy the file backup_091qp into your sd card and switch on the cam.

I forgot a "b": backup_b091qp
If the cam renames Factory to Factory.done when you try to apply the hack, it means that the cam starts.
So the restore procecure described above with the file backup_b091qp must work.

@roleoroleo Continues not working for me. I try with different sd card's and nothing. The camera simple don't work... Probably a internal problem or something. The cam renames Factory to Factory.done but the hack simple not work...

You need a better and a faster way to unhack the camera like the same way you hack. If the camera don't works more, i will buy a new camera and put this in garbage.

Yi doesn't publish the firmwares so it's not possible to create a standard restore file.
Each user must create their own restore file.
I can assure you that your cam is not broken (or at least not completely).
If the folder is renamed, the cam boots and the main script (init.sh) is executed.
And the sd card is properly read and mounted.
We should restart from scratch and check every step.


Pull request. Havent started main work yet

I can' merge this.
Too many conflicting files.
PLease pull from upstream main and make push to update this PR.

Also you said you were done with this nndpoint as at 2pm this afternoon. Now you said you;re just starting work. I don't even know what to say.


Update .smalltalk.ston

Pull Request Test Coverage Report for Build 130

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 32.958%

Totals

Change from base Build 124:
0.0%

Covered Lines:
439

Relevant Lines:
1332

💛  - Coveralls


Automate - Fixed differences between tenant, group and tag values.

Fixed validate quota issue where request was not denied properly
when max and warn exceeded.
https://bugzilla.redhat.com/show_bug.cgi?id=1319292

@tinaafitz
Please Review

@billfitzgerald0120
Looks good.
@mkanoor
Please review.

@mkanoor
I changed the 2 if's into elsif ...

@tinaafitz @mkanoor
Please Review

@tinaafitz @mkanoor
Please Review

@tinaafitz @mkanoor
Please Review

@billfitzgerald0120
Looks good.

@mkanoor
I applied all of your changes.
Please Review

👍

@gmccullo
Please Review

@gmccullo
Please Review


Add ability to use newline separators in response files and update WinMDExp task to use it

Update WinMDExp task to use a response file with new line separators
Add new property to CommandLineBuilder and CommandLineBuilderExtension class to use newlines instead of spaces
Add new property to ToolTaskExtension to use newlines in response files instead of spaces.
Update WinMDExp task to use a response file with newline separators and unquoted arguments
WinMDExp fails if there are quotes anywhere in the response file passed
to it, but correctly understands spaces in paths passed to /reference:
when the arguments are newline separated. Avoid ToolTask's default
quoting rules when populating the response file.
Reverts 0174f995. Fixes #2140.

@dotnet-bot test OSX Build for CoreCLR please

@yizhang82 signed off via email. This should be good to go.


react tags v5.x.x

Release 5.x.x
Features/Enhancements

[ ] key code is deprecated (#196)
[ ] custom tag component (#48, #211)
[ ] tags inside 'input field' or default (#211, #132)
[ ] choose to sort tags (#21)
[ ] inline styling (#45)
[ ] ability to show suggestions drop up (#208)
[ ] should work with redux-form (#51)
[ ] two react-tag-input in one place (#57, #173)
[ ] suggestions can be array of objects (#74, #131, #185)
[ ] ability to render input above tags (#233)
[ ] auto-add (#77)
[ ] different options for input placement (#130)
[ ] get rid of dnd or conditionally use dnd when needed (#141, #198)
[ ] hide used suggestions (#171)
[ ] optional submit button (#186)
[ ] read-only tags (#194)
[ ] 'handlePaste' option (#104)
[ ] improve tests coverage
[ ] setup eslint
[ ] precommit hook
[ ] add storybook with examples
[ ] setup semantic-release
[ ] update docs
[ ] release react-tags 5.x.x

Bugs

[ ] react-tags breaks other dnd's (#98)

Non-related to library

[ ] add stale bot

If you have any other requests feel free to write them down below. :beers:

@mrdannael added pre commit hook in the list


Uniquely identify existing PR comment

Summary
The PR makes the action include fields testScript, coverageFile, and baseCoverageFileto the report tag along with theworkingDirectory` field. The action will identify an existing PR comment more precisely using these fields.
I didn't include all the options into the report tag because this might have caused the action not to identify existing comments because some of the "cosmetic" settings were changed. For example, the custom-title option may change during a sequence of action invocations. I can imagine a use-case when a user adds the invocation counter or commit ID to the custom-title. The same considerations apply to other options that were not included in the report tag.
Note: this PR also fixes a small bug with the customTitle it basically was not propagated to the createReport function in run.ts. Now it is passed to the function along with other options.
This PR fixes #146
cc @ArtiomTr

@all-contributors please add @BohdanPetryshyn for code

Hey @ArtiomTr! Thank you for reviewing my changes! When can I expect the changes to be released?

@BohdanPetryshyn Within a few days I will release version 2.0-rc.5. It will contain all the latest changes.

@BohdanPetryshyn the new version 2.0-rc.5 is now available!


Fix generated type definitions

Fixed the wrong paths in the generated type definitions, this should fix errors when using the client with TypeScript.
Also removed console.log from the vue sample since the production build doesn't allow it

:shipit:


TextStyleTween has unexpected lerping behavior for underlines

When TextStyleTween is lerping between two TextStyles, one with and the other one without an underline, I expect it to provide fading transition on the underline. Currently, the underline suddenly appears/disappears in the middle of transition.
Steps to Reproduce

Define two text styles one with an underline and another one without an underline. You can use these samples that I'm actually using in my app:

static const sectionCurrent = TextStyle(
    color: ConstantColors.lightIndigo,
    fontFamily: ConstantFontFamilies.brandonTextBlack,
    letterSpacing: 0.5,
    fontSize: 13.0,
    decoration: TextDecoration.underline,
  );
  static const sectionFuture = TextStyle(
    color: ConstantColors.dusk50,
    fontFamily: ConstantFontFamilies.brandonTextBlack,
    letterSpacing: 0.5,
    fontSize: 13.0,
  );

Define a TextStyleTween for lerping between these two text styles.
Try lerping with different values and see that the underline appears/disappears suddenly in transitions instead of fading in/out.

Flutter Doctor
[✓] Flutter (Channel stable, v1.9.1+hotfix.2, on Mac OS X 10.14.6 18G95, locale en-US)
    • Flutter version 1.9.1+hotfix.2 at /Users/behnam/dev/tools/flutter
    • Framework revision 2d2a1ffec9 (4 weeks ago), 2019-09-06 18:39:49 -0700
    • Engine revision b863200c37
    • Dart version 2.5.0

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.1)
    • Android SDK at /Users/behnam/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.1
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.0, Build version 11A420a
    • CocoaPods version 1.7.4

[✓] Android Studio (version 3.4)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 37.0.1
    • Dart plugin version 183.6270
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)

[✓] VS Code (version 1.38.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.4.1

Example Output
Here is an actual recording showing the issue:

After looking into the implementation I was able to solve this in my specific case with a workaround that works due to the fact that I have a solid color background under the text.
What I did was adding the underline to both TextStyles and hide it with setting the decorationColor to the background color.
Notice the first I tried having the underline with a transparent color which didn't work! It actually behaved in a weird way which could be a separate bug. So if you are looking into this issue it's a good idea to look into that too.

The problem with fading the transition is that it can be ambiguous what a "fade" entails. For example, what is the proper way to fade between a wavy underline and a double underline?
This is one of those instances where there is no good solution for all cases, and a boolean-like switch is the best we can do, which should still be relatively easy to hack around for custom solutions. For example, we do lerp colors. This means you can define an intermediate TextStyle with the same underline but transparent color, and lerp to that one before lerping to your final TextStyle. This would achieve your fade without too much hacking about.
I don't think there is any work to do here, so I will close this issue for now, please let me know if you have any additional questions/issues/concerns!

@GaryQian
I meant the case in which there is underline in one TextStyle while the other one doesn't have an underline or any other decoration. There is no ambiguity in that case!
I know that there is no solution for all cases but there actually a solution for this specific case. Knowing the fact that we can't handle all cases should not impact the case that we actually know how to handle. So I believe not doing it altogether is not the solution.
And please refer to my comment where I mentioned using an underline with transparent color didn't work! I'll open another issue for that then if we're not looking into this here.

I'll take a look at the transparent behavior. It looks like there should already be handling for fading the alpha in the Color.lerp implementation, but the framework's TextSpan.lerp may be masking it.
Regarding treating special cases, we should be wary of adding very specific edgecase code as that can lead to unpredictable behavior in the future/many layers up the stack. We can consider this case, but it has to be a particularly safe assumption.

@GaryQian Thanks!
I understand the special case treatment. It might come down to personal preference here. I think however due to having a small number of cases (none, underline, overline, and lineThrough) it seems fine to have special treatment.
And by the way, now that I'm looking at it, seems like the ambiguity is present for TextDecorationStyles (solid, double, dotted, dashed, wavy) and not for TextDecoration  (none, underline, overline, and lineThrough).
For TextDecoration I can easily see how the transition would occur.


fix: add missing strict mode directives

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
176

Relevant Lines:
176

💛  - Coveralls


Add words to ignore list

I suggest we add the words 'obviously', 'simple' and 'easy' to ignore list

Ah, the word 'easy' seems to be brought up in #327

These words we should try to avoid using, but might be overkill for the bot to respond to these every time 👍

My thoughts exactly! Also, silly question but... where's the ignore list?

Its currently in a PR, so you will need to branch from issue-303 and make your PR to that branch too https://github.com/EddieJaoudeCommunity/EddieBot/pull/307/files#diff-c3095d5010e65c52737a98a5d618ea24049ebe90c8470752426081d70ed6e012R90


Fix for Instance Pool API and Enum JSON Handling

This is a fix for #103

@microsoft-github-policy-service agree


SideNav menu was moving on X-axis

when menu contents overrides the height

Fixes https://github.com/Dogfalo/materialize/issues/4814
PS: overflowY without quotes should also work.

This is the wrong file, please change the right lines in the sidenav.js file.

I agree with @DanielRuf. I believe there is something wrong with the css that is causing this error and changing the overflow is not the way to go fixing this.

https://github.com/Dogfalo/materialize/commit/a8d472fca1abf72afa638aa5e7e9bf1882d9cd15


Please provide a definition for 'scopes' that does not use the word 'scope'

It is very unhelpful and I still don't know what a scope is.

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

ID: f2086405-ef16-89f4-6308-b015fed23c8f
Version Independent ID: e5c04494-8e45-2942-0c1f-f2029ae2450a
Content: TokenRequestContext.Scopes Property (Azure.Core) - Azure for .NET Developers
Content Source: xml/Azure.Core/TokenRequestContext.xml
Service: azure
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Azure.Core:0.23026885,Managed Identity:0.15519804,Docs:0.14798568'

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Azure.Core:0.23026885,Managed Identity:0.15519804,Docs:0.14798568'

@christothes :  Would you mind taking a look?

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

@scottaddie  What are your thoughts on this? The definition of scope in the oauth context may be too large to add in the doc comments, but perhaps we can link to something like this? https://docs.microsoft.com/en-us/azure/active-directory/develop/v2-permissions-and-consent

@christothes I propose that we link directly to this section: https://docs.microsoft.com/azure/active-directory/develop/v2-permissions-and-consent#scopes-and-permissions

Updated link to scopes and permissions is here: https://learn.microsoft.com/azure/active-directory/develop/scopes-oidc


Update Removed Keys

Updated the delete keys by running diff between getConf in 1.8 and
2.0-RC4
Update the reference to these deleted keys in doc and src code
Simplify the RemovedKey implementation a bit to make my life a bit
easier

Cherry-pick of existing commit.
orig-pr: Alluxio/alluxio#9353
orig-commit: Alluxio/alluxio@79cc37711d07ddc73b3f5f68fa67f6d8f34467c7
orig-commit-author: Bin Fan<EMAIL_ADDRESS>pr-link: Alluxio/alluxio#9353
change-id: cid-e8a91a8251510c3a6160d83229fe355b5954b5f9

Only conflict was to replace a .port property with rpc.port in the docs.

alluxio-bot, merge this please

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4379/
Test FAILed.


ARROW-7082: [Packaging][deb] Add apache-arrow-archive-keyring package

Debian recommends that third party APT repository should provide
apache-arrow-archive-keyring.
https://wiki.debian.org/DebianRepository/UseThirdParty#Key_rollover_and_updates
We can update APT repository sign keys automatically by this
approach. We will add more new release managers. If we can't update
APT repository sign keys automatically, users need to update them
manually when we add a new release manager.
apache-arrow-archive-keyring includes APT source. So install step can
be simplified (no manual keyring download and no manual APT source
creation):
apt install -y -V wget
wget https://.../apache-arrow-archive-keyring.deb
apt install -y -V ./apache-arrow-archive-keyring.deb
apt update
apt install libarrow-dev

Updating install document and release script will be done as follow-up
tasks.

@ursabot crossbow submit -g linux

@ursabot crossbow submit -g linux

@ursabot crossbow submit -g linux

@ursabot crossbow submit -g linux

@ursabot crossbow submit -g linux

@ursabot crossbow submit -g linux

@ursabot crossbow submit -g linux

CentOS 7 build failure is unrelated to this change: https://issues.apache.org/jira/browse/PARQUET-1678
I'll merge this once the running CI is green except CentOS 7.

https://travis-ci.org/kou/arrow/builds/608943059
Green. I'll merge this.


ci: fix rust tool chain file name for rust-rocksdb

The toolchain file name has been changed from rust-toolchain to rust-toolchain.toml

/cc @Connor1996 @purelind

/cc @overvenus

https://ci.pingcap.net/blue/organizations/jenkins/rust_rocksdb_ghpr_test/detail/rust_rocksdb_ghpr_test/586/pipeline is the result with this PR.


fix tests and packaging

pytest wasn't passing locally.  This addresses a few issues including:

#4
#5

closing in favor of #2


New official collaborators

@torfmaster and @Woyten, since you have been doing the lion's share of work bring this library up to speed, do you want to be added as contributors?
This would basically mean you'd be able to merge pull requests yourselves. I think the one thing we'd want us just to make sure there is a good way to contact you (e.g. that you are on the mailing list).
There is no formal process for this library (yet), but managing it core-team-esque, where mostly big design decisions, as well as things like when to do a release version, are done by consensus seems like a good idea.

Hi @alevy, thank you very much for appreciation! :slightly_smiling_face:
Having the ability to merge PRs ourselves would help a lot integrating new features easily! However, for non-trivial, possibly breaking changes I would still want to wait for one of the Tock pros to start and approve a code review. This is the process anyway, right?
Regarding the mailing list, you can just add me.
I am only speaking for myself here. Thus, before adding @torfmaster I would wait for his feedback.

@Woyten you can add yourself to the mailing list (it's just a Google group): https://groups.google.com/forum/#!forum/tock-dev it's not a closed group or anything.
The process for the kernel/c-userland repo is for major changes to be approved by all the "core team". I

@Woyten you can add yourself to the mailing list (it's just a Google group): https://groups.google.com/forum/#!forum/tock-dev it's not a closed group or anything.
The process for the kernel/c-userland repo is for major changes to be approved by all the "core team". We can do something similar for this repo, though I think playing it by ear for now seems fine. Basically, if you are not comfortable merging something, yeah, totally fine to wait to hear from one of us.

Hi @alevy,
thank you for this proposal. I would like join as collaborator as well.


How to set a variable from response?

First of all, great work on the Flashpost plugin. I really love how intuitive it is, and how much effort you've put in it.

I'm not sure if I have to configure something, or it's an issue, but when I go to the "set variable" tab, it looks empty.

Hi
Looks like something is wrong with your collection. Please try creating a new collection and a new request and see if you can set the response variable.


word error in doc for chinese at : 文档/生产者/基本概念  => "可以支持不用的应用场景"  should be  => "可以支持不同的应用场景"

word error in doc for chinese at : 文档/生产者/基本概念  => "可以支持不用的应用场景"  should be  => "可以支持不同的应用场景" ;
The great difference between the two words is easy to cause misunderstanding;
两个词意思相差比较大, 容易造成误解;

Nice, you can go to RocketMQ-site and ask for an issue and PR.

ok, 好的;

I found the place. url This might help you.

thinks , 谢谢;


no grand_average for TFRs

There is a function to average a list of evokeds, but not for AverageTFRs (right?).
Should be fairly simple.

Should be fairly simple.

Except the data needs to be interpolated for bad channels.

just intersect channels as you cannot interpolate properly phase or power.

@agramfort how about an option to either intersect, or just average over the good subset?

just average over the good subset?

that's what I mean by intersect (the good ones)

@agramfort I think the "subset" refers to subset of subjects with the given channel good, i.e. for each channel, you give the average over the subjects for which the channel was good. I guess that means each channel could have a different subset of subjects averaged over. I guess it's okay with me as an option, so long as it's not the default. People should only see that one if they know the number of averages and subset of subjects could differ channel-by-channel.

ok agreed

Yes that's what I meant.

I have an implementation. It's seemingly actually pretty straight forward (at least the intersection solution). I'll PR once any other of my current PRs is closed.

I also have something, but it's not that straight forward. It would be interesting to compare the implementations.

Here is mine:
def grand_average(all_tfr, drop_bads=True):
    """Make grand average of a list TFR data

    Selects the subset of good channels across all datasets.

    The grand_average.nave attribute will be equal the number
    of datasets used to calculate the grand average.

    Note: Grand average TFR shall not be used for source localization.

    Parameters
    ----------
    all_evoked : list of AllTFR data
        The TFR datasets.
    drop_bads : bool
        If True, bad MEG and EEG channels are dropped.

    Returns
    -------
    grand_average : AllTFR
        The grand average data.

    Notes
    -----
    .. versionadded:: 0.10.0
    """
    # check if all elements in the given list are evoked data
    if not all(isinstance(e, AverageTFR) for e in all_tfr):
        raise ValueError("Not all the elements in list are AllTFR data")

    # Copy channels to leave the original evoked datasets intact.
    all_tfr = [e.copy() for e in all_tfr]

    # Interpolates if necessary
    if drop_bads:
        for tfr in all_tfr:
            tfr.drop_channels(tfr.info['bads'], copy=False)
    else:
        pass
        # implement good channel subset here somehow??

    equalize_channels(all_tfr)  # apply equalize_channels
    # make grand_average object using combine_evoked
    grand_average = combine_tfr(all_tfr)
    # change the grand_average.nave to the number of Evokeds
    grand_average.nave = len(all_tfr)
    # change comment field
    grand_average.comment = "Grand average (n = %d)" % grand_average.nave
    return grand_average

def combine_evoked(all_tfr, weights='nave'):
    """Merge TFR data

    Data should have the same channels and the same time instants.
    Subtraction can be performed by passing negative weights (e.g., [1, -1]).

    Parameters
    ----------
    all_tfr : list of TFR
        The evoked datasets.
    weights : list of float | str
        The weights to apply to the data of each TFR instance.
        Can also be ``'nave'`` to weight according to tfr.nave,
        or ``"equal"`` to use equal weighting (each weighted as ``1/N``).

    Returns
    -------
    evoked : AllTFR
        The new TFR data.

    Notes
    -----
    .. versionadded:: 0.10.0
    """
    tfr = all_tfr[0].copy()
    if isinstance(weights, string_types):
        if weights not in ('nave', 'equal'):
            raise ValueError('Weights must be a list of float, or "nave" or '
                             '"equal"')
        if weights == 'nave':
            weights = np.array([e.nave for e in all_tfr], float)
            weights /= weights.sum()
        else:  # == 'equal'
            weights = [1. / len(all_tfr)] * len(all_tfr)
    weights = np.array(weights, float)
    if weights.ndim != 1 or weights.size != len(all_tfr):
        raise ValueError('Weights must be the same size as all_tfr')

    ch_names = tfr.ch_names
    for e in all_tfr[1:]:
        assert e.ch_names == ch_names, ValueError("%s and %s do not contain "
                                                  "the same channels"
                                                  % (tfr, e))
        assert np.max(np.abs(e.times - evoked.times)) < 1e-7, \
            ValueError("%s and %s do not contain the same time instants"
                       % (evoked, e))

    # use union of bad channels
    bads = list(set(tfr.info['bads']).union(*(ev.info['bads']
                                                 for ev in all_tfr[1:])))
    tfr.info['bads'] = bads

    tfr.data = sum(w * e.data for w, e in zip(weights, all_tfr))
    tfr.nave = max(int(1. / sum(w ** 2 / e.nave
                                   for w, e in zip(weights, all_tfr))), 1)
    return tfr

I guess yours is a lot better :)

Hey, that looks pretty good. I wasn't aware of this equalize_channels function.  The documentation should be changed that it also accepts AverageTFR. Just fix the typos and make a PR.
Also, I think there's something wrong with the plot_topo of TFR. I'll look into it.

And by typos I mean references to evoked object.

And in more general, should the weights parameter be exposed to the user? With this implementation, the channel equalization would have to be made by the user before calling the combine_tfr, if he/she wants to use weights='equal'. Same goes for evoked grand average.

I had actually written a setup all on my own to do the job of equalize_channels before I found out the function exists ...

And in more general, should the weights parameter be exposed to the user? With this implementation, the channel equalization would have to be made by the user before calling the combine_tfr, if he/she wants to use weights='equal'. Same goes for evoked grand average.
Right now I'd tend to implement the function as similar to the one for evokeds as possible, and if we want to change evokeds later, that's a different story. Thoughts?

Closed by #2569


nav-list padding and margin off?

Shouldn't the padding and negative margin for the nav-list be equal? The way it is now it's wider than the containing element.
.nav-list {
padding-left: 14px;
padding-right: 14px;
margin-bottom: 0;
}
.nav-list > li > a,
.nav-list .nav-header {
margin-left:  -15px;
margin-right: -15px;
text-shadow: 0 1px 0 rgba(255,255,255,.5);
}

It allow us to click inside the padding area to navigate

I know, but see the image for what i mean; a little hard to spot but see the hover effect, the nav-list overlaps the blue bg by a pixel. Setting the padding also to 15 px (equal to the negative margin) seems to solve this.

@SiNNuT what is your markup and bootstrap version? Here it renders well...
This snippet overrides the one you posted:
.nav-list > li > a {
  padding: 3px 15px;
}

It is located here: https://github.com/twitter/bootstrap/blob/master/docs/assets/css/bootstrap.css#L2211


Adding support for Multipath TCP

More information on https://pluginized-protocols.org/tcp/

@ohault is the ask to have a hook exposed by an MP-TCP implementation, or for someone to implement MP-TCP in an eBPF program, or something else?

@ohault We don't have enough information here to know the details of the ask.  Please add more details of what is being requested and then please reopen this issue.  Thanks!

Indeed, there are two parts for the implementation of this feature :

In one side, there is the core question about making the Windows TCP stack more extensible with eBPF and this point is far wider that the support of Mutipath TCP (see https://inl.info.ucl.ac.be/system/files/tcp-ebpf.pdf ). According to me, this is something at the heart of ebpf-for-windows to bring a symmetry with TCP-BPF implementation in Linux kernel starting at version 4.13 and later ( see https://netdevconf.info/2.2/papers/brakmo-tcpbpf-talk.pdf)
AFIAK, in the ebpf-for-windows architecture, this point is directly related to eBPF Shim for hooks/helpers (netebpfext.sys) and could require updates in TCP/IP stack (tcpip.sys) and/or Layer 2 Network Driver Subsystem (ndis.sys).

On the other side, there are the plugins running eBPF bytecode and Multipath TCP could become one such plugin in the future. In follow-up to https://pluginized-protocols.org/tcp/ , a first start could be supporting the implementation of new TCP options like the Timestamps option described in RFC1323 still not supported by Windows TCP/IP stack.


[bug] Does not compute

How to reproduce

Compile last build
Run elasticsearch-dump dump --file D:\dump\<IP_ADDRESS>.txt --url http://<IP_ADDRESS>:9200

Where <IP_ADDRESS>:9200 is your Elasticsearch server url
Logs
2020/04/27 22:00:05 main.go:117: opt: &main.Opt{filename:"D:\\dump\\<IP_ADDRESS>.txt", url:"http://<IP_ADDRESS>:9200", host:"http://<IP_ADDRESS>:9200", index:"", opType:"data", batchSize:1000, max:0}

2020-04-27T14:00:05Z GET http://<IP_ADDRESS>:9200?q=%2A%3A%2A&scroll=60000ms&size=1000 [status:400 request:157ms]

2020/04/27 22:00:05 main.go:322: Error [400]: {"error":{"root_cause":[{"type":"illegal_argument_exception","reason":"request [/] contains unrecognized parameters: [q], [scroll], [size]"}],"type":"illegal_argument_exception","reason"
:"request [/] contains unrecognized parameters: [q], [scroll], [size]"},"status":400}

For some reason, your HTTP request goes to:
http://<IP_ADDRESS>:9200?q=%2A%3A%2A&scroll=60000ms&size=1000
although it should go to:
http://<IP_ADDRESS>:9200/_search/?q=%2A%3A%2A&scroll=60000ms&size=1000

Oh sorreh, forgot about specifying index lol
Do: elasticsearch-dump dump --file D:\dump\<IP_ADDRESS>.txt --url http://<IP_ADDRESS>:9200/INDEX


Does prompt make sense?

This PR includes a simple script to judge the quality of the SFT prompts in the dataset.

As a very basic first pass this makes sense, but I wonder if:
(a) we can be more specific / ask for fine-grained scores (e.g., coherence, length, etc.). I feel like recent llm-as-a-judge work is trending to more fine-grained scores (e.g. ultrafeedback finegrained vs overall).
(b) If we can make this more quantitative somehow. Can we have some analysis of false positives or similar? Or maybe some small val set that we can reason about and quality-check with?


filter out request from the error log entries

Follow up issue: https://github.com/commercetools/commercetools-sdk-typescript/issues/240

Could you provide an example of how it looked before and now? If we remove complete payload is it still enough to figure out what the issue is?

Could you provide an example of how it looked before and now? If we remove complete payload is it still enough to figure out what the issue is?

Check from the gist here simply the request going to CTP will be deleted from the logs, only response will stay. Let me know what do you think ?

@ahmetoz Thanks, I commented on the gist.


[Feature] Implementation of mx.meshgrid

The equivalent of torch.meshgrid is needed in mlx for easier and more convenient way of creating 2D maps from 1D tensors. Please refer to here for Pytorch's documentation on torch.meshgrid

Sure! I will start working on that

@awni Feature merged. This issue can be closed.


customRequestHeaders always overwrite exist header value

It seems that it also overwrites when there is already the same header in the request.
I want the headers to remain as they are when they already exist in the request.
For example, if you set it like below, even if you request with app-id: world, it will be overwritten with the hello header.
    http:
      middlewares:
        add-dapr-app-id:
          headers:
            customRequestHeaders:
              app-id: hello

Is there a setting to enable this? Or does it always overwrite?
Thank you.

Hello @nagisatosanae,
Thanks for your interest in Traefik!

Is there a setting to enable this? Or does it always overwrite?

No, there is currently no workaround (https://github.com/traefik/traefik/issues/9471 tracks the feature request).
I'm closing this issue as duplicate accordingly.


Switch to E.164 format

What has been done

Switch to E.164 formatted phone value

Return e164 phone in onChange callback by default
Allow get full phone when using component with disabledDialCodeAndPrefix

Refactor user input handling: move logic from hook into separate helper files
Small cursor handling updates

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket: