-
Notifications
You must be signed in to change notification settings - Fork 65
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
More fixes to the LDO mask handling, etc. #394
More fixes to the LDO mask handling, etc. #394
Conversation
is apparently imported without the mixins
I have a case where I'm getting an error that I can't come up with a test for:
|
How do we return a |
This reverts commit 359d565.
@keflavich -- I reverted all of the changes for 2D masking. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
The other issues discussed here are with regards to 2D masking and WCS objects without both celestial axes defined.
This revisits issue #392, which wasn't actually solved by that PR. However, it
turns out we need to make deeper changes.