Mihai Cara

Results 196 comments of Mihai Cara

I would prefer to review myself in order to understand (for my own education) the proposed changes but I will not be able to look at this PR in detail...

1. When testing https://github.com/spacetelescope/stcal/pull/418 one can see that downstream tests (true, it's `romancal` right now) fail because of deprecation warnings being converted into errors. 2. @braingram was concerned about backwards...

Removing upper pin is something that can be done although we can run regression tests right now and force a specific version of `drizzle` to be used regardless of this...

> Turns out the dev requirements does not include dev version of drizzle, plus there is an upper pin, so we actually won't see these warnings until drizzle is released...

Also, > Wait for drizzle to release with these changes. The point is to test `drizzle` with existing `jwst` (`stcal`) code to see that it does not break anything _before_...

@melanieclarke I was not aware of this issue. Feel free to create a JP ticket.