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

Copter: SmartRTL causing Internal Error 100000 #22780

Open
andrefreitas97 opened this issue Jan 30, 2023 · 11 comments · May be fixed by #28284
Open

Copter: SmartRTL causing Internal Error 100000 #22780

andrefreitas97 opened this issue Jan 30, 2023 · 11 comments · May be fixed by #28284

Comments

@andrefreitas97
Copy link

Bug report

Issue details

If I engage SmartRTL with a switch 3 consecutive times while the drone is disarmed I get Internal Error 100000 at the third time. I believe this should be some type of bug.

Version
4.3.3

Platform
[ ] All
[ ] AntennaTracker
[ X ] Copter
[ ] Plane
[ ] Rover
[ ] Submarine

Airframe type
Quad

Hardware type
Kakute H7

@rmackay9 rmackay9 changed the title SmartRTL causing Internal Error 100000 Copter: SmartRTL causing Internal Error 100000 Jan 31, 2023
@shivam969619
Copy link

I am new here. Can i take this issue.

@prathamesh7255
Copy link
Contributor

@andrefreitas97 when it shows the error for the third time does it take time of 10 seconds or does it show the error immediately?

@prathamesh7255
Copy link
Contributor

I am new here. Can I take this issue.

Hello Shivam, I would recommend you start with good first issues.

@prathamesh7255
Copy link
Contributor

@andrefreitas97 when it shows the error for the third time does it take time of 10 seconds or does it show the error immediately?
@rmackay9 if you know, please help.

@andrefreitas97
Copy link
Author

@prathamesh7255 the error shows immediately after the third time I engage SmartRTL

@prathamesh7255
Copy link
Contributor

@andrefreitas97 I think the issue is almost resolved but as I was not able to reproduce the issue on my GCS, would it be fine for you to have an online meeting to clear some doubts? if it's fine, you can mail me the time which suits for you. you will find my email on my profile.

@prathamesh7255
Copy link
Contributor

@andrefreitas97, how many total waypoints were there when engaging smart RTL, three or more?

@andrefreitas97
Copy link
Author

There were none because it had just been turned on on my desk with no gps fix

@prathamesh7255
Copy link
Contributor

There were none because it had just been turned on on my desk with no gps fix

@rmackay9 sir, I think the absence of fixed GPS is causing the error.
What should we do then? as we have several choices:

  1. we can consider adding a check for GPS lock before engaging SmartRTL. If the GPS lock is unavailable, we can either prevent the SmartRTL mode from being engaged or we can warn the user that SmartRTL is not recommended without a GPS lock.
  2. we may also consider implementing a timeout mechanism for GPS lock acquisition, where the SmartRTL mode is automatically disengaged if the GPS lock is not acquired within a certain period of time.

@rmackay9
Copy link
Contributor

rmackay9 commented Mar 9, 2023

I think this issue is a duplicate of #17416

@rmackay9
Copy link
Contributor

rmackay9 commented Mar 9, 2023

This PR could resolve the issue #17419

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

Successfully merging a pull request may close this issue.

4 participants