-
Notifications
You must be signed in to change notification settings - Fork 81
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
Improve error message when there is a missing schema #376
Comments
Just a heads up that its my first time commenting on an issue so if things are out of place, that's why and please point me in the right direction :) I'm facing a similar problem. I saw that issue #375 was closed, where it was recommended to use I've tried using the I can't find any changes since these issue that makes it possible/easier to point to message definitions, do you have any update or idea on if this is possible? |
@martijnhabers thanks for reporting. We were actually having an internal discussion last week about making this better, so this is timely. First, hopefully some advice to get it working. The Second, what we're discussing internally is how we can make the schema detection behavior smarter, including whether we need to mandate a particular directory layout or whether there is a common one users are ending up with that we should support. Would you be able to share a sanitized example of the layout of the directory you are working with? |
Hi, thanks for the reply, haven't really been working on it this past month but now I want to see if I can get it working. A simplified structure we have implemented for custom messages used in our workspace looks like this:
I have tried to use what you mentioned and mimic the directory structure in opt/ros/foxy
However, when running the command Hope this clears things up a bit. |
When running mcap convert, if there is a missing message schema, I get:
This error should tell me what to do next to fix the issue. Ideally related to #375 so I can just point it at the missing message file.
The text was updated successfully, but these errors were encountered: