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

Initial implementation of comms watchdog #316

Merged
merged 3 commits into from Mar 19, 2019

Conversation

Projects
None yet
3 participants
@Nomadluap
Copy link
Contributor

commented Mar 6, 2019

  • Added watchdog timeout property to Axis::Config_t:
    axis.config.watchdog_timeout

  • Added axis protocol function to feed watchdog timer:
    axis.watchdog_feed()

  • Axis::run_control_loop now checks for watchdog expiration.

  • Ascii protocol: add support for watchdog

    • The following ASCII commands now automatically update the watchdog:
      p, v, t, c, q

    • Added a 'u' command to update the watchdog of a motor without
      modifying setpoints.

    • Updated ascii protocol documentation to reflect new commands and
      effects.

  • Updated getting started guide to mention watchdog settings and
    functions in protocol.

Please note: due to unavailability of hardware at this time, I have been
unable to test this code on an Odrive.

Initial implementation of comms watchdog
- Added watchdog timeout property to Axis::Config_t:
  axis.config.watchdog_timeout

- Added axis protocol function to feed watchdog timer:
  axis.watchdog_feed()

- Axis::run_control_loop now checks for watchdog expiration.

- Ascii protocol: add support for watchdog

  - The following ASCII commands now automatically update the watchdog:
    p, v, t, c, q

  - Added a 'u' command to update the watchdog of a motor without
    modifying setpoints.

  - Updated ascii protocol documentation to reflect new commands and
    effects.

- Updated getting started guide to mention watchdog settings and
  functions in protocol.

Please note: due to unavailability of hardware at this time, I have been
unable to test this code on an Odrive.
@CLAassistant

This comment has been minimized.

Copy link

commented Mar 6, 2019

CLA assistant check
All committers have signed the CLA.

@madcowswe madcowswe changed the base branch from master to devel Mar 19, 2019

@madcowswe

This comment has been minimized.

Copy link
Owner

commented Mar 19, 2019

Thanks for making this, after adding a fix it works great!

@madcowswe madcowswe merged commit aa08901 into madcowswe:devel Mar 19, 2019

1 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
license/cla Contributor License Agreement is signed.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.