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 bindings bug and update #1435

Open
knmcguire opened this issue Nov 12, 2024 · 3 comments
Open

Python bindings bug and update #1435

knmcguire opened this issue Nov 12, 2024 · 3 comments
Labels

Comments

@knmcguire
Copy link
Contributor

The spiral and updated go_to PR (#1410) caused compatibility issues with the Crazyswarm2 with the python bindings.

See this issue: IMRCLab/crazyswarm2#587

We need to update the plan_to function in the python bindings and add the spiral function itself.

Also let's keep an eye on this PR too: #1377. This will also cause incompatibility with Crazyswarm2... maybe it is better to first merge this and then fix both issues together.

@knmcguire knmcguire added the bug label Nov 12, 2024
@knmcguire
Copy link
Contributor Author

knmcguire commented Nov 12, 2024

@matejkarasek just letting you know about this. I was half working on this already for ROSCon but haven't gotten the time to finish it.

@matejkarasek
Copy link
Contributor

Thanks for heads up, and my apologies... Unfortunately I have zero experience with Python bindings and by quickly looking into the code I could not find the pieces of code that need to be updated.
Will keep an eye on this issue to learn and see if I can help resolve it...

@knmcguire
Copy link
Contributor Author

No problem. The current plan is to first merge #1377 and fix the bindings in one PR after that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants