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

crazyGUI added #592

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

santiagorg2401
Copy link

Hey, I made a GUI for the Crazyswarm repository, it's still under development but it can already be used and it'll save you a lot of terminal usage.

@whoenig
Copy link
Contributor

whoenig commented Feb 17, 2022

Thanks for your contribution! It is difficult to add/merge/review this, since you only point to a submodule. I think the code should either be part of this repository, or you need at least add some documentation on how your GUI is supposed to be started/used.

@knmcguire
Copy link
Contributor

knmcguire commented Feb 24, 2022

I tried to gui myself out as well and it indeed has some great elements to it! But currently it is hardcoded that the crazyswarm folder needs to be in a crazyflie folder in home, and I noticed also other bugs as well.

The idea about the cfclient in this tool is interesting, but perhaps the cfclient should show the state of the crazyflie as connected to the crazyflieserver, which is not something it can do yet... perhaps it should be a ros package instead?

I like the idea! but I do agree with wolfgang that the gui should be part of the crazyswarm repo directly.

@santiagorg2401
Copy link
Author

Alright I'll try better and post some results when they're ready, thank you.

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.

3 participants