Skip to content

crazyGUI added#592

Open
santiagorg2401 wants to merge 1 commit into
USC-ACTLab:masterfrom
santiagorg2401:crazyGUI_branch
Open

crazyGUI added#592
santiagorg2401 wants to merge 1 commit into
USC-ACTLab:masterfrom
santiagorg2401:crazyGUI_branch

Conversation

@santiagorg2401
Copy link
Copy Markdown

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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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