Skip to content

Avoid pyav v14 to prevent AttributeError #1112

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

Merged
merged 15 commits into from
Jan 18, 2025
Merged

Conversation

shaform
Copy link
Contributor

@shaform shaform commented Dec 4, 2024

Closes: #1111.

Work around the issue described in #1111 by restricting pyav version.

@shaform shaform changed the title Fix AttributeError in pyav v14 and fixes #1111 Fix AttributeError in pyav v14 Dec 4, 2024
@shaform shaform changed the title Fix AttributeError in pyav v14 Prevents AttributeError in pyav v14 Dec 12, 2024
@shaform shaform changed the title Prevents AttributeError in pyav v14 Prevent AttributeError in pyav v14 Dec 12, 2024
@shaform shaform changed the title Prevent AttributeError in pyav v14 Avoid pyav v14 to prevent AttributeError Dec 17, 2024
@FirefoxMetzger
Copy link
Contributor

[Sorry for ghosting this PR. I've had a neat sequence of events that, unfortunately, kept me away from open-source (work deadline, vacation in China, being sick). Now I am back.

I took the liberty to not just go over the PR but also checked out other changes that come with av v14, which are mostly removals of various convenience methods that used to exist before. I added changes for these here.

Copy link
Contributor

@FirefoxMetzger FirefoxMetzger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added all the changes I would make to this PR, and then I will merge once I#ve updated CI.

@FirefoxMetzger FirefoxMetzger merged commit 08d2cc1 into imageio:master Jan 18, 2025
17 checks passed
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.

latest pyav av.video.codeccontext.VideoCodecContext does not have a close() method
3 participants