maho3 / ltu-ili

Robust ML in Astro
https://ltu-ili.readthedocs.io/en/latest/
45 stars 9 forks source link

35 separate data and training config files #38

Closed maho3 closed 1 year ago

maho3 commented 1 year ago

Some refactoring and reorganization of the data loaders and runner objects. Changes:

DeaglanBartlett commented 1 year ago

Nice! There are just a few things that need fixing

installation:

Example inference:

Quijote example:

notebook/compare_summaries.ipynb

DeaglanBartlett commented 1 year ago

Just a few more things in INSTALL.md:

maho3 commented 1 year ago

Thanks for catching this. ili-summarizer should not be cloned into the ltu-ili/ directory, but instead into the same level as ltu-ili. I've cleared that up as well as removed some unnecessary back-and-forth cd'ing.

DeaglanBartlett commented 1 year ago

Thanks for changing this. One final thing: now for the installation tests, I think you need to change cd examples to cd ltu-ili/examples as you aren't in the ltu-ili repository at this point