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

After uploading a "bad" mission, Bad tones (beeps) happen everytime, even when uploading a good mission #9746

Closed
Antiheavy opened this issue Jun 22, 2018 · 9 comments
Assignees

Comments

@Antiheavy
Copy link
Contributor

Antiheavy commented Jun 22, 2018

When the vehicle rejects a mission the first time we hear the "bad" tones from the autopilot beeper (this is expected). Any subsequent uploads of ANY mission (good or bad) the "bad" tones happen.

This is happening on 1.8.0.

I think has been happening for a long time, but we finally just realized it is 100% repeatable. The way we usually see it is we upload a mission with a landing pattern that is too short and it gets rejected.

@dagar dagar added the bug label Jun 22, 2018
@dagar dagar self-assigned this Jun 22, 2018
@dagar
Copy link
Member

dagar commented Jun 22, 2018

Something must be wrong with the logic here.

https://github.com/PX4/Firmware/blob/master/src/modules/commander/commander.cpp#L2069-L2075

Possibly a previous warning that hasn't been cleared.

@Antiheavy
Copy link
Contributor Author

@dagar the tweak we just tried seems to have resolved this issue. Is it worth making a smaller stand-alone PR here in upstream?

@Antiheavy
Copy link
Contributor Author

@dagar Did the fix for this make it upstream?

@dagar
Copy link
Member

dagar commented Aug 13, 2018

No, the more thorough fix is here. #9747

@Antiheavy
Copy link
Contributor Author

@dagar the fix for this bug seems like something that should be part of v1.9.0. thoughts?

@Antiheavy
Copy link
Contributor Author

still valid issue?

@stale stale bot removed the Admin: Wont fix label Jun 25, 2019
@dagar
Copy link
Member

dagar commented Jun 25, 2019

This isn't fixed in master?

@PX4 PX4 deleted a comment from stale bot Jun 25, 2019
@dagar dagar added this to the Release v1.10.0 milestone Jun 25, 2019
@PX4 PX4 deleted a comment from stale bot Sep 25, 2019
@stale stale bot removed the Admin: Wont fix label Sep 25, 2019
@stale
Copy link

stale bot commented Dec 24, 2019

This issue has been automatically marked as stale because it has not had recent activity. Thank you for your contributions.

@stale stale bot added the stale label Dec 24, 2019
@Antiheavy
Copy link
Contributor Author

Yep, this is fixed in Master. closing.

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

No branches or pull requests

2 participants