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

server (cflib) implement full state command #350

Merged
merged 7 commits into from
Dec 14, 2023
Merged

Conversation

knmcguire
Copy link
Collaborator

@knmcguire knmcguire commented Nov 13, 2023

Since of this PR bitcraze/crazyflie-lib-python#382, there is a full state implementation now in cflib. This is not yet included in the latest release so let's keep it in draft for now.

fixes this discussion: #347

This requires cflib install from source: https://www.bitcraze.io/documentation/repository/crazyflie-lib-python/master/installation/install/#install-from-source

@knmcguire knmcguire changed the title Update crazyflie_server.py server (cflib) implement full state Nov 14, 2023
@knmcguire knmcguire changed the title server (cflib) implement full state server (cflib) implement full state command Nov 14, 2023
@knmcguire knmcguire marked this pull request as ready for review December 13, 2023 08:30
@knmcguire
Copy link
Collaborator Author

Since the release is out of the cflib, we can go ahead and merge this!

Copy link

@whoenig whoenig left a comment

Choose a reason for hiding this comment

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

Cool!

crazyflie/scripts/crazyflie_server.py Outdated Show resolved Hide resolved
docs2/overview.rst Outdated Show resolved Hide resolved
@knmcguire
Copy link
Collaborator Author

I'll wait for the CI to finish checking and then I'll merge!

@knmcguire knmcguire merged commit 735f625 into main Dec 14, 2023
3 checks passed
@knmcguire knmcguire deleted the cflib-full-state-cmd branch December 14, 2023 13:16
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