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

Jump to bottom

[MNT]: FFMpegWriter does not check if out path exists when initialized. #25233

Closed
waltsims opened this issue Feb 16, 2023 · 0 comments · Fixed by #25238
Closed

[MNT]: FFMpegWriter does not check if out path exists when initialized. #25233

waltsims opened this issue Feb 16, 2023 · 0 comments · Fixed by #25238
Labels
API: argument checking topic: animation
Milestone
v3.7.1

Comments

@waltsims
Copy link

Summary

FFMpegWriter does not ensure the outputpath exists when initialized (here). This leads to a broken pipe error with no mention of a non-existent path which can be misleading to the user.

Proposed fix

When setup is called, check the output path exists and if not, throw an error to inform the user.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
API: argument checking topic: animation
Projects
None yet
Milestone
v3.7.1
Development

Successfully merging a pull request may close this issue.

Check file path for animation and raise if it does not exist oscargus/matplotlib
3 participants
@QuLogic @anntzer @waltsims