Improve Travis / tox setup + coverage

Mainly inspired by using pytest directly in tox, for easy test selection
etc.

Added coverage reports via codecov, which can be seen at https://codecov.io/gh/pallets/click/pull/851/tree.

@untitaker @axu2
Can you take a look, please?

Also please authorize/install codecov on https://codecov.io/gh/pallets/click/.

I wouldn't include codecov, particularly its notification settings are
extremely buggy in my experience. I feel like it's more maintenance effort than
it brings insight
On Thu, Sep 28, 2017 at 02:14:18PM -0700, Daniel Hahler wrote:

@untitaker @axu2
Can you take a look, please?
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/pallets/click/pull/851#issuecomment-332965879

@untitaker
What do you mean by "extremely buggy"?
What I am seeing sometimes is multiple comments, also only one should be there and edited.

maintenance effort

Do you mean that you have or want to delete the duplicated comments then?
If this is the issue you mean it might be solvalble by @codecov just fixing it in the end.. ;)
From my point of view codecov provides a lot of valuable insights.

IMO even just a single codecov comment on every PR is somewhat annoying. I like getting notifications from GitHub for new PRs and comments since that's convenient to see if there's anything to comment on, a PR to review, etc. But codecov comments? I'm tempted to overcome my laziness and configure my email filter to send the notifications about them straight to the trash folder. But that aside, IMO these comments are more noise than adding useful comment.

@ThiefMaster
The recommended way is to install codecov's GitHub integration, where you would not see comments in your emails (AFAIK): https://github.com/integration/codecov.
Anyone with admin rights here could try it already.
Otherwise you could also just block the bot user (https://github.com/codecov-io), and be done with it I guess.

Especially for PRs codecov's reporting is valuable, since it will tell you if coverage drops and if the diff is covered etc.

If codecov were about to "just fix" anything they would have done that a long time ago.
On 6 October 2017 21:38:40 GMT+02:00, Daniel Hahler<EMAIL_ADDRESS>wrote:

@untitaker
What do you mean by "extremely buggy"?
What I am seeing sometimes is multiple comments, also only one should
be there and edited.

maintenance effort

Do you mean that you have or want to delete the duplicated comments
then?
If this is the issue you mean it might be solvalble by @codecov just
fixing it in the end.. ;)

From my point of view codecov provides a lot of valuable insights.

--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/pallets/click/pull/851#issuecomment-334850731

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

@untitaker
It is still the best service for it, isn't it?  coveralls is much worse and there is no other.

Hence my suggestion to not have any such service.
On 7 October 2017 00:09:43 GMT+02:00, Daniel Hahler<EMAIL_ADDRESS>wrote:

@untitaker
It is still the best service for it, isn't it?  coveralls is much worse
and there is no other.
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/pallets/click/pull/851#issuecomment-334882005

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

What about the benefit to see if a PR covers its diff?

And even in general, I think you can grasp a lot of useful info from https://codecov.io/gh/pallets/click/tree/improve-travis-tox/click already.

I never disputed that the information provided by the service would be useful. But for that it would have to work properly, and that just has not been my experience at all. Flaky reports and/or random build failures because their weird CLI tool was unable to contact S3 or whatever.
Bot comments are a non-starter, way too spammy,
On 7 October 2017 00:13:51 GMT+02:00, Daniel Hahler<EMAIL_ADDRESS>wrote:

And even in general, I think you can grasp a lot of useful info from
https://codecov.io/gh/pallets/click/tree/improve-travis-tox/click
already.
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/pallets/click/pull/851#issuecomment-334882725

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

Ah, I see you used travis-retry for that purpose, fair enough.
I guess we can merge this and also immediately block the bot org-wide. That should do it.
On 7 October 2017 00:13:51 GMT+02:00, Daniel Hahler<EMAIL_ADDRESS>wrote:

And even in general, I think you can grasp a lot of useful info from
https://codecov.io/gh/pallets/click/tree/improve-travis-tox/click
already.
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/pallets/click/pull/851#issuecomment-334882725

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

@untitaker
Thanks.

btw: it might be better to use pytest-cov here, since it would also track subprocess automatically - not sure if that is used in tests?!

Thanks! Codecov seems to be already activated.


Not working after browser update

After the self-update of Microsoft Edge, the extension is not working properly;
Here is how it looks like ;
See here
Browser Version : 101.0.1210.32
Extension Version : 1.1.16

Curretly Microsoft Edge version is 101.0.1210.39 and the extension started to work right again,
Also there was a disk repair process on boot, maybe there was few corrupted file causing that problem.


Avoid hardcoding location of backend services

While currently all hatrac, authn, and ermest services are installed under a predefined location in the same server, this assumption might eventually not be the case anymore.
Just like we have a ermrestLocation chaise-config property, we might want to include other properties for other services.
As part of this issue, we may want to experiment and see what other changes we could make in chaise/ermrestjs to allow the installation of frontend components in a different server or even locally.

We took a closer look at our code and discussed this. The following is the summary of it:

As the description mentioned, we have an ermrestLocation chaise-config. Apart from using this to communicate with ermrest, we're using this location's origin to find where export, and hatrac services are.

While these services might eventually be in different origins, we do not want Chaise to solve this on its own for now. It requires a broader discussion and solution that involves other clients of deriva. So we're going to wait to tackle this.

We just need to make sure Chaise is consistent. Currently, only the authn service is not using this chaise-config property. And that's because we're fetching the session before configuring the app. So chaise-config properties are not ready to be used. There are two ways to solve this:

Change the order of events and, therefore, the structure of AppWrapper. So the AuthnProvider is conditionally used when we're done with the configuration (the same as navbar). This requires more exploration

Directly look for the ermrestLocation that might be defined under global window.chaiseConfig object. Or in other words, get it from the chaise-config.js file and not the catalog request. This is similar to what we described in #1978, where we might want to show the navbar based on chaise-config.js so there's less "load lag".


Simulation Start Date and End Date parameters didn't work

No matter I use the  --simstartdate or --simenddate  parameters I never was able of get a period of simulation different from the period between  current_date - 300 random sets   / current date
pycryptobot --sim fast-sample --simenddate 2021-03-16
The Result (Logger.info (tradingData))
                                   date    market granularity      low     high     open    close       volume
ts
2021-06-09 01:00:00 2021-06-09 01:00:00  DOGEUSDT          1h  0.31306  0.32079  0.31706  0.31378   56854499.9
2021-06-09 02:00:00 2021-06-09 02:00:00  DOGEUSDT          1h  0.31227  0.32375  0.31378  0.31885   81523460.3
2021-06-09 03:00:00 2021-06-09 03:00:00  DOGEUSDT          1h  0.31605  0.32155  0.31885  0.31823   51623931.5
2021-06-09 04:00:00 2021-06-09 04:00:00  DOGEUSDT          1h  0.31500  0.32176  0.31825  0.32149   48966785.8
2021-06-09 05:00:00 2021-06-09 05:00:00  DOGEUSDT          1h  0.31986  0.32523  0.32147  0.32132   74383260.4
...                                 ...       ...         ...      ...      ...      ...      ...          ...
2021-06-21 08:00:00 2021-06-21 08:00:00  DOGEUSDT          1h  0.25528  0.26096  0.25700  0.25613   53692761.2
2021-06-21 09:00:00 2021-06-21 09:00:00  DOGEUSDT          1h  0.25074  0.25977  0.25614  0.25312  168043657.6
2021-06-21 10:00:00 2021-06-21 10:00:00  DOGEUSDT          1h  0.23011  0.25349  0.25306  0.23210  376215758.5
2021-06-21 11:00:00 2021-06-21 11:00:00  DOGEUSDT          1h  0.21223  0.23587  0.23214  0.23464  803087469.1
2021-06-21 12:00:00 2021-06-21 12:00:00  DOGEUSDT          1h  0.22563  0.23735  0.23454  0.22678  226139837.2

[300 rows x 8 columns]

OR
pycryptobot --sim fast-sample --simstartdate 2021-03-16
The Result (Logger.info (tradingData))
                                   date    market granularity      low     high     open    close       volume
ts
2021-06-09 01:00:00 2021-06-09 01:00:00  DOGEUSDT          1h  0.31306  0.32079  0.31706  0.31378   56854499.9
2021-06-09 02:00:00 2021-06-09 02:00:00  DOGEUSDT          1h  0.31227  0.32375  0.31378  0.31885   81523460.3
2021-06-09 03:00:00 2021-06-09 03:00:00  DOGEUSDT          1h  0.31605  0.32155  0.31885  0.31823   51623931.5
2021-06-09 04:00:00 2021-06-09 04:00:00  DOGEUSDT          1h  0.31500  0.32176  0.31825  0.32149   48966785.8
2021-06-09 05:00:00 2021-06-09 05:00:00  DOGEUSDT          1h  0.31986  0.32523  0.32147  0.32132   74383260.4
...                                 ...       ...         ...      ...      ...      ...      ...          ...
2021-06-21 08:00:00 2021-06-21 08:00:00  DOGEUSDT          1h  0.25528  0.26096  0.25700  0.25613   53692761.2
2021-06-21 09:00:00 2021-06-21 09:00:00  DOGEUSDT          1h  0.25074  0.25977  0.25614  0.25312  168043657.6
2021-06-21 10:00:00 2021-06-21 10:00:00  DOGEUSDT          1h  0.23011  0.25349  0.25306  0.23210  376215758.5
2021-06-21 11:00:00 2021-06-21 11:00:00  DOGEUSDT          1h  0.21223  0.23587  0.23214  0.23464  803087469.1
2021-06-21 12:00:00 2021-06-21 12:00:00  DOGEUSDT          1h  0.22563  0.23735  0.23454  0.22678  226139837.2

[300 rows x 8 columns]

As you may see, all the time I get the time frame between 2021-06-09 and 2021-06-21 no matther what start or end date I provided
I find out searching in models.PyCryptoBot.py that in simulation section of the code, when calling getHistoricalData  Method to fill trandingData It was missing to provide the endDate parameter:
       # run the first job immediately after starting
        if self.isSimulation():
            if self.simuluationSpeed() in ['fast-sample', 'slow-sample']:
                tradingData = pd.DataFrame()
                ....
                ....
                ....
                        tradingData = self.getHistoricalData(self.getMarket(), self.getGranularity(),
                                                             startDate.isoformat(timespec='milliseconds'))

SO I ADD IT :
       # run the first job immediately after starting
        if self.isSimulation():
            if self.simuluationSpeed() in ['fast-sample', 'slow-sample']:
                tradingData = pd.DataFrame()
                ....
                ....
                ....
                        tradingData = self.getHistoricalData(self.getMarket(), self.getGranularity(),
                                                             startDate.isoformat(timespec='milliseconds'),endDate.isoformat(timespec='milliseconds'))

And I did the same for the following elif statements in simulation section every time getHistoricalData  Method is called.
It seems now it is working propperly :
pycryptobot --sim fast-sample --simenddate 2021-03-16
                                                      date    market granularity       low      high      open     close       volume
ts
2021-03-03 00:00:00 2021-03-03 00:00:00  DOGEUSDT          1h  0.049956  0.050402  0.050402  0.050177   37592312.0
2021-03-03 01:00:00 2021-03-03 01:00:00  DOGEUSDT          1h  0.050119  0.050687  0.050159  0.050565   42081335.0
2021-03-03 02:00:00 2021-03-03 02:00:00  DOGEUSDT          1h  0.050289  0.050630  0.050577  0.050332   27460799.0
2021-03-03 03:00:00 2021-03-03 03:00:00  DOGEUSDT          1h  0.050050  0.050408  0.050320  0.050253   33783875.0
2021-03-03 04:00:00 2021-03-03 04:00:00  DOGEUSDT          1h  0.050219  0.050796  0.050254  0.050617   50004029.0
...                                 ...       ...         ...       ...       ...       ...       ...          ...
2021-03-15 08:00:00 2021-03-15 08:00:00  DOGEUSDT          1h  0.055872  0.057320  0.056289  0.056295  119631264.0
2021-03-15 09:00:00 2021-03-15 09:00:00  DOGEUSDT          1h  0.055000  0.056622  0.056273  0.055961  183610617.0
2021-03-15 10:00:00 2021-03-15 10:00:00  DOGEUSDT          1h  0.055597  0.056663  0.055921  0.056573  126304106.0
2021-03-15 11:00:00 2021-03-15 11:00:00  DOGEUSDT          1h  0.056239  0.057078  0.056573  0.056681   94111315.0
2021-03-15 12:00:00 2021-03-15 12:00:00  DOGEUSDT          1h  0.055556  0.056760  0.056681  0.056726  113589910.0

[300 rows x 8 columns]

OS: Windows
Exchange : Binance
Release : v2.37.1

Im so free to post here too, as in version 2.37.2 its still same.

Is the 300 rows limited by the api provider ? Not sure 300 gives anything meaningful in terms of simulations

No, that should not be the issue imho.

@itsaLca @cbutler99 please, get the latest (2.39.x or later) and check if the behaviour persists? Recent changes were made to fix this problem, please check if those changes work for you?

Looks good, its using my configured date for now to start with, but stops after 13 days after.

testing with; pycryptobot.py --market BTCGBP --granularity 3600 --sim fast --verbose 0 --simstartdate 2020-01-01 --simenddate 2021-01-01

starts somewhere random.
Also produces then;
Attempting to retrieve data from 2021-07-01T18:15:00.000
Exception('Data range too small.')
Traceback (most recent call last):
File "C:\Projects\pycryptobot\models\Trading.py", line 611, in addSMA
raise Exception('Data range too small.')
Exception: Data range too small.

300 candles of 1 hour is about 12.5 days, exactly what you have got.
when you use --simstartdate 2020-01-01 --simenddate 2021-01-01 with 1h granularity, what did you expect on 24 candles?

So how can i backtest lets say 1 year or more? Not possible? Can't figure it out.

this might have something to do with the exchanges only return 300 data points, for longer time intervals we need to store data locally and use that for the backtesting / simulation. not sure how much of this is already implemented.

Hi I am currently testing / building a way of testing more candles. Coinbase only returns 300 bars and with smart switching in simulation that is not much data at all. I am working on a way of downloading more bars to allow you to run longer tests.

this should work in the current version 2.49.x or later
please use --sim fast-sample --simstartdate 2020-01-01 --simenddate 2021-01-01

It looks fixed. Thks !


Random error with conv_grad_filter_ops

Please make sure that this is a bug. As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 18.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): source
TensorFlow version (use command below): TF 2.2
Python version: 3.6
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: 11.0.194 / 8.0.1
GPU model and memory: GeForce RTX 2070 SUPER

Describe the current behavior
We have a repo, with a few hundreds "unit" tests.
During the CI, in 5-10% of the runs, one of the largest tests (always the same one) is failing with the following log (repeated many times):
tensorflow/core/framework/op_kernel.cc:1753] OP_REQUIRES failed at conv_grad_filter_ops.cc:1101 : Not found: No algorithm worked!
The test suite is a mix of eager execution and graph mode tests.
This particular failing test is building a network built with CNNs (Resnet backbone, then object detection), and it's using the estimator API. Its only exotic feature is the use of RaggedTensors.
Another test is very similar (but no RaggedTensor) and has no issue.
Both these tests are running towards the end of the suite, so after a lot of models/variables have been instantiated and deleted.
Describe the expected behavior
A more informative error message.
Standalone code to reproduce the issue
Unfortunately, this error can be reproduced only randomly and with the whole repo (even changing the order of the tests can have an impact).
I'm aware this ticket is not super helpful, but I hope someone will have an idea about what's happening ...
Other info / logs
Stack trace looks like:
[2020-10-27T07:40:30.393Z] tensorflow.python.framework.errors_impl.NotFoundError: No algorithm worked!
[2020-10-27T07:40:30.393Z] 	 [[node SGD/gradients/gradients/.../Conv2D_grad/Conv2DBackpropFilter (defined at ...) ]]
[2020-10-27T07:40:30.393Z]
[2020-10-27T07:40:30.393Z] Errors may have originated from an input operation.
[2020-10-27T07:40:30.393Z] Input Source operations connected to node SGD/gradients/gradients/.../Conv2D_grad/Conv2DBackpropFilter:
[2020-10-27T07:40:30.393Z]  .../BiasAdd (defined at ...)

@jnd77
Please provide with minimal reproducible code or a colab gist with the error reported.

@Saduf2019 Thanks for looking at it.
As I mention in the ticket, it's impossible for me to reproduce the error with some minimal code. It's happening randomly even with a stable code base and the same machine.
I'm more hoping on someone being able to suggest what situation could create such error (since the message is not that helpful).
I guess in some cases, CUDA/Cudnn is in some particular state due to the accumulation of past ops (RaggedTensors in particular have changing shapes from one training loop to the next).
If you believe not much can be done to help me, then I will close the ticket.

You may want to try this hack suggested by the user.

@ymodak Thanks a lot for your message. I will try revert  to an older version of Cudnn and monitor.

@ymodak I ended up updating Cudnn to 8.0.4 (looks like the first versions 8.0 and 8.0.1 were unstable), and it looks good so far.
Thanks.

@jnd77 I just posted this workaround on the other linked issue. It may or may not be relevant for you too.
Put this at the start of your code.
from tensorflow.compat.v1 import ConfigProto
from tensorflow.compat.v1 import InteractiveSession

config = ConfigProto()
config.gpu_options.allow_growth = True
session = InteractiveSession(config=config)

Thanks @maxeonyx but that wasn't the fix for me.
allow_growth is always true in my setup via the env variable (no choice for RTX cards).


Add generated Resource Info And Management APIs.

Adding generated functions and interfaces for the
Resource Info And Management APIs.

Adding unit tests.

Resolves: OLPEDGE-2330
Signed-off-by: Oleksii Zubko<EMAIL_ADDRESS>

Codecov Report

Merging #380 into master will decrease coverage by 0.0573%.
The diff coverage is 87.6000%.

@@               Coverage Diff                @@
##             master       #380        +/-   ##
================================================
- Coverage   88.1506%   88.0933%   -0.0572%
================================================
  Files            71         71
  Lines          2152       2402       +250
  Branches        233        264        +31
================================================
+ Hits           1897       2116       +219
  Misses          161        161
- Partials         94        125        +31

Impacted Files
Coverage Δ

...-sdk-dataservice-api/lib/authorization-api-v1.1.ts
92.5197% <87.6000%> (-4.7671%)
:arrow_down:

Continue to review full report at Codecov.

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


EDM licensing information missing

[Enter feedback here]
The link "See the [data loss prevention service description] for complete licensing information" does not mention anything regarding license rights for EDM.

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

ID: e5c5906d-c5e2-dbee-7e57-9f00510bac48
Version Independent ID: e5c5906d-c5e2-dbee-7e57-9f00510bac48
Content: Get started with exact data match based sensitive information types - Microsoft Purview (compliance)
Content Source: microsoft-365/compliance/sit-get-started-exact-data-match-based-sits-overview.md
Service: o365-seccomp
GitHub Login: @chrfox
Microsoft Alias: chrfox

@chrfox Please help us in resolving this issue. Thanks


Add tests for sf::VertexArray

Description
The journey to test the Graphics module continues with sf::VertexArray. Since this is mostly just a wrapper for std::vector it was fairly simple. I was able to write all the tests without reading the implementation which is always a good sign.
Tasks

[ ] Tested on Linux
[ ] Tested on Windows
[x] Tested on macOS
[ ] Tested on iOS
[ ] Tested on Android

SFMLBuildMaster: Build this please.

SFMLBuildMaster: Build this please.

SFMLBuildMaster: Build this please.

Thanks for this new set of tests 🥳


Bug: non clear name

In name local  type in registry and array

https://github.com/goodwinxp/ida-info-code-gen/commit/50c08483d7937c975f6296ffcc06a44ccaf4aeab


Where can I learn how to use advanced APIs like SqlFragmentExpression or IMethodCallTranslator?

I'd like to know more about these advanced APIs (and others) and how I can customize EF-Core by implementing my own IMethodCallTranslator and generate some custom SQL with the SqlFragmentExpression. I'd also like to know which services are replacable and to what extent they can be customized.
Could you share some links to articles about such more advanced techniques?

@he-dev I'm not aware of any documentation or articles for these areas. I suspect your best bet is to read the code and look at the tests. Leaving this open for now to discuss with @divega as to whether or not to create a docs issue for this.

@ajcvickers I was afraid you might say that reverse-engineering is the only way. However, I'd be great if EF-Core's docs had the same quality as ASP.NET-Core where virtually every interface and service and customization is described (even with examples) and you can adjust things much more easily.

@he-dev Can you post a link to that documentation for ASP.NET Core?

@ajcvickers Here're a couple of them:
https://docs.microsoft.com/en-us/aspnet/core/mvc/controllers/filters?view=aspnetcore-2.1
https://docs.microsoft.com/en-us/aspnet/core/mvc/advanced/app-parts?view=aspnetcore-2.1
https://docs.microsoft.com/en-us/aspnet/core/mvc/advanced/custom-model-binding?view=aspnetcore-2.1
https://docs.microsoft.com/en-us/aspnet/core/mvc/views/tag-helpers/intro?view=aspnetcore-2.1
https://docs.microsoft.com/en-us/aspnet/core/fundamentals/host/hosted-services?view=aspnetcore-2.1
https://docs.microsoft.com/en-us/aspnet/core/fundamentals/dependency-injection?view=aspnetcore-2.1
https://docs.microsoft.com/en-us/dotnet/api/microsoft.aspnetcore.mvc.razor.iviewlocationexpander?view=aspnetcore-2.1
https://docs.microsoft.com/en-us/dotnet/api/microsoft.aspnetcore.mvc.razor.languageviewlocationexpander?view=aspnetcore-2.1
or an extensive list of feature-interfaces:
https://docs.microsoft.com/en-us/aspnet/core/fundamentals/request-features?view=aspnetcore-2.1
ASP.NET-Core seems to be a customize-anything product whereas EF-Core a use-as-is one. As you've said in the first response, there are no such pages about EF-Core that describe various services and interfaces to override, add, replace, implement or extend in order to customize EF-Core or provider behaviour.

@he-dev Thanks for the links. Those seem much more like mainline scenarios to me. EF has a lot of application-level configuration, but almost none of it is done through implementation of services. That's only for the most advanced things that we expect only a handful of people to do, or for people writing database providers, for which we are lacking some documentation. When we find something that a lot of people need, we try to make a good configuration experience that does not require replacement of services.


[I #3373] edit recon service url

Fixes #3373
Changes proposed in this pull request:

Add icon to edit reconciliation service
Select the changed reconciliation service after a successful change

Hi @v1scal, are you still working on this?

Closed per inactivity.


Change 'upcoming only' to 'upcoming and ongoing'?

I have a lot of conflicts in my bookmarks. During online FOSDEM it is less problematic as there is no need to walk between rooms/buildings.
When I watch session X which ends at 15:30 I want to switch to one which is 15:20 - 16:00. But as is 'ongoing' session it is not displayed in Bookmarks view anymore.
What do you think about doing such change?

I changed the feature name to "hide past sessions". The meaning is easier to understand, and sessions in progress will now always be shown.


Manifest-Metadata-Consistency not considering root nodes

https://github.com/microsoft/winget-pkgs/pull/138983

In the above PR, the Manifest-Metadata-Consistency label was applied for

=== manifests\f\Fly-io\flyctl\0.2.2 ===
Missing Properties value based on version 0.1.76:
NestedInstallerFiles

However, the commit did have NestedInstallerFiles, it was just placed at the root node instead of the installer node

[Policy] Area Validation Pipelines
[Policy] Area Bots

https://github.com/microsoft/winget-pkgs/pull/139940#top


permission

I am trying to add my mate to be able to use permission as he cant like turn up the volume up or restart the bot with the commands but it wont let him do anything other than skip songs and play songs to here is the permission can anyone help [MusicMaster]
; GrantToRoles =578618650223706132
; UserList =578618650223706132
MaxSongLength = 10
MaxSongs = 10
MaxPlaylistLength = 0
MaxSearchItems = 20
AllowPlaylists = yes
InstaSkip = yes
Remove = yes
SkipWhenAbsent = no
BypassKaraokeMode = yes
Extractors =
and this is what it says
578618650223706132/Colonel_Erax#4090: /volume 100
[ERROR:bot] Error in volume: PermissionsError: You don't have permission to use that command.
Reason: This command is not enabled for your group (Default).
Traceback (most recent call last):
File "C:\Users\home\Desktop\discord bot\MusicBot\musicbot\bot.py", line 2788, in on_message
expire_in=20)
musicbot.exceptions.PermissionsError: This command is not enabled for your group (Default).

Uncomment the lines by removing the ;  from the start of the lines with id's


BigQuery Transport Basic Schema Evolution Support

Since it sounds like we agree on the approach and implementation for the most part, any objections to calling this PR accepted and continue on with reviewing the rest of the planned work in the next PR? Sounds like we have plans for a 1.0.2-18 release with @r2d2li's changes. In that case, I can merge this PR into the next version release branch, wayfair_release_1_0_2-19, and rebase my next PR on top of that for a future release.


Investigate why results vary with n_agents

Try checking population sizes with Starsim with demographics but no diseasees

Make an analyzer that prints out the average number of edges per agent in the first timesteps

added devtests and confirmed this has been resolved


updates-for-new-java-and-flutter-versions

Setup script bash setup.sh android updates Flutter which requires newer versions of these libraries.

lgtm @audriu 🥳


ADD 1.19 COMPATIBILITY

Describe the Suggestion
i want to use mangrove wood and stuff in my builds with the create mod, BUT THE ONLY VERSION AVAILABLE IS 1.18.2
Screenshots and Videos
No response
Additional Context
No response

eventually


update bundle check to consider release candidate versions to be valid

Currently the bundle check considers the following bundle name as invalid crc_libvirt_4.18.0-ec.2_amd64.pr980.crcbundle, it expects the version to be x.y.z without containing the -ec.2 part and the suffix pr980 should have the separator _ instead of a .
the CI on snc [0] is failing due to this strict checking, i think we can update the validation check to allow the version formats x.y.z-rc.1
[0] https://prow.ci.openshift.org/view/gs/test-platform-results/pr-logs/pull/crc-org_snc/980/pull-ci-crc-org-snc-master-e2e-snc/1848580407633645568

I think it is happen because of recent work @redbeam did around validation. @redbeam can you please take a look into this ?

@praveenkumar @anjannath We can modify the regex to allow additional characters in the version specification. But I'm not sure how to interpret the suffix - currently there is the CustomBundleSuffix that accepts just numbers. Is the pr980 something different than the existing suffix? Or is it just a variant of it?

@redbeam this is something different which is used as part of CI bits so we need to modify the regex to allow that also -ec.2 or -rc.0 can be part of bundle string.

@praveenkumar But what about the suffix? What does pr980 mean in this context?

@praveenkumar But what about the suffix? What does pr980 mean in this context?

pr980 is also the custom suffix, the CI job is just adding the pull request number at the end of pr, so we could modify the regex to accept alphanumeric for the custom suffix instead of just numbers, or we could instead update the snc job to create the custom suffix containing only numbers
and the separator between the bundle name and the custom suffix is currently a . while the regex expects it to be a _ for this i think best to update the job on the snc side, here: https://github.com/crc-org/snc/blob/b4d9754b20d5636d62eea2c43e42100403048ce0/createdisk-library.sh#L9

We can remove pr from the  suffix and have only the number and use _ but we still need to update our version regex to make sure it accepts ec, alpha , rc ..etc.

@anjannath Updating the job on snc side would be fine, but if there are any other places that could put their own suffix, maybe then we should change the regex.
@praveenkumar I will submit a PR with this change.

@praveenkumar after we merge #4413 you will probably need to rebase crc-org/snc#982 to make the pipelines pass

fixed in #4413


Fix minor typos in sharing_components.rst

What does this PR do?

Fix minor typos in sharing_components.rst.
Does your PR introduce any breaking changes? If yes, please list them.

None
Before submitting

[ ] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes? (if necessary)
[ ] Did you write any new necessary tests? (not for typos and docs)
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] Did you list all the breaking changes introduced by this pull request?
[ ] Did you update the CHANGELOG? (not for typos, docs, test updates, or minor internal changes/refactors)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing, make sure you have read the review guidelines. In short, see the following bullet-list:

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[ ] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃

Thank you ❤

I misunderstood the .rst file to be .md for referencing pages.
Thank you for resolving the issues! :)


Support bindings to CompactRange

It would be very nice of the library had a binding to CompageRange.
See: https://github.com/google/leveldb/blob/803d69203a62faf50f1b77897310a3a1fcae712b/include/leveldb/db.h#L141

Fixed by https://github.com/kim/leveldb-haskell/commit/5bc30702ebbafd27fb3b9ca0a1a7d2ad7245c645


what is the default username/password ?

I cant' find anything about default login information in README or config files.
so what is the default username/password ? ??

Username: admin
Password: voiduser

not admin:voiduser not working

@AlphaNecron

admin:voiduser is the default credential. If you can't login, try admin:draconicuser

@AlphaNecron > User not found

Are you using docker?

no, direct install

sudo -u postgres psql
\x
TABLE "User";

The output is the username.

lol nothing

Use yarn prisma db seed if you are using yarn or npx prisma db seed if you are using npm.

admin:voiduser or admin:draconicuser  are both wrong .

I'm using docker-compose

Try executing yarn prisma db seed in the docker container.

ok no way

Do export DATABASE_URL=postgres://postgres:postgres@postgres/postgres first.

Or whatever you put in your config.

Is the issue fixed?

yes , after running the commands you provided , now I can login with admin/voiduser
/void # export DATABASE_URL=postgres://postgres:postgres@postgres/postgres
/void # yarn prisma db seed
yarn run v1.22.15
$ /void/node_modules/.bin/prisma db seed
Running seed command `ts-node --compiler-options {"module":"commonjs"} prisma/seed.ts` ...
 12:41:15 PM  INFO/SEED  Created default user with username "admin" and password "voiduser"

🌱  The seed command has been executed.
Done in 4.92s.
/void #

and I also want to point out the $PWD variable in docker-compose was not working. I had to update $PWD to .
like
    volumes:
      - './uploads:/void/uploads'
      - './public:/void/public'
      - './config.toml:/void/config.toml'

hope these problems can be solved in next release.

Hi @AlphaNecron , I'm encountering an error when attempting this (docker-compose). Any ideas?
/void # export DATABASE_URL=postgres://postgres:<password>@postgres/postgres
/void # yarn prisma db seed
yarn run v1.22.15
$ /void/node_modules/.bin/prisma db seed
Running seed command `ts-node --compiler-options {"module":"commonjs"} prisma/seed.ts` ...
PrismaClientKnownRequestError:
Invalid `prisma.user.aggregate()` invocation:

  The table `public.User` does not exist in the current database.
    at cb (/void/node_modules/@prisma/client/runtime/index.js:38537:17)
    at async main (/void/prisma/seed.ts:8:17) {
  code: 'P2021',
  clientVersion: '3.3.0',
  meta: { table: 'public.User' }
}

An error occured while running the seed command:
Error: Command failed with exit code 1: ts-node --compiler-options {"module":"commonjs"} prisma/seed.ts

I've also tried npx... with the same result.
Thank you,

Check whether void_postgres is running.


[CI][Packaging] Pin vcpkg to release that includes OpenSSL linux-headers fix

Describe the enhancement requested
Newer versions of vcpkg are required when building manylinux_2_28 arm64 platforms due to the following build issue with OpenSSL

https://github.com/microsoft/vcpkg/issues/29674

since fixed in:

https://github.com/microsoft/vcpkg/pull/29966

#34818 pins vcpkg to an untagged commit:  see https://github.com/apache/arrow/pull/34818#discussion_r1164203060, but it would be better to pin vcpkg to an official release tag.
Component(s)
Continuous Integration, Packaging, Release

@sjperkins It seems that a new version was released: https://github.com/microsoft/vcpkg/releases/tag/2023.04.15
Could you work on this?

@sjperkins It seems that a new version was released: https://github.com/microsoft/vcpkg/releases/tag/2023.04.15 Could you work on this?

Opened #35430


todo: merge AVX-512 version of PQFastScan

Faiss PR https://github.com/facebookresearch/faiss/pull/3276
AVX-512 implementation for PQFastScan for QBS.
For local benchmarks on 4th gen Xeon, the QPS is up to 10% higher, mostly for a single query case. But as far as I remember, production cases would show higher performance improvements.

It cannot be cherry-picked or copy-pasted, because Faiss baseline uses #ifdef mechanism for deciding whether to use AVX-512, and Knowhere uses dynamic hooks. Some additional code is needed.

/reopen

/reopen


checkbox  Array Values don't work

Your OS : MacOS Mojave
Node.js version : 10.15.0
Vuesax version: ^3.8.31
Browser version : chrome & mozilla latest version
Which package manager did you use for the install? npm

checkbox  Array Values don't work and not checked after click on checkbox .
please visit this page :
https://lusaxweb.github.io/vuesax/components/checkbox.html#array-values

@webafra  check sulotion here #496

@LaughingCoy : test this version but not worked

Hey @LaughingCoy use vuesax 3.8.20 latest vuesax still have some issue with vue. and if you can try using latest vue release I haven't tested latest vue release and vuesax 3.8.20
If latest vue and vuesax 3.8.20 dont work then use vue 2.5.19 & vuesax 3.8.20

hum.... i check today why bug with new releases of vuejs and new version of chrome

As @luisDanielRoviraContreras commented in above pull(Change is reverted). So, issue is still not solved.  This issue should re-open.


[Feature] echarts实现点击图标全屏 有没有全屏的属性

What problem does this feature solve?
echarts实现全屏浏览
What does the proposed API look like?
1

借助浏览器全屏 API 即可。 Demo


Can this be built with Java 17? Do you see a issue with that?

Can this be built with Java 17? Do you see a issue with that? @KilianB

Hey this is a little late, but I've got it working with Java 20:
build.gradle
plugins {
    id 'java'
    id 'org.openjfx.javafxplugin' version '0.0.14'
}

...

dependencies {
   implementation group: 'dev.brachtendorf', name: 'JImageHash', version: '1.0.0'
   implementation group: 'org.openjfx', name: 'javafx', version: '20.0.2', ext: 'pom'
   ...
}

javafx {
    version = "20"
    modules = [ 'javafx.base', 'javafx.graphics' ]
}


0.9.1 Compatibility

Compatibility with 0.9.1 appears to be blocked until the purescript-datetime rewrite is complete.
The only other issue I can see is changing import Data.Maybe.Unsafe (fromJust) to import Data.Maybe (fromJust). I'll include that in a PR once purescript-datetime is ready. 👍

Hopefully the datetime limbo will end soon, it's just waiting a review before release... assuming the review outcome is good :wink:

what is necessary to update the version tag to something different from current?

Just someone to find 10 minutes to do it is all. In the meantime you can point to master or a specific commit.

@hdgarrood Is anything stopping us from making a 1.0 release? I can do it if not.

Not that I'm aware of. I was planning on doing it tomorrow but sure, go ahead :)

I've made a release now.


Typo in the documentation

Hello.
The heading of this page has a typo. It should be menu not alert.
[menu]https://www.material-tailwind.com/docs/html/menu)
👍 thanks to the creators material tailwind.

Hey @drnkwati,
This issue has been fixed.
Cheers,
Sajad


Removed Unnecessary imports

Just removed a double import. One time it was imported as fp and another as f.

@Sh0g0-1758  yeah it happens in many files. I'm in favor of not merging this, as we currently have a few Dict related hints that use f.Element
merging this would lead too many conflicts, we should address this once all dict hints are merged
@cicr99

@Sh0g0-1758  now we only have one dict hint , we can merge this i guess!  can you update what needs to be updated to make it pass?

I think the changes I intended to makes with this PR already happened in the previous PRs which you merged. Closing this.


Are anchored link supported?

When someone click a link with anchor and take them to another page, is it possible to expand that section open?

Hi @llahnoraa
damn, I've thought to this case with my tabpanel script, but not with this plugin :)
For the moment, there is only an option to make it open by default, see http://a11y.nicolas-hoffmann.net/hide-show/
I'll have a look asap.

+1

Hi @llahnoraa ,
could you test ? It is okay for me https://a11y.nicolas-hoffmann.net/hide-show/#anims (test with or without it) but I prefer another test. ;)
Cheers,
Nicolas

It worked. Looks good to me so far. Can anyone else test on this, such as mobile-wise?
Do we need to update the plugin file for this?

Yes, as you may see in the diff, https://github.com/nico3333fr/jquery-accessible-hide-show-aria/commit/942cd066dad9555164fe4287f86ed04510686356 there are not many differences.

Perfect! Thank you for your help on this!


node version

Do your module work with node 0.12.7?
This error comes only if to many links will be checked (~ more than 150). With node 4.x are there no problems.
output:
  throw er; // Unhandled stream error in pipe.
        ^
  Error: not implemented
at Writable._write (_stream_writable.js:430:6)
at doWrite (_stream_writable.js:301:12)
at writeOrBuffer (_stream_writable.js:288:5)
at Writable.write (_stream_writable.js:217:11)
at Request.ondata (stream.js:51:26)
at Request.emit (events.js:129:20)
at IncomingMessage.<anonymous> (C:\Users\<user>\Documents\GitHub\Tutorial-C
 ontribution\node_modules\markdown-link-check\node_modules\request\request.js:985
 :12)
at IncomingMessage.emit (events.js:107:17)
at IncomingMessage.Readable.read (_stream_readable.js:373:10)
at flow (_stream_readable.js:750:26)
at resume_ (_stream_readable.js:730:3)

Thanks for the report. Fixed in version 1.0.2


Name for new feed source disappears

 Issue by abyrd
Saturday Sep 09, 2017 at 15:08 GMT
Originally opened as https://github.com/catalogueglobal/datatools-ui/issues/21

If you're creating a new feed source and move the pointer outside the UI box representing the new feed source, once you've typed a name for the feed source and move the pointer back into the text box the text will disappear.

 Comment by landonreed
Tuesday Nov 13, 2018 at 18:04 GMT

#41 will address this once merged.


MQTT Toggle messages

I have set my Sonoff T1 switches to have a 'ButtonTopic'. This effectively separates the 'button' from the relay. I require this as I want to do some complex automations within homekit. When the button is pressed, it sends an MQTT message something like this
cmnd/upstairs-landing/POWER1 TOGGLE
However, the plugin doesn't seem to know what to do with this and nothing gets sent up to Homekit. I was hoping to  use this to drive some automations, but this doesn't appear to be possible, as the plugin seems to rely upon ON/OFF values rather than TOGGLE?
Maybe I am missing something fundamental here. Thanks for your help.

I figured that a new plugin was in order as it wasn't working this way. I created one for the Service.StatelessProgrammableSwitch type.


Store file count on file_count event for future use

Fixes an undefined length issue introduced by #3502

Codecov Report

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

@@            Coverage Diff             @@
##           master    #3505      +/-   ##
==========================================
- Coverage    14.6%   14.59%   -0.01%
==========================================
  Files         267      267
  Lines       10794    10798       +4
  Branches     1485     1485
==========================================
  Hits         1576     1576
- Misses       9049     9053       +4
  Partials      169      169

Impacted Files
Coverage Δ

src/util/deploy/process-deployment.ts
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Default Sass implementation to Dart Sass

Recently, in PR #186 support for multiple sass implementations (e.g. Dart Sass, LibSass, etc.) was added. This PR defaults the Sass implementation to Dart Sass, while adding instructions for those who want to use alternatives.
Now users can simply run: ember install ember-cli-sass

Oh good. I was just working on this myself! Glad you beat me to it.
I was exactly at this point when I had a thought: maybe we should fully default to the JS dart sass implementation and instead of putting it into the app with addPackageToProject it should just be in the dependencies section of package.json. This would provide it by default to apps, but It would still be possible to install a different implementation if you want to and I don't think the extra NPM install will bother anyone as it wont add any weight to the final output.
I don't think it matters too much either way, but it feels to me like the sass implementation is a detail that I (as an app author) would rather hand over to this addon instead of needing to manage it myself.

Thanks @jbailey4! Seems to be a build error...
@jrjohnson I think adding the SASS implementation as a devDependency is the right approach.

Seems to be a build error...

@aexmachina I saw this same syntax error locally and thought I pushed up the change. May need to restart the CI build

@aexmachina Tests look good now - had one node eslint issue.

I just want to say that Dart Sass made the SCSS re-compilation 6x slower in our app with ~140 .scss files (which basically means that the overall re-compilation got 5x slower).
This project really needs a changelog so people don't miss these "breaking" changes 😅

Good suggestion @javve. See #208.


Add support to RTL to Grid

This PR adds support to dynamically update the direction (rtl|ltr) of the Grid component.
Direction of ltr has no effect on the actual implementation and should produce no changes. Setting rtl however will make the Grid be scrollable from the right, and column 0 will be the first starting from the right.
I am having trouble handling the state update of rtl->ltr and ltr->rtl, which should either reset the scroll to column 0 or (harder) keep the current scroll but from the other direction. Personally, resetting the scroll would work for my use case, but I am happy to add support for the scroll preservation.

[x] The existing test suites (npm test) all pass
[ ] For any new features or bug fixes, both positive and negative test cases have been added
[ ] For any new features, documentation has been added
[ ] For any documentation changes, the text has been proofread and is clear to both experienced users and beginners.
[x] Format your code with prettier (yarn run prettier).
[x] Run the Flow typechecks (yarn run typecheck).

Codecov Report

Merging #2 into master will decrease coverage by 0.56%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master       #2      +/-   ##
==========================================
- Coverage   94.66%   94.10%   -0.57%
==========================================
  Files          47       48       +1
  Lines        1801     1848      +47
==========================================
+ Hits         1705     1739      +34
- Misses         96      109      +13

Impacted Files
Coverage Δ

source/Grid/utils/getRTLOffsetType.js
86.95% <0.00%> (ø)

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 8d32bf6...eb996e8. Read the comment docs.


Dodat link za video reportazu

👷 Deploy Preview for balkantransport processing.
🔨 Explore the source changes:<PHONE_NUMBER>e7a7af8380c24da4c0651ffb64f1a4
🔍 Inspect the deploy log: https://app.netlify.com/sites/balkantransport/deploys/61ce0f3b34fc990007501183


check-config.sh returns git is unclean error on certain files

Running check-config.sh on the latest version 9367c9540a419fa4e5bff807ba64759b582091e5  which is an upgrade from previous versions returns the following error:
./check-config.sh

# ==============================================================================
# Checking git
# ==============================================================================
[ERR]  git is unclean
M mod/README.md
M mod/php-fpm-5.4/.keepme
M mod/php-fpm-5.5/.keepme
M mod/php-fpm-5.6/.keepme
M mod/php-fpm-7.0/.keepme
M mod/php-fpm-7.1/.keepme
M mod/php-fpm-7.2/.keepme

# ==============================================================================
# Checking .env file
# ==============================================================================
[SUCC] .env file exists
[SUCC] .env file is readable
[SUCC] All variables are present in .env file
[SUCC] No variables is duplicated in .env file

# ==============================================================================
# Checking .env file values
# ==============================================================================
[SUCC] All .env file variables have correct values
[SUCC] All .env file variables have correct values

# ==============================================================================
# Checking required directories
# ==============================================================================
[SUCC] All PHP cfg/ sub directories are present
[SUCC] All PHP log/ sub directories are present
[SUCC] All PHP mod/ sub directories are present
[SUCC] All HTTPD cfg/ sub directories are present
[SUCC] All HTTPD log/ sub directories are present

# ==============================================================================
# Checking directory permissions
# ==============================================================================
[SUCC] All devilbox directories have correct permissions
[SUCC] All devilbox directories have correct uid
[SUCC] All devilbox directories have correct gid

# ==============================================================================
# Checking file permissions
# ==============================================================================
[SUCC] All devilbox files have correct permissions
[SUCC] All devilbox files have correct uid
[SUCC] All devilbox files have correct gid

# ==============================================================================
# Checking projects
# ==============================================================================
[SUCC] All project dirs have correct permissions
[SUCC] All project dirs have correct uid
[SUCC] All project dirs have correct gid

# ==============================================================================
# SUMMARY
# ==============================================================================
[ERR]  Found 1 error(s)
[ERR]  Devilbox might not work properly
[ERR]  Fix the issues before submitting a bug report
[INFO] Ensure to run 'docker-compose stop; docker-compose rm -f' on changes in .env

I can't find any reason why these 7 files are flagged except all files are from commit d6bf9692126aafa04d3501460d157d523e483b94. If I make these files executable and then run  git status -s I don't see any modified files. However the script check-config.sh complains that the files have incorrect permissions
[ERR]  File 'mod/xxxx' should have 0644, 0664 or 0666 permissions. Has: 0775 permissions

Can you provide git status and git diff  outputs

$ git status
HEAD detached from v1.0.2
Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git restore <file>..." to discard changes in working directory)
	modified:   mod/README.md
	modified:   mod/php-fpm-5.4/.keepme
	modified:   mod/php-fpm-5.5/.keepme
	modified:   mod/php-fpm-5.6/.keepme
	modified:   mod/php-fpm-7.0/.keepme
	modified:   mod/php-fpm-7.1/.keepme
	modified:   mod/php-fpm-7.2/.keepme

no changes added to commit (use "git add" and/or "git commit -a")

$ git diff
diff --git a/mod/README.md b/mod/README.md
old mode 100755
new mode 100644
diff --git a/mod/php-fpm-5.4/.keepme b/mod/php-fpm-5.4/.keepme
old mode 100755
new mode 100644
diff --git a/mod/php-fpm-5.5/.keepme b/mod/php-fpm-5.5/.keepme
old mode 100755
new mode 100644
diff --git a/mod/php-fpm-5.6/.keepme b/mod/php-fpm-5.6/.keepme
old mode 100755
new mode 100644
diff --git a/mod/php-fpm-7.0/.keepme b/mod/php-fpm-7.0/.keepme
old mode 100755
new mode 100644
diff --git a/mod/php-fpm-7.1/.keepme b/mod/php-fpm-7.1/.keepme
old mode 100755
new mode 100644
diff --git a/mod/php-fpm-7.2/.keepme b/mod/php-fpm-7.2/.keepme
old mode 100755
new mode 100644

Can you simply do a git checkout . to reset your changes and run check-config.sh again

$ git checkout
M	mod/README.md
M	mod/php-fpm-5.4/.keepme
M	mod/php-fpm-5.5/.keepme
M	mod/php-fpm-5.6/.keepme
M	mod/php-fpm-7.0/.keepme
M	mod/php-fpm-7.1/.keepme
M	mod/php-fpm-7.2/.keepme

$ ./check-config.sh

# ==============================================================================
# Checking git
# ==============================================================================
[ERR]  git is unclean
 M mod/README.md
 M mod/php-fpm-5.4/.keepme
 M mod/php-fpm-5.5/.keepme
 M mod/php-fpm-5.6/.keepme
 M mod/php-fpm-7.0/.keepme
 M mod/php-fpm-7.1/.keepme
 M mod/php-fpm-7.2/.keepme

$ git status
HEAD detached from v1.0.2
Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git restore <file>..." to discard changes in working directory)
	modified:   mod/README.md
	modified:   mod/php-fpm-5.4/.keepme
	modified:   mod/php-fpm-5.5/.keepme
	modified:   mod/php-fpm-5.6/.keepme
	modified:   mod/php-fpm-7.0/.keepme
	modified:   mod/php-fpm-7.1/.keepme
	modified:   mod/php-fpm-7.2/.keepme

no changes added to commit (use "git add" and/or "git commit -a")

You seem to have forgotten the . on the checkout command

Opps sorry.
$ git checkout .
Updated 7 paths from the index

$ git status
HEAD detached from v1.0.2
nothing to commit, working tree clean

# ==============================================================================
# Checking file permissions
# ==============================================================================
[ERR]  File 'mod/php-fpm-7.1/.keepme' should have 0644, 0664 or 0666 permissions. Has: 0775 permissions
[ERR]  File 'mod/php-fpm-5.5/.keepme' should have 0644, 0664 or 0666 permissions. Has: 0775 permissions
[ERR]  File 'mod/README.md' should have 0644, 0664 or 0666 permissions. Has: 0775 permissions
[ERR]  File 'mod/php-fpm-7.2/.keepme' should have 0644, 0664 or 0666 permissions. Has: 0775 permissions
[ERR]  File 'mod/php-fpm-5.4/.keepme' should have 0644, 0664 or 0666 permissions. Has: 0775 permissions
[ERR]  File 'mod/php-fpm-5.6/.keepme' should have 0644, 0664 or 0666 permissions. Has: 0775 permissions
[ERR]  File 'mod/php-fpm-7.0/.keepme' should have 0644, 0664 or 0666 permissions. Has: 0775 permissions

That is truely strange. Do these files actually have 0775 permission?

They really do have 0775.
$ tree -apug
.
├── [drwxrwxr-x james    james   ]  php-fpm-5.2
│   └── [-rw-rw-r-- james    james   ]  .keepme
├── [drwxrwxr-x james    james   ]  php-fpm-5.3
│   └── [-rw-rw-r-- james    james   ]  .keepme
├── [drwxrwxr-x james    james   ]  php-fpm-5.4
│   └── [**-rwxrwxr-x** james    james   ]  .keepme
├── [drwxrwxr-x james    james   ]  php-fpm-5.5
│   └── [**-rwxrwxr-x** james    james   ]  .keepme
├── [drwxrwxr-x james    james   ]  php-fpm-5.6
│   └── [**-rwxrwxr-x** james    james   ]  .keepme
├── [drwxrwxr-x james    james   ]  php-fpm-7.0
│   └── [**-rwxrwxr-x** james    james   ]  .keepme
├── [drwxrwxr-x james    james   ]  php-fpm-7.1
│   └── [**-rwxrwxr-x** james    james   ]  .keepme
├── [drwxrwxr-x james    james   ]  php-fpm-7.2
│   └── [**-rwxrwxr-x** james    james   ]  .keepme
├── [drwxrwxr-x james    james   ]  php-fpm-7.3
│   └── [-rw-rw-r-- james    james   ]  .keepme
├── [drwxrwxr-x james    james   ]  php-fpm-7.4
│   └── [-rw-rw-r-- james    james   ]  .keepme
├── [drwxrwxr-x james    james   ]  php-fpm-8.0
│   └── [-rw-rw-r-- james    james   ]  .keepme
├── [drwxrwxr-x james    james   ]  php-fpm-8.1
│   └── [-rw-rw-r-- james    james   ]  .keepme
└── [-rwxrwxr-x james    james   ]  README.md

If for example I change to 0664 then we are back to the git is unclean.
None of these files are crucial for devilbox to work correctly so we can leave for the moment.

These six files have some strange values indicatedby the ** before and after the permissions.

├── [drwxrwxr-x james    james   ]  php-fpm-5.4
│   └── [**-rwxrwxr-x** james    james   ]  .keepme
├── [drwxrwxr-x james    james   ]  php-fpm-5.5
│   └── [**-rwxrwxr-x** james    james   ]  .keepme
├── [drwxrwxr-x james    james   ]  php-fpm-5.6
│   └── [**-rwxrwxr-x** james    james   ]  .keepme
├── [drwxrwxr-x james    james   ]  php-fpm-7.0
│   └── [**-rwxrwxr-x** james    james   ]  .keepme
├── [drwxrwxr-x james    james   ]  php-fpm-7.1
│   └── [**-rwxrwxr-x** james    james   ]  .keepme
├── [drwxrwxr-x james    james   ]  php-fpm-7.2
│   └── [**-rwxrwxr-x** james    james   ]  .keepme

All others are fine and have correct permission.

├── [drwxrwxr-x james    james   ]  php-fpm-5.2
│   └── [-rw-rw-r-- james    james   ]  .keepme
├── [drwxrwxr-x james    james   ]  php-fpm-5.3
│   └── [-rw-rw-r-- james    james   ]  .keepme
...
├── [drwxrwxr-x james    james   ]  php-fpm-7.3
│   └── [-rw-rw-r-- james    james   ]  .keepme
├── [drwxrwxr-x james    james   ]  php-fpm-7.4
│   └── [-rw-rw-r-- james    james   ]  .keepme
├── [drwxrwxr-x james    james   ]  php-fpm-8.0
│   └── [-rw-rw-r-- james    james   ]  .keepme
├── [drwxrwxr-x james    james   ]  php-fpm-8.1
│   └── [-rw-rw-r-- james    james   ]  .keepme

Do you have any idea what those two ** mean? And btw are you on a Mac or Linux?

Also can you share your global ~/.gitconfig if it does not contain any sensitive data?

I will have to close this as it is a local git issue and not related to the project.


HitsMetadata total hits is mandatory, but shouldn't be.

https://github.com/opensearch-project/opensearch-java/blob/e050cb03d5c90198da98010137460215be31eeea/java-client/src/main/java/org/opensearch/client/opensearch/core/search/HitsMetadata.java#L73
E.g. when creating a NativeQuery using the query builder such as
NativeQuery.builder()
   ...
   .withTrackTotalHits(false)
   .build()

This results in  org.opensearch.client.util.MissingRequiredPropertyException: Missing required property 'HitsMetadata.total'
Not tracking the totals is completely valid and can result in better performance when scrolling through the data (e.g. searchafter).

Is this a bug in the client or a feature request in the server? Either way maybe you can write a small unit test for this and PR a fix?


Make dependency 'nos' an extra

Nos is already an extra in pyproject.toml. However, there are several places in the code where nos is imported unconditionally, which means it needs to be installed.

Obviated (NOS has been removed)


different score ranges are confusing

it's confusing bleu scores are 0-100 & rouge 0-1 in this repo; I think either all scores should 0-100 or 0-1, probably the former

I agree that standardizing on the $[0, 100]$ range is ideal for the readability of these scores. The difference here is that the underlying sacreblue package scales BLEU scores by $100$. These are the only metrics in the harness that are scaled (accuracy, ROUGE, SAR, etc. are not). So, to make everything consistent for now, we can re-scale BLEU back to its "natural" units in $[0, 1]$ and follow up with an optional per-metric "results-formatter". What do you think?

I think this suggestion makes a lot of sense. Additionally, it would be nice to have the option to get rounded answers, e.g., 17.7%.


virt-api Webhook Informer Refactor

This PR does two things to our webhook code.

Removes usage of a global structure used to pass informers around with Get/Set functions and a mutex
Adds a DataSource informer to virt-api to avoid calling GET for every volume on a VM on every create/update.

The refactor of the informer logic should make it more approachable for people to use informers for webhooks moving forward. Without this change, it was very tedious to add and test logic that involved new informers.
The transition to using a cdi DataSource informer rather than api GETs avoids api amplification, where every create/update of a VM causes multiple more API calls. Avoid this reduces api server load and reduces latency in our core components.
NONE

/retest

/retest

/retest

@akrejcir maybe you'd be interested in reviewing this? This impacts your flavor PR since it lays the pattern for how to expose new informers to the validation webhooks.

Looks good.
/lgtm

@akrejcir can i get another lgtm when you get a chance. I added another commit to remove a function that's no longer used due to the refactor.

/lgtm


st.experimental_connection API docs

Nice!!!
It appears that the attributes grabs a larger section of the docstring for the table at the top than the methods does... Am I understanding it right? Should it be just the first line and not 3 paragraphs?

@vdonato is there a cherry picked release branch / commit that Snehan can use to re-pull the updated docstrings into this PR? Assuming it's a bit risky to do from develop although that might work too.

Should it be just the first line and not 3 paragraphs?

Good catch, I've fixed the issue and pulled the docstrings from the 1.22.0 branch 😄
Are we good to merge now @sfc-gh-jcarroll ?


Fix typo in src/components/modal/types.ts

What problem does this feature solve?
There is a typo in the word IBackgroudProps src/components/modal/types.ts
What is expected?
To remove typo.

I have raised PR


Dependency graph as new command

Implemented issue from (#3665):

Moved graph switches from search command to depend-info command.
Added option to generate dgml graph (considers also features).

Example (dgml):
.\vcpkg.exe depend-info ffmpeg --dgml // print specific port
.\vcpkg.exe depend-info --dgml // print all ports

Example (dot):
.\vcpkg.exe depend-info ffmpeg --dot

this is awesome

Sorry for the delay on this. It looks great!
I also agree with moving the graph code into depend-info; I have been meaning to do more graph analysis in depend-info, specifically gather transitive dependents & transitive dependencies (but hadn't had time yet).


tidb_tools: update dependency

What problem does this PR solve?
update filter dependency to solve https://github.com/pingcap/tidb-tools/pull/382
What is changed and how it works?
update go mod
Check List
Tests

No code

Related changes

Need to cherry-pick to the release branch

/merge


osm-controller_test.go failed in the CI

Bug description:
osm-controller_test.go failed in the CI.
https://pipelines.actions.githubusercontent.com/FqYyUlFBwgcC4xlcAlRHSOZjGFqxSg5UdT6unwZ3oA6NkQIoht/_apis/pipelines/1/runs/6464/signedlogcontent/34?urlExpires=2020-08-27T16%3A00%3A24.1924727Z&urlSigningMethod=HMACV1&urlSignature=jrS8ptfBKjM00Rv0FJfwHARFog34LXEWaOjABiaDeNE%3D
2020-08-27T05:07:27.8429040Z {"level":"info","component":"osm-controller/main","time":"2020-08-27T05:07:27Z","file":"osm-controller.go:279","message":"Created CA bundle Kubernetes secret --secret--name-- in namespace --namespace--"}
2020-08-27T05:07:27.8429442Z • Failure [1.831 seconds]
2020-08-27T05:07:27.8429696Z Test creation of CA bundle k8s secret
2020-08-27T05:07:27.8430145Z /home/runner/work/osm/osm/cmd/osm-controller/osm-controller_test.go:17
2020-08-27T05:07:27.8430409Z   Testing createCABundleKubernetesSecret
2020-08-27T05:07:27.8430832Z   /home/runner/work/osm/osm/cmd/osm-controller/osm-controller_test.go:18
2020-08-27T05:07:27.8431234Z     creates a k8s secret [It]
2020-08-27T05:07:27.8431685Z     /home/runner/work/osm/osm/cmd/osm-controller/osm-controller_test.go:19
2020-08-27T05:07:27.8431854Z
2020-08-27T05:07:27.8432191Z     Expected
2020-08-27T05:07:27.8432420Z         <int>: 1960
2020-08-27T05:07:27.8432638Z     to equal
2020-08-27T05:07:27.8432995Z         <int>: 1915
2020-08-27T05:07:27.8433150Z
2020-08-27T05:07:27.8433557Z     /home/runner/work/osm/osm/cmd/osm-controller/osm-controller_test.go:35

Affected area (please mark with X where applicable):

Install                    [ ]
SMI Traffic Access Policy  [ ]
SMI Traffic Specs Policy   [ ]
SMI Traffic Split Policy   [ ]
Permissive Traffic Policy  [ ]
Ingress                    [ ]
Egress                     [ ]
Envoy Control Plane        [ ]
CLI Tool                   [ ]
Metrics                    [ ]
Certificate Management     [ ]
Sidecar Injection          [ ]
Logging                    [ ]
Debugging                  [ ]
Tests                      [X]
CI System                  [ ]

Expected behavior:
Test should not fail.

The test hardcodes the byte length of the CA certificate:
https://github.com/openservicemesh/osm/blob/main/cmd/osm-controller/osm-controller_test.go#L35
This is not going to be reliable when the package versions are upgraded.


fix _moveDomModuleStyleIntoTemplate

[x] fixed  https://github.com/Polymer/polymer-cli/issues/614

I signed it!

@baocang Thank you for this fix!

Going to move this into a separate branch and adjust a couple items for style before internal review and merge.  Thanks again, @baocang


Make WebhookClient (sync/async) #send method accept link unfurl params

Summary
Resolves #1045, and addresses a bug where bool values being converted to "0" and "1" were not correctly being interpreted by Slack API.
Changes:

Modifies WebhookClient#send and AsyncWebhookClient#send to accept unfurl_links and unfurl_media as optional arguments)
Previously booleans False and True were being converted to "0" and "1" in the outgoing http request body, but the options were not correctly being applied. See "recreate bug" section below. I believe this is due to the value being returned as a string. With a small modification to _to_0_or_1_if_bool() method to return integer instead, the unfurl options provided are correctly applied.

To recreate the bug:
Using the #send_dict method only, you can recreate this with
dict = {
   "text": ""<https://imgs.xkcd.com/comics/new_sports_system_2x.png>", "unfurl_links": False, "unfurl_media": False
}

response = webhook.send_dict(dict)
assert response.status_code == 200
assert response.body == "ok"

In this case, outgoing http body looks like:
{"text": "hello webhook world\n<https://imgs.xkcd.com/comics/new_sports_system_2x.png>", "unfurl_links": "0", "unfurl_media": "0"}

Per docs, example, this link should be prevented from unfurling, but it does unfurl.
With the changes, the outgoing http body looks like:
{"text": "hello webhook world\n<https://imgs.xkcd.com/comics/new_sports_system_2x.png>", "unfurl_links": 0, "unfurl_media": 0}

And the link is properly prevented from unfurling.
Category (place an x in each of the [ ])

[ ] slack_sdk.web.WebClient (sync/async) (Web API client)
[x] slack_sdk.webhook.WebhookClient (sync/async) (Incoming Webhook, response_url sender)
[ ] slack_sdk.socket_mode (Socket Mode client)
[ ] slack_sdk.signature (Request Signature Verifier)
[ ] slack_sdk.oauth (OAuth Flow Utilities)
[ ] slack_sdk.models (UI component builders)
[ ] slack_sdk.scim (SCIM API client)
[ ] slack_sdk.audit_logs (Audit Logs API client)
[ ] slack_sdk.rtm_v2 (RTM client)
[ ] /docs-src (Documents, have you run ./docs.sh?)
[ ] /docs-src-v2 (Documents, have you run ./docs-v2.sh?)
[ ] /tutorial (PythOnBoardingBot tutorial)
[ ] tests/integration_tests (Automated tests for this library)

Requirements (place an x in each [ ])

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've run python3 -m venv .venv && source .venv/bin/activate && ./scripts/run_validation.sh after making the changes. <---- Ran up until test_interactions_websocket_client.py and then hung again

Ran up until test_interactions_websocket_client.py and then hung again

Hmm, this test basically takes much longer than others (it verifies many patters of data chunk size). You can check the progress in logs/pytest.log. Also, for convenience, you can run specific tests by ./scripts/run_unit_tests.sh tests/slack_sdk/webhook/.

@seratch I've put some questions in alongside my latest changes (which add to the sync webhook's integration tests). Depending on the answers to those questions, I will go ahead an implement for async as well. Thanks in advance for taking a look!

@seratch Yes, the level of expected behavior for unit-level tests you describe makes sense to me, and I have added them alongside the integration tests. It also makes more sense to me to add the channel_id logic to setUp(). Thanks for the detailed feedback!


Object not assignable to type even after narrowing

Originally posted on StackOverflow. Re-posting here as I hope this is something which can be improved in TypeScript, so we don't have to resort to type assertions.

TypeScript Version:  3.9.3

Search Terms: narrow object type guard
Code
type DictionaryLike = {
    [index: string]: unknown;
};

type ObjectWithFooStringProp = {
    foo: string;
};

const getObjectWithFooStringProp = (
    input: DictionaryLike,
): ObjectWithFooStringProp | null => {
    if (typeof input.foo === 'string') {
        input.foo; // Type is `string` ✅

        // Unexpected error ❌
        // Property 'foo' is missing in type 'DictionaryLike' but required in type 'ObjectWithFooStringProp'.
        return input;

        // Workaround:
        // return { foo: input.foo };
    } else {
        return null;
    }
};

Expected behavior:
No errors
Actual behavior:
Errors
Playground Link: Link
Related Issues: https://github.com/microsoft/TypeScript/issues/20470

Duplicate #10065

Thanks. From the discussion in that issue it's not clear what the current status of this is. Is it still considered an open suggestion or is it a "won't fix"?


Core: Connector not starting, OAuth2Extension cannot resolve certificates

Bug Report
Describe the Bug
When starting the EDC the OAuth2Extension.getEncodedClientCertificate always throws an org.eclipse.dataspaceconnector.spi.EdcException: Public certificate not found exception.
Expected Behavior

Connector starting and resolving certificates from vault

Observed Behavior

Connector not starting

Steps to Reproduce
Steps to reproduce the behavior:

Start IDS Connector

Context Information
Add any other context about the problem here.

Used version 0.0.1-20220818-SNAPSHOT

Detailed Description

Extension is always using the NoopCertificateResolver

Possible Implementation

Probably the DefaultCertificateResolver should be used instead

Dominik Pinsel<EMAIL_ADDRESS>Mercedes-Benz Tech Innovation GmbH, legal info/Impressum

If you use a vault custom extension you need to explicitly register the VaultCertificateResolver.
For an example, look at the hashicorp-vault extension:
https://github.com/eclipse-dataspaceconnector/DataSpaceConnector/blob/main/extensions/common/vault/hashicorp-vault/src/main/java/org/eclipse/dataspaceconnector/core/security/hashicorpvault/HashicorpVaultExtension.java#L75

If you use a vault custom extension you need to explicitly register the VaultCertificateResolver.
For an example, look at the hashicorp-vault extension:
https://github.com/eclipse-dataspaceconnector/DataSpaceConnector/blob/main/extensions/common/vault/hashicorp-vault/src/main/java/org/eclipse/dataspaceconnector/core/security/hashicorpvault/HashicorpVaultExtension.java#L75

With our custom extension everything works fine. Our problem occurs with the Azure Vault extension from this repository, but I assume the HashiCorp vault extension from this repository has this problem, too.
But I just saw that you fixed this issue 7 days ago in PR #1842  -> will close this issue.
Dominik Pinsel<EMAIL_ADDRESS>Mercedes-Benz Tech Innovation GmbH, legal info/Impressum


Double spaces

There are 2 spaces in Typescript types as interface (between TypeName and extends):
export interface TypeName  extends InterfaceName {
  readonly __typename?: 'TypeName';
  readonly type: PropertyType;
}

Thanks @jer-sen , this is a minor lint issue, and we usually don't fix those.
It's recommended to ignore generated files from your linter/formatter.
I fixed this specific one in: https://github.com/dotansimha/graphql-code-generator/commit/247a8b0b5bc79b8aff08f60ba337c9f6892a4b16

Thanks @dotansimha

Fixed in v1.15.2


Refactoring a multi-line step should retain the step in multiple lines

Expected behavior
Refactoring a multi-line step should retain the step in multiple lines
Actual behavior
After refactoring the step is in single line
Steps to replicate

Create a multi-line step in a spec
Refactor it

The multi-line format must be retained

Version
Gauge version: 1.0.1.nightly-2018-08-22
Commit Hash: 3750b26

Child of https://github.com/getgauge/gauge/issues/265

Closing this as an old issue. This may be fixed in the newer releases.
Will reopen if it still needs fixing.


Add support for RP2040

I used svd2nim for SVD file for RP2040 in https://github.com/raspberrypi/pico-sdk/blob/master/src/rp2040/hardware_regs/RP2040.svd
But procedures in generated module rp2040.nim doesn't write registers atomitically.
Please read '2.1.2. Atomic Register Access' in RP2040 Datasheet in https://datasheets.raspberrypi.com/rp2040/rp2040-datasheet.pdf
This header file in Raspberry Pi Pico SDK wraps atomic memory accesses:
https://github.com/raspberrypi/pico-sdk/blob/master/src/rp2_common/hardware_base/include/hardware/address_mapped.h
modifyIt templates should not be generated for RP2040 as it seems cannot be atomic.
I think following procs should to be added:
# Common procs for writing RP2040 registers.
# Not exported.
const
  RegAliasXorBits = 0x1000'u32
  RegAliasSetBits = 0x2000'u32
  RegAliasClrBits = 0x3000'u32

proc storeToReg(reg: object; mask: uint32; regAlias: uint32) {.inline.} =
  volatileStore(cast[ptr uint32](cast[uint32](reg.loc) or regAlias), mask)

# You can use setBits/clearBits/xorBits for modifying 1 bit bool fields in a register.
# When you write int or enum fields in the register, use `writeMasked`.
# Because for example, if you want to set '0b10' to the 2bit field of the register,
# you need to call `setBits` and `clearBits` but that requires write to the register twice.
# Write to some registers have side effect and you might want to write only once.
proc setBits*(reg: object; mask: uint32) {.inline.} =
  storeToReg(reg, mask, RegAliasSetBits)

proc clearBits*(reg: object; mask: uint32) {.inline.} =
  storeToReg(reg, mask, RegAliasClrBits)

proc xorBits*(reg: object; mask: uint32) {.inline.} =
  storeToReg(reg, mask, RegAliasXorBits)

# Set new values for the bits of register.
# You need to use this when you want to set int/enum type field of a register.
# Concurrent write to different bits is safe but same bits is unsafe.
proc writeMasked*(reg: object; values, writeMask: uint32) =
  xorBits(reg, (volatileLoad(cast[ptr uint32](reg.loc)) xor values) and writeMask)

import std/options

type
  SomeRegister_Fields* = distinct uint32

# It seems `set[SomeEnum]` cannot be used for specific size sets.
# https://github.com/nim-lang/Nim/issues/21789
# So use bitwise operators.
const
  SomeRegisterFieldBit0* = 0x1.SomeRegister_Fields
  SomeRegisterFieldBit1* = 0x2.SomeRegister_Fields
  SomeRegisterFieldBit2* = 0x4.SomeRegister_Fields
  SomeRegisterFieldBit3* = 0x8.SomeRegister_Fields
  ...

# Set only bits that is 1 in `val`.
proc setBits*(reg: Register_Type, val: SomeRegister_Fields) =
  setBits(reg, val)

# Clear only bits that is 1 on `val`.
proc clearBits*(reg: Register_Type, val: SomeRegister_Fields) =
  clearBits(reg, val)

# Flip only bits that is 1 on `val`.
proc xorBits*(reg: Register_Type, val: SomeRegister_Fields)
  xorBits(reg, val)

# For int/enum fields.
proc writeMasked*(reg: Register_Type, Field0 = none(uint32), Field1 = none(SomeEnum), Field2 = none(bool)) =
  var x: uint32
  if Field2.isSome:
    x.setMask((Field2 shl 24).masked(24 .. 24))
  if Field1.isSome:
    x.setMask((Field1.uint32 shl 16).masked(16 .. 23))
  if Field0.isSome:
    x.setMask((Field0 shl 0).masked(0 .. 15))
  reg.writeMasked x

Thanks for reporting. I'll have to look into it in more detail. I'm also not a very deep expert on low-level stuff, so if someone wants to propose a PR, I'm very open.

The atomic access this microcontroller offers is a neat feature.  But there isn't anything in the SVD 1.3 file spec that reliably serves as an indicator for this capability.  So, for now, I recommend that support for atomic access be done outside the svd2nim tool.
Here is a snippet from the cited RP2040.svd file for a register that has atomic access:
            <register>
                <name>GPIO_OUT_SET</name>
                <addressOffset>0x00000014</addressOffset>
                <description>GPIO output value set</description>
                <resetValue>0x00000000</resetValue>
                <fields>
                    <field>
                        <name>GPIO_OUT_SET</name>
                        <description>Perform an atomic bit-set on GPIO_OUT, i.e. `GPIO_OUT |= wdata`</description>
                        <bitRange>[29:0]</bitRange>
                        <access>write-only</access>
                    </field>
                </fields>
            </register>

As you can see, the only mention of "atomic" is in the description.  The field access write-only, which is not indicative of atomic access.

The atomic access this microcontroller offers is a neat feature. But there isn't anything in the SVD 1.3 file spec that reliably serves as an indicator for this capability.

That right. So how about to generate code for RP2040, when svd2nim see that following device name and vendor in the input SVD file?
    <vendor>Raspberry Pi</vendor>
    <name>RP2040</name>
    <series>RP</series>

According to '2.1.2. Atomic Register Access' in RP2040 Datasheet, all registers in RP2040 supports atomic access excepts some registers in SIO.
If it is still not good and should not be merged to svd2nim, I will fork svd2nim and implement the feature.

I don't mind "special case" codegen for the rp2040. I'd accept a PR.
It might be a bit of PITA with the current (poor) organization of the codegen module in svd2nim. One improvement I planned to make at some point was creating some sort of "CodegenOptions" global singleton in the codegen module that would contain all the info necessary for all the procs in there. It could also contain some sort of "rp2040" flag or device name string.


Revisions should be worth 2.0, not 0.0

The value should be configurable in the assignment. This would simplify bCourses exports.
I'm also considering manually changing all past revisions to be 2.0, as well.

We can even publish these scores after some time, when #872 is complete.


Initial support for extract(x from time)

Which issue does this PR close?

Closes #8692
Rationale for this change

What changes are included in this PR?

Extend support for extract and date_part to Time32 and Time64 types, for relevant precisions (hours, minutes, seconds, etc.)
Are these changes tested?

Are there any user-facing changes?

Depends on https://github.com/apache/arrow-rs/pull/5262
Will keep this in draft until arrow-rs support is merged and available to Datafusion
(Also need to update docs)

Release of arrow is tracked by https://github.com/apache/arrow-rs/issues/5234

Pending https://github.com/apache/arrow-rs/pull/5337 which will come as part of arrow-rs v51.0.0 (next release)


Allow admins to unsubscribe others irrespective of setting.

First commit is to allow admins to unsubscribe others.
Second commit is to mention the admin privilege with label.
Third commit is to optimize DB queries for computing group setting values.

Fixes:  https://chat.zulip.org/#narrow/channel/101-design/topic/permissions.20for.20admin.20to.20unsubscribe.20others

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Not sure label does look good since the text does not fit in one line -

For the last commit, the resulting SQL is -
(
    SELECT
        "zerver_usergroupmembership"."user_group_id" AS "col1",
        "zerver_usergroupmembership"."user_profile_id" AS "col2",
        'user' AS "member_type",
        "zerver_namedusergroup"."usergroup_ptr_id" IS NOT NULL AS "is_named_group"
    FROM
        "zerver_usergroupmembership"
    INNER JOIN
        "zerver_usergroup"
        ON (
            "zerver_usergroupmembership"."user_group_id" = "zerver_usergroup"."id"
        )
    LEFT OUTER JOIN
        "zerver_namedusergroup"
        ON (
            "zerver_usergroup"."id" = "zerver_namedusergroup"."usergroup_ptr_id"
        )
    WHERE
        "zerver_usergroupmembership"."user_group_id" IN group_ids
)
UNION
(
    SELECT
        "zerver_groupgroupmembership"."supergroup_id" AS "col1",
        "zerver_groupgroupmembership"."subgroup_id" AS "col2",
        'group' AS "member_type",
        "zerver_namedusergroup"."usergroup_ptr_id" IS NOT NULL AS "is_named_group"
    FROM
        "zerver_groupgroupmembership"
    INNER JOIN
        "zerver_usergroup"
        ON (
            "zerver_groupgroupmembership"."supergroup_id" = "zerver_usergroup"."id"
        )
    LEFT OUTER JOIN
        "zerver_namedusergroup"
        ON (
            "zerver_usergroup"."id" = "zerver_namedusergroup"."usergroup_ptr_id"
        )
    WHERE
        "zerver_groupgroupmembership"."supergroup_id" IN group_ids
);

Query and query plan looks good, but I am still confused about the query counts not improving -- worth debugging.


Pyscenedetect Gui Interface and Resolve like threshhold manipulation

I have an idea.

Will you make pyscenedetect gui with this repo>https://github.com/mherrmann/fbs-tutorial?
https://www.youtube.com/watch?v=clnlOImwCuc, after watching this tutorial Davinci Resolve can manipulate to threshold or content detection. It is hard via CLI, thus we need Gui option to alter visually.
Hope you will get my idea. Thanks.

After reviewing that video, I might consider doing something like this in Rust with egui once PySceneDetect is a bit more stable.


The response body param wrong

Expected behavior
{"useName": null, "iName": "xxx"}

Actual behavior

Steps to reproduce
use entity
    class Hello {
        String iName;

        String userName;
    }

Env

Restlight version: 1.0.0-SNAPSHOT
JVM version: jdk11
OS:window 10

We use Jackson to serialize by default. Jackson serializes the iName to iname by default. Users can configure it according to their own needs or use other serialization.


compiling problem

Problem Description
My OS is Ubuntu18.04.
First, I installed PInocchio successfully following the instructions on this website https://stack-of-tasks.github.io/pinocchio/download.html :

Add robotpkg apt repository
Install Pinocchio
sudo apt install robotpkg-py27-pinocchio
Configure environment variables(add the following to the file ~/.bashrc and source it)
export PATH=/opt/openrobots/bin:$PATH
export PKG_CONFIG_PATH=/opt/openrobots/lib/pkgconfig:$PKG_CONFIG_PATH
export LD_LIBRARY_PATH=/opt/openrobots/lib:$LD_LIBRARY_PATH
export PYTHONPATH=/opt/openrobots/lib/python2.7/site-packages:$PYTHONPATH
Then, I started installing tsid following the instructions on this website and failed when executing cmake:

I am really confused.

Dear @xuhuairuogu
Several solutions are possible
1/ You want to continue with sources:
I would suggest to try:
export PKG_CONFIG_PATH=/opt/openrobots/lib/pkgconfig/:$PKG_CONFIG_PATH

Note that there is '/' just after pkgconfig
2/ You might try to use the binaries:
apt-get install robotpkg-tsid
apt-get install robotpkg-py27-tsid
Note that there is currently a problem with the python bindings, and I will push a PR to fix this problem this week-end.

Thanks for your reply. Now I have successfully compiled and installed pinocchio, OpenSceneGraph3.4, PythonQt, gepetto-viewer, gepetto-viewer-corba and tsid all from source. However, when I execute examples related to Gepetto GUI, for example exercizes/ex_1.py, 3D models cannot be displayed. It seems that everything except for model displaying works well.

Hi @xuhuairuogu,
It seems that you did not install the osg-dae pluging necessary to load your meshes (because they are collada files).
You can do that using the following command:
sudo apt-get install robotpkg-osg-dae
See https://github.com/stack-of-tasks/pinocchio/issues/705 for additional details.
Best,
Justin

HI, on Ubuntu18.04, there is not robotpkg-osg-dae. Therefore, I install collada using the following command:
sudo apt-get install libcollada-dom2.4-dp-dev
and recompile OpenSceneGraph3.4 and osgdb_dea.so is generated.
However, when executing python ex_1.py, a new error appears:

I have converted all dae files to stl files by Blender and changed the 3D model file path in romeo.urdf. Although it is not an elegant way, it really works.


Stage and deploy App Engine Standard projects

Related to #283 , implements item 5)

With the exception of the promote flag, lgtm.

@briandealwis PTAL

LGTM once testing the XML reading for comments.  The others issues we can defer to later.


Always show instance label on target page.

It may be different from the address shown in the first column.
Closes #2061.

make assets, make format :)

FYI @fabxc make format is not longer necessary as independent step,  make assets ensures that the generated asset files are formatted correctly.


feat: 4889 - new simplified nutriscore widget

What

This PR adds a simplified version of the nutriscore display conform to the big redesign, on top of the KP and only if the "reordering KP" flag is set on DEV mode.
While coding, several questions:

Should we try to be a11n compliant? At least I tried with colors AND icons. Can easily be turned off back to "just the color"
We don't have any localizations for "salt", and I don't know if we should expect them from the server or if we should localize them in the app
I had to extract the data from the server (e.g. "1.2%") - again, should the server provide us with that raw data?

The simplified nutriscore is about 50% less tall than the current version. That goes in the right direction.

Screenshots

test 1
test 2

Fixes bug(s)

Fixes: #4889

Files
New files:

evaluation_extension.dart: a11y helper for Evaluation.
knowledge_panel_enum.dart: Helper around knowledge panel ids.
knowledge_panel_simplified_row.dart: Row that displays two widgets on same width columns (half max width).
knowledge_panel_simplified_title.dart: Full width title for a simplified knowledge panel.
knowledge_panel_simplified_widget.dart: Simplified widget for knowledge panel.
nutriscore_simplified_widget.dart: Simplified nutriscore widget, with nutriscore and 4 other attributes.

Impacted files:

knowledge_panel_title_card.dart: moved code to new class EvaluationExtension
new_product_page.dart: added simplified nutriscore widget triggered by dev mode only
reorderable_knowledge_panel_page.dart: minor refactoring using new class KnowledgePanelEnum

Codecov Report
Attention: 123 lines in your changes are missing coverage. Please review.

Comparison is base (5ffdca3) 9.68% compared to head (c12d035) 9.62%.

Files
Patch %
Lines

...ig_redesign/knowledge_panel_simplified_widget.dart
0.00%
54 Missing :warning:

...big_redesign/knowledge_panel_simplified_title.dart
0.00%
24 Missing :warning:

...uct/big_redesign/nutriscore_simplified_widget.dart
0.00%
17 Missing :warning:

...ges/product/big_redesign/evaluation_extension.dart
0.00%
14 Missing :warning:

...t/big_redesign/knowledge_panel_simplified_row.dart
0.00%
8 Missing :warning:

...l/knowledge_panels/knowledge_panel_title_card.dart
0.00%
2 Missing :warning:

...smooth_app/lib/pages/product/new_product_page.dart
0.00%
2 Missing :warning:

...ages/product/reorderable_knowledge_panel_page.dart
0.00%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           develop   #4890      +/-   ##
==========================================
- Coverage     9.68%   9.62%   -0.06%
==========================================
  Files          318     323       +5
  Lines        16121   16218      +97
==========================================
  Hits          1561    1561
- Misses       14560   14657      +97

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Hey @monsieurtanuki :-)

the PR is a good start
as was mentioned in the tracker issue for the PoC implementation, it should eventually rely on server side evolutions (issue there: https://github.com/openfoodfacts/openfoodfacts-server/issues/9368 mentioned in https://github.com/openfoodfacts/smooth-app/issues/4842 )
So we can expect some further refactoring so that the server can trigger this layout, including for the environment page
there are some other issues that rely on server side changes (server side issues are listed). Please ensure to ping @stephanegigandet or @alexgarel before moving

@g123k @monsieurtanuki We discussed the square layout yesterday in the Product Opener call. John is going to work on it, we made the necessary decision, and hopefully it will arrive soon.
We also felt it was necessary to uncouple the a11y decision, although we discussed it along the way. There are several options (smileys, thumbs up and down, or simply unambiguous text) and there's no agreement on it. We also want to bring the website on the same level.

Closing as stale.


An issue? Possibly? IDK ...

Just ran into a weird issue where I tried to run the script, got prompted to install requests, installed it, but when I try and run the script it says I need to install requests :(
2016techmba06:~ user$ sudo easy_install requests
Password:
Searching for requests
Best match: requests 2.18.3
Processing requests-2.18.3-py2.7.egg
requests 2.18.3 is already the active version in easy-install.pth

Using /usr/local/lib/python2.7/site-packages/requests-2.18.3-py2.7.egg
Processing dependencies for requests
Finished processing dependencies for requests
2016techmba06:~ user$ appleLoops.py -v
requests module must be installed. sudo easy_install requests

If you're packaging up the requests module rather than using sudo easy_install requests, then the package may not have all the required files, or the permissions may not be correct.
What happens when you try and import requests from an interactive python session?
[user@valkyrie]:~ # python
Python 2.7.10 (default, Feb  7 2017, 00:08:15)
[GCC 4.2.1 Compatible Apple LLVM 8.0.0 (clang-800.0.34)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import requests
>>>

Yeah I think it was a fluke.
I just came up with my own solution for nicely installing these dependencies and then removing them when everything is done.
I have 3 packages - 1 x for the appleLoops folder w/ a postinstall script, 1 x payload-free package w/ a postinstall script for installing the dependencies, and 1 x payload-free package with a postinstall script for uninstalling everything.
All three write to the local appleLoops.log file.  I opt to go with pip for requests and dependencies installs because they're easier to uninstall and pip can uninstall itself.
appleLoops.py_2.1.3.pkg postinstall
#!/bin/sh
## postinstall

DATE=$(date "+%Y-%m-%d %H:%M:%S")
LOG="/Library/Logs/TMSTech/appleLoops.log"
APPLELOOPSPY="/usr/local/bin/appleLoops-test/appleLoops.py"

writelog () {
	/bin/echo "$DATE" "${1}" >> "$LOG"
}

writelog "========= Start appleLoops.py Install ========="

if [ -f "$APPLELOOPSPY" ]; then
	writelog "Creating appleLoops.py symbolic link …"

	/bin/ln -s "$APPLELOOPSPY" /usr/local/bin

	if [ -h "/usr/local/bin/appleLoops.py" ]; then
		writelog "appleLoops.py link: successful!"
	else
		writelog "appleLoops.py link: failed."
		result=1
	fi
else
	writelog "appleLoops.py does not exist. Exiting."
	exit 1
fi

# Install PIP to install additional appleLoops.py dependencies
sudo easy_install pip
PIP_INSTALL=$?

if [ "$PIP_INSTALL" = "0" ]; then
	writelog "pip install: successful!"
else
	writelog "pip install: failed."
	result=1
fi

writelog "========= Finish appleLoops.py Install ========="

exit $result

appleLoops.py_Dependencies.pkg postinstall
#!/bin/sh
## postinstall

DATE=$(date "+%Y-%m-%d %H:%M:%S")
LOG="/Library/Logs/TMSTech/appleLoops.log"

writelog () {
	/bin/echo "$DATE" "${1}" >> "$LOG"
}

writelog "========= Start appleLoops.py Dependencies Install ========="

# install requests and dependencies for appleLoops.py
sudo /usr/local/bin/pip install requests
REQUESTS_INSTALL=$?

if [ "$REQUESTS_INSTALL" = "0" ]; then
	writelog "requests and dependencies installs: successful!"
else
	writelog "requests and dependencies installs: failed."
	exit 1
fi

writelog "========= Finish appleLoops.py Dependencies Install ========="

exit

appleLoops.py_Uninstall.pkg
#!/bin/sh
## postinstall

DATE=$(date "+%Y-%m-%d %H:%M:%S")
LOG="/Library/Logs/TMSTech/appleLoops.log"
PIP="/usr/local/bin/pip"
APPLELOOPSPY="/usr/local/bin/appleLoops-test"
REQUESTS_DEPENDENCIES=$($PIP show requests | /usr/bin/grep Requires | /usr/bin/sed 's/Requires\: //g; s/,//g')

writelog () {
	/bin/echo "$DATE" "${1}" >> "$LOG"
}

writelog "========= Start appleLoops.py Uninstall ========="

# Remove appleLoops directory
/bin/rm -rf "$APPLELOOPSPY"

if [ ! -d "$APPLELOOPSPY" ]; then
	writelog "appleLoops.py removal: successful!"
else
	writelog "appleLoops.py removal: failed."
	result=1
fi

# Remove appleLoops.py link
/bin/rm /usr/local/bin/appleLoops.py

if [ ! -h "/usr/local/bin/appleLoops.py" ]; then
	writelog "appleLoops.py link removal: successful!"
else
	writelog "appleLoops.py link removal: failed."
	result=1
fi

# Uninstall all requests dependencies
for egg in $REQUESTS_DEPENDENCIES
do
	sudo $PIP uninstall $egg -y

	if [ $? = 0 ]; then
		writelog "${egg} uninstall: successful!"
	else
		writelog "${egg} uninstall: failed"
		result=1
	fi

done

# Uninstall requests
sudo $PIP uninstall requests -y

if [ $? = 0 ]; then
	writelog "requests uninstall: successful!"
else
	writelog "requests uninstall: failed."
	result=1
fi

# Uninstall pip
sudo $PIP uninstall pip -y

if [ $? = 0 ]; then
	writelog "pip uninstall: successful!"
else
	writelog "pip uninstall: failed."
	result=1
fi

writelog "========= Finish appleLoops.py Uninstall ========="

exit $result

Ah cool. I'll close this out. I'm thinking that it's better to stick with the inbuilt python urllib2 modules than requiring requests be installed, it's just so damn easy to work with the requests module tho! :)

Removed the requests module dependency for version 2.1.4.


Advanced matching

If I wanted to use react-facebook-pixel to do advanced matching -
https://developers.facebook.com/docs/facebook-pixel/pixel-with-ads/conversion-tracking#advanced_match
fbq('init', '<FB_PIXEL_ID>', {
  em: '{{_email_}}',
  // Data will be hashed automatically via a dedicated function in FB pixel
    ph: '{{_phone_number_}}',
    fn: '{{_first_name_}}'
    ....
})

Can you tell me the equivalent of that with your wrapper?
i.e.
ReactPixel.init('yourPixelIdGoesHere', {
  em: '{{_email_}}',
  // Data will be hashed automatically via a dedicated function in FB pixel
    ph: '{{_phone_number_}}',
    fn: '{{_first_name_}}'
    ....
});

Should work the same way, right?

@bkrall Not sure if you stil need this, but I made a pull request to this feature: https://github.com/zsajjad/react-facebook-pixel/pull/6
You can initialize multliple times as well - ReactPixel.init(someId); after login - ReactPixel.init(someId, advancedMatchingInfo).

Merged #6


安装luci-app-passwall提示错误255

Collected errors:

opkg_download: 下载失败 https://op.supes.top/packages/aarch64_generic/packages/libuv1_1.40.0-2_aarch64_generic.ipk, wget returned 8.
opkg_install_pkg: Failed to download libuv1. Perhaps you need to run 'opkg update'?
opkg_install_cmd: Cannot install package luci-app-passwall.

系统→软件包→更新列表


Incompatibility with VS 2022 due to old Cuda toolkit

Nvcc officially supports VS 2022 since v.11.6.
Earlier versions exit with error unless --allow-unsupported-compiler option is explicitly added.

Fixed in

https://github.com/neoml-lib/neoml/pull/778


fix: changed class theme in language toggle to use the base active

instead of link--secondary when selected

Why:
Changes the active theme from Link.secondary to just active
Closes https://github.com/github/docs/issues/268
What's being changed:
Before:
Japanese Example:

English Example:

After:
Japanese Example:

English Example:

Check off the following:

[x] I have reviewed my changes in staging. (look for the deploy-to-heroku link in your pull request, then click View deployment)
[x] For content changes, I have reviewed the localization checklist
[x] For content changes, I have reviewed the Content style guide for GitHub Docs.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (eg. a "before and after video")

@Xenolithes Thanks so much for opening a PR with this fix! I'll get this triaged for review ⚡
I slightly edited your OP to remove the [ ] from around the issue link. Now the issue and PR are linked, so that when the PR is merged, the issue will close automatically ✨


add panel Query Concurrency to dashboard mixin.

[ ] I added CHANGELOG entry for this change.
[x] Change is not relevant to the end user.

Changes
add a panel "Query Concurrency" to the dashboard mixin, showing the available capacity to serve concurrent queries.
Verification
add the dashboard to grafana to check it.
The panel should look like this:

how to rerun the failing test? it seems like a transient error.

Hey @raptorsun, can you have a look at the feedback so that we can proceed with merging your contribution, please? 🙏
To rerun the build in case it fails, you can push an empty commit. Here's how you can create an empty commit: git commit -s --allow-empty -m "Empty commit to trigger CI"

Small tip: maybe rebasing with main can help reducing the random failures. We've been working on this recently.

Small tip: maybe rebasing with main can help reducing the random failures. We've been working on this recently.

The PR has been rebased on latest main branch now.