LSSTDESC / rail

Top level "umbrella" package for RAIL
MIT License
5 stars 3 forks source link

Move in bpz notebooks, update paths; copy over cmnn nb changes #109

Closed OliviaLynn closed 7 months ago

OliviaLynn commented 8 months ago

Problem & Solution Description (including issue #)

Also copies over updates to CMNN notebook made in CMNN repo to the notebook here

That was on me, should have deleted it sooner. PR is out to remove it at LSSTDESC/rail_cmnn#13

Code Quality

codecov[bot] commented 8 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (1de4475) 100.00% compared to head (453792d) 100.00%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #109 +/- ## ========================================= Coverage 100.00% 100.00% ========================================= Files 1 1 Lines 1 1 ========================================= Hits 1 1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

OliviaLynn commented 7 months ago

Ok, I changed some text and added a commented rail get-data command, so now I think it looks good, I'll approve, if my changes look good go ahead and merge.

Nice catch! That looks great, thank you