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

[FEATURE] Configurable Z-Rotate on import #10651

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mjonuschat
Copy link
Contributor

Automatically rotate objects on th Z axis by a configurable amount (typically 45°) on import.

Rotating objects by 45° often allows better (automatic) seam positioning in combination with the "rear" seam position, improving layer consistency. CoreXY printers might have the additional benefit that 45° movements only utilize one motor which can lead to better quality on straight walls (VFAs when both motors are in use).

@foreachthing
Copy link

CoreXY printers might have the additional benefit that 45° movements only utilize one motor ...

Our Ultimaker 2 uses two steppers for 45°. I don't see any benefit from this PR.

Can"t you use the rotate function in PS to rotate your models? Or use the CLI...

@mjonuschat
Copy link
Contributor Author

Can"t you use the rotate function in PS to rotate your models?

Absolutely, but it is much more convenient to make it happen automatically instead of repeating a manual process over and over again for every single object or plate.

@foreachthing
Copy link

If you make it optional, I don't have anything against it 😉

@mjonuschat
Copy link
Contributor Author

It’s optional and configurable per printer, default to zero degrees. So no change from previous versions unless explicitly opted in.

@mjonuschat mjonuschat changed the title [FEATURE] Z-Rotate on import [FEATURE] Configurable Z-Rotate on import May 24, 2023
@foreachthing
Copy link

I just noticed, that SuperSlicer has that function! Even in 2.4.58.5;
image

@mjonuschat mjonuschat force-pushed the feature-initial-rotation branch from 4f57308 to cc8d341 Compare February 19, 2024 08:08
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