Skip to content
Snippets Groups Projects

Resolve "components/RobotUnitSimulation: Wrong timestamps in 'RobotState/Proprioception'-memory"

Merged Patrick Dormanns requested to merge fix/robotunit-timestamps into master
All threads resolved!

Fixes components/RobotUnitSimulation: Wrong timestamp... (#26 - closed)

Fixes the issue by setting the timestamps using the system's raw monotonic clock with armarx::rtNow(), as expected by the used RobotAPI code.

This is inconsistent with the rest of the RobotUnitSimulation-component, which exclusively uses the system's realtime or the time provided by a timeserver, but consistent with the timestamps set by the real-robot units.

I don't expect this to break anything (at least as long as the timeserver-functionality isn't used), as this change just adapts the behavior of the simulated RobotUnits to match the (correct) behavior of the real-robot RobotUnits.

Edited by Patrick Dormanns

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Christian Dreher resolved all threads

    resolved all threads

  • added 1 commit

    • 66262f81 - Add note about why changes with armarx::rtNow where necessary

    Compare with previous version

  • Great, thanks a lot!

  • Christian Dreher approved this merge request

    approved this merge request

  • Christian Dreher mentioned in commit 165457f2

    mentioned in commit 165457f2

  • Please register or sign in to reply
    Loading