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

Flowdeck based estimate seems to diverge faster #1329

Open
knmcguire opened this issue Oct 31, 2023 · 5 comments
Open

Flowdeck based estimate seems to diverge faster #1329

knmcguire opened this issue Oct 31, 2023 · 5 comments
Labels

Comments

@knmcguire
Copy link
Member

So with the release testing, I found out that flowdeck based flight is a bit iffy.... It seems to diverge faster than usually!

Some things that worked:

  • moving the flowdeck higher from the ground (if you have some long header pins)
  • Waving the crazyflie to get some flow info before flying.

Still, the general perspective is that it 'used' to work much better than before. There is no black and white error here but it is good to investigate what recent changes caused this.

@knmcguire knmcguire added the bug label Oct 31, 2023
@knmcguire
Copy link
Member Author

Users have started to notice this too, especially when it is still on the ground. We are going to look to this a bit more closer in the next couple of weeks https://github.com/orgs/bitcraze/discussions/1033. First step is to figure out which release it still worked okay

@williamleong
Copy link

Hi @knmcguire, we have been using [2022.12] as you suggested and it seems to have improved flow deck performance (otherwise the Crazyflie just shoots off and doesn't stop on some floor types).

However, I am trying to use the cpxRegisterAppMessageHandler function to receive messages from the AI deck, which seems to be only introduced in [2023.06]. Is there an alternative? I am wondering how it would have been done before the introduction of cpxRegisterAppMessageHandler.

@knmcguire
Copy link
Member Author

@williamleong

Your question about cpx, please ask it on https://discussions.bitcraze.io/. This is out of scope for this ticket, which is here to indicate an issue that needs to be fixed on th flowdeck

@williamleong
Copy link

Apologies, I meant to ask if this would be fixed soon so that we could use the latest version with the flow deck as well as the cpx. In the meantime I've manually merged some code to get it to work.

@knmcguire
Copy link
Member Author

I think the flowdeck based flight still works okay, but there are some circumanstances to keep in mind. I wrote a blogpost about it here https://www.bitcraze.io/2023/11/go-with-the-flow-relative-positioning-with-the-flow-deck/.

CPX question should still be referred to discussions still

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