Closed bearecinos closed 2 years ago
@dngoldberg there is no need to review this I think (there are a lot of changes.. ) I will keep making changes anyways to activate the melt rate param once I get the alpha init constant values and the new velocity input.
FYI @dngoldberg
I think this is ready to be merge. Its a very big PR but the code that previously lived in meshtools/meshtools.py has been re-organized as a ficetools toolbox
ficetools now has the right imports (tml_adjoint back ends) following @jrmaddison changes to the fenics_ice repo.
__init__.py
backend.py
: Organises fenics and tlm_adjoing importsgraphics.py
: A set of tools for plotting input data and all the experiments results from fenics_icemesh.py
: A set of tools for mesh generation.utils_funcs.py
: Useful functions to organised data, compute vertex for fenic_ice parameters, and other utils that did not fit in any other module, lets try to keep this one short!velocity.py
: A set of tools for the generation of velocity input data over a specific domain.This should make the development easier as we don't have a huge file of over 3000 lines anymore!
All of this required changes in the prepro_stages and plotting_stages scripts.
A big change now its on the smith.toml and on the param_sweep.py script to generate the "tolms" for the l-curve experiments. These files now reflects the phase_suffix and new output/diagnostic dir changes.
Solves issues #9, #3 , #11 and #10