Skip to content
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

jigsaw using CSM sensor models and "standard" ISIS parameters returns a non-helpful error #5486

Closed
jlaura opened this issue Apr 24, 2024 · 0 comments · Fixed by #5562
Closed
Assignees
Labels
bug Something isn't working

Comments

@jlaura
Copy link
Collaborator

jlaura commented Apr 24, 2024

ISIS version(s) affected: 8.0.0+

Description

When using csminited sensor models, one needs to use csmsolveset=VALID amongst other optional arguments. If you do not, the error returned is:

**PROGRAMMER ERROR** Could not solve bundle adjust.
**PROGRAMMER ERROR** CHOLMOD: Failed to load Triplet matrix.

The docs describe needing a different parameter set, but this is buried and not super obvious.

How to reproduce

Bundle adjust pair of csminited cubes with jigsaw without specifying csmsolveset.

Possible Solution

Inside ISIS, if a way exists to have a more obvious error message print, I think that would be good. Something along the lines of: "Data are using CSM sensor models. Therefore, jigsaw must use the CSM parameters. Please see the documentation."

Additional context

@jlaura jlaura added the bug Something isn't working label Apr 24, 2024
@AustinSanders AustinSanders self-assigned this Jul 23, 2024
@github-project-automation github-project-automation bot moved this from In Progress to Done in Astro Software Support Sprint Jul 26, 2024
This was referenced Nov 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Development

Successfully merging a pull request may close this issue.

2 participants