maho3 / ltu-ili

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

Cleanup #53

Closed maho3 closed 1 year ago

maho3 commented 1 year ago

Cleanup and refactoring of ltu-ili after many changes over the last months. Addresses #52.

maho3 commented 1 year ago

A lot of minor changes have occurred in this PR, resulting in many files changed. The primary functionality remains exactly the same, this PR is just to make everything consistent over the past series of changes. Here's a summary:

For reviewers Since a lot of files changed, it will be difficult to check everything of them. So I recommend you check that at

Note, SNLE/SNRE support and custom sampling will be added in PR #54 .

DeaglanBartlett commented 1 year ago

Replies to checks:

I compiled these without issues locally (some warnings were raised, but did not affect the result).

I see no issues with the code structure

Other changes I have made:

A request: