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 Request: Brim only on inside #2853

Closed
Drayson80 opened this issue Nov 26, 2017 · 43 comments
Closed

Feature Request: Brim only on inside #2853

Drayson80 opened this issue Nov 26, 2017 · 43 comments
Labels
Status: On Backlog The issue / feature has been reproduced and is deemed important enough to be fixed. Type: Improvement Improvement to existing functionality.

Comments

@Drayson80
Copy link

As we have the possibility to place "Brim only on outside", it would be great to have the opposite too...
Currently printing an RC plane, would make the life easier in glueing the parts together...

@Drayson80 Drayson80 changed the title Feature Request: Brin only inside Feature Request: Brim only on inside Nov 26, 2017
@Ando120
Copy link

Ando120 commented Nov 27, 2017

Or maybe width setting for inside brim. I need only small inside brim+ large outside brim

@Ghostkeeper
Copy link
Collaborator

So if you load an ordinary model which has no holes in the bottom layer you just see no brim at all. I'm putting it up in our issue planner but we have to consider usability a bit, because it might be unexpected to a user that just selects "Brim" but accidentally enabled this feature that no brim appears.

@Ghostkeeper Ghostkeeper added component: CuraEngine Type: Improvement Improvement to existing functionality. labels Dec 4, 2017
@Drayson80
Copy link
Author

Of course, it depeds on the model. if it has a solid bottom, then it is useless... but if something like a tube or a single layer wall print is required, then it is better to have a brim on the inside than outside. (easier to remove via a deburring tool than on outside).
Example: I need good adhesion AND a good outer surface at a single perimeter wall print of an RC plane (3dlabprint). Without brim, the print has limited adhesion, with brim, I have issues at cleanup and gluing the parts together...

@nickthetait
Copy link

Rather than a sub setting of brim, what if these were three different adhesion strategies? Such as:
Brim (interior)
Brim (exterior)
Brim (both)

@Ando120
Copy link

Ando120 commented Dec 5, 2017

Sub setting for internal brim width will give all options (external or internal width can be 0 )

Materials with big thermal expansions need large external and small internal brim.

@Ghostkeeper
Copy link
Collaborator

Our project manager removed this enhancement from our planning because it was not considered important enough.

@Ghostkeeper Ghostkeeper added the Status: Deferred We don't have time to work on this for now but intend to in the future. label Feb 28, 2018
@GWRon
Copy link

GWRon commented Oct 25, 2018

So how to achieve the desired result: clean outside with adhesive brim on the inside? And nope, adding a thin 3D-model s simulated "brim" is not a convenient solution.

Cura adds so much stuff I never used and this "easy" feature (as the opposite exists already) is "not important enough"? Laughable...

@Feidias04
Copy link

"Brim only on inside" - I need this option badly now. Because I would like to use "Spiralize Outer Contour" so I cannot prepare model with my CAD. When I import this model and I would like to use "Spiralize Outer Contour", this bottom part isn't printed. But I would like to use this part - brim on inside for gluing parts together

@ghost
Copy link

ghost commented Sep 14, 2019

WE ALL NEED IT.

@retif
Copy link

retif commented Apr 1, 2020

i need an option to turn on brim from both sides (for thin parts)

@Ghostkeeper
Copy link
Collaborator

The option to turn the brim on for both the outside and the inside already exists: Brim Only on Outside. There is no option to turn that on automatically for thin parts, because something being thin is kind of ambiguous. It's not a per-object setting either since the brim is shared between all objects within a mesh group.

@hazedav
Copy link

hazedav commented Dec 12, 2020

Bumping this. It definitely makes way more sense to be able to have inside-only, outside-only, both pulldown for brim instead of a checkbox for "outside only". Wouldn't inside-only be just as important as outside-only depending on the model? I'm printing some wall switch covers and am definitely in need of some inside-only right now.

@gloomytree
Copy link

I'd like this, too. My usecase at the moment: I have a model where it is really easy to remove the brim from the inside. The outside features a complex geometry, where it is really tedious to remove the brim.

@yet-another-average-joe
Copy link

yet-another-average-joe commented Apr 27, 2021

Here's a typical example of a part that would benefit from a inside only brim:

Focus Gear

Printing upside down (gear up) is not an option here, and a brim on outside would be a mess (80 teeth...)

Creating the inside brim at CAD level + concentric 1st layer is not the same as a brim.

@TCandee
Copy link

TCandee commented May 7, 2021

I have a very similar situation as "yet-another-average-joe". Difficult to clean the brim from the teeth. Please enable this function.

@aaronse
Copy link

aaronse commented Jun 11, 2021

+1 Given inside-outside and outside are options, hopefully this isn't too hard to implement.

@LutzFassl
Copy link

LutzFassl commented Oct 3, 2021

I need this too! I am maxing out the print bed with a thin part. So I need brim for adhesion, but cannot print on the outside --> PROBLEM. Need Brim Inside Only! Please

@Ghostkeeper you said you might pick this up again? Do you know when? Thanks!!

@Ghostkeeper
Copy link
Collaborator

@Ghostkeeper you said you might pick this up again? Do you know when? Thanks!!

I put this on our backlog in 2017. In 2018 it was removed from our backlog by our project manager because it wasn't deemed important enough.

@LutzFassl
Copy link

@Ghostkeeper
Thanks for your answer! I guess the simple workaround is creating the inner brim in CAD if required, so the feature missing is not the biggest of deals. BR, Lutz

@yet-another-average-joe

@Ghostkeeper Thanks for your answer! I guess the simple workaround is creating the inner brim in CAD if required, so the feature missing is not the biggest of deals. BR, Lutz

Not the same results ! An inside brim should print from inner to outer contours. A "CAD brim" prints as walls with infill. It is very different, and not as efficient as a real brim. And you do not benefit from "Brim distance", making removal more difficult.

@bechhansen
Copy link

I have often needed this and today I need it again.

@LukasMueller187
Copy link

I'd like this feature too.

@psiberfunk
Copy link

Also need this! @Ghostkeeper : maybe your project/product manager could reconsider.. I would imagine it would be relatively easy to implement if you've already got the outside only function...

@davidlip-cura
Copy link

Add me to the list of people wanting inside only brim. The argument to use CAD to add it can also apply to any brim or raft so if inside only is not important, then any brim could be considered not important.

Inside only would make cleanup of many parts I make so much easier. On some parts I can leave the inside brim.

@Fredyy90
Copy link

A vital feature that is still missing from cura. Any updates on an ETA?

@1knueller
Copy link

hello! id like to print something round with a thin, rough shell. it has to look good on the outside. brim only on inside would be an amazing feature.

@smartavionics
Copy link
Contributor

It's trivial to implement and will be in my next release (4.20.6). Check https://github.com/smartavionics/Cura/releases now and again.

Screenshot_2022-10-02_22-34-32

@yet-another-average-joe
Copy link

@smartavionics : do you plan Arachnid integration for your branch in a near future ? (Arachnid was a game changer for what I do ; I can't use your branch anymore :( )

@smartavionics
Copy link
Contributor

smartavionics commented Oct 2, 2022

Sorry, no. That's not going to happen.

PS - please everyone, don't spam this issue with any further comments related to my builds, for that go to https://github.com/smartavionics/Cura and open or contribute to an issue or discussion there, thanks.

And what happens? Some (expletive deleted) immediately spams this issue.

@psiberfunk
Copy link

psiberfunk commented Oct 2, 2022

Sorry, no. That's not going to happen.

PS - please everyone, don't spam this issue with any further comments related to my builds, for that go to https://github.com/smartavionics/Cura and open or contribute to an issue or discussion there, thanks.

Would you be able to submit a pull request to the Cura team to take it up though? That's not related to your build exactly, more like.. if you've developed this, it would be great if you could ask them to bring it in upstream :). Alternatively, it's possible your code wouldnt merge cleanly if it's based off Cura 4 and not 5?

@aaronse
Copy link

aaronse commented Oct 2, 2022

@Someone familiar with Cura and CuraEngine (I'm not) that's interested in putting a pull request together:

Cura edit to Implement brim_inside_only smartavionics@e7c965c

CuraEngine edit to Implement brim_inside_only smartavionics/CuraEngine@56d86b7

@aaronse
Copy link

aaronse commented Oct 3, 2022

Noticed @BagelOrb submitted pull request with very significant rewrite/improvements to Brim #11678 maybe brim_inside_only will appear in, or after, that merges.

@MariMakes MariMakes added Status: On Backlog The issue / feature has been reproduced and is deemed important enough to be fixed. and removed Status: Deferred We don't have time to work on this for now but intend to in the future. labels Nov 2, 2022
@MariMakes
Copy link
Contributor

@aaronse is right, it's easier for us to implement this now the PR from @BagelOrb is merged.

We've added a ticket to the backlog with the intent to enable this.
For internal reference CURA-9838

Thanks for the report! 👍

@aaronse
Copy link

aaronse commented Nov 2, 2022

Thank you the update and info @MariMakes

Couldn't access CURA-9838 details, guessing jira item details are accessible to just Cura folks.

Hopefully the dev task description references smartavionics's change lists that seem to have implemented this feature already in their fork #2853 (comment) . Hopefully dev estimate and risk are low enough this'll turn up soon.

Cheers!

@poldim
Copy link

poldim commented Mar 13, 2023

It looks like this is still not implemented in 5.3.0. Can anyone with access to Jira advise when this might be added?

image

@MariMakes
Copy link
Contributor

Hey @poldim,

I took a look for you.
The contribution we referred to earlier has been merged in Cura 5.3 so it should be easier to implement this feature.

I'll flag it to the team again to increase the priority👍

@aaronse
Copy link

aaronse commented Mar 14, 2023

Hello @MariMakes, cheers for the update!

Looks like this was first requested on on Nov 26, 2017, which probably seems like a lifetime ago for all of us. Is there something we can do to motivate the team to get this done? Where are your Devs/Office located?

If they're in the Seattle area I promise to drop off a 6-pack when this is done. Maybe someone will offer something similar if you're in a diff city, e.g. NYC.

@DP-M
Copy link

DP-M commented Jan 9, 2024

It would sure be great to see this. Add me to the long list of people who have a thin walled part, which needs to have a clean outside. I'm using 5.6 and the option is not available on this release. @MariMakes any idea where this is in the pipeline? Thanks!

@rff255
Copy link

rff255 commented Jan 22, 2024

I second this request. I have the very same need, for a thin tool with small marks on the outside:
image

@ppriest
Copy link

ppriest commented Feb 5, 2024

I just encountered a case where this would have been useful - an objects sloping up from the outside, which an external brim usually makes even more visible when removed, but a hollow inside.
For the hopefully small amount of development and maintenance effort required (given inside brim is already supported as path of the !'Brim Only on the Outside', I reckon this is a win...

@aaronse
Copy link

aaronse commented Feb 5, 2024

Happy New Year Everyone!

@MariMakes, due to inflation, and ludicrous interest rates, I will raise my bounty from 6 beers, to 7 beers.

There's at least one code fix mentioned in this topic already, and the offer of beer. Befuddled as to why https://ultimaker.atlassian.net/browse/CURA-9838 is not resolved.

Would be nice if bug database for open source project was openly shared, read access at least so community would have more context about what's happening. The context risks more questions, but it could also inspire more contributions too.

Ideally Customers could +1 up vote feature requests like this here, or on UserVoice Forum or something similar.

@nallath
Copy link
Member

nallath commented Apr 5, 2024

Implemented in 5.7 :D

@nallath nallath closed this as completed Apr 5, 2024
@LutzFassl
Copy link

Thanks ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: On Backlog The issue / feature has been reproduced and is deemed important enough to be fixed. Type: Improvement Improvement to existing functionality.
Projects
None yet
Development

No branches or pull requests