csdms / ivy

Course material on scientific software development for researchers in earth and planetary surface processes
https://csdms.colorado.edu
Other
75 stars 58 forks source link

Use functions from solver module in Diffusion class #132

Closed mdpiper closed 1 year ago

mdpiper commented 1 year ago

In this PR, I modified the ivy-diffusion Diffusion class to use the functions from the solver module. This provides a nice way to show:

In order to make relative imports work, I had to include a __init__.py file in the ivy-diffusion package (see #131). I had been hoping to avoid this extra step.

I also changed the way the time step is calculated in the solver module. It was way unstable otherwise.

This resolves #129.