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

Support X and Y axis flip, mk. 2 #223

Merged
merged 2 commits into from
Apr 19, 2024
Merged

Conversation

TheAssassin
Copy link
Contributor

Adds another commit to #206 to fix the calculation of the bounding box.

kkaempf and others added 2 commits December 27, 2023 19:15
Signed-off-by: Klaus Kämpf <kkaempf@gmail.com>
Fixes oversight in previous commit.
@mgmax
Copy link
Collaborator

mgmax commented Apr 19, 2024

Looks good, please let me know if everything works correctly (cut and engrave match, engrave image not mirrored, ...).

@mgmax mgmax mentioned this pull request Apr 19, 2024
@TheAssassin
Copy link
Contributor Author

I tested all of this on an OMTech Polar. I could send pictures but I think they wouldn't prove anything (could be mirrored afterwards or so).

@mgmax mgmax merged commit e6bae6f into t-oster:master Apr 19, 2024
1 check passed
@mgmax
Copy link
Collaborator

mgmax commented Apr 19, 2024

That's good. I just asked because recently we had some PRs by someone else where it turned out that the code doesn't work in real life.

@TheAssassin TheAssassin deleted the issue-198 branch April 19, 2024 20:30
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.

3 participants