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

Toggle auto point collecting #5

Merged
merged 1 commit into from
Jul 30, 2024

Conversation

rovo89
Copy link
Contributor

@rovo89 rovo89 commented Jul 30, 2024

Cherry-picked from jkaflik/OpenMowerApp@632e50e

Copy link
Owner

@ClemensElflein ClemensElflein left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

ClemensElflein pushed a commit to ClemensElflein/open_mower_ros that referenced this pull request Jul 30, 2024
…d) (#137)

This supersedes PR #56. The first two commits are the same as over
there, just rebased on the latest main branch and re-formatted with the
new rules.

The slicer has been updated with
ClemensElflein/slic3r_coverage_planner#11 and
#135, so no need
for the fourth commit.

The app enhancements are in
ClemensElflein/OpenMowerApp#5. I didn't include
the rebuilt assets here - let me know if I should do that or if you'll
take care (in case you have special build requirements).

I have recorded my map using these commits and have been mowing it with
the slicer fix for a few weeks, all working very nicely.

---------

Co-authored-by: Kuba Kaflik <kuba.kaflik@clickhouse.com>
@ClemensElflein ClemensElflein merged commit f1bc7c6 into ClemensElflein:main Jul 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants