upkie / vulp

Robot/simulation switch for the mjbots stack
Apache License 2.0
61 stars 4 forks source link

Update palimpsest to 2.0.0 #62

Closed stephane-caron closed 8 months ago

stephane-caron commented 8 months ago

Although this PR only updates palimpsest to version 2, there is a major change in that version that directly impacts spine behaviors:

As a consequence, we will be able to clear out asyncio loggers on the Python side, leaving the spine as the single source of truth for logs.

Performance regression check

foxplot([data.spine.clock.measured_period, data.spine.clock.slack], right=[data.spine.clock.skip_count])

Before

2023-10-16_185356_pi3hat_spine.mpack

image

After

2023-10-16_190040_pi3hat_spine.mpack

image

coveralls commented 8 months ago

Pull Request Test Coverage Report for Build 6537019066


Changes Missing Coverage Covered Lines Changed/Added Lines %
vulp/spine/Spine.cpp 11 14 78.57%
<!-- Total: 21 24 87.5% -->
Totals Coverage Status
Change from base Build 6420685880: 0.006%
Covered Lines: 1654
Relevant Lines: 1882

💛 - Coveralls