Introduce an endpoint to serve user avatars

curl --request POST \
  --url 'https://dev-sqa2k3wwnhpxk36d.eu.auth0.com/oauth/token' \
  --header 'content-type: application/json' \
  --data '{ "grant_type":"client_credentials", "client_id":"...", "client_secret":"...", "audience":"https://dev-sqa2k3wwnhpxk36d.eu.auth0.com/api/v2/" }'

How to add a permission to a Machine to Machine application through the Auth0 CLI tenant.yaml:
clientGrants:
  - client_id: Sciety
    audience: https://dev-sqa2k3wwnhpxk36d.eu.auth0.com/api/v2/
    scope:
      - read:users
      - read:clients

(this still has the audience hardcoded for the dev tenant)

https://auth0.com/docs/troubleshoot/customer-support/operational-policies/rate-limit-policy
https://auth0.com/docs/troubleshoot/customer-support/operational-policies/rate-limit-policy/rate-limit-configurations/enterprise-public

We couldn't provoke rate limiting with Apache Bench using this command:
ab -c 120 -n 1200  https://sciety.org/users/scotted400/avatar


Conversion of GIF/PNG to JPEG raises 'cannot write mode P as JPEG'

I'm using pilbox as a proxy to convert images created by some older software that output to
PNG and GIF. For my purposes I need to convert them to JPEG format. Here is the error I get when using either the command line or the tornado application:
$ python  -m pilbox.image --width=300 --height=300 --format=jpeg http://agilitycoursemaps.com/Content/courseMapImages/636373724322750029_Sun11%20-%20Starters%20Standard.gif
Traceback (most recent call last):
  File "/usr/local/Cellar/python/2.7.13/Frameworks/Python.framework/Versions/2.7/lib/python2.7/runpy.py", line 174, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "/usr/local/Cellar/python/2.7.13/Frameworks/Python.framework/Versions/2.7/lib/python2.7/runpy.py", line 72, in _run_code
    exec code in run_globals
  File "/Users/saschwarz/dev/pilbox/pilbox/image.py", line 526, in <module>
    main()
  File "/Users/saschwarz/dev/pilbox/pilbox/image.py", line 520, in main
    preserve_exif=options.preserve_exif)
  File "/Users/saschwarz/dev/pilbox/pilbox/image.py", line 283, in save
    raise errors.ImageSaveError(str(e))
pilbox.errors.ImageSaveError: HTTP 415: Unsupported Media Type (cannot write mode P as JPEG)

I've added an explicit check for the scenario where the output format is JPEG and the mode of the input file is 'P' for Palette. I then apply the solution proposed here:
https://mail.python.org/pipermail/python-list/2000-May/036017.html

Coverage decreased (-0.2%) to 99.455% when pulling 00dc08457be8697e10ec2f87d3a9967338fb7ebb on saschwarz:gif-to-jpeg into 7e4824a01d38414986dc145ba251752468067e28 on agschwender:master.

Will add tests for this and bump version for this over the weekend.

Merged in 9c9b7148f7fb0dfb8060d2a57f344723efe5542c.


Integration test requiring dbt-utils 0.8.4 reduces package compatibility

https://github.com/emilyriederer/dbtplyr/blob/main/integration_tests/packages.yml
It would be great to support dbt-utils 0.9.0+. Is there any reason that the integration tests require specificially 0.8.4? This is preventing compatibility for using this package and packages requiring 0.9.0+.

Done! Apologies - I had been waiting for a bugfix that was messing with the tests. Looks like all is resolved now.
For my education: does the dependency in the test suite versus the main package impede using the package or trying to further develop it? I wouldn't have expected the test suite version to matter, but that's great to know


Update COVID19_Fallzahlen_Kanton_LU_total.csv

change dates to ISO

there is already another pull request with the same changes... closing


math contract more functions

A few more transitions like max, min and average needs to be added in maths contract.
https://github.com/OpenZeppelin/openzeppelin-solidity/blob/master/contracts/math/Math.sol

@iamjaspreetsingh can we close this or are there more functions to be made ?

The functions are completed but I am getting doubt in one of the function i.e. Average.
In reference to
https://github.com/OpenZeppelin/openzeppelin-solidity/blob/master/contracts/math/Math.sol#L18
I need to use the formula (a / 2) + (b / 2) + ((a % 2 + b % 2) / 2) to avoid underflow. Using this generates a runtime error.
I can't use mod function directly.
Can it be the reason?
(At present for mod I replace and calculate it by divide, multiply and subtract.)

as per suggestions from gitter group , we should remove integer operations as those can be handled safely by scilla , we should however implement functions like average , GCD , LCD etc as a library like this .


possible bugs or mis documentation in multiple scopes

I have done the above example and it works , now I add one extra scope which is https://dynamics.microsoft.com/business-central/overview/user_impersonation (I need it for Business central )
but it fails and said bad request, how can I add the extra scope ?
here is what I have done, how can I add the extra scope?
POST {tenant}/oauth2/v2.0/token
Host: login.microsoftonline.com
Content-Type: application/x-www-form-urlencoded
client_id=6731de76-14a6-49ae-97bc-6eba6914391e
&scope=user.read%20openid%20profile%20offline_access%20user_impersonation
<EMAIL_ADDRESS>&password=SuperS3cret
&grant_type=password

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

ID: b3eac288-4ddd-1099-1c86-885aa42a566f
Version Independent ID: 190585ca-8337-0644-2182-75b77f53c175
Content: Use Microsoft identity platform to sign in users using resource owner password credential (ROPC) grant
Content Source: articles/active-directory/develop/v2-oauth-ropc.md
Service: active-directory
Sub-service: develop
GitHub Login: @rwike77
Microsoft Alias: ryanwi

@Farzad-Jalali Thanks for your feedback! We will investigate and update as appropriate.

Hey @Farzad-Jalali
The reason you're getting this error is because only those 4 scopes can be used in that format due to them being required to login. See the permissions docs here for further information in regards to that : https://docs.microsoft.com/en-us/azure/active-directory/develop/v2-permissions-and-consent
See specifically this section on how to request proper permissions : https://docs.microsoft.com/en-us/azure/active-directory/develop/v2-permissions-and-consent#requesting-individual-user-consent
scope=
https%3A%2F%2Fgraph.microsoft.com%2Fcalendars.read%20
https%3A%2F%2Fgraph.microsoft.com%2Fmail.send
&state=12345
So you need to get the App ID URI for your resource accordingly. For your dynamics 365 app id uri see : https://docs.microsoft.com/en-us/dynamics365/business-central/dev-itpro/administration/configure-server-instance
Essentially your url will be :
URI_ENCODED_APP_ID_URI/
@Farzad-Jalali Please let us know if there are anymore questions within the scope of this git issue. If not, I will be closing out this git issue by end of day tomorrow. Please file a new git issue with a reference to this one if you have anymore concerns. Thanks

Hi @FrankHu-MSFT
thanks for the reply,
if you don't mind, could you please update the provided AAD example with your suggestion in a way that it works?
thanks
Farzad

@Farzad-Jalali it would look like the following below.
If you need anymore help in regards to this issue, I would suggest filing a uspport ticket for further engagement.
The request will have the following values per your example.
client_id=6731de76-14a6-49ae-97bc-6eba6914391e
&scope=user.read%20openid%20profile%20offline_access%20<URL_ENCODED_APP_ID_URI>/user_impersonation
<EMAIL_ADDRESS>&password=SuperS3cret
&grant_type=password
As said before I will be closing out this github issue by eod today.


standardize lint setting

We use flake8 and black to lint and format our code, notebooks, and docs. we invoke it by running pkgmt lint read more here
Running black locally might modify unrelated files and apply lint changes based on our local settings (i.e: IDE).
To fix it we can:

Define a standard black lint setting
Make sure our local lint settings don't affect it
Add black phase to CI

References:
Black configuration format
cc: @edublancas @idomic @tonykploomber

this sounds like a duplicate of https://github.com/ploomber/contributing/issues/44


Fix C# distribtest on kokoro macos

Kokoro mac workers have a new version of mono, which picks up DistribTest.project.json file in the same directory. Deleting the project.json files fixes the problem (and we should migrate the distribtest to use the "new" .csproj format in the future - project.json files are deprecated).

Confirmed that is actually fixes the distribtest.


[BUG] CPU Memory Leakage when using reader_tfds

I have found that the CPU memory usage exhibits a abnormal increase when a new epoch is about to start. I have been able to located the problem down to reader_tfds.py, as I am reading from a set of tfrecord files and I do not observe this issue when I am directly reading from images files like jpeg or png. This is not a big issue in the current implementation, as the real shuffle size for each worker is divided by global_num_workers here, significantly decreasing the cpu memory usage. So I guess a little memory usage increase every epoch would not cause a big problem.
However, I'd like to use a larger shuffle size as I find the default value TFDS_SHUFFLE_SIZE=8192 results in worsened accuracy.  For example, if my understanding is correct, the default value in reader_wds.py is also WDS_SHUFFLE_SIZE=8192, but the critical difference here is that reader_wds.py create a shuffle buffer of size 8192 for each worker, while in the reader_tfds.py the total (all workers together) shuffle buffer is of size 8192. Yet, a larger shuffle size (like 32x or 64x) means my program get killed every a few epochs for out-of-memory error.
I have been able to pinned down the root cause as the persistent_workers=True argument when creating pytorch dataloader. Since this argument allows the dataset instance to be kept alive across epochs, I wonder is it because each epoch a new dataset instance is created, but the old ones are not released? What if I set persistent_workers=False? Is that the right way to fix this problem? I have also seen a related issue when reading webdataset data, but it seems to me whenever a new epoch is about to start, the set_epoch method is called here, so we do not actually need to keep the dataset instances alive all the time?
Thanks a lot!

@rwightman You are right! After the setting up using the tcmalloc, the issue is gone. So I guess the problem lies in the memory allocator, but not in the timm code. This is interesting, though, as I am using a gpu machine, but not a tpu-vm instance. I thought that issue only exists with PyTorch XLA on a TPU-VM and TFDS, but clearly I was wrong.


input without default style after update

Current Behavior
input without default style after update
Before

After

Expected Behavior
input with default style
Tamagui Version
1.39.3

Reproduction
Put an input on the screen, and see that it has no default style

System Info
No response

Needs platform and repro, not seeing it myself


Support generics in the object_wrapper macro

Before I add this to the outer wrapper macro, is this something we are comfortable with? should I add it also to glib_boxed_wrapper, glib_boxed_inline_wrapper and glib_shared_wrapper?

After discussion in the chat channel, the updated patch:

handles multiple bounds (eg <T: A + B>)
uses ident instead of tt for the generic name (it still uses tt for the bounds since they can be both types and lifetimes and we do not care about making the distinction)
it is implemented for all wrapper macros (object, boxed, boxed_inline, and shared)

After discussion in the chat channel, the updated patch:

handles multiple bounds (eg <T: A + B>)
uses ident instead of tt for the generic name (it still uses tt for the bounds since they can be both types and lifetimes and we do not care about making the distinction)
it is implemented for all wrapper macros (object, boxed, boxed_inline, and shared)

Adding some notes (also for myself):

The wrapper macro with generics seems to work (yay!)
To make Object work I had to add PhantomData with the generic in the struct.  I added them for task, not sure it is what we want
The patch includes the port of Task to try out the new functionality. However the open questions in https://github.com/gtk-rs/gtk-rs-core/pull/443 with regard to the API and implementation still hold true

Adding some notes (also for myself):

The wrapper macro with generics seems to work (yay!)
To make Object work I had to add PhantomData with the generic in the struct.  I added them for task, not sure it is what we want
The patch includes the port of Task to try out the new functionality. However the open questions in https://github.com/gtk-rs/gtk-rs-core/pull/443 with regard to the API and implementation still hold true

It sucks that if you use T as the name of the generic type things explode because it conflicts with what used internally in object.rs

Maybe we could switch to something less common than T?

It sucks that if you use T as the name of the generic type things explode because it conflicts with what used internally in object.rs

Maybe we could switch to something less common than T?

It sucks that if you use T as the name of the generic type things explode because it conflicts with what used internally in object.rs

Maybe we could switch to something less common than T?

I now changed the internal use to OT since it is an ObjectType. I wonder if there is a better workaround to make generic types hygienic rather than "pick an uncommon name"

I think this is now more or less ready, so I am dropping the "Draft" flag. I have updated the title and initial description to be more accurate.
Known things to check before merging:
[ ] two lines in the task part that need double checking (see comments above)
[ ] one change in the macro part that changes the macro logic a bit (see comment above)
[ ] the macro for boxed, boxed_inline, and shared do not yet have the PhantomData to hold generics... I would rather do this as a second step once there is a use case to test with
[ ] check if you agree with the API changes for task, some of them are my preference but are not strictly needed for the soundness issue
Clearly any other feedback outside of the list above is also welcome

As discussed in chat, this last patch reworks the Task API and splits LocalTask and Task... see commit message for details


Cfn autoexpand capability

Please check if the PR fulfills these requirements

[x] The commit message describes your change
[na] Tests for the changes have been added if possible (for bug fixes / features)
[na] Docs have been added / updated (for bug fixes / features)
[x] Changes are mentioned in the changelog (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Allowed CAPABILITY_AUTO_EXPAND  as in Capabilities.member.N

What is the current behavior? (You can also link to an open issue here)
Stacks being updated with template macros display an error on stack create/update:
Requires capabilities : [CAPABILITY_AUTO_EXPAND]

What is the new behavior (if this is a feature change)?
Stacks with template macros can be created

Does this PR introduce a breaking change? (What changes might users need to make in their setup due to this PR?)
No business logic has been changed.

Other information:

Hah, I didn't even see the Issue already listed! 😄


lora-phy: refactor sx127x to have chip trait instead of enum

This is a rebased and conflict-resolved branch that supersedes #221

Also, this PR reduces binary size, for example the stm32l0 with LTO enabled:

lora_lorawan 94.4KiB -> 93.1KiB
lora_p2p_send 45.3KiB -> 44.3KiB
lora_p2p_receive 52.1KiB -> 52.0KiB


option scoreOnly cannot work

Expected behavior
When option scoreOnyl is used, it should return the similarity score which is a floating type.
Actual behavior
With the option scoreOnly on, the score value is -nan and the volume of reference and database molecule are 0.
Step to reproduce scoreOnly failure

shape-it -r ex1.sdf -d ex1.sdf -o ex1_aligned.sdf -s ex1_aligned.score --scoreOnly

+++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
  Shape-it v2.0.0 | Aug  9 2022 19:11:26

  -> GCC:       11.2.0
  -> OpenBabel: 3.1.0

  Copyright 2012-2021 by Silicos-it, a division of Imacosi BVBA, Hans De Winter,
     and the Shape-it contributors

  Shape-it is free software: you can redistribute it and/or modify
  it under the terms of the MIT license.

  Shape-it is distributed in the hope that it will be useful,
  but WITHOUT ANY WARRANTY; without even the implied warranty of
  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  MIT License for more details.

  Shape-it is linked against OpenBabel version 2.
  OpenBabel is free software; you can redistribute it and/or modify
  it under the terms of the GNU General Public License as published by
  the Free Software Foundation version 2 of the License.
+++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

COMMAND_LINE OPTIONS:

  -> Reference file:    ex1.sdf
  -> Database file:     ex1.sdf
  -> Output file:       ex1_aligned.sdf
  -> Scores file:       ex1_aligned.score
  -> Best hits:         no
  -> Scoring only:      yes
  -> Extra iterations:  no
  -> Rank by:           Shape-it::Tanimoto
  -> Cutoff:            no
  -> Output reference   yes

Processed 1 molecules
1 molecules in 0.022083 seconds (45.2837 molecules per second)

show scoring file:

$ cat ex1_aligned.score
dbName  refName Shape-it::Tanimoto      Shape-it::Tversky_Ref   Shape-it::Tversky_Db    overlap refVolume       dbVolume
EX1     EX1     -nan    -nan    -nan    0       0       0

This issue was solved. #18


[Request For External] codeunit 7000 "Sales Price Calc. Mgt."

Hi,
Is it possible to get these functions global:
    local procedure SalesHeaderExchDate(SalesHeader: Record "Sales Header"): Date
    begin
        with SalesHeader do begin
            if "Posting Date" <> 0D then
                exit("Posting Date");
            exit(WorkDate);
        end;
    end;

    local procedure SalesHeaderStartDate(var SalesHeader: Record "Sales Header"; var DateCaption: Text[30]): Date
    begin
        with SalesHeader do
            if "Document Type" in ["Document Type"::Invoice, "Document Type"::"Credit Memo"] then begin
                DateCaption := FieldCaption("Posting Date");
                exit("Posting Date")
            end else begin
                DateCaption := FieldCaption("Order Date");
                exit("Order Date");
            end;
    end;

    local procedure ServHeaderExchDate(ServHeader: Record "Service Header"): Date
    begin
        with ServHeader do begin
            if ("Document Type" = "Document Type"::Quote) and
               ("Posting Date" = 0D)
            then
                exit(WorkDate);
            exit("Posting Date");
        end;
    end;

    local procedure ServHeaderStartDate(ServHeader: Record "Service Header"; var DateCaption: Text[30]): Date
    begin
        with ServHeader do
            if "Document Type" in ["Document Type"::Invoice, "Document Type"::"Credit Memo"] then begin
                DateCaption := FieldCaption("Posting Date");
                exit("Posting Date")
            end else begin
                DateCaption := FieldCaption("Order Date");
                exit("Order Date");
            end;
    end;

Thanks,
Robin Fjelldal

This codeunit have been obsoleted in 16.0 and will be removed in 19.0 (Wave 2 2021). Are you sure you would like to use it or better switch to new pricing engine?
codeunit 7000 "Sales Price Calc. Mgt."
{
ObsoleteState = Pending;
ObsoleteReason = 'Replaced by the new implementation (V16) of price calculation.';
ObsoleteTag = '16.0';

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


[codegen] Use layout in data builder names

layout properly escapes type names in case of name clash.
See https://github.com/apollographql/apollo-kotlin/issues/5537

🛠 Docs Preview Building...

Configuration
{
  "repoOverrides": {
    "apollographql/apollo-kotlin@main": {
      "remote": {
        "owner": "apollographql",
        "repo": "apollo-kotlin",
        "branch": "escape-data-builders"
      }
    }
  }
}

4a2a50d0b5b1862dc4647b30e4e15509b4814e65


Using it with electron builder ?

Hi, I try to build using electron builder, and after build, it seems cannot open the 'app://-' location, is it because the
"build" folder that contains single-page web app files is not accessible ?
My electron builder config :
  ...
  "build": {
    "extraFiles": [
      "build"
    ],
  ...

the "build" folder is the single-page web app folder
Anybody has clue ?
Thanks

Found the cause : cannot use build folder name for static files

Hi @x4080, i'm having the same issue, could you share your setup to make it work with electron builder?


[Bug][tapd] error on extract namespace from repository url

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
Failed to extract namespace part of a repo url like : https://gitlab.com/merico-dev/ee/tapd-test.git
Namespace should be "merico-dev/ee", it only extracted first segment in url path "merico-dev".

What do you expect to happen
Use url path without the last segment as namespace
How to reproduce
repository url path has more than 2 segments, e.g. https://gitlab.com/merico-dev/ee/tapd-test.git  has 3 segments "merico-dev", "ee", "tapd-test.git"
Anything else
No response
Version
v0.18
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I think this procedure will work when you want to extract issue's relation commits info when you using jira.
Is it right ? @leric (If yes, please update the description, and make it clear and understable.)
As to the question you have mentioned, the namespace field is  extract by a regexp, and  it's configurated in blueprint or scope's config. Please paste your config  content, so we have dive into it.
cc @narrowizard

Not in this case, it just split out the first segment of url path:
https://github.com/apache/incubator-devlake/pull/5994/files#diff-d28cea07c6ce6ec7e3a62264712c15c4f156bf04abc53c362353039b512bc830

I don't think there are configuration like jira's remotelinkRepoPattern in tapd plugin. @d4x1

remotelinkRepoPattern

You are right.  I only find remotelinkRepoPattern in Jira plugin.


Download binaries?

Hi,
Where can I download binaries?
regards,

You can download via releases section.
https://github.com/burki169/kbvault/releases


Monero miner for Bjorn

Posible have a monero miner for bjorn i foudn this repo :
https://gist.github.com/henrytriplette/0ddea865d0d46060c7c9227034a73127
https://github.com/tpruvot/cpuminer-multi

What is this? May i ask

Sorry not planned, out of rpi resources and not compatible with the project.


⚠ Processors has degraded performance

In 9fac2bf, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 125 ms

Resolved: Processors performance has improved in 32453a8.


Get QA Auth Smoke Tests passing

Description
Get rid of the --record option when running auth tests against QA. The big con is that if the tests fail, we'll have to manually run them locally to see what the problem is, as the --record option lets us save videos where we can see the failures.
I spent about 8 hours trying to get the tests to pass on CircleCI and this is all I could figure out to get them passing. Here's what I tried, none of which worked; if anybody else has any other ideas, I'm happy to give them a shot first.

Upped the CircleCI resource_class from medium (the default) to both medium+ and large.
Tried a newer Docker image, that went from Node 16 to Node 18 (as well as updated the Java version). Not only did it not work, it also broke the regular integration tests. I didn't bother looking into the regular failures since it didn't fix the smoke tests anyway.
Tried a different testing reporter, teamcity instead of junit.
Tried a new cache
Tried restructuring the auth-tests.ts file.

Some additional observations:

Auth tests run locally against QA with the record option, although I'm on a Mac/ARM, so it's not an exact comparison.
The non-auth tests run on CircleCI against QA with record with no problems.

Review Instructions
Nightly auth tests should be passing on CircleCI.
Issue
SEAB-6459
Security
If there are any concerns that require extra attention from the security team, highlight them here.
Please make sure that you've checked the following before submitting your pull request. Thanks!

[x] Check that your code compiles by running npm run build
[x] Ensure that the PR targets the correct branch. Check the milestone or fix version of the ticket.
[x] If this is the first time you're submitting a PR or even if you just need a refresher, consider reviewing our style guide
[x] Do not bypass Angular sanitization (bypassSecurityTrustHtml, etc.), or justify why you need to do so
[x] If displaying markdown, use the markdown-wrapper component, which does extra sanitization
[x] Do not use cookies, although this may change in the future
[x] Run npm audit and ensure you are not introducing new vulnerabilities
[x] Do due diligence on new 3rd party libraries, checking for CVEs
[x] Don't allow user-uploaded images to be served from the Dockstore domain
[x] If this PR is for a user-facing feature, create and link a documentation ticket for this feature (usually in the same milestone as the linked issue). Style points if you create a documentation PR directly and link that instead.
[x] Check whether this PR disables tests. If it legitimately needs to disable a test, create a new ticket to re-enable it in a specific milestone.

I don't remember the backstory on this.
Why do only the test-local-no-auth, test-qa-no-auth and test-qa-auth jobs have --record in the first place. i.e. we don't record in staging or prod

I don't know the background, but my guess is because QA is changing daily, the possibility for breakage is higher, and it's more convenient to have the artifacts so you can debug. Whereas once the tests are passing in staging/prod, it'll be months before the possibility of them breaking again happens (unless they tests themselves change).
For test-local-no-auth, my guess is so that we could test the Cypress Cloud/record integration.

Overall, not objecting, but how did you come to suspect this as a potential fix and is there a theory for why? Or why it continues to work for those other jobs?

I looked at what's different between the passing and failing jobs and the --record was a thing. I was surprised it made a difference, especially given that test-qa-no-auth works with the switch.
After that worked, I looked around, and there were some bug reports/comments of --record being an issue. Here (old ticket but still) and here.
Also, FWIW, I noted this in the JIRA issue but not in the PR, these failures started when Cypress 13 was merged to develop.
In any case, my theory is there is some flakiness in Cypress. :)

Does staging-auth work with recording?
Also, this probably won't make a difference but I noticed that there's a new browser-tools version so could try updating that to 1.4.8 https://github.com/dockstore/dockstore-ui2/blob/6a4ee281ce5a18e7daf1ff7ca8b40130ae4a52c6/.circleci/config.yml#L6

Does staging-auth work with recording?

Good idea, that would see if the problem stems from something new in Dockstore 1.16 or if it's purely Cypress.

Also, this probably won't make a difference but I noticed that there's a new browser-tools version so could try updating that to 1.4.8
https://github.com/dockstore/dockstore-ui2/blob/6a4ee281ce5a18e7daf1ff7ca8b40130ae4a52c6/.circleci/config.yml#L6

I think I tried that and it didn't make a difference. But I'll try again to be sure, and we might as well update anyway.

"Get rid of the --record option when running auth tests against QA"

Recording the tests was causing them to fail? If yes, is there clues/evidence as to why that's happening?

Ooops, nevermind. missed some of the comments, they discuss it.

Does staging-auth work with recording?

It also failed against staging with recording.

Thanks for testing it!


How to install node csgo web tradebot ?

I set up this script as follows. But I can not login in the website.

Installed DigitalOcean.com 64Bit Ubuntu
sudo apt-get update
sudo apt-get install nginx
sudo apt-get install mysql-server
sudo mysql_install_db
sudo mysql_secure_installation
sudo apt-get install php5-fpm php5-mysql php5-mcrypt
sudo apt-get install curl libcurl3 libcurl3-dev php5-curl
sudo php5enmod curl
curl -sL https://deb.nodesource.com/setup_0.12 | sudo bash -
sudo apt-get install -y nodejs
mkdir -p /home/www/html
Uploaded web files
and no login to websites.

NOTE: The work you do is really worth a job. But I really ask you to share some explanations for those who want to do this job. At least I think it should be an installation instruction.
Thank you.

you should only set up nginx as a reverse proxy, you simply run the script with node index.js. After changing the details in the config file.

If you have 0 experience you should not run this.

Thank you IDefineHD.

Np. Close the issue please.


Unsupported URL

First off, this is a phenomenal plugin! It saved me years of work during development.
Now, trying to use it in production, I have come to realize that HCP does not download updates from S3. I am not using the JS Module, but when trying chcp.fetchUpdate() with callbacks, it returns

{description: "unsupported URL", code: -1}

What could this be? My bucket is in eu-central-1 and the content_url is set to s3-eu-central-1.amazonaws.com/bucketname (tried with or without slash in the end).
Any ideas?

Okay, got it. You need to add
<chcp>     <config-file url="https://s3-eu-central-1.amazonaws.com/bucketname/chcp.json"/> </chcp>
to config.xml.
Now it's working nicely.

Yes, you need to tell plugin, from what location to download chcp.json :) If it's solved - closing the issue?

Oops, sorry – yes, it's solved :-)


add proper CLI

We should consider what features we want from a demes CLI, and how we expect folks to use it. Probably format conversion would be most useful. Maybe some options like:
usage: python -m demes [-s] [-f] [-y] [-j] [input.yaml | input.json]  > output.format

-s --simplified        output simplified model (default for YAML and JSON output)
-f --fully-resolved    output fully-resolved model
-y --yaml              output YAML (default output)
-j --json              output JSON

Or maybe also support an ms subcommand to convert ms command lines? That will need a mandatory N0 option to convert units. Maybe that would work better as it's own python -m demes.ms command?

Probably better to have some subcommands, I think. I can imagine validate being a very useful one.


Permission issue preveting posting on sandbox dockers

when creating containers from Sandbox artifacts, a permission error prevents certain transactions from being posted.

You need to give me a little more than this.
There are 1000s of containers started every day on sandbox artifacts, where this doesn't happen, so you need to at least provide script and output if you want any assistance.

Hello Freddy
Thank you for your response and sorry for the delay in mine.
I have the feeling that the phenomenon occurs with all sandbox containers.
Scritp:
$credential = New-Object pscredential 'admin', (ConvertTo-SecureString -String @.' -AsPlainText -Force)
New-BcContainer -accept_eula -containerName BC250W1T -artifactUrl https://bcartifacts-exdbf9fwegejdqak.b02.azurefd.net/sandbox/25.0.23364.25359/w1 -assignPremiumPlan -auth NavUserPassword -Credential $credential -isolation hyperv -shortcuts None -updateHosts
last artifact BC is used.
@.
Thanks again,
Pedro
De : Freddy Kristiansen @.>
Envoyé : jeudi 10 octobre 2024 07:49
À : microsoft/navcontainerhelper @.>
Cc : PERERA AGUILA Pedro @.>; Author @.>
Objet : Re: [microsoft/navcontainerhelper] Permission issue preveting posting on sandbox dockers (Issue #3680)
Closed #3680https://github.com/microsoft/navcontainerhelper/issues/3680 as completed.

Reply to this email directly, view it on GitHubhttps://github.com/microsoft/navcontainerhelper/issues/3680#event-14584170172, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BFZ7PEREYWQAC7DD7GHTDGDZ2YIMDAVCNFSM6AAAAABO2D25QGVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJUGU4DIMJXGAYTOMQ.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

Yeah, well the error message leaves a lot for interpretation/guessing - which wastes a lot of time for people looking at this.
If you could describe what you are doing to provoke this - whether this happens in 24 versions as well and a lot more information, which makes it possible for us to run the script and do the same things - or maybe even say that this is expected with the license file used.

Thanks a lot,
important to specify that this problem does not exist on on-prem artifacts
De : Freddy Kristiansen @.>
Envoyé : jeudi 10 octobre 2024 10:27
À : microsoft/navcontainerhelper @.>
Cc : PERERA AGUILA Pedro @.>; Author @.>
Objet : Re: [microsoft/navcontainerhelper] Permission issue preveting posting on sandbox dockers (Issue #3680)
Yeah, well the error message leaves a lot for interpretation/guessing - which wastes a lot of time for people looking at this.
If you could describe what you are doing to provoke this - whether this happens in 24 versions as well and a lot more information, which makes it possible for us to run the script and do the same things - or maybe even say that this is expected with the license file used.

Reply to this email directly, view it on GitHubhttps://github.com/microsoft/navcontainerhelper/issues/3680#issuecomment-2404436234, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BFZ7PERRDUDINOKYSV5DURTZ2Y23XAVCNFSM6AAAAABO2D25QGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBUGQZTMMRTGQ.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

with BC24 there is the same problem.
We try to post a simple General Journal.
The license file used is that of demo, cronus by default.
Thanks Freddy for your answers.. (we get around the problem by using on-prem artifacts)
De : Freddy Kristiansen @.>
Envoyé : jeudi 10 octobre 2024 10:27
À : microsoft/navcontainerhelper @.>
Cc : PERERA AGUILA Pedro @.>; Author @.>
Objet : Re: [microsoft/navcontainerhelper] Permission issue preveting posting on sandbox dockers (Issue #3680)
Yeah, well the error message leaves a lot for interpretation/guessing - which wastes a lot of time for people looking at this.
If you could describe what you are doing to provoke this - whether this happens in 24 versions as well and a lot more information, which makes it possible for us to run the script and do the same things - or maybe even say that this is expected with the license file used.

Reply to this email directly, view it on GitHubhttps://github.com/microsoft/navcontainerhelper/issues/3680#issuecomment-2404436234, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BFZ7PERRDUDINOKYSV5DURTZ2Y23XAVCNFSM6AAAAABO2D25QGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBUGQZTMMRTGQ.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>


chore: rename BasicTracer to SDKTracer

Which problem is this PR solving?

Short description of the changes

This change is

Codecov Report

Merging #12 into master will decrease coverage by 0.04%.
The diff coverage is 100%.


live metrics release: update top level banner

Minor update. I think we need to move on from the MR now and use a bit different message. It won't be affecting the MLEM.ai website.

It will work for dvc.org and cml.dev.
For iterative.ai it still needs some changes https://github.com/iterative/iterative.ai/blob/main/src/components/NavBar/Alert/index.js. We should be able to update this to use the same content.

Yep, I can update iterative.ai as a next step. MLEM can stay as is.


Allow migration to be reversed

Add the down function to the migration file to allow it to be reversed without causing any issues when rolling back the migrations

Hey, thanks for taking the time to PR this, however personally I  don't ever use the down function in migrations so it doesn't really make sense for me to merge it. You can always add the down function into the migration yourself when publishing the migrations.


Flip phone device overlay

A new overlay for flip phone device emulation is needed (currently by SquirrelJME). Note that any overlays for ColecoVision, Intellivision, and Atari Jaguar systems will not work here due to lack of information on buttons and the number of available buttons.
Specifically the following keys are required:

Up
Down
Left
Right
Select
Soft 1 (Top-left button)
Soft 2 (Top-right button)
Talk/Call (vertical green phone facing left)
End/Hangup (horizontal red phone facing down)
Camera
Back
Speaker
Volume Up
Volume Down
1 🖂 or 1 ➿ (Voicemail)
2 ABC
3 DEF
4 GHI
5 JKL (tactile aid)
6 MNO
7 PQRS
8 TUV
9 WXYZ
* Shift (Star, Shift)
One of these pairs:

Space on zero:

0 + ␣  (Zero, Plus, Space)
# (Pound)

Space on pound:

0 +  (Zero, Plus)
# ␣ (Pound, Space)

This can be seen in the following picture of a Kyocera Dura XT:

Modifier keys like shift are weird in RetroArch. You can't just call shift plus whatever and have it do what it's supposed to, but if someone presses and holds shift on the overlay and then presses another key, it should work. Is that good enough?

There is no actual shift key on phones, that key is just the star key. If you press it, it would send * to the device (same with the number keys and pound for their respective symbol). The stuff below the big number part is just a helper because there will be text input on the device where you have to press a button to enter in a letter (for example b requires pressing 2 twice, but the device will only ever know that 2 was pressed). I will note though that having the extra letters and symbols below the primary symbol just have to be on the key graphic itself.

I do have some questions, though: what are you mapping these functions to in the core? It looks like too many things for the retropad, so I'm assuming retrokeyboard?

That is to be determined whether it be a Retropad or Retrokeyboard. The minimum set of buttons are: Up, Down, Left, Right, Select, Back, End, Soft 1 (L), Soft 2 (R), 1, 2, 3, 4, 5, 6, 7, 8, 9, 0, Star, and Pound.
SquirrelJME right now (outside of RetroArch) has the following key bindings on the keyboard:

Up/Down/Left/Right/0/1/2/3/4/5/6/7/8/9/Star/Pound to their respective keys.
Back to Backspace.
End to Escape.
Select to Enter.
Soft 1 (L) to F1.
Soft 2 (R) to F2.

Okay, so I defined a bare minimum that will fit on RetroPad and it is defined as follows:
/* SOFT LEFT. */
{0, RETRO_DEVICE_JOYPAD, 0,
	RETRO_DEVICE_ID_JOYPAD_SELECT, "Soft 1 (Left Shoulder)"},

/* SOFT LEFT. */
{0, RETRO_DEVICE_JOYPAD, 0,
	RETRO_DEVICE_ID_JOYPAD_START, "Soft 2 (Right Shoulder)"},

/* [1  o~o] */
{0, RETRO_DEVICE_JOYPAD, 0,
	RETRO_DEVICE_ID_JOYPAD_X, "1 Voicemail (Game A)"},

/* [2  abc] */
{0, RETRO_DEVICE_JOYPAD, 0,
	RETRO_DEVICE_ID_JOYPAD_UP, "2 ABC (Game Up)"},

/* [3  def] */
{0, RETRO_DEVICE_JOYPAD, 0,
	RETRO_DEVICE_ID_JOYPAD_A, "3 DEF (Game B)"},

/* [4  ghi] */
{0, RETRO_DEVICE_JOYPAD, 0,
	RETRO_DEVICE_ID_JOYPAD_LEFT, "4 GHI (Game Left)"},

/* [5  jkl] */
{0, RETRO_DEVICE_JOYPAD, 0,
	RETRO_DEVICE_ID_JOYPAD_L2, "5 JKL (Game Fire/Select)"},

/* [6  mno] */
{0, RETRO_DEVICE_JOYPAD, 0,
	RETRO_DEVICE_ID_JOYPAD_RIGHT, "6 MNO (Game Right)"},

/* [7 pqrs] */
{0, RETRO_DEVICE_JOYPAD, 0,
	RETRO_DEVICE_ID_JOYPAD_Y, "7 PQRS (Game C)"},

/* [8  tuv] */
{0, RETRO_DEVICE_JOYPAD, 0,
	RETRO_DEVICE_ID_JOYPAD_DOWN, "8 TUV (Game Down)"},

/* [9 wxyz] */
{0, RETRO_DEVICE_JOYPAD, 0,
	RETRO_DEVICE_ID_JOYPAD_B, "9 WXYZ (Game D)"},

/* [*  shf] */
{0, RETRO_DEVICE_JOYPAD, 0,
	RETRO_DEVICE_ID_JOYPAD_L, "Star (Shift)"},

/* [0    +] */
{0, RETRO_DEVICE_JOYPAD, 0,
	RETRO_DEVICE_ID_JOYPAD_R2, "0 Operator (Plus)"},

/* [# spac] */
{0, RETRO_DEVICE_JOYPAD, 0,
	RETRO_DEVICE_ID_JOYPAD_R, "Pound (Space)"},

It looks like this:

Ok, that's very doable. Would you care to try this one I whipped up? It's meant for portrait orientation:
https://www.mediafire.com/file/brvy3w3fnmnxfm9/j2me_overlay.zip/file

Ok, that's very doable. Would you care to try this one I whipped up? It's meant for portrait orientation:
https://www.mediafire.com/file/brvy3w3fnmnxfm9/j2me_overlay.zip/file

Hi! This one will work! Thank you!

hey, no problem :)
I'll get it pushed into the repo tomorrow.

hey, no problem :)
I'll get it pushed into the repo tomorrow.

Cool! Thank you!

Alright, closed by ecccb45912d36ad0506f96783da03eda6afbb84d
Let me know if it needs anything else added to it in the future.


What is vis_utils?

Hi Alex, thank you for sharing this great work.
I am currently having a hard time tracing this line:
https://github.com/alexrich021/3dvnet/blob/8ea11dca65964d952986fe33ef7b922f0aa0c300/mv3d/dsets/dataset.py#L254
Neither was it mentioned in the readme file nor in the dependencies. Could you please let me know what this vis_utils pkg links to?
Thank you!

Problem solved. I can raise a pr if needed. Thanks.


Fix message formatting in the IP validator

Q
A

Is bugfix?
yes

New feature?
no

Breaks BC?
no

Tests pass?
yes

Fixed issues

yii\validators\Validator::formatMessage() accepts two parameters. Three were provided.

Merged. Thanks!

Glad I could help!


Is this how hidenavbar should work?

if I have
Router hideNavBar={true} I get a screen like this (with the little '<' on the top). Also it shows the title from that Route.
http://screencast.com/t/ugUrtfCcW
if I have  Router hideNavBar={false}, it does not show the '<' and shows the title of the first route.
http://screencast.com/t/2T7Bpj04W8
Is this the correct behaviour?

No, "<" is not displayed when current route (scene) if first within navigation stack (i.e. have 'switch' type or have wrapRouter function or inner Router)

the behaviour I described is in the second route (scene) of the application. hidenavbar(true) shows the < and false hides it.

Post your full routes here.

<Router hideNavBar={true}>
          <Schema name="modal" sceneConfig={Navigator.SceneConfigs.FloatFromRight}/>
          <Schema name="default" sceneConfig={Navigator.SceneConfigs.FloatFromRight}/>
          <Schema name="withoutAnimation"/>
          <Schema name="tab" type="switch" icon={TabIcon} />

          <Route name="location" component={Location} title="Property Location" initial={true} wrapRouter={true} />
          <Route name="quote" component={Quote} title="Quote"/>
      </Router>

As you can see "location" has wrapRouter which means it runs under his own navigator (that is why you don't see "<")

Thanks much!


Update code generator for PySAM

Current code generator options do not provide inputs in the same format as used in PySAM. While conversion is possible, directly producing PySAM formatted input json files would allow for quicker setup as well as direct comparison between python and the GUI.

@dguittet - what is the PySAM formatted JSON input?

@sjanzou It's nested by SSC group names. Here's an example from GenericSystem:
{
	'Plant': {
		'conv_eff': 34.11804962158203,
		'derate': 4.0,
		'energy_output_array': (0.0, ),
		'heat_rate': 10.0,
		'spec_mode': 0.0,
		'system_capacity': 100000.0,
		'user_capacity_factor': 43.6
	},
	'Lifetime': {
		'analysis_period': 25.0,
		'generic_degradation': (0.0, ),
		'system_use_lifetime_output': 0.0
	},
	'AdjustmentFactors': {
		'constant': 0.0
	}
}

From @dguittet, Here’s a short example of importing a SAM case from the JSON exported: https://nrel-pysam.readthedocs.io/en/2.0.2/Import.html
One nice thing that might not be as time-consuming as the original idea but which would be greatly helpful is if the JSON from the code generator could be grouped by compute_modules.
Right now it’s a giant single json, which is why we have to call “dict_to_ssc_table” twice and do an unweidly “so.assign(Singleowner.wrap(so_dat).export())”.
If we could separate the json by compute module, then we can call the “value” function like this, where “filename” is read in from the Pvwatts json.
system_model.value('solar_resource_file', filename)
Using this method, the nesting by Group names isn’t even required :D

Updated and separate JSON file for each compute module. However, pySAM value function fails for adjust:constant

Commit and push to SAM_217 branch of SAM and merge into patch and develop branches and close issue

Just a couple of items!

the “adjust:constant” variable doesn’t work as a PySAM variable since you can’t do gs.adjust:constant so I had taken out the “adjust” prefix. Here’s a list of all the adjustment factor variable [names] (https://nrel-pysam.readthedocs.io/en/master/modules/Pvwattsv7.html#adjustmentfactors-group)

“nan” isn’t legit JSON. Could there be a check so that Nan-valued variables aren't printed?

Update and strip out prefix for now and test several cases. The link provided does not have all the adjustment factors for all of the different technologies. Also, no NaN values encountered in inputs for CSP, PV, Generic generations. Issue with MSPT:

Python code much simpler and running example for generic_system single owner:

Moving out of patch 2 milestone until resolution for all adjustment factors in flat JSON file determined. Fixes and running code checked into SAM_217 branch of SAM and merged into patch and develop branches.

Working example script for PySAM and generic single owner PySAM JSON code generation:
import json
import PySAM.GenericSystem as GenericSystem
import PySAM.Singleowner as Singleowner
from PySAM.PySSC import *
ssc = PySSC()
dic = json.load(open("untitled_generic_system.json"))
gs = GenericSystem.new()
for key, value in dic.items():
if key != 'number_inputs' :
gs.value(key,value)
#grid compute module here when in PySAM
dic = json.load(open("untitled_singleowner.json"))
so = Singleowner.from_existing(gs)
for key, value in dic.items():
if key != 'number_inputs' :
so.value(key,value)
gs.execute()
so.execute()
print ('\n Annual Energy (year 1) = ', gs.Outputs.annual_energy , ' (kWh)\n PPA price (year 1) = ' , so.Outputs.ppa, '\u00a2/kWh\n')

Script is now working for MSPT


When I used svtr_tiny_ch.yaml to configure training, there was a loss of nan

When I used svtr_tiny_ch.yaml to configure training, I encountered a Loss of nan.I used the pre-trained model of svtr_tiny_ch-2ee6ade4.ckpt.I am using the GPU version of MindSpore.The display is as follows:
[WARNING] ME(3613:140537237112640,ForkServerPoolWorker-1:1):2024-03-07-06:39:44.569.244 [mindspore/train/summary/_summary_adapter.py:304] There are no valid values in the ndarray(size=40960, shape=(1, 640, 64))
[WARNING] ME(3613:140537237112640,ForkServerPoolWorker-1:1):2024-03-07-06:39:44.569.675 [mindspore/train/summary/_summary_adapter.py:304] There are no valid values in the ndarray(size=864, shape=(32, 3, 3, 3))
[WARNING] ME(3613:140537237112640,ForkServerPoolWorker-1:1):2024-03-07-06:39:44.569.892 [mindspore/train/summary/_summary_adapter.py:304] There are no valid values in the ndarray(size=32, shape=(32,))
[WARNING] ME(3613:140537237112640,ForkServerPoolWorker-1:1):2024-03-07-06:39:44.570.100 [mindspore/train/summary/_summary_adapter.py:304] There are no valid values in the ndarray(size=32, shape=(32,))
[WARNING] ME(3613:140537237112640,ForkServerPoolWorker-1:1):2024-03-07-06:39:44.570.365 [mindspore/train/summary/_summary_adapter.py:304] There are no valid values in the ndarray(size=18432, shape=(64, 32, 3, 3))
[2024-03-07 06:40:10] mindocr.utils.callbacks INFO - epoch: [1/3] step: [100/1000], loss: nan, lr: 0.000033, per step time: 879.835 ms, fps per card: 18.19 img/s
[WARNING] ME(3621:140537237112640,ForkServerPoolWorker-1:9):2024-03-07-06:40:10.246.843 [mindspore/train/summary/_summary_adapter.py:304] There are no valid values in the ndarray(size=40960, shape=(1, 640, 64))
[WARNING] ME(3621:140537237112640,ForkServerPoolWorker-1:9):2024-03-07-06:40:10.247.187 [mindspore/train/summary/_summary_adapter.py:304] There are no valid values in the ndarray(size=864, shape=(32, 3, 3, 3))
[WARNING] ME(3621:140537237112640,ForkServerPoolWorker-1:9):2024-03-07-06:40:10.247.432 [mindspore/train/summary/_summary_adapter.py:304] There are no valid values in the ndarray(size=32, shape=(32,))
[WARNING] ME(3621:140537237112640,ForkServerPoolWorker-1:9):2024-03-07-06:40:10.247.616 [mindspore/train/summary/_summary_adapter.py:304] There are no valid values in the ndarray(size=32, shape=(32,))
[WARNING] ME(3621:140537237112640,ForkServerPoolWorker-1:9):2024-03-07-06:40:10.247.852 [mindspore/train/summary/_summary_adapter.py:304] There are no valid values in the ndarray(size=18432, shape=(64, 32, 3, 3))
[2024-03-07 06:40:35] mindocr.utils.callbacks INFO - epoch: [1/3] step: [200/1000], loss: nan, lr: 0.000066, per step time: 252.681 ms, fps per card: 63.32 img/s

Hello, issue 670 have a similar problem with this problem, you could refer to it.
We suggest you could try the following solutions:

Check the MindSpore and CANN version is matched. You could run the following code to check it.import mindspore
mindspore.set_context(device_target="Ascend")
mindspore.run_check()
exit()

Please refer to https://www.mindspore.cn/install/en#configuring-environment-variables and find more details. If some errors occur in the version check, please refer to the installation guide and reinstall CANN and MindSpore.
O2 amp_level might cause unsteady training. You could try to set amp_level to O0. For example, if you are using configs/rec/svtr/svtr_tiny.yaml, you could findsystem:
  ...
  amp_level: O2
  amp_level_infer: O2 # running inference in O2 mode
  ...

replace O2 with O0, and the yaml file would be changed like:system:
  ...
  amp_level: O0
  amp_level_infer: O0
  ...

And would you mind offering the following information, so that we could relocate the problems faster:

your GPU information
cuda/cudnn version
mindspore version
dataset you use
the yaml config, if you have modified it

In my config，I haved set the amp_level and amp_level_infer to O0。How can I solve this problem?

There is no problem training with English datasets and svtr_tiny.yaml.

It seems you train ic15 dataset on svtr_tiny_ch, but ic15 does not contain chinese characters. How about using svtr_tiny.yaml instead?

Please offer us more information, so that we could locate the problems faster:

your GPU information
cuda/cudnn version
mindspore version

CUDA Version: 11.4 and mindspore2.2.11


Update tutorial-dotnetcore-sqldb-app.md

Updated text for Step 1.2.7, as SQL database isn't necessarily the default DB engine.  (I got PostgreSQL as the default engine while running through this exercise)

@cephalin
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@cephalin - can you help us resolve this aging public-repo PR that's assigned to you? Should the change be made?
Thanks, PR review team

@cephalin - can you provide any update about this aging public-repo PR that's assigned to you? Should the change be made?
Thanks, PR review team

I sent email to the content owner today.
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


ActionView::Template::Error for hu.yml although language is :en

I have installed this gem alongside devise in my project, but I receive the following error in my production deployment (not in development on my machine):
ActionView::Template::Error (can not load translations from /.../bundle/ruby/2.2.0/gems/devise-i18n-views-0.3.4/lib/../locales/hu.yml: #<Errno::EACCES: Permission denied @ rb_sysopen - /.../bundle/ruby/2.2.0/gems/devise-i18n-views-0.3.4/lib/../locales/hu.yml>):
    1:
<%= t('.sign_in', :default => "Sign in") %>

I'm not even using the :hu language so this is a bit annoying. Is this gem loading all languages if no available locales are set? And is it possible something's wrong with the hu.yml?

After having a deeper look it turns out that this issue is related to this one: https://github.com/mcasimir/devise-i18n-views/issues/80


[ACR] Updates for new RP data models

Updated acr client to support adding/switching environment using cloud/context command modules.
Updated repository return value to support table format.
Updated SDK.
Changed storage account update as part of "az acr update" command. Removed storage subgroup.
Removed client side validation on registry name and storage account. Added a separate "az acr check-name" command to check name availability.
Updated format to match new models.
Changed "az acr list" command from ARM list to RP list.
Added regenerate admin user credental command.

Hi @DavidObando, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (daobando). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

/cc @SajayAntony @sivagms

/cc @amarzavery

/cc @tjprescott


feat(core): flatten structure of core package

affects: @fremtind/jkl-accordion, @fremtind/jkl-bullet-list, @fremtind/jkl-button,
@fremtind/jkl-card, @fremtind/jkl-checkbox, @fremtind/jkl-core, @fremtind/jkl-divider-line,
@fremtind/jkl-dropdown, @fremtind/jkl-footer, @fremtind/jkl-grid, @fremtind/jkl-header,
@fremtind/jkl-logo, @fremtind/jkl-radio-button, @fremtind/jkl-text-input
📥 Proposed changes
Flatten structure of core package as proposed in issue #139
☑ Submission checklist

[x] I have read the CONTRIBUTING doc
[x] yarn build works locally with my changes
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation (if appropriate)

💬 Further comments
Fixes #139

Also cpx-depdendency can be deleted

Also checkout the _index.scss file instead of _variables.scss and _mixin.scss

Not possible yet in node-sass as of yet (new feature in LibSass from May)


Proposal: attributes as first-class concept of Istio

I would like to propose to define attributes as a first-class concept of Istio at the same level of proxy, mixer, adapter and others, and use it to connect many sub-systems together, such as billing, logging, monitoring, analytics.
The main reason to define attributes as a first-class concept instead of part of proxy or mixer is to establish a stable foundation that we can quickly build many components on top it, including proxy, chemist, logging, monitoring, billing, rate limiting and more. So we don't let one component overrule others. Istio ecosystem must interact with many other systems, including standards like HTTP, gRPC, REST, Swagger/OpenAPI. With first-class attributes, we can develop the system as N modules instead of (N * N) mesh. Per our experience with Google API Platform, I am confident this proposal is significantly better and simpler than previous metadata designs and should meet Istio's needs comfortably.
An Istio attribute is a piece of metadata describing activities with an Istio service, such as the origin.ip (the attribute) of an API request (the activity) received by an Istio API service. Being a first-class concept, the same attribute will have the same value and semantics across all sub-systems, e.g. origin.ip in the logs will mean the same thing in the rate limiting system. If you want to allow one request per IP address per second, you can define the rate limit as "1/{origin.ip}/s".
The schema of the attribute will be simple:

name: Each attribute has a qualified name. The namespace will be organized as "organization.product.property".

For example, origin.ip.

type: Each attribute has a type, which specifies the value type. The type will be defined by an enum type.

For example, STRING.

value: Each attribute has a string value, which must be interpreted according to the type.

For example, <IP_ADDRESS>.

To define the above scheme using proto3, it would look like this:
package istio.type;
message Attribute {
enum Type {
// The value is a valid UTF-8 string.
STRING = 0;
// The value is a 64-bit integer printed as a decimal number text. It must not have leading "+".
INT = 1;
// The value is a 64-bit floating point printed as text.
FLOAT = 2;
// The value is a timestamp with RTC 3339 format.
TIMETAMP = 3;
}
string name = 1;
// Most attributes have well known type, so this field is often omitted.
Type type = 2;
string value = 3;
};
The attribute can be used in various systems. For example:

To log attributes about an API request, we can simply specify a list of attribute names in the logging config.
To monitor a system against certain attributes, we can use attributes as dimensions of metrics.
To specify rate limits, we can use attributes in the limits.

Example: "1/{origin.ip}/s" and "3/{origin.user}/min".

To control access, we can use attributes in access policy.

Example: origin.user != null.

To express simple resource ownership.

Example: origin.user == resource.owner.

To send attributes from proxy to REST servers, we can map attribute to HTTP header.

Example: origin.ip => X-Origin-IP

Thoughts?

Can you provide rationale for why value is untyped? ( a string?)
is this only definition of type? Or we will actually define standard vocabulary like '''src.ip'''

We did extensive study with labels, see https://github.com/googleapis/googleapis/blob/master/google/api/label.proto, and we use it for logging, monitoring, billing just like attributes. In practice, most of values are strings, integers can be quickly converted to/from strings, other value types are much less used.
If we use strongly typed oneof, you have to mentally remember which attribute uses which type. This is a huge burden and error prone. Try to print the attribute to console or generate a hash key or copy the value from a to b, you will find 2/3 of your code are dealing with oneof. Now if you want to write an UI editor for attributes, there is no way for the UI to know how to save input "123" into the oneof field. I spent a lot of time on this and eventually led to Google API labels dropping the type.
There is only one type definition, because many components (proxy, mixer, policy engine) need to operate on attributes, and they can only handle small set of types. We can consider attributes as primitive types for our configuration language. In any language, the number of primitive types is very limited, and generally fixed. Even so, it takes a lot of time to build a language runtime.
Regarding standard vocabulary, yes, Istio will define a standard set of attributes like "origin.ip". Almost all services will be happy with just the standard set. Google has hundreds of public API services, we have plenty experience to believe it. However, any company is free to introduce their own attributes. Istio will provide good support for them as well.

Can we close this now, with the istio/api repo and the PR for adding Attributes there?

Closing this issue, as I believe the top-level concern has been addressed.


Code examples can't scroll horizontally on mobile

None of the code example boxes in the guides can scroll horizontally when viewed via Chrome on Android. Since the interesting parts of queries are often on the right end of the string usually the interesting part is what is missing and impossible to see.
I'd rather see the whole query on multiple lines without any interaction needed but if that's not possible then allowing the query to scroll horizontally would at least make the interesting parts visible.
Thanks

Thanks for the report @setharnold. Side-scrolling on Chrome on Mac OS X works so it appears that this may be a mobile specific issue. I'm partial to scrolling since wrapping code examples will make multi-line examples confusing to read.
cc/ @alexpaxton

@setharnold what version of Android and Chrome?
Does it still happen? We just pushed an updated design with new CSS.

I'm sorry to say that the new version is even worse -- many of the code boxes are now completely blank.
For example, https://docs.influxdata.com/influxdb/v0.13/introduction/getting_started/
Of the eleven code boxes, only three actually have text rendered inside:

CREATE DATABASE mydb
SHOW DATABASES
USE mydb

The other eight are blank. If I hold my finger down inside them, I can blindly select the text, and Chrome's bottom-bar shows what text is selected, but it remains invisible when selected. (Which is a neat trick.)
I'm on Android 6.0.1, April 2, 2016 patch level. Chrome is version 50.0.2661.89. It's a Nexus 6P.
Thanks

thanks @setharnold, I can repro what you are seeing in Chrome Dev Tools.
@alexpaxton @gunnaraasen let me know if I can help, but basically just load the page in Chrome, launch developer tools, emulate the Nexus 6P, and you'll see that most of the code blocks are just blank until you interact with them. Maybe a z-axis issue? The problem is changing any of the CSS properties automatically makes all code blocks on the page appear. Most likely will need to hack local code while running the emulation in Chrome to see what finally fixes it.

I was able to reproduce using the Nexus 6P emulator, and out of curiosity tried on on an iPhone 6 and found the same issue. Very bizarre. I will look into this.

@willpiers @wfro @mark-rushakoff @121watts any ideas?
@toddboom @pauldix possible hackday project post 1.0 release.

Should this fix be live on the website?
The code box after "The following lines are all examples of points that can
be written to InfluxDB" for example is still completely blank on the page
https://docs.influxdata.com/influxdb/v0.13/introduction/getting_started/
Many others are blank, some have content.
I'm on a nexus 6p 6.0.1 July update, chrome version 51.0.2704.81
Thanks

@setharnold we do not immediately deploy the site, the fix should be live after tonight's automated push.


gulp-concat with gulp-changed

Hi,
When the destination file is passed in as a string, gulp-concat clones the attributes of its first source file. That includes stats.mtime, so piping the joined file through gulp-changed will give different results for the case where the first source had changed since the previous build and the case where any of the other sources had changed...
Would it make sense to clone the attributes of the most recently changed source file, instead of the first? Or is it some kind of convention (I'm a bit new to gulp) to use the first file in such situations?
Thanks!

Figured I'd illustrate this with an example.
gulpfile.js
var gulp = require( 'gulp' );
var changed = require( 'gulp-changed' );
var concat = require( 'gulp-concat' );

gulp.task( 'default', function() {
  return gulp.src( 'part-*.js' )
    .pipe( concat( 'list.js' ) )
    .pipe( changed( '.' ) )
    .pipe( gulp.dest( '.' ) )
  ;
} );

part-1.js
console.log( 'part-1' );

part-2.js
console.log( 'part-2' );

Running gulp produces the correct list.js:
console.log( 'part-1' );

console.log( 'part-2' );

Re-running gulp without any changes leaves list.js untouched, as expected.
Editing part-1.js and re-running gulp updates list.js, as expected:
console.log( 'part-1 changed' );

console.log( 'part-2' );

But editing part-2.js and re-running gulp does not pass through gulp-changed and so it leaves list.js untouched!

Yeah that makes a lot of sense, want to send a PR?

Thanks for the quick reply!
Well, since my earlier post, I've decided I will try my hand at first very own gulp plugin, gulp-touch or something like that, to simply set state.mtime of all sources to the maximum amongst them.
That would be a slightly  more elegant solution, because then I can swap gulp-concat and gulp-changed around in my task, meaning that the concatenation can be avoided as well, when the inputs were not changed. No offence, natch.
var touch = require( 'gulp-touch' );  // <--
gulp.task( 'default', function() {
  return gulp.src( 'part-*.js' )
    .pipe( touch() )                  // <--
    .pipe( changed( '.' ) )           // <--
    .pipe( concat( 'list.js' ) )
    .pipe( gulp.dest( '.' ) )
  ;
} );

Also, this might prove useful with other "merging plugins" that produce one output from several inputs.
But anyway, to the extent that cloning the first encountered file's attributes in gulp-concat is kind of an arbitrary choice, I still think my original suggestion is valid. I've spent a little time with the code earlier today, but it looks like it's slightly more complicated than just s/firstFile/latestFile/ and adding  the mtime test...
For one thing, I wouldn't know what to do when the dest argument is an object, maybe even an actual File with its own stats to consider (I noticed the tests pass {path:'...'} for that case?)
Anyway, as you can probably tell I'm new around here -- so I'd like to spend just a little more time playing around before I'm comfortable sending PRs... And I'd first like to do this gulp-touch experiment!

@erikkemperman I think that should be built into the plugin though, not something the user has to specify. It's good to have it as a module that other plugins can rely on though

Thanks for merging my PR! One more thing: is this something you'd consider releasing as a new version?

@erikkemperman 2.6.0


Limit updates to segment stats history so as to get a better learning curve

The segment stats history is a circular buffer of last N segments characteristics. We average values from the
previous segments to provide a value for the new one. Since all partitions within a topic tend to have
similar characteristics within the same period of time, it is best that we take only one partition from
each time period for computing statistics. Then, average over time will actually capture trends rather than
being thrown off by one set of segments during a time period. (as can happen with samza jobs, or daily
variations, for example).
We attempt to get to "one update per bunch of completing partitions" by introducing a min time interval
between consecutive additions to the stats history. This will lead us to take stats from the first partition
that completes, and then hold off taking stats from other partitions that will complete around the same
time.

Codecov Report

Merging #2876 into master will increase coverage by 0.11%.
The diff coverage is 92.3%.

@@            Coverage Diff            @@
##           master   #2876      +/-   ##
=========================================
+ Coverage   69.18%   69.3%   +0.11%
=========================================
  Files         909     909
  Lines       43502   43513      +11
  Branches     5913    5914       +1
=========================================
+ Hits        30098   30157      +59
+ Misses      11390   11335      -55
- Partials     2014    2021       +7

Impacted Files
Coverage Δ

...ata/manager/realtime/RealtimeTableDataManager.java
41.04% <100%> (+0.89%)
:arrow_up:

.../core/indexsegment/mutable/MutableSegmentImpl.java
67.15% <100%> (ø)
:arrow_up:

...ore/realtime/impl/RealtimeSegmentStatsHistory.java
81.41% <90%> (+0.45%)
:arrow_up:

...a/manager/realtime/RealtimeSegmentDataManager.java
75% <0%> (-25%)
:arrow_down:

.../impl/dictionary/LongOffHeapMutableDictionary.java
80% <0%> (-12.73%)
:arrow_down:

...kedin/pinot/core/util/SortedRangeIntersection.java
83.82% <0%> (-7.36%)
:arrow_down:

...e/impl/dictionary/LongOnHeapMutableDictionary.java
88.88% <0%> (-6.67%)
:arrow_down:

...r/filter/predicate/PredicateEvaluatorProvider.java
65.21% <0%> (-4.35%)
:arrow_down:

...nsport/netty/PooledNettyClientResourceManager.java
85.41% <0%> (-4.17%)
:arrow_down:

...om/linkedin/pinot/transport/netty/NettyServer.java
79.59% <0%> (-4.09%)
:arrow_down:

... and 25 more

Continue to review full report at Codecov.

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


SW-5814: An org with no projects that are not currently associated with an application autoselects "Select Existing Project" option

This PR includes changes to resolve an issue where the "Select Existing Project" option is auto-selected when there are no available projects without an application.

Thanks for the assist @nickgraz 🙌


fixes error where get_history returns start_time date for each row in…

…stead of correct date.

Nice catch, thank you!


[WIP] Implement code snippet completer

References
Starts working on implementing code snippet completion.
This pull request is related to the issue #208
Code changes

[x] Create a package called code-snippet-completion
[x] Add @krassowski/completion-manager to dev-dependencies in package.json
[x] Add ICompletionProviderManager as optional token in activate function of your extension
[ ] Need to fix the error on Plugin '@krassowski/jupyterlab-lsp:completion' failed to activate and No provider for: @krassowski/completion-manager:ICompletionProviderManager. - could be related to the issue of setting up the environment.
[ ] Need to create a codeSnippetCompletionProvider class that implements ICompletionProviderManager
[ ] Need to register the class

User-facing changes

User will be able to see the autocomplete options for code snippets they have saved.

Backwards-incompatible changes

Chores

[ ] linted
[ ] tested
[ ] documented
[ ] changelog entry

But, now I think that it would be actually better to work in the snippets extension directly because it would be easier to fetch the saved snippets and so on.

Exactly, it will be just easier for you. Plus I once the ICompletionProvider lands in JupyterLab core (again, probably for 4.0) your users will be able to have completion with snippets without having to have this LSP extension :)

But of course, if it is convenient for you to start working/experimenting here on this PR - absolutely feel free to do so :)

Hi, I am going to close this one as #600 on which one it is based was superseded by the work upstream. Please see https://github.com/jupyter-lsp/jupyterlab-lsp/pull/600#issuecomment-1060030197 for references. Hopefully we can revisit it after JupyterLab 4.0 release :)


build failed on jitpack always

https://jitpack.io/com/github/LiquidPlayer/LiquidCore/0.7.8/build.log

Jitpack is deprecated since 0.7.0


The error message for too short/long password for VM is spelled incorrectly (pssword)

We even have a test that verifies that it is incorrectly spelled :)

The test was to verify that error message occurred. @troydai, do we have lint rules to catch English typos? I need it!


Add initial React app stub

Initially created by:
npx create-react-app web --template typescript

...and then adjusted.
It doesn't do anything yet besides rendering an empty "Perses" app bar with Material UI.
To start (on http://localhost:3000/):
npm i
npm run start

Signed-off-by: Julius Volz<EMAIL_ADDRESS>

Codecov Report

Merging #62 (8516768) into master (4bc07a2) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #62   +/-   ##
=======================================
  Coverage   48.24%   48.24%
=======================================
  Files          54       54
  Lines        2390     2390
=======================================
  Hits         1153     1153
  Misses       1112     1112
  Partials      125      125

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 4bc07a2...8516768. Read the comment docs.

Would be cool to have the CI to build the react-app. But if you want I can do it in another PR if you don't want to be bothered by that :).

Later :)

Wondering how you think to serve the webapp through echo (the http framework), since it's quite outside of the internal package. (It's ok to think about it later)

I haven't looked into that yet, I was planning on using https://github.com/rakyll/statik to bake the React assets into the Go code.

Also any reason to not put it in the internal package ?

IMO internal should only be for Go packages that are intentionally hidden for outside consumers, I prefer keeping the web frontend code outside of that and ideally easily accessible from the repo root.

@Nexucis An unrelated Go test is failing:
=== RUN   TestUpdateDatasource
    datasource_test.go:122:
        	Error Trace:	datasource_test.go:122
        	Error:      	Received unexpected error:
        	            	invalid character '}' after top-level value
        	Test:       	TestUpdateDatasource
--- FAIL: TestUpdateDatasource (0.00s)

I haven't looked into that yet, I was planning on using https://github.com/rakyll/statik to bake the React assets into the Go code.

ah yeah interesting. For Angular I use a code generation, but certainly better to not use an artisanal way.

IMO internal should only be for Go packages that are intentionally hidden for outside consumers, I prefer keeping the web frontend code outside of that and ideally easily accessible from the repo root.

Like I said in a previous conversation, I have been told it was painful to have the UI so deep in the arbo. And on my side I wasn't so confortable to have it in the internal package etheir. So yeah totally fine with that. I was just curious :).

@Nexucis An unrelated Go test is failing

yeah that's odd. I just rerun the pipeline and it works. Will have to find what happens one day.

Thanks for merging!

Like I said in a previous conversation, I have been told it was painful to have the UI so deep in the arbo. And on my side I wasn't so confortable to have it in the internal package etheir. So yeah totally fine with that. I was just curious :).

Yes, and I would also not see this primarily as a Go repo, but primarily as a web / React repo, with some Go pieces to support the web application :)

Yes, and I would also not see this primarily as a Go repo, but primarily as a web / React repo, with some Go pieces to support the web application :)

Haha I will have a different opinion here. But I think for the moment it doesn't really matter :)


MRG: Fix raw PSD

Does two things:

Adds SEEG and ECoG to raw.plot_psd, which formely just did mag/grad/eeg.

Allows for a list of axes to be passed when picks=None.

Ready for review/merge from my end.

LGTM

Coverage increased (+0.004%) to 90.811% when pulling 788267dba8e9d14364e2eb5a81434dea003a115e on Eric89GXL:fix-psd into 199c8e93810dd54e1d65c35a35ef99776b0f5055 on mne-tools:master.

thx @Eric89GXL


python311Packages.slowapi: 0.1.8 -> 0.1.9

Diff: https://github.com/laurentS/slowapi/compare/refs/tags/v0.1.8...v0.1.9
Changelog: https://github.com/laurentS/slowapi/blob/v0.1.9/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

@ofborg eval

Fixes for failures:

#288244
#282824

Result of nixpkgs-review pr 287965 run on x86_64-linux 1

14 packages failed to build:

apache-airflow
apache-airflow.dist
irrd
irrd.dist
libretranslate (python311Packages.libretranslate)
libretranslate.dist (python311Packages.libretranslate.dist)
python311Packages.flask-appbuilder
python311Packages.flask-appbuilder.dist
python312Packages.flask-limiter
python312Packages.flask-limiter.dist
python312Packages.limits
python312Packages.limits.dist
python312Packages.slowapi
python312Packages.slowapi.dist

18 packages built:

calibre-web
calibre-web.dist
fit-trackee
fit-trackee.dist
octoprint
octoprint.dist
python311Packages.aetcd
python311Packages.aetcd.dist
python311Packages.flask-limiter
python311Packages.flask-limiter.dist
python311Packages.limits
python311Packages.limits.dist
python311Packages.slowapi
python311Packages.slowapi.dist
python312Packages.aetcd
python312Packages.aetcd.dist
theharvester
theharvester.dist

Result of nixpkgs-review pr 287965 run on x86_64-linux 1

10 packages marked as broken and skipped:

apache-airflow
apache-airflow.dist
python311Packages.flask-appbuilder
python311Packages.flask-appbuilder.dist
python312Packages.flask-limiter
python312Packages.flask-limiter.dist
python312Packages.limits
python312Packages.limits.dist
python312Packages.slowapi
python312Packages.slowapi.dist

4 packages failed to build:

irrd
irrd.dist
libretranslate (python311Packages.libretranslate)
libretranslate.dist (python311Packages.libretranslate.dist)

18 packages built:

calibre-web
calibre-web.dist
fit-trackee
fit-trackee.dist
octoprint
octoprint.dist
python311Packages.aetcd
python311Packages.aetcd.dist
python311Packages.flask-limiter
python311Packages.flask-limiter.dist
python311Packages.limits
python311Packages.limits.dist
python311Packages.slowapi
python311Packages.slowapi.dist
python312Packages.aetcd
python312Packages.aetcd.dist
theharvester
theharvester.dist


Image component doesn't rerender

<Image src={src} />
If I change the variable src the image doesn't update. Is this a bug?

I have a similar problem and I'm wondering if I'm missing something.  When I change the src on an image component via a state change, it doesn't rerender on the canvas.  Isn't this the expected behavior or no?

@benjamingeorge Yes, if you change the state, all components should update. That's the common behavior of React components.
You can try to use my fix but I just noticed that it's not working perfectly if the image is not loaded already.

Yes your fix works for me but only if the images are preloaded by placing them in the DOM in img tags.

Is it possible to fix this bug somehow? Such a showstopper for me.
Thanks

+1, you can workaround by using src (or a hash of src) as key


Refactoring of truncate helper method

Description
I discovered couple of issues with models.PyCryptoBot.truncate() method:

the precision was not always honored, e.g. (1.123, 4) emitted "1.123" which, when printed by logging, broke the alignment of
"columns" in the output:

2021-05-24 11:10:00 | ADAUSDT | 5m | Close: 1.4568 | EMA12/26: 1.4568 = 1.4568 | MACD: 0.0000 = 0.0000OBV: 3534629.9200 (0.0000%)ERI: | WAIT
2021-05-24 11:15:00 | ADAUSDT | 5m | Close: 1.458 | *^ EMA12/26: 1.4569 > 1.4568 ^* | *^ MACD: 0.0001 > 0.0000 ^* | ^ OBV: 6328470.5900 (79.0400%) ^ | ERI: buy | WAIT
2021-05-24 11:20:00 | ADAUSDT | 5m | Close: 1.4625 | ^ EMA12/26: 1.4578 > 1.4573 ^ | ^ MACD: 0.0005 > 0.0001 ^ | ^ OBV: 9257233.7600 (46.2800%) ^ | ERI: sell | WAIT

(Close: 1.458 should really be printed as Close: 1.4580)

inconsistent return value: a string, but, with carefuly picked inputs, a float. AFAICT, every time truncate() is called, its return value is either converted to string (str(app.truncate(...))), or to float.
instance method yet it uses no instance attributes, it's pretty much a pure function since its output depends only on its inputs \o/

So I took a shot and refactored it to become a standalone function, fixed the return value to be a string, always - these two steps reduced quite common str(app.truncate(...)) to just truncate(...). In the process, I fixed the issue with precision not being honored, and I added a unit test to protect it against future regressions.
Fixes # (issue)
Type of change
Please make your selection.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] This change requires a new dependency
[x] Code Maintainability / comments
[x] Code Refactoring / future-proofing

How Has This Been Tested?
Simulation and actual runs, new unit test. Focuses on the logging, shouldn't hurt trading even if there's a mistake somewhere.
Checklist:

[ ] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added pytest unit tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings

The unit tests are blocked by https://github.com/whittlem/pycryptobot/pull/235

@happz, the new unit test is failing. Can you check it please?
pytest: error: unrecognized arguments: tests/unit_tests
=========================== short test summary info ============================
ERROR tests/unit_tests/test_formatting.py - SystemExit: 2
!!!!!!!!!!!!!!!!!!!! Interrupted: 1 error during collection !!!!!!!!!!!!!!!!!!!!
=============================== 1 error in 1.25s ===============================
Thanks

@happz, the new unit test is failing. Can you check it please?
pytest: error: unrecognized arguments: tests/unit_tests
=========================== short test summary info ============================
ERROR tests/unit_tests/test_formatting.py - SystemExit: 2
!!!!!!!!!!!!!!!!!!!! Interrupted: 1 error during collection !!!!!!!!!!!!!!!!!!!!
=============================== 1 error in 1.25s ===============================
Thanks

Yes, I'll rebase the patch - the failure is expected, it was caused by an issue fixed in #235.

Great, thanks.

This fix leads to error !
The truncate() method returns a string so in line 583 of PyCryptoBot, marketBuy is called with a string and raise an error since quote_quantity should be a int or a float.
EUR balance before order: 2251.88
Telegram send:https://api.telegram.org/bot1858946724:AAGevzd487VThh3_VeRB563JX3rc4M5L5xA/sendMessage?chat_id=572632714&parse_mode=Markdown&text=Bot for LTC-EUR got an exception: TypeError('The funding amount is not numeric.')
TypeError('The funding amount is not numeric.')
Traceback (most recent call last):
  File "/app/pycryptobot.py", line 1038, in <module>
    main()
  File "/app/pycryptobot.py", line 1003, in main
    runApp()
  File "/app/pycryptobot.py", line 998, in runApp
    executeJob(s, app, state)
  File "/app/pycryptobot.py", line 741, in executeJob
    resp = app.marketBuy(app.getMarket(), state.last_buy_size, app.getBuyPercent())
  File "/app/models/PyCryptoBot.py", line 583, in marketBuy
    return api.marketBuy(market, truncate(quote_currency, 2))
  File "/app/models/exchange/coinbase_pro/api.py", line 275, in marketBuy
    raise TypeError('The funding amount is not numeric.')
TypeError: The funding amount is not numeric.```

This fix leads to an error !
The truncate() method returns a string so in line 583 of PyCryptoBot, marketBuy is called with a string and raise an error since quote_quantity should be a int or a float.
EUR balance before order: 2251.88
Telegram send:https://api.telegram.org/bot1858946724:AAGevzd487VThh3_VeRB563JX3rc4M5L5xA/sendMessage?chat_id=572632714&parse_mode=Markdown&text=Bot for LTC-EUR got an exception: TypeError('The funding amount is not numeric.')
TypeError('The funding amount is not numeric.')
Traceback (most recent call last):
  File "/app/pycryptobot.py", line 1038, in <module>
    main()
  File "/app/pycryptobot.py", line 1003, in main
    runApp()
  File "/app/pycryptobot.py", line 998, in runApp
    executeJob(s, app, state)
  File "/app/pycryptobot.py", line 741, in executeJob
    resp = app.marketBuy(app.getMarket(), state.last_buy_size, app.getBuyPercent())
  File "/app/models/PyCryptoBot.py", line 583, in marketBuy
    return api.marketBuy(market, truncate(quote_currency, 2))
  File "/app/models/exchange/coinbase_pro/api.py", line 275, in marketBuy
    raise TypeError('The funding amount is not numeric.')
TypeError: The funding amount is not numeric.```

Is truncate() call needed here then? Since the api.marketBuy expects a number, would it be possible to drop truncate and proceed with api.marketBuy(market, quote_currency)?

That's what I did in my version of the bot and it's okay.

I submitted #249 as a quick fix, to restore the original behavior. Dropping truncate() can be done later if deemed reasonable (seems to me, but I don't know the API well enough).


How to specify number of threads?

Hi I'm just wondering is it possible to specify the number of threads? Many thanks!

Hey Linda. Thanks for the comment. I've added a command line flag for the fold_tree workflow for this. The readme is updated with an example of how to use this flag. Hope this helps

Hi, sorry only seeing this now! Brilliant thank you very much! :)


Add package to conda-forge

We should use conda-forge to create a feedstock and add this package to be installable from conda

It's now on conda so I think we can close this: https://anaconda.org/conda-forge/alteryx-open-src-update-checker


ci: disable swagger check temporarily

Finschia/finschia-sdk#471
Don't need to apply it to upstream


Marker (circle) is not removed when removing breakpoint

VSCode Version: Code 1.18.0 (dcee2202709a4f223185514b9275aa4229841aa7, 2017-11-08T21:22:49.932Z)
OS Version: Windows_NT x64 10.0.15063
Extensions:

Extension
Author (truncated)
Version

vscode-eslint
dba
1.4.3

react-native-react-redux
EQu
1.4.1

auto-close-tag
for
0.5.3

auto-rename-tag
for
0.0.15

vscode-redux-devtools
jin
0.0.11

resharper9-keybindings
ms-
1.1.0

debugger-for-chrome
msj
3.5.0

vscode-icons
rob
7.18.1

Steps to Reproduce:

When I try to remove an existing breakpoint by clicking on the red circle in my code the red marker is not removed from the code line.
However the breakpoint is actually removed in the list of breakpoints. So it seems like this is only a graphics update error and that breakpoints work.

Reproduces without extensions: Yes/No

Just to clarify: I had no issues until I installed the latest update. Not certain if I have skipped some previous updates, but I use VS Code dialy and update often.

Duplicate of #37880. Please try Insiders. Fix will appears in Stable soon.


Love the Icons! Can you add a license?

Would love to use these but we need a license - preferably MIT of course :)

Much appreciated sir.


Bookmark Support in release page

Bookmark Support in release page

Fixes https://github.com/adoptium/adoptium.net/issues/666
cc @hendrikebbers @gdams
Checklist

[ ] npm test passes
[ ] documentation is changed or added (if applicable)
[ ] permission has been obtained to add new logo (if applicable)
[ ] contribution guidelines followed here

@Ndacyayisenga-droid what is the state on that PR?


CFE fails to resolve extension imported via prefix when multiple import statements use the same prefix

Consider the following two Dart libraries:
test.dart:
import 'dart:async' as other; // (1)
import 'other.dart' as other;

f(other.C c) {
  c.f();
}

main() {}

other.dart:
class C {}

extension E on C {
  f() {}
}

I believe this code is statically valid, and the analyzer accepts it without complaint.  But the CFE reports the following error:
../../tmp/proj/test.dart:5:5: Error: The method 'f' isn't defined for the class 'C'.
 - 'C' is from '../../tmp/proj/other.dart'.
Try correcting the name to the name of an existing method, or defining a method named 'f'.
  c.f();
    ^

Removing the line marked (1) makes the error go away.

I also ran into this issue. When I moved the import for the file containing the extension above the other import, it worked.


New plugin creater

Supporting the 5.2 and legacy plugin system for sw plugin:create.
Working on top of @mohrekopp pull request: #70.
Added:

Move Views/ to Resources/views
Move Snippets/ to Resources/snippets
Resources/menu.xml
Resources/config.xml
Service registrations in Resources/services.xml
Create widget
Separate directories for legacy and 5.2 plugin system
Little unit test fixes

Thanks @mohrekopp for your Pull-Request #70, these big changes must be done due to missing documentation at the moment.

Shouldn't Commands and Subscribers be written in the services.xml with their according tags instead of adding methods in the plugin class for them?

@FiveDigital commands and subscribers are registered in the services.xml if I see this correctly. the onStartDispatch event in the plugin.php is only used for registering the API components

The according part was removed exactly one minute after my comment with https://github.com/shopwareLabs/sw-cli-tools/pull/71/files/1f5edd94f005e0ff43386f9bd8ea204fe851216b..9b4004c391cafc68622126a0ccb114d7dc4da4e0 👍

I improved the pull-request a little bit more by adding the Subscriber/ApiSubscriber.php and better support for the command arguments.

new Plugins created in root directory

install and uninstall function in plugin class are empty, unnecessary boiler plate code (php storm override function generates this code within 2sec )

controller path service definition exists even if when the controllerPath subscriber class not exist.

--have-filter
4.0 please consider SearchBundleES too
4.1 Add a decorator for ESIndexingBundle/Product/*
4.2 Add a sorting too please, with template extension
4.3 move CriteriaRequestHandler into SearchBundle namespace

--have-widget
5.0 Mixed with backend module implementation
5.2 ControllerPath Subscriber contains template extends command for backend widgets => extract
5.3 Backend/Application controller contains php code for backend widget
5.4 views/backend/application/widgets contains extjs widgets
==> please split in seperate namespace

$this->container->get('application')->Loader() => $this->container->get('Loader');

@OliverSkroblin The 1. and 2. point of your list should be done in additional PRs, because this PR just handles  the compatibility to the 5.2 plugin system.
In this case the target installation directory and SearchBundleES integration would be new functionality.
All other points on your list were fixed.

Added:

Root folder detection, created plugins will be created in the plugin path
legacy plugins in `engine/Shopware/Plugins/Local/your_namespace'
5.2 plugins in custom/plugins
if the command will not be executed from shopware root it will be created in the same directory
Added elastic search integration
Added product search decorator
Added sorting

This PR is ready to merge.
Thank you for your contribution, @mohrekopp ;)


Connection manager transition order test using IOSimPOR

This PR adds a quickcheck property test which validates order of connection manager transitions using IOSimPOR.  It also fixes a connection manager bug discovered thanks to the new test.
Fixes #3613.

connection-manager-tests: test transition order
connection-manager: don't blindly override state
connection-manager-tests: code style

bors merge


Weather report and erlang TLS distribution problem

Description
Versions

COUCHDB
"couchdb":"Welcome","version":"3.3.1"
erl
Erlang/OTP 25 [erts-13.2.2] [source] [64-bit] [smp:4:4] [ds:4:4:10] [async-threads:1] [jit:ns]
Rocky 8.7

2 node cluster configured and looks like starts well, log output ->
couch_replicator_clustering : cluster stable -> couch_replicator_clustering : cluster stable
When I run /weatherreport --etc /opt/couchdb/etc/ without erlang TLS distribution configured, it works ->
xxxx
_Configuration Settings:
[admins] rocky="-pbkdf2-092b2a2214c80a33e720c76cf1b778b23cd95d9d,0d8c8fcd652bc1acc14f291eaead743f,10"
[admins] vagrant="-pbkdf2-6160777f3986cf1c320b01c46e631ecbea8f2910,5a95dcffdfcdc0a9cacfdb7d13a9f392,10"
[chttpd] bind_address="<IP_ADDRESS>"
[chttpd] port="5984"
[chttpd_auth] hash_algorithms="sha256, sha"
[chttpd_auth] secret="ed35b213650525d1987c810617000a23"
[couch_httpd_auth] authentication_db="users"
xxxx
When I run /weatherreport --etc /opt/couchdb/etc/ with erlang TLS distribution configured, it does not work ->
xxxx
<EMAIL_ADDRESS>[warning] Could not connect to the local cluster node<EMAIL_ADDRESS>some checks will not run.
<EMAIL_ADDRESS>[crit] Bad rpc call executing check weatherreport_check_tcp_queues: nodedown
<EMAIL_ADDRESS>[crit] Bad rpc call executing check weatherreport_check_search: nodedown
<EMAIL_ADDRESS>[crit] Bad rpc call executing check weatherreport_check_safe_to_rebuild: nodedown
<EMAIL_ADDRESS>[crit] Bad rpc call executing check weatherreport_check_process_memory: nodedown
xxxxx
Is it something wrong with the configuration of just weatherport is used somehow wrong?
A test table with new documents  gets replicated either when erlang TLS distribution configured or not.
If I try to connect to the port 9100 SSL handshake starts ->
s_client -connect couchdb1.dp.home:9100
Steps to Reproduce

install elrang (this step is needed as otherwise 22 version will be installed and weathereport will not run)
curl -s https://packagecloud.io/install/repositories/rabbitmq/erlang/script.rpm.sh | sudo bash
sudo yum install erlang -y

Install couchdb as per https://docs.couchdb.org/en/stable/install/unix.html#enabling-the-apache-couchdb-package-repository
sudo yum install couchdb-3.3.1-1.el8.x86_64 -y

Prepare the nodes for the cluster -> https://docs.couchdb.org/en/stable/setup/cluster.html

Create CA, server1cert.pem / server1key.pem && server2cert.pem / server2key.pem

Enable erlang TLS distribution ->https://docs.couchdb.org/en/stable/cluster/tls_erlang_distribution.html

The final changes will be

/opt/couchdb/etc/vm.args
xxxxx
-name<EMAIL_ADDRESS>#-name<EMAIL_ADDRESS>-setcookie SbFch4ESZSNBAF5ivZn34oKEDAT8H684V4TATER0dCxhGZwjk
-kernel inet_dist_use_interface {0,0,0,0}
-kernel inet_dist_listen_min 9100
-kernel inet_dist_listen_max 9200
-proto_dist inet_tls
-ssl_dist_optfile /etc/couchdb/newcert/couch_ssl_dist.conf
xxxxx
/etc/couchdb/newcert/couch_ssl_dist.conf (node1)
xxxxx
[{server,
[{certfile, "/etc/couchdb/newcert/couchdb1withprivate.pem"},
{secure_renegotiate, true}]},
{client,
[{cacertfile, "/etc/couchdb/newcert/CAcert.pem"},
{secure_renegotiate, true}]}].
xxxxx
/etc/couchdb/newcert/couch_ssl_dist.conf (node2)
xxxxx
[{server,
[{certfile, "/etc/couchdb/newcert/couchdb2withprivate.pem"},
{secure_renegotiate, true}]},
{client,
[{cacertfile, "/etc/couchdb/newcert/CAcert.pem"},
{secure_renegotiate, true}]}].
xxxxx
Expected Behavior
I would like to get the same output as when erlang TLS distribution is not configured.
Your Environment
https://couchdb1.dp.home:6984
{"couchdb":"Welcome","version":"3.3.1","git_sha":"1fd50b82a","uuid":"ed35b213650525d1987c8106170001fd","features":["access-ready","partitioned","pluggable-storage-engines","reshard","scheduler"],"vendor":{"name":"The Apache Software Foundation"}}

@pavlov2000uk good find, I don't think weatherreport was fixed to work with the TLS distribution yet
(Also hopefully those are not production credentials the password hashes, secret and coookie values, if so please consider editing them out and rotate your credentials).

Thank you. Test values everywhere .... all form my lab.

If the team is already aware of the issue, by any chance maybe some ETA can be shared?


[PERFORMANCE] Config file writing spam

This plugin is currently writing to the config file (CollapsibleUI.config.json) around 10 times per second.
The config file should only really be written to when the configuration is changed.
It should be quite clear as to why writing the file 10 times per second is bad. Not only for performance but also SSD health.

Fixed in v5


ADD ability to disable max-height restriction (to use browsers page scrollbar to scroll through content)

I would like to use the main document body to scroll through the dataset that Clusterize.js optimizes, not have a limited height area where the rows are shown
I tried:
.clusterize-scroll {
	overflow: visible;
}

and
.clusterize-scroll {
	max-height: none;
	max-width: none;
}

..but Clusterize only shows the first 200 rows then :(
I also tried using 99999px instead of none, but it had the same effect

This PR might be interesting for you.