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/Object Base Additional XY Distance #2619

Closed

Conversation

lovelytwo
Copy link
Contributor

@lovelytwo lovelytwo commented Nov 5, 2023

This pr, adds a new support option that allows for the base material between interface layers to have a greater xy offset than the xy offset used for interface layers.

This solves a problem I have with printing sloped overhangs that have low gradients. In this case the support/object xy distance pushes the support interface layer away futher than the top z distance.

Ideally in this case i want a small support/object xy distance, however this causes problems with the base support material getting too close the object.

So i've added a simple fix that allows you to specify an additional base support material offset.

Interface layers are printed with "support/object xy distance".
Base support layers are printed with "support/object xy distance" + "support/object base additional xy distance".

I think this implementation is far from ideal, and is more of quick hack.

Additional this will allow zero/small support/object xy distances when using a support filament for interface only, and preventing the base from getting too close (this might be handled differently already as i haven't tested support filaments but i have seen others request the following Cura option, so i assume it's a problem).

Cura has a support distance priority option, that would probably be better but i have zero clue how to implement that.

Any feed back is greately appreciated.

@SoftFever
Copy link
Owner

@lovelytwo
Thanks for the PR and sorry for delayed response.
Do you mind to attach a sample project?
Some photos of the prints to show case the before and after would be perfect.

@@ -729,6 +729,7 @@ PRINT_CONFIG_CLASS_DEFINE(
// Overhang angle threshold.
((ConfigOptionInt, support_threshold_angle))
((ConfigOptionFloat, support_object_xy_distance))
((ConfigOptionFloat, support_object_base_additional_xy_distance))
Copy link
Owner

Choose a reason for hiding this comment

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

It would be better to define it as parameter: support_object_top_interface_xy_distance
So top contacts us this parameter while other support layers use support_object_xy_distance

@@ -3274,7 +3275,8 @@ void PrintObjectSupportMaterial::generate_base_layers(
++ iRun;
#endif /* SLIC3R_DEBUG */

this->trim_support_layers_by_object(object, intermediate_layers, m_slicing_params.gap_support_object, m_slicing_params.gap_object_support, m_support_params.gap_xy);
this->trim_support_layers_by_object(object, intermediate_layers, m_slicing_params.gap_support_object, m_slicing_params.gap_object_support, m_support_params.gap_xy + m_support_params.base_additional_gap_xy);
Copy link
Owner

Choose a reason for hiding this comment

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

Does it work for normal support only? Or it affects tree/organic support too?

@lovelytwo
Copy link
Contributor Author

Sorry for the long delay, christmas holidays.

I will have a look at the comments and suggested changes.

@SoftFever
Copy link
Owner

Sorry for the long delay, christmas holidays.

I will have a look at the comments and suggested changes.

Thank you very much.
Take your time :)

@SoftFever
Copy link
Owner

Seems this PR has gone stale.
I will temporarily close it as I'm trying to keep the pending PR list short.
Feel free to reopen it when you think it's ready

@SoftFever SoftFever closed this Jan 27, 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