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

Gracefully handle missing files #21

Closed
bpurinton opened this issue Jul 3, 2024 · 1 comment · Fixed by #31
Closed

Gracefully handle missing files #21

bpurinton opened this issue Jul 3, 2024 · 1 comment · Fixed by #31

Comments

@bpurinton
Copy link
Contributor

For the final disparity, match points, etc. plots, currently an error is raised if e.g. the DEM cannot be found. This should not be the case and instead the plots that depend on the DEM should simply not be shown and no error should crash the other independent plots:

Don't do this:

except:
raise ValueError("Could not find DEM file in stereo directory")

(cc @ShashankBice)

@bpurinton
Copy link
Contributor Author

More generally, add intelligent handling of any missing files. Scripts and plots should fail gracefully for a User. Perhaps output a plot but just show text on it that says "could not find X file(s)" along with perhaps helpful message depending on the file type of how to generate it.

@bpurinton bpurinton changed the title Add conditionals to show partial subplots Gracefully handle missing files Jul 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant