Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Adding error parsing as per TODO#7

Open
williammlu wants to merge 2 commits into
neomanic:masterfrom
williammlu:dump_error2
Open

Adding error parsing as per TODO#7
williammlu wants to merge 2 commits into
neomanic:masterfrom
williammlu:dump_error2

Conversation

@williammlu

@williammlu williammlu commented Apr 16, 2019

Copy link
Copy Markdown

Removing old logic and replacing with most recent version of odrivetool's dump_error()

Example python interactive script runs:

>>> od.get_errors(clear=True)
Axis0:
  axis: no error
  motor: no error
  encoder: no error
  controller: no error
Axis1:
  axis: no error
  motor: no error
  encoder: no error
  controller: no error


>>> od.calibrate(
... )
Vbus 48.33V
Calibrating axis 0...
Calibrating axis 1...
Failed calibration with axis error 0x40, motor error 0x1
False
>>> od.get_errors()
Axis0:
  axis: no error
  motor: no error
  encoder: no error
  controller: no error
Axis1:
  axis: Error(s):
    ERROR_MOTOR_FAILED
  motor: Error(s):
    ERROR_PHASE_RESISTANCE_OUT_OF_RANGE
  encoder: no error
  controller: no error
>>> od.preroll

@neomanic

neomanic commented Apr 21, 2019

Copy link
Copy Markdown
Owner

Hi William, thanks for this. I'm on leave at the moment so I won't have a chance to look for another week or two. The only issue at the moment is that I'm in the slow process of upgrading from an older firmware (which doesn't support dump_errors()) across our robots, so I can't rely on that function being present just yet.

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@williammlu @neomanic