Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge with upstream #1

Merged
merged 67 commits into from
Apr 12, 2023
Merged

Merge with upstream #1

merged 67 commits into from
Apr 12, 2023

Conversation

yuqian5
Copy link
Owner

@yuqian5 yuqian5 commented Apr 12, 2023

No description provided.

Dimitri Vranken and others added 30 commits February 13, 2023 11:07
Changes can be made in a single place if neeeded
Required for successful test execution on my rather slow machine
They are now also supported on Linux because we are using SkiaSharp instead of System.Drawing.Common
Required for execution on Linux, otherwise we get the following error: Unable to load shared library 'libSkiaSharp' or one of its dependencies
See mono/SkiaSharp#1341 for more information
The two constructors were causing issues with ambiguous method calls but are a good idea, so this is more of a workaround.
Error message:
The call is ambiguous between the following methods or properties:
'FFMpegCore.Pipes.RawVideoPipeSource.RawVideoPipeSource(System.Collections.Generic.IEnumerator<FFMpegCore.Pipes.IVideoFrame>)' and
'FFMpegCore.Pipes.RawVideoPipeSource.RawVideoPipeSource(System.Collections.Generic.IEnumerable<FFMpegCore.Pipes.IVideoFrame>)'
…png-images

[Bugfix] Handle joining of non png images
Dimitri Vranken and others added 28 commits February 20, 2023 13:18
Execution time is now approximately one fifth of what it was previously
Existing (untouched) tests are not stable in CI test execution
…windows

Only upload to codecov on windows
Add a helper method to generate GIF Snapshots
@yuqian5 yuqian5 self-assigned this Apr 12, 2023
@yuqian5 yuqian5 merged commit 7c333a7 into yuqian5:main Apr 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants