-
Notifications
You must be signed in to change notification settings - Fork 28
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove SourceDetection rename source_detection to source_catalog #1533
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1533 +/- ##
===========================================
+ Coverage 28.01% 76.77% +48.76%
===========================================
Files 116 118 +2
Lines 7728 7747 +19
===========================================
+ Hits 2165 5948 +3783
+ Misses 5563 1799 -3764 ☔ View full report in Codecov by Sentry. |
4e8f576
to
f3bab1a
Compare
pyproject.toml
Outdated
# "roman_datamodels>=0.22.0,<0.23.0", | ||
"roman_datamodels @ git+https://github.com/spacetelescope/roman_datamodels.git", | ||
#"roman_datamodels>=0.22.0,<0.23.0", | ||
"roman_datamodels @ git+https://github.com/braingram/roman_datamodels.git@source_catalog", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Temporarily pointing to the source branch for spacetelescope/roman_datamodels#428 to pick up required changes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
98a938a
to
24b7c7a
Compare
for more information, see https://pre-commit.ci
24b7c7a
to
967c03f
Compare
This PR:
SourceDetectionStep
meta.source_detection
tometa.source_catalog
meta.cal_step.source_detection
tometa.cal_step.souce_catalog
SourceCatalog
toImageSourceCatalog
(to deal with schema name conflicts with the above renaming and to make this agree withMosaicSourceCatalog
)As the above renaming breaks reading current files a new set of regtest data was generated:
roman-pipeline/dev_20241127/
all tests pass locally.
This PR also makes several changes to
make_regtest_data.sh
to deal with a few issues. I will call them out in comments.Regtests (with existing files showing many expected failures): https://github.com/spacetelescope/RegressionTests/actions/runs/11942934325
Running regtests with new truth files: https://github.com/spacetelescope/RegressionTests/actions/runs/12144666599
Tasks
24Q4_B15
(use the latest build if not sure)no-changelog-entry-needed
)changes/
:echo "changed something" > changes/<PR#>.<changetype>.rst
(see below for change types)docs/
pageokify_regtests
to update the truth filesnews fragment change types...
changes/<PR#>.general.rst
: infrastructure or miscellaneous changechanges/<PR#>.docs.rst
changes/<PR#>.stpipe.rst
changes/<PR#>.associations.rst
changes/<PR#>.scripts.rst
changes/<PR#>.mosaic_pipeline.rst
changes/<PR#>.patch_match.rst
steps
changes/<PR#>.dq_init.rst
changes/<PR#>.saturation.rst
changes/<PR#>.refpix.rst
changes/<PR#>.linearity.rst
changes/<PR#>.dark_current.rst
changes/<PR#>.jump_detection.rst
changes/<PR#>.ramp_fitting.rst
changes/<PR#>.assign_wcs.rst
changes/<PR#>.flatfield.rst
changes/<PR#>.photom.rst
changes/<PR#>.flux.rst
changes/<PR#>.source_detection.rst
changes/<PR#>.tweakreg.rst
changes/<PR#>.skymatch.rst
changes/<PR#>.outlier_detection.rst
changes/<PR#>.resample.rst
changes/<PR#>.source_catalog.rst