kammerje / jwst-kpi

Kernel phase pipeline for JWST.
GNU General Public License v3.0
3 stars 2 forks source link

Loop over vectorized code #18

Closed vandalt closed 1 year ago

vandalt commented 1 year ago

I think there is a loop over vectorized code in the trim step:

https://github.com/kammerje/jwst-kpi/blob/41abac728f870538973dd4293f130141ccfc2206/jwst_kpi/calwebb_kpi3.py#L324-L339

The loop variable is not used and the code does the operation on all frames with : I believe.

kammerje commented 1 year ago

Yes, that seems like a bug!