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

[Firmware] Movements retry too many times #990

Open
gabrielburnworth opened this issue Sep 19, 2019 · 0 comments
Open

[Firmware] Movements retry too many times #990

gabrielburnworth opened this issue Sep 19, 2019 · 0 comments

Comments

@gabrielburnworth
Copy link
Contributor

FarmBot OS version

8.0.2

Behavior

Movements are currently attempted n^2 + n times, where n is the number of retries selected by the user. FarmBot OS attempts a movement n + 1 times, and every time FarmBot OS tries a movement, the firmware attempts the movement n times.

Expected

The movement retry mechanism should only be implemented in (one of) the firmware or FarmBot OS.

Relevant code

do_move_absolute(x, y, z, speed, retries - 1, [reason | errors])

@ConnorRigby ConnorRigby changed the title Movements retry too many times [Firmware] Movements retry too many times Oct 31, 2019
@roryaronson roryaronson moved this from Bugs to To do in FarmBot Project Board Feb 7, 2020
RickCarlino added a commit that referenced this issue Feb 8, 2020
@roryaronson roryaronson moved this from To do to QA'd, Ready to Deploy in FarmBot Project Board Feb 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
FarmBot Project Board
QA'd, Ready to Deploy
Development

Successfully merging a pull request may close this issue.

1 participant