BPF kernel probes based on bpftool

This PR implements BPF feature probes for the kernel config with bpftool. Based on #8094, but now I'm trying to submit in smaller pieces with hope that CI is not going to complain. Please review per commit.
BPF kernel probes based on bpftool

This change is

test-me-please

test-me-please

Coverage decreased (-0.02%) to 45.934% when pulling ed8aed156b9f4b981d300643b71bfd29deaedd88 on mrostecki:bpftool-probes-kernel into 7acf04658413d99947450803cf083f6611a4b8ca on cilium:master.

Coverage decreased (-0.005%) to 45.946% when pulling ed8aed156b9f4b981d300643b71bfd29deaedd88 on mrostecki:bpftool-probes-kernel into 7acf04658413d99947450803cf083f6611a4b8ca on cilium:master.

@borkmann @joestringer CI seems to be OK now. Can we possibly include that in 1.7? And should I try the other tests (missing-k8s, upstream-k8s)?

test-me-please


SVG question difficulty filter on question finder

This adds a question filter on question difficulty as requested here: https://docs.google.com/presentation/d/1zSgtpLJmDnwQ1LmMM5VpX3AprneaXvBv
As part of this I've refactored Hexagon.tsx for the interactive SVG shape logic to be more easily extended to new shapes.
Still to do:

[x] Add extra shapes to the difficulty filters to visually show difficulty levels
[ ] Add a summary of difficulties chosen at the top of the question finder (this wasn't on the spec so I've left it for now)
[x] Add SVG shapes to question entries on the finder to indicate difficulty

Superseded by https://github.com/isaacphysics/isaac-react-app/pull/448.


reset the http status code to 500 in case we encounter an error

Reason for Change:

During NC publish and unpublish, we do not reset the httpStatusCode that we return back to DNC. As a result, even if we encounter an error we send back HTTP 200 back to DNC
Issue Fixed:

https://msazure.visualstudio.com/DefaultCollection/One/_workitems/edit/17035213
Requirements:

[x] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests

Notes:

Since we do not want CNS to make any decisions and just pass the encoded response from NMA to DNC, we should not handle this bug in the manner this PR does. Closing this PR. It will be handled here: #1799


Style updates to accommodate addition of swiper \w refactoring

We've added the logic for Swiper use in #12   but we still need to integrate it visually into the site. I believe some initial refactoring will make this easier + make the layout pattern easier to reuse in the future.
Its current appearance is viewable here.

[x] Refactoring

Look at moving CardContent from ShowCards into its own component.
What's left of ShowCards could be renamed 'CardCollection' and should be generic enough to be located alongside the Card and Container component inside a new folder in card elements. I think it's worth coupling these 3 together more tightly as CardCollection will need to borrow css from the other two to style the swiper components.

[ ] Styling

With the above in place let's attempt to normalize the card styles to work with swiper and see if we can get the pagination feature of swiper better positioned and using compodio colours.

Swiper needs the Main Swiper component and SwiperSlide. To both be in the same component. This caused me to make some compromises to how I would have liked to structure generic vs project specific modules. So I ended up with a ShowCollection module that makes use of the generic CardCollection + Card, and handles the logic for displaying the Swiper components.
Not immediately obvious how to best override Swiper CSS. There's all kinds of wild ideas here. I used !important in my existing CSS files for now to make view the Swiper cards with the widths used without swiper.

There's some work that could be done with sizing, and spacing of cards at different breakpoints. Overall, Swiper styling is good enough for now.
CardContent was changed to ShowCollection. I'd still like to get rid of it all together by merging it into the less specific CardCollection in the future.
In the mean time I think we can move this feature in its current state into production.

From here it would be nice to merge ShowCollection into the more generic CardCollection, and clean up the use of shared CSS modules. Probably using composition to share common properties for Card and SwiperSlide.


Completion of Svlangsever's of Astronwim

How to use svlangserver normally in mason in astronvim,I haven't been able to get it right,Text completion has been using astronvim's friendly-snippets,How to solve it,Can you give detailed steps.

Mason's download of Sronsever's was also not done with the code 'wrong.'

The svlangserver downloaded by Mason also did not complete the code correction

You should specify the project settings when using svlangserver. Please see the example settings for neovim in README.


Code optimisations

Improved performance of Map and Set;
Fix: #35;
Improved statements and branches coverage;

Benchmarks (not default):
* local/es6           # Map (equal) x 5,164,902 ops/sec ±0.46% (92 runs sampled)
* fast-deep-equal/es6 # Map (equal) x 3,509,397 ops/sec ±0.22% (94 runs sampled)

> Fastest is local/es6 # Map (equal)

* local/es6           # Map (unequal) x 5,920,068 ops/sec ±0.75% (91 runs sampled)
* fast-deep-equal/es6 # Map (unequal) x 4,076,101 ops/sec ±1.62% (92 runs sampled)

> Fastest is local/es6 # Map (unequal)

* local/es6           # Set (equal) x 2,844,644 ops/sec ±0.52% (94 runs sampled)
* fast-deep-equal/es6 # Set (equal) x 1,424,040 ops/sec ±0.25% (94 runs sampled)

> Fastest is local/es6 # Set (equal)

* local/es6           # Set (unequal) x 3,349,728 ops/sec ±0.43% (93 runs sampled)
* fast-deep-equal/es6 # Set (unequal) x 2,319,039 ops/sec ±0.37% (93 runs sampled)

> Fastest is local/es6 # Set (unequal)

Benchmarks (default one):
fast-deep-equal x 212,783 ops/sec ±0.35% (83 runs sampled)
fast-deep-equal/es6 x 172,554 ops/sec ±0.23% (96 runs sampled)
nano-equal x 157,959 ops/sec ±0.44% (89 runs sampled)
shallow-equal-fuzzy x 129,607 ops/sec ±0.28% (93 runs sampled)
underscore.isEqual x 86,097 ops/sec ±0.63% (90 runs sampled)
lodash.isEqual x 30,583 ops/sec ±0.92% (93 runs sampled)
deep-equal x 43,955 ops/sec ±1.21% (95 runs sampled)
deep-eql x 29,686 ops/sec ±1.11% (94 runs sampled)
ramda.equals x 9,582 ops/sec ±0.77% (94 runs sampled)
util.isDeepStrictEqual x 40,327 ops/sec ±0.44% (91 runs sampled)
assert.deepStrictEqual x 557 ops/sec ±0.87% (80 runs sampled)

Coverage decreased (-0.8%) to 94.969% when pulling 7425c32762dc589b5efe809c2a9cd0bbcdc6d196 on xobotyi:code-optimisations into a7bf530713a16217625069091e7abb1a8beb58e5 on epoberezkin:master.

Coverage decreased (-0.8%) to 94.969% when pulling 7425c32762dc589b5efe809c2a9cd0bbcdc6d196 on xobotyi:code-optimisations into a7bf530713a16217625069091e7abb1a8beb58e5 on epoberezkin:master.

@epoberezkin shall i make a separate PR for tests?

Thanks @xobotyi. I didn’t quite understand the reasoning behind these changes in tests, we can discuss separately if a separate PR is needed. To merge this PR l, I suggested removing changes that aren’t required - type changes in tests and other code style changes.

Aren’t required = not required to address issues of this PR - bigint array and Map/Set iteration performance.

Pushed the version without tests and style related changes.

@epoberezkin hey =)
ive done what you've asked.


Unable to find joyride in 6+ version

What should happen?
We are currently using foundation 4 and upgrading to latest(6.6.0) but one of key feature foundation.joyride is missing after 6 upgrade.
I did investigated on the history and found that some sources said it to be ready and released as part of 6.2 but never released. No conclusion found anywhere.
Please share resolution on this and alternatives if any ( Posting this here for help since Zurb foundation forum not allowing new help queries ).
I really appreciate getting any response on this from anyone.

Hi @vimalnair007,
It was extracted from Foundation and it is a standalone jQuery plugin.
https://github.com/zurb/joyride

The new forum is at https://foundation.discourse.group/.

@DanielRuf Thanks for the quick reply :)
Yes i was able to find this and trying it out with foundation 6.6.0.
But it looks like this repository is not maintained. Would you suggest to use this?

2 years: https://github.com/zurb/joyride/commits/master
I can not say much, never used it.


Upgrade of Winston library from version 2.4.4 to 3.2.1

Issue #, if available:
#10 #48 #103
Description of changes:
Upgrade of Winston library from version 2.4.4 to 3.2.1
I tried to keep the changes to a minimum and hence configured the new version to get the behaviour as close as possible to that of the previous version. As part of this change the date-fns package was removed.
There are additional changes/issues that might be worth considering. First, I can't see anything adding meta information to the logs so it might be better to remove the metadata formatter.
Second, it looks like the logging for lambda is treated differently (see lines 31 to 38 of logger.js) and I'm not sure why. In particular it would seem that this code removes any filtering on log level.
Third, as mentioned in one of the issues, using winston might be overkill for what we want to achieve. Maybe it would just be better to have a standard logger that writes everything to the console and then let users to override if they want?
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Hi Brian,
Thank you very much for your contribution. We will be taking a look shortly.

In the initial comment I mentioned that it might be preferable to remove Winston as a dependency as the module doesn't use much of it's functionality. Pull request #190 was created to show this different approach.

I will leave this open until certain but it appears that this will be dropped in favour of #193


feat: Bundle passport-oauth2 types

Fixes #25
This is an attempt to setup api-extractor to automatically bundle types from dependencies, without adding the whole dependency tree.
This also generates a public types report which can be used to check backwards compatibility.

Bundling types isn't way to go, I will close this in favor of #32.


[REQ] Component Request: Sensirion SHT40 Temperature & Humidity Sensor

What's the full name of the component you'd like to see in Adafruit.io WipperSnapper?
Sensirion SHT40 Temperature & Humidity Sensor
What type of component is this (i.e.: Pin, I2C)?
I2C
Product URL or Datasheet
https://www.adafruit.com/product/4885
Additional context
N/A

Yes please it the best sensor on the market please add support

I did my best to add it myself but am getting an error

Ok i got it to verify the code I wrote for this sensor just waiting for it to be approved. fingers crossed
https://github.com/adafruit/Wippersnapper_Components/pull/49
in case you wanted to take a look

Thanks for adding this. I took a look, let's continue this convo on #49.

@TheGodMotherG Support for SHT4x-family of sensors has been added in WipperSnapper firmware beta 43
https://github.com/adafruit/Adafruit_Wippersnapper_Arduino/releases/tag/1.0.0-beta.43

@TheGodMotherG  I used your component and some of the driver work, very.

Thank you again already using it and ordering more sensors!
How much time do you assume it took you to edit my stuff to get it to work?
I’m just genuinely curious how close I got.
Also
Any chance to get it to work in F or a document you can point me to to get
it to work that way?
I get the whole world uses C and I do agree we should be on the metric
system. But It would be great to have a C or F feature
On Thu, Aug 18, 2022 at 11:08 AM Brent Rubell @.***>
wrote:

@TheGodMotherG https://github.com/TheGodMotherG I used your component
and some of the driver work, very.
—
Reply to this email directly, view it on GitHub
https://github.com/adafruit/Wippersnapper_Components/issues/17#issuecomment-1219610420,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXPQAUOBK4TSZCDHJLY5CXDVZZGWJANCNFSM5NX34H6A
.
You are receiving this because you were mentioned.Message ID:
@.***>

How much time do you assume it took you to edit my stuff to get it to work?. I’m just genuinely curious how close I got.

Probably 15min incl. testing, very close!

Any chance to get it to work in F or a document you can point me to to get it to work that way?

Does it not show up under "Edit Component"? Could you link me to the component definition for this sensor?


fix: Publisher should increment key time for batch publish

Codecov Report
Base: 29.78% // Head: 29.78% // Decreases project coverage by -0.00% :warning:

Coverage data is based on head (08ce37f) compared to base (5f4ad35).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #533      +/-   ##
==========================================
- Coverage   29.78%   29.78%   -0.01%
==========================================
  Files          88       88
  Lines       10313    10315       +2
==========================================
  Hits         3072     3072
- Misses       6951     6953       +2
  Partials      290      290

Impacted Files
Coverage Δ

server/services/v1/query_runner.go
4.86% <0.00%> (-0.01%)
:arrow_down:

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

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


containers: Fix the CUDA build to not use an SSH URL

This was failing during the build process:
Cloning<EMAIL_ADDRESS>(to revision stable) to /tmp/pip-req-build-en5sp88r
Running command git clone --filter=blob:none --quiet<EMAIL_ADDRESS>/tmp/pip-req-build-en5sp88r
<EMAIL_ADDRESS>Permission denied (publickey).

Switch to an HTTP URL instead fixes the problem.
Changes
Which issue is resolved by this Pull Request:
Resolves #
Description of your changes:

This is now fixed bi #988 which was merged. Closing.


feat: add integration test

:tada: This PR is included in version 0.1.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Style errors aren't highlighted if inroduced by out of vim editing

Scenario

Open file in Vim
Open file outside the Vim
Add trailing spaces in external editor and save
Confirm file reloading in Vim

Expected
Trailing spaces are shown
Actual
Trailing spaces are not shown

It seems, that issue is probably introduced by some changes in autocmd behavior.

The issue was caused by this bug in another plugin.


drivers: ieee802154: support selective TX channel

@fgrandel This (draft) PR is another approach instead of https://github.com/zephyrproject-rtos/zephyr/pull/79782
Could you please have a look and let me know if it is justified enough? Please also refer to our discord discussion (my nickname is ankuns) and what we agreed there.
The first commit is the most relevant to you, later commits simply implement the proposed API within nrf5 and openthread.
As you see the approach doesn't enforce any driver to implement/support the new Kconfig and related capability.
Just these higher layers that are interested in this feature (currently OpenThread) need to know and possibly turn the feature on. If the feature is on, the higher layer can check if underlying driver supports the selective tx channel feature.


Circular imports bring a poor experience to users

Describe the bug
For example, if you introduce a tool function into the .vue file in some places, the modified page will be refreshed immediately after the call. The experience is too bad, and there is no problem with this usage.
This may be a blow to many projects 😭
Reproduction
none
Steps to reproduce
No response
System Info
vite5.0.0-beta.18

Used Package Manager
pnpm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Modifying any content in the template or script tag in the .vue file will trigger reloading, but modifying the style will not trigger it.

Duplicate of https://github.com/vitejs/vite/issues/10208. In Vite 5, Vite will try its best to avoid circular imports when possible.  There are cases where we can't apply HMR without the risk of breaking your app due to circular execution order, it's documented at https://main.vitejs.dev/guide/troubleshooting.html#a-full-reload-happens-instead-of-hmr. Ideally you should avoid it if possible.


Search bar

There is no search bar
And do u guys have an app for this thing?


Start of Room Editor

Moved the data files into /data
Updated the Git Actions to run in a more consistent cadence.
Create a Notebook for the future Room Editor Voila application.
Update the data loader for the new directory.

The pip problem is that there is a Windows specific package than the Linux utility can't deal with as is. I need to adjust the Git Action.


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-main" workflow

Codecov Report

Merging #529 (52ccf75) into main (09d83b8) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #529   +/-   ##
=======================================
  Coverage   95.69%   95.69%
=======================================
  Files           1        1
  Lines         395      395
  Branches       53       53
=======================================
  Hits          378      378
  Misses         17       17

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


Feature Suggestion: Kill SoundKeeper on Workstation Lock, Start it again on Workstation Unlock

I have a feature suggestion to perhaps help with sleep issues.
Perhaps an option can be added for SoundKeeper to create two scheduled tasks in Task Scheduler. One to kill the process on workstation lock, to help with automatic sleep issues and another to start SoundKeeper back up on Workstation Unlock.
I've done this manually with two batch files and it works quite well. At least with Windows 11 the screen will automatically shut off if audio is playing allowing the workstation to lock and SoundKeeper to be killed. Once the workstation is unlocked SoundKeeper will start up again. At least on Windows 11 this should help with sleep issues.

It was one of the things that I considered, but it's not flawless. For example, when my workstation is locked, I expect to hear notifications from IM apps, and without Sound Keeper these short sounds will be cut.
I may add it as an option in the future.

It's a good option for a laptop perhaps. That's my use case.

If your system is configured to go into sleep mode when laptop lid is closed, streaming any audio shouldn't prevent your system from sleeping.

@mackid1993 you should be able to accomplish this by creating a task in task scheduler that triggers "on workstation lock of any user" that kills the process using taskkill for instance, and vice versa, restarts it on logon. It doesn't necessarily need a software feature at all.

 Please fix the docs and it would be super helpful to have a /? help usage switch :)

Also, thanks for making this tool. I just bought the Q Acoustics M20 HD's for my PC and the USB seems to be a glorified s/pdif and when the signal goes dead, the speaker kills the amp and it comes alive with an annoying pop every time audio playback starts. Glorious sound, but unusable without your tool :)

you should be able to accomplish this by creating a task in task scheduler that triggers "on workstation lock of any user" that kills the process using taskkill for instance, and vice versa, restarts it on logon / unlock. It doesn't necessarily need a software feature at all. You can implement all kinds of complex event based scenarios using task scheduler pretty easily.

I actually did this but I had to stop using SoundKeeper as keeping the audio hardware active all of the time was shaving 2 hours off the battery life of my laptop 🙁It's a shame because it stopped the audio popping issues when the hardware woke up.

Oh, that's weird. Laptops usually have class D amplifiers that are very energy efficient. 2 hours off sounds like too much. How did you measure?

Did you try the OpenOnly mode? That just opens the audio device but doesn't stream anything.

Did you try the OpenOnly mode? That just opens the audio device but doesn't stream anything.

That helps the battery issue, but not the audio popping. I'm going to try running it stock and benchmark it a little bit better because it really does solve all of my audio problems.

I wanted to clarify further what actually you are using?

Built-in speakers?
Wired headphones?
Bluetooth headphones?

I wanted to clarify further what actually you are using?

Built-in speakers?
Wired headphones?
Bluetooth headphones?

Sorry I'm using the built in speakers on my Surface Laptop Studio 2.

This is really wild that built-in speakers have this issue.

Yup leave it to Microsoft!

@vrubleg So I tested a little more. SoundKeeperZero.exe seems to stop the popping and also lets the battery drain drop a little bit lower at idle.
Hopefully Microsoft will eventually fix this with a driver or firmware but my hopes aren't high.


Future of Spark

As you probably have noticed the project has been in hibernation for a couple of months. The reason to this are mainly personal reasons.
However, the plan is to ramping up development of Spark. The plan is to make a 3.0 release where the API is revised and improved where possible and relevant extensions are made. The kotlin framework also needs an official release.

You're my hero!

@RyanSusana
Thanks!
That is one possible way, I have yet to investigate that option on a deep level. However, there are several parameters to weigh in when taking that decision. The working plan has been continuing on the Spark baseline with the solution to build upon javelin as a possibility if deemed beneficial.
The plan is either way to fixing som crucial issues and make a 2.9.0 release soon and thereafter start work on the 3.0 release. I am currently compiling a list of must-haves and nice-to-haves in Spark 3.0 and I'll be posting continuously here in this tread and on the Spark twitter page everytime there's any interesting news.

Looking forward to it, I have written some interfaces to Sparkjava/Javalin in my CMS framework, I would be happy to contribute

I would love to see a 2.9 release soon. Happy to help on PRs if we can make a roadmap or list of features we need to see.

@robax the plan is to make a 2.9.0 release next week. It'd very much appreciate any help. Is there any particual issues/PRs you'd like to see in 2.9.0?

I had a couple.

Native support for uploaded files instead of getting the raw request.
multiple static file locations

It would be awesome if #1049 could make it into 2.9.0.

I just want to see #1049. If you want to release next week (which would be awesome), then probably just going through and merging open PRs is the best plan of attack? Then we can work on new features for the 3.0 release.

Thanks for the quick release! @perwendel

Would it be possible to add internal metrics (usages, perfs, custom) to answer my personal needs of control :)  More seriously, in a world of containers, metrics are mandatory to monitor services. Maybe a look at Microprofile-metrics and their annotations could inspire developers ?

Please support render raw jsp. for example:
get("/url", (q, a) -> jsp("/x.jsp");  // or support servletDispatch to WEB-INF/jsp file.

this would help to upgrade the legacy code. Thanks!!


docs: close bracket on helm chart reference

SUMMARY
This is a minor Markdown change in the README to close brackets on the Helm charts reference so that it will be properly presented as a hyperlink
This is a minor improvement to make the README more polished for readers
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
Before

After

No tests were created or modified because this is documentation written in Markdown

Thank you @brnnnfx! I see this is your first PR in Superset, which is great. Let me know if you would like to contribute more, I'd be happy to assist.

Thanks @geido I would be interested in contributing more!

That's great @brnnnfx. If you haven't already, join the Superset Slack. We can chat there. I am geido there too.


fix build comment in generator

Pretty sure this comment was meant to include :client?

@ttrmw You are correct, npm run build:dev is not a valid run command. The client webpack should be run with npm run build:dev:client.

Thanks @ttrmw


Availability Outside German App Store

Feature description
Make app available to global app store users
Problem and motivation
A large number of expats and military personnel live and work in Germany for short periods of time and typically leave their app store set to their home country.  As changing app stores can occasionally be difficult and cumbersome, making the Corona-Warn app would make it far easier for those individuals to participate in the system while inside of Germany.
Is this something you're interested in working on
Unable To.

The same issue is true for cross-border commuters (e.g. for people living in Austria and working in Germany) and tourists who want to use the German app while spending a week(end) in Germany.
The usage of the app should not be limited to the country which is set in the app store.

This unfortunately has already been rejected, see:
https://github.com/corona-warn-app/cwa-app-ios/issues/623
https://github.com/corona-warn-app/cwa-backlog/issues/9
They'll only enable it country-by-country :(

Dear colleagues,
as we have this new process now, we will close this generic issue. Please create issues and vote on country by country as outlined on https://github.com/corona-warn-app/cwa-backlog/issues/9#issuecomment-648745301
Thanks for your understanding!
Mit freundlichen Grüßen/Best regards,
SW
Corona Warn-App Open Source Team


Revert "Configure cloud provider"

Reverts dutt-sanjay/github-actions-continuous-delivery-azure#2

Spin up environment

Done.


New interface

With your last change, the interface changed and I am trying to move from archiver.Tar.Make(... to the new interface, but not having any luck. A bit of a Go novice, so please bare with me.
"github.com/mholt/archiver"
...
err := archiver.Archiver.Archive([]string{
		"src",
		"test",
		"help"
	}, "/tmp/some.tar")

gives me:
./main.go:32:34: not enough arguments in call to method expression archiver.Archiver.Archive
	have ([]string, string)
	want (archiver.Archiver, []string, string)

The new equivalent of the old archiver.Tar.Make() is something like this:
t := archiver.Tar{
    // ... configuration; see godoc
}
err := t.Archive([]string{"file1", "file2", "..."}, "output.tar")
if err != nil {
    return err
}

You can find the configuration fields for the Tar type here: https://godoc.org/github.com/mholt/archiver#Tar

Thanks @mholt for the quick response.
Thanks for great lib/tool, obviously ;-)


SQL as of 2017-10-03

Codecov Report

Merging #1503 into master will decrease coverage by 3.52%.
The diff coverage is 46.15%.

@@            Coverage Diff             @@
##           master    #1503      +/-   ##
==========================================
- Coverage   55.44%   51.92%   -3.53%
==========================================
  Files        3532     3575      +43
  Lines       85566    93191    +7625
==========================================
+ Hits        47442    48388     +946
- Misses      38124    44803    +6679

Impacted Files
Coverage Δ

...sql/operations/virtual_network_rules_operations.py
15.84% <0%> (ø)
:arrow_up:

...ure/mgmt/sql/models/sql_management_client_enums.py
100% <100%> (ø)
:arrow_up:

...e-mgmt-sql/azure/mgmt/sql/sql_management_client.py
100% <100%> (ø)
:arrow_up:

azure-mgmt-sql/azure/mgmt/sql/models/__init__.py
100% <100%> (ø)
:arrow_up:

...ure-mgmt-sql/azure/mgmt/sql/operations/__init__.py
100% <100%> (ø)
:arrow_up:

azure-mgmt-sql/azure/mgmt/sql/version.py
100% <100%> (ø)
:arrow_up:

...l/azure/mgmt/sql/operations/database_operations.py
28.26% <28.26%> (ø)

...mt-sql/azure/mgmt/sql/models/database_operation.py
29.41% <29.41%> (ø)

...-sql/azure/mgmt/sql/models/virtual_network_rule.py
55.55% <33.33%> (-15.88%)
:arrow_down:

.../azure/mgmt/sql/models/database_operation_paged.py
80% <80%> (ø)

... and 117 more

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 cce4e87...34666bf. Read the comment docs.

Not ready for release


feat: move mev rpc types to alloy

ref https://github.com/paradigmxyz/reth/issues/9027
Waiting for alloy dep pump to get rpc_types_mev from alloy

should be out now

https://crates.io/crates/alloy-rpc-types-mev

thanks, will merge after next alloy release

already out

This g2g?

it's waiting for the next release of alloy to be merged.


Improvements to parent & scope Documentation

parent and scope both document similar things (i.e. references to other resources), but they do so in different ways - e.g. parent uses parentSymbolicName whilst scope uses resourceName.  There could be a bit more consistency here.
In the value (or possibly description), when exact types are known then that exact type should be included(e.g. for a cosmos sql database the parent must be Microsoft.DocumentDB/databaseAccounts).  This should also link to the documentation for that resoruce type.  This should always be possible for parent (in which the parent should be of exactly one resource type), but I'm not sure of the practicality of doing so for scope.
The symbolicName referenced in parent samples feels a bit misleading to me

For anyone not familiar with compiler design (I'd guess the vast majority of people), it adds more confusion with "What is a symbolic name?".
There are also many legal values that are not symbolic names (e.g. resourceGroup(), subscription(), tenant() (and eventually resource() 🤞)

When the exact type of a parent/scope is known, rather than use a generic name such as resource or parent, could use something more specific based on the parent type e.g. databaseAccountResource.
Putting this all together, something like:

Lots of good suggestions here. Adding @mumian / @stephaniezyen to take a look.

I think using resourceSymbolicName for both parent and scope makes sense. I also like the idea of adding the resource type for parent resources. Using more specific values for different parent types might lead to confusion. It's always just a resource symbolic name but using different values like databaseAccountResource implies it is something different.
Extension resources that can be applied to the tenant have tenant() as a value. See roleAssignments. For other scope functions, the scope is set on the module, not the extension type.

This work has been completed.
Here's an extension resource type.
Here's a child resource type.

Looks good, thanks Tom! Closing.


Wrong Word comparison output

This issue is raised in https://forum.groupdocs.com/t/groupdocs-comparison-for-net-mvc-example-problems/8273
Hi,
Please have a look at the attached videos and document with issue details below:
files_2.zip
I am also uploading the problematic files:
Problematic files.zip
When I compare the attached Word files, output has changed position of the content. Have a look at the wrong output.zip.
However, I am not able to reproduce this issue in console application using API version 20.3.1.
Please take a look.
Thanks

Hi @atirtahirgroupdocs
This issue comes with oldest API version (19.6) which is used in this project.
We already working on API version update, so please stay tuned.
Regards

The issue is not longer reproducible on a current version of the Comparison sample app from the master-branch:


yii2 updateAllCounters

I realized the addition operation is as follows
Count::updateAllCounters(['follower'=>1],['user_id'=>$user_id]);

But I realized the subtraction operation
To achieve a similar way
Count::updateAllCounters(['follower'=>-1],['user_id'=>$user_id]);

This can not be achieved

Why?


Reconfigure x86 CPU benchmarks

Reconfigures x86 CPU benchmarks to:

Benchmark with threads 1 and 8 for small workloads (models <10M params).
Benchmark with threads 1, 8 and 13 threads for large workloads (models >= 10M params or large batch size). We use 13 because a c2-standard-60 machine will be used which has 15 cores and MAX_CORES - 2 has empirically shown to be an optimum configuration.

Before this change, configurations used sync, thread 1, 4, and 8 for all models.
x86 benchmarks completed in 21 mins (from 30 mins).

Note the description isn't quite right, I think. The c2-standard-60 actually has 30 cores. By default that gives 60 vCPU with hyperthreading, but we can set to a 1:1 ratio of vCPU to physical CPU. The VM has two NUMA sockets though, so using one socket, that gets us 15 cores. So you've got the right setting, but the description is inaccurate when it says that c2-standard-60 machines "have 15 cores"

Also, thank you for optimizing this stuff :-) I'm happy to see improvements in the usefulness of our benchmarks and reduced CI latency


[1pt] Create a schema.sql file with tables that will be analogical to the structure of the classes that you created

books table (add all properties and associations from the parent Item class as table columns)
labels table

We have done it


uv init hangs forever unless I add --python-preference only-managed

Hi,
Just trying out uv for the first time (previous experience with rye), and weirdly I ran into problems running the first example command in the Getting Started docs:
$ uv -v init example
DEBUG uv 0.4.15
DEBUG Searching for default Python interpreter in managed installations or system path
DEBUG Searching for managed installations at `.local/share/uv/python`

and it just hangs forever until killed, using almost no CPU resources.
I guessed it was a problem with the search for pre-existing local python installations, so I tried it with
$ uv -v init example --python-preference only-managed

and that works fine.
System is macOS Sonoma 14.2.1 (23C71) running on intel
$ uv --version
uv 0.4.15 (0d81bfbc6 2024-09-21)

$ uname -a
Darwin gris.local 23.2.0 Darwin Kernel Version 23.2.0: Wed Nov 15 21:54:10 PST 2023; root:xnu-10002.61.3~2/RELEASE_X86_64 x86_64

Thanks
Will

That's really weird — can you share the output of RUST_LOG=uv=trace uv python find -v?

So, now that I have run once with only-managed it seems to be working fine even without that flag. I get
$ RUST_LOG=uv=trace uv python find -v
DEBUG uv 0.4.15
DEBUG Searching for default Python interpreter in managed installations or system path
DEBUG Searching for managed installations at `/Users/will/.local/share/uv/python`
DEBUG Found managed installation `cpython-3.12.6-macos-x86_64-none`
TRACE Cached interpreter info for Python 3.12.6, skipping probing: /Users/will/.local/share/uv/python/cpython-3.12.6-macos-x86_64-none/bin/python3
DEBUG Found `cpython-3.12.6-macos-x86_64-none` at `/Users/will/.local/share/uv/python/cpython-3.12.6-macos-x86_64-none/bin/python3` (managed installations)
/Users/will/.local/share/uv/python/cpython-3.12.6-macos-x86_64-none/bin/python3

However, if I try and get back to the initial conditions after installing uv for the first time by nuking .local/share/uv, then I can reproduce the hanging behavior:
$ rm -r ~/.local/share/uv
$ RUST_LOG=uv=trace uv python find -v
DEBUG uv 0.4.15
DEBUG Searching for default Python interpreter in managed installations or system path
DEBUG Searching for managed installations at `/Users/will/.local/share/uv/python`
TRACE Searching PATH for executables: python3, python
TRACE Checking `PATH` directory for interpreters: /Users/will/.cargo/bin
TRACE Checking `PATH` directory for interpreters: /Users/will/.gem/ruby/2.6.0/bin
TRACE Checking `PATH` directory for interpreters: /Users/will/mambaforge/condabin
TRACE Checking `PATH` directory for interpreters: /Users/will/.rye/shims
TRACE Found possible Python executable: /Users/will/.rye/shims/python3
TRACE Querying interpreter executable at /Users/will/.rye/shims/python3
^C

So it looks like the problem might be with the rye-installed python. I will check this next

I wonder if the Rye shim is invoking uv so it cycles?

Could be ...
$ ls -l /Users/will/.rye/shims/python3
lrwxr-xr-x  1 will  staff  26 Aug 14 13:17 /Users/will/.rye/shims/python3 -> /Users/will/.rye/shims/rye

$ rye --version
rye 0.38.0
commit: 0.38.0 (3e3c8540f 2024-08-02)
platform: macos (x86_64)
self-python<EMAIL_ADDRESS>symlink support: true
uv enabled: true

I did rye self update so it is now on 0.39.0 but that did not help

Could you run the rye shim (in the same directory as previous experiments) and print(sys.executable)

OK, we seem to be getting somewhere. It turns out that the rye shim hangs when I try to run it outside of any rye or uv managed project. So this explains the uv init hangs
Looking at the rye docs, it seems that it should be finding the system python in this case, which I guess is the part that seems to be broken on my machine.
If I do
$ rye config --set-bool behavior.global-python=true

then the shim now does work in non-managed directories:
# will @ gris in ~/tmp [10:46:53]
$ /Users/will/.rye/shims/python
Python 3.12.1 (main, Jan  8 2024, 06:38:22) [Clang 16.0.3 ] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import sys; print(sys.executable)
<EMAIL_ADDRESS>
and uv init now works fine, even starting from an empty ~/.local/share/uv folder:
$ uv -v init example-4
DEBUG uv 0.4.16
DEBUG Searching for default Python interpreter in managed installations or system path
DEBUG Searching for managed installations at `/Users/will/.local/share/uv/python`
DEBUG Found `cpython-3.12.1-macos-x86_64-none` at `/Users/will/.rye/shims/python` (search path)
DEBUG Writing Python versions to `/Users/will/tmp/example-4/.python-version`
Initialized project `example-4` at `/Users/will/tmp/example-4`

Oddly, this now seems to be picking up a homebrew version of python
$ (cd example-4; uv run hello.py)
Using CPython 3.12.4 interpreter at<EMAIL_ADDRESS>Creating virtual environment at: .venv
Hello from example-4!

Then I double-checked by running
rye config --set-bool behavior.global-python=false
mv ~/.local/share/uv/python ~/.local/share/uv/python-MOVED
uv -v init example-6

and that reproduced the hang again.

Happened to me as well.
Running rye config --set-bool behavior.global-python=true solved the issue.


Print file name and slot after calling add_file()

Hey Prime,
I have this problem with harpoon: When I call the add_file() command, I usually don't know in which slot the file will be placed. This usually results in calling toggle_quick_menu() right after to check the slot. This is especially a problem when harpoon has some files inside already. When I create a new project I can keep the slots in my head. Later on, I have to constantly check slots when adding new files.
I thought it might be a good idea to print to which slot was the current file placed in, after calling add_file(). Also, if the file is already placed in a slot, I would like it to print in which one.
I am not sure what the print message should look like (I am not that good at English :)). For now, this is how it looks like:
print("File", buf_name, "added to slot", found_idx)
print("File", buf_name, "already exists in slot", buf_idx)
Also, there could be an option in config when calling the setup(), to turn prints like this on.
If you don't like this idea, feel free to delete this PR. I will think of a different way to print the files just for me :)

I like this idea.  But I think that we should try something a bit different.
You can update your status line H: % where you replace % with a harpoon#(i_forgot_the_function_call)
If you really want this to work out, then we need to have an option that defaults to false and you can just have this feature only and set that option to true.

@ThePrimeagen, great idea! I like seeing it in the status line even more than printing it. This fixes my problem, and this PR can be closed. Thank you :)
@asbjornhaland Nice! I am using galaxyline, so I can use the function you created like this:
require("galaxyline").section.left = {
    -- other galaxyline components
    {
     Harpoon = {
        provider = require("harpoon.mark").status,
        separator = '  ',
        highlight = { colors.white, colors.bg, 'bold' },
        separator_highlight = { 'NONE', colors.bg },
      }
    },
    -- other galaxyline components
}

Result (M1):

Thank you for help!


Does Typhoeus support HTTP/2

Hi,
I have some large assets I need to download (>2GB) with Typhoeus, but the downloads consistently fail using HTTP/1.1 and I've only gotten them to work with HTTP/2. Does Typhoeus support HTTP/2 and if so, how would I implement that?

Yes. Pass a http_version: :httpv2_0 option to Typhoeus::Request.new.
https://github.com/typhoeus/ethon/blob/453c6f0ba37a7d42978c90f2399f5c2cd66b32a6/lib/ethon/curls/options.rb#L355

It should use HTTP/2 by default where available, same as curl.
> Typhoeus::Request.new("https://www.google.com", verbose: true).run

  Trying <IP_ADDRESS>:443...
Connected to www.google.com (<IP_ADDRESS>) port 443 (#1)
...
Using HTTP2, server supports multiplexing

How do you multiplex requests? Does Hydra automatically multiplex requests when using an HTTP/2 server?


forever: true causes memory to leak like the deepwater horizon

var request = require('request');

var concurrency = 128;
var inProgress = 0;
var startedCount = 0;
var finishedCount = 0;

function fillQueue() {
  while (inProgress < concurrency) {
    makeRequest();
  }
}

function makeRequest() {
  inProgress++;
  startedCount++;

  request.post({
    uri: 'http://localhost/',
    forever: true
  }, function(){
    inProgress--;
    finishedCount++;
    fillQueue();
  });
}

fillQueue();

setInterval(function(){
  console.log(inProgress+' requests in progress, '+startedCount+' started, '+finishedCount+' finished');
}, 1000);

When executed, this script will leak memory until you run out of it. This effect has been observed on ubuntu 14.04 using node 0.11.16 and 0.12.4. There is no memory leak in 0.10.25.
Don't run this script if you have a server running locally on port 80, this might DOS it.
Commenting out forever: true will cause this script to run as normal, but without the memory leak, and presumably without keepalive support.

@devTristan do you know what is going on?

My first guess would be something in here preventing request data from being garbage-collected. Very possible that I'm wrong, though.

@simov gentle ping

0.12+ makes this feature totally unnecessary. We should check if the node version is greater than 0.10.x and print a warning when using this feature and silently turn it off.

:+1: I was just trying to figure out what's the difference between forever-agent and the current Agent implementation in core.

@devTristan, theoretically this should give you the same results, without the memory leak. Let me know what do you think.
var request = require('request');
var http = require('http');

var concurrency = 128;
var inProgress = 0;
var startedCount = 0;
var finishedCount = 0;

function fillQueue() {
  while (inProgress < concurrency) {
    makeRequest();
  }
}

function makeRequest() {
  inProgress++;
  startedCount++;

  request.post({
    uri: 'http://localhost/',
    agentOptions: {keepAlive:true, maxSockets:1},
    agentClass: http.Agent
    // forever: true
  }, function(){
    inProgress--;
    finishedCount++;
    fillQueue();
  });
}

fillQueue();

setInterval(function(){
  console.log(inProgress+' requests in progress, '+startedCount+' started, '+finishedCount+' finished');
}, 1000);

It does indeed. Same results, no memory leak.

Ok, I should probably play around with these options and figure out some fix, as mikeal pointed out. Our docs should be improved too.

btw {keepAlive:true, maxSockets:1} closes the socket in node 0.12.6 even if you specify big enough keepAliveMsecs timeout, works in iojs though :astonished:


Option to group packages by Go module?

Can we have an option to group packages by their Go module?
Meaning all packages inside a Go module be combined into one rectangle/node.
Including sub-packages of current module (root of the tree).
So that the nodes are modules, not packages.
Because that's what we generally mean by "dependency".
Once your module depends on a package from another module, your module depends on that module.

We can get the data from go mod graph.
This tool gives a web page for it, but I still like to have the dot and svg files.

Sorry, this tool does that: https://github.com/lucasepe/modgv


Multi out

I seem to have accidentally goofed on #502, let's try this again.

@ckuethe, Chris, how much of this is still needed?  keogram.cpp has been changed since you first created this PR.  I believe many (most or all?) the changes are in the current keogram.cpp.

All of it and none of it. I'll have to update multi_out to match current code style but the point of the tool remains: avoid wasting time on loading files multiple times when we could do more with each frame while we have it in memory

@ckuethe are you thinking all that processing should happen after the image is written to disk, or while it's still in memory in the capture program?

After it's written to disk. We don't want to clutter up the capture tool which could be running on a different machine. The point here is to avoid unnecessary work by reusing the same decompressed image in memory to generate startrails, keograms, or whatever else.


SearchDescriptor.Query(q=>q.Ids()) results in empty JSON object on serialization

SearchDescriptor<NestRelease> searchDescriptor = new SearchDescriptor<NestRelease>();
searchDescriptor.Query(q => q.Ids(i => i.Values(id)));
ISearchResponse<NestRelease> searchResponse = this.elasticContext.Search<NestRelease>(searchDescriptor);

returns too many documents, because the searchDescriptor is an empty json object after serialization.
Proofed here:
FileStream fileStream = new FileStream("D:\\searchDescriptor .json", FileMode.Create);
this.elasticContext.Serializer.Serialize(searchDescriptor, fileStream);

I can't get one document by it's id because i use an alias for many indices. Now i do a search via term as a workaround. Any other ideas?
searchDescriptor.Query(q => q.Term(t => t.Field("_id").Value(id)));

best regards

Which version of NEST are you using / branch are you building?
I couldn't replicate generating an empty json object with the following
void Main()
{
	var client = new ElasticClient(connection: new InMemoryConnection());
	var id = new long[] { 1, 2 };

	SearchDescriptor<NestRelease> searchDescriptor = new SearchDescriptor<NestRelease>();
	searchDescriptor.Query(q => q.Ids(i => i.Values(id)));

	var result = client.Search<NestRelease>(searchDescriptor);
	Console.WriteLine(Encoding.UTF8.GetString(result.RequestInformation.Request));
}

public class NestRelease
{
}

This generates the following query DSL
{
  "query": {
    "ids": {
      "values": [
        "1",
        "2"
      ]
    }
  }
}

Is id an empty IEnumerable<long>? Did you also see multi-get?

I'm using branch 2.0 with latest changes. I forgot to say.
My id is just one value, but i've tried. It makes no difference.
Multi-get is another workaround. I do it via term instead of Ids.
searchDescriptor.Query(q => q.Term(t => t.Field("_id").Value(id)))
Is a multi-get the better way?

Would you be able to show what id is in your first code example please?

IsConditionless was taking into account types, but they are optional. This is fixed with the above commit.  Thanks for reporting @danielweisss!


ExtendViewport does not Always Preserve Aspect Ratio

A square shape with x = y = 10 does not render as a square, when using ExtendViewport. Instead a y-stretched rectangle is presented. When switching to FitViewport with the same parameters, the aspect ratio is maintained.
	public void create () {
	    camera = new OrthographicCamera(800, 480);
	    viewport = new ExtendViewport(800, 480, camera);
	    stage = new Stage(viewport);

		shapeRenderer.begin(ShapeType.Filled);
		shapeRenderer.identity();
		shapeRenderer.setColor( Colors.get(square.color));
		shapeRenderer.rect(square.x, square.y, square.width, square.height);
		shapeRenderer.end();

I noticed that FitViewport extends ScalingViewport while ExtendViewport extends Viewport. The difference might be due to using enum fit in ExtendViewport:
		Vector2 scaled = Scaling.fit.apply(worldWidth, worldHeight, screenWidth, screenHeight);

In the FitViewport this is simply:
		Vector2 scaled = scaling.apply(getWorldWidth(), getWorldHeight(), screenWidth, screenHeight);

The sample code you've given doesn't even use the viewport.


[Bug] Impersonating Service Account - Defaults GCP quota_project to project where SA is defined

Is this a new bug in dbt-bigquery?

[X] I believe this is a new bug in dbt-bigquery
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
When using the impersonate_service_account setting in the profiles.yml for BigQuery it will default the quota_project for the client_options to None. It will then use the project that the service account is defined in as the quota project.
This means that if the service account a user sets is defined in a project that does not have the BigQuery API enabled in it will fail, no matter what execution_project or project are set to in the profiles.yml as these are separate to the quota project.
Expected Behavior
A user should be able to override and set the quota project in the configuration of the dbt profile rather than it defaulting to the project the service account is defined in. A user should not have to enable the BigQuery API for a project that a service account lives in if BigQuery is not required in that project.
Steps To Reproduce

Setup a python venv and pip install the following versions

pip install dbt-core==1.8.6 dbt-bigquery==1.8.2

Setup three separate GCP projects (the names of which will have to be unique to yourself)

MY_SERVICE_ACCOUNT_PROJECT:

Create a service account called dbt-sa
Give your user service account token creator role on the above SA so that you can impersonate it

MY_DATA_PROJECT:

Enable BQ API and create a dataset called foo within it
Give service account above roles/bigquery.dataViewer to the dataset

MY_EXECUTION_PROJECT:

Enable BQ API
Give service account above roles/bigquery.jobUser to the project

Create a profiles.yml configuration that contains the following and substitute the names of your GCP projects with those you have created.

  target: dev
  outputs:
    dev:
      type: bigquery
      method: oauth
      schema: foo
      location: europe-west2
      priority: interactive
      project: $MY_DATA_PROJECT
      execution_project: $MY_EXECUTION_PROJECT
      impersonate_service_account<EMAIL_ADDRESS>

Run the following dbt command and point to wherever you have stored your profiles.yml file defined above

dbt debug --connection --profiles-dir .

This will error on the connection and complain that the BQ API is not enabled in the project, it will link you to the project that the service account is defined in and crucially not the project or execution_project set in the profile.
Relevant log output
❯ dbt debug --connection --profiles-dir .
22:22:11  Running with dbt=1.8.6
22:22:12  dbt version: 1.8.6
22:22:12  python version: 3.12.1
22:22:12  python path: /Users/jackcarpenter/.pyenv/versions/3.12.1/envs/dbt-bigquery-bug/bin/python
22:22:12  os info: macOS-13.2.1-x86_64-i386-64bit
22:22:13  Using profiles dir at .
22:22:13  Using profiles.yml file at ./profiles.yml
22:22:13  Using dbt_project.yml file at /Users/jackcarpenter/dbt-bigquery-bug/dbt_project.yml
22:22:13  adapter type: bigquery
22:22:13  adapter version: 1.8.2
22:22:13  Skipping steps before connection verification
22:22:13  Connection:
22:22:13    method: oauth
22:22:13    database: MY_DATA_PROJECT
22:22:13    execution_project: MY_EXECUTION_PROJECT
22:22:13    schema: foo
22:22:13    location: europe-west2
22:22:13    priority: interactive
22:22:13    maximum_bytes_billed: None
22:22:13    impersonate_service_account<EMAIL_ADDRESS>22:22:13    job_retry_deadline_seconds: None
22:22:13    job_retries: 1
22:22:13    job_creation_timeout_seconds: None
22:22:13    job_execution_timeout_seconds: 5000
22:22:13    timeout_seconds: 5000
22:22:13    client_id: None
22:22:13    token_uri: None
22:22:13    dataproc_region: None
22:22:13    dataproc_cluster_name: None
22:22:13    gcs_bucket: None
22:22:13    dataproc_batch: None
22:22:13  Registered adapter: bigquery=1.8.2
22:22:15  BigQuery adapter: https://console.cloud.google.com/bigquery?project=MY_EXECUTION_PROJECT&j=bq:europe-west2:7388df4b-c857-4aef-8af5-5f97d9a5ff74&page=queryresults
22:22:15    Connection test: [ERROR]

22:22:15  2 checks failed:
22:22:15  Project loading failed for the following reason:
 project path </Users/jackcarpenter/dbt-bigquery-bug/dbt_project.yml> not found

22:22:15  dbt was unable to connect to the specified database.
The database returned the following error:

  >Database Error
  BigQuery API has not been used in project xxxxxxxxxxxxx before or it is disabled. Enable it by visiting https://console.developers.google.com/apis/api/bigquery.googleapis.com/overview?project=xxxxxxxxxxxxx then retry
. If you enabled this API recently, wait a few minutes for the action to propagate to our systems and retry.

Check your database credentials and try again. For more information, visit:
https://docs.getdbt.com/docs/configure-your-profile

### Environment

```markdown
- OS: macOS Ventura 13.2.1
- Python: 3.12.1
- dbt-core: 1.8.6
- dbt-bigquery: 1.8.2

Additional Context
This relates to a feature request I have also raised #1343 that outlines a solution/improvement that I think may work

@jcarpenter12 Thank you for opening this issue! I appreciate the level of detail you provided here and in the related PR. Would you be able to explain to me in what instances a user would need to impersonate a service account and also not use the related quota account? Is it due to access?

@jcarpenter12 Thank you for opening this issue! I appreciate the level of detail you provided here and in the related PR. Would you be able to explain to me in what instances a user would need to impersonate a service account and also not use the related quota account? Is it due to access?

Hi @amychen1776 thanks for getting back to me. This is actually a bit misleading this issue. It actually isn't related to impersonating a service account at all it applies when using any auth method as far as I can tell. I've raised this issue #1347 to cover that and will close this one as it's not the core problem


Feature timeline and Epic Roadmap by Microsoft DevLabs is taking longer than expected to load

Describe the bug
Getting a message as 'Feature timeline and Epic Roadmap by Microsoft DevLabs is taking longer than expected to load.'  Epic Roadmap(beta) has been added as a new option.
Browser
Choose a browser
[x] IE11
[x] Chrome
[ ] FireFox
[ ] Safari
Version
[x] VSTS
[ ] TFS 2017 Update 2/3
[ ] TFS 2018
Screenshots
If applicable, add screenshots to help explain your problem.

This is working in TFS 2018 though and there is no Epic roadmap (beta) tab.

We are experiencing the same problem within VSTS. Unable to view the Feature Timeline as it is taking too long to Load.

Thanks for reporting the issue, can you Clear the cache and do a hard refresh using Ctrl + F5?

Its still not working after clearing cache and Ctrl+F5.

Mine is not working either after following the steps provided

I am really sorry for the inconveniences, I am on my way to office, I will take a look ASAP.

Can you open Developer tools by pressing F12 in Chrome and then do Ctrl + F5?

Tried it. Console still gives the error

This really strange ... as I am not able to repro it anymore. Can you send me an email at<EMAIL_ADDRESS>I can setup a Skype call and diagnose with you.

The issue seems to be with the CDN ... we are investigating.

Thanks for reporting the issue @SEnglishSMP , the root cause is an issue with the CDN, we are trying to work with them for a resolution.

Is there any update on this issue?
Thank you

As a workaround you can disable http/2 in Firefox browser.

Go to about:config
Set network.http.spdy.enabled.http2 to false

Using Chrome on Windows 10.  Azure region is Central US.
Chrome debugging console logs the following error:
Content?bundle=vss-bundle-basejs-vI1IxAQ10AVWwEoAF6E9t0idR3ZSaZnasnpanuPQzGJ0=:57 GET https://ms-devlabs.gallerycdn.vsassets.io/extensions/ms-devlabs/workitem-feature-timeline-extension/0.0.316/1532924896769/dist/featuretimeline.js 404 ()

The issue is fixed, please confirm that it works for you.

Closing at this is resolved now.

This works fine now. Thanks.

Thanks for confirming @ssriprashanth .

Fixed.  Thanks.


Could not find 'uncrustify'. The program may not be installed.

Could not find 'uncrustify'. The program may not be installed.
See http://sourceforge.net/projects/uncrustify/ for program installation instructions.
Your program is properly installed if running 'where.exe uncrustify' in your CMD prompt returns an absolute path to the executable. If this does not work then you have not installed the program correctly and so Atom Beautify will not find the program. Atom Beautify requieautify requires that the program be found in your PATH environment variable.
Note that this is not an Atom Beautify issue if beautification does not work and the above command also does not work: this is expected behaviour, since you have not properly installed your program. Please properly setup the program and search through existing Atom Beautify issues before creating a new issue. See https://github.com/Glavin001/atom-beautify/search?q=uncrustify&type=Issues for related Issues and https://github.com/Glavin001/atom-beautify/tree/master/docs for documentation. If you are still unable to resolve this issue on your own then please create a new issue and ask for help.
Hide Stack Trace
Error: Could not find 'uncrustify'. The program may not be installed.
at Uncrustify.module.exports.Beautifier.commandNotFoundError (file:///C:/Users/yu/.atom/packages/atom-beautify-master/src/beautifiers/beautifier.coffee:204:14)
at file:///C:/Users/yu/.atom/packages/atom-beautify-master/src/beautifiers/beautifier.coffee:304:22
at tryCatcher (C:\Users\yu.atom\packages\atom-beautify-master\node_modules\bluebird\js\release\util.js:16:23)
at Promise._settlePromiseFromHandler (C:\Users\yu.atom\packages\atom-beautify-master\node_modules\bluebird\js\release\promise.js:510:31)
at Promise._settlePromise (C:\Users\yu.atom\packages\atom-beautify-master\node_modules\bluebird\js\release\promise.js:567:18)
at Promise._settlePromise0 (C:\Users\yu.atom\packages\atom-beautify-master\node_modules\bluebird\js\release\promise.js:612:10)
at Promise._settlePromises (C:\Users\yu.atom\packages\atom-beautify-master\node_modules\bluebird\js\release\promise.js:687:18)
at Async._drainQueue (C:\Users\yu.atom\packages\atom-beautify-master\node_modules\bluebird\js\release\async.js:133:16)
at Async._drainQueues (C:\Users\yu.atom\packages\atom-beautify-master\node_modules\bluebird\js\release\async.js:143:10)
at Async.drainQueues (C:\Users\yu.atom\packages\atom-beautify-master\node_modules\bluebird\js\release\async.js:17:14)
at process._tickCallback (internal/process/next_tick.js:103:

I have already installed.Well, the mouse right button has this menu.but it dosen't working.

When you type where.exe uncrustify in your command prompt what do you get?

This command is used ,and then the tip is used to provide the mode,unable to find the file.
"C:\Users\yu>where.exe uncrustify
信息: 用提供的模式无法找到文件。"
@prettydiff

same issue:(

me 2 :(

I can safely say that the build is not passing... furthermore, The PATH env:var has the exe available, as I can execute it through the Command Line.

Are there any sspecific solutions? Or having a corresponding course？

Same issue

Same issue

I encount the same problem. After trying many times, I give up installing uncrustify in Atom. It is difficult to work right.

Hi, I find a solution. My OS is windows 7.

download uncrustify-0.64-win32.zip, which contains .exe file
unzip zip file, and copy it in disk d, i.e., D:\uncrustify-0.64-win32
next set environment path, i,e.,Path = ...; D:\uncrustify-0.64-win32
so, OK

I think this problem is caused by windows access authority， it is possible that Windows OS can't permit atom to invoke .exe file in Disk C.
I hope you to give better suggestion

Yes, your method is very useful. @jiangwest
Thanks you ,you suggestion is working.

I really want to focus on improving the installation experience for users. I have created a new Issue, #1687, to target this problem. Please provide your feedback! Thanks in advance.

Go there and the issue is CLOSED. Something about Dockers. 20+ years in IT and never heard of them. Here's an idea: what about a little program that puts all the files in the right place and makes the necessary config file changes? To make it easy to remember we can call it an 'installer' lol

Go there and the issue is CLOSED. Something about Dockers. 20+ years in IT and never heard of them. Here's an idea: what about a little program that puts all the files in the right place and makes the necessary config file changes? To make it easy to remember we can call it an 'installer' lol

Something about Dockers. 20+ years in IT and never heard of them.

You may find this article helpful: https://www.docker.com/what-docker

Here's an idea: what about a little program that puts all the files in the right place and makes the necessary config file changes? To make it easy to remember we can call it an 'installer' lol

Feel free to recommend this innovative idea to the Uncrustify maintainers over at https://github.com/uncrustify/uncrustify
Since installing these third-party beautifiers, such as Uncrustify, is out of scope for Atom-Beautify, it is up to developers like yourself and Uncrustify maintainers to implement the means of getting them installed and supporting Windows, Mac, and Linux.
This is a great application of Docker so I have integrated it into Atom-Beautify and created https://github.com/Unibeautify/docker-beautifiers such that installation is a way of the past, provided Docker is installed. However, if Docker is too new or unfamiliar of a technology for you, then the conventional manual installation methods (i.e. using an "installer" or otherwise) are always available to you and provided by the third-party beautifier maintainers. Once again, writing these installers is out of scope for Atom-Beautify itself.

The point was that if it's new to me, it's going to be new to a lot of people. Many are already struggling and frustrated and you're just tossing another layer of confusion and ways for things to go wrong. Was: is it OS, Atom, Atom-Beautify, or Uncrustify. Now:  is it OS, Atom, Atom-Beautify, or Uncrustify ...or Docker or whalebrew or the Docker image or whatever support is required to get this all to work together seamlessly.
Doesn't matter as I'm one of only 94% of computer users who don't use Mac/Linux. Whalebrew: "Windows support is theoretically possible, but not implemented yet."
I, and many others here, have followed the manual instructions and they don't work reliably. I would suggest that the simplest answer may be to fix the instructions.
Thanks

I, and many others here, have followed the manual instructions and they don't work reliably.

If the manual installation instructions for Uncrustify are insufficient then I encourage you to contribute to https://github.com/uncrustify/uncrustify and improve their documentation. Same goes for Atom-Beautify's automatically generated installation instructions: https://github.com/Glavin001/atom-beautify/blob/master/docs/index.coffee#L207-L226

Doesn't matter as I'm one of only 94% of computer users who don't use Mac/Linux. Whalebrew: "Windows support is theoretically possible, but not implemented yet."

Whalebrew is not required for Atom-Beautify, only Docker. Whalebrew is optional and a nice-to-have for those who wish to use it. Docker is the requirement, provided the user wishes to try it instead of going the conventional manual installation instructions path.

Many are already struggling and frustrated and you're just tossing another layer of confusion and ways for things to go wrong.

Yes, I know many are struggling and there is only so much which can be done.
I propose to you this problem: you need to write a cross-platform (Windows, Mac, Linux) installer which supports multiple types of programs/executables (scripting languages, compiled languages, prebuilt binaries which are architecture specific, etc). To make the problem more concrete, consider Uncrustify: on some distributions of Linux you can run apt-get install uncrustify, on Mac you can run brew install uncrustify (and only if Homebrew is already installed), on Windows you can follow the instructions recommended in https://github.com/Glavin001/atom-beautify/issues/772#issuecomment-216685220 . One program (i.e. Uncrustify) and multiple different installation processes. Once again, all of which are out of scope for Atom-Beautify. Atom-Beautify intends to use Uncrustify, not handle its installation.
Let's introduce Docker and see what we can do now. After installing Docker (which has an official "installer" for many Operating Systems) we can simply run docker pull unibeautify/uncrustify and utilize the pre-built Docker image generated from https://github.com/Unibeautify/docker-beautifiers . Thus, the cross-platform issue becomes a non-issue. This is why the new Docker layer was added to Atom-Beautify: to target the very problem which leads you to search for this Issue on GitHub in the first place.
Now I understand Docker is new and unfamiliar, and so it is by no means required. However, I trust you can understand -- with a little bit of research -- it's purpose and how it applies in this situation. Also, I hope you understand how building a universal "installer" is non-trivial.
That being said, if you build such a thing, by all means, please submit a Pull Request, as I would love to improve the installation experience for all users!

In the end, if you have a problem the solution is always to contribute. If you do not like the documentation, please submit a Pull Request to improve it. If you do not like having these many layers in place, please feel free to suggest a new idea which meets the growing needs of the users. This is open-source software which I develop in my personal time along with other open-source contributors. We do our best. You have the ability to make changes here, just as we do. Let's work together and improve this.

I would suggest that the simplest answer may be to fix the instructions.

I agree. I look forward to seeing your Pull Request(s).

I cannot improve the procedures if i can't get it to work myself. I spent another hour this afternoon googling for solutions online and trying to figure it out myself.
The Docker website does not list Win7/8 support only win10 so how many hours should I spend trying to get it to work before I find the bug(s) in the Docker non-install are unresolvable OS compatibility issues?
I've been working with Atom + Platformio + Atom-Beautify + Uncrustify + stm32duino for just a little over a week trying to get it all to work together. Part of that was spent testing new board support for Platformio with one of the developers. Not having seen any of it work at all and not knowing where the boundaries of the products lie, it's a bit challenging to debug it. Once all the obvious errors are gone, i have no good way of telling if it's doing what it's supposed to do or not. Is it a bug or a feature? lol
And on top of having to actually write working code on a new microcontroller- "is it my code or the dev system or the microcontroller or the external hardware?"
All i want is indents that line up and at the moment doing it by hand is quicker than troubleshooting layers and layers of 'stuff' that even more experienced and knowledgeable people cannot get to work :-(
Thanks for your efforts

I looked at Docker. It sounds great for some applications- no argument there. But as a way to get a plug-in installed into a text editor....? I'm not going to install a virtualization environment just for that. I couldn't if I wanted to. They are not planning on supporting Win7/8. "The current version of Docker for Windows runs on 64bit Windows 10 Pro, Enterprise and Education... In the future we will support more versions of Windows 10." Win 10 total is only 25% of the market and I'd guess the great majority of those are Home users.
https://docs.docker.com/docker-for-windows/install/#what-to-know-before-you-install

this totally confusing , the problem is NOT if it is complicated or simple , the problem is the lack of clarity in my opinion.
How to install the docker image? , almost all beautify images didn't pass the docker build , as stated on the page linked .
So i guess , the first question to be answered is : Is there any Atom - platform Beautifier working ?
Then , in case yes , what is the best step by step procedure to install it , as the normal add-on installation doesn't work .
thank you


Fail fast when "Error with page"

We recently had a huge YML file (taking many minutes to capture) - where one of the URLs was bad (causing phantomjs to console.log "Error with page"). The URL was bad in that the server was inaccessible (due to redirect) so it would be impossible for Wraith to take any screenshot.
However, the first the tester knew about this was right at the end, when Wraith attempts to generate a gallery. We get an obscure error on gallery.rb:114 instead.
It would be good to fail fast by killing Wraith as soon as any of the paths are bad, given that Wraith otherwise has an impossible task ahead of it.

That would be great, I recently came across with the same problem.

indeed, would love this feature too


Example on how to automatically refresh the browser when Blazor Server reconnection fails

Example on how to automatically refresh the browser when Blazor Server reconnection fails
This PR adds an example showing how to configure a Blazor Server app to automatically refresh the browser if reconnection to the server fails.
Description
Customers have been asking for a way to keep the existing Blazor Server reconnection logic and UI, but modified slightly so that refreshing the browser happens automatically rather than requiring direct user action. We don't have an extensibility point in our current JavaScript API that allows this do be done without reimplementing the full reconnection logic and UI. That said, it's not that much code, so it could be sufficient to provide an example in the docs showing how to achieve this.
I'm opening this initially as a draft PR because there are a few open questions:

How complete should the example code be?

Should we make the example more complete by including some example CSS for the reconnect-modal?
Or, should we take the opposite approach and reduce the example by simplifying the reconnect functionality?

I'm personally not in favor of this, because the customer ask is to retain existing reconnect functionality but with automatic browser refreshing.

Is pasting the code inline the best approach?

While the code is straightforward, it's a bit on the longer end. Should we put this code elsewhere and link to it from the docs?

Rather than having a docs example for this at all, should we address https://github.com/dotnet/aspnetcore/issues/32113 and add new APIs to enable this scenario without requiring reconnection logic to be reimplemented?

Fixes https://github.com/dotnet/aspnetcore/issues/44715

@MackinnonBuck It looks like https://github.com/dotnet/aspnetcore/issues/32113 may be a duplicate of https://github.com/dotnet/aspnetcore/issues/44669, which is already being considered for .NET 8. Thoughts?

Should we make the example more complete by including some example CSS for the reconnect-modal?

Your call; but if you do include it and want to cross-link, we cover that at ...
[Reflect the connection state in the UI (Blazor Server)](#reflect-the-connection-state-in-the-ui-blazor-server)

@MackinnonBuck It looks like dotnet/aspnetcore#32113 may be a duplicate of dotnet/aspnetcore#44669, which is already being considered for .NET 8. Thoughts?

@danroth27 I think those seem like two different issues.
From my understanding, the former is asking for an extra onReconnectionFailed event on the default reconnection handler so you can add the automatic refreshing behavior in just a few lines of code.
The latter seems to be asking for a way for Blazor.reconnect() to signal whether reconnection failed due to the circuit ID being invalid or due to something like the server being down. Addressing this would still require that customers provide a full implementation of the reconnection logic in order to automatically refresh the browser on reconnection failure.
But now that you called out https://github.com/dotnet/aspnetcore/issues/44669, I think this is something we already have support for, and it's what my sample in this PR utilizes. Our own default reconnection handler even handles both cases explicitly ("failed" vs "rejected"). Maybe that issue can be closed? CC @SteveSandersonMS

@MackinnonBuck ... You might be waiting on Steve to look. I'll let you pull the trigger on merging this when ur ready.


Cu-1z64abr Display the names of subjects on the instrument panel

Task linked: CU-1z64abr Display the names of subjects on the instrument panel

Vital lesson learned: React is brittle.


admins in default zone should be able to invite users in other zones - Story Id: 127027759

Link: https://www.pivotaltracker.com/story/show/127027759
Labels:
Requester: Kevin Huang
Estimate: 0.0

As an administrator and zone administrator, I want to be able to invite users to the current zone.
Acceptance:

An admin can successfully call the invite_users endpoint
An zone admin can successfully call the invite_users endpoint

Link: https://www.pivotaltracker.com/n/projects/997278/stories/127027759/comments144847185
User: Kevin Huang

@stummidi @madhurab
Comment Id: 144847185


Feature : LDAP authentication.

Do we have LDAP feature?

Hi @bhushangahire,
no not at the moment. If you are willing to, send a PR and discuss it with @melvinsembrano.
All the best
Andy

Ok thanks will see what I can do.


opt prune runtime redundant filters

Proposed changes
after set expand_runtime_filter_by_inner_join, RFGenerator generates more rfs, and hence there are some rfs have the same filter effect. This pr detect the redunant rfs, and remove one of them
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Istio controller must process only k8s ingress resources annotated with "istio"

This is to allow co-existence of other ingress controller in a cluster.

@andraxylia: it looks like the NGINX and GCE ingress controllers consider non-annotated ingress resources fair game:
NGINX:
https://github.com/kubernetes/ingress/tree/master/controllers/nginx#running-multiple-ingress-controllers
GCE:
https://github.com/kubernetes/ingress/blob/116fbe8c33b5d51b825038d4c7f2a481ab7cee74/controllers/gce/controller/utils.go#L115

This is a known problem with ingress controller design in k8s. Doing what other k8s controllers do seems reasonable.

We can support the two behaviors with a flag, e.g.:
--ingress_class="istio"            // specifies the ingress class annotation to match
--ingress_class_strict=false    // specifies whether only annotated ingress resources should be processed

With non-strict being the common behavior for ingress controllers, and strict being the desirable one (IMO).

The ingress controller processing the ingress resource, assigns it an IP:port equal to the ::
loadBalancer:
ingress:
- ip: <IP_ADDRESS>
$ kubectl get ing
NAME            HOSTS     ADDRESS         PORTS     AGE
basic-ingress   *         <IP_ADDRESS>   80        18h
This is the address where traffic coming from outside of the cluster should be sent. Traffic should not be sent directly to the ingress controller service, like it is currently the case with the bookinfo demo. The user is not supposed to know about the ingress controller service name or other internal details.
With multiple ingress controllers in the same cluster, we can annotate the ingress with "kubernetes.io/ingress.class" to ensure a single controller processes the ingress. However, if the annotation is not set, and more then one controller processes the ingress and sets the IP address, we end up with a race condition.
Note that, unlike nginx,  traefik controller does behave, there is long thread here on the same topic:
https://github.com/containous/traefik/issues/1163
I checked with Tim Hockin and the behavior of the nginx controller is incorrect, and it has to be fixed. Each controller has to know for which class it is responsible. One of the controller has to be the default.
To me, it makes sense to have the default controller be the cloud LB. For deployments in public cloud, this relieves the user from knowing the type of cloud LB, and eases portability between clouds, since no change will be needed in the yaml files.
I re-open this issue to fix three things:

Istio default behavior should be to only process istio annotated ingress resources. The extra config flag can be enabled to allow istio to process non-annotated resources. Since this is related to the fact that istio is the default LB, I propose this to be renamed --default_lb=true. This is easier to understand than "ingress-class_strict" or "parse_annotated".

Assign an IP address:port to the ingress resource

Fix the bookinfo demo to use ingress resource IP and not the external IP of the istio-ingress-controller service

4)optional: Rename istio-ingress-controller to istio-ingress-lb. In fact, that container runs both management & data plane. The mgmt plane is build with a control loop, similar with k8s controller, and it is a k8s ingress controller, no doubt,  hence the name.  But it is in general an HTTP LB, and traffic coming from outside the cluster does pass through the envoy proxy/data plane running inside istio-ingress-controller.

@andraxylia

I'll add the --default_ingress_controller=true and --ingress_class="istio" flags. Note the flag rename, as explained in (4) below.
This will be covered in #214.
Depends on #214, therefore not part of this issue.
Given that the container runs both management and data plane functions, but can have only a single name, I propose we stick to "ingress controller", so that we're consistent with kubernetes terminology. IMO, "ingress lb" mixes up two distinct (although related) kubernetes concepts, which might lead to confusion.

can we close it?

We can close it, since @zcahana will address all the mentioned points as part of #214 .


[LOGISTICS] Set up automatic testing

What needs to happen in the codebase?
Testing needs to be setup in a Github action
Describe the solution you'd like
How we did it for hw2b would be solid.

This include the badge in the readme


Nick: Watch (?:cn.)?etherscan.com

Nick requests the watch of the watch_keyword (?:cn\.)?etherscan\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

(?:cn\.)?etherscan\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Rejected by cigien in Charcoal HQ. Reason: 'several FP hits across SE'


Report Generator for Screengrab

Hi,
Are you going to include Reports Generator (to html) in Screengrab? This would be a great.
I could contribute to this feature and send PR.

Hi @maszin 👋  - Yes, that is something we'd like to have. If you'd like to make the PR, I would be happy to review it 👍

Implemented in #9213 🎉

Thanks 👍


missing downward inference involving lists & typedefs

class DartType {}
typedef void Asserter<T>(T type);
typedef Asserter<T> AsserterBuilder<S, T>(S arg);

main() {
  AsserterBuilder<List<Asserter<DartType>>, DartType> assertAOf;
  Asserter<DartType> _isInt;
  Asserter<DartType> _isString;
  assertAOf([_isInt, _isString]);
}

produces:
$ ./xcodebuild/ReleaseX64/dart-sdk/bin/dartanalyzer --strong test.dart
Analyzing [test.dart]...
[error] Type check failed: [_isInt, _isString] (List<dynamic>) is not of type List<(DartType) → void> because [_isInt, _isString] cannot be typed as List<(DartType) → void> (test.dart, line 9, col 13)
1 error found.

Not sure what's going on there.

(we just chatted, I think I figured out what's going on here)

https://codereview.chromium.org/1513603002/


Ajay Joshi WMS 11453 OraGitPush1:main Data Sovereignty Lab uds23c folder initial upload

WMS 11453
General requirements

Information in the workshop is adequate and updated
Code is correct and working
Links are correct
Make sure you enter the help email link in your manifest.json
Please make sure WMS URLs are updated as needed after your PR is approved

Checklist - Refer to the QA document for the complete list
Please confirm that the following is completed before submitting your PR

[X] All filenames are lower case (including folders, images, files, etc.)
[X] Filenames are descriptive
[X] Your workshop folder structure should be similar to the one used in the sample workshop (https://github.com/oracle-livelabs/common/tree/main/sample-livelabs-templates/sample-workshop)
[X] Are you using multiple versions (desktop/, sandbox/, tenancy/)? Make sure that each of them contains a manifest.json and an index.html
[X] Image references in markdown contain an alternate text

Ajay Joshi WMS 11453 OraGitPush1:main Data Sovereignty Lab uds23c folder initial upload #366

Updates for QA check

WMS ID 11453 for all updates in this pull request


Implement standard retry policies for failed API requests

It seems that SDK supports one-time retry out of the box for failed requests. We need to support more sophisticated policies, especially for internal clients. I suggest to implement the following policy as a default retry policy:

For HTTP 4xx errors (except HTTP 429) no retries are made
For 429 and 5xx

Exponential backoff policy with jitter
Initial retry interval is configured: default = minimum = 1 second)
Number of retries is configured: minimum = 0, default = 3 retries, maximum should be limited, e.g. 20)
Jitter randomization  factor is configured: minimum = 0 (no jitter), default = 0.3, maximum =0.5
If "Retry-After" header is returned by backend it has to be obeyed (current retry interval must NEVER be less than returned "Retry-After")

Client can define global policy or customize per API call.
SHould be eventually added to other SDKs as well.
See also:

https://aws.amazon.com/blogs/architecture/exponential-backoff-and-jitter/
https://www.baeldung.com/resilience4j-backoff-jitter

@embbnux Could you please add some comments about how integration team handles 429 issue in some of the popular production integration apps? We would like to take advantages of your experience. Thanks.

The JS widget lib listen to 429 error, and show a message about the error to user with a time counter (from Retry-After) in UI. After time go to zero, it will request the API again.


formatCodeOptions are ignored

Whatever I specify in formatcodeoptions if tsconfig is ignored during transpilation.
For example specifying ""indentSize": 2" is ignored and typescript keeps transipiling with 4.
Thanks

For example specifying ""indentSize": 2" is ignored

The formatCodeOptions are to format TypeScript (and not the generated JavaScript). Apprently the generated JavaScript is always 4 indent (you can double check / feature request from the team if you want https://github.com/Microsoft/TypeScript/issues/new) :rose:


Send Realtime Frame to python project using Socket❓

Question
i have a react native app with frame processor enabled. I have a senerio where i want to send the frame image to a python app using socket. i have try to convert the frame to base64 image but could able to achieve it as the community library is not maintained which cause issue while building the app. I just want to know what is the best to achieve the senerio of sending the image to python app without dependent on any external library.
What I tried
No response
VisionCamera Version
3.3.1
Additional information

[ ] I am using Expo
[X] I have read the Troubleshooting Guide
[X] I agree to follow this project's Code of Conduct
[X] I searched for similar questions in the issues page as well as in the discussions page and found none.

Hey, you can easily create a simple native Frame Processor Plugin inside your app for this. This is just very few lines of native code that can efficiently convert the Frame and send them over the Web


proposal: Split services into separate repos

[x] coreutils should be moved to another repo
[ ] internal/cmd/gomod sholud be moved to cns-io/tools maybe
[x] internal/cmd/definitions should be split into several parts so that other repos can use it as well
[x] services/* should be moved to different repos

Fixed in #419


Bug

──(root㉿kali)-[~]
└─# ghost
Traceback (most recent call last):
File "/usr/local/bin/ghost", line 8, in
sys.exit(cli())
^^^^^
File "/usr/local/lib/python3.11/dist-packages/ghost/init.py", line 35, in cli
console.shell()
File "/usr/local/lib/python3.11/dist-packages/ghost/core/console.py", line 225, in shell
self.badges.print_empty(self.banner, translate=False)
File "/usr/local/lib/python3.11/dist-packages/badges/main.py", line 44, in print_empty
self.print(*args, **kwargs)
TypeError: IO.print() got an unexpected keyword argument 'translate'

Fixed, please reinstall


add proxied generator to access Parameters and Seed within a Gen

This generator will pass the Parameters and Seed supplied from the outer generator to the inner/proxied one. I needed to access the framework generated seed in one of the generators I was building it would have been useful to have this function. I'm not sure if it is really necessarily useful for anyone else. Happy to hear thoughts on this even if this PR is not merged.

Shouldn't withPerturb allow you to access the generated seed?

@ashawley you're right! Not sure how I missed it! Thanks!


Track tax - Track tax rate is disabled when changing unit

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.80-1
Reproducible in staging?: y
Reproducible in production?: no new feature
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause internal team
Slack conversation:
Action Performed:
Precondition: Enable 'Distance rate' and 'Taxes'

Workspace Settings page > Distance Rate > Settings
Enable Track tax
Change Unit and wait in the screens for few seconds

Expected Result:
Track tax should remain enabled
Actual Result:
Track tax is disabled
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
https://github.com/Expensify/App/assets/38435837/1e96c44a-447a-4d19-a977-c55b37848dfa

View all open jobs on GitHub

@trjExpensify FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

We think that this bug might be related to #wave-collect - Release 1

Checking.

Hmm we are returning null for taxEnabled when changing the unit, so not a blocker for the App.

Oh, actually it seems that we should send the taxEnabled attribute when updating the distance rate unit.

cc @nkdengineer @eVoloshchak

Was just looking for the original PR, thanks!
Anyway, I don't think this needs to be a blocker, do you agree @MonilBhavsar ?

Removing the blocker from Web-E as this has to be solved in the FE.

This is going to be demoed at Xerocon, and app looks buggy with this. So I think we should fix it cc @trjExpensify

Asked for CP here https://expensify.slack.com/archives/C07J32337/p1717688056480579

Thanks @rlinoz 🙇 ⚡ !

Tested the CP and it worked, so closing.


Vue and Vuex 2.0

Vue and Vuex 2.0 are currently in RC, you have plans for migrate cloudradioo to this new version?
Has some breaking changes with Vuex, like the deprecation of vuex inside components
Vuex 2.0
Vue 2.0

I have been following the changes and the discussions for vue and vuex, and i will definitely migrate cloudradioo to version 2.0.
However, I have currently very little time to work on cloudradioo (Again, thank you for the notify pull request!).


Added new information on README.md

This helps new contributors to understand more about the project.

Ah, I forgot to send the favicon one to another pull request.


A shorter release this time?

Our last release took longer. IMO releases should be frequent for OSS projects to formalize recent changes.
We have had some decent set of features of going in till now (not as much as 1.8 though). Would the maintainers be interested in releasing now ?

Am I the only one who can push a new release? I think at least in cocoapods I'm the only one, but I think I can add other maintainers too.
@piv199 @lfarah
Ready for release, agree/disagree?

I also agree we took to long to push a new release. You can also add us as mainteiners

Just so we're organized, can we add things to the project

@Khalian did I miss anything? If not, please make a PR changing the CHANGELOG.md``` to 1.9and add a1.9` tag, so @goktugyil can push it to CocoaPods

Yeah, that should we about it, I will try to post out the fixes and update the changelog and send a CR. Also tag the release to 1.9.

Perfect. Thanks @Khalian 👍

@lfarah You have pod access now, you should push 1.9 to learn how things are.
Heres a checklist: https://github.com/thellimist/EZPods
@Khalian @piv199
I got this error message:
[!] No owner found with the specified email address.
Its probably because you don't have a library or registered in cocoapods.

I haven't been watching for a while, so I can't tell about release :) If it's needed then do.
I wish I could blend in the project asap. I was really missing the open source.

@lfarah we need a pod release

@Khalian we have a problem tho: we forgot to update the podspec, so we gotta remove tag 1.9 and add it again since we need to add this commit


不支持H5的drag/drop？

浏览器测试：正常
https://github.com/user-attachments/assets/cc96be01-7a21-4cb7-9e1a-86adf1d437af
energy2.4.4 测试 ：被禁止放置
https://github.com/user-attachments/assets/d1d380b6-e44d-4f51-9b2e-eafbba0206f9
测试的HTML：
`
  |
-- | --
  |
  |
  |
  |
  | Draggable List
  |
  |
  |
  |
  |
  | Item 1
  | Item 2
  | Item 3
  | Item 4
  | Item 5
  |
  |
  |
  |
  |
  |
`

It is possible to implement drag-and-drop functionality.

Reason: Unlike CEF and browsers, browsers have already implemented most of the features used by most users. CEF, however, is more geared towards developers. Some features in CEF are not fully implemented like they are in browsers, and we need to implement them ourselves through the CEF API.
This allows us to have more flexibility in controlling the browser to implement custom features.

Your problem is solved by implementing the SetOnDragEnter callback event
SetOnDragEnter :
	cef.BrowserWindow.SetBrowserInit(func(event *cef.BrowserEvent, window cef.IBrowserWindow) {
		event.SetOnDragEnter(func(sender lcl.IObject, browser *cef.ICefBrowser, dragData *cef.ICefDragData, mask consts.TCefDragOperations, window cef.IBrowserWindow, result *bool) {
			// mask is const
			if mask == consts.DRAG_OPERATION_MOVE {
				*result = false // can drag
			} else {
				*result = true
			}
		})
	})

demo - drag-file


cocomac: Watch welcome to stack overflow buddy

cocomac requests the watch of the watch_keyword welcome to stack overflow buddy. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

welcome to stack overflow buddy has been seen in 2 true positives, 0 false positives, and 0 NAAs.

!! REJECT ME PLEASE !!


DeepOmit index

my type has
    [IS_AUTO_COMMIT_ENABLED]: boolean;

how do I omit that, I've tried a few things, but no luck
this is the declaration from a library
declare const IS_AUTO_COMMIT_ENABLED: unique symbol;
export declare abstract class AggregateRoot<EventBase extends IEvent = IEvent> {
    [IS_AUTO_COMMIT_ENABLED]: boolean;

ask and ye shall figure it out yourself in the next 10 minutes
    [index: symbol]: never;


Housekeeping: Improved readme samples and descriptions.

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Readme updates include:

Including a new section for Android extensions and a sample.
A simplified generic sample for validations.

What is the current behavior? (You can also link to an open issue here)
The sample is too complex with many references.
What is the new behavior (if this is a feature change)?
Simplified sample.
Please check if the PR fulfills these requirements

[ ] Tests for the changes have been added (for bug fixes / features)
[X] Docs have been added / updated (for bug fixes / features)

Other information:
I would like to add a sample for ValidationHelper property in a ViewModel and binding to that from the View, but I think that is unnecessary.

Looks good otherwise


Add a check for the setting of watch flag in config file

Fixes #1074

Oh, and one last thing ... You do very good work, but remember to run "go fmt" on the code before commit.
I use LiteIde for Go coding, but I know other editors too have "go fmt on save".

Rats!  Sorry, I could have sworn I did.  I will double check the README before my next commit.  Thanks for your patience.

Closing this PR; new one opened #1286.

Closing and open a new PR is good for the project, as it shows a lot of activity ... but:
I notice you do your changes on your own master branch.

Please create a new branch for every new pull request (a Git Branch is cheap)
When doing changes, just do a "commit --amend" or similar followed by "git push -f" (force push, nobody cares about the history of a temporary branch)

Got it. Thanks.


Common: create dry-run for vagrant push kinesis

To allow testing of the process without Bintray write access

there is cd, closing


npm package seems to be down

npm ERR! code E404
npm ERR! 404 Not Found - GET<EMAIL_ADDRESS>npm ERR! 404
npm ERR! 404  '<EMAIL_ADDRESS>is not in the npm registry.
npm ERR! 404 You should bug the author to publish it (or use the name yourself!)
https://www.npmjs.com/package/asn1

This seems to be an issue with npm, had the same problem with 'encoding'


Updated golang-ci lint to 1.46.2

If not urgent, I recommend waiting for the next release since golangci-lint has cover some useful features and work hard for release.

Sure, we can wait. I'll periodically keep this up-to-date

Closed as superseeded by https://github.com/dapr/dapr/pull/5041


issue when building on Mac (xorgxrdp doesn't find xrdp already built)

Hi,
I have tried to install xrdp on a Mac (Sierra) as mentionned in "Building on OSX"
At step "Build xorgxrdp", when doing "./configure PKG_CONFIG_PATH=/opt/X11/lib/pkgconfig", I have the following error:
checking for XRDP... no
configure: error: Package requirements (xrdp >= 0.9.0) were not met:

No package 'xrdp' found

Consider adjusting the PKG_CONFIG_PATH environment variable if you
installed software in a non-standard prefix.

Alternatively, you may set the environment variables XRDP_CFLAGS
and XRDP_LIBS to avoid the need to call pkg-config.
See the pkg-config man page for more details.

All steps before are ok. Including building xrdp.
Can anyone help me to solve this issue.
Thanks in advance

same issue for me on Mojave

Same issue on Sierra

See also Platform Support Tier.
We dev team do not provide any support for using xrdp on Mac. Users need to solve this issue by their own effort. However, I'll leave this open for users to exchange information with each other.

Same issue on Sonoma

I figured it out! I ran only this at the beginning:
./configure PKG_CONFIG_PATH=/opt/X11/lib/pkgconfig:/usr/local/lib/pkgconfig
but actually I need to run:
./configure PKG_CONFIG_PATH=/opt/X11/lib/pkgconfig:/usr/local/lib/pkgconfig
**make
sudo make install**

when I ran all the commands below, the error disappears...
——When I checked the configure script, I was still thinking "Why is there no make command?"😅


image is in high accurancy ，however video predict is extremely worse！

Execuse me，is these angbody solving my problem， thanks

Hi.
If we come to the your problem, it may have many different reason. The things that come to my mind first:

It could be that your dataset is small or doesn't have enough variety.
Object sizes may be out of range of the anchor boxes

You can save the frames of your video into the images using OpenCV, so you can test the model on these images to see if does it work better or not while they are images instead of a video?

我也是这个问题，请问怎么解决呢

recompose detect_video in yolo.py :
it is about line 190
frame=frame[:,:,::-1]
image = Image.fromarray(frame)
image = yolo.detect_image(image)
result = np.asarray(image)
result=result[:,:,::-1]
this is because cv2 cut video is not in RGB format


How to get Image file name

Please guide us how to get Filestream instead of Stream to get image file name.
Thanks!

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

ID: 22cd76be-2702-6a58-498c-a84c723d9588
Version Independent ID: 9f98549e-ff3d-5b8d-67e6-523834c4dd6d
Content: Picking a Photo from the Picture Library - Xamarin
Content Source: docs/xamarin-forms/app-fundamentals/dependency-service/photo-picker.md
Product: xamarin
GitHub Login: @davidbritch
Microsoft Alias: dabritch

Hi @thanguc - this repository is only for issues specifically related to the documentation. For support or to report bugs, please take a look at Developer Community. Thank you!


Colorwheel is not reactive in InDesign CS6

While using in InDesign CS6, it's impossible to change the tint using the color wheel : nothing is happening while clicking on it.

Photoshop CC 2020 seems to have this same issue.