LSSTDESC / rail

Top level "umbrella" package for RAIL
MIT License
8 stars 4 forks source link

Clean up evaluation demo notebook and utils.py file. #99

Closed eacharles closed 3 months ago

eacharles commented 10 months ago

rail/examples/evaluation_examples/utils.py -> This stuff is only used by the evaluation demo notebook. I think the answer here is to clean up that notebooks and either get stuff from somewhere else, or just add it to the notebook. If we have other notebooks that start needed some of this stuff, we can think about where it belongs in the wider RAIL ecosystem. This should be it's own issue.

eacharles commented 10 months ago

Though, if we refactor the Evaluation code, it might be better to wait...

ztq1996 commented 3 months ago

Util refactoring is done in PR #138, closing issue.