open-atmos / PyMPDATA

Numba-accelerated Pythonic implementation of MPDATA with examples in Python, Julia, Rust and Matlab
https://open-atmos.github.io/PyMPDATA/
GNU General Public License v3.0
24 stars 17 forks source link

issues with the new 2D advection-diffusion example #453

Closed slayoo closed 3 weeks ago

slayoo commented 2 months ago

@pawelmagnu, here's an update on the post-merge status for #427:

Let's aim at fixing it next week

pawelmagnu commented 1 month ago

@Kojec1 could you check which points have already been addressed in #456 ?

Kojec1 commented 1 month ago

There's still an issue with the figures displayed on GitHub, missing header information and I haven't managed to check if the .gif file is successfully uploaded to the "tip" release. The other issues seem to be solve.

pawelmagnu commented 1 month ago

I'm pretty sure we can't check if the .gif file uploads before actually merging to main.

slayoo commented 1 month ago

.gif file upload works! The notebook does not work on Colab yet as it fails on first line with: image This is due to lack of pip install ... commands for Colab in an initial notebook cell, like we do in other notebooks. Here's a PR addressing it: https://github.com/open-atmos/PyMPDATA/pull/458

slayoo commented 1 month ago

Just tested both on Colab and mybinder.org, and everything works! So, we're still missing two things:

pawelmagnu commented 1 month ago

addressed in #462

slayoo commented 3 weeks ago

kudos!