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

Refactor Python examples with a new "util.py" file #347

Merged
merged 3 commits into from
Aug 14, 2024

Conversation

ManifoldFR
Copy link
Member

@ManifoldFR ManifoldFR commented Aug 13, 2024

This PR refactors a bunch of Python examples to use a function from the same util.py file

This helps with my draft PR #345 but should have value in itself

@hrp2-14
Copy link
Member

hrp2-14 commented Aug 13, 2024

Hi ! This project doesn't usually accept pull requests on the main branch.
If this wasn't intentionnal, you can change the base branch of this PR to devel
(No need to close it for that). Best, a bot.

@ManifoldFR ManifoldFR changed the base branch from main to devel August 13, 2024 13:53
Copy link
Collaborator

@fabinsch fabinsch left a comment

Choose a reason for hiding this comment

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

good idea @ManifoldFR , thanks for the PR. I think we should address the reg and its good to merge.

examples/python/util.py Outdated Show resolved Hide resolved
fabinsch
fabinsch previously approved these changes Aug 13, 2024
fabinsch
fabinsch previously approved these changes Aug 13, 2024
examples/python/util.py Outdated Show resolved Hide resolved
@jcarpent jcarpent merged commit 315f4d3 into Simple-Robotics:devel Aug 14, 2024
74 of 78 checks passed
@ManifoldFR ManifoldFR deleted the topic/refactor-py-examples branch August 15, 2024 21:10
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.

4 participants