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

mavlogdump: show clear error message when mat_file arg is missing #928

Merged
merged 1 commit into from
Mar 15, 2024

Conversation

shancock884
Copy link
Contributor

I was trying to use mavlogdump to generate a .mat file from a .bin file, and forgot to specify the --mat_file argument. I found the exception thrown by the call to scipy.io.savemat was not very clear as to my mistake.
This PR adds a check before processing the input file that the required argument is specified, and gives the user a clearer message as follows:
mat_file argument must be specified when mat format is selected

tools/mavlogdump.py Show resolved Hide resolved
tools/mavlogdump.py Outdated Show resolved Hide resolved
@peterbarker peterbarker merged commit 636a733 into ArduPilot:master Mar 15, 2024
12 checks passed
@shancock884 shancock884 deleted the mat_file-arg-missing-msg branch March 15, 2024 07:54
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 this pull request may close these issues.

2 participants