LSSTDESC / rail_base

Base classes for RAIL
MIT License
0 stars 1 forks source link

Drop the spectroscopic_selection in rail_base and host it in rail_astro_tools #95

Closed ztq1996 closed 3 months ago

ztq1996 commented 3 months ago

Problem & Solution Description (including issue #)

We are bringing the spectroscopic selection to rail_astro_tools, and dropping the one in rail_base. Here I am bringing the up-to-date version in rail_base to rail_astro_tools. The differences are made by @OliviaLynn, and are all reviewed in rail_base, so this should be straightforward.

Code Quality

codecov[bot] commented 3 months ago

Codecov Report

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

Project coverage is 96.78%. Comparing base (8c2733e) to head (dbc4911).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #95 +/- ## ========================================== - Coverage 97.11% 96.78% -0.34% ========================================== Files 32 31 -1 Lines 1770 1585 -185 ========================================== - Hits 1719 1534 -185 Misses 51 51 ```

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