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

xarray_to_video and coastlines #100

Closed
JimCircadian opened this issue Feb 16, 2023 · 7 comments · May be fixed by #282
Closed

xarray_to_video and coastlines #100

JimCircadian opened this issue Feb 16, 2023 · 7 comments · May be fixed by #282
Assignees

Comments

@JimCircadian
Copy link
Member

JimCircadian commented Feb 16, 2023

Related to #97, probably worth tackling as one. Worth offering non contour options for landmasks too.

@JimCircadian
Copy link
Member Author

Cartopy and FuncAnimation don't yet appear to operate well together, so placing a hold on investigating this for the driving force of the functionality, but have added ax_init and ax_extra arguments that help in allowing this to happen.

@JimCircadian
Copy link
Member Author

@tom-andersson mentioned this alternative to the matplotlib native animation library for producing videos. Might be worth a look (perhaps as an optional dependency?)

@JimCircadian
Copy link
Member Author

A quick look suggests that it's directly integrated with matplotlib, so testing alongside cartopy will be prudent

@ots22
Copy link
Member

ots22 commented Feb 21, 2023

I was playing around with FuncAnimation with cartopy and have something working in a notebook. I could try to get this into xarray_to_video, but is this still worth pursuing given the above?

@JimCircadian
Copy link
Member Author

Nice one @ots22. Is it a solution that can hijack those new options or something that requires re-engineering of xarray_to_video? If it won't break the existing mechanics of then definitely go for it, I'd prefer that option than including new dependencies if I'm honest! 😉

@ots22
Copy link
Member

ots22 commented Feb 21, 2023

I'm not completely sure yet! I suspect it might need some changes to xarray_to_video but I'll try to avoid breaking the existing interface.

@JimCircadian
Copy link
Member Author

JimCircadian commented Feb 21, 2023

That's perfect! Don't worry toooooo much about investing loads of time on this though, it's really very very simple to stitch the necessary frames together for what we're trying to achieve and we have to produce individual frames anyway. 😉 I'll send you a message on slack

@ots22 ots22 self-assigned this Feb 21, 2023
bnubald added a commit to bnubald/icenet that referenced this issue Jul 9, 2024
@bnubald bnubald assigned bnubald and unassigned ots22 Jul 9, 2024
@bnubald bnubald linked a pull request Jul 9, 2024 that will close this issue
@bnubald bnubald closed this as completed Aug 21, 2024
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 a pull request may close this issue.

3 participants