-
Notifications
You must be signed in to change notification settings - Fork 11
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
Numerical instability in Mapping.rate #28
Comments
I believe this is with version 9.2.11 (but it doesn't look like 9.2.12 has had any changes in this area). |
@dsberry this is the output of the test script:
|
I've been able to reproduce this. I'll look into a fix some time over the next week or so. Sorry it can't be a bit sooner but I'm in the thick of things at the moment. |
Hopefully the commit I've just made will fix the issue. |
@dsberry would it be possible to make a new release with this fix? |
Yes, but it will be next week before I can get round to it. |
@timj New release 9.2.13 now available on github. |
I'm seeing an unexpected near-zero value in
Mapping.rate
for a transform that otherwise looks perfectly fine (with the correct rate many orders of magnitude larger). In particular, this is a 2-d composite mapping, and if I compute the first derivative matrices of the two component mappings viaMapping.rate
, those matrices and their product looks fine, but the matrix derived from the composite mapping's rate is singular.I've included below a how-to-reproduce Python script that uses the Python bindings in Rubin Observatory's astshim package, but I think it should be easy translatable into any other kind of script that calls AST under the hood. If you do need to install it, I'd recommend installing the full Rubin software stack.
The text was updated successfully, but these errors were encountered: