Remove the ScarletModelData to LsstScarletModelData converter (DM-55150) - #1386
Remove the ScarletModelData to LsstScarletModelData converter (DM-55150)#1386fred3m wants to merge 1 commit into
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1386 +/- ##
==========================================
- Coverage 89.90% 89.89% -0.01%
==========================================
Files 373 373
Lines 51089 51089
Branches 5953 5953
==========================================
- Hits 45930 45929 -1
Misses 3766 3766
- Partials 1393 1394 +1 ☔ View full report in Codecov by Harness. |
|
@TallJimbo I am guessing we need to keep this converter around for a while given #1411 |
|
I'm not sure; #1411 is about the other direction, which in practice is about supporting new pipelines/tasks on old repos. That's important because most extant repos qualify as "old" - i.e. they have the old storage class for the dataset type where this is used. So I think this PR is just for supporting old pipelines/tasks (which now just means older software versions) against new repos ( |
|
Oh, no, nevermind: we do need this in the long term, because if #1411 makes a new |
A base ScarletModelData cannot be promoted to an LsstScarletModelData: the latter requires a single catalog-wide observed PSF and band list, while a ScarletModelData carries them per blend. The conversion never produced a usable model in the science pipelines, so drop the storage-class `converters` entry (and the now-removed `scarlet_model_to_lsst_scarlet_model` it referenced, deleted in a companion commit on this ticket). Old ScarletModelData datasets continue to load at their own storage class. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
A base ScarletModelData cannot be promoted to an LsstScarletModelData: the latter requires a single catalog-wide observed PSF and band list, while a ScarletModelData carries them per blend. The conversion never produced a usable model in the science pipelines, so drop the storage-class
convertersentry (and the now-removedscarlet_model_to_lsst_scarlet_modelit referenced, deleted in a companion commit on this ticket).Old ScarletModelData datasets continue to load at their own storage class.
Checklist
doc/changesconfigs/old_dimensionsand update the list indoc/lsst.daf.butler/dimensions.rst