deepl: Add version 2.5.0

What type of Apps related PR is this? (check all applicable)

[x] ✨ Add Manifest
[ ] ♻ Refactor
[x] 🐛 Bug Fix
[ ] 👷 Optimization
[ ] 🚩 Other

Description
DeepL is a tools that can help translate text in differente language
The x86 version seems to be limited to 2.4.0, so I enforced it. It will require a modification if/when the issue at DeepL is resolved
Related Tickets & Documents
Not related
Did you verify the followings
Add/Modify a description in the manifest?

[x] 👍 yes
[ ] 🙋 yes, but I need help
[ ] 🙅 no, because they aren't needed

Check URLs?

[x] 👍 yes
[ ] 🙋 yes, but I need help
[ ] 🙅 no, because they aren't needed

Update to the latest version?

[ ] 👍 yes
[x] 🙋 yes, but I need help I had to force the x86 version to the prior version (2.4.0)
[ ] 🙅 no, because they aren't needed

Update Hashes?

[x] 👍 yes
[ ] 🙋 yes, but I need help
[ ] 🙅 no, because they aren't needed

Format the json manifest?

[x] 👍 yes
[ ] 🙋 yes, but I need help
[ ] 🙅 no, because they aren't needed

Did you identify extra steps
Notes in the manifest?

[ ] 👍 yes
[ ] 🙋 yes, but I need help
[x] 🙅 no, because they aren't needed

Application Dependencies?

[ ] 👍 yes
[ ] 🙋 yes, but I need help
[x] 🙅 no, because they aren't needed

Application Suggestions?

[ ] 👍 yes
[ ] 🙋 yes, but I need help
[x] 🙅 no, because they aren't needed

Pre-installation steps to perform?

[x] 👍 yes
[ ] 🙋 yes, but I need help
[ ] 🙅 no, because they aren't needed

Post-installation steps to perform?

[ ] 👍 yes
[ ] 🙋 yes, but I need help
[x] 🙅 no, because they aren't needed

File persistence to perform?

[ ] 👍 yes
[ ] 🙋 yes, but I need help
[x] 🙅 no, because they aren't needed

[optional] What gif/emoji best describes this PR or how it makes you feel?

:flushed:

Your changes do not pass checks.
deepl

[x] Description
[x] License
[x] Hashes
[ ] Checkver
[x] Autoupdate
[x] Autoupdate Hash Extraction

/verify

All changes look good.
Wait for review from human collaborators.
deepl

[x] Description
[x] License
[x] Hashes
[x] Checkver
[x] Autoupdate
[x] Autoupdate Hash Extraction


Add support for socket activation

[x] endlessh: Add a new option (-i) to expect a socket on stdin.
[x] systemd: Add two templated unit files (endlessh@.{service,socket})
to support socket activation on arbitrary ports.
[x] endlessh: Error-out when -i is specified on the command-line and a port
was explicitely configured (either by config file or CLI argument).
[ ] systemd: Rework the default service (endlessh.service) to use socket
activation, or remove it outright.

Rationale

Supports listening on port 22 without issue. (see #39)
Lowers overhead, as the service isn't started until the first connection.
Improves security, as the socket-activated service:

runs under a transient user (not root!), even with ports <1024;
does not have any access to the network, aside from the socket it was
started with.

@skeeto I just updated the feature branch to solve the merge conflict, and while I was at it added some improvements to the systemd unit that I was experimenting with locally.
Were you able to confirm this works as desired?

Ping?

This worked well for me and it obviated the need for #21 .  ListenStream supports IP's, IP/port, sockets, etc. so it's very flexible.  I made an array of the listens I wanted in a puppet class and a loop to deploy them.  Slick!
https://www.freedesktop.org/software/systemd/man/systemd.socket.html


Week 6 update: Home Page completed

Project
Unnati
Changes
Home page completion

@Apoorvgarg-creator, please review.


Feature/coverage

Added more coverage to arcgis client.
Added some more control in arcgis client.
Expanded situations in which the tests are executed.

Codecov Report

Merging #22 (8f12a21) into development (2630d9f) will increase coverage by 8.45%.
The diff coverage is 100.00%.

@@               Coverage Diff               @@
##           development      #22      +/-   ##
===============================================
+ Coverage        50.72%   59.18%   +8.45%
===============================================
  Files               27       26       -1
  Lines              826      833       +7
===============================================
+ Hits               419      493      +74
+ Misses             407      340      -67

Flag
Coverage Δ

unit-tests
59.18% <100.00%> (+8.45%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

mango/clients/arcgis.py
100.00% <100.00%> (+100.00%)
:arrow_up:

mango/shared/__init__.py
100.00% <100.00%> (ø)

mango/shared/exceptions.py
100.00% <100.00%> (ø)

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


Refactor the CatalogPanel

To enable to merge the Weather Forecast models selectors and the Weather layers

The basic idea is to assigned a specific selector to a category.


Test connection with new user

What
At the moment, the user and password created, are not tested when
establishing the connection. To prevent issues down the line, we'd like
to cover that case with some tests.
It has pointed out the issues in our code, which we didn't use back
ticks to identify the database in a query.
How to review

Code review
Run MySQL (and postgres optionally) services locally - or rely on travis being happy
Execute make unit and see mysql test pass

Who can review
Not @LeePorte nor @paroxp

The problem actually has been related to the line 88 of that file, where mysql expected some back ticks around the dbname.
It has been briefly mentioned in the commit message:

It has pointed out the issues in our code, where we didn't use back
ticks to identify the database in a query related to GRANT.

Should I add some more details? :)

Should I add some more details? :)

Yes please. Why the backticks are necessary.

Commit message amended.


feat(Styles|Documentation): Design Tokens/Utilities typescript parity and usage docs

Overview
Better emotion and TS support for design tokens and utilities.  Initial documentation pass.
PR Checklist

[ ] Related to Abstract designs:
[ ] Related to JIRA ticket: ABC-123
[x] I have run this code to verify it works
[ ] This PR includes unit tests for the code change

Description

Changed gamut-styles ts config to be aware of dev dependencies.
Added a few exclusions to the .npmignore.
Split typography to its own top-level section and added token documentation to Foundations.
Added documentation for all existing and converted utilities.
Added a few storybook native blocks to help with examples.

📬Published Alpha Packages:@codecademy/gamut-labs@1.9.4-alpha.681e5c.0@codecademy/gamut-styles@4.2.6-alpha.681e5c.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-labs@1.9.4-alpha.67ccc7.0@codecademy/gamut-styles@4.2.6-alpha.67ccc7.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5f35891ebd7d3451207c119f--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://5f358a9cc00e0913b18aa8bc--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@1.9.4-alpha.e2a639.0@codecademy/gamut-styles@4.2.6-alpha.e2a639.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5f35a11f4963bf6ad7f89a86--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@1.9.4-alpha.2408ac.0@codecademy/gamut-styles@4.2.6-alpha.2408ac.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-labs@1.9.4-alpha.caa464.0@codecademy/gamut-styles@4.2.6-alpha.caa464.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-labs@1.9.4-alpha.9827fc.0@codecademy/gamut-styles@4.2.6-alpha.9827fc.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5f36db8e186dfa74f49eda82--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://5f3bd926f3701e6d98c1e109--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@1.9.4-alpha.5e3910.0@codecademy/gamut-styles@4.2.6-alpha.5e3910.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-labs@1.9.4-alpha.9a2807.0@codecademy/gamut-styles@4.2.6-alpha.9a2807.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5f3bf19c0c21b4487435c7b0--gamut-preview.netlify.appDeploy Logs


Dynamically register SemanticTokenProviders in VSCode

Currently we manually handle SemanticToken requests, and more specifically make a call to a special endpoint on the server to retrieve the legend. This is off-spec and bad practice, so we should switch to using the standard systems. Of note: we'll have to deal with this issue (https://github.com/microsoft/language-server-protocol/issues/1107, https://github.com/OmniSharp/csharp-language-server-protocol/pull/401)

This should be fixed in v1.25.4 (hurrah)!


Get Need admin approval dialog when 'Call API' in chapter 4 scenario 1 example

Issue
I setup 2 apps in Azure to run this chapter 4, 1 scenario.  I am able to login to my spa.
But when I click the  'Call API' button, I get this dialog saying
Need admin approval
needs permission to access resources in your organization that only an admin can grant. Please ask an admin to grant permission to this app before you can use it.

Please provide us with the following information:

This issue is for the sample

    - [ ] 1-1) Sign-in with Azure AD
    - [ ] 1-2) Sign-in with Azure AD B2C
    - [ ] 2-1) Acquire a Token and call Microsoft Graph
    - [ ] 3-1) Protect and call a web API on Azure AD
    - [ ] 3-2) Protect and call a web API on Azure AD B2C
    - [ x] 4-1) Call a web API that calls Microsoft Graph on-behalf-of user
    - [ ] 4-2) Call a web API that calls another web API on-behalf-of user
    - [ ]   5) Deploy to Azure Storage and App Service

This issue is for a

    - [ x] bug report -> please search issues before submitting
    - [ ] question
    - [ ] feature request
    - [ ] documentation issue or request

Minimal steps to reproduce

Login via 'localhost:3000' and click 'Call API'.

Any log messages given by the failure

Network trace:
{"error":"invalid_grant","error_description":"AADSTS65001: The user or administrator has not consented to use the application with ID '592f65fc-d5cf-4269-8394-95e8fd467d31' named 'ms-identity-javascript-tutorial-c4s1-spa'. Send an interactive authorization request for this user and resource.\r\nTrace ID: e27b4539-c15c-4deb-b0e3-5e95fa390400\r\nCorrelation ID: 9304a21f-9409-4dee-9838-004d4283c21c\r\nTimestamp: 2022-01-29 18:41:40Z","error_codes":[65001],"timestamp":"2022-01-29 18:41:40Z","trace_id":"e27b4539-c15c-4deb-b0e3-5e95fa390400","correlation_id":"9304a21f-9409-4dee-9838-004d4283c21c","suberror":"consent_required"}
Expected/desired behavior

Library version

Browser and version

Chrome, Edge, Firefox, Safari?

Mention any other details that might be useful

Thanks! We'll be in touch soon.

I am able to resolve this issue by going to Azure Portal and Under 'Authorized client applications
' and 'Expose an API' and add a client id of the SPA. to it.


[google-cloud-cpp] update to latest release (v1.25.0)

Describe the pull request
Updates google-cloud-cpp to the latest release (v1.25.0)

What does your PR fix? Fixes #

N/A

Which triplets are supported/not supported? Have you updated the CI baseline?

No change.

Does your PR follow the maintainer guide?

Yes.

Thanks!


kindred 1st mark

kindred mark on scuttle doesnt show

When playing kindred you should use the default skin when pick a champion and then go to the game to choose a skin formm R3nZ, the kindred mark will appear in the minimap
Sorry, im bad ENG, hope u know !!!!!!!!!

#1


Update build-tools version to 30.0.2

Fixes #

@... PTAL?

Codecov Report

Merging #1179 (69ea1d3) into master (614ac2e) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1179   +/-   ##
=========================================
  Coverage     28.78%   28.78%
  Complexity      843      843
=========================================
  Files           304      304
  Lines          8371     8371
  Branches        369      369
=========================================
  Hits           2410     2410
  Misses         5830     5830
  Partials        131      131

Help us with your feedback. Take ten seconds to tell us how you rate us.


insertMany bug: dotted string value inserted with incorrect data type

With MongoClient 1.0.1 and MongoDB 3.2.6, inserting a field with a dotted string value can produce unexpected results. I haven't tested earlier versions of MongoDB, but this seems to be a client side bug changing some strings with dotted values to different types on insertion.
insertMany of:
{webscale":"<IP_ADDRESS>"}

... results in the webscale string created as an ObjectID (checked via mongo shell):
{
  "_id": ObjectId("572ba076f5e431b52061d68d"),
  "webscale": ObjectId("3134342e32392e31302e3934")
}

insertMany using an array of strings:
[{"ip":"<IP_ADDRESS>"}, {"ip":"<IP_ADDRESS>"}]

... results in the following documents (checked via mongo shell):
{
  "_id": ObjectId("572ba227f5e431b52061d69d"),
  "ip": ObjectId("3139322e3136382e312e3934")
}
{
  "_id": ObjectId("572ba227f5e431b52061d69e"),
  "ip": "<IP_ADDRESS>"
}

Originally reported at: http://stackoverflow.com/questions/37055525/mongodb-changes-string

Hello,
that's an expected behaviour unless you uncheck the valid ObjectID conversion checkbox which placed at upper right corner at queries.
But I didn't know valid IP addresses are also valid ObjectIDs. I'll investigate this further. Currently you can simply uncheck the mentioned checkbox before inserting a document.

Hi,
I do not have this checkbox (using windows x64 last version 1.1)
How come?
Thanks

@stennie in the link you mentioned is really interesting:

Another interesting thing is that it does not happen on all ip addresses. for example this one : {"ip":"<IP_ADDRESS>"} works

will investigate this further as I said.
I'm glad @vnicolet 's issue is fixed.

I guess there's a some sort of bug in nodejs mongodb driver, you can keep tracking this issue from here: https://jira.mongodb.org/browse/NODE-702

@rsercano Thanks for investigating! I would suggest turning off "valid ObjectID conversion" as a default (particularly given it can unexpectedly change types). Perhaps it can be a user preference to enable as default?

I will add a notification text under selectors, thanks for the suggestion :) @stennie


No text output when id is not equal

https://github.com/Nelliosis/DSA-FMP/blob/a6bf4b9c9508d7ea280cbab79b64eb5c07dd1457/data.h#L852

Resolved in v1.1


Consider adding an extra argument to @queryAssignedNodes to query in the result

Description
This argument would accept a simple css selector and filter the results by running matches on that selector. This would also limit the output to elements.
Acceptance criteria
Feature implemented and tests added.

cl/319987735


Backspace speed overriding Typing speed

Hi, @adamjking3. Thank you for this project. I'm using it on the new docs of https://github.com/diegohaz/reas
I'm trying to set different speeds for the typing and backspace, but it seems that backspace speed is overriding typist speed.
Reproduction: https://codesandbox.io/s/k25p7r924v
I was able to fix it with a workaround:
<Typing.Backspace speed={20} count={word.length - 1} />
<Typing.Backspace speed={200} count={1} />

But it would be better if it worked out of the box.

One possible fix for this, would be to only apply that speed for the next x characters, where x is specified by props.count in the Backspace component. Otherwise, adding a Speed component after the Backspace is meant for this purpose.


mkdirp-promise is deprecated and replaced with mkdirp

I have signed the CLA before with this github username.

Will do it in another pull request.


Feature addvalidation

Overview/Summary
Adds validation by adding a validate job for each deploy job. Also adds dependencies for validate to run successfully before deploying.
This PR fixes/adds/changes/removes

Adds validation by passing whatif to the deploy jobs
Adds dependencies by using needs in the GitHub workflow
Updated the wiki to add the environment production

Breaking Changes

This code requires the environment production to be added to the GitHub repo.
This has been added to the wiki

Testing Evidence
Limited testing as in the test environment VWAN is not deployed:

As part of this Pull Request I have

[x] Read the Contribution Guide and ensured this PR is compliant with the guide
[x] Ensured the resource API versions in .bicep file/s I am adding/editing are using the latest API version possible
[x] Checked for duplicate Pull Requests
[x] Associated it with relevant GitHub Issues
[ ] (ALZ Bicep Core Team Only) Associated it with relevant ADO Items
[x] Ensured my code/branch is up-to-date with the latest changes in the main branch
[x] Performed testing and provided evidence.
[ ] Updated one or more of the following tests (if required)

Unit
Linting
E2E (End-To-End)
ValidateAzCloud (Base validation in Azure Cloud)
ValidateMcCloud (Base validation in Azure China Cloud)

[x] Updated relevant and associated documentation (e.g. Contribution Guide, Module READMEs, Wiki Docs etc.)
[ ] If relevant, created or updated Code Tours here

@microsoft-github-policy-service agree [company="Airwair International Limited"]

@MilesCameron-DMs please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]

Options:

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.

@microsoft-github-policy-service agree

(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement
Contribution License Agreement
This Contribution License Agreement (“Agreement”) is agreed to by the party signing below (“You”), and conveys certain license rights to Microsoft Corporation and its affiliates (“Microsoft”) for Your contributions to Microsoft open source projects. This Agreement is effective as of the latest signature date below.

Definitions.
“Code” means the computer software code, whether in human-readable or machine-executable form,
that is delivered by You to Microsoft under this Agreement.
“Project” means any of the projects owned or managed by Microsoft and offered under a license
approved by the Open Source Initiative (www.opensource.org).
“Submit” is the act of uploading, submitting, transmitting, or distributing code or other content to any
Project, including but not limited to communication on electronic mailing lists, source code control
systems, and issue tracking systems that are managed by, or on behalf of, the Project for the purpose of
discussing and improving that Project, but excluding communication that is conspicuously marked or
otherwise designated in writing by You as “Not a Submission.”
“Submission” means the Code and any other copyrightable material Submitted by You, including any
associated comments and documentation.
Your Submission. You must agree to the terms of this Agreement before making a Submission to any
Project. This Agreement covers any and all Submissions that You, now or in the future (except as
described in Section 4 below), Submit to any Project.
Originality of Work. You represent that each of Your Submissions is entirely Your original work.
Should You wish to Submit materials that are not Your original work, You may Submit them separately
to the Project if You (a) retain all copyright and license information that was in the materials as You
received them, (b) in the description accompanying Your Submission, include the phrase “Submission
containing materials of a third party:” followed by the names of the third party and any licenses or other
restrictions of which You are aware, and (c) follow any other instructions in the Project’s written
guidelines concerning Submissions.
Your Employer. References to “employer” in this Agreement include Your employer or anyone else
for whom You are acting in making Your Submission, e.g. as a contractor, vendor, or agent. If Your
Submission is made in the course of Your work for an employer or Your employer has intellectual
property rights in Your Submission by contract or applicable law, You must secure permission from Your
employer to make the Submission before signing this Agreement. In that case, the term “You” in this
Agreement will refer to You and the employer collectively. If You change employers in the future and
desire to Submit additional Submissions for the new employer, then You agree to sign a new Agreement
and secure permission from the new employer before Submitting those Submissions.
Licenses.

Copyright License. You grant Microsoft, and those who receive the Submission directly or
indirectly from Microsoft, a perpetual, worldwide, non-exclusive, royalty-free, irrevocable license in the
Submission to reproduce, prepare derivative works of, publicly display, publicly perform, and distribute
the Submission and such derivative works, and to sublicense any or all of the foregoing rights to third
parties.
Patent License. You grant Microsoft, and those who receive the Submission directly or
indirectly from Microsoft, a perpetual, worldwide, non-exclusive, royalty-free, irrevocable license under
Your patent claims that are necessarily infringed by the Submission or the combination of the
Submission with the Project to which it was Submitted to make, have made, use, offer to sell, sell and
import or otherwise dispose of the Submission alone or with the Project.
Other Rights Reserved. Each party reserves all rights not expressly granted in this Agreement.
No additional licenses or rights whatsoever (including, without limitation, any implied licenses) are
granted by implication, exhaustion, estoppel or otherwise.

Representations and Warranties. You represent that You are legally entitled to grant the above
licenses. You represent that each of Your Submissions is entirely Your original work (except as You may
have disclosed under Section 3). You represent that You have secured permission from Your employer to
make the Submission in cases where Your Submission is made in the course of Your work for Your
employer or Your employer has intellectual property rights in Your Submission by contract or applicable
law. If You are signing this Agreement on behalf of Your employer, You represent and warrant that You
have the necessary authority to bind the listed employer to the obligations contained in this Agreement.
You are not expected to provide support for Your Submission, unless You choose to do so. UNLESS
REQUIRED BY APPLICABLE LAW OR AGREED TO IN WRITING, AND EXCEPT FOR THE WARRANTIES
EXPRESSLY STATED IN SECTIONS 3, 4, AND 6, THE SUBMISSION PROVIDED UNDER THIS AGREEMENT IS
PROVIDED WITHOUT WARRANTY OF ANY KIND, INCLUDING, BUT NOT LIMITED TO, ANY WARRANTY OF
NONINFRINGEMENT, MERCHANTABILITY, OR FITNESS FOR A PARTICULAR PURPOSE.
Notice to Microsoft. You agree to notify Microsoft in writing of any facts or circumstances of which
You later become aware that would make Your representations in this Agreement inaccurate in any
respect.
Information about Submissions. You agree that contributions to Projects and information about
contributions may be maintained indefinitely and disclosed publicly, including Your name and other
information that You submit with Your Submission.
Governing Law/Jurisdiction. This Agreement is governed by the laws of the State of Washington, and
the parties consent to exclusive jurisdiction and venue in the federal courts sitting in King County,
Washington, unless no federal subject matter jurisdiction exists, in which case the parties consent to
exclusive jurisdiction and venue in the Superior Court of King County, Washington. The parties waive all
defenses of lack of personal jurisdiction and forum non-conveniens.
Entire Agreement/Assignment. This Agreement is the entire agreement between the parties, and
supersedes any and all prior agreements, understandings or communications, written or oral, between
the parties relating to the subject matter hereof. This Agreement may be assigned by Microsoft.

@microsoft-github-policy-service agree company="Airwair International Ltd"

Great to have some more validation before deployment! 👍
The PR itself adds an approval gate, so maybe there is not need for the requirement to use environments in github.
The pipelines should limit the number of duplicated steps, it would be simpler to define each step only one time and trigger pipeline runs on both pull request into main and push into main. Some of the steps could have if-expressions to only run in certain scenarios:

The last step, Deploy X module, could have an expression so it only runs actual deployment on push into main, not pull requests.
And a step named "Push Whatif Output to PR" that only runs on pull requests.

Also I had a look at the updates to the Deploy-ALZ*.ps1 scripts. Just using the -WhatIf flag only prints the pending changes to the host/console, without any way to store the result in a variable. If the result of a WhatIf should be published as text into the PR; have a look at Get-AzManagementGroupDeploymentWhatIfResult, etc.
Code examples
e.g. Deploy-ALZSubscriptionPlacement.ps1 could return the WhatIf-results as text
if ($WhatIf) {
  return (Get-AzManagementGroupDeploymentWhatIfResult @inputObject | Out-String)
}

New-AzManagementGroupDeployment @inputObject

e.g alz-bicep-3.yml could use this results in a message on the PullRequest, as long as the env-var to enable this feature is true (allows others easier to enable/disable this)
on:
  push:
    branches: [ "main" ]
  pull_request:
    branches: [ "main" ]
  workflow_dispatch: {}

env:
  ENV_FILE: ".env"
  WHATIF_TO_PR: false

permissions:
    id-token: write
    contents: read
    pull-requests: write

...

      - name: What-If
        id: whatif
        if: github.event_name == 'pull_request'
        uses: azure/powershell@v1
        with:
          inlineScript: |
            $whatif = .\pipeline-scripts\Deploy-ALZSubscriptionPlacement.ps1 -WhatIf $true
            $whatif | Out-String
            -join (1..15 | ForEach {[char]((48..57)+(65..90)+(97..122) | Get-Random)}) | Set-Variable -Name EOF
            "WHATIF<<$EOF" >> $env:GITHUB_OUTPUT
            $whatif >> $env:GITHUB_OUTPUT
            "$EOF" >> $env:GITHUB_OUTPUT
          azPSVersion: "latest"

      - name: Push Whatif Output to PR
        if: github.event_name == 'pull_request' && env.WHATIF_TO_PR == true
        uses: actions/github-script@v6
        env:
          WHATIF: "${{ steps.whatif.outputs.WHATIF }}"
        with:
            github-token: ${{ secrets.GITHUB_TOKEN }}
            script: |
              const output = `#### Deployment WhatIf 📖\`${{ steps.whatif.outcome }}\`

              <details><summary>Show Plan</summary>

              \`\`\`text\n
              ${process.env.WHATIF}
              \`\`\`

              </details>

              *Pushed by: @${{ github.actor }}, Action: \`${{ github.event_name }}\`*`;

              github.rest.issues.createComment({
                issue_number: context.issue.number,
                owner: context.repo.owner,
                repo: context.repo.repo,
                body: output
              })

      - name: Deploy Subscription Placement
        if: github.ref == 'refs/heads/main' && github.event_name == 'push'
        uses: azure/powershell@v1
        with:
          inlineScript: |
            .\pipeline-scripts\Deploy-ALZSubscriptionPlacement.ps1
          azPSVersion: "latest"

@picccard Thanks for the feedback, i was working on using the output - perhaps for the step summary. One change at a time i guess.
Agree about the duplication of code and i changed my PR to minimise this.
Also, the environment approval runs before the deployment job runs, straight after the validation - unlike the approval gate via the branch protection which runs when there is a PR. Try it yourself and you will see it flows better. Take  your point about changing the triggers, i have similar workflows for some of our repo's.
I am just trying to get validation in to the workflow with what we have at the moment and we can work on the rest next?

Thanks @MilesCameron-DMs! I like the idea of using environments, especially if we end up incorporating the option to deploy to [canary environments](https://learn.microsoft.com/en-us/azure/cloud-adoption-framework/ready/enterprise-scale/testing-approach) within the Accelerator.
We were also thinking about adding the output to a comment as @picccard suggested as well. I will play around with this a bit this week and I may end up opening a PR with some potential changes as a heads up.

@MilesCameron-DMs I just opened a PR against your branch if you wouldn't mind taking a look. I am holding off on adding on adding the comments to outputs as the approach I want to go with requires some modification of the ALZ Powershell module.

@oZakari thanks for that. I have reviewed and merged. Much neater solution 👍

@oZakari thanks for that. I have reviewed and merged. Much neater solution 👍

Thank you, @MilesCameron-DMs! I resubmitted a PR as I realized I originally requested it against your main branch and not your feature branch.

/azp run e2e


fix oci.WithImageConfigArgs for windows

fix parameters passing for windows when use  oci.WithImageConfigArgs

Codecov Report

Merging #2887 into master will decrease coverage by 0.02%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #2887      +/-   ##
==========================================
- Coverage   43.89%   43.87%   -0.03%
==========================================
  Files         101      101
  Lines       10800    10806       +6
==========================================
  Hits         4741     4741
- Misses       5328     5334       +6
  Partials      731      731

Flag
Coverage Δ

#linux
47.48% <0%> (-0.03%)
:arrow_down:

#windows
41.04% <0%> (-0.03%)
:arrow_down:

Impacted Files
Coverage Δ

oci/spec_opts.go
22.32% <0%> (-0.19%)
:arrow_down:

Continue to review full report at Codecov.

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

LGTM to me. @crosbymichael @jterry75 PTAL


"bin/mango-submit -h" does not display anything

I am very confused by mango documentation
When I do "bin/mango-submit -h" I do not get any output even though I have $SPARK_HOME setup in a right way.

Hi @antonkulaga! Can you paste the output when you run ./bin/mango-submit -h? It should look something like this:

There is NO output on one machine but everything works fine on another, I am confused =(

And even on the machine where console works, here is what I see when I run
./example-files/run-example.sh

That is correct. you can click on available chromosomes to view the data.

That is correct. you can click on available chromosomes to view the data.

Maximum that I manage to see there is nucleotides. When I saw Readme I thought it is more like a normal genome viewer (with different tracks and useful info) and that image was generated from the example


The "--add-metadata" does not set the URL in the MP4 metadata

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2016.04.06. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2016.04.06

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add -v flag to your command line you run youtube-dl with, copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
$ youtube-dl -v <your command line>
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'http://www.youtube.com/watch?v=BaW_jenozKcj']
[debug] Encodings: locale cp1251, fs mbcs, out cp866, pref cp1251
[debug] youtube-dl version 2016.04.06
[debug] Python version 2.7.11 - Windows-2003Server-5.2.3790-SP2
[debug] exe versions: ffmpeg N-75573-g1d0487f, ffprobe N-75573-g1d0487f, rtmpdump 2.4
[debug] Proxy map: {}
...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Description of your issue, suggested solution and other information
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue required an account credentials please provide them or explain how one can obtain them.
Hi,
The --add-metadata does not set the URL in the MP4 metadata:
youtube-dl -v --ignore-config -f18 --add-metadata --prefer-ffmpeg https://www.youtube.com/watch?v=PUYs3KoBFOc
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'--ignore-config', u'-f18', u'--add-metadata', u'--prefer-ffmpeg', u'https://www.youtube.com/watch?v=PUYs3KoBFOc']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2016.04.05
[debug] Python version 2.7.8 - Linux-3.16.0-44-generic-x86_64-with-Ubuntu-14.10-utopic
[debug] exe versions: avconv 11.2-6, avprobe 11.2-6, ffmpeg 3.0.1-1, ffprobe 3.0.1-1, rtmpdump 2.4
[debug] Proxy map: {}
[youtube] PUYs3KoBFOc: Downloading webpage
[youtube] PUYs3KoBFOc: Downloading video info webpage
[youtube] PUYs3KoBFOc: Extracting video information
[youtube] PUYs3KoBFOc: Downloading MPD manifest
[debug] Invoking downloader on u'https://r4---sn-25ge7n76.googlevideo.com/videoplayback?signature=C930F7B110E213CDA797AC776669D7557468659F.2ECD3527EF19F3753B878F3350C578C1AFE407F6&sver=3&requiressl=yes&mt=1460492309&pl=21&dur=5526.743&lmt=1387500766085194&ip=<IP_ADDRESS>&ms=au&mv=m&source=youtube&id=o-AMQW0Bz8iKZoqba1jru8Pvj8GlMxtQ_wuzGm6ltMWxb_&mn=sn-25ge7n76&mm=31&key=yt6&expire=1460514012&sparams=dur%2Cid%2Cinitcwndbps%2Cip%2Cipbits%2Citag%2Clmt%2Cmime%2Cmm%2Cmn%2Cms%2Cmv%2Cnh%2Cpl%2Cratebypass%2Crequiressl%2Csource%2Cupn%2Cexpire&upn=91icWEVKEC8&mime=video%2Fmp4&itag=18&ratebypass=yes&fexp=9405995%2C9416126%2C9416891%2C9420452%2C9422596%2C9423662%2C9424490%2C9426927%2C9427324%2C9427413%2C9427902%2C9428398%2C9429119%2C9429379%2C9430020%2C9430066%2C9431366%2C9432684%2C9433114%2C9433892&ipbits=0&nh=IgpwcjAxLnBhcjAxKgkxMjcuMC4wLjE&initcwndbps=913750'
[download] Resuming download at byte 247006371
[download] Destination: mamadou karambiri - Comment liberer la nouvelle creation prise en otage-PUYs3KoBFOc.mp4
[download] 100% of 272.28MiB in 00:21
[ffmpeg] Adding metadata to 'mamadou karambiri - Comment liberer la nouvelle creation prise en otage-PUYs3KoBFOc.mp4'
[debug] ffmpeg command line: ffmpeg -y -i 'file:mamadou karambiri - Comment liberer la nouvelle creation prise en otage-PUYs3KoBFOc.mp4' -c copy -metadata comment= -metadata description= -metadata artist=herveekis -metadata 'title=mamadou karambiri - Comment liberer la nouvelle creation prise en otage' -metadata date=20130716 -metadata 'purl=https://www.youtube.com/watch?v=PUYs3KoBFOc' 'file:mamadou karambiri - Comment liberer la nouvelle creation prise en otage-PUYs3KoBFOc.temp.mp4'

mp4info 'mamadou karambiri - Comment liberer la nouvelle creation prise en otage-PUYs3KoBFOc.mp4' | egrep -i "url|www|http"
echo $?
1
ffprobe 'mamadou karambiri - Comment liberer la nouvelle creation prise en otage-PUYs3KoBFOc.mp4' | egrep -i "url|www|http"
echo $?
1

There is no URL field in mp4.


Prometheus adaptor fails to load custom metrics

What happened?
Used a new rule in promethues adaptor-config ConfigMap to use a custom metric in HPA via CustomMetric API.Works fine when the metric used in 'seriesQuery' is valid. If the metric in 'serierQuery' is invalid (metric became invalid when the pods go down or the metric does not exist any longer), all other valid metrics/series fail to load.
E0722 15:00:58.683001       1 provider.go:206] unable to update list of all metrics: no metrics retrieved for query

Did you expect to see some different?
Other than the rule/seriesQuery with invalid metric, the rest should load and should be available for custom metric API
How to reproduce it (as minimally and precisely as possible):
Deploy the manifest from https://github.com/coreos/kube-prometheus/blob/master/experimental/custom-metrics-api/custom-metrics-configmap.yaml
kubectl get --raw /apis/custom.metrics.k8s.io/v1beta1

Will give you results of custom metrics, which looks correct

Add a new rule with a invalid metric name like below.
```

seriesQuery: 'invalid_metric_name{namespace!="",pod!=""}'
seriesFilters: []
resources:
overrides:
namespace:
resource: namespace
pod:
resource: pod
name:
matches: "^(.*)_total"
as: "${1}_per_second"
metricsQuery: 'sum(rate(<<.Series>>{<<.LabelMatchers>>}[2m])) by (<<.GroupBy>>)'

kubectl get --raw /apis/custom.metrics.k8s.io/v1beta1
- Will give you empty results.

**Environment**

* Prometheus Operator version: v0.31.0

    `Insert image tag or Git SHA here`
    <!-- Try kubectl -n monitoring describe deployment prometheus-operator -->

* Kubernetes version information:

Client Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.0", GitCommit:"e8462b5b5dc2584fdcd18e6bcfe9f1e4d970a529", GitTreeState:"clean", BuildDate:"2019-06-19T16:40:16Z", GoVersion:"go1.12.5", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.0", GitCommit:"e8462b5b5dc2584fdcd18e6bcfe9f1e4d970a529", GitTreeState:"clean", BuildDate:"2019-06-19T16:32:14Z", GoVersion:"go1.12.5", Compiler:"gc", Platform:"linux/amd64"}
    `kubectl version`
    <!-- Replace the command with its output above -->

* Kubernetes cluster kind:
        Local minikube
    insert how you created your cluster: kops, bootkube, tectonic-installer, etc.

* Manifests:

apiVersion: v1
kind: ConfigMap
metadata:
name: adapter-config
namespace: monitoring
data:
config.yaml: |
rules:
- seriesQuery: '{name=~"^container_.",container_name!="POD",namespace!="",pod_name!=""}'
seriesFilters: []
resources:
overrides:
namespace:
resource: namespace
pod_name:
resource: pod
name:
matches: ^container_(.)seconds_total$
as: ""
metricsQuery: sum(rate(<<.Series>>{<<.LabelMatchers>>,container_name!="POD"}[1m])) by (<<.GroupBy>>)
- seriesQuery: '{name=~"^container.",container_name!="POD",namespace!="",pod_name!=""}'
seriesFilters:
- isNot: ^container_.seconds_total$
resources:
overrides:
namespace:
resource: namespace
pod_name:
resource: pod
name:
matches: ^container(.)total$
as: ""
metricsQuery: sum(rate(<<.Series>>{<<.LabelMatchers>>,container_name!="POD"}[1m])) by (<<.GroupBy>>)
- seriesQuery: '{name=~"^container.",container_name!="POD",namespace!="",pod_name!=""}'
seriesFilters:
- isNot: ^container_.total$
resources:
overrides:
namespace:
resource: namespace
pod_name:
resource: pod
name:
matches: ^container(.)$
as: ""
metricsQuery: sum(<<.Series>>{<<.LabelMatchers>>,container_name!="POD"}) by (<<.GroupBy>>)
- seriesQuery: '{namespace!="",name!~"^container_."}'
seriesFilters:
- isNot: .total$
resources:
template: <<.Resource>>
name:
matches: ""
as: ""
metricsQuery: sum(<<.Series>>{<<.LabelMatchers>>}) by (<<.GroupBy>>)
- seriesQuery: '{namespace!="",name!~"^container."}'
seriesFilters:
- isNot: ._seconds_total
resources:
template: <<.Resource>>
name:
matches: ^(.)total$
as: ""
metricsQuery: sum(rate(<<.Series>>{<<.LabelMatchers>>}[1m])) by (<<.GroupBy>>)
- seriesQuery: '{namespace!="",name!~"^container."}'
seriesFilters: []
resources:
template: <<.Resource>>
name:
matches: ^(.)_seconds_total$
as: ""
metricsQuery: sum(rate(<<.Series>>{<<.LabelMatchers>>}[1m])) by (<<.GroupBy>>)
- seriesQuery: 'tomcat_requestcount_total{namespace!="",pod!=""}'
seriesFilters: []
resources:
overrides:
namespace:
resource: namespace
pod:
resource: pod
name:
matches: ^(.)_total
as: "${1}_per_second"
metricsQuery: sum(rate(<<.Series>>{<<.LabelMatchers>>}[2m])) by (<<.GroupBy>>)
- seriesQuery: 'http_requests_total{namespace!="",pod!=""}'
seriesFilters: []
resources:
overrides:
namespace:
resource: namespace
pod:
resource: pod
name:
matches: ^(.)_total
as: "${1}_per_second"
metricsQuery: sum(rate(<<.Series>>{<<.LabelMatchers>>}[2m])) by (<<.GroupBy>>)
- seriesQuery: 'invalid_metric_name{namespace!="",pod!=""}'
seriesFilters: []
resources:
overrides:
namespace:
resource: namespace
pod:
resource: pod
name:
matches: "^(.)_total"
as: "${1}_per_second"
metricsQuery: 'sum(rate(<<.Series>>{<<.LabelMatchers>>}[2m])) by (<<.GroupBy>>)'
resourceRules:
cpu:
containerQuery: sum(rate(container_cpu_usage_seconds_total{<<.LabelMatchers>>}[1m])) by (<<.GroupBy>>)
nodeQuery: sum(rate(container_cpu_usage_seconds_total{<<.LabelMatchers>>, id='/'}[1m])) by (<<.GroupBy>>)
resources:
overrides:
node:
resource: node
namespace:
resource: namespace
pod_name:
resource: pod
containerLabel: container_name
memory:
containerQuery: sum(container_memory_working_set_bytes{<<.LabelMatchers>>}) by (<<.GroupBy>>)
nodeQuery: sum(container_memory_working_set_bytes{<<.LabelMatchers>>,id='/'}) by (<<.GroupBy>>)
resources:
overrides:
node:
resource: node
namespace:
resource: namespace
pod_name:
resource: pod
containerLabel: container_name
window: 1m

* Prometheus Adapter Logs:

E0722 15:39:50.045412       1 provider.go:206] unable to update list of all metrics: no metrics retrieved for query "invalid_metric_name{namespace!="",pod!=""}"

* Prometheus Logs:

Insert Prometheus logs relevant to the issue here

**Anything else we need to know?**:

cc @s-urbaniak @metalmatze

@gogineni99 do you mind to open an issue in https://github.com/DirectXMan12/k8s-prometheus-adapter itself? As far as I see this is a legitimate issue, thanks for reporting!

Opened https://github.com/DirectXMan12/k8s-prometheus-adapter/issues/225. Thank you

Moving the discussion to the other repo. Thanks for opening!


"Unexpected extra semicolon" in nested block

Howdy! Thanks for putting this syntax together. For the most part, it works great. However, somehow it seems to be allergic to semicolons in nested blocks. For example:
const RedBox = styled.div`
  ${css`
    background-color: red; // Triggers "Unexpected extra semicolon" (no-extra-semicolons)
  `}
`;

This rule is autofixable, which produces some strange results. For example, both semicolons in this example are flagged as rule violations:
const RedBox = styled.div`
  ${css`
    background-color: red;
    color: white;
  `}
`;

That causes the autofix to generate this code, which is erroneous:
const RedBox = styled.div`
  ${css`
    background-color: red // Triggers "Missed semicolon" (CssSyntaxError)
    color: white
  `}
`;

None of this strange behavior occurs if the css block is extracted from the styled block.

For reference, here's my .stylelintrc:
{
  "extends": [
    "stylelint-config-recommended",
    "stylelint-config-styled-components"
  ],
  "customSyntax": "postcss-styled-syntax"
}

And the relevant dependencies:
"postcss": "^8.4.20",
"postcss-syntax": "^0.36.2",
"postcss-styled-syntax": "^0.1.0",
"styled-components": "^5.3.6"
"stylelint": "^14.16.0",
"stylelint-config-recommended": "^9.0.0",
"stylelint-config-styled-components": "^0.1.1",

It might be an issue with stylelint. I'll take a look in coming days.

This is the issue with Stylelint rule. However, this rule will be deprecated in upcoming v15 https://github.com/stylelint/stylelint/issues/6563#issuecomment-1374682768
I suggest to remove this rule from config.


[TEST] benchmark gas unit price 1

/canary

:broken_heart: Test Failed - Build images and run cluster test


HealthCheckRegistry.unregister() should return the removed HealthCheck

The HealthCheckRegistry should return the removed HealthCheck when you unregister it. This would allow you for the user to manipulate the check, wrapping it in another class, or in my particular use case. It would allow me to move a check from one registry to another.

Tracking in #781


Added two new tests to Archive & Unarchive locations

As part of fix #5, we have to add the tests to for the following workflows:

Archive & Unarchive Locations
Download & Upload Web Users

Creating a PR for review for Archive & Unarchive location tests added.

Hi @kiran-tripathy , I have added the remaining following tests

Download/ Upload Web Users
Creation of Dedupe Cases rule
Upload and Download Data Dictionary

And have also made changes to the Archive and Unarchvie locations as per your feedback.

I added Kankana to review this, just skimmed over -- If works on CI as well, we can merge this.


Fix #49777 - Emmet balance In after balance out should go back to initial selection and not first child

This should fix #49777 according to the instructions of @ramya-rao-a.
This is my first contribution, I hope everything is alright 😃.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: Heldenkrieger01 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@Heldenkrieger01 I have made the necessary changes


Fix for cv2.imread terminations (issue #25) and python 2 cross-compatibility

Let me know if you want me to create an issue for the python 2 division errors.

thank you very much for your PR, I've merged it.
For Python 2 compatibility, I used future instead such that both print and division now behave as known from Python 3.


Unknown error from PostCSS plugin. Your current PostCSS version is 8.1.7, but postcss-preset-env uses 7.0.35

This is the error-message:

Unknown error from PostCSS plugin. Your current PostCSS version is 8.1.7, but postcss-preset-env uses 7.0.35. Perhaps this is the source of the error below.

ready - started server on <IP_ADDRESS>:3000, url: http://localhost:3000
info  - Using external babel configuration from /home/a/Websites/devilbox/data/www/project-nextjs/htdocs/frontend/babel.config.json

Unknown error from PostCSS plugin. Your current PostCSS version is 8.1.7, but postcss-preset-env uses 7.0.35. Perhaps this is the source of the error below.

error - ./styles/utilities.css ((webpack)/css-loader/cjs.js??ref--5-oneOf-6-1!(webpack)/**postcss-loader/cjs.js??ref--5-oneOf-6-2!./styles/utilities.css)**

RangeError: Maximum call stack size exceeded
    at Function.keys (<anonymous>)
Error: Cannot find module '/home/a/Websites/devilbox/data/www/project-nextjs/htdocs/frontend/.next/build-manifest.json'
Require stack:
- /home/a/Websites/devilbox/data/www/project-nextjs/htdocs/frontend/node_modules/next/dist/next-server/server/load-components.js
- /home/a/Websites/devilbox/data/www/project-nextjs/htdocs/frontend/node_modules/next/dist/next-server/server/api-utils.js
- /home/a/Websites/devilbox/data/www/project-nextjs/htdocs/frontend/node_modules/next/dist/next-server/server/next-server.js
- /home/a/Websites/devilbox/data/www/project-nextjs/htdocs/frontend/node_modules/next/dist/server/next.js
- /home/a/Websites/devilbox/data/www/project-nextjs/htdocs/frontend/node_modules/next/dist/server/lib/start-server.js
- /home/a/Websites/devilbox/data/www/project-nextjs/htdocs/frontend/node_modules/next/dist/cli/next-dev.js
- /home/a/Websites/devilbox/data/www/project-nextjs/htdocs/frontend/node_modules/next/dist/bin/next
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:880:15)
    at Function.mod._resolveFilename (/home/a/Websites/devilbox/data/www/project-nextjs/htdocs/frontend/node_modules/next/dist/build/webpack/require-hook.js:4:1715)
    at Function.Module._load (internal/modules/cjs/loader.js:725:27)
    at Module.require (internal/modules/cjs/loader.js:952:19)

My Postcss.config.js
module.exports = {
  plugins: [
    "@tailwindcss/jit",
    "postcss-flexbugs-fixes",
    "postcss-nested",
    "autoprefixer",
    [
      "postcss-preset-env",
      {
        autoprefixer: {
          flexbox: "no-2009",
        },
        stage: 3,
        features: {
          "custom-properties": false,
        },
      },
    ],
  ],
};

Babel.config.json
{
    "presets": ["next/babel"],
    "plugins": [
    ["inline-react-svg"],
    ["styled-components", { "ssr": true }]
  ]
  }

package.json
  "devDependencies": {
    "@babel/cli": "^7.13.0",
    "@babel/core": "^7.13.1",
    "@tailwindcss/jit": "^0.1.1",
    "@types/sortablejs": "^1.10.6",
    "autoprefixer": "^10.2.5",
    "babel-plugin-inline-react-svg": "^2.0.1",
    "babel-plugin-styled-components": "^1.12.0",
    "next-server-context": "^2.0.0",
    "postcss": "^8.2.8",
    "postcss-flexbugs-fixes": "4.2.1",
    "postcss-nested": "^4.2.3",
    "postcss-preset-env": "^6.7.0",
    "tailwindcss": "^2.0.2"
  }

Any idea?

The error seems very clear: postcss-preset-env doesn't support PostCSS v8, see csstools/postcss-preset-env#191.


Added BIP path text field to transaction signer

Adds support for any SEP-0005 path #193 by providing additional text field. Pre-filled with default value (44'/148'/0'). This field also has basic format validation. See attachment

Looks good but can you move the input field to the "Ledger Wallet" section? Other than that 👍 .

@bartekn Done :)

OK, great. I'm sorry but I forgot to ask you to sign the Contributor License Agreement. I'll merge it after that.

Your response has been recorded.

Signed with blood :)


App service relase2022.06

Generate Code with this branch
Tag: package-2021-03-01

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.AppService


Per default don't strip IRC colors to IRC channels (tengo)

Per default don't strip IRC colors to IRC channels (tengo).
Added missing variable and documentation to example tengo script in contrib/

I am unsure why but tengo doesn't seem to enjoy the way I think. I am trying the following as can be seen in https://github.com/42wim/matterbridge/pull/807/commits/e475e380e9d605516451ef7354b97d6bd98d93e3 :
if inProtocol == "irc" {
    if outProtocol != "irc" {
    re := text.re_compile(`\x03(?:\d{1,2}(?:,\d{1,2})?)?|[[:cntrl:]]`)
    msgText=re.replace(msgText,"")
    }
}

I currently have no [tengo] configuration in my .toml config file, however this would still strip IRC colors to both IRC and other outProtocols though. Can I have a helping hand on this?

Maybe inProtocol/outProtocol are flipped>

I just flipped the variables. It would still strip colors (?). So I reverted this and I used just this:
if outProtocol != "irc" {
    re := text.re_compile(`\x03(?:\d{1,2}(?:,\d{1,2})?)?|[[:cntrl:]]`)
    msgText=re.replace(msgText,"")
}

...and... it would still strip all colors. So I looked at the commits and realized, this probably has to get patched in internal/bindata.go too?
var _bindataTengoOutmessagetengo = []byte(
"\x1f\x8b\x08\x00\x00\x00\x00\x00\x00\xff\xc4\x91\x3d\x8f\xda\x40\x10\x86\xfb\xfd\x15\x13\x37\xb1\x2d\x07\xe7\xa3" +
"\xb3\x64\x59\x11\x45\x94\x2e\x8a\x92\x0a\xd0\xb1\xac\x07\x33\xd2\x7a\xc7\x1a\x8f\x31\x88\xe3\xbf\x9f\xcc\x01\x47" +
"\x7f\xc5\x75\xef\xae\x9e\x9d\x77\x1f\x4d\x9e\x9a\xbd\x15\xb2\x1b\x8f\x3d\xd8\xbd\x25\x3f\x45\x30\x82\xb6\xfe\xc2" +
"\xc1\x1f\x0b\x43\xe1\xa7\x73\x3c\x04\xcd\x80\xc2\x1f\x61\x65\xc7\x7e\xca\xf3\x9d\x0d\x01\x2f\xf1\x97\x55\x1c\xed" +
"\xd1\xf0\xa0\x77\x98\x07\x7d\xa3\x79\xd0\x3b\xce\x83\xde\xf8\xd7\x9e\x51\x48\xb1\x30\x6d\xdf\xfc\xc3\x83\x66\xd0" +
"\xf6\xcd\xff\x1e\x25\xd8\x16\x4d\x9a\x1b\xa3\x78\x50\x28\x4a\xa0\xb6\x63\xd1\x38\x9a\xce\x51\x62\x4c\x9e\x43\xaf" +
"\x42\x1d\x90\x38\x70\xec\x59\xfa\xe9\x8e\xb6\x30\xe2\x67\x41\x08\xac\xd0\x63\xa8\x29\x34\xa0\x0c\x36\x5c\xc0\x8d" +
"\x50\xdd\x20\x8c\x78\x7d\xac\x3b\x84\xdf\x7f\xe7\xb7\x01\xb4\x7d\xd0\x84\xb2\x84\x88\xc4\x45\x70\x32\x00\x00\x82" +
"\xd3\x3f\xa6\xfe\x99\xe0\x93\xe3\xb6\x23\x8f\xf1\x7a\x79\xf8\xfa\x23\xae\x8a\x65\x7d\xfa\x96\x7d\x3f\xc7\x55\x91" +
"\x5d\x63\x52\x25\xd5\xf3\x62\x51\xb8\xa0\xe2\x8b\xd5\x6a\x9d\x5c\xc6\x5c\x4d\x4b\xc1\x99\x60\xe7\xad\xc3\xf8\x26" +
"\x1f\x45\x89\x39\x9b\xf7\x6b\xe4\x29\x6d\x1f\x57\x00\x9f\x3e\xc6\x24\xcd\xcd\x4b\x00\x00\x00\xff\xff\x40\xb8\x54" +
	"\xb8\x64\x02\x00\x00")

I've added some comments.
You can't modify internal/outmessage.tengo without regenerating the bindata.go (which is generated code) which adds the asset into the binary so that users still have 1 single binary.

Closing for now as https://github.com/42wim/matterbridge/tree/tengo-out got merged. I'll re-open later.


Added top level documentation for classes and modules.

In this PR

Added a simple documentation line
Added missing frozen_string_literal to gemfiles/*.gemfile
Fixed minor format inconsistencies in .rubocop.yml

This PR is stale and will be closed.


Fix failing tests

This PR will investigate and hopefully eventually solve the issues we currently observe when it comes to testing the library.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.05%) to 91.693%

Files with Coverage Reduction
New Missed Lines
%

lib/server.ts
1
86.4%

Totals

Change from base Build<PHONE_NUMBER>:
-0.05%

Covered Lines:
2887

Relevant Lines:
3103

💛  - Coveralls

Apparently, it's only two tests that are broken under Node 20. I will look into how to solve the problem here.

overlaps replaces https://github.com/coapjs/node-coap/pull/362 ?

overlaps replaces #362 ?

Hmm, good point! Maybe updating will actually solve the issue here – I will test it out :)

I think this PR is now ready for review :) The tests are passing now again (I also added Node 22 to the test matrix) – however, I needed to skip a few of them since there seem to be some changes in Node 20 and above that cause issues with sinon.
In general, the tests are somewhat flaky at the moment, but I think we can probably deal with that in future PRs. For now, it is probably more important to be able to move forward again (and maybe also release a new version).


Automatically Post to GitHub on Failure

It'd be awesome to have this project automatically comment on the GitHub pull request with a link to the failures HTML page.

This could be handled by Danger.

Was easy to add: https://github.com/artsy/eigen/blob/4454d6bb595a4fcc0d491c75eed07e24cae52300/Dangerfile#L69-L73


Serializing CountVectorizer and Models

Hello,
Is there a way to serialize the preprocessing steps like CountVectorizer?
Any help would be greatly appreciated!

linfa-preprocessing currently has no Serde support, but I don't see any reason why it can't be added. If you open a PR adding Serde support under a feature flag (similar to other Linfa crates) then I'd be happy to look over it.

Added the serde support in a patched version. Seems to work. Will push after testing. Thanks!

Resolved by #292


remove "extend" hack

https://github.com/kamilsk/CilexServiceProviders/blob/4.x/tests/ClassAvailabilityTest.php#L39-L43
see https://github.com/kamilsk/Common/issues/55

will not do


Missing KIC guide on Key Authentication with ACL per service or route

Where is the problem?
https://docs.konghq.com/hub/kong-inc/key-auth/
What happened?
It would be good if there is a guide describing non-trivial case of configuration:

Two services or routes, A and B
Two API keys, 1 and 2
Key 1 can access A, but can not access B
Key 2 can access B, but can not access A

It requires combination of the Key Authentication plugin and ACL plugin.
It would be useful if presented with declarative configuration.
Such configuration for Kong Ingress Controller is especially not trivial, because it requires

Creation of 'conceptual/logical' ACL group via Kubernetes secret which is not very intuitive and not well documented, if at all, especially after deprecating kind: KongCredential
API key stored in secret
Consumer using the two secrets as credentials: one to authorise access to specific service/route and the other to authenticate consumer

What did you expect to happen?
Add a guide for Kong Ingress Controller with non-trivial authentication and authorisation scenarios.
Code of Conduct and Community Expectations

[X] I agree to follow this project's Code of Conduct
[X] I agree to abide by the Community Expectations

I'm also interested in this

Hey @mloskot, @eschbach-dd 👋
At this moment we have the guide for using ACLs in KIC: https://docs.konghq.com/kubernetes-ingress-controller/latest/plugins/acl/
Please let us know if that works for you or there's anything missing to be covered. Looking at the description I believe we should be good. LMK.


jdk_jfr tests fail on jdk8 hotspot xLinux

The following tests from jdk_jfr in the extended openjdk test suite fail on xLinux
https://ci.adoptopenjdk.net/job/Test_openjdk8_hs_extended.openjdk_x86-64_linux_testList_1/30/testReport/
jdk/jfr/event/compiler/TestCompilerCompile.java.TestCompilerCompile
[2022-01-19T14:19:55.882Z] TEST RESULT: Error. Program
`/home/jenkins/workspace/Test_openjdk8_hs_extended.openjdk_x86-64_linux_testList_1/openjdkbinary/j2sdk-image/bin/java'
timed out (timeout set to 960000ms, elapsed time including timeout handling was 960365ms).

jdk/jfr/event/runtime/TestShutdownEvent.java.TestShutdownEvent
[2022-01-19T14:27:29.508Z] TEST RESULT: Error. Program
`/home/jenkins/workspace/Test_openjdk8_hs_extended.openjdk_x86-64_linux_testList_1/openjdkbinary/j2sdk-image/bin/java'
timed out (timeout set to 960000ms, elapsed time including timeout handling was 960366ms).

jdk/jfr/event/runtime/TestShutdownEvent.java.TestShutdownEvent has been excluded on aarch64 linux but not yet for xLinux
https://github.com/adoptium/aqa-tests/pull/3007

Deep history shows this test passes occasionally.  Can we verify the status of the jdk_jfr tests from the October release on this platform?  If we hit the same issues with last release (and reran to see it passing), we will do the same with this release.  Try grinding on the machine its shown to pass in the deep history view (test-docker-ubuntu2010-x64-3).  If passes, what is special about that machine in relation to what this test is doing (look at test code).  If fails, run with ITERATIONS=10 to see what is the pass rate.
We need to take extra action to try and figure out root cause (test / infra / product) and raise an issue in the appropriate location.
If test issue:
We should determine if this is a 'flaky' test (JBS report for test material issue)
If infra issue:
one that needs a particular machine setup and raise/fix infra requirements (adoptium/infrastructure issue).
If intermittent product issue:
JBS report for JFR issue

Rerunning on the machine it last passed on, test-docker-ubuntu2010-x64-3, https://ci.adoptopenjdk.net/job/Grinder/3263/console

@Haroon-Khel @smlambert
The deep history log provides a dump when the timeout occurs:
https://trss.adoptium.net/output/test?id=61e848cba7df1a2924028e9a
TestCompilerCompiles has:
[2022-01-19T14:46:27.450Z] "MainThread" #9 prio=5 os_prio=0 tid=0x00007f0e2c12a800 nid=0x158efa waiting on condition [0x00007f0df70ee000]
[2022-01-19T14:46:27.450Z]    java.lang.Thread.State: TIMED_WAITING (sleeping)
[2022-01-19T14:46:27.450Z] 	at java.lang.Thread.sleep(Native Method)
[2022-01-19T14:46:27.450Z] 	at jdk.test.lib.Utils.waitForCondition(Utils.java:530)
[2022-01-19T14:46:27.450Z] 	at jdk.test.lib.Utils.waitForCondition(Utils.java:499)
[2022-01-19T14:46:27.450Z] 	at jdk.jfr.event.compiler.TestCompilerCompile.doTest(TestCompilerCompile.java:85)
[...]
[2022-01-19T14:46:27.450Z] Warning: 'NoSuchMethodError' on register of sun.hotspot.WhiteBox::NMTNewArena(J)J
[2022-01-19T14:46:27.450Z] Warning: 'NoSuchMethodError' on register of sun.hotspot.WhiteBox::NMTFreeArena(J)V
[2022-01-19T14:46:27.450Z] Warning: 'NoSuchMethodError' on register of sun.hotspot.WhiteBox::NMTArenaMalloc(JJ)V

which is the test waiting for the request to compile the method via WhiteBox.
TestShutdownEvent has:
[2022-01-19T14:54:31.488Z] "MainThread" #9 prio=5 os_prio=0 tid=0x00007fd7dc12a000 nid=0x159ece in Object.wait() [0x00007fd787db3000]
[2022-01-19T14:54:31.488Z]    java.lang.Thread.State: WAITING (on object monitor)
[2022-01-19T14:54:31.488Z] 	at java.lang.Object.wait(Native Method)
[2022-01-19T14:54:31.488Z] 	- waiting on <0x00007fd7c309e3a0> (a java.lang.UNIXProcess)
[2022-01-19T14:54:31.488Z] 	at java.lang.Object.wait(Object.java:502)
[2022-01-19T14:54:31.488Z] 	at java.lang.UNIXProcess.waitFor(UNIXProcess.java:395)
[2022-01-19T14:54:31.488Z] 	- locked <0x00007fd7c309e3a0> (a java.lang.UNIXProcess)
[2022-01-19T14:54:31.488Z] 	at jdk.test.lib.process.ProcessTools.getOutput(ProcessTools.java:98)
[2022-01-19T14:54:31.488Z] 	at jdk.test.lib.process.OutputAnalyzer.<init>(OutputAnalyzer.java:48)
[2022-01-19T14:54:31.488Z] 	at jdk.test.lib.process.ProcessTools.executeProcess(ProcessTools.java:412)
[2022-01-19T14:54:31.488Z] 	at jdk.jfr.event.runtime.TestShutdownEvent.runSubtest(TestShutdownEvent.java:82)

where it's waiting on the code to actually start up the subprocess. However the stdout has:
[2022-01-19T14:54:31.493Z] ...
[2022-01-19T14:54:31.493Z] Output overflow:
[2022-01-19T14:54:31.493Z] JT Harness has limited the test output to the text
[2022-01-19T14:54:31.493Z] at the beginning and the end, so that you can see how the
[2022-01-19T14:54:31.493Z] test began, and how it completed.
[2022-01-19T14:54:31.493Z]
[2022-01-19T14:54:31.493Z] If you need to see more of the output from the test,
[2022-01-19T14:54:31.493Z] set the system property javatest.maxOutputSize to a higher
[2022-01-19T14:54:31.493Z] value. The current value is 100000
[2022-01-19T14:54:31.493Z] ...

So we can't see exactly which subprocess was stuck.
I'd consider them both test/infra issues rather than product issues.

Thanks for this assessment @jiekang - based on it and the fact that this test passes intermittently on some machines (test-docker-ubuntu2010-x64-3 has both passed and failed before which also potentially indicates infra/test issues), we will not block the release on these failures, but indeed we should figure out which subprocess is stuck and why...
@Haroon-Khel - could it be the case that there are some processes laying around that then block some of this tests subprocesses from running?  and when we do a cleanup of the machines (static docker containers) we see 1 passing run before the machine is back to a state where this test fails again.

On test-docker-ubuntu2010-x64-3 there was a rogue Test_openjdk17_hs_extended.openjdk_x86-64_linux_testList_1 process from jan 18 still running.
Also, I noticed that test-docker-ubuntu2010-x64-3 is actually test-docker-fedora35-x64-1, I replaced test-docker-ubuntu2010-x64-3 with test-docker-ubuntu2110-x64-1 a few weeks back, and I must have forgotten to remove the jenkins node for test-docker-ubuntu2010-x64-3, and then when setting up test-docker-fedora35-x64-1 I gave it the same port on the dockerhost. That may explain why the test no longer passes on test-docker-ubuntu2010-x64-3, and so does hint towards this issue being a machine specific problem
Anyway, rerunning on test-docker-fedora35-x64-1 https://ci.adoptopenjdk.net/job/Grinder/3291/console


side chains

Any plans to add side chains in the demo ?

I'm going to close this as I haven't heard back from you. Feel free to reopen any time.


Allow anonymous signing

When the user wants to update e.g. votes, the voting widget does not need to know the identiy stored in companion.
For that purpose a placeholder account id can be used, that the companion fills in when receiving a transaction, e.g. 1.2.0 (no one will sign as committee using companion).

We still need to block access unless a user has allowed it.
So even if we drop the getAccount call from connect() , we still need a popup of "Allow app "YYYY" to interact with companion?"
You dont wanna get swamped with popups

The companion already has a popup for that .
"App "Name" wants to sign a transaction"
That is sufficient IMO. One could initially ask one time if this application should be whitelisted to send transactions.

those calls can be abused...you need to find a way to ask permission to interact first and deny subsequent requests altogether...i was using getAccount for that as you're very likely to need the account ID.
"One could initially ask one time if this application should be whitelisted to send transactions." <- this is what I mean...if you wanna use a separate confirmation then getAccount, fine with me...I just thought it saves on double requests cause 90% of the time you will also need the accountId


Reopen of #209 - TabView - iOS bottom tabs can get pushed off screen

Please see #209 for full details.

Let's stay focused on #209, though I don't think this is an issue anymore.


Minor formatting fix.

Description
Fixes a really minor formatting issue in one of the example scripts.

Checklist

[x ] Does npm test pass with this change (including linting)?
[x ] Is the new or changed code fully tested?
[ x] Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?

Codecov Report

Merging #1890 (91fce0e) into master (16afa7d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1890   +/-   ##
=======================================
  Coverage   95.05%   95.05%
=======================================
  Files         358      358
  Lines        3337     3337
  Branches      405      405
=======================================
  Hits         3172     3172
  Misses         83       83
  Partials       82       82

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 16afa7d...91fce0e. Read the comment docs.


Update to tokio 1.0

This upgrade was attempted, but it had to be reverted in #2379 because it caused at least two bugs that remain mysterious:

Seemly randomly, a very loaded node would not send chain head updates to syncing subgraphs. This could be worked around by a fallback such as #2378, but ideally chain head updates should always work.
Serious performance regressions in block processing time, at least in very loaded nodes. Uniswap in a community node had a 2x slowdown in block processing time.

We also had an update for web3 in #2362 which had to be reverted along with tokio. Attempting to upgrade again will require deep investigation into these two bugs.

Is there urgency around this tokio upgrade or is this just good hygiene?

Not urgent, but it prevents us from updating other networking dependencies such as hyper (the http server) and web3. This makes it unpleasant to contribute to the networking parts of the code and outdated dependencies may generate security issues. I'd classify this is high-interest technical debt. But resolving it would require commitment because the above issues are difficult to reproduce and root cause, it could take weeks.

This is done!


Re-Size the Mini-Browser Window

In many cases, when the mini-browser appears, the size is too small to fit all of the information inside the window.  This results in the user having to scroll to the bottom of the page to hit the submit button.
Can the mini-browser window dynamically adjust the size to fit all of the elements into the window, or at the very least, make the window larger to fit all of the elements without having to scroll to the bottom?
Thanks!

Thanks for highlighting this, appreciate it! We have plans to improve this, I'll update here.


Predicting of non-linear sequences

No need for this, might do another library for polynomial

No need for this, since this library takes only care of linear regression predicting

No need for this, since this library takes only care of linear regression predicting


Strange behavior using "path" parameter since 1.2.0

Hello,
Related to the problem that i commented here (https://github.com/karatelabs/karate/issues/2012), please found a repository to reprocude the problem locally.
Repo to reproduce the problem: https://github.com/kdefives/karate_path_issue
Java: 18
Maven: 3
Extract from the repo's README:
How to test the strange behavior?

Ensure to use Karate version 1.2.0 inside pom.xml, refresh maven dependencies en then execute the class src/test/java/features/SuiteTest.java (using IDEA for instance).
Normally, you will see the test using path parameter is failing.
Workaround 1: Inside file src/test/java/features/tests/test-feature-fail.feature, if you put the Given path .... after the call read, you will see the test passed. Normal? I do not know.
Workaround 2: Just rollback to Karate version 1.1.0 without any changes, all tests passed perfectly. Regression?

Regards,

I also face similar issue after I update Karate version from 1.2.0.RC1 to 1.2.0. All my tests failed after the upgrade. The "path" variable in the scenario is passed to the called feature file which is being called in configure headers js function. This "path" value is being appended to the url in the called feature file.
If this issue cannot be solved, I have to re-write all my test scenarios using url without path variable.

@cheanwei this is not a bug in karate and please read this discussion to understand what you need to do differently: https://github.com/karatelabs/karate/issues/1990
there is not such thing as a path variable. I am closing this issue also because I forgot to earlier
and to all who are reading this - please try to do a test upgrade when we release RC versions in future


Bug in responsive styles

Hey Kheva,
I've found a bug. Sorry for the amount of discussion that I'm throwing at you, I hope it's not too much.
In the example below there's a Text component that is supposed to change ONLY it's fontSize according to the device size.
Eventhough I explicitly state the fontWeight should be 300, it still changed it to 700 ( from the textbold style ) in screen sizes medium and smaller.
I'm fairly certain this is the intended use ( but not the intended result ). I'm suspecting the regex has something to do with this, not sure.

import { CreateResponsiveStyle, DEVICE_SIZES, maxSize } from 'rn-responsive-styles'
import { Text, View } from 'react-native';

export default function App() {
    const { styles } = useStyles();

    return (
        <View style={styles('center')}>
            <Text style={styles('text')}>Open up App.tsx to start working on your app!</Text>
        </View>
    );
}

const useStyles = CreateResponsiveStyle(
    {
        center: {
            flex: 1,
            justifyContent: "center",
            alignItems: "center",
            textAlign: "center",
        },
        text: {
            fontSize: 40,
            fontWeight: '300'
        },
        textlight: {
            fontWeight: '300'
        },
        textbold: {
            fontWeight: '700'
        },

    },
    {
        [maxSize(DEVICE_SIZES.MEDIUM_DEVICE)]: {
            text: {
                fontSize: 30,
                fontWeight: '300'
            },
            textlight: {
                fontWeight: '300'
            },
            textbold: {
                fontWeight: '700'
            },
        },
        [maxSize(DEVICE_SIZES.SMALL_DEVICE)]: {
            text: {
                fontSize: 20,
                fontWeight: '300'
            },
            textlight: {
                fontWeight: '300'
            },
            textbold: {
                fontWeight: '700'
            },
        }
    }
)

Wow! Good catch, yes this is an important issue. Thank you for finding this!
Also sorry for the late reply, I love that you are opening issues and contributing, it is nice to have someone else pushing development and improvements.


GraphQL Transformer V2 External Lambda Calls

Note: If your question is regarding the AWS Amplify Console service, please log it in the
AWS Amplify Console repository

Which Category is your question related to?
GraphQL API
Amplify CLI Version
10.0.0
You can use amplify -v to check the amplify cli version on your system
What AWS Services are you utilizing?
Lambda, AppSync
Provide additional details e.g. code snippets. Be sure to remove any sensitive data.
Hello,
We are in the process of migrating to the new transformer v2 and have noticed that the auth for IAM has changed a bit for AWS resources (lambda) calling the API. Specifically, we had some backend lambda functions calling the API and they used to work by just granting an IAM policy with the appropriate permissions. With this new transformer, based on the document https://docs.amplify.aws/cli/graphql/authorization-rules/#grant-lambda-function-access-to-graphql-api it looks like we need to have an iam role added to this custom-roles.json file and then assume that role on all our lambda functions which call the API.
Is there anyway to support the original way we had done this auth? Migrating our entire backend system to have to follow this new pattern has a large blast radius and scope of work, we would prefer to not have to do this.
Thank you

Hey @gakinson :wave: thanks for raising this! How were the functions previously set up to call the GraphQL API? The linked documentation is still active, however after granting access to the API you will need to make an empty update to the GraphQL API in order to trigger the resolvers to regenerate with the Lambda name. This is an active bug being tracked here https://github.com/aws-amplify/amplify-category-api/issues/679


your NN's : difference ?

i see many other NNUE files in your package .. can they be used instead of the default one ? What's their difference ? Could we train with specific pawn structures / opening setups ?
https://github.com/zzzzz151/z5/assets/1109281/228b60f4-ed35-462d-ba70-8a3fd08f250d

Those are NNs I've trained and tested.
You can change to a .nnue NN by adapting the NET_FILE and HIDDEN_LAYER_SIZE constants in nnue.hpp.
For net_512_1B_25wdl.nnue, you'll also need to change outputWeights to int16_t.


Supporting other web sites for plugins or support for web sites with port(?)

When I tried with my http://ip:port/ or http://ip:port/plugin, shelter doesn't add my plugin to list and I tried with github, it worked.
(Ps: I'm sure my port is open to web, I tried to access site with my phone with VPN.)
Question is: Why Shelter doesn't support links other than github?
Edit: I came across with other links like blabla.dev and now I think its because my ip have port?
Edit2: If it's because there is a port in my IP, is there a way to fix this? I would appreciate it if you could release an update if there is one.

It's most likely because of http. Switch to https to make it work, it's a web limitation and not shelter.

Oh, ok. Thanks.

It's most likely because of http. Switch to https to make it work, it's a web limitation and not shelter.

CONTENT SECURITY POLICY STRIKES AGAIN!!!!!!!


BeanCurrentlyInCreationException when create minioNotificationConfiguration bean on spring boot 2.6

using version:

com.jlefebure
spring-boot-starter-minio
1.10

spring boot version: 2.6.6
problem:
org.springframework.beans.factory.BeanCurrentlyInCreationException: Error creating bean with name 'minioNotificationConfiguration': Requested bean is currently in creation: Is there an unresolvable circular reference?
at org.springframework.beans.factory.support.DefaultSingletonBeanRegistry.beforeSingletonCreation(DefaultSingletonBeanRegistry.java:355)
at org.springframework.beans.factory.support.DefaultSingletonBeanRegistry.getSingleton(DefaultSingletonBeanRegistry.java:227)
at org.springframework.beans.factory.support.AbstractBeanFactory.doGetBean(AbstractBeanFactory.java:333)
at org.springframework.beans.factory.support.AbstractBeanFactory.getBean(AbstractBeanFactory.java:208)
at org.springframework.context.support.AbstractApplicationContext.getBean(AbstractApplicationContext.java:1154)
at com.jlefebure.spring.boot.minio.MinioNotificationConfiguration.setApplicationContext(MinioNotificationConfiguration.java:63)
at org.springframework.context.support.ApplicationContextAwareProcessor.invokeAwareInterfaces(ApplicationContextAwareProcessor.java:128)
at org.springframework.context.support.ApplicationContextAwareProcessor.postProcessBeforeInitialization(ApplicationContextAwareProcessor.java:102)
after debuing in MinioNotificationConfiguration.java:63 i found exception on beanName equal MinioNotificationConfiguration self bean.

As exception stack tells, the bug is raised by an 'unresolvable circular reference'.
When spring container is initializing this bean , as this bean implements SpringContextAware ,  the method setApplicationtContext(ApplicationContext applicationContext) is invoked.
However, in the method , author uses a for-loop to get every bean name in the spring context and pass the name to getBean, which will trigger the bean of that  beanName to be initialized(if that bean has not been initialized).
And this causes the bug, in the for-loop , the beanName of this bean itself will also appear and be passed into getBean. So it's a typical  unresolvable circular reference bug.
Though it is not recommended in the setApplicationContext to invoke getBean by Spring because this would trigger some irrelevant  beans to be  initialized too early .
Still author chooses to do so , maybe because the annotation MinioNotification (actually
Asnyc annotation) will  cause the target bean to be a proxy bean. And therefore the annotation meta data can not be obtained here. So he/she invokes the getBean method to get target object firstly , and then uses AopUtils to get the real class.
Actually, if you want get meta data from target class , you can use the ClassUtils which will get the parent class of that proxy class (which in CGLIB proxy style is the target class). This is suggested code(:

Hi @MusicGun
Where do you find the configurableListableBeanFactory ?
Thanks ;)

How to solve this issue? I have the same problem

Hello,
the issue is when beans are scanned, the bean minioNotificationConfiguration  is also in the pipe. Add this line at first line of the foreach :
`    @Override
public void setApplicationContext(ApplicationContext applicationContext) throws BeansException {
for (String beanName : applicationContext.getBeanDefinitionNames()) {
    	// ignore self
    	if("minioNotificationConfiguration".equals(beanName)) {
    		continue;
    	}

`
You may see my pull request: https://github.com/jlefebure/spring-boot-starter-minio/pull/26
It's solved the issue and migrate the starter to spring boot 3.
Enjoy.


Why different norm cfg in caffe style pretrained resnet50?

for example:
https://github.com/open-mmlab/mmdetection/blob/d40e19b09b19dd3dd55627ecf0f8d2f0796a1a03/configs/legacy_1.x/retinanet_r50_caffe_fpn_1x_coco_v1.py#L8
https://github.com/open-mmlab/mmdetection/blob/d40e19b09b19dd3dd55627ecf0f8d2f0796a1a03/configs/cascade_rcnn/cascade_rcnn_r50_caffe_fpn_1x_coco.py#L9

Hi @YongtaoGe
In caffe model, the std is absorbed by the backbone.

@xvjiarui The mean value [102.9801, 115.9465, 122.7717] and [103.530, 116.280, 123.675] is different while using the same ImageNet pre-trained model.

Ah, I see. My bad.
It is due to the minor difference in the pertained backbone. The legacy one is from detectron, the other one is from detectron2.

Got it! Thank you!


[AMORO-2988] Add more base info for the table detail page

Why are the changes needed?

Close #2988.
Brief change log

How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before making a pull request

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

It is OK to display other dimensions of information under the title, but I have a small suggestion. The information under the current table title and the metrics column on the right are a lot of duplication. I think the information under the title can be simplified. For example, the last commit time, table size, file count, avg file size and other information can be placed on the right.

It is OK to display other dimensions of information under the title, but I have a small suggestion. The information under the current table title and the metrics column on the right are a lot of duplication. I think the information under the title can be simplified. For example, the last commit time, table size, file count, avg file size and other information can be placed on the right.

If the optimizing status, records and table format is remained under the current table title, is it OK or do you have any suggestion which else information can be displayed there?

It is OK to display other dimensions of information under the title, but I have a small suggestion. The information under the current table title and the metrics column on the right are a lot of duplication. I think the information under the title can be simplified. For example, the last commit time, table size, file count, avg file size and other information can be placed on the right.

@XBaith  If the optimizing status, records and table format is remained under the current table title, is it OK or do you have any suggestion which else information can be displayed there?

@XBaith If the optimizing status, records and table format is remained under the current table title, is it OK or do you have any suggestion which else information can be displayed there?

That's ok for me.
We can display full table identifier as title, <catalog>.<database>.<table>

I think some of the metrics under the table identifier that duplicate the right hand side detail metrics should be removed, WDYT?

I think some of the metrics under the table identifier that duplicate the right hand side detail metrics should be removed, WDYT?

frontend or backend code?

last commit time, table size, file count, avg file size

Front side, metrics: last commit time, table size, file count and avg file size can be removed from the title?

last commit time, table size, file count, avg file size

Front side, metrics: last commit time, table size, file count and avg file size can be removed from the title?

Updated.

@tcodehuber Thanks for the contribution!
I'm wondering if Optimizing Status would be better. Optimizing highlights our continuous optimization capability, and here the status perfectly corresponds to the ongoing actions of continuous optimization.

@tcodehuber Thanks for the contribution!
I'm wondering if Optimizing Status would be better. Optimizing highlights our continuous optimization capability, and here the status perfectly corresponds to the ongoing actions of continuous optimization.

OK, I have followed your suggestion.


Command Migration: ('PFADD', 'PFCOUNT', 'PFMERGE')

This issue tracks the migration of the mentioned commands - ('PFADD', 'PFCOUNT', 'PFMERGE') to make them compatible across the three protocols supported by the Dice database: RESP, HTTP, and WebSocket.
DiceDB now supports more than one protocols (Resp/http/websocket). We don't want eval operation of each command to be strictly bound with any of the protocols. Currently eval function return statements of each command is bounded to the RESP protocol.
The migration is required because:

The current implementation is specific to the RESP protocol
Evaluation functions of each command must be refactored to ensure protocol-agnostic behavior.
Evaluation functions should return raw responses instead of protocol specific responses.

The goal is to make the command logic protocol-independent, allowing all three protocols to call the same core functionality seamlessly.
Requirements

Refactor the evaluation function of [Command Name] to be generic.
Ensure the evaluation function does not include protocol-specific logic.

Migration Steps

Analyze Current Implementation

Review the current codebase to understand how the command logic is implemented.

Refactor Return Logic

Create a new function with the function definition as evalXXX(args []string, store *dstore.Store) *EvalResponse under the file /internal/eval/store_eval.go
Analyse return statements of the eval and modify them to send raw types without encoding
Use errors in the return statements from /errors/migrated_errors.go file
Use perdefined responses from /internal/clientio/resp.go file as RespType variables

Command/Worker specific Changes

Make IsMigrated flag to true in the commands information under /internal/eval/commands.go file.
Use the newly written eval function against the NewEval parameter in the same command structure.
Delete the old eval function from the /internal/eval/eval.go file.
Add the migrated command to the /internal/worker/CommandsMeta map and make it's type as SingleShard

Update Unit Tests

Refactor existing unit tests to accommodate the new implementation.
Add new unit tests if necessary to cover all possible cases.

Integration Tests

Run all integration tests to ensure successful migration.
Ensure that each protocol (RESP, HTTP, WebSocket) works correctly after migration.

Checklist

[ ] Migrated the evalXXX function with the latest definition
[ ] Update or add unit tests for the new implementation.
[ ] All unit tests pass successfully.
[ ] Ensure all integration tests pass successfully.

Additional Notes

Describe any edge cases that need to be handled in the generic function.
Mention any protocol-specific optimizations required in wrappers.

If there are any questions or concerns about this migration, please mention them here.
Related Issues/PRs

Sample implementation for the Get, Set, GetSet and SetEx can be found in this Pull Request.

@AshwinKul28, @lucifercr07 @JyotinderSingh I would love to work on this. Please assign this to me

Thanks @surya0180 go for it 🚀

HI @surya0180 , I hope you are doing well. Do you happen to have any updates on this? If you've any doubts please let us know on the discord.

Hi @AshwinKul28. I am still exploring the codebase to better understand how things get triggered through different protocols. Working on Go language after a long time, its taking sometime for me to understand few things, but its actually done now. I am going to send a PR by EOD or at max by tomorrow morning!.
Thanks!

@AshwinKul28, There is a method in eval_test called runEvalTests, which runs the tests for the given eval function. Initially, this function was written for RESP only. However, since we are now supporting multiple protocols, we need to add a new function that does the same but accepts an eval function with a return type of EvalResponse. Based on what you shared in this issue, you referenced a PR that includes the migration of GET, SET, GETSET, etc., but I am unable to find the new method required to execute these new eval functions.
Could you please advise if we need to write the logic for this function inside our eval functions, or is there a reusable function we can use?

Hi @surya0180 thanks for the update.
Based on the sample PR, you need to migrate the current Eval function to the new one by moving it to the store_eval.go file and making required changes which returns *EvalResponse in return. You may need to make changes in the respective unit test functions.


Add support for specific images

Problem Description
Currently it takes longer than expected to find suitable photos for a design as the results are sometimes too random to meet specific design requirements. Also, search terms longer than a couple of words result in a network connection error which leads to making broad searches that end up compounding the problem.
Suggested Solution

Find photo on unsplash.com with a browser, and switch to Sketch;
Select a symbol or layer to add or refresh data feed photo;
Open 'Search Photo...' window, paste photo url, and hit return;
See the expected image show up in selected layer or symbol.

Additional Info

Sketch 52.6 (67491)
Unsplash Plugin v1.0.3
MacOS 10.14.1

Hi @untouchable, I think my pull request might be similar to what you're searching.

Sorry about that, I totally missed your PR! I'll review and merge it ASAP

@untouchable you said:

search terms longer than a couple of words result in a network connection error

can you please provide an example of this issue?

@patrickdaze's PR was just merged, and will be part of the next release of the plugin.
Thanks Patrick!


Convert scripts to modules, add console_scripts entry points

This makes it easier to unit test the code that implements the scripts. It also causes setuptools to install a .exe for each script on Windows. The modules can also be run with e.g. python -m u2flib_host.register
See

https://packaging.python.org/en/latest/distributing.html#console-scripts
https://pythonhosted.org/setuptools/setuptools.html#automatic-script-creation

Thank you, I was unaware of the automatic script creation. Much cleaner!


team: Rename Alumni section to Former members

Alumni means "past student", whereas the list includes Jean-Paul Laumond for instance.

@fabinsch looks good to you?


[cocoon] Refactor LuciBuildService to generally work

There's several methods like rescheduleProdBuild rescheduleTryBuild scheduleTrybuild scheduleProdBuild that we could have share logic instead of duplicating. This would make it easier to read and change buildbucket related logic.
Ideally, we want a _scheduleBuild that these all internal refer to, with minor additions in their respective calls.
Additionally, we want to migrate off the terms try and prod to presubmit and postsubmit to make it clearer.

Is this still open? Should we close this issue?

Is this still open? Should we close this issue?

This is still open. There's a lot of duplicate code in that class that's causing issues like https://github.com/flutter/flutter/issues/99523

@CaseyHillers only one of the four functions you mentioned are present in the file is present (scheduleTrybuild) and the remaining functions don't seem to share a significant amount of logic. is this issue still open?


New version: TeraTermProject.teraterm5 version 5.2

Pull request has been created with komac v2.1.0 :rocket:
Microsoft Reviewers: Open in CodeFlow

Hi @TejasMate,
The MinimumOSVersion has been removed - has this package been modified to work on all OSes?

Yes supports both Win 10, 11, You can merge this..

Yes supports both Win 10, 11, You can merge this..


Feature Proposal: Automatic Priority Channel for Scanning

Overview:
This proposal suggests adding an automatic priority channel function to the scanning feature in the Quansheng UV-K5 custom firmware.
Functionality:
The proposed feature would work as follows:
Initially, when scanning begins, there is no priority channel set.
Upon receiving a transmission on any channel:
This channel becomes Priority Channel 1 for a duration of one minute.
It will be scanned more frequently, out of the regular sequence of other channels.
If a transmission is received on another channel:
The current Priority Channel 1 becomes Priority Channel 2.
The new channel becomes Priority Channel 1.
Priority status is time-based:
After one minute without activity, a channel loses its Priority 1 status.
After two minutes without activity, a channel loses all priority status.
If no transmissions are received for two minutes, the scanning reverts to normal mode with no priority channels.

There is already a notion of priority channel in this firmware, easily configurable from Chirp. This is different from your suggestion, but it exist.

Yes, I'm aware that the priority channel feature exists, but it requires access to a computer to change the parameters. When in the field in a dynamic situation, it would be beneficial to have an extension of this functionality accessible from the radio's menu.
This proposed feature would allow users to dynamically adapt to changing communication needs without requiring pre-configuration via Chirp. It would enhance the radio's flexibility and usability in situations where quick adjustments are necessary.

About your last request, simply use SList (20/65) to set your default preferred scan list. Mine is list [1], not [0].


Add support for alarm

Is this module still under development?
Or is it wiser to start from python icalendar?

Hi! So sorry for the late response - must have missed the notification! I just started maintaining this module after the original author became too busy.
Are you referring to this part of the icalendar RFC? This project is pretty handy for making icalendar work well with Django specifically, but the icalendar module probably supports that feature already if you're looking for something quick. I'd definitely love to see this module support everything that icalendar does!
PRs are welcome 😃

I will come back to this. I use it now and it works fine! But I need alarm, so I will try to create a patch if I have time :)

After 2 years of inactivity this is a wontfix


Feature/app rating

Added a feature through which users can rate DocLense. Whenever the user will exit from the app, a popup will appear for rating DocLense.

I have moved the rating feature to the drawer. Users can now rate DocLense. On clicking on submit button they will be redirected to the play store. ( Please note: I haven't used the play store link there, just for demo purposes I have used the GitHub repo link, once it will get uploaded on the play store we can replace that link with the app link).
Kindly review this PR.

@Niteshkr123 Can you please close this PR and give it again. Since I have removed the .vscode/ and .gitignore it. We would have some merge conflicts


Ignore macOS .DS_Store file for VisualStudio

Development on macOS often leads to having a .DS_Store file that needs to be ignored. For information about the .DS_Store file, refer to https://wikipedia.org/wiki/.DS_Store.

I was pointed toward Added missing ignore on VisualStudio.gitignore. (github/gitignore#3715), which was closed with the guidance of Global/macOS.gitignore for my global .gitignore.


Add lading screen

Related issues:

AEROGEAR-3033

Identity Manager
Security
Push
Metrics

@danielpassos can we make the bullet points aligned properly?

@wei-lee is that ok now?!

Identity Manager
Security
Push
Metrics

@danielpassos not really. If you look at other platforms, the text are aligned when there are multiple lines. But it is different on Android.

@wei-lee do you mean all the lines should start after the bullet point space?

@danielpassos yes

@wei-lee Is this what do you want?

Identity Manager
Security
Push
Metrics

@danielpassos yes 👍


Automatic page view tracking

@dhindrik  following on from my previous comment - https://github.com/dhindrik/TinyInsights.Maui/pull/14#issuecomment-2325211500
I've updated the code to use mauis built in WeakEventManager and that has the same issue as yours. On the first time returning to the MainPage it fires off the OnAppearing event, as it still has a subscriber -

But on subsequent returns the OnAppearing event is triggered, but the WeakEventManger doesn't have any subscribers -

Not really sure where to go from here tbh

Strange, seems like the Page object is handled by the GC, but in it can not be because it is possible to navigate back to ut. I mean when it still is in the navigation stack it should not be handled by the GC.
Will it be the same if you make the OnAppearing method static as in my example?
An option for auto-tracking can also be to use Application.Current.PageAppearing. Maybe that is the preferred way over a handler? At least it is a more simple solution.
In that case we can create an Initialize method that we call from the both constructors. SendCrashes and AddMetaData can slo be moved to that method.
public void Initialize()
{
     AddMetaData();
     Task.Run(SendCrashes);

     var weakHandler = new WeakEventHandler<Page>(OnAppearing);
    Application.Current.PageAppearing += weakHandler.Handler;
}

private static void OnAppearing(object sender, Page e)
{
 //TrackPageView
}

What do you think?
I also have to say that I really appreciate your work with the library!

@dhindrik  No prob, its a really cool project.
I've implemented you suggestion here, on a separate branch (just for testing) - https://github.com/IeuanWalker/TinyInsights.Maui/blob/6505e96d47d447ee975f30223febf8a0c4821f40/TinyInsights/ApplicationInsightsProvider.cs#L107C5-L111C6
The static OnAppearing method seems to work completely fine. Im just not sure how to call the  TrackPageViewAsync method from within the static method.
If I make it a non static method it never gets triggered

What do you think of adding a static field for the instance of the provider? The extension sets it as a singleton, so it should not be a problem.
private static ApplicationInsightsProvider instance;
It is ok to remove the ILogger stuff from the sample project, I also noticed that it makes it harder to debug. And I guess it is not the primary way to use the library. I am not using it in my apps. If some people want to read about it, they can read the docs.

@dhindrik Ye that works.
Think this PR is ready now, I've left ILogger on the sample project. The static IsInitialized variable i added, avoided any issue i was having with it.

Great work! Thank you very much.