Skip to content
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

Fix call to ELP #73

Merged
merged 1 commit into from
Jan 8, 2025
Merged

Conversation

nden
Copy link
Contributor

@nden nden commented Jan 5, 2025

This PR does two things

  • Fixes the way the exposure level pipeline and a step are called by using the .call method.
    There are two ways to execute a step or a pipeline.
    .call : combines parameter values from the defaults in the code, CRDS parameter files and user supplied parameters
    .run: uses the default parameter values in the code and ignores parameter files in CRDS.
  • Changes the step name : source_detection to source_catalog as this has changed in the pipeline. The change was made in romancal v 16.

@tddesjardins

@tddesjardins tddesjardins merged commit f4a9989 into spacetelescope:main Jan 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants