sunkit-spex icon indicating copy to clipboard operation
sunkit-spex copied to clipboard

Refactor redundant legacy

Open samaloney opened this issue 1 year ago • 2 comments

Remove redundant 'legacy' in module from ~sunkit_spex.legacy.fitting_legacy to sunkit_spex.legacy.fitting.

samaloney avatar Jul 17 '24 11:07 samaloney

Codecov Report

Attention: Patch coverage is 38.88889% with 66 lines in your changes missing coverage. Please review.

Please upload report for BASE (main@3a9a4a4). Learn more about missing BASE report.

Files Patch % Lines
sunkit_spex/legacy/fitting/data_loader.py 10.34% 52 Missing :warning:
sunkit_spex/legacy/fitting/instruments.py 38.09% 13 Missing :warning:
sunkit_spex/legacy/fitting/fitter.py 90.00% 1 Missing :warning:
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #158   +/-   ##
=======================================
  Coverage        ?   60.50%           
=======================================
  Files           ?       25           
  Lines           ?     3408           
  Branches        ?        0           
=======================================
  Hits            ?     2062           
  Misses          ?     1346           
  Partials        ?        0           

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

codecov-commenter avatar Jul 17 '24 11:07 codecov-commenter

Note I haven't update the notebooks as they will be removed in #153

samaloney avatar Jul 17 '24 12:07 samaloney

Reviews please

samaloney avatar Sep 10 '24 15:09 samaloney

Why was it called legacy originally?

nabobalis avatar Sep 10 '24 17:09 nabobalis

Needed a way to delineate older code which won't be updated and will eventually be removed from what we are now trying to do

samaloney avatar Sep 12 '24 14:09 samaloney