Error: 8 RESOURCE_EXHAUSTED: Bandwidth exhausted

We get this error often:
Error: 8 RESOURCE_EXHAUSTED: Bandwidth exhausted
    at Object.callErrorFromStatus (/root/repo/node_modules/@grpc/grpc-js/build/src/call.js:30:26)
    at Http2CallStream.<anonymous> (/root/repo/node_modules/@grpc/grpc-js/build/src/client.js:96:33)
    at Http2CallStream.emit (events.js:215:7)
    at Http2CallStream.EventEmitter.emit (domain.js:476:20)
    at /root/repo/node_modules/@grpc/grpc-js/build/src/call-stream.js:75:22
    at processTicksAndRejections (internal/process/task_queues.js:75:11) {
  code: 8,
  details: 'Bandwidth exhausted',
  metadata: Metadata { internalRepr: Map {}, options: {} },
  note: 'Exception occurred in retry method that was not classified as transient'
}

Fully described in grpc repo: https://github.com/grpc/grpc-node/issues/1158
Cross-posting here for visibility.

@kirillgroshkov RESOURCE_EXHAUSTED is a server-side error that I would expect to be thrown if your project is hitting a quota see.
It seems strange to me that this error would have started cropping up more frequently in a specific version of @grpc/grpc-js.
As an experiment, perhaps try the grpc dependency, and see if you are continuing to see these errors:
const {Datastore} = require('@google-cloud/datastore');
const grpc = require('grpc');
const datastore = new Datastore({grpc});

So, I've conducted this experiment with require('grpc') on a same script that failed after ~7 minutes 4 times in a row. With require('grpc') it passed 2 times (taking 1 hour each, to load few millions of records from Datastore). So, error is not happening then. If I switch back - error is there.
Project we're running it on has a billing account and we're paying for it, so it's not a free quota limit.

So, native gprc was running in production for 2 days and we see 0 errors from grpc. Neither from our cron jobs. Good for our production service, but it also proves that @grpc/grpc-js is not working as expected, the bug still exists there.

@kirillgroshkov thank you :+1: for the follow up. I've been having a discussion with @murgatroid99 and @alexander-fenster internally.
I'm wondering, is there a chance that grpc is still hitting the RESOURCE_EXHAUSTED exception, but recovering gracefully? is there anything in the logs that might point to that?

I have quickly checked our StackDriver and see no logs with RESOURCE_EXHAUSTED string.
Also, when I ran the 1M-rows cron job script that originally was crashing after 7 minutes, with native grpc it was not only not crashing, but it wasn't printing any errors or anything at all from grpc. I can try with more debugging enabled (if you tell me which env variables to set)

This issue is interesting: https://github.com/googleapis/nodejs-firestore/issues/765. Same error, traces back to grpc-js using Node.js HTTP/2, which has an open issue that could address this when fixed. The workaround of using grpc is the best bet for now. @murgatroid99 -- does that sound right?

@kirillgroshkov, @stephenplusplus, reading this thread on Node.js, @kirillgroshkov if you would be willing to run your workload one additional time with @grpc/grpc-js, but on<EMAIL_ADDRESS>  this would isolate the issue to the regression Stephen mentions.
At which point we might be able to update both this thread and grpc/grpc-node#1158, noting that they're pending on an upstream fix in Node.js.

@kirillgroshkov if you would be willing to run your workload one additional time with @grpc/grpc-js, but on<EMAIL_ADDRESS>
Yes, running it now.
First, I ran it again on Node 12.13.0 with @grpc/grpc-js and verified that it still crashes.
Now running on<EMAIL_ADDRESS>

So, with Node 12.13.0 it failed after ~10 minutes with the RESOURCE_EXHAUSTED error.
With Node 10.16.2 it failed after 53 minutes with this error:
uncaughtException: 13 INTERNAL:
Error: 13 INTERNAL:
    at Object.callErrorFromStatus (/Users/kirill/Idea/NCBackend3/node_modules/google-gax/node_modules/@grpc/grpc-js/build/src/call.js:30:26)
    at Http2CallStream.call.on (/Users/kirill/Idea/NCBackend3/node_modules/google-gax/node_modules/@grpc/grpc-js/build/src/client.js:96:33)
    at Http2CallStream.emit (events.js:203:15)
    at Http2CallStream.EventEmitter.emit (domain.js:448:20)
    at process.nextTick (/Users/kirill/Idea/NCBackend3/node_modules/google-gax/node_modules/@grpc/grpc-js/build/src/call-stream.js:75:22)
    at process._tickCallback (internal/process/next_tick.js:61:11)  events.js:203
onFatalError { Error: 13 INTERNAL:
    at Object.callErrorFromStatus (/Users/kirill/Idea/NCBackend3/node_modules/google-gax/node_modules/@grpc/grpc-js/build/src/call.js:30:26)
    at Http2CallStream.call.on (/Users/kirill/Idea/NCBackend3/node_modules/google-gax/node_modules/@grpc/grpc-js/build/src/client.js:96:33)
    at Http2CallStream.emit (events.js:203:15)
    at Http2CallStream.EventEmitter.emit (domain.js:448:20)
    at process.nextTick (/Users/kirill/Idea/NCBackend3/node_modules/google-gax/node_modules/@grpc/grpc-js/build/src/call-stream.js:75:22)
    at process._tickCallback (internal/process/next_tick.js:61:11)
  code: 13,
  details: '',
  metadata: Metadata { internalRepr: Map {}, options: {} },
  note:
   'Exception occurred in retry method that was not classified as transient' }

We have been getting a combination of these errors recently. I haven't tried rolling back to a previous version, but are currently using "@google-cloud/datastore": "^4.5.2". I don't believe we're hitting any of our quotas.

I'm getting the same errors trying to delete a million + rows...
Error: 8 RESOURCE_EXHAUSTED: Bandwidth exhausted
    at Object.callErrorFromStatus (c:\dev\vsts\kinetix-backend\node_modules\google-gax\node_modules\@grpc\grpc-js\build\src\call.js:30:26)
    at Http2CallStream.<anonymous> (c:\dev\vsts\kinetix-backend\node_modules\google-gax\node_modules\@grpc\grpc-js\build\src\client.js:96:33)
    at Http2CallStream.emit (events.js:228:7)
    at c:\dev\vsts\kinetix-backend\node_modules\google-gax\node_modules\@grpc\grpc-js\build\src\call-stream.js:98:22       at processTicksAndRejections (internal/process/task_queues.js:76:11) {
  code: 8,
  details: 'Bandwidth exhausted',
  metadata: Metadata { internalRepr: Map {}, options: {} },
  note: 'Exception occurred in retry method that was not classified as transient'
}

I ran into the problem with
    "@google-cloud/datastore": "^4.5.2",
I upgraded to
    "@google-cloud/datastore": "^5.0.6",
and I still have the issue...
Anything else I can add?
--
fwiw -- I think the issue is I'm doing a Promise.All(...delete 75 rows)... on all 1.5 million rows.
After ~300-400k deletes, everything starts failing.
I assume this should not be the case, but I'm not sure.
Thanks, Chad

Experiencing 8 RESOURCE_EXHAUSTED: Bandwidth exhausted errors also.

Just FYI, it's not related to any quotas by google APIS, it's an issue with node grpc client

Am having the same issue
    ... 474 more items
  ],
  code: 8,
  details: 'Bandwidth exhausted',
  metadata: Metadata { internalRepr: Map(0) {}, options: {} }
}

@Ikxyz , @lokoArt
What versions of this library are you using? There have been a number of (largish) changes to the grpc transport in our node clients since this issue was opened.

@bcoe Looking back through this it seems the stack is in GAX. But this is also an older issue. I am leaning towards closing this and reopening if it comes up again?

@bcoe do you think this is something we can improve with better logging out of GAX? As a user it seems the hard bit is this doesn't indicate where it exhausted resources.
To unblock as a user, it is likely best to retry with exponential backoff when this happens.

@kirillgroshkov are you continuing to see this issue on newer versions of the library, it looks like this thread has been quite for a while.
I've created a tracking issue to provide better contextual information about what requests are triggering the exception. Also, there's been work to add more contextual information in the errors thrown by gRPC (for some varieties of errors).

@kirillgroshkov are you continuing to see this issue on newer versions of the library, it looks like this thread has been quite for a while.

We actually stopped trying after my message in November 2019. Native grpc works well for us and honestly I don't have that much incentive to experiment with non-fully-working versions on our paying users. I can try once a year maybe.
Do you expect this issue to be solved, or is it just additional debugging information was added?

Closing and marking as external, as it seems the issue here is around grpc-js and not @google-cloud/datastore. Feel free to reopen or start a new issue if I am mistaken.

I had same problem for reading all data from firebase, after a while I figured out if we stop this process and run a new process we can pass the error and continue the job.
then I used child-process and it helped me:

I did write a parent script and a child script,
Parent script runs child script as a child-process,
The child goes through a collection until get [8 RESOURCE_EXHAUSTED] error, then send a message to parent to inform it from the error.
Then parent kills child and create a new one and tells it where to start reading again.

This is a solution works 100 percent, but it's a little advance and beginners-intermediates may could not able to implement it.

@navidshad That sounds perfect. Do you have a code example by any chance?


typo: adhybridhealthservice/resource-manager/Microsoft.ADHybridHealthService

Double word "the"

Can one of the admins verify this patch?

Automation for azure-sdk-for-js

Automation for azure-sdk-for-python

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-go

Automation for azure-sdk-for-java

@nschonni  closed/re-opened the PR to kick off CI again. The changes in the PR should not cause a CI break.

@praries880 looks like the only left over failure is from Avocado complaining about some unreferenced examples files


updates cp to keep the previous behavour

my new implementation of the cp function got merged
https://github.com/JuliaLang/julia/pull/10888
sets remove_destination=true otherwise if you rerun you get an:
ERROR: ArgumentError: 'static/custom.css' exists.
`remove_destination=true` is required to remove
'static/custom.css' before copying.

Excellent, thanks! Once AppVeyor gives a green light I'll merge.

I guess this requires a 0.1.8 tagging since 0.1.7 breaks on master.

could be


[feature request] have switch input default to null if no connection.

I did a scan, but have not seen this request before. Apologies if that's been covered already.
This is important when you turn on and off groups. If a group is off, then the connection is off as well. Comfy fails..
This would be really really handy... See this thread.
https://www.reddit.com/r/comfyui/comments/18z3a41/any_way_to_default_a_node_value_int_or_float_to_0/
Thanks bunches!

I'm a little confused. The Any Switch inputs work like null in that when their muted, they are essentially null (or None in python). Same as when not connected. If there are no connected or unmuted inputs, the Any Switch output will be None.
That is all intended, and is "default to null" essentially.
Do you have an example workflow screenshot of what you mean if that is not the case?


wiki link points to wiki

You pasted the link in a second time. Fixed and merged. Thanks for spotting my oversight.


CMake refactor

Extracted a subset of the CMake file that is consistent between all my projects.
Also added some warning fixes for building the library with the legacy Xbox One XDK.

https://github.com/walbourn/directxtextest/pull/38


Live Demo doesn't work

The Live demo doesn't work. Seems like the libraries can't be loaded from cloudflare.

@krebbl The live demo seems to be working for me. If you're still seeing this issue, can you share the steps to reproduce it and the error you're getting, or a screenshot? Thanks in advance!

@greg-db its now working again. maybe it was a DNS problem on my side.


Two new components to center content and to add a field to a form

Cool stuff, Im going to merge it. @vicmator and Rocío can you close this issue and update the kanban?Thanks


org.kohsuke/graphviz-api 1.1

Type: Missing
Summary:
org.kohsuke/graphviz-api 1.1
Details:
There's a file in the package that points to this website where you can see the license is CPL https://www.graphviz.org/license/
Resolution:
CPL-1.0
Affected definitions:

graphviz-api 1.1

This package has a file that points to the website that has the license info:


Update argument passed for assetRoots.

Fixes #4388

@martinbigio can you import this?

Thanks @christopherdro!

@facebook-github-bot shipit


Fix #7141 Use compressed IPv6 for comparisons

The IPv6 of an interface may have been entered in full form, e.g.
2001:470:dead:beef:0:0:0:1
Which does not compare directly with the compressed form:
2001:470:dead:beef::1
So compress IPv6 addresses in functions before using them.

Note1: There may be a whole lot more places where we should do a compress before using an IPv6 address, for consistency to ensure that any comparisons between IPv6 addresses will succeed. Or we should use some compare_ipv6_addresses() function whenever doing comparisons and that function can compress the addresses before doing a string comparison or...
Note2: The example in the bug report uses a "compressed" address that is not valid according to the combination of standards;
https://tools.ietf.org/html/rfc4291#section-2.2
https://tools.ietf.org/html/rfc5952
Net_IPv6::compress("2001:470:dead:beef:1:2:0:3") returns the input string (no compression possible)
2001:470:dead:beef:1:2:0:3
does not compress to:
2001:470:dead:beef:1:2::3
because rfc5952 says that you are not allowed to compress a single zero.
Unfortunately 2001:470:dead:beef:1:2::3 is accepted by the GUI validation as an IPv6 address. So I guess the Net_IPv6 package is being inconsistent in allowing this as a valid IPv6 address, but not compressing to it. I will have a look at that separately.

The underlying issue is really that Net_IPv6 is being "nice" about uncompressing a string that is thought to be IPv6 "text" format. It will uncompress any single "::" sequence, e.g in the following the "::" is actually only compressing a sequence of 2 "0" fields. According to the standard, the 3 "0" fields at the end are the ones that should have been compressed.
require_once('pfsense-utils.inc');
$ipaddr = "2001:2::1:0:0:0";
var_dump($ipaddr);
$x = is_ipaddrv6($ipaddr);
var_dump($x);
$cip = Net_IPv6::compress($ipaddr);
var_dump($cip);
$cip = Net_IPv6::compress($ipaddr, true);
var_dump($cip);
$uip = Net_IPv6::uncompress($ipaddr);
var_dump($uip);
$cip = Net_IPv6::compress($uip);
var_dump($cip);

and the output is:
/tmp/o753Ay:7:
string(15) "2001:2::1:0:0:0"
/tmp/o753Ay:9:
bool(true)
/tmp/o753Ay:11:
string(15) "2001:2::1:0:0:0"
/tmp/o753Ay:13:
string(14) "2001:2:0:0:1::"
/tmp/o753Ay:15:
string(18) "2001:2:0:0:1:0:0:0"
/tmp/o753Ay:17:
string(14) "2001:2:0:0:1::"

So an address like this is accepted by validation. If we do an ordinary compress() on it, it stays the same. If we "force" compress by compress($ip, true), then the underlying code does uncompress() and then compresses the full address, resulting in the expected canonical form.
That is a bit of a quirk of the Net_IPv6 code, that if the user has "pre-compressed" the address into some "reasonable" form then Net_IPv6 does not fuss about it. However it means that 2 "compressed" addresses that validate as IPv6 addresses can be different strings.

See #3414 for what I think is a better way to start on this.


[EOSTAT-14] Odin integration

Add odin dependency
Use odin to run checks against EO source code

There is a problem with the Program class. When I pass it "\\Phi" as a locator I expect it to return me all the objects from the sources to be analyzed, but instead I get an exception, so OdinAnalysis::errors now throws an exception at the stage of xmir.apply("\\Phi").

@Sitiritis I left some comments. As for the Program bug, please make a ticket.

@Sitiritis I'm in general in favor of "fail fast" approach: let the app die as soon as possible, read: https://www.yegor256.com/2015/08/25/fail-fast.html

@Sitiritis also, run mvn clean install -Pqulice on your branch, you will see issues

@Sitiritis I left some comments. As for the Program bug, please make a ticket.

#16

@Sitiritis see a few comments above

Codecov Report

Merging #13 (8b4fd8d) into master (3f2af5f) will decrease coverage by 2.22%.
The diff coverage is 68.75%.

@@             Coverage Diff              @@
##             master      #13      +/-   ##
============================================
- Coverage     95.62%   93.40%   -2.23%
- Complexity       44       45       +1
============================================
  Files             5        6       +1
  Lines           183      197      +14
  Branches         21       21
============================================
+ Hits            175      184       +9
- Misses            4        9       +5
  Partials          4        4

Impacted Files
Coverage Δ

src/main/java/org/polystat/odin/OdinAnalysis.java
37.50% <37.50%> (ø)

src/main/java/org/polystat/Polystat.java
90.32% <100.00%> (+2.32%)
:arrow_up:

Continue to review full report at Codecov.

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

@Sitiritis see above. Also, maybe you can add a simple unit test for this new class, similar to what we have for Reverses. In this case, you put your .eo file into the right place and it will be used properly during testing.


feat: add support for signed user metadata

Adds support for signed user metadata in notation sign and notation verify. Relevant spec
example sign usage:
chienb@a07817b52895 notation % notation sign $IMAGE --user-metadata io.wabbit-networks.buildId=123 --user-metadata io.wabbit-networks.buildTime=123
Warning: Always sign the artifact using digest(`@sha256:...`) rather than a tag(`:v1`) because tags are mutable and a tag reference can point to a different artifact than the one signed.
Resolved artifact tag `v1` to digest `sha256:5a07385af4e6b6af81b0ebfd435aedccdfa3507f0609c658209e1aba57159b2b` before signing.
Successfully signed localhost:5000/net-monitor@sha256:5a07385af4e6b6af81b0ebfd435aedccdfa3507f0609c658209e1aba57159b2b

example metadata displayed on verification (without metadata flag)
chienb@a07817b52895 notation % notation verify $IMAGE
Resolved artifact tag `v1` to digest `sha256:5a07385af4e6b6af81b0ebfd435aedccdfa3507f0609c658209e1aba57159b2b` before verification.
Warning: The resolved digest may not point to the same signed artifact, since tags are mutable.
Successfully verified signature for localhost:5000/net-monitor@sha256:5a07385af4e6b6af81b0ebfd435aedccdfa3507f0609c658209e1aba57159b2b

The artifact was signed with the following user metadata.

KEY                          VALUE
io.wabbit-networks.buildTime   123
io.wabbit-networks.buildId     123

example verification:
chienb@a07817b52895 notation % notation verify $IMAGE --user-metadata io.wabbit-networks.buildTime=123
Resolved artifact tag `v1` to digest `sha256:5a07385af4e6b6af81b0ebfd435aedccdfa3507f0609c658209e1aba57159b2b` before verification.
Warning: The resolved digest may not point to the same signed artifact, since tags are mutable.
Successfully verified signature for localhost:5000/net-monitor@sha256:5a07385af4e6b6af81b0ebfd435aedccdfa3507f0609c658209e1aba57159b2b

The artifact was signed with the following user metadata.

KEY                            VALUE
io.wabbit-networks.buildTime   123
io.wabbit-networks.buildId     123

example verification failure
chienb@a07817b52895 notation % notation verify $IMAGE --user-metadata foo=bar
Resolved artifact tag `v1` to digest `sha256:5a07385af4e6b6af81b0ebfd435aedccdfa3507f0609c658209e1aba57159b2b` before verification.
Warning: The resolved digest may not point to the same signed artifact, since tags are mutable.
Error: signature verification failed: signature verification failed

Signed-off-by: Byron Chien<EMAIL_ADDRESS>

related: https://github.com/notaryproject/notation/pull/507

Codecov Report

Merging #242 (a6a6e72) into main (c90d521) will increase coverage by 0.66%.
The diff coverage is 76.66%.

@@            Coverage Diff             @@
##             main     #242      +/-   ##
==========================================
+ Coverage   73.31%   73.97%   +0.66%
==========================================
  Files          23       23
  Lines        1746     1802      +56
==========================================
+ Hits         1280     1333      +53
- Misses        372      375       +3
  Partials       94       94

Impacted Files
Coverage Δ

notation.go
57.41% <12.50%> (-2.45%)
:arrow_down:

signer/plugin.go
65.71% <40.00%> (-0.63%)
:arrow_down:

verifier/verifier.go
76.29% <85.18%> (+2.82%)
:arrow_up:

signer/signer.go
76.92% <100.00%> (+5.49%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

@byronchien  could you fix the conflicts?

resolved conflict


Fix disk_caching_stats

App is not compiling
Explain how you tested your changes:

dune build src/app/disk_caching_stats

Checklist:

[x] Dependency versions are unchanged

Notify Velocity team if dependencies must change in CI

[ ] Modified the current draft of release notes with details on what is completed or incomplete within this project
[x] Document code purpose, how to use it

Mention expected invariants, implicit constraints

[x] Tests were added for the new behavior

Document test purpose, significance of failures
Test names should reflect their purpose

[x] All tests pass (CI will check this if you didn't)
[x] Serialized types are in stable-versioned modules
[x] Does this close issues? None

!ci-build-me


Update Time and Time::Span methods for 0.24

Works with newer version of Crystal (0.24), see epoch

This was merged via #21, closing


Search Filter Component Scoped Style

Hello Chris,
I am having a hard time getting the scoped style to apply to my SearchFilter component to position correctly with the right size just above the results.

The search filter scss file
.filters {
    display: flex;
    margin-bottom: 20px;
    align-items: baseline;
    justify-content: flex-end;

    label {
        text-transform: uppercase;
        margin-right: 10px;
    }

    input {
        margin-right: 20px;
        width: 100px;
    }

    button:first-of-type {
        margin-right: 10px;
    }
}

The SearchFilter razor

@inject NavigationManager NavManager

    Max Length
    <input id="maxLength"
           type="number"
           class="form-control"
    @bind="_maxLength" />
    <button class="btn btn-outline-primary"
    @onclick="FilterSearchResults">
        Filter
    </button>
    <button class="btn btn-outline-secondary"
    @onclick="ClearSearchFilter">
        Clear
    </button>

@code {
    private int _maxLength;

    [Parameter, EditorRequired] public string SearchTerm { get; set; } = default!;

    private void FilterSearchResults()
        => NavManager.NavigateTo($"/search/{SearchTerm}/maxlength/{_maxLength}");

    private void ClearSearchFilter()
    {
        _maxLength = 0;
        NavManager.NavigateTo($"/search/{SearchTerm}");
    }
}

Snippet of the SearchPage razor

Home

Search

Search Results for "@SearchTerm"

<SearchFilter SearchTerm="@SearchTerm" />

When I inspect the element with Chrome Dev Tools
BlazingTrails.Client.styles.css

/* /Features/Home/SearchFilter.razor.rz.scp.css */
@charset "UTF-8";
ï»¿.filters[b-vca69nmj2w] {
  display: flex;
  margin-bottom: 20px;
  align-items: baseline;
  justify-content: flex-end;
}
ï»¿.filters label[b-vca69nmj2w] {
  text-transform: uppercase;
  margin-right: 10px;
}
ï»¿.filters input[b-vca69nmj2w] {
  margin-right: 20px;
  width: 100px;
}
ï»¿.filters button:first-of-type[b-vca69nmj2w] {
  margin-right: 10px;
}

Elements from the page

Max Length
    <input id="maxLength" type="number" class="form-control" b-vca69nmj2w="">
    <button class="btn btn-outline-primary" b-vca69nmj2w="">
        Filter
    </button>
    <button class="btn btn-outline-secondary" b-vca69nmj2w="">
        Clear
    </button>

Any ideas what I have done wrong? I have gone over the section several times thinking I missed something, but can't find the issue.

Is it a caching issue? I've found this sometime and have needed to do a hard reload on the browser for the new styles to be picked up.

Hi Chris,
I don't believe that to be the case. I have done a hard reload and cleared all cache files and still get the same result
I am using chrome version 117.0.5938.150

Something in the scss file that i was using. I deleted it, recreated it after downloading yours for Chapter 4 and now it works. Dunno it was anomalous


Update  package name for left springboot module

Update  package name for left springboot module

@wujack778 I think you can run the unit test and integration test in your local env first, it helps you figure out the problem as soon as possible. you can refer to these doc:
integration-test: here
unit test: here

@zhoufenqin understood

I will open a new pr for this until Moary finish the Update for cosmos modules.


Deploys kill code of conduct

Looks like saving as a MD file was a bad idea. Let's make the code of conduct more persistent

Fixed by linking a static code of conduct


Frequent compile error after rustup update

Recently I frequently encounter errors similar to the following:
   Compiling rustc-ap-rustc_data_structures v148.0.0
error[E0599]: no method named `start` found for type `R` in the current scope
   --> /home/topecongiro/.cargo/registry/src/github.com-1ecc6299db9ec823/rustc-ap-rustc_data_structure
    |
122 |         let start = match range.start() {
    |                                 ^^^^^
    |
    = help: items from traits can only be used if the trait is implemented and in scope
    = note: the following trait defines an item `start`, perhaps you need to implement it:
            candidate #1: `arrayvec::range::RangeArgument`

This happens after I run rustup update, and gets fixed after updating the rustc-ap-* crates to the latest version.
AFAIK rustfmt is not broken in rustc repo, so that is good. But it is quite unfortunate that we need to update rustc-ap-* crates everyday (or at least with every PR) to make ci work.
@nrc Do you have any idea what we can do?

Fixed now, I think


fix(learn): improve landing page

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #38658

Changed text and increased size of CTA button
Change company names on landing pages to company logo in react svg component
Added featured on section in landing page

Thank you for your patience.
I am just double checking with the team to see if we need to add a couple of tweaks before merging.

current state

current /learn

I would LGMT this but I modified the PR as well.

I do not like the clutter on /learn, it's going to be an annoying experience to get to challenges on the page.
Should be reverted.

Presumably people are going to land on https://www.freecodecamp.org/ and then navigate to https://www.freecodecamp.org/learn, right?  If that's the case, it's probably not necessary to repeat the landing page stuff on /learn.

Also, while we are at this, we do can change the CTA on the landing page for signed in users.
We have the state of the user available on the page and can show continue to challenges instead.

I do not like the clutter on /learn, it's going to be an annoying experience to get to challenges on the page.

The As featured in section was not added to /learn. Only the company names has been replaced with images.
Just to clarify, you would like to revert that back to text?

We have the state of the user available on the page and can show continue to challenges instead.

I will look into that.

Just to clarify, you would like to revert that back to text?

I am not sure of the solution.
IMHO I would let you be the best judge on the UX here. As a user, I hit the learn page and see a bunch of text (or images with this PR) every time (which I am not interested in), I would like to get to the challenge I was on, see my progress etc.
My advice would be to start thinking and using the page as a user.

As a user, I hit the learn page and see a bunch of text (or images with this PR) every time (which I am not interested in), I would like to get to the challenge I was on, see my progress etc.

I completely agree. the signed in users see a different view that does not include the intro text and images. maybe we add a button to the stack that takes users to the next challenge.

How about replacing "If you are new to coding, we recommend you start at the beginning" with "Jump to your current challenge" (or similar) once the user has completed some challenges?

Presumably people are going to land on https://www.freecodecamp.org/ and then navigate to https://www.freecodecamp.org/learn, right? If that's the case, it's probably not necessary to repeat the landing page stuff on /learn.

That could be one route. if users go to landing page first and sign in they would not see the same info twice. I believe the direction is to have learn lean and showcase our services at some point.

Can we not automatically redirect if a user is logged in?

the signed in users see a different view that does not include the intro text and images.

Right, but my comment wasn't about signed in users either. It was about what any user would see, when they are trying out the curriculum.

Can we not automatically redirect if a user is logged in?

We can, and should. This was broken back in november launch, was fixed long ago and was forgotten 🤷

Thank you all for your valuable feedback.
Here is the proposal:
I will revert the changes to /learn on this pr and limit the scope to the landing page. Additionally, I created another issue (38658) to discuss the improvement to learn which we could bring it to Quincy's attention for final approval.

The last push removes changing the logo row with image row.
I left the essential changes such as keeping the cta size in proportion with text.


Help with nodejs setup

Hi,
I know that I am a newbie in nodejs and its build tool.
So I tried to look looking to add vbscript (most of syntax somewhat like c) and I tried to use https://glitch.com as dev environment. I got error when doing cmd
npm install -g typescript
npm install -g eslint

permission denied, mkdir '/home/nvm/.nvm/versions/node/v8.11.1/pnpm-global/1_shamefully_flatten'

I am not familiar with PNPM.  I believe it is https://github.com/pnpm/pnpm which looks like a different form of package manager.  Perhaps it could be related to https://github.com/pnpm/pnpm/issues/1097
Did you attempt to install typescript and eslint as separate commands?  If so did either of them install successfully?


chore: implement cards carousel

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #229
Test URLs:

Before: https://main--moleculardevices--hlxsites.hlx.page/drafts/mhaack/carousel
After: https://carousel-cards--moleculardevices--hlxsites.hlx.page/drafts/mhaack/carousel
All Carousels for cross-check: https://carousel-cards--moleculardevices--hlxsites.hlx.page/drafts/tuicu/carousels
Real Sample page: https://carousel-cards--moleculardevices--hlxsites.hlx.page/products/clone-screening

@andreituicu I change export async function createCarousel (removed default) so we can actually use the carousel directly. With that we can maybe also change "feature-products-carousel" to "carousel" and remove one extra block. WDYT?

@mhaack Since the defaultStyling carousel configuration was changed, these too need to be adapted:
https://github.com/hlxsites/moleculardevices/blob/main/blocks/citation-list/citation-list.js#L63
https://github.com/hlxsites/moleculardevices/blob/main/blocks/blog-list/blog-list.js#L47
afterwards should be good to go.


Empty check boxes need more contrast with adjacent colors

https://www.w3.org/WAI/WCAG21/Understanding/non-text-contrast.html

For active controls any visual information provided that is necessary for a user to identify that a control is present and how to operate it must have a minimum 3:1 contrast ratio with the adjacent colors

For designing focus indicators, selection indicators and user interface components that need to be perceived clearly, the following are examples that have sufficient contrast...

All of our ~input controls~ checkboxes in the dark & light theme need to have a 3:1 contrast ratio between its background and adjacent color in the empty state. So either the border (if it has one) or the container's background.

Dark
Light

cc @isidorn @roblourens @rzhao271

@misolori I think dropdown doesn't need 3:1 border if it won't be empty. This rule only applies to controls without text or icon inside. If it has text, then the text will be required to have 4.5:1 CCR from another rule (icon will have 3:1). Either of the content inside will satisfy the overall active indicator rule.

Gửi từ điện thoại thông minh Samsung Galaxy của tôi.
-------- Tin nhắn gốc --------Từ: Cherry Wang @.> Ngày: 05/04/2022  23:58  (GMT+07:00) Đến: microsoft/vscode @.> Cc: Subscribed @.***> Chủ đề: Re: [microsoft/vscode] Input controls need more contrast with
  adjacent colors (Issue #146834)
@misolori I think dropdown doesn't need 3:1 border if it won't be empty. This rule only applies to controls without text or icon inside. If it has text, then the text will be required to have 4.5:1 CCR from another rule (icon will have 3:1). Either of the content inside will satisfy the overall active indicator rule.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.>
[
{
@.": "http://schema.org",
@.": "EmailMessage",
"potentialAction": {
@.": "ViewAction",
"target": "https://github.com/microsoft/vscode/issues/146834#issuecomment-1089034581",
"url": "https://github.com/microsoft/vscode/issues/146834#issuecomment-1089034581",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
@.***": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]

Verified with Light+ and Dark+ themes in the Settings editor.
The v2 versions of those themes might have an issue, though.


Sorted set processed in reverse

It appears that the zadd callback is called in reverse order (descending, highest to lowest score). Is there any way to process the sorted set in ascending order?

No, there isn't a way to control that. We process the entries in the order in which they are stored in the rdb file. The parser is stateless, it is designed to use as little memory as possible during parsing. Returning records in a different order will require holding all elements in memory, and that isn't a good idea.
You can easily solve this by writing a new class that extends RdbCallback. Accumulate records in zadd, in end_sorted_set method sort all the records and apply your business logic.


Release 2.2.1

Could this be merged?
I'm feeling dirty using -e git+https://github.com/onefinestay/nameko.git#egg=nameko in my requirements. :)

hi, yes sorry. given the required eventlet bump i think i want to at least mention that in the changelog and make the bump a bit bigger. apologies for the delay

Superseded by https://github.com/onefinestay/nameko/pull/315


JWTClaims doesn't support fractional-seconds timestamps

I've run into issues verifying JWT-formatted VPs that have fractional timestamps (in particular nbf).  It seems that ssi::vc::JWTClaims uses Option<i64> for timestamps, but the JWT standard calls for these to be NumericDate values, which allow for representing fractional-seconds values using JSON numeric values (which I believe are really f64) -- see:

https://www.w3.org/TR/vc-data-model/#jwt-encoding
https://datatracker.ietf.org/doc/html/rfc7519#section-2 under "NumericDate"

I tried to modify JWTClaims to use Option<f64>, and modified the relevant code, but this didn't fix the issue.  I'm probably missing some subtlety about the encoding or JWT standard, perhaps about the precision (e.g. millis, micros, nanos).  Here's a diff (against the current main branch) of what I did, if that helps.  I can also produce a PR if that's easier to work with.
diff --git a/src/vc.rs b/src/vc.rs
index 8b183dc..ec66311 100644
--- a/src/vc.rs
+++ b/src/vc.rs
@@ -312,13 +312,13 @@ pub enum StringOrURI {
 pub struct JWTClaims {
     #[serde(skip_serializing_if = "Option::is_none")]
     #[serde(rename = "exp")]
-    pub expiration_time: Option<i64>,
+    pub expiration_time: Option<f64>,
     #[serde(skip_serializing_if = "Option::is_none")]
     #[serde(rename = "iss")]
     pub issuer: Option<StringOrURI>,
     #[serde(skip_serializing_if = "Option::is_none")]
     #[serde(rename = "nbf")]
-    pub not_before: Option<i64>,
+    pub not_before: Option<f64>,
     #[serde(skip_serializing_if = "Option::is_none")]
     #[serde(rename = "jti")]
     pub jwt_id: Option<String>,
@@ -737,7 +737,9 @@ impl Credential {
             None => return Err(Error::MissingCredential),
         };
         if let Some(exp) = claims.expiration_time {
-            vc.expiration_date = Utc.timestamp_opt(exp, 0).latest().map(|time| VCDateTime {
+            let exp_secs = exp.floor() as i64;
+            let exp_nsecs = ((exp - exp.floor()) * 1.0e9) as u32;
+            vc.expiration_date = Utc.timestamp_opt(exp_secs, exp_nsecs).latest().map(|time| VCDateTime {
                 date_time: time.into(),
                 use_z: true,
             });
@@ -750,7 +752,9 @@ impl Credential {
             }
         }
         if let Some(nbf) = claims.not_before {
-            if let Some(time) = Utc.timestamp_opt(nbf, 0).latest() {
+            let nbf_secs = nbf.floor() as i64;
+            let nbf_nsecs = ((nbf - nbf.floor()) * 1.0e9) as u32;
+            if let Some(time) = Utc.timestamp_opt(nbf_secs, nbf_nsecs).latest() {
                 vc.issuance_date = Some(VCDateTime {
                     date_time: time.into(),
                     use_z: true,
@@ -795,7 +799,7 @@ impl Credential {
             expiration_time: vc
                 .expiration_date
                 .as_ref()
-                .map(|date| date.date_time.timestamp()),
+                .map(|date| date.date_time.timestamp_nanos() as f64 / 1.0e9),
             issuer: match vc.issuer.take() {
                 Some(Issuer::URI(uri)) => Some(StringOrURI::URI(uri)),
                 Some(_) => return Err(Error::InvalidIssuer),
@@ -804,7 +808,7 @@ impl Credential {
             not_before: vc
                 .issuance_date
                 .as_ref()
-                .map(|date| date.date_time.timestamp()),
+                .map(|date| date.date_time.timestamp_nanos() as f64 / 1.0e9),
             jwt_id: vc.id.take().map(|id| id.into()),
             subject: Some(StringOrURI::String(subject_id)),
             verifiable_credential: Some(vc),
@@ -1902,14 +1939,18 @@ fn jwt_matches(
         assert_local!(allowed_vms.contains(kid));
     }
     if let Some(nbf) = claims.not_before {
-        if let Some(time) = Utc.timestamp_opt(nbf, 0).latest() {
+        let nbf_secs = nbf.floor() as i64;
+        let nbf_nsecs = ((nbf - nbf.floor()) * 1.0e9) as u32;
+        if let Some(time) = Utc.timestamp_opt(nbf_secs, nbf_nsecs).latest() {
             assert_local!(created.unwrap_or_else(Utc::now) >= time);
         } else {
             return false;
         }
     }
     if let Some(exp) = claims.expiration_time {
-        if let Some(time) = Utc.timestamp_opt(exp, 0).earliest() {
+        let exp_secs = exp.floor() as i64;
+        let exp_nsecs = ((exp - exp.floor()) * 1.0e9) as u32;
+        if let Some(time) = Utc.timestamp_opt(exp_secs, exp_nsecs).earliest() {
             assert_local!(Utc::now() < time);
         } else {
             return false;

@vdods yes, looks like a bug that it was not allowing fractional seconds. The changes look good. But I don't think we can maintain nanosecond precision with UNIX timestamps in f64. Would millisecond precision be sufficient for your purposes?

Yeah, for my particular use case, it would.  It also looks like microsecond precision with f64 will be possible until about the year 2255.  The calculation I used was 53 bits of mantissa in f64 gives 2^53 =<PHONE_NUMBER>740992.0, then divide by the number of microseconds in a year (60.060.024365.251000000), giving approx 285.  Add that to 1970 (the beginning of the Unix epoch) and you get 2255.  So given that the lifespan of any software is almost certainly less than what would carry it to the year 2255, it seems safe to use microseconds.  What do you think?
I'll give the implementation another go using microseconds (and maybe also milliseconds).  I think the criteria here is to exactly roundtrip the transformation between f64-valued Unix epoch seconds and DateTime<Utc>.

Here's a PR for this: https://github.com/spruceid/ssi/pull/315

Closing since #315 was merged; thanks @vdods.


Diagram Typo, ExpressRoute (ExressRoute).

The diagram has typo for ExpressRoute (ExressRoute).

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

ID: 251f9147-c9e9-a17a-3316-0f8a3a3d6f3e
Version Independent ID: e0714fbd-98f1-d125-f565-a68871b186d0
Content: Configure ExpressRoute encryption: IPsec over ExpressRoute for Azure Virtual WAN
Content Source: articles/virtual-wan/vpn-over-expressroute.md
Service: virtual-wan
GitHub Login: @cherylmc
Microsoft Alias: cherylmc

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

Thank you for you dedication to our documentation.
Unfortunately, we have been unable to review this issue in a timely manner. We sincerely apologize for the delayed response. We are closing this issue. If you feel that the problem persists, please respond to this issue with additional information.
Please continue to provide feedback about the documentation. We appreciate your contributions to our community.
#please-close


[Port v2int 7.4] Follow up changes to GC sweep op that are needed to enable sweep

This PR ports the following changes to internal-8.0 branch. internal-8.0 contains changes that will help enable sweep and these are follow ups that did not make it to the branch.

Initialize GC state as soon as possible and update it on (re)connecti… · microsoft/FluidFramework@40ad2d4 (github.com)
GC: Fix GC deleted stats not correctly updated for data stores by agarwal-navin · Pull Request #18890 · microsoft/FluidFramework (github.com)
GC: Added whether tombstone is enabled to GarbageCollection_end event by agarwal-navin · Pull Request #18927 · microsoft/FluidFramework (github.com)

Let's update the description to say 7.4 instead of 8.0 just in case someone runs into this later.


Compare to NSG service tags

Behind the scenes, are the Firewall service tags using exactly the same IPs as the NSG service tags described at this link?
https://docs.microsoft.com/en-us/azure/virtual-network/security-overview
Just want to confirm that if there is guidance such as "allow X service tag outbound", that would apply equally to NSG and Firewall.

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

ID: 47182f22-284b-7a8b-8a18-64bc38051ce6
Version Independent ID: 54e00bc3-3ba5-bcf8-b2e2-efb98d62acb6
Content: Overview of Azure Firewall service tags
Content Source: articles/firewall/service-tags.md
Service: firewall
GitHub Login: @vhorne
Microsoft Alias: victorh

@dcbrown16 , We are looking into this and will update you shortly.

@dcbrown16 , Both Firewall service tags and NSG services tags will be using same IPs in the backend for those services which are common between them except for (virtual network,azure loadbalancer and internet) as azure firewall doesn't support these technically.
We are closing this issue for now.If there are further questions regarding this matter, please reply and we will gladly continue the discussion.


fix overlapped issue of create Iteration modal and workItem detail.

Reduce the z-index of detail panel.
Fix: #810

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Seems like a fix for now, but we'll hit a point where unknown order/precedence of z-index of multiple elements will cause regressions.
I suggest having a z-index order/precedence range guide of our UI, and get it reviewed by @fabric8io/uxd. Knowing that a modal can have 1000-1200 z-index, a floating button or something can have 500-600 z-index and a sliding sidebar can have 100-200 z-index etc (all these are for e.g. only, please don't nitpick on values) will make a huge difference on how many complicated regressions we will avoid in later stages of the app.

@debloper yes, I agree with you. Need a feedback from @fabric8io/uxd.

@vikram-raj please create an issue on https://github.com/fabric8io/fabric8-ux for this (which element would sit on top of which)?

Questions:

why does the panel have a z-index of 100? Why so high? Also, I was unable to find a z-index for any other element - modal, navigation, or anything else.
Do other elements already have z-indexes?
Is Bootstrap / PatternFly being used to control the layering, as it should? Those frameworks cover layering for modals, slideouts, and other aspects. Custom code should not be there for that. @michaelkleinhenz, @michaelvirgil - how much of Bootstrap/PatternFly is being used at this point?

We will most likely not be able to give you a z-index for every single element - that is what the embedded frameworks are for. The CSS for each framework will properly handle the button layering, modal layering, etc.

z-index property only affects elements that have position value other than static. And modal have z-index 1050. which is set by bootstrap and to make sure it will be always on top.

Not all elements have z-index property. Z-index property set to only those elements which have position fixed or absolute property.

Bootstrap / Patternfly control the layering by giving z-index property.

@mindreeper2420  Here we just wanted to know the order of layering of elements. (modal, detail page, kabal menu.)

@vikram-raj - As you stated, Bootstrap / PatternFly control the layering by giving the z-index property. For anything not covered by either of those libraries, please use what makes logical sense - i.e. the kabob should not appear above the detail panel.

Ok @mindreeper2420. Thanks.

[test]


[redis_proxy] Assert proper transaction initialization

Fixes #37825
Making sure we know early that transaction initialization was not done correctly.
Tested manually, was unable to reproduce issue again. I'm guessing the assertion caused compiler to not do bad optimization there 🤷

Sorry this doesn't make sense. We need to better understand the problem. Are you sure this path is properly covered in test and by ASAN/TSAN?
Also use the Envoy ASSERT macro which is not compiled on release builds, which if this is an optimization issue (very doubtful) won't help.
/wait

Oh yeah it doesn't make any sense 😂 Just created PR to have the CI build to see if it wasn't just local the issue (if the assertion would fire).
I'll keep investigating. Maybe it was something transient on that specific build.
Will try tomorrow latest main, to see if the issue is there or not. If not, I'll abandon this PR.

Closing as not able to reproduce with latest binaries. Will reopen investigation if we notice this again 😃


Use borderless window during non-active

Use borderless window to display idol more clearly.

Note: This PR contains #6 to avoid conflict.
Scerenshots

Active
Non-active

How about 436ea7caebc9fe844d4a5cadd2e38894f2b9e302 that avoids position change on active/deactive?

How about 436ea7c that avoids position change on active/deactive?

good!

Nice blue sky.


bug fix dfget client timeout set error when fileLength is 0

Ⅰ. Describe what this PR did
bug fix dfget client timeout set error when fileLength is 0
Ⅱ. Does this pull request fix one issue?

bug fix dfget client timeout set error when fileLength is 0
Ⅲ. Why don't you add test cases (unit test/integration test)? (你真的觉得不需要加测试吗？)
Ⅳ. Describe how to verify it
use dfget and set timeout
Ⅴ. Special notes for reviews

Thanks for your contribution, Please help to sign https://github.com/dragonflyoss/Dragonfly/pull/1032/checks?check_run_id=277021510. @ww3122000

Codecov Report

Merging #1032 into master will decrease coverage by 0.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #1032      +/-   ##
==========================================
- Coverage   46.64%   46.62%   -0.02%
==========================================
  Files         115      115
  Lines        7002     7002
==========================================
- Hits         3266     3265       -1
  Misses       3481     3481
- Partials      255      256       +1

Impacted Files
Coverage Δ

dfget/core/core.go
45.83% <0%> (ø)
:arrow_up:

supernode/daemon/mgr/scheduler/manager.go
21.91% <0%> (-0.69%)
:arrow_down:

Continue to review full report at Codecov.

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

In addition, if you need to real-time chatting, please see https://github.com/dragonflyoss/Dragonfly/issues/291 to join us.

LGTM

LGTM.


Compiler in watch mode consumes 5% CPU with no activity

OS: Windows 1809
TSC version: 3.1.4
To reproduce:

build VS Code from source
execute npm run strict-null-check-watch
check the corresponding node process when compiler is waiting for file changes. It consumes at minimum 5% CPU.

@dbaeumer After looking at this, it doesn't seem like any of the file watchers or directory watchers are invoked unexpectedly which could cause the CPU usage.
Intead it looks like the cause of this is too many files in the program are being watched. File watching by default is always static polling which is accurate (and needed when watching changes in files in the program) but watching it without polling is inaccurate but efficient since it uses file system events.
So we have environment options to configure this behavior...  I tried using DynamicPriorityPolling which gives more accurate result without impacting cpu as much (I see the cpu usage is 0.3-0.4 as against 7-8% on my windows machine). Here is the link to details about these options.

@sheetalkamat thanks for getting back.
Wouldn't under Windows this be the best setting: UseFsEventsWithFallbackDynamicPolling or did you experience any problems with this.

Wouldn't under Windows this be the best setting: UseFsEventsWithFallbackDynamicPolling or did you experience any problems with this.
Yes on windows that works well but I wasn't sure if you were using windows. So from all platform perspectives DynamicPriorityPolling works well.

Thanks. I will use UseFsEventsWithFallbackDynamicPolling then.


--obfuscate should obfuscate AFTER reduction (as an option)

it would be nice to option to combine --obfuscate and reduction to make reductions more unique across problems.
Obfuscate would be run on the reduced output only (for speed), and would be deterministic, so different people running into same compiler bug from their own different programs would be more likely to end up with word-for-word identical reductions, make it especially useful to find your bug in bugzilla.
This could even be a default.
Furthermore, it would be nice if dustmite attempted to reduct strings to "" (again for generating as identical as possible outputs)

Um.

If you want to reduce and then obfuscate, you can write a batch file to do that. A full reduction is usually a several step process involving human assistance. I don't see the point in making reduction part of obfuscation.

Unless you use the --keeplength switch, reductions are already deterministic.

I'll look into splitting up strings.

Maybe I didn't explain my motivation properly:
the point of using obfuscation here was to generate outputs that are more likely to be identical for a class of compiler bug regardless of the details of the input file such as variable/function names:
right now when i reduce a bug i get:
void runInference(int num_steps){ badcode("/tmp/"); };
but another person may get a different output module variable renaming and other factors (ordering etc).
With obfuscation, there's some chance everyone would end up with the same output:
void A(int B){ badcode(""); };
I know it's a small thing, but it does make bug sorting/merging easier.

Honestly, I think that it's exceedingly unlikely to get the same reduction result from different codebases even with obfuscation. Almost always there are too many ways to write the same code, as far as syntax and semantics are concerned.

bug sorting/merging

A somewhat related project of mine I haven't really announced anywhere:
https://github.com/CyberShadow/DBugTests

Unless you use the --keeplength switch, obfuscations are already deterministic.

Erm, they're deterministic in all cases - I use a RNG with a default seed for --keeplength.

A somewhat related project of mine I haven't really announced anywhere:

That's really cool. I wonder why you haven't announced it yet.
Automation is really the way to go indeed.
I still think maximal reductions are likely to create identical outputs automatically in a number of cases, with each of those heuristics being attempted:

reformatting
renaming via obfuscate
removing comments
reducing strings to ""
removing ith argument to a function (is that easy across modules? within a module?)
harder: pulling in automatically import dependencies into the working set so they can be reduced too (eg std.range etc). That could be doable since dmd spits -v out the file for each module

That's really cool. I wonder why you haven't announced it yet.

Mainly because I don't really expect that anyone is going to use it, or help me improve it.

I still think maximal reductions are likely to create identical outputs automatically in a number of cases, with each of those heuristics being attempted:

I think this is a task for a separate project. But I've reduced a lot of test cases, and I'm skeptical of the idea. In many cases reductions that you may think are minimal really aren't, but you need insight in compiler internals to reduce further (because at some point you need to add or replace code in order to remove other code).

I'll look into splitting up strings.

Dustmite currently treats strings as a single token. Since reducing a string to an empty one is very unlikely to allow a reduction to occur elsewhere, I think there is little value in refactoring DustMite to allow this type of reduction, so it would have to be done by the above-mentioned separate project.


Update READme

fix grammar

@BaoTran91 - can you review the content again with the latest changes, and send in a commit if you still find some improvements can be done.

The changes were merged as part of:
https://github.com/openebs/jiva/pull/20
https://github.com/openebs/jiva/pull/19


new: switch website to Falco 0.28 version

Signed-off-by: Leonardo Di Donato<EMAIL_ADDRESS>
What type of PR is this?
/kind content
Any specific area of the project related to this PR?
/area documentation
What this PR does / why we need it:
It switches the Falco website to Falco 0.28.0 version.
Which issue(s) this PR fixes:
NONE
Special notes for your reviewer:
/hold

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please ask for approval from leodido after the PR has been reviewed.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 8d2f4967f519cf9065fd6441ee2df720378cfbdc

/hold cancel

/lgtm


Addition link

https://web.page-support.account.verification.bangtuanga.org/?sgn_re   |  ip : <IP_ADDRESS>
https://siddqpoisntislqpsoa.ddnsking.com/?bibimemek_   | ip : <IP_ADDRESS>
https://akuhsapplckssuwipos.ddnsking.com/?bibimmk_  | ip : <IP_ADDRESS>
will add more link if found it suspicious.

Will be added on next commit.


Endpoint path incorrect for router group index routes

When using a router group in combination with index routes, the endpoint path for those requests incorrectly (IMO) has a trailing slash:
let group = router.group("/users")
group.get(use: list)

func list(request: HBRequest) {
  // here I'd expect request.endpointPath to be "/users", but it is "/users/"
}

This might be a regression from https://github.com/hummingbird-project/hummingbird/pull/164.

So should group.get("/", use: list) have a trailing slash, even though the router would treat them as the same route.

Good point 🤔 My reasoning for expecting /users is that if I wouldn't use a router group, I'd write router.get("/users"), not router.get("/users/"), resulting in /users and not /users/.
In the case you shared I'd probably still expect /users, but that's even more opinionated I suppose.
If we say group.get("/", use: list) shouldn't have the trailing slash it should be consistent though and all trailing slashes in routes should be stripped, except for the case of the root index route which shouldn't be empty.

#167 consistency is a good thing


Invalid Transaction when using Bank to make payment

Please all i get is Invalid Transaction when trying to make payment using bank below is my function. what could be the problem ?
processPaystackPayment({required skKey, context}) async {
Charge charge = Charge()
  ..amount = (widget.paymentAmount * 100)
  ..reference = _getReference()
  ..accessCode = await _paymentRepository.createAccessCode(
      _getReference(),
      widget.paymentAmount * 100,
      userDetails.emailAddress.toString(),
      skKey)
  ..email = userDetails.emailAddress.toString();

CheckoutResponse response = await plugin.checkout(context,
    method:
        CheckoutMethod.selectable,
    charge: charge,
    fullscreen: true);

print(response);

if (response.status == true) {
  var res = await _paymentRepository.verifyOnServer(
      response.reference.toString(), skKey);

  if (res == 'success') {
    var obj = widget.bookingModel;
    obj.paymentReference = response.reference.toString();
    transacRefernceId = response.reference.toString();
    log(jsonEncode(obj));

    ref.read(bookingFutureProvider.notifier).initiateReservation(obj);
  }
} else {
  if (kDebugMode) {
    print('failed here');
  }
  Navigator.of(context).pop();
}

}

Please can someone answer me Please i running out of time.

have you found solution ?

have you found solution ?

Nope, what I did was, I just made the app process card payment instead.


GetBoardHistorySince returns broken BoardHistoryEvents

GetBoardHistorySince returns BoardHistoryEvent with EventDateTime always set to datetime min value
EventDateTime is missing the time part makign impossible to know when it happen or even see them in order.
See example below
Important ids haev been changed to 9999's for security reasons
{
"ReplyData": [
[
{
"CardId": 9999,
"EventType": "CardMoveEvent",
"EventDateTime": "20/07/2015",
"Message": "Daniel Portella moved the Card [(S95) Necromancer Ticket DO NOT DELETE] from Release: Ready to Deploy to Release: Pre-Deploy Huddle.",
"ToLaneId": 9999,
"FromLaneId": 9999,
"RequiresBoardRefresh": false,
"IsBlocked": false,
"BlockedComment": null,
"UserId": 0,
"AssignedUserId": 0,
"IsUnassigning": false,
"CommentText": null,
"WipOverrideComment": null,
"WipOverrideLane": 0,
"WipOverrideUser": 0,
"TaskboardParentCardId": 0,
"TaskboardId": 0
}
]
],
"ReplyCode": 200,
"ReplyText": "Board history since last update successfully retrieved."
}

GetBoardHistorySince only returns the most recent 5 events since the given board version number, and is therefor not a very reliable way to keep track of all changes that have occurred to a board since the last known version number. CheckForUpdates will return all the events that have occurred, including date + time of those events.
Hope this helps!

ok but shouldnt it return the time too - consistency is important.
but i am glad i will check the CheckForUpdates method and see what it returns.

Yes, I agree it should return a date/time for consistency. I also think GetBoardHistorySince is not very useful ;)
FYI, I noticed there was an issue parsing dates of events. I've added an overload for the Create() method so you can specify a date format (e.g. "dd/MM/yyyy", "yyyy/MM/dd", or "MM/dd/yyyy"). For example:
var api = new LeanKitClientFactory().Create(leanKitAuth, "dd/MM/yyyy");

Yeah i was thinking that but didnt want to say anything :) GetBoardHistorySince isnt really useful.
Ok the date on events i noticed it too the json returns the right date but the object has a default date time {01/01/0001 00:00:00} will your fix resolve that?

Yes, the latest version should fix the issue where the date/times returned in JSON formatted to the current user's date format were not getting parsed correctly. You will need to specify your date format when creating the API client.

yep I added it and all is working now. I will close this ticket


The reason using reimplemented Euler to Quaternion instead of using scipy

Hi KosukeFukazawa, great work, it helps me a lot!
I'm quit new to the 3D modeling field, and I'm curios why using the reimplemented approach to convert Euler angles into quaternion (rots = quat.from_axis_angle(rots))? I use scipy's Rotation ( scipy.spatial.transform.Rotation.from_euler().as_quat() ) and yields different result.
And also, in the following code, why convert Euler angles to quaternion and then convert back to Euler angles, instead of directly convert raw Euler angles to degree?
`
# to quaternion
rots = quat.from_axis_angle(rots)
order = "zyx"
pos = offsets[None].repeat(len(rots), axis=0)
positions = pos.copy()
positions[:,0] += trans * 100
rotations = np.degrees(quat.to_euler(rots, order=order))

`
Thanks!

Hi hallazie,
thank you for your interest!
SMPL uses the axis-angle representation (available in scipy as rotvec: https://docs.scipy.org/doc/scipy/reference/generated/scipy.spatial.transform.Rotation.from_rotvec.html) instead of Euler angles.
I use quaternion as an intermediate representation because it can prevent gimbal locks, which is useful when adding processing to compensate for motion. (additionally, scipy.spatial.transform.Rotation class does not support arrays with dimensions greater than 3, so I implemented a separate solution.)
I think that using scipy to change SMPL's axis-angle representation directly to bvh's Euler angles will give similar results.

Hi hallazie, thank you for your interest!
SMPL uses the axis-angle representation (available in scipy as rotvec: https://docs.scipy.org/doc/scipy/reference/generated/scipy.spatial.transform.Rotation.from_rotvec.html) instead of Euler angles. I use quaternion as an intermediate representation because it can prevent gimbal locks, which is useful when adding processing to compensate for motion. (additionally, scipy.spatial.transform.Rotation class does not support arrays with dimensions greater than 3, so I implemented a separate solution.) I think that using scipy to change SMPL's axis-angle representation directly to bvh's Euler angles will give similar results.

Thanks for your reply, that was really helpful, I misunderstood the SMPL default rotation format. Thanks!


chore: introducing bulkRead API with veneer client

Towards #2454
This is an attempt to maintain the same behavior of existing BulkRead with veneer bulk read API.
The Veneer's Batcher API does not accept filter per request whereas existing BulkRead accepts a filter and sends entries with the same filters in batches.

@rahulKQL take a look at this rough sketch:
https://github.com/igorbernstein2/cloud-bigtable-client/commit/6e4927a2216737655d2ea8da5b909d1b788c7440
I'm


fix travis configuration

anchorj dependency is only view github or local mvn repo available.
is it possible to configure travis to build anchorj and extensions from source?

Snapshot is on mvn central. Ok for you?


Other Storage Options

It would be great if we could configure this to use a different storage mechanism that I can run within the network, rather than a cloud hosted one.  Many organizations have policies against "unapproved" cloud solutions.

This would be nice but right now the app is totally reliable on firebase. In order to accomplish other storage options we would have to basically rewrite the whole app from scratch. We would have to implement also real time behaviour using socket.io or something similar because firebase gives you that for free.
One option would be start a fork that implements another storage like http://feathersjs.com/, http://gun.js.org/ or http://parse.com (that now it's open source).

It would be helpful to abstract all database calls so that we could add our own custom data stores for implementations.  Personally, I would probably go the DynamoDB route for what I am working on: http://docs.aws.amazon.com/AWSJavaScriptSDK/latest/AWS/DynamoDB.html

FeathersJS sounds promising. Have you tried plugging it in yet?

@chapmanjw I didn't try to change the storage yet. And I am not sure I will have time for that in the near future. But if anyone else wants to try it I would really appreciate it.

@glauberramos what Firebase alternative do you think would be the easiest to integrate into Fun Retro right now? Would the whole app actually need to be rewritten? I only see Firebase references here - https://github.com/funretro/distributed/blob/master/js/services/firebaseService.js .

I think a lot of code would have to be changed in order to support a new integration. Do you have anything in mind @obv-mikhail ? Maybe not a lot, but's it's not very easy task.

I think it's fine if it's centralised, it would be nice to have an option where people can install the database locally instead of using on the cloud like firebase.

@obv-mikhail check out my fork
https://github.com/waninlezu/distributed
http://waninlezu.github.io/distributed

@waninlezu nice! I was able to port funretro to use Gun like I mentioned in the previous comment, and got authentication working through GitHub. The problem I faced with Gun was with setting up ACLs of any kind. ACLs should probably be simpler with feathers.


Create component tree using model method.

Part of fix for OLCS-447

The update of "ProductTreeNode" is not included in the MR, but should be similar.

Some of the changes e.g., update_or_create_component is used in https://github.com/RedHatProductSecurity/openlcs/pull/136 . You may need to resolve conflicts.

Updated

@RichardXuan btw, one pull could have multiple commits. You could commit your own changes and push to this PR, so that it's easier to track newly added changes from log.

@rhyw Ok, got it

LGTM


Crash after starting organization in v1.8.0

v1.8.0 is crashing for me after starting organization. NSX, XCI files are on a SMB Share:

{
"versions_json_url": "https://raw.githubusercontent.com/blawar/titledb/master/versions.json",
"versions_etag": "W/"132bbcb0ae6ffaefdd06137e54b69df68a5cf781fa0794f46d2c7f28fcdfda5f"",
"titles_json_url": "https://tinfoil.media/repo/db/titles.json",
"titles_etag": "W/"7e4bc4ad2fdb1:0"",
"prod_keys": "",
"folder": "\\smb\path\",
"scan_folders": [],
"gui": true,
"debug": false,
"check_for_missing_updates": true,
"check_for_missing_dlc": true,
"hide_missing_games": true,
"hide_demo_games": true,
"organize_options": {
"create_folder_per_game": false,
"dlc_folder": "",
"updates_folder": "",
"rename_files": true,
"delete_empty_folders": false,
"delete_old_update_files": true,
"folder_name_template": "{TITLE_NAME}",
"switch_safe_file_names": true,
"file_name_template": "{TITLE_NAME} ({DLC_NAME})[{TITLE_ID}][v{VERSION}]",
"process_when_missing_base_game": false
},
"scan_recursively": true,
"gui_page_size": 100,
"ignore_dlc_updates": false,
"ignore_dlc_title_ids": [],
"ignore_update_title_ids": [],
"ignore_file_types": []
}

If starting from cmd:

as you seem to use a smb share it might share the root cause with #61

Tested with a windows mapped SMB share, same issue.

i'm using the same and i'm not able to reproduce this error, so its a bit hard for me to debug currently, but will see if i can find anything

Are you using the same settings as I am using in the json?

will try and see if it makes a difference

if you know your way around you could try and see if you can find anything, otherwise i dont really know for now, i will try a bit more to see if i can recreate the problem

I am having the very same issue...it will run briefly and then just crash. Same settings as OP...using a mapped "X" drive to network address.

thanks for reporting back and helping with this, i'm still unable to recreate this problem.
with my own digging it doesnt seem be related to a SMB share, but rather a memory leak problem
so next step i guess will be to start going through each changed row between 1.7 and 1.8
but how large are you libraries if you are willing to share? ex: X files and X gb

@xadox-1st does your library work without this file otherwise?
if that is the cause, i think i know what is broken and that is that the change between 1.7 and 1.8 broke homebrew games that does not exist in the titles.json
will work on a fix for that asap

restored support for homebrew files, so hopefully this build should work now: https://github.com/trembon/switch-library-manager/actions/runs/11162189381

That fixed it for me too! Awesome! Thanks for the quick work!

fix is included in the 1.9.0 release


MAINT: Fix for pydata-sphinx-theme

Closes #10598
Fix for backward incompat change https://github.com/pydata/pydata-sphinx-theme/pull/604 which fails silently and gives bad URLs

Hah, https://nodejs.org/download/release/v16.13.2 is down with a 500 error. I guess we'll try later!

All better! https://output.circle-artifacts.com/output/job/aea5ead8-6ef0-4364-812c-c3a09a62d850/artifacts/0/dev/index.html

@larsoner I don't think this one has been backported, should we do it?

All of the links seem to work in https://mne.tools/stable/ so I'm not sure we need to

... but if we continue backporting things and want to push a website update, we might need to. Not sure

Let's just drop this then :)


need help --- Import not working in pyscript via HACS on HA

Hi,
I'm struggling to get import working in pyscript via HACS on HA 2024.7.1.
any python file without an import works fine and I can run as service and show up in dev tools
then i simply add import requests for example I get this:
Exception in </config/pyscript/hello_world.py> line 2: i ^ SyntaxError: invalid syntax (hello_world.py, line 2)
Exception in </config/pyscript/hello_world.py> line 2: import requests ^ SyntaxError: invalid syntax (hello_world.py, line 2)
I tried to follow this guide by adding removing from yaml
https://hacs-pyscript.readthedocs.io/en/stable/reference.html#importing
my yaml config
pyscript:
allow_all_imports: true
hass_is_global: true
the doc above says this yaml will be ignored if GUI was used to install so I'm not sure what my settings need to be
my hello world.py file:
@service
import requests << won't load thus don't show up under dev tools and errors in log as provided
def hello_world():
thanks anyone in advance for any help to get this working
lastly when I listed all in requirment doc  logs stated  requests was already installed and ignoring
2024-08-08 16:49:06.244 DEBUG (MainThread) [custom_components.pyscript] Skipping unpinned version of package 'requests' because version '2.32.3' is already installed

also i now see this
Logger: homeassistant.helpers.frame
Source: helpers/frame.py:184
First occurred: 8:23:19 PM (4 occurrences)
Last logged: 8:23:46 PM
Detected that custom integration 'hacs' calls hass.http.register_static_path which is deprecated because it does blocking I/O in the event loop, instead call await hass.http.async_register_static_paths([StaticPathConfig("/hacsfiles/frontend", "/config/custom_components/hacs/hacs_frontend_experimental", False)]); This function will be removed in 2025.7 at custom_components/hacs/frontend.py, line 49: hass.http.register_static_path(, please create a bug report at https://github.com/hacs/integration/issues
Detected that custom integration 'hacs' calls hass.http.register_static_path which is deprecated because it does blocking I/O in the event loop, instead call await hass.http.async_register_static_paths([StaticPathConfig("/hacsfiles/iconset.js", "/config/custom_components/hacs/iconset.js", True)]); This function will be removed in 2025.7 at custom_components/hacs/frontend.py, line 57: hass.http.register_static_path(, please create a bug report at https://github.com/hacs/integration/issues
Detected that custom integration 'hacs' accesses hass.components.frontend. This is deprecated and will stop working in Home Assistant 2024.9, it should be updated to import functions used from frontend directly at custom_components/hacs/frontend.py, line 68: hass.components.frontend.async_register_built_in_panel(, please create a bug report at https://github.com/hacs/integration/issues
Detected that custom integration 'hacs' accesses hass.helpers.event. This is deprecated and will stop working in Home Assistant 2024.11, it should be updated to import functions used from event directly at custom_components/hacs/base.py, line 642: self.hass.helpers.event.async_track_time_interval(, please create a bug report at https://github.com/hacs/integration/issues

import requests

@service
def hello_workd():
    pass

@ALERTua
@ALERTua  thank you so much !!!   that helped having import above @service thanks   gosh I feel stupid!!
script now shows up in dev tools!

please close this issue if you have no further questions :)

again thanks for the help .. I am having an issue seeing anything in logs but I keep at it
I now  have this:
import requests
@service
def hello_world():
testing requests
url = "https://raw.githubusercontent.com/custom-components/pyscript/master/README.md"
resp = task.executor(requests.get, url)
log.info(f"Hello World")
log.info(resp.text)

closing a simple reboot of HA resolved @ALERTua thanks again

thanks


Regression: environment variables can no longer contain a dot

This issue is a: (mark with an x)

[ ] bug report -> please search issues before submitting
[ ] documentation issue or request
[x] regression (a behavior that used to work and stopped in a new release)

Issue description
We have an environment variable as such:
SolutionName.Feature however, the dot . is no longer allowed (last week it still was allowed).
Steps to reproduce

Set environment variable with dot

Expected behavior [What you expected to happen.]
The dot should be allowed. The allowed character range should be less strict.
Actual behavior [What actually happened.]
Error:
Env variable name 'SolutionName.Feature' contains invalid character, allowed characters are 'a-z, A-Z, 0-9 and _'

Additional context
Ex. Did this issue occur in the CLI or the Portal?
Both. Even our CI/CD deployments broke because of this change.

We are seeing the same problem. Our last CD run at 7/8/2022 8:08:00 PM UTC worked fine, but the next run at 7/12/2022 4:35:00 PM UTC failed so the change happened somewhere between those two timestamps.

Same here, my CD failed as well. I store my database connection string in an Azure secret that gets injected into the Container. I haven't deployed in several days, and I have no changes that would upset the Azure deployment process.

Can confirm, environment variables containing a dot are working again. Thank you!

I know its an old issue, also closed. but @mjqin I am facing this issue on container deployment. Did something change?


Malfunction: Export, save functions  (to  .xml) Windows 10 V0.35 Roster(opta)planner

Export, save functie (to .xml) at Windows10 computer:
-> Rosterplanner message: "Uncaught exception" Thread "AWT-EventQueue-0" java lang illegalArgumentException Could not writethe file  ...xml.

Finally figured out the solution for the export/print errors before solving, will be fully implemented tomorrow

Just committed solution


Bumping version

<!-- Make sure you have read http://napalm.readthedocs.io/en/latest/contributing/index.html --!>

Changes Unknown when pulling 948bb71ff8f14f2bec4b3d67fe255ab0a0b91f1c on GGabriele:bump into ** on napalm-automation:develop**.


MecanumDrive odometry and tf functions implementation

Signed-off-by: danilo_gsch<EMAIL_ADDRESS>Danilo Schneider
🎉 New feature
Closes #683 #1619
Summary
Implementation of Mecanum drive odometry and tf based on the implementation of the Differential robot.
Test it
Run mecanum_drive.sdf and listen for odometry and tf topics.
Checklist

[ ] Signed all commits for DCO
[ ] Added tests
[ ] Added example and/or tutorial
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if the library has them)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

Depends on #486

Waiting on https://github.com/gazebosim/gz-math/pull/486

@danilogsch if you can address feedback on the math pull request, I can begin reviewing this one.

@danilogsch the math PR has been landed, can you add a test here to verify the behavior of the system?

I've removed the beta label. We can merge this after Harmonic is released.

Hi all, any information regarding when this PR will be included?
I see some failures in the automatic build, is this due to the PR or still the math dependency ?
In case I can help.
Thanks

I didn't realize https://github.com/gazebosim/gz-sim/pull/2297 replaced this. Closing.


License

Is there a reason GPL license was used for this repo? It would be better to have MIT, if you don't have a reason for using GPL over MIT. The latter is better, as it enables people to use whichever license they want, if they wish to use your code in other projects. MIT enables others to use their code as they wish, as long as they give credit.

I chose to remove all code related to this project (in my own separate repo), to avoid running into this GPL-virus nonsense.
I would recommend you switching to MIT, if you don't have a good reason for having GPL.

Just observed that you have changed to MIT license. Thats great! :D


Update --scripts tag to run only associated scripts

Instead of "adapting" the tests to whatever now fits their behaviour, make them relevant again.


Shellcheck, Enhancements, Fixes

🤔 Check before merge:

[x] 🏗 All functions implemented thoroughly

[x] Fix unintentional multiline device picker
[x] Fix "useless cat" lint warnings
[x] Fix all lint issues (or ignore those which arent easy to resolve) ¨
[x] ainstall without any device connected

[x] 🔨 All changes tested

[x] aanimationspeed (on & off)
[x] aanimationspeed <num>
[x] afontscale (on & off)
[x] afontscale <num>
[x] abounds (on & off)
[x] ascreenshot
[x] iscreenshot
[x] ...dozens of other scripts and their variants...

[x] 📝 Changes in usage / new scripts added to README.md
[x] 📣 Major changes listed in changelog.txt

✅ All good? -> Merge! 🎉🎉🎉

Let's be brave. 🤞


Avoid collisions between essential packages

Motivation for this change
Fix remaining collision between essential packages.
See issue #55886.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS)
Built on platform(s)

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

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

The priority thing is really not optimal... We should have relative priorities between packages.

On 19.03, nix-build -E 'with (import <nixpkgs> {}); buildEnv { name = "test"; paths = [coreutils procps]; }' creates a environment where kill and uptime come from coreutils.

I'm assuming this PR retains this behavior?

Yes, it does. I opened this PR just to fix those annoying warnings, nothing more.
Anyway, I agree that it would make more sense for procps to override and coreutils.


translate="no" in Vue3 is not work

Version
3.2.31
Reproduction link
sfc.vuejs.org/
Steps to reproduce

npm init vue@latest
write code like this:

Hello Vue.js

Hello Vue.js

What is expected?
Expect something like this.

What is actually happening?
Actually:

This is because translate attribute exists as dom properties but must be set as a boolean. Maybe we should handle this special case for the "no" value.
For the moment you can either set boolean values or use the ^ in front of the attribute:

    Hello Vue.js

    Hello Vue.js

@posva can you explain more about the ^ in front of the attribute trick?

Same a .attr modifier for v-bind

@Rolanddoda  .attr - force a binding to be set as a DOM attribute.
https://vuejs.org/api/built-in-directives.html#v-bind


reverseDuration parameter doesn't make sense for implicit animations

Recently we added the reverseDuration parameter for animations. The thing is, it doesn't really make sense for Implicit animations, since there is no "reverse"... they just animate to whatever value was provided. I suggest we remove the parameter in this case.

cc @gspencergoog


GP ADMX file name has been changed

i have verified in policy definations folder , there is no file called  edgeupdate.admx
but  msedgeupdate.admx file is there
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 7c2c2341-3489-41ce-9223-71e27bf5d586
Version Independent ID: 599462c4-4725-a8b3-29e9-d790238db1bf
Content: Microsoft Edge Update Policy Documentation
Content Source: edgeenterprise/microsoft-edge-update-policies.md
Product: microsoft-edge
GitHub Login: @Brianalt-Msft
Microsoft Alias: stmoody

I think the article has updated with a changed name,
Please @Brianalt-Msft conform it .

I think the article has updated with a changed name,
Please @Brianalt-Msft conform it .

Hi @RAJU2529 where are you seeing this?  In the documentation or the policy archive?

@tamarahills , in the documentation only.  Name has been changed from edgeupdate.admx  to  msedgeupdate.admx.
article has updated.

@tamarahills , I am closing this issue, issue has been solved , hence closing.


Use updated ansible 2.4 inventory interface

Ansible devel (2.4) provides an updated implementation for inventory internals.  This change:

adds a 2.4 version check
sets the plugin variable_manager and adopts new constructor where applicable
sets the plugin inventory_manager to use the new InventoryManager where applicable.

The InventoryManager still provides list_hosts(), list_groups(), and subset and I have confirmed functionality w/ both dynamic and static inventory files.

thanks for the contribution!  I'm ignoring the test failures for the time being since they are often caused by upstream changes in core. Will research.


[Win32]: display.newText() , text rendering errors in RTL languages

As shown in the screenshot, there is a piece of Arabic text, and it can be clearly observed that the position where the number '3' appears is inaccurate.
Then I modified the logic here to determine whether the string contains RTL characters. If so, add the DT_RTLREADING flag to the flags variable and adjust the alignment attribute logic. It seems to work normally.
Judging whether a string is mainly RTL text seems to be a very troublesome thing. There are some very complicated implementations in the ICU library. Of course, I only did a simple process here, that is, as long as it contains RTL unicode characters, It is considered to be RTL text, but this does not seem to be a perfect approach.
Test text:
مرحبًا ، اسمي # name # ، عمري 3 سنوات هذا العام ، وأتيت إلى هذه الروضة للذهاب إلى المدرسة ، آمل أن تعطيني المزيد من النصائح!
Where to modify the code:
https://github.com/coronalabs/corona/blob/2c211d807a5ef039b6f885b2deb85f49b9a3f68e/platform/windows/Corona.Native.Library.Win32/Rtt/Rtt_WinBitmap.cpp#L670
ScreenShot:

Code modify screen Shot:

Judge RTL character code snippet
`
// aft_isrtl impl:
//1  https://stackoverflow.com/questions/5423960/how-can-i-recognize-rtl-strings-in-c
//2  https://www.appsloveworld.com/cplus/100/247/how-can-i-recognize-rtl-strings-in-c
bool ContainsRTLCharacters(const WinString& winStr)
{
	const wchar_t* str = winStr.GetUTF16();
	size_t len = wcslen(str);
	for (int i = 0; i < len; i++)
	{
		wchar_t c = str[i];
		if (aft_isrtl(c))
		{
			return true;
		}
	}
	return false;
}`

See https://github.com/coronalabs/corona/pull/579
But the suggestion this would not constant across platform
You can change text alignment already
https://docs.coronalabs.com/api/library/display/newText.html#align-optional


adding uniswapv2 token voting

Fixes # .
Changes proposed in this pull request:

@stalker474 Hi! I'm closing this PR because we have now a strategy that can do the same, have a look here on the examples query if you want to use it: https://github.com/snapshot-labs/snapshot.js/blob/master/src/strategies/uniswap/examples.json or feel free to ask in Discord or open an issue if you need more informations.


action-used-by-link - Drop deduplicate

Test URLs
https://github.com/marketplace/actions/title-replacer
Screenshot
https://github.com/user-attachments/assets/cd0cb81a-49de-4024-b5b3-e3999d244dd3

As mentioned in #7000:

There's no need to manually go through and change all the features, but they're worth keeping in mind if we're already working on a specific feature

Does this fix a specific bug? observe is always heavier than a plain $ so if the latter suffices we shouldn't change it.

You can see the bug in the left browser window. And it also relates to https://github.com/refined-github/refined-github/issues/5871


Prevent weird text boxes to appear in the chapter

Steps to reproduce

inject this js: https://cdn.discordapp.com/attachments/1055525753325883403/1141673728732827801/aaa.js
Go to AllNovelFull, The Incubus System, Glossary (Chapter 0).
open Chapter 0 (Glossary)

Expected behavior
Weird text box never appears
Actual behavior
https://github.com/LNReader/lnreader/assets/75091899/fb5389ff-e65b-4a00-876d-1cec09aca151
LNReader version
1.1.17
Android version
Android 8 miui 8
Device
Huawei p8 lite 2017
Other details
These text boxes appear rarely, but when they do they are a pain.
Maybe directly deleting them before before the JS is applied.
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with a source, I should be opening an issue in the sources repository.
[X] I have updated the app to version 1.1.17.
[X] I will fill out all of the requested information in this form.

There is no input present in the chapter's HTML and neither is the text box present in the chapter without applying any custom JS.
If this only happens after applying your custom JS then it's not an app issue.


'Application Development Engine with Epinio' Landing Page

SURE-6866
Request description:
Add landing page as top page in hierarchy for each 'Integrations' section.
Page should include:

brief description of integrated product/project

benefits of integrating product/project with Rancher

additional benefits received by Prime customers

Reopening as additional review/updates need to be made. Initial PR was merged so some stakeholders had access to a preview.


Replace OPA JWT example with long-lived token to fix #3579

Summary
Replace OPA JWT example with long-lived token to fix #3579
Reason
#3579
Testing
Complete the full Example in this docs page

There are broken links, but that's unrelated to this PR. Let's ship and fix the original issue and we can fix broken links afterwards


Add Sony DualShock 4 USB Wireless Adaptor

The Sony DualShock 4 USB Wireless Adaptor (part number CUH-ZWA1) allows connection of PS4 controllers to PCs wirelessly:
https://blog.playstation.com/archive/2016/08/22/introducing-the-dualshock-4-usb-wireless-adaptor/
Mapping created with SDL2 Gamepad Mapper.

Have you tested if the device works with SDL without this mapping ? 6800 tail indicates this mapping is not necessary: https://github.com/mdqinc/SDL_GameControllerDB/issues/478

Right you are, I wasn't testing properly. Thank you!


Activity Not Defined in the ChangeNotification Class (Issue Confirmed with MS Support)

Hello,
I'm using the following test app to retrieve presence:
https://docs.microsoft.com/en-us/samples/microsoftgraph/aspnetcore-webhooks-sample/microsoft-graph-change-notifications-sample-for-aspnet-core/
In the following function which receives the webhooks callbacks. It initially does the following code to retrieve the plain text notifications:
    // The notificationUrl endpoint that's registered with the webhook subscription.
    [HttpPost]
    [AuthorizeForScopes(ScopeKeySection = "SubscriptionSettings:Scope")]
    public async Task<IActionResult> Listen([FromQuery]string validationToken = null)
    {

        if (string.IsNullOrEmpty(validationToken))
        {
            try
            {

                // Parse the received notifications.
                var plainNotifications = new Dictionary<string, ChangeNotification>();
                var options = new JsonSerializerOptions
                {
                    PropertyNameCaseInsensitive = true
                };
                options.Converters.Add(new JsonStringEnumConverter(JsonNamingPolicy.CamelCase));
                var collection = await JsonSerializer.DeserializeAsync<ChangeNotificationCollection>(Request.Body, options);
                foreach (var notification in collection.Value.Where(x => x.EncryptedContent == null))
                {
                    SubscriptionStore subscription = subscriptionStore.GetSubscriptionInfo(notification.SubscriptionId.Value);

                    // Verify the current client state matches the one that was sent.
                    if (notification.ClientState == subscription.ClientState)
                    {
                        // Just keep the latest notification for each resource. No point pulling data more than once.
                        plainNotifications[notification.Resource] = notification;
                    }
                }

                if (plainNotifications.Count > 0)
                {
                    // Query for the changed messages.
                    await GetChangedMessagesAsync(plainNotifications.Values);
                }

The issue is that neither Activity nor Availability are defined in the ChangeNotification class and so the presence updates cannot be retrieved. They are definitely being received as they can be seen on the Ngrok interface.
Note that there is a workaround in that you can do the following (this was provided by MS support gclark@microsoft.com):
                string l_szReqBody = await new StreamReader(Request.Body).ReadToEndAsync();
                string l_szDisplay = "";
                dynamic data = JsonConvert.DeserializeObject(l_szReqBody);

                // Verify if the request body is not empty and has json data
                if (!string.IsNullOrEmpty(l_szReqBody))
                {
                    JObject body = JObject.Parse(l_szReqBody);
                    if (body.ContainsKey("value"))
                    {
                        //JValue resource = (JValue)body["value"][0]["resource"];
                        l_szDisplay = string.Concat("WEB HOOK:" + "ID=" + body["value"][0]["resourceData"]["id"] +
                            ", Activity=" + body["value"][0]["resourceData"]["activity"] +
                            ", Availability=" + body["value"][0]["resourceData"]["availability"]);
                    }
                }

Are there any plans to add Activity and Availability to the ChangeNotification class or should I stick with the workaround ?
Thanks
Ed James

Hi @edjames99
Thanks for reaching out. I'm not sure why you need a workaround to access the data? After token validation, symmetric key decryption, signature validation, and payload decryption, the full JSON representation of the data is available in the decryptedpublisherNotification string
Just adding var presence = JsonSerializer.Deserialize<Presence>(decryptedpublisherNotification, options); after that line will give you the deserialized payload using the types provided by the SDK.

It doesn't get that far as collection.ValidationTokens is null so the decryptedpublisherNotification string isn't populated.

Have you set IncludeResourceData to true in the configuration as well as created the key vault, granted permissions to it for your application, generated the self-signed certificate and updated the keyvault settings in the configuration file?

Geoff Clark from MS support said I didn't need IncludeResourceData to be true so I haven't created a key vault. I have granted delegate permissions to the app (as you can't get presence with application permissions). Geoff said the issue was that the Microsoft.Graph.ChangeNotification class doesn't contain Activity so it's not available.

change notifications do not contain resource data by default. They tell you "something has changed there", you then need to query the information, which you can see being done in GetChangedMessagesAsync (might need to be adapted in the case of Presence) or you need to request change notifications to include resource data so it gets delivered with the notification. (only teams presence and teams chat messages support it at this time)

OK thanks, I'll take a look tomorrow. Have you got any code that can request change notifications to include resource data ? How do I do that ?

This sample does that, as long as you configure it to do so.
As the question has been answered, I'll close this issue for now.

One more question, if I've already done the Azure registration without creating a key vault can I now go back and create one and re-save the same registration or do I need to do a fresh registration from scratch ?

no you can keep your current registration, just go through the steps you haven't been through yet

Thanks for your help, I appreciate it. I wonder if I could pick your brain about the best way forward for our production strength application. What we need ideally is an application which does not require any user input and which can retrieve presence data from potentially thousands of users.
So far I've discovered the following:

Presence is only available with delegated permissions and not with application permissions.
Delegated permissions requires a user to log in and in the example ASP.NET applications currently available this means manual user input.
A subscription supports 650 users max so we will need multiple simultaneous subscriptions.

When we did the Skype for Business integration we had a trusted application running as a windows service subscribing to presence states of thousands of users but it looks like we can't have presence via application permissions (is there a reason for this ?) so we have to have delegated permissions but we don't want manual user login so we have to find a way of doing the login to MS programmatically.
Do you have any example apps that retrieve presence and log in to MS without manual user input ? How difficult would it be to suppress the manual login in the ASP.NET application ?
Any comments or ideas would be welcome as we're currently not sure on the best way forward.

Hi, I'm trying to set up the Azure Key Vault as described on https://docs.microsoft.com/en-us/samples/microsoftgraph/aspnetcore-webhooks-sample/microsoft-graph-change-notifications-sample-for-aspnet-core/ but when I click through on the link it says 404 site not found.
Do you have a link to the steps to set up the key vault so I can see the presence coming through ?

without user input: this would require the API to support app only, and it doesn't today. You should upvote this idea to help prioritize the work for app only (and also be notified when it comes online).
Key vault: https://github.com/microsoftgraph/aspnetcore-webhooks-sample/blob/master/KEYVAULT.md

Thanks, I'm trying to add a resource group and it's saying we don't have a subscription yet the rest of the app registration works in the Azure Active Directory. So is the Azure Portal different and so we need a separate subscription to the portal to be able to create this key vault ?

you need an active azure subscription yes. It is possible to use the feature (change notifications with resource data) without using the Azure KeyVault by generating and storing the certificate somewhere else/yourself, but this sample doesn't show that.