Skip to content

Parameters support #7

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

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

Parameters support #7

wants to merge 16 commits into from

Conversation

ipa-nhg
Copy link
Contributor

@ipa-nhg ipa-nhg commented Mar 20, 2020

  • Added the support for parameters
  • Add a second service call to return not only the ros model but also the rossystem model
  • Fix minor grammar and syntax errors for the java snapshot model

ToDo:
- Test
- Clean the history

@ipa-led I just open the PR to don't forget which is my current working version. I want to still do a couple of tests and clean the history

@ipa-nhg
Copy link
Contributor Author

ipa-nhg commented Mar 20, 2020

@ipa-hsd FYI

@ipa-nhg ipa-nhg changed the title [DO NOT MERGE] Parameters support Parameters support Mar 24, 2020
@ipa-nhg
Copy link
Contributor Author

ipa-nhg commented Mar 24, 2020

Tested for the cob4 simulation and on the new V1.2.1 if the ROS tooling and working! 👍

hsd-dev pushed a commit to hsd-dev/ros_graph_parser that referenced this pull request Apr 13, 2021
update the python API to support the new models format
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