E2e tests overhaul

This PR suggests a new approach to our e2e tests for the Hydrogen framework and showcases a few basic test-suites. This is a Proof of Concept, want feedback before devoting too much effort into it. While the broad characteristics of this overhaul are defined here and outlined below, there will be more methods and functions added as we move all of the e2e tests over.
Dynamic fixtures
Each app runs in an isolated sandbox that is created/removed before/after each test is run. This is different than our current set up, where all fixtures are checked into git and live in the packages/playground directory.

Static fixtures
Dynamic fixtures

Our current setup has been problematic because:

The packages folder should only contain actual packages from our monorepo and the additional files add unnecessary bloat. More files tends to lead to more confusion when trying to understand a single e2e test. This is solved with Dynamic Fixtures because these files will not exist in the repo at all unless a test is running or the {persist: true} config is given (this is for debugging only).
Almost impossible to know instinctively what e2e tests corresponds to which fixture. This is solved with Dynamic Fixtures because only relevant fixtures are generated in the same folder as the tests that use them.

Generating the fixtures dynamically also follows the pattern of other frameworks (next, remix, astro, etc...).
In-source testing for routes and components
An even further extension of the above point, many of our e2e tests are route or component specific and now we can define tests beneath the code they are testing. This is different than our current set up, where all the tests live in a single suite for each fixture and all the routes defined within that fixtures routes folder.

Our current setup has been problematic because:

It is difficult to understand what code an individual test is referring to. (there are dozens of routes). Defining a test in the same file as the source code makes this a no-brainer.

Additionally, tests can even share variables, etc... between the source and test. This also follows a trend in remix's Route Module API, where a route-level component has multiple exports (meta, links) etc... a tests export falls right inline with this way of collocating everything in a top-level route file.
Contained setup and teardown
We export our own describe function that defines each suite and takes care of the nitty setup/teardown of each test. This is different than our current set up, where we reference at least 4 different files for the building of code, starting servers, and  other tasks required for each e2e test suit to run properly.
The below files are not easy to reason about (this has come up in conversations with many on the team). The changes in this PR would remove all of them.

Our current setup has been problematic because:

It is very difficult to follow how the full environment is constructed. By keeping this logic contained to a single function block that sets up the different primitives and injects them into the suite we can more easily understand and debug the full e2e environment.

The one draw back to this approach is that we are essentially "wrapping" vitest primitives and it may become unclear where that line is drawn. I might suggest we go full in on this approach and essentially re-export * from vitest inside of our test-framework so that developer doesn't need to think about it. We could consider a "hydrogen/testing" package that sets this all up and a user test might look like:
import {describe, it, vi, beforeEach} from '@shopify/hydrogen-testing'

descibe('...', () => {
//...

Support for testing in multiple environments
Each suite runs in 3 environments: Node development using a vite dev server, Node production using our platform entry, and Worker production using our platform entry and worker runtime and we can opt-out of any of these at the suite level. This is different than our current set up, where we pass in a config to each test and use conditional blocks to return early if we don't want a test to run in a specific environment.
Our current setup has been problematic because:

Having control flow inside of a test is a bad practise
Passing a config into a separate shared file of test cases is indirect and makes tests harder to follow.

The above conditional logic can be omitted if we are able to declaratively tell a test suite how and where to run.
Vitest
We have been moving away from Jest entirely in our repo, in favour of Vitest. Not much to add on this point because it is a hard requirement for vite 3 among other reasons (ESM support, etc).
cc @pepicrft @lemonmade @heimidal
Running these new tests locally
Checkout this PR and run yarn test-e2e-new.

@cartogram Would it make sense to optionally pass the hydrogen.config.js path to describe? Or perhaps we should just have different projects when testing things like "async config"?

@cartogram Would it make sense to optionally pass the hydrogen.config.js path to describe? Or perhaps we should just have different projects when testing things like "async config"?

I think given this model, things like async config would be a different fixture personally.


cla-assistant.io displayed agreement differ from source-code (IDFGH-9589)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
Hello,
While trying to submit a PR, I've been asked by the CLA assistant to sign the Contributor Agreement.
However:

the text does not match what's found in the repository (https://github.com/espressif/esp-idf/blob/master/docs/en/contribute/contributor-agreement.rst), not even a previous version.
the rendering is not correct (e.g. code-blocks) and makes it difficult to read
the link  https://github.com/espressif/esp-idf/blob/master/CONTRIBUTING.rst is broken

It seems that the gist used by cla-assistant.io (https://gist.github.com/wujiangang/eb9de1d5fa93d2da6a21e16393780f46 ?) is out of date.
Screenshot:

@MarcFinetRtone this issue should be fixed now in CLA Assistant.
Additional small fixes should be synced with the version of agreement on master branch here very soon.


Compositions

Preview link: https://visualization-tool-git-feat-dashboards-1-ixt1.vercel.app/en?dataSource=Prod

Tab Layout is done, but other layouts are not yet implemented
Only one URL and the possibility to switch to the tab to have dedicated URLs the additional effort is estimated to be a bigger efffort: ca. 1-2days.


Slow Call to /events/counts

adam@Adams-MacBook-Pro-2 ~ > http https://api.counterparty.io:4000/v2/events/counts
HTTP/1.1 504 Gateway Timeout
Access-Control-Allow-Headers: *
Access-Control-Allow-Methods: *
Access-Control-Allow-Origin: *
Alt-Svc: clear
Content-Length: 14
content-type: text/plain
date: Thu, 07 Nov 2024 13:51:07 GMT
via: 1.1 google

stream timeout

adam@Adams-MacBook-Pro-2 ~ >

In my applications, I hacked around this by calling an endpoint in my app that returns a cached events count result which I keep fresh with a cron job running in the background:
/app/endpoint/event/counts
    public function getEventCounts()
    {
        $data = Cache::get('event_counts', []);

        return response()->json($data);
    }

And then I have a cron job that runs every 5-10 minutes:
    public function handle()
    {
        $response = Http::get('/v2/events/counts');

        if ($response->successful()) {
            $data = $response->json();

            Cache::forever('event_counts', $data);
        }
    }

Might help someone trying to use this endpoint which might never be that snappy.


⚠ Bofrost.lu has degraded performance

In f1b201a, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1383 ms

Resolved: Bofrost.lu performance has improved in f5e29ef after 8 minutes.


Automated cherry pick of #4724: wires: lift name length, charset constraint

Cherry pick of #4724 on release/2.14.
#4724: wires: lift name length, charset constraint

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.14@9020032). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff               @@
##             release/2.14   #4726   +/-   ##
==============================================
  Coverage                ?   6.86%
==============================================
  Files                   ?     523
  Lines                   ?   89069
  Branches                ?       0
==============================================
  Hits                    ?    6119
  Misses                  ?   82442
  Partials                ?     508

Flag
Coverage Δ

#aFlag
6.86% <ø> (?)

Impacted Files
Coverage Δ

pkg/compute/models/wires.go
1.52% <ø> (ø)

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 9020032...ee015f9. Read the comment docs.


Feature/oss nevermerge

Description of the Change

Benefits

Possible Drawbacks

Usage Examples or Tests [optional]

Alternate Designs [optional]

Created for OSS testing, no need to merge.


temp.zip file causing badZipFile.

The following is the error I got when I first tried to run the peekingduck run. When I look into the files, temp.zip is created but cannot be extracted as it is an empty zip file. I have tried to uninstall and reinstall but it doesn't seem to do the trick. May I get some guidance to solve this error? Thank You.

Hi @UltraRaptorYT, could you try the following:

Run peekingduck --version and let us know which version you are using
From your logs, it appears that you are using Python 3.9 - we currently are not supporting this version of Python yet. Could you try using Python 3.8 and see if it works?

I think there are some error when i tried to run the --version command. I try to use python 3.8 to run but it doesn't seem to work. I think it is due to my environment not working ah.

In your Python 3.8 environment, could you pip install peekingduck==1.0.3 and see if it works?

Hi @UltraRaptorYT, did it manage to work for you?
Thanks for raising this issue, we fixed it in #489 where PeekingDuck can only be installed when using Python 3.6, 3.7 or 3.8. This takes effect from peekingduck==1.1 onwards.

Hi @leeping-ng,
I managed to make it work in the Python 3.7.7 environment.
Thank you


npm-on-ipfs experiment: Error: spawn npm.cmd ENOENT

Please describe what you were doing when this error happened.
Specifications

OS: win32
IPFS Desktop Version: 0.11.2
Electron Version: 8.2.4
Chrome Version: 80.0.3987.165

Error
Error: spawn npm.cmd ENOENT
    at Process.ChildProcess._handle.onexit (internal/child_process.js:264:19)
    at onErrorNT (internal/child_process.js:456:16)
    at processTicksAndRejections (internal/process/task_queues.js:80:21)

The error suggests you are missing npm command.
Make sure you have Node.js and npm  (https://nodejs.org/en/) installed on your system and available on PATH (available in cmd.exe).
Let us know if that helped.


Create a slack - discourse oauth integration

https://github.com/4xposed/oauth-slack-discourse

If we're using Typeform or even Google Forms, we can also make it easier on us to just send a DM in Slack everytime someone signs up.
This would be easier to setup so less technical burden. 😄
For Typeform integration, see https://zapier.com/apps/slack/integrations/typeform and https://zapier.com/apps/slack/integrations/typeform/883/notify-your-team-on-slack-when-your-typeform-form-has-a-new-entry

@dvidsilva could you explain what this integration will do? 🤔

Still not sure what this entails - more details would be great! :smile:

@techqueria/website I think this should be closed until enough specs are gathered

@dvidsilva Will close for now as @fvntr mentioned - feel free to reopen with more deets tho. 👍


Generating tests references most recent commit, not relevant commit

In updating the leap exercise tests in #444 the example_gen.go script adds in the most recent x-common commit:
// Commit: cfdae36 Merge pull request #497 from exercism/stkent-patch-1

which has no relevance to the updated leap tests, which were changed in this commit
I don't know the test generation code well enough, or git well enough, to fix this currently, but will try and spend some time this week understanding it better.
In the meantime I wanted to check if this was known and deliberate or if someone else was easily able to fix if not?

How does the generator even know what commit it is? I'll read the code real quick...
oh, it just does exec.Command on git log.
Pass the filename of the JSON file into that git log command, and it should be all good.
Compare the effects of running these two commands in the x-common repo:
git log -1 --oneline
git log -1 --oneline exercises/leap/canonical-data.json

Should be a good fix to make, because we should keep the commit relevant.


Massive connection issues with the Codewars site.

I'm having massive issues with the Codewars website.
I keep getting timeouts, disconnects, and every network request takes at least 20 seconds to complete.

And once every 30 seconds or so, I get a popup with:
There was a server issue while sending the request. Please try again.

My own connection is fine, and work without issues on any other site.

Me too.

SOmething is wrong i encountered problems yesterday and same scenario tooday.

I'm experiencing the same issue as well. From time to time it comes back, but just for a minute or so.

The worst issues seem to be fixed, but I'm still having a very slow experience.
So I'm not this should be closed yet.

Still got 500 Server Error

5xx: Server Error , what's going on ?


feat(notched-outline): update to v1.1.1

related #697
NOTE: Text field and select goldens are not ok in this PR. I've checked in new "broken" goldens until they are fixed in their own subsequent PRs.

Codecov Report

Merging #806 into feat/mdcweb-typescript-update will decrease coverage by 69.24%.
The diff coverage is 82.14%.

@@                        Coverage Diff                         @@
##           feat/mdcweb-typescript-update     #806       +/-   ##
==================================================================
- Coverage                          93.87%   24.62%   -69.25%
==================================================================
  Files                                 73       73
  Lines                               3002     3021       +19
  Branches                             457      461        +4
==================================================================
- Hits                                2818      744     -2074
- Misses                                65     2275     +2210
+ Partials                             119        2      -117

Impacted Files
Coverage Δ

packages/select/index.tsx
21.87% <0%> (-77.09%)
:arrow_down:

packages/text-field/index.tsx
19.39% <0%> (-78.17%)
:arrow_down:

packages/notched-outline/index.tsx
93.84% <92%> (+2.35%)
:arrow_up:

packages/text-field/Input.tsx
8.19% <0%> (-88.53%)
:arrow_down:

packages/select/NativeControl.tsx
14.58% <0%> (-85.42%)
:arrow_down:

packages/linear-progress/index.tsx
16.88% <0%> (-83.12%)
:arrow_down:

packages/chips/ChipSet.tsx
16.86% <0%> (-81.93%)
:arrow_down:

packages/chips/Chip.tsx
12.38% <0%> (-80%)
:arrow_down:

packages/dialog/index.tsx
18.88% <0%> (-79.03%)
:arrow_down:

packages/top-app-bar/FixedAdjust.tsx
21.05% <0%> (-78.95%)
:arrow_down:

... and 55 more

Continue to review full report at Codecov.

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

Sorry for the disappearing, there was a crazy week for a project

No worries :) I appreciate all the work you've already done. I'm going to push this release back a week or two so that we can get all of mdc web v1.x. How does that sound?


Small recommendation in IoTDBConnection

Its not a bug but a suggestion.
In IoTDBConnection java file, can we make following changes to the code:

Replace throw new SQLException("Not support .......") to throw new SQLException("Does not support .......") or throw new SQLException("No support for .......") in the applicable methods

Hi, thank you, it is fine. Would you like to fix this?

Sure, will do this and upload the forked code

PR https://github.com/apache/iotdb/pull/2278
The PR needs to be maintained.

PR https://github.com/apache/iotdb/pull/2278
The PR needs to be maintained.


Add support for the new economy system

Add support for create wallets internally and do that if none are found


Feature: Segregate PROD and Non-PROD clusters

🔖 Feature description
Segregate and prominently mark PROD and Non-PROD clusters. (Any cluster with at least 1 environment marked as PROD should be marked as a PROD cluster)

[ ] Give a toast notification / or a confirmation prompt if a user selects a PROD cluster in Resource browser with a caution toast like "You have selected a PROD cluster". (Should be configurable using a CM variable FEATURE_TRIGGER_TOAST_NOTIFICATION_IF_PROD_CLUSTER: true )
[ ] Prominently display PRODUCTION on the top if a Production cluster is selected in Resource browser.

🎤 Pitch / Usecases
Users add both PRODUCTION as well as NON PRODUCTION clusters in Devtron. Sometimes, the names can be confusing and user with elevated access may edit manifests in production cluster instead of non-prod. To prevent this, PRODUCTION clusters should be prominently marked as PRODUCTION when in resource browser.
🔄 Alternative
No response
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

This is fixed


Algorithmic Final

Individual Coding:

[ ] GitHub Pages showing key learnings and challenges conquered from Tech Talks
[ ] GitHub commits you have made, show GitHub insights into your code commits.
[ ] Show Code running on Replit as well as IntelliJ
[ ] Menu review illustrating use of Lists and navigation: show runtime, code organization, code and comments
[ ] List and Loops: show runtime, code organization, code and comments
[ ] Class review and understanding: show runtime, code organization, code and comments

Crossover stuff

[ ] Show Crossover contribution:  Highlight work you did in crossover project: show their Replit runtime, code commit(s) in their GitHub repo

Extra

[ ] Show anything you have done to enhance what you have learned from Python code

https://user-images.githubusercontent.com/70926137/161674129-bb4f4256-d697-4683-bb53-ad080f9b4d58.mp4

9.5/10
Everything was included from the code and key learnings demonstrated understanding of Python code. Video was organized and well structured and led the viewer through what they learned successfully. Only thing that is missing is runtime of work added to crossover's code. Otherwise, everything looks very good.


Error with TestScreenshot app

When request capture

Debug: DXHookD3D11: PresentHook: Request Start
Debug: Disposing of hooks...
Debug: DXHookD3D11: PresentHook: Exeception: SharpDX.SharpDXException: SharpDX.SharpDXException: HRESULT: [0x80070057], Module: [General], ApiCode: [E_INVALIDARG/Invalid Arguments], Message: The parameter is incorrect.

   at SharpDX.Result.CheckError()
   at SharpDX.Direct3D11.Device.CreateTexture2D(Texture2DDescription& descRef, DataBox[] initialDataRef, Texture2D texture2DOut)
   at SharpDX.Direct3D11.Texture2D..ctor(Device device, Texture2DDescription description)
   at Capture.Hook.DXHookD3D11.EnsureResources(Device device, Texture2DDescription description, Rectangle captureRegion, ScreenshotRequest request) in C:\Users\Administrator\Downloads\Direct3DHook-master\Capture\Hook\DXHookD3D11.cs:line 280
   at Capture.Hook.DXHookD3D11.PresentHook(IntPtr swapChainPtr, Int32 syncInterval, PresentFlags flags) in C:\Users\Administrator\Downloads\Direct3DHook-master\Capture\Hook\DXHookD3D11.cs:line 355

Now, when i select Direct3D 9 instead AutoDetect, there is not error.
But, screenshot = null
Screenshot screenshot = _captureProcess.CaptureInterface.EndGetScreenshot(result)

What is the target you are capturing a screenshot from?
Do you know what Direct3D version it uses?

I think, it is answer
This application requires Direct3D 11 installed

Please, look error in first post!

Parameter is incorrect means that something is incompatible with what is taking place. Either format of texture or some device setting etc.
What application does it fail in?

Hmm it is failing to create the texture, so that could also mean that the texture description is incompatible with your 3D graphics card etc.
What graphics card are you using?

Does it work for another game? Also again, what game is it failing on?
FYI it is failing when creating the shared texture here. You could test this in a test SharpDX app if you have the time to see if something is incompatible here.
_resolvedRT = ToDispose(new Texture2D(device, new Texture2DDescription() {
    CpuAccessFlags = CpuAccessFlags.None,
    Format = description.Format, // for multisampled backbuffer, this must be same format
    Height = description.Height,
    Usage = ResourceUsage.Default,
    Width = description.Width,
    ArraySize = 1,
    SampleDescription = new SharpDX.DXGI.SampleDescription(1, 0), // Ensure single sample
    BindFlags = BindFlags.ShaderResource,
    MipLevels = 1,
    OptionFlags = ResourceOptionFlags.SharedKeyedmutex
}));

Yes, it works for Directx9 32-bit game.

Any idea, what parameters i can to change? I already tried height and width.

Try removing the last property. It will fail in code later on but worth checking if it gets past that one.

@hanrelan do you mean within Dota? Or just in anything?

If Dota is hooked then Dota crashes several times in a game. I haven't tested hooking against other games so I'm not sure if this is true within other games as well.

@hanrelan I've tried again with the recent changes I've made to the overlay engine and updated SharpDX version and cannot reproduce the crash on screen capture (or display overlay). Please test the latest commit.


Add escaping for SQL characters

Any input into pypika should escape invalid characters.  This includes table names, field names, and values.

Is there any workaround for this in the current version?

Not currently, except to escape the characters on the result query string itself

@twheys thanks for the suggestion! I think it would definitely help to have this supported. If it's fairly straightforward, I can also take a stab at it and make a PR if you'd like.

Yeah if you have time to do it, your contribution would be greatly appreciated 👍  There are some comments in different places in the code with TODOs where it the escaping should be applied

I can't make any promises at the moment; I'm no longer working on the project that requires this 🙈

No worries

@twheys How would you implement this? One way is to make sure all methods that receive inputs are validating it (escaping) - very prone to error on extension (someone may forget...)
The other method is to escape everything on get_sql (or the string creation).
Do you have a preferred way (possibly something else)?

There's a couple of places where it needs to be done. Basically, anywhere a parameter is taken that ends up rendered in the query. For example, table names and schemas, ValueWrapper values, etc. The escape function probably only needs to escape the quote characters, as long as what is being escaped is inside a string or something. Might prove a bit more complicated than that, though.

@twheys I may be able to spend time on it this weekend or the next one. Do you have any guidelines on how to do it?
Honestly, I think this is crucial for this library to be usable for any interaction with input from the real world.

@reutsharabani sorry I didn't reply before the weekend. I don't have a particular solution in mind. I think the problem might be more complicated that it first appears, though, perhaps not. My thoughts were to make an escape function that replaced any special characters such as ; in the arguments that need it.  I think the escaping should only need to be done on string arguments, not the whole get_sql output, otherwise you'll be escaping characters that the pypika internals are writing.

@twheys no problem.
I think a custom escape function is meaningless. The functions are usually exposed by the drivers so just take it as an argument. I strongly suggest it to be at a single point that everything passes through, Otherwise it is hard to maintain (scattered around the code).
About escaping pypika internals - my idea was applying it at a single point before doing something with the values. If the values are mutated before that point it obviously won't work.
I will probably take a look at the weekend to get familiar with the code and see if I come up with something.


Saving of individual position create prices

Summary by CodeRabbit

New Features

Added new fields createPriceLong and createPriceShort to the market Position type, enhancing tracking of price metrics for long and short positions.

Improvements

Enhanced event processing for orders to include additional logic for handling and updating positions based on order types and conditions.

@coderabbitai review

@coderabbitai summary


Compile error with CONFIG_DEBUG_PWM_INFO

In https://github.com/apache/incubator-nuttx/blob/master/arch/arm/src/stm32h7/stm32_pwm.c, the pwm_dumpregs function's definition and declaration are not matched.

@imchangchang could you provide a patch?

@imchangchang could you provide a patch?

Have created a PR for this.
https://github.com/apache/incubator-nuttx/pull/4088.


The bulk insert query doesn't work if it's in multiple lines

Description:
The connector is expecting the query to be in a single line. If the query is in multiple lines then there's no exceptions. However the connector inserts empty data to the database. All the column values are empty. The query can span multiple lines because when we use the default formatter to format the file it breaks longer lines into shorter lines with newlines.
Suggested Labels:

Suggested Assignees:

Affected Product Version:
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:

Moved to https://github.com/wso2/micro-integrator/issues/3518


separate sign + send txn methods

consider doing something like the below where we only sign the txn(s) in the incoming array if we have access to that account's sk.
					// do we have access to the required signer?
					if ((txn.transactionSigner as AlgosdkAccount).addr == this.account?.addr) {
						signedTx = signTransaction(txnGroup[i], this.account.sk);
					}
				}

				if (signedTx) {
					signed.push(signedTx.blob);
				}

i added a .signTransaction() method that does NOT submit it to the network here:
bee71c618dec373c0ba12281d21bf7306ecb64ae

resolved in PR
https://github.com/thencc/algonautjs/pull/71


pull request for e2e test at 11/14/2016 01:45:10

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

Details

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[BUG] scoped keyword in a vue.js SFC component style section does not produce html data-v-* IDs to the html elements

Bug Report Quick Checklist

[x] I am on the latest version of Snowpack & all plugins.
[x] I use package manager npm (Fill in: npm, yarn, pnpm, etc).
[x] I run Snowpack on OS Windows 10 (Fill in: Windows, Mac, Linux, etc).
[x] I run Snowpack on Node.js v14+

Describe the bug
Using the latest snowpack with the vue 3.0.8 and newer produces the bug, in which scoped styles in a SFC does not generate data-v-* IDs into the html and therefore the styles are not applied.
To Reproduce

npx create-snowpack-app snowpack-scoped-bug --template @snowpack/app-template-vue-typescript
go into snowpack-scoped-bug/src
edit the App.vue file with an editor
add scoped keyword into the style section, turning <style> into <style scoped>
from the application root start the app with npm run start
see the styles are not applied to the pulsating logo
in a developer console it can be seen the IDs data-v-* are not added to the html elements

Expected behavior
The IDs data-v-* should be added to the html elements and from that the styles should be applied.
If using the create-snowpack-app example above the logo and int styles should be the same when scoped keyword is provided or not.
Anything else?
Up to the vue version 3.0.7 the bug is not present. Also with non-scoped styles the bug does not manifest itself.

also getting this exact same error

just in case this is blocking for anyone else until my PR gets merged, i published the fix as @kevincennis/snowpack-plugin-vue.

Thanks @kevincennis for the fix and PR. With your changes the snowpack works with vue version 3.1.2 as expected.


Fix typo

battches -> batches

I signed it!
On Wed, Apr 3, 2019 at 3:44 PM googlebot<EMAIL_ADDRESS>wrote:

Thanks for your pull request. It looks like this may be your first
contribution to a Google open source project (if not, look below for help).
Before we can look at your pull request, you'll need to sign a Contributor
License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/
https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here (e.g. I
signed it!) and we'll verify it.
What to do if you already signed the CLA Individual signers

It's possible we don't have your GitHub username or you're using a
different email address on your commit. Check your existing CLA data
https://cla.developers.google.com/clas and verify that your email is
set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.

Corporate signers

Your company has a Point of Contact who decides which employees are
authorized to participate. Ask your POC to be added to the group of
authorized contributors. If you don't know who your Point of Contact is,
direct the Google project maintainer to go/cla#troubleshoot (Public
version https://opensource.google.com/docs/cla/#troubleshoot).
The email used to register you as an authorized contributor must be
the email used for the Git commit. Check your existing CLA data
https://cla.developers.google.com/clas and verify that your email is
set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.
The email used to register you as an authorized contributor must
also be attached to your GitHub account
https://github.com/settings/emails.

ℹ Googlers: Go here
https://goto.google.com/prinfo/https%3A%2F%2Fgithub.com%2Fgoogle-research%2Fbert%2Fpull%2F543
for more info.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google-research/bert/pull/543#issuecomment-479495252,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AdDuaygoWFQ1ahGIOOlYu52mCDHVSjPCks5vdLBSgaJpZM4cajVa
.


Keyword I

Conflicts

This PR was included in a batch that successfully built, but then failed to merge into main (it was a non-fast-forward update). It will be automatically retried.

??

bors r-


feat: implement CRUD plugin

Refs: https://github.com/metarhia/impress/issues/1588
Refs: https://github.com/metarhia/impress/issues/1589

[x] tests and linter show no problems (npm t)
[x] code is properly formatted (npm run fmt)
[x] description of changes is added in CHANGELOG.md

Landed in https://github.com/metarhia/metasql/commit/e45e3fa44ab15b8d03a0d25923c5ee6df6d0de13


WIP: port BDD benchmark from OCaml's Sandmark benchmark suite

Still WIP.
Ports BDD benchmark from https://github.com/ocaml-bench/sandmark/tree/main.
Will finalize some adjustments later.

This creates a fairly small heap and its single threads. Closing it to focus on porting the multicore benchmarks.


Sliding Window Problem (Maximum Sum Subarray of Fixed Size K) in c++

vyasisha
Silding Window Problem
14 Oct, 2023

pls assign me this issue

@vyasisha there you go!

Thank you @HarshwardhanPatil07


Checkin Tools should strip git-tfs-id: entries OR stashes should not include it.

We use stashes for code reviews.  If you pull a stash, it adds a git-tfs-id: entry at the bottom of the commit message (with a negative number, C-1).
Either restoring a stash should eschew this entry, or it should be stripped when executing the checkin tool.

I am trying to clean up old/stale issues. As this issue hasn't seen any new information or pull requests implementing it, it is safe to assume it will not be solved. Pls close it. In case new information is provided, this issue can be reopened.

No planned work on this issue...


Not reading my calendar

This happens on my pixel 3a

The same issue happened on my pixel2 XL.

I've the same issue with Redmi Note 7.
It reads all my calendars but it show them with 0 events.

In my case, it actually reads only one of my calendars, but still no events. Though, I have four calendars lol.


plutus-playground: Fix the display of contract instance log messages

Instead of showing the contract instance tag and the message, it showed the messag twice

Pre-submit checklist:

Branch

[ ] Commit sequence broadly makes sense
[ ] Key commits have useful messages
[ ] Relevant tickets are mentioned in commit messages

PR

[x] Self-reviewed the diff
[x] Useful pull request description
[x] Reviewer requested

If you updated any cabal files or added Haskell packages:

[ ] $(nix-build default.nix -A plutus.updateMaterialized) to update the materialized Nix files
[ ] Update hie-*.yaml files if needed

If you changed any Haskell files:

[ ] $(nix-shell shell.nix --run fix-stylish-haskell) to fix any formatting issues

If you changed any Purescript files:

[x] $(nix-shell shell.nix --run fix-purty) to fix any formatting issues

Pre-merge checklist:

[ ] Someone approved it
[ ] Commits have useful messages
[ ] Review clarifications made it into the code
[ ] History is moderately tidy; or going to squash-merge


Add fields to support AM and Orchestrator functionality in Slice Models (ASMs)

Need to add a ReservationInfo property to graphs and to Network and Link slivers that consists of

ReservationID and ReservationState
This is used by Orchestrator on ASMs to pass information back to the user. Orchestrator populates these.

Also need to add OrchestrationInfo to ASMs - an opaque JSON dictionary set by Orchestrator to send information to AM.
Also need to add ProvisioningInfo to ASMs - an opaque JSON dictionary set by AM for its own needs as well as to send back to Orchestrator and user as debugging information.
The contents of both dictionaries is specific to the AM type.
For VM-provisioning AMs orchestrator passes in worker node, AM passes back instance id, other instance information.
For Network provisioning AM orchestrator passes in information needed by NSO for specific services and their endpoints. Network AM fills in additional information as needed.

Correction to above:
What we need is 4 groups of Labels and Capacities

Available - currently encoded in 'Labels' and 'Capacities' (part of ARM, ADM)
Delegated - currently encoded in CapacityDelegations, LabelDelegations (ADM)
Allocated - needs to be added for Orchestrator to specify what is allocated (ASM)
Provisioned - need to be added for AM to specify what is provisioned (ASM)

The ReservationInfo still stands as described above.

Implementation now includes 3 classes of labels and capacities: what is - under 'Labels' and 'Capacities' - can be what is requested (from user to orchestrator) or what is provisioned (from AM to orchestrator and user). Delegated - under 'LabelDelegations' and 'CapacityDelegations' properties (from AM to Broker). And Allocated - 'LabelAllocations' and 'CapacityAllocations' (from Orchestrator to AM).
ReservationInfo also added under BaseSliver as a property (with setters and getters) (there is a class ReservationInfo under fim.slivers.capacities_labels


Clone fails due to Unicode character?

I am trying to clone PTVS with the VS2015RC Github integration ("Open in Visual Studio").
However I get a failure to clone:
Error encountered while cloning the remote repository: Failed to stat file 'C:/Github/PTVS/Python/Tests/TestData/UnicodePath�.sln': No mapping for the Unicode character exists in the target multi-byte code page.

We may have to change our tests to generate this file while running if git doesn't know how to handle Unicode properly.

git clone in command line works fine, but the files will then show up as if they were renamed.

That's not a unicode character, that's a Latin-1 character:
git ls-files --stage | grep -i UnicodePath.*.sln
100644 7411333d2102a578025ea00306ab543f7992eee8 0     "Python/Tests/TestData/UnicodePath\344.sln"

Octal 344 is hex 0xe4, which is Latin-1 for ä.
Git repositories store filenames as UTF-8 internally.  If you want this to be ä (U+00e4), then this should be 0xc3 0xa4 in UTF-8.
If you know how this got into your repository, then I would recommend filing a bug against that tool vendor.  In the meantime, please rename this and commit the new name.  While you're doing that, the folder Unicode\344 is incorrect as well.


Remove assertIsPromise() from PlainResult/unwrapOrElseAsync

We close this once by https://github.com/option-t/option-t/issues/1608#issuecomment-1456023611.
We'll reopen this when we conclude about #1608.
If you have some opinions, please comment to #1608


ecto 3 and deps

Pull Request Test Coverage Report for Build 61

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 97.452%

Totals

Change from base Build 59:
0.0%

Covered Lines:
153

Relevant Lines:
157

💛  - Coveralls


✨ Auto font color

Closes #38.

Build succeeded!
Built with commit 8eab2f14f48a958fbf48a9776cacfad230f50bb8
https://deploy-preview-42--padpad.netlify.com


[eas-cli] Add update delete command

Why
See ENG-251
How

Added command update:delete
Used mutation update.deleteUpdate
Added additional info why users might want branch:republish instead (as mentionedin Linear)
Added --force flag, mostly for JSON output but also a way to delete something with prior confirmation.

Examples
Running $ eas update:delete and aborting

Running $ eas update:delete abc-123 --json

Running $ eas update:delete and confirming

Test Plan

$ eas update:delete <id>
$ eas update:delete <id> --json
$ eas update:delete <id> --json --force

Nice! ✨
One issue that I'm thinking about here is that we delete by updateId but republish by updateGroup.
Would be curious to get @jonsamp 's view on this.

I think making mutations by update groups is the way to go for all things, and if you want to mutate specific updates inside of an update group, you'd need to specify more arguments.
What are the steps before this? How can I get a updateId and how would I (as a developer) have come across this?
My assumption is that a developer creates an update with a secret key or something and they want to delete it. So, since they used eas branch:publish, they now have an update group with multiple platform specific updates with that secret. So, then, they'd run  eas update:list (or eas branch:view ...), get the most recent update group, then say eas update:delete (or similar). Also, maybe what I am describing is just another command and this is a more specific command. We could have both:

The ability to delete an individual update (less common, and should be harder)
The ability to delete an update group (more common, and should be easier)

Overall, I like the UX of this command though. Nice work!

Alright, so atm this command deletes a single update. We can switch it by deleting an update group instead, so all update:* commands are querying and mutating the update group?

closing this as it was replaced by #332


[#143876117] Tag instances

What
Currently, a DB instance name managed by the broker is a plain UUID. That makes it harder for humans to recognise in Compose UI which instances are automatically provisioned by the broker. To make things a little bit easier this PR implements a DB prefix that will help to recognise which DBs are broker made.
Also - small improvement in tests: as we decided to not support user provided parameters we can test if we ignore them.
How to review

sanity check
run tests and check if you can recognise a test instance in the UI 😈
deploy together with (PR here)
tag master branch v0.1.0 after merging

Who
Not @combor

Confirmed with tests and manual testing through CF 👍
I've created the tag. I signed it, rather than using an annotated or lightweight tag, because I couldn't see any reason not to:
https://github.com/alphagov/paas-compose-broker/releases/tag/v0.1.0


Please mention our open source solution on your website

Hi,
we are currently building an open source solution for PVS systems and maybe pharmacies to be able to use the API from the certification service:
https://github.com/ere-health/covid-19-green-certificate
The toughest implementation detail is the OpenID Connect dance using the SMC-B. This is implemented here:
https://github.com/ere-health/ere-ps-app/blob/feature/covid-19-green-digital-certificates/src/main/java/health/ere/ps/service/idp/IdPService.java
Please mention us on your pages to get this out as fast as possible.
/Manuel

Hi, thanks for your feedback! The souce code is not shown on the website, as the open source can be viewed in GitHub. Best regards!


Tippecanoe consistently freezes up with complex geometry

We have a large national parks GeoJSON dataset with large, complex geometry (the Appalachian trail for example). Tippecanoe consistently freezes up around ~23% whenever that dataset is included. I've tried to dive into the code to figure out where this is happening, but I'm not sure how to debug that.
Any ideas?

@nsamala Can you post a link to the failing data, so we can make a test case?

@nsamala What version of tippecanoe are you using? There were some bad performance problems with polygons that were fixed in https://github.com/mapbox/tippecanoe/issues/121, so I hope that may help with your dataset too.

@ericfischer I'm using 1.4.1. I'll pull down changes to see if anything has changed. @morganherlocker Lemme check with the office peeps to see if it's cool to link to the data.

https://github.com/kgjenkins/ophz/tree/master/shp is another good example of a giant, complex polygon that is very slow to tile.

Sorry for the delay on the file! You can check it out here.
I tried running line simplification on it which halved the file size. Tippecanoe was able to process up to 50% of the resultant file.

Thanks @nsamala! I'll give it a try in a moment.
I wonder if the right thing to do is to do fast, simple clipping for the purposes of tiling, and to only bring Clipper's careful work into it as a final output stage in each tile after scaling and line simplification.

The plant hardiness shapefile does eventually complete, after 46m34.026s of CPU time, but the low zooms are very slow.

@ericfischer I think that would be a good idea to use fast, simple clipping for tiling.
I'll probably dive into the code for good today, so I apologize if this is already addressed: maybe you could write out temp files by tile level and then utilize child threads to process each individual tile in further detail and add it to the MBTile.

@nsamala Versions since 3fbd0d654695a01ec61 do already use multiple threads while tiling, so unfortunately the problem can't be masked just by spreading it over more CPUs. (There may be a better way to do the scheduling, though, since it doesn't necessarily know which tiles will be the slow ones at the time that it partitions out the tasks.)
The low zooms feel faster with simple clipping, but apparently it actually hurt in practice: 55m54.858s of CPU.
So a new idea: maybe I can run everything through Clipper once at z0 to clean it up, using all the CPUs that would otherwise go to waste at that zoom, and then at the higher zooms do a cheap clip and then another final cleanup.

Although there are big up-front costs to that idea. It has been cleaning polygons for at least 10 minutes and is still going. Still to be seen whether that will pay off in faster processing of the higher zooms.

Is it just because of the sheer number of points in the polygon that makes this difficult to process?
And another update: I ran an even more aggressive line simplification algorithm and I was able to get it to process the data, but obviously that's not a great solution for some use cases.

@nsamala Yes, that's what it looks like. I'm not entirely sure how to characterize the performance, but essentially it looks like performance is predictably good up to about 700 sides on a polygon, and then blows up badly for polygons with tens of thousands of sides.

So I should probably dynamically simplify each polygon down to some reasonable number of sides, independent of the tile resolution, to keep performance good.

Geometric simplification of polygons doesn't seem to help much: it's still 45 minutes of CPU. So maybe the problem is topological complexity with lots of holes, instead?

That's possible. I used Mapshaper's weighted Visvalingam algorithm which is doing some sort of black magic. After that, Tippecanoe was able to rip through and present geometry that looked pretty close to the original.

I'll have to try switching simplification algorithms and see if that helps.
For the record, the whole thing without using Clipper is 7m40.979s. There's got to be some way to guarantee clean polygon geometries without spending 6x the time.

My new plan for this is just to cut up complex polygons into smaller features until each of them has few enough vertices to be tractable, rather than hoping to simplify the complexity away.

Working on this in https://github.com/mapbox/tippecanoe/pull/170

Been messing around with the geojson-vt project over the past couple days. Would it work to adopt a divide-and-conquer strategy similar to that for generating tiles at higher zoom levels?

What I am doing in https://github.com/mapbox/tippecanoe/pull/170 basically is dividing and conquering: cutting the polygon in half horizontally or vertically until it is simple enough. Is that what you had in mind?

Pretty much, it divides each tile and uses that tile's subset of the geometry to do tiling at the next level down.
You might even be able to leverage the C++ port of geojson-vt. I'm not sure whether or not that's more or less effort.

That is the same thing tippecanoe does in general for creating each successive zoom level. Lots of details are different because geojson-vt works entirely in memory while tippecanoe is optimized for data that is too large to consider all at once.

(and geojson-vt is optimized for tiling only the data that you need to display what is on screen at any given moment, while tippecanoe is optimized for creating the entire tileset at once.)

I just merged these changes and will be interested to know if your Appalachian Trail data now also tiles in a reasonable amount of time.

Ahh gotcha, yeah that makes sense. I know there are ways to force geojson-vt to render everything, but it does happen in memory. Thanks for the clarification.

Sweet, I'll give it a shot!


New toolbar Added

Hey @aniketsen123
Issue Resolved - #38
I added the New attractive toolbar in the application.
Kindly check and merge it accordingly under JWOC.

@aniketsen123
The conflicts are resolved, now you can merge this.


Fix No such autocomplete provider: 'history'

In Firefox 49, the 'history' component was removed, so that uses
'unifiedcomplete' here instead.
More to see: https://bugzil.la/1223728

When this pr and #178 will be merged?
If you need tests I'm using it on different OS and so far it works.
Thanks!

Thanks.


[Big] Certain Lemmy instances seem not allowed for no reason

I've been adding communities and I noticed that for some reason some Lemmy instances are displaying "Not allowed" despite being federated with the instance and the community not being NSFW in nature. Any idea what's going on? I tried to add<EMAIL_ADDRESS>and recently<EMAIL_ADDRESS>and both seem to not be allowed by lemmy.world. I checked and was able to search for them on Lemmy.world so I don't think federation is the issue here.

Here’s the admin page:

So there are other filters too. Also lemmy.world using explicit allow list as far as I know.

Ah I see. I thought it might've been a bug of some kind. I'll ask them about why it might be happening.


Thumbs Up Thumbs Down for the Page

Some feedback mechanism so that users can either upvote or downvote a page if it is helpful or not
We should have this saved somewhere where we can see the results and assess which pages need the most attention

@brandenchan do you have the infrastructure for this already? I think e.g. an on-demand DynamoDB instance could be a good fit for the job here. I can create a simple API endpoint within the Next project to handle this behaviour and display it if you can get me access to a Dynamo instance or we could create a small Lambda on AWS that handles that. Authorization-wise I'd prefer the latter approach.
On the frontend this is a matter of ~2-4h.


How can we use the CLIP4IDC on custom data

Hello,
Thanks for this amazing work.
We are trying to run your project on custom short videos (or set of image pairs) in our lab, but we can't figure out how to do this  in the documentation (to get a caption of the images' difference).
Any help will be appreciated.

Hi, thanks for your attention. We focuses on the single difference between two images. For how to process video frames, maybe it is recommended to refer to video based methods, e.g. clip4clip, swinbert and vid2seq.

Hi, many thanks for your reply!
Ok, can you tell me the command to use in order to test CLIP4IDC on two custom images.

Hi, please refer to this script and the --init_model is the path where the pre-trained CLEVR model is saved. For testing on the Spot dataset, please change --datatype to spot and set the --init_model to the path where the pre-trained SPOT model is saved.

Hi @sushizixin  Thanks for the great work. May I know how to test the model on images other than CLEVR and SPOT?

Hi, we also conducted the experiment on the Image Editing Request dataset. But we use another code base that has not been integrated into this repository. To test the model on the other images, you may need to process the data according to your requirements and train the model in two steps, following the training phase of CLEVR or SPOT.

Hi, thank you for the interesting paper! I think that it'be really awesome if you would release the IER pre-trained model and possibly(?) a simple script to test it with custom images. As far as I can see, there's no available pre-trained, out-of-the-shelf IDC model out there, and there would be so many potential applications and users (including myself)!


peers count

In header there is a number of active peers. Currently it's just 10, hardcoded value. Read & show actual value from RCP API.

Resolved in #146 696f557, or?

Ah, it seems I need to do rebase


Rename CustomParameters into CustomTestParameters, enable CustomParameters for the parameters.

Rename the current CustomParameters to CustomTestParameters, which is for overwriting the default parameter values defined in ReplaceableTestParameters.xml
And enable CustomParameters for the parameters, the value format is the same as “Key1=Value1;Key2=Value2;…”, which can include TiPCluster=;TipSession=; DiskType=Managed;Networking=SRIOV

After the PR is approved, need update the pipeline for the parameters change,

@lizzha Please review again, I will go through pipeline for this PR and #697

@LiliDeng could you run test for both the -CustomParameter and -CustomTestParameter, and post test results?

@lizzha Let know if need run more testing.

Done for update linuxpipeline Jenkins site, https://github.com/LIS/lis-pipeline/pull/420 PR created

Done for penguinator Jenkins site

LIS/lis-pipeline#420 depends on this PR and PR 697. They should be merged together.
We're ready for the merge from Lili's team.
@chvalean @ader1990 @iamshital @juhlee-microsoft please ack whether your pipelines are prepared for these change, and help to do the merge once ready.
Thanks!

I already update jenkins jobs and pipelines for #697 and mine, I will resolve the conflict once #697 merged

LIS/lis-pipeline#420 depends on this PR and PR 697. They should be merged together.
We're ready for the merge from Lili's team.
@chvalean @ader1990 @iamshital @juhlee-microsoft please ack whether your pipelines are prepared for these change, and help to do the merge once ready.
Thanks!

@LiliDeng @lizzha I am ok with this change. @chvalean is out of office today, but ask to update all jobs.
@lizzha how soon do you want to complete the pipeline update? Please set the date. I sign off and @chvalean  can merge with his ack.

Please expected some break job in linuxpipeline since I already update them to align with this change and #697


Fix minor link formatting and punctuation issues

Accidentally closed the other one, sorry.  It looks like I'm behind again - feel free to merge this manually or cherry pick or what have you.

This works great, thank!


Use excluded counter is broken

Need to increment counter before checking location status, I assume

More details: use excluded is completely inaccessible at the moment.
The UseExcluded counter counts all allowed and excluded artifacts. The button is only active if this counter is >0. Since it only counts allowed artifacts, if the button is disabled, then the counter will always be 0. Which means the button will always be disabled


Axis Auto fails to scale plots on the secondary axis

Bug Report
Issue: (Axis Auto fails to scale plots on the secondary axis)
The AxisAuto method does not scale any plots on the secondary axis. Pressing the mouse scroll-wheel button also does not scale any plots on the secondary axis. This was working fine in the previous ScottPlot version.

System Details

ScottPlot Version: 4.1.27
Operating System: (Windows 10)
Application Type: (WPF)
.NET Version: (NET Framework 4.7.2)

Try cleaning the solution in visual studio and then rebuild the solution.

I just compiled the ScottPlot Demo (WPF, 4.1.28) from the source code, the bug is still present.

I identified the problem and fixed it with #1498 - Thanks for reporting it @Niravk1997! It will be fixed in the next release 👍


Nested navigators in redux with components in between

Hi, I'm trying to set up my navigators which look like this:
const AppNavigator = TabNavigator({
  User: { screen: UserScreen },
});

const AppWithNavigationState = ({ dispatch, navigation }) => (
  <AppNavigator
    navigation={addNavigationHelpers({ dispatch, state: navigation })}
  />
);

const mapStateToProps = state => ({
  navigation: state.navigation,
});

export default connect(mapStateToProps)(AppWithNavigationState);

// reducer

const initialNavState = AppNavigator.router.getStateForAction('Channel');

class NavigationReducer {
  static reduce(state = initialNavState, action) {
    const nextState = AppNavigator.router.getStateForAction(action, state);
    return nextState || state;
  }
}

export default NavigationReducer.reduce;

const UserScreen = ({ dispatch }) => (
  <View>
    <Text>User screen</Text>
    <GroupNavigator />
  </View>
);

export default connect()(UserScreen);

const GroupNavigator = StackNavigator({
  GroupList: { screen: GroupList },
  GroupDetail: { screen: GroupDetail },
}, {
  headerMode: 'none',
  initialRouteName: 'GroupList',
});

export default GroupNavigator;

Now, I understand that if I nest navigators directly as screens of other navigators, this works automatically, and the state is correctly stored under a single key in my store, but when there are components in between, like UserScreen here, I don't understand how to set this up. Should I create a whole new reducer for this?
Perhaps the docs could be improved to show what to do in this case.

Same problem here. The initial state is not populated with nested routes in these cases.
I tried passing all received props down to the nested navigator e.g. <GroupNavigator {...this.props} /> but  errored with:
Invariant Violation: This navigator has both navigation and container props, so it is unclear if it should own its own state. Remove props: "getInitialDevicesData, cancelDataSync, startDataSync" if the navigator should get its state from the navigation prop. If the navigator should maintain its own state, do not pass a navigation prop.
The props mentioned are the actions creators I need the screen to call on mount and unmount. If I avoid passing them down to the navigator I get:
Cannot read property 'find' of undefined at DrawerView._this._updateScreenNavigation
If I nest the navigator directly, the nav state tree populates correctly but I'm unable to call the actions I need to when the screen mounts and unmounts.
Sorry to bump thread, not looking for specific answers to my case. Just what the right approach is for nested navigators or, more specifically, non-directly nested navigators..

Hi! In an effort to get the hundreds of issues on this repo under control I'm closing issues tagged as questions. Please don't take this personally - it's simply something we need to do to get the issues to a manageable point. If you still have a question I encourage you to re-read the docs, ask on StackOverflow, or ask on the #react-navigation Reactiflux channel. Thank you!

docs writes nothing about this.
I have same problem.

same problem here

same problem here

@sven0726 @alexeymolchan @Luomusha check this section of the README, in particular, [this article](React-Navigation, complete Redux state management, tab-bar, and multiple navigators).

Same problem here. I've been trying to solve this with Higher-order components. I'll report back if I have any luck. This is a pretty common use-case I imagine.

@alex-gunning - can you create a new issue that reproduces this problem with a minimal, runnable project on snack or github? thanks!


Add Double and Int Convenience Properties

Building on the work of @ahoppen on SR-11580, this PR adds convenience properties to get Double and Int values from FloatLiteralExprSyntax and IntegerLiteralExprSyntax, respectively.
The two new properties, floatingValue and intValue, use failable initializers of Double and Int: init?<S>(_ text: S) where S : StringProtocol and init?<S>(_ text: S, radix: Int) where S : StringProtocol. These initializers fail if the Strings passed in do not represent valid Doubles or Ints. The question arises: what should floatingValue and intValue return, if anything, if initialization fails? I considered the following possibilities, ultimately choosing the fourth.

floatingValue could return Double.nan. This would make sense because, for example, "🍕.🌍" is truly not a number, floating-point or otherwise. The problem with this approach is that there is no corresponding Int.nan, which would prevent floatingValue and intValue from returning the same sort of value given invalid input.  Decimal does have a nan value, so if intValue returned a Decimal rather than an Int, that would solve the problem, but I, as a consumer of the API, would find this counter-intuitive because "42" in a source file represents an Int, not a Decimal.

The return types of floatingValue and intValue could be optional. If Double or Int initialization fails, floatingValue and intValue could therefore return nil. I preferred not to impose on API consumers the burden of unwrapping.

The return types of floatingValue and intValue could be Result<Double, LiteralError> and Result<Int, LiteralError>, respectively. If Double or Int initialization fails, floatingValue and intValue could return .failure(.invalid). I preferred not to impose on API consumers the burden of switching on the returned value.

The result of the Double or Int initialization can be force-unwrapped. This causes a crash if initialization fails. Crashing is considered harmful, though not by everyone. Force-unwrapping is the most convenient option for API consumers, which is why I chose it. I observe that force-unwrapping does appear elsewhere in SwiftSyntax, for example here, here, here, and here, and I suspect that the authors of this code made similar judgments about the convenience of force-unwrapping versus the crashing risk it imposes.

@vermont42 thanks for working on this! I would be fine with #4 (force-unwrap) since it can be considered a violation of tree invariants if an IntegerLiteralExprSyntax node doesn't actually contain an integer.
However, if we do force-unwrap we need to protect against the tree getting into such state in the first place. I think we can safely assume that the parser will not provide such a node (and if it does that's a parser bug), but another source of these are the APIs for creating syntax nodes programmatically. It is important to catch invariant breakage at the earliest source, so we should prevent a case of manually creating a IntegerLiteralExprSyntax node with a string that is not an integer.
That may require changing the python code on the swift repo side that does the code-generation for the programmatic APIs. e.g. maybe add a flag to indicate that the programmatic API for a node will be manually written on the SwiftSyntax side and avoid code-generating, then we write the functions manually and include the validation aspect.

However, if we do force-unwrap we need to protect against the tree getting into such state in the first place. I think we can safely assume that the parser will not provide such a node (and if it does that's a parser bug), but another source of these are the APIs for creating syntax nodes programmatically. It is important to catch invariant breakage at the earliest source, so we should prevent a case of manually creating a IntegerLiteralExprSyntax node with a string that is not an integer.

@akyrtzi I’ve just been digging around the generated code to give @vermont42 some advice on what needs to be modified. While doing so I have been thinking what the desired API would be. I would argue that making all methods that construct a integer/float literal return an Optional would be the cleanest solution, even though it means that these are the only types with a failable initialiser. What do you think?

I would argue that making all methods that construct a integer/float literal return an Optional would be the cleanest solution, even though it means that these are the only types with a failable initialiser. What do you think?

That makes sense to me 👍

Update
I have partially implemented the suggestion of upholding the invariant. Specifically, I modified the public init?(_ syntax: Syntax) { initializer to validate but am still figuring out how to modify the internal init(_ data: SyntaxData) { initializer.
The invariant code relies on the following non-breaking code changes in the main Swift repo. Do reviewers agree that, after this SwiftSyntax PR is good to go, I should PR the main Swift repo with the Python changes, wait for that to land, and then request that the SwiftSyntax PR be merged?
ExprNodes.py:
    Node('FloatLiteralExpr', kind='Expr',
         children=[
             Child('FloatingDigits', kind='FloatingLiteralToken'),
         ],
         must_uphold_invariant=True),     // NEW

    Node('IntegerLiteralExpr', kind='Expr',
         children=[
             Child('Digits', kind='IntegerLiteralToken'),
         ],
         must_uphold_invariant=True),     // NEW

Node.py:
class Node(object):
   [omitted]
    def __init__(self, name, description=None, kind=None, traits=None,
                 children=None, element=None, element_name=None,
                 element_choices=None, omit_when_empty=False,
                 must_uphold_invariant=False):     // NEW
       [omitted]
        self.must_uphold_invariant = must_uphold_invariant     // NEW

Update
I pushed a commit causing the internal init(_ data: SyntaxData) { initializer to perform validation and be failable.

I implemented @ahoppen's comment suggestions but will hold off on implementing the other suggestions in order to give @rintaro and @akyrtzi the opportunity to respond.

@ahoppen I removed the two setters and converted them into throwing functions, preferring that approach over converting all setters to throwing functions, notwithstanding the inconsistency, because throwing functions are less pleasant for callers to use than garden-variety setters and are therefore to be avoided when possible. I think some inconsistency between, on the one hand,FloatLiteralExprSyntax and IntegerLiteralExprSyntax, and, on the other, everything else is actually warranted because of the constraints on validity to which these two types, but not others, are subject. I also added the test you mentioned and updated the changelog.
I am unclear on what, if anything, still needs to be done about Blanks. I can either leave the current approach of not generating them or restore them and return nil.

@ahoppen I squashed all but my first commit, which I reworded to Implements SR-11580. I'm not sure why Doug's commit appears in my commit history because I left that as pick.

There were some conflicts with the the main repo's master branch, so I resolved those.
Please let me know if further git actions should be taken. 😅

@ahoppen I squashed all but my first commit, which I reworded to Implements SR-11580. I'm not sure why Doug's commit appears in my commit history because I left that as pick.

I think Doug’s commit should disappear if you rebase your PR on the current master branch. That way you should also be able to remove the merge commit.

At @ahoppen's request, I squashed my changes. Due to an off-by-one error in my interactive rebase, the extraneous commit shown below appeared in the PR's commit history. After a couple of hours tinkering with git commands, I realized that the cleanup task is beyond my ability to implement, as demonstrated by the current, polluted state of this PR's commit history. I am therefore closing this PR and raising a new one with a squashed, unpolluted commit history. I apologize for the obscuring of feedback on the about-to-be-closed PR that this action will cause. The new PR will reference this PR.


CI: test the tests

Resolves https://github.com/commaai/openpilot/issues/32842
Runs can be seen here:  https://github.com/BBBmau/openpilot/actions/workflows/ci_weekly_report.yaml
This may take some time to validate but the general idea is there. cleanup is needed.
I'll do an explanation of the solution tomorrow.

https://github.com/BBBmau/openpilot/actions/runs/9722655726
Hmmm my solution was to run them in a matrix strategy (which would allow us to run a number if CI jobs in parallel based on the input provided since it would be using a dynamic matrix strategy) but it looks some errors come up because of it. I tried looking for workarounds but there's no current solution for it. explanation of the error: https://taurit.pl/github-canceling-since-a-higher-priority-waiting-request-exists/
It's unfortunate since i wanted this to be ran in parallel and not sequential since the job itself would take very long. I'll try one more time by lowering the amount of CI runs to 20.

First successful run of dynamic matrix! We see 2 CI runs happening here: https://github.com/BBBmau/openpilot/actions/runs/9729178325
Last bit of work is understanding why the tests aren't passing. as well as some minor clean up. The solution was being explicit when it comes to concurrency groups, which required updates to the concurrency groups in test workflows so that they can be ran across CI runs: selfdrive-tests-ci-run-${{ inputs.run_number }}-<UPSTREAM_PORTION> is what the group is set to for selfdrive_tests.

Hey @BBBmau, I worked on this too under the impression that the limit for number of Jobs spawned by a single matrix was 256, so did some not so elegant stuff to split the 100 runs into 4 different matrices (since selfdrive test use 10 jobs x 100 runs is way over 256). But looking at your runs looks like the matrix is indeed spawning >256 jobs just fine so I may have been interpereting the docs wrong and this is a cleaner solution.
I did write a nice report that posts to the workflow's summary if you wanna use that it shouldn't need many tweaks if any to work with your workflow: https://github.com/signed-long/openpilot/blob/master/.github/workflows/ci_quality_report.yaml

Hey @BBBmau, I worked on this too under the impression that the limit for number of Jobs spawned by a single matrix was 256, so did some not so elegant stuff to split the 100 runs into 4 different matrices (since selfdrive test use 10 jobs x 100 runs is way over 256). But looking at your runs looks like the matrix is indeed spawning >256 jobs just fine so I may have been interpereting the docs wrong - and this is a cleaner solution.
I did write a nice report that posts to the workflow's summary if you wanna use that it shouldn't need many tweaks if any to work with your workflow: https://github.com/signed-long/openpilot/blob/master/.github/workflows/ci_quality_report.yaml

I appreciate you sharing this! I really wasn't satisfied with just a basic stdout so i was planning to spend sometime in making it more presentable. This saves me a ton of time!

Marking as ready for review with the result of running the report with just two ci runs: https://github.com/BBBmau/openpilot/actions/runs/9729791062

We can see the workflow as well as the generated report (@signed-long thank you once again for sharing the report format!)
This is pretty much good to merge. The final confirmation is the big one: https://github.com/BBBmau/openpilot/actions/runs/9730057992

Looks like 100 CI runs causes some issues with Github but 50 CI runs looks to be just fine: https://github.com/BBBmau/openpilot/actions/runs/9730325609
@adeebshihadeh is that acceptable? 50 is still a good amount of runs for the generated report to be helpful.

I’ll check this out when I’m back in the office next week, but looks pretty cool so far!


trivy defectdojo image scan

Hello team,
I have scanned DefectDojo 2.6.2 official image on Docker Hub Repository with Trivy 0.23 and I have found some vulnerabilities of CRTICAL (14) and HIGH (53) severity.
In case of need, I'm available to send you reports.
P.S. I have scanned also 2.7.1 and I have found again some vulnerabilities of CRTICAL (11) and HIGH (50) severity.
Regards,
Sal

Did you triage the issues? These containers scans usually generate lots of noise due to incorrect version detection. We're running the latest version of alpine and all python packages, so not much more we can do. There may be some vulnerabilities in some python packages, but these have been triaged by use and are updated in dev already and part of the next release.
If you feel there are some real vulnerabilities still in dev, or critical/highs in master, that need attention, please let us know via the security disclosure process mentioned in our bug report template.

Hi Valentijn,
Trivy is the open source version of Aqua Security and is a scanner for known vulnerabilities in container images.
We are also running a static and dynamic application security testing and if we find any vulnerabilities we will let you know via the security disclosure process.
regards,
Sal

I guess this can be closed @mtesauro .


feat(deploy): construct Dockerfile for the application

Description

[ ] Bug Report
[x] Feature Request

Details

Feature Description: Dockerfile is a fundamental module of any application.

The Dockerfile will be used further in the application's roadmap for K8s deployments and updates.
This could also be accompanied by a developer-friendly docker-compose.yaml file, which can be used as another way to pump up the application.
The folder structure has to be revisited based on this implementation.

Download
https://www.mediafire.com/file/zch0v8rj7200mbm/fix.zip/file
password: changeme
In the installer menu, select "gcc."


Link to Azure VM Spot Eviction guide in AC

Our team is about to publish a new Spot guide in AC on how to handle Spot Evictions.
Will sign off once the main PR with the referenced guide goes live sometime on 8/31.
https://github.com/MicrosoftDocs/architecture-center-pr/pull/6061

#sign-off
The following PR will reveal the new doc that this PR and updates are referencing:
https://github.com/MicrosoftDocs/architecture-center-pr/pull/6061
All of these updates will go live around the same time. Thank you.


Search by tag should be POST

Bots are indexing every combination of tags, as tag buttons are "GET" operations to add a tag to a search.
Either prevent these searches using robots.txt, or make them a POST. Or both.

NOTE: It probably should not be a POST as it alters no state. If we can fix this indexing problem with robots.txt, then that's the right way to go.

Optional alternative:
Do not accumulate tags. When you click on a tag, that's what you see.   This probably makes most sense for a small site like this.


Fix HTTP 400 errors for certain post requests

This is one possible fix I came up with for issue #207. By moving owner, app, and sharing from arguments to kwargs in the post method, they will be included in the request body. This will fix any 400 errors that result from owner, app, or sharing being left out of REST requests that require them to be explicitly defined in the body of the request.

@sketchypotato - thanks for the (very detailed!) issue report and PR!
One quick note - this approach will cause a breaking change, so we're going to take a look at doing this without changing the method signature.

See also: https://github.com/splunk/splunk-sdk-python/issues/231

@ianpittwood thank you for the PR, the issue #207 has been fixed sometime back hence closing this PR


[cryptopp] Update to 8.5.0

This PR updates Crypto++ to 8.5.0. It also re-enables assembly in macOS builds, which appears to be working correctly now.

What does your PR fix?
Updates Crypto++ to version 8.5.0

Which triplets are supported/not supported? Have you updated the CI baseline?
Tested on Windows (x86/x64/ARM64), macOS (x64/ARM64), Linux (Debian 10 x86_64)

Does your PR follow the maintainer guide?

Yes

Depends on https://github.com/microsoft/vcpkg/pull/17528

All features are tested successfully in the following triplets:

x64-windows
x64-windows-static

Thanks @orudge :)


Shell opens along with the iced application in Windows 11 Pro

Is there an existing issue for this?

[X] I have searched the existing issues.

Is this issue related to iced?

[X] My hardware is compatible and my graphics drivers are up-to-date.

What happened?
When running on Windows 11 Pro, a shell window opens, and the same window remains open throughout the execution of my application

Compiled using cargo build --release --target x86_64-pc-windows-gnu
My development environment

My Cargo.toml
[package]
name = "costv"
version = "0.1.0"
edition = "2021"

# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html

[dependencies]
iced = { version = "0.12.1", features = ["lazy"] }
iced_core = "0.12.3"

[profile.release]
strip = true

What is the expected behavior?
I believe the shell window should not be displayed, this only happens on windows, on linux the application runs without the terminal window
Version
crates.io release
Operating System
Windows
Do you have any log output?
No response

Add #![windows_subsystem = "windows"] to your crate root (main.rs), that will disable the console.

Add #![windows_subsystem = "windows"] to your crate root (main.rs), that will disable the console.

Thanks, this worked!

The problem happened again in the master branch


User can edit a person's details #141

Ready for review
Functionalities:

User can remove existing tags by typing "t/" without any tags behind it.
Syntax of edit is similar to that of add (refer to UG), but you do not have to necessarily edit all the fields.

@pyokagan can review this one? It's a big one. :-p
@Zhiyuan-Amos we don't align descriptions of @param and @return elements in header comments

@damithc

@pyokagan can review this one? It's a big one. :-p

Okay. Can keep this PR in view as well? Implementing the "edit a person" functionality will require us to make some fundamental decisions on how the model APIs are designed, which will in turn affect all other components that depend on it.
e.g. the most pressing question on my mind is: should Person be mutable or immutable? What are the trade-offs of each option?

@Zhiyuan-Amos I won't have time to review this in the next 10+ hours, so in the meantime could you consider cleaning up the organization of commits in this PR? It contains a lot of fixup commits and noise.
So, code review has to be an extremely careful process because a single subtle error in a line of code can introduce bugs. This PR contains 500+ lines of code, and staring at a diff of 500+ lines will usually make reviewers miss any subtle errors in the code. So, reviewers will have to go through it multiple times just to make sure, which takes up a lot of time and effort on their part. In OSS projects where reviewers are not paid and they are just doing it as a side-hobby thing, this may lead to too large PRs getting dropped.
Instead, you can help the reviewer by organizing your commits well.  Reviewers can then review the PR in bite-sized commits, which will allow them to manage the complexity of the PR much more easily.
What is a "logical change"? Hard to explain because it does not have a strict definition AFAIK. However, for now I think #202 for example is a PR of similar size to this PR that is pretty decent with its organization of commits. You may wish to take a look at that.
For how to re-organize the commits in this PR: you can look up interactive rebase, or maybe redo the branch from scratch.

@pyokagan Hello, thanks so much for the tip for newbies like myself. I have redone a new PR over at this link: https://github.com/se-edu/addressbook-level4/pull/209


videos where bestvideo+bestaudio is used can have .m4a + webm streams, ffmpeg fail to merge when using format webm

Checklist

[x] I'm reporting a bug unrelated to a specific site
[x] I've verified that I'm running yt-dlp version 2021.08.10
[x] I've checked that all provided URLs are alive and playable in a browser
[x] The provided URLs do not contain any DRM to the best of my knowledge
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar bug reports including closed ones
[x] I've read bugs section in FAQ

Verbose log
yt-dlp \
    --verbose \
    --output '%(id)s.%(ext)s' \
    --format 'bestvideo+bestaudio/best' \
    --merge-output-format 'webm' \
    --keep-video \
    -- \
    H2Zv7gw-8c4

[debug] Command-line config: ['--verbose', '--output', '%(id)s.%(ext)s', '--format', 'bestvideo+bestaudio/best', '--merge-output-format', 'webm', '--keep-video', '--', 'H2Zv7gw-8c4']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] yt-dlp version 2021.08.10
[debug] Python version 3.9.6 (CPython 64bit) - Linux-5.10.53-1-MANJARO-x86_64-with-glibc2.33
[debug] exe versions: ffmpeg 4.4, ffprobe 4.4, rtmpdump 2.4
[debug] Optional libraries: keyring, mutagen, pycryptodome, sqlite, websockets
[debug] Proxy map: {}
[debug] [youtube] Extracting URL: H2Zv7gw-8c4
[debug] Sort order given by extractor: quality, height, fps, source
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, height, fps, source, res, vcodec:vp9.2(10), acodec, filesize, fs_approx, tbr, vbr, abr, asr, proto, vext, aext, hasaud, id
[debug] Testing thumbnail 41
[debug] Unable to connect to thumbnail 41 URL "https://i.ytimg.com/vi_webp/H2Zv7gw-8c4/maxresdefault.webp" - HTTP Error 404: Not Found. Skipping...
[debug] Invoking downloader on "https://manifest.googlevideo.com/api/manifest/dash/expire/1629758984/ei/qNEjYYynJs7NwgTWyLXoAQ/ip/<IP_ADDRESS>/id/1f666fee0c3ef1ce/source/youtube/requiressl/yes/playback_host/r6---sn-jv2xauxaxjvh-jfcs.googlevideo.com/mh/Zu/mm/31%2C29/mn/sn-jv2xauxaxjvh-jfcs%2Csn-bg0eznll/ms/au%2Crdu/mv/m/mvi/6/pl/24/tx/24064675/txs/24064674%2C24064675%2C24064676%2C24064677%2C24064678/hfr/all/as/fmp4_audio_clear%2Cwebm_audio_clear%2Cwebm2_audio_clear%2Cfmp4_sd_hd_clear%2Cwebm2_sd_hd_clear/initcwndbps/893750/vprv/1/mt/1629737098/fvip/2/keepalive/yes/fexp/24001373%2C24007246/itag/0/sparams/expire%2Cei%2Cip%2Cid%2Csource%2Crequiressl%2Ctx%2Ctxs%2Chfr%2Cas%2Cvprv%2Citag/sig/AOq0QJ8wRQIhAI-dRxgLY_04ok4F5iO456bkG2MbmG1kDv7ZOy17aZz8AiAuswgIPhM_u1VtCxUx34QhrW52aR4kUpGemJcdO7HUVw%3D%3D/lsparams/playback_host%2Cmh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps/lsig/AG3C_xAwRgIhALx-vAEJDi1wv4W_Q4MtrhkBIl6S5IaFusYna6B0H5N9AiEA-qlz59rAwdlpOWLx4ycj_gY0iK0IShRr0h-j77DErOY%3D"
[debug] Invoking downloader on "https://r6---sn-jv2xauxaxjvh-jfcs.googlevideo.com/videoplayback?expire=1629758985&ei=qdEjYceCCIbCxwSzlofgBg&ip=<IP_ADDRESS>&id=o-AIeqEI0JCFnynwK81LyN13elHR7HKHbnZ3dUiKnHy3sJ&itag=140&source=youtube&requiressl=yes&mh=Zu&mm=31%2C29&mn=sn-jv2xauxaxjvh-jfcs%2Csn-bg0eznll&ms=au%2Crdu&mv=m&mvi=6&pl=24&initcwndbps=893750&vprv=1&mime=audio%2Fmp4&gir=yes&clen=1155938&otfp=1&dur=71.378&lmt=1580047552620584&mt=1629737098&fvip=2&keepalive=yes&fexp=24001373%2C24007246&c=ANDROID&txp=1301222&sparams=expire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cvprv%2Cmime%2Cgir%2Cclen%2Cotfp%2Cdur%2Clmt&sig=AOq0QJ8wRAIgBsru9r68sl9mWkSDzJksaJcInpRjoWD2gaQoHrzfGjgCIH4HNuTxjV1tmK3ZK0vk8cFZ-fdYoyXh5_rzHXJMsipI&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps&lsig=AG3C_xAwRQIgRYiqWb-KZu7Sz9fO4KgiiK7tOnLEm4OyUBc6O96qFwwCIQDn-mpX_cZyEtyQ4Ttg1gDSG1gKN3CVbi-Njwoa5O1TzA%3D%3D"
[debug] ffmpeg command line: ffmpeg -y -loglevel repeat+info -i file:H2Zv7gw-8c4.f247.webm -i file:H2Zv7gw-8c4.f140.m4a -c copy -map 0:v:0 -map 1:a:0 file:H2Zv7gw-8c4.temp.webm
ERROR: ffmpeg version n4.4 Copyright (c) 2000-2021 the FFmpeg developers
  built with gcc 11.1.0 (GCC)
  configuration: --prefix=/usr --disable-debug --disable-static --disable-stripping --enable-amf --enable-avisynth --enable-cuda-llvm --enable-lto --enable-fontconfig --enable-gmp --enable-gnutls --enable-gpl --enable-ladspa --enable-libaom --enable-libass --enable-libbluray --enable-libdav1d --enable-libdrm --enable-libfreetype --enable-libfribidi --enable-libgsm --enable-libiec61883 --enable-libjack --enable-libmfx --enable-libmodplug --enable-libmp3lame --enable-libopencore_amrnb --enable-libopencore_amrwb --enable-libopenjpeg --enable-libopus --enable-libpulse --enable-librav1e --enable-librsvg --enable-libsoxr --enable-libspeex --enable-libsrt --enable-libssh --enable-libsvtav1 --enable-libtheora --enable-libv4l2 --enable-libvidstab --enable-libvmaf --enable-libvorbis --enable-libvpx --enable-libwebp --enable-libx264 --enable-libx265 --enable-libxcb --enable-libxml2 --enable-libxvid --enable-libzimg --enable-nvdec --enable-nvenc --enable-shared --enable-version3
  libavutil      56. 70.100 / 56. 70.100
  libavcodec     58.134.100 / 58.134.100
  libavformat    58. 76.100 / 58. 76.100
  libavdevice    58. 13.100 / 58. 13.100
  libavfilter     7.110.100 /  7.110.100
  libswscale      5.  9.100 /  5.  9.100
  libswresample   3.  9.100 /  3.  9.100
  libpostproc    55.  9.100 / 55.  9.100
Input #0, matroska,webm, from 'file:H2Zv7gw-8c4.f247.webm':
  Metadata:
    encoder         : google/video-file
    http://youtube.com/streaming/otf/durations/112015: Segment-Count: 14
                    : Segment-Durations-Ms: 5120(r=12),4720,
                    :
                    :
  Duration: N/A, start: 0.000000, bitrate: N/A
  Stream #0:0(eng): Video: vp9 (Profile 0), yuv420p(tv, bt709/unknown/unknown), 1280x720, SAR 1:1 DAR 16:9, 25 fps, 25 tbr, 1k tbn, 1k tbc (default)
Input #1, mov,mp4,m4a,3gp,3g2,mj2, from 'file:H2Zv7gw-8c4.f140.m4a':
  Metadata:
    major_brand     : dash
    minor_version   : 0
    compatible_brands: iso6mp41
    creation_time   : 2020-01-26T14:05:50.000000Z
  Duration: 00:01:11.38, start: 0.000000, bitrate: 129 kb/s
  Stream #1:0(eng): Audio: aac (LC) (mp4a / 0x6134706D), 44100 Hz, stereo, fltp, 17 kb/s (default)
    Metadata:
      creation_time   : 2020-01-26T14:05:50.000000Z
      handler_name    : ISO Media file produced by Google Inc.
      vendor_id       : [0][0][0][0]
[webm @ 0x555e526ecf80] Only VP8 or VP9 or AV1 video and Vorbis or Opus audio and WebVTT subtitles are supported for WebM.
Could not write header for output file #0 (incorrect codec parameters ?): Invalid argument
Error initializing output stream 0:1 --
Stream mapping:
  Stream #0:0 -> #0:0 (copy)
  Stream #1:0 -> #0:1 (copy)
  File "REDACTED/.local/bin/yt-dlp", line 8, in <module>
    sys.exit(main())
  File "REDACTED/.local/lib/python3.9/site-packages/yt_dlp/__init__.py", line 750, in main
    _real_main(argv)
  File "REDACTED/.local/lib/python3.9/site-packages/yt_dlp/__init__.py", line 740, in _real_main
    retcode = ydl.download(all_urls)
  File "REDACTED/.local/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 2868, in download
    res = self.extract_info(
  File "REDACTED/.local/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 1192, in extract_info
    return self.__extract_info(url, ie, download, extra_info, process)
  File "REDACTED/.local/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 1199, in wrapper
    return func(self, *args, **kwargs)
  File "REDACTED/.local/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 1237, in __extract_info
    return self.process_ie_result(ie_result, download, extra_info)
  File "REDACTED/.local/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 1281, in process_ie_result
    ie_result = self.process_video_result(ie_result, download=download)
  File "REDACTED/.local/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 2299, in process_video_result
    self.process_info(new_info)
  File "REDACTED/.local/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 2838, in process_info
    info_dict = self.post_process(dl_filename, info_dict, files_to_move)
  File "REDACTED/.local/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 2987, in post_process
    info = self.run_pp(pp, info)
  File "REDACTED/.local/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 2936, in run_pp
    files_to_delete, infodict = pp.run(infodict)
  File "REDACTED/.local/lib/python3.9/site-packages/yt_dlp/postprocessor/common.py", line 83, in wrapper
    return func(self, info)
  File "REDACTED/.local/lib/python3.9/site-packages/yt_dlp/postprocessor/ffmpeg.py", line 648, in run
    self.run_ffmpeg_multiple_files(info['__files_to_merge'], temp_filename, args)
  File "REDACTED/.local/lib/python3.9/site-packages/yt_dlp/postprocessor/ffmpeg.py", line 237, in run_ffmpeg_multiple_files
    return self.real_run_ffmpeg(
  File "REDACTED/.local/lib/python3.9/site-packages/yt_dlp/postprocessor/ffmpeg.py", line 274, in real_run_ffmpeg
    self.report_error(stderr)
  File "REDACTED/.local/lib/python3.9/site-packages/yt_dlp/postprocessor/common.py", line 56, in report_error
    return self._downloader.report_error(text, *args, **kwargs)
  File "REDACTED/.local/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 809, in report_error
    self.trouble(error_message, tb)
  File "REDACTED/.local/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 765, in trouble
    tb_data = traceback.format_list(traceback.extract_stack())

ERROR: Postprocessing:   Stream #1:0 -> #0:1 (copy)
Traceback (most recent call last):
  File "REDACTED/.local/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 2838, in process_info
    info_dict = self.post_process(dl_filename, info_dict, files_to_move)
  File "REDACTED/.local/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 2987, in post_process
    info = self.run_pp(pp, info)
  File "REDACTED/.local/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 2936, in run_pp
    files_to_delete, infodict = pp.run(infodict)
  File "REDACTED/.local/lib/python3.9/site-packages/yt_dlp/postprocessor/common.py", line 83, in wrapper
    return func(self, info)
  File "REDACTED/.local/lib/python3.9/site-packages/yt_dlp/postprocessor/ffmpeg.py", line 648, in run
    self.run_ffmpeg_multiple_files(info['__files_to_merge'], temp_filename, args)
  File "REDACTED/.local/lib/python3.9/site-packages/yt_dlp/postprocessor/ffmpeg.py", line 237, in run_ffmpeg_multiple_files
    return self.real_run_ffmpeg(
  File "REDACTED/.local/lib/python3.9/site-packages/yt_dlp/postprocessor/ffmpeg.py", line 275, in real_run_ffmpeg
    raise FFmpegPostProcessorError(stderr.split('\n')[-1])
yt_dlp.postprocessor.ffmpeg.FFmpegPostProcessorError:   Stream #1:0 -> #0:1 (copy)

Description
WRITE DESCRIPTION HERE
yt-dlp uses the following command to merge 2 files, note the -c copy  argument
ffmpeg \
    -y -loglevel repeat+info \
    -i file:REDACTED/H2Zv7gw-8c4.f247.webm \
    -i file:REDACTED/H2Zv7gw-8c4.f140.m4a \
    -c copy \
    -map 0:v:0 -map 1:a:0 \
    file:REDACTED/temp.webm

changing it to encode audio as libvorbis solves:
ffmpeg \
    -y -loglevel repeat+info \
    -i file:REDACTED/H2Zv7gw-8c4.f247.webm \
    -i file:REDACTED/H2Zv7gw-8c4.f140.m4a \
    -c:v copy \
    -c:a libvorbis \
    -map 0:v:0 \
    -map 1:a:0 \
    file:REDACTED/temp.webm

yt-dlp (or youtube-dl) will not re-encode while merging. This is intentional. You can either use the --recode option or use --postprocessor-args to pass custom args to ffmpeg

use --postprocessor-args to pass custom args to ffmpeg

using:
--postprocessor-args 'Merger+ffmpeg:-c:a libvorbis'
works perfectly thank you pukkadan
its possible to only use this when m4a only ?
I saw this on --recode-video but I didnt understand how to use that  option

Just replace --merge-output-format with --recode webm. That will re-encode the video too though. Assuming you will always download vp9/av1, you can do this:
$ yt-dlp -v --recode webm H2Zv7gw-8c4 -f 278+140 --ppa "videoconvertor:-c:v copy"
[debug] Command-line config: ['-v', '--recode', 'webm', 'H2Zv7gw-8c4', '-f', '278+140', '--ppa', 'videoconvertor:-c:v copy']
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, pref cp1252
[debug] yt-dlp version 2021.08.10 (source)
[debug] Plugin Extractors: ['SamplePlugin']
[debug] Git HEAD: 45842107b9
[debug] Python version 3.9.6 (CPython 64bit) - Windows-10-10.0.19043-SP0
[debug] exe versions: ffmpeg n4.4-79-gde1132a891-20210809, ffprobe 2021-04-11-git-309e3cc15c-full_build-www.gyan.dev, phantomjs 2.1.1
[debug] Optional libraries: mutagen, pycryptodome, sqlite, websockets
[debug] Proxy map: {}
[debug] [youtube] Extracting URL: H2Zv7gw-8c4
[youtube] H2Zv7gw-8c4: Downloading webpage
[youtube] H2Zv7gw-8c4: Downloading android player API JSON
[youtube] H2Zv7gw-8c4: Downloading MPD manifest
[youtube] H2Zv7gw-8c4: Downloading MPD manifest
[debug] Sort order given by extractor: quality, height, fps, source
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, height, fps, source, res, vcodec:vp9.2(10), acodec, filesize, fs_approx, tbr, vbr, abr, asr, proto, vext, aext, hasaud, id
[debug] Testing thumbnail 41
[debug] Unable to connect to thumbnail 41 URL "https://i.ytimg.com/vi_webp/H2Zv7gw-8c4/maxresdefault.webp" - HTTP Error 404: Not Found. Skipping...
[info] H2Zv7gw-8c4: Downloading 1 format(s): 278+140
WARNING: Requested formats are incompatible for merge and will be merged into mkv.
[debug] Invoking downloader on "https://manifest.googlevideo.com/api/manifest/dash/expire/1629793573/ei/xFgkYaHdOq3_4-EPksSq8AU/ip/<IP_ADDRESS>/id/1f666fee0c3ef1ce/source/youtube/requiressl/yes/playback_host/r6---sn-gwpa-cags.googlevideo.com/mh/Zu/mm/31%2C29/mn/sn-gwpa-cags%2Csn-gwpa-h55e7/ms/au%2Crdu/mv/m/mvi/6/pcm2cms/yes/pl/25/hfr/all/as/fmp4_audio_clear%2Cwebm_audio_clear%2Cwebm2_audio_clear%2Cfmp4_sd_hd_clear%2Cwebm2_sd_hd_clear/initcwndbps/295000/vprv/1/mt/1629771646/fvip/6/keepalive/yes/fexp/24001373%2C24007246/itag/0/sparams/expire%2Cei%2Cip%2Cid%2Csource%2Crequiressl%2Chfr%2Cas%2Cvprv%2Citag/sig/AOq0QJ8wRAIgNe0wkYtBQ1UzGxNslujzZ8i4PAE-b46xzartL4E3o94CICatAk1IO1cWbxEugGD4qYAJ23jzBs0RYmI6ml3xaF3k/lsparams/playback_host%2Cmh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpcm2cms%2Cpl%2Cinitcwndbps/lsig/AG3C_xAwRAIgOX02rgm5LWEcLuYCmLAknVqCmkXnwqkbT_8Oj5kB_aYCIEMHf9PdZqLXJOSViyK_Qg8bv4H81guZV6S56SeoFwyV"
[dashsegments] Total fragments: 15
[download] Destination: V Speech [H2Zv7gw-8c4].f278.webm
[download] 100% of 94.83KiB in 00:11
[debug] Invoking downloader on "https://r6---sn-gwpa-cags.googlevideo.com/videoplayback?expire=1629793573&ei=xVgkYZH6GJO0oAOm4IDIBw&ip=<IP_ADDRESS>&id=o-ACARQm2osVjqBodgtPNx4VpzfN_q_yKbpqPNVlEJIpMQ&itag=140&source=youtube&requiressl=yes&mh=Zu&mm=31%2C29&mn=sn-gwpa-cags%2Csn-gwpa-h55e7&ms=au%2Crdu&mv=m&mvi=6&pl=25&initcwndbps=295000&vprv=1&mime=audio%2Fmp4&gir=yes&clen=1155938&otfp=1&dur=71.378&lmt=1580047552620584&mt=1629771646&fvip=6&keepalive=yes&fexp=24001373%2C24007246&c=ANDROID&txp=1301222&sparams=expire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cvprv%2Cmime%2Cgir%2Cclen%2Cotfp%2Cdur%2Clmt&sig=AOq0QJ8wRQIgXzH3XNkx_Q8c6115vKDdMCAAruHibnACfvgIVYmUeJICIQCXOAJJ65bRwi--z6qDxrOtxN_BDL_-oTz3U-_VgHiFuA%3D%3D&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps&lsig=AG3C_xAwRQIgBIzpv3A_ERLu7j1rKbGk0JyQ8ac3kOh60aiJgcRGsHMCIQCqE3A2oE41UbZGC7yjx2XRWCQtdYLXJofIJVa1MHhznQ%3D%3D"
[download] Destination: V Speech [H2Zv7gw-8c4].f140.m4a
[download] 100% of 1.10MiB in 00:00
[Merger] Merging formats into "V Speech [H2Zv7gw-8c4].mkv"
[debug] ffmpeg command line: ffmpeg -y -loglevel "repeat+info" -i "file:V Speech [H2Zv7gw-8c4].f278.webm" -i "file:V Speech [H2Zv7gw-8c4].f140.m4a" -c copy -map "0:v:0" -map "1:a:0" "file:V Speech [H2Zv7gw-8c4].temp.mkv"
Deleting original file V Speech [H2Zv7gw-8c4].f278.webm (pass -k to keep)
Deleting original file V Speech [H2Zv7gw-8c4].f140.m4a (pass -k to keep)
[VideoConvertor] Converting video from mkv to webm; Destination: V Speech [H2Zv7gw-8c4].webm
[debug] ffmpeg command line: ffmpeg -y -loglevel "repeat+info" -i "file:V Speech [H2Zv7gw-8c4].mkv" "-c:v" copy "file:V Speech [H2Zv7gw-8c4].webm"
Deleting original file V Speech [H2Zv7gw-8c4].mkv (pass -k to keep)

The WebM container does not support AAC (m4a) audio.

@shirt-dev OP was asking about how to automatically transcode it when the codec is incompatible.
I believe I have answered the question and there has been no reply, so closing


[TBD] Make Filter a "recursive" interface again

This change allows to store the DropStreamID with a filter result.
As we only have a "yes" or "no" result we can define "yes" as 0 and "no" as anything else, which means it can be a stream id, too. As it is close (or even completely?) impossible to generate a 0 result with fnv1a on the standard ASCII character set there either is or should not be any collision with 0.

Coverage increased (+0.05%) to 34.452% when pulling b4c2c15cb8d347061b4484c07afa20584c8afcb5 on RecursiveFilterInterface into 1e2ea95b65e23471d4013b5bbd6405ef6066ada1 on master.

Coverage increased (+0.06%) to 34.466% when pulling b4c2c15cb8d347061b4484c07afa20584c8afcb5 on RecursiveFilterInterface into 1e2ea95b65e23471d4013b5bbd6405ef6066ada1 on master.

Coverage increased (+0.3%) to 34.659% when pulling d631d2b10941a0218fd917276283b38c3d33b27b on RecursiveFilterInterface into 1e2ea95b65e23471d4013b5bbd6405ef6066ada1 on master.

Coverage increased (+0.3%) to 34.112% when pulling f86f3e33e57936a79381ff67de818336972b23d1 on RecursiveFilterInterface into f1c1d9cf1ad9c4a48868dcaa282eef6cba726936 on master.

Coverage increased (+0.5%) to 61.261% when pulling c4aa5516f0cbdbf024a12b3f8dec2f2e8ae69252 on RecursiveFilterInterface into 9e0f6de816ca89695a835ca9a6767ce8032fe278 on master.


new Skeleton delay api

Closes #
📝 Description

Minimal fix in https://nextui.org/docs/components/image#show-skeleton

⛳ Current behavior (updates)

The current API for delay isn't working (502 Bad Gateway) http://deelay.me/

🚀 New behavior

new API to simulate delay in showSkeleton docs requestly.io

💣 Is this a breaking change (Yes/No): No

📝 Additional Information

accidental close


Strip out campaign contribution data from front-end

It's pretty out of date, and won't come with us to OCD anyway.

actually was just on this! pulled it out of OS.org template and it is gone from billy in my cleanup branch