hzeller / beagleg

G-code interpreter and stepmotor controller for crazy fast coordinated moves of up to 8 steppers. Uses the Programmable Realtime Unit (PRU) of the Beaglebone.
http://beagleg.org/
GNU General Public License v3.0
122 stars 51 forks source link

Added command M181. #66

Closed lromor closed 1 year ago

lromor commented 1 year ago
codecov-commenter commented 1 year ago

Codecov Report

Patch coverage: 100.00% and project coverage change: +1.72 :tada:

Comparison is base (4d55223) 45.93% compared to head (302fd25) 47.66%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #66 +/- ## ========================================== + Coverage 45.93% 47.66% +1.72% ========================================== Files 37 37 Lines 4245 4284 +39 ========================================== + Hits 1950 2042 +92 + Misses 2295 2242 -53 ``` | [Impacted Files](https://codecov.io/gh/hzeller/beagleg/pull/66?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Henner+Zeller) | Coverage Δ | | |---|---|---| | [src/gcode-machine-control.cc](https://codecov.io/gh/hzeller/beagleg/pull/66?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Henner+Zeller#diff-c3JjL2djb2RlLW1hY2hpbmUtY29udHJvbC5jYw==) | `30.41% <100.00%> (+10.30%)` | :arrow_up: | | [src/planner.cc](https://codecov.io/gh/hzeller/beagleg/pull/66?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Henner+Zeller#diff-c3JjL3BsYW5uZXIuY2M=) | `83.74% <100.00%> (+0.79%)` | :arrow_up: | ... and [2 files with indirect coverage changes](https://codecov.io/gh/hzeller/beagleg/pull/66/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Henner+Zeller) Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Henner+Zeller). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Henner+Zeller)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

lromor commented 1 year ago

@hzeller , I've added a planner test to check the lookahead resize behavior. Let me know what you think!

hzeller commented 1 year ago

In the gcode_finished() callback the GCodeMachineControl receives: might be useful to also reset the planner buffer to normal.