-
Notifications
You must be signed in to change notification settings - Fork 0
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
Merge new changes from paltas main branch #1
Conversation
smericks
commented
Sep 6, 2023
- Resolve some merge conflicts & accept new changes
The 'self' argument is included in the caching calculation, and the cache is kept globally with the class (not with each instance, as you might expect). One of these even has an unbound size! Finally, these don't actually provide an appreciable speedup.
Remove lru_cache on LOS methods
Images double in size following mag_cut failure
Argument is that generator does not need it so user should not have to suffer its installation...
Fixing deprecation issues causing test errors
…thub.com:drphilmarshall/paltas into drphilmarshall/feature/contributor-documentation
…tributor-documentation Contributor documentation
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.
note bug with MagnificationError and _draw_image_standard() in drizzle function might need to be changed with my updated handling of errors
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.
note kwargs_synthesis in main_deflectors line 66, 133