search

spacetelescope / romancal

Python library to process science observations from the Nancy Grace Roman Space Telescope
https://roman-pipeline.readthedocs.io/en/latest/
Other
32 stars 28 forks source link

Refactor resample (and other step) input handling #1309

Open braingram opened 4 months ago

braingram commented 4 months ago

As noted https://github.com/spacetelescope/romancal/pull/1241#discussion_r1629650840

The resample step has a large block of logic to handle different input types: https://github.com/spacetelescope/romancal/blob/459e8e345bbb1ba6bfd76d0d8a90f2058089791e/romancal/resample/resample_step.py#L70

There are similar blocks in other steps, for example for flux: https://github.com/spacetelescope/romancal/blob/459e8e345bbb1ba6bfd76d0d8a90f2058089791e/romancal/flux/flux_step.py#L50

Investigate moving as much of this logic as possible to common code.

braingram commented 4 months ago

This is related to https://jira.stsci.edu/browse/RCAL-875