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

use a validation library #18

Open
asmodehn opened this issue May 16, 2017 · 3 comments
Open

use a validation library #18

asmodehn opened this issue May 16, 2017 · 3 comments

Comments

@asmodehn
Copy link
Member

To validate our data, we currently use custom code.
We should use an existing library when possible.
Ex : https://github.com/alecthomas/voluptuous

@asmodehn
Copy link
Member Author

We could also rely on python3 types, and corresponding tools like mypy

@asmodehn
Copy link
Member Author

asmodehn commented Sep 1, 2017

@asmodehn
Copy link
Member Author

asmodehn commented Sep 1, 2017

maybe we also could have that at a higher level...
The plan so far is to have pyros-msgs expose the same API as ROS but with improvements, and pyros-schemas / marshmallow is probably python specific, since we have an interface of ROS messages as dictionaries...

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

No branches or pull requests

1 participant