DEEPDIP-project / CoupledNODE.jl

Apache License 2.0
2 stars 0 forks source link

import statements #27

Closed luisaforozco closed 6 months ago

luisaforozco commented 6 months ago

Context

Issue #19

Changes

As mentioned in #19 instead of having a big set of using statements in the beginning of the examples I tried to use import whenever possible as well as importing the requirements when needed. I also tried to do this for the include files.

⚠️ There are changes on the name of the object Grid because Plots has also a grid. So declaring using Plots would already define a grid element in the namespace.

Functionality/tests

I've tested the examples 01.00, 02.00, 02.01, 02.02, 02.03. Also I have merged @SCiarella branch #23 into this one and solved the conflicts. I've also run the JuliaFormatter (manually).

luisaforozco commented 6 months ago

Yes, at this state we cannot avoid the includes. But indeed once the members become part of the module CoupledNODE.jl we can import them as the other external dependencies.