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

Arrange room for skirt and brim #9181

Conversation

individ-divided
Copy link

@individ-divided individ-divided commented Nov 19, 2022

Hello

I've changed the Arrange function so that it makes sure that any skirt or brim added also fits the bed. No longer does one press the arrange button and then after one slices, the skirt/brim is outside the bed!

brim-demo

I've contained the libnest2d changes so that they are backwards compatible with earlier versions, to make it easier to merge with upstream.

I believe these changes fixes #3477 and #3072

I wasn't aware of #7653 until a couple of minutes ago. The major difference is that these changes I made are capable of handling individual brim settings, per object. I also found out that thread safety issue (but on my own, why didn't I read this earlier!!) and implemented the flow calculations directly from the configuration, much like bubnikv suggested here

Best,
David

…bject)

Make the arrange function place objects so that the skirt and all the
brims fit inside the bed.
I can't figure out how to make fff_print valid here so I decided to
just reimplement the flow calculation.
@Eldenroot
Copy link

Cool!

@rlesac
Copy link

rlesac commented Mar 30, 2023

That's super useful! Hope it gets merged soon!

@individ-divided
Copy link
Author

@rlesac Glad you like it. Unfortunately I haven't heard from Prusa at all. It has been months. I guess they are busy with other things. I'll just keep using it myself. shrug

@individ-divided
Copy link
Author

@tamasmeszaros Is this something you'd like to have in Prusa Slicer? I don't want to impose, but you seem to be the guy to ask about the arrange side of things.

@jmd73
Copy link
Contributor

jmd73 commented Apr 4, 2023

This is from the releasenotes for 2.6.0 alpha 6:
'Arrange' and 'Fill bed with instances' functions now take into account skirt and brim separation (#3072, #3477, #5286, #5310, #9301, #9309). Many thanks to @jschuh (#7653) and @individ-divided (#9181), whose pull requests were very useful sources of inspiration, despite not being used verbatim.

@individ-divided
Copy link
Author

@jmd73 Thank you for letting me know, I wasn't aware of this!

@tamasmeszaros
Copy link
Contributor

Hi @individ-divided!

as @jmd73 stated, this feature will be shipped with the next PS release, many thanks for your contributions. I will close this PR as unmerged, as there is no need for an explicit merge. I've tagged you alongside with @jschuh in the appropriate commits where your changes were used (directly or indirectly).

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.

None yet

5 participants