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

[python] Update files in ros2controlcli to use format strings #358

Merged
merged 17 commits into from
Apr 29, 2021
Merged

[python] Update files in ros2controlcli to use format strings #358

merged 17 commits into from
Apr 29, 2021

Conversation

NovusEdge
Copy link
Contributor

@NovusEdge NovusEdge commented Mar 30, 2021

Fixes #355

@bmagyar
Please be sure to review the changes and let me know if something is wrong; or if more changes are to be made.

@NovusEdge NovusEdge changed the title Updated format strings Update files in ros2controlcli to use format strings Mar 30, 2021
@NovusEdge NovusEdge changed the title Update files in ros2controlcli to use format strings [python] Update files in ros2controlcli to use format strings Mar 31, 2021
@destogl destogl mentioned this pull request Apr 2, 2021
@destogl
Copy link
Member

destogl commented Apr 2, 2021

@NovusEdge This looks great thanks! Did you also test it?

@NovusEdge
Copy link
Contributor Author

NovusEdge commented Apr 2, 2021

@destogl
I'm afraid i didn't, do you mind running the tests and letting me know if there's any problem?

@destogl
Copy link
Member

destogl commented Apr 2, 2021

@NovusEdge I am guessing you are new in ros2_control and would like to learn more about the framework. Am I right?

If so, I propose to check out ros-controls/ros2_control_demos and test the call using rrbot demo hardware. Also, before doing anything further with this PR, you should check these errors with linters.

@destogl destogl self-requested a review April 2, 2021 18:42
destogl
destogl previously requested changes Apr 2, 2021
Copy link
Member

@destogl destogl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As per comment. Also, check the errors with linters. You can also execute them locally using ament_* commands.

@Karsten1987
Copy link
Contributor

@NovusEdge friendly ping. Are you still working on this?

@NovusEdge
Copy link
Contributor Author

NovusEdge commented Apr 26, 2021

@NovusEdge friendly ping. Are you still working on this?

I currently can't work on it. So, no.

@bmagyar bmagyar merged commit c0824d7 into ros-controls:master Apr 29, 2021
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.

use format strings for ros2controlcli
4 participants