Implement stcal.alignment sub-package.

spacetelescope / stcal

https://stcal.readthedocs.io/en/latest/

Other

10 stars 32 forks source link

Implement stcal.alignment sub-package. #177

Closed mairanteodoro closed 1 year ago

mairanteodoro commented 1 year ago

This PR addresses the initial structure for the modules of the alignment sub-package for stcal.

Checklist

[ ] added entry in CHANGES.rst (either in Bug Fixes or Changes to API)
[ ] updated relevant tests
[ ] updated relevant documentation
[ ] updated relevant milestone(s)
[ ] added relevant label(s)

codecov[bot] commented 1 year ago

Codecov Report

Patch and project coverage have no change.

Comparison is base (e0c6f69) 74.46% compared to head (a4fed92) 74.46%.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #177 +/- ## ======================================= Coverage 74.46% 74.46% ======================================= Files 29 29 Lines 5937 5937 ======================================= Hits 4421 4421 Misses 1516 1516 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

nden commented 1 year ago

@mairanteodoro, @s-goldman Do you mind if this work is kept on a separate branch until ready? Then we can merge into main. In this way it won't interfere with releases.

Also I think we should add modules only when we need them, to avoid adding too much clutter. So perhaps keep only utils.py and resample_utils.py.

mairanteodoro commented 1 year ago

I don't mind keeping a separate branch for the new sub-package work.

mairanteodoro commented 1 year ago

Moving changes to a separate branch (stcal-alignment).