Christina Hedges
Christina Hedges
Hi @orionlee thanks for bringing this up! In this PR, if a user were to input mask data e.g. because of a flare or a cosmic ray, this data would...
Hi @orionlee thanks for the explanation, when you say > The current behavior is to replace nan values with mask. what do you mean here? There is also a comment...
@Nschanche here are some quick tips; 1. Put your GREAT comments in the first post on the PR, so people can easily see (myself included) 2. Install and use a...
In the workflow we've talked about you need to be careful that you aren't constantly re-reading in that PRF file if you don't need to. You can use `lru_cache` potentially...
Hi @Nschanche this is looking really close! Here are the last things you need to wrap up to get the WIP label removed: 0. Get a linter. 1. Update tpf.py...
This would be addressed by #1345 if we merge that in, maybe @rebekah9969 wants to take a look at that PR and provide feedback?
This is really awesome! Can't wait to see how this evolves. @ojhall94 what do you envision partial stitching might be?
- [x] Make a TPF with all nan errors, and check that this fails gracefully - [x] Make a TPF with partial nan errors, and check that passes - [...
Hi @kristalynnesmith, sorry for the delay! I believe this is due to the recent `numpy` update to v1.24. I actually can't replicate this problem easily. If you're still experiencing this...
@vvilhelmus thanks for commenting and corroborating, that's a really good point about the tutorials. I hadn't seen this issue in v2.3. We need to do some digging to figure out...