left
_revolve splitAtTangents = yes / or _divideAlongCrease
middle
_revolve splitAtTangents = no
right
deleted planar surfaces, _cap
two aspects
(a) additional surface seams result in more complex polysurfaces and case problems for details, projecting, booleans, offsetting, filleting…
(b) surfaces of revolution have unnecessary dense meshes close to the pole.
(not this topic: divideAlongCreases affects meshing / makes it less complex)
for a lot of workflows I show my students to remove the planar section (disk or ring) of a revolve.
and then cap it. the simiple trimmed surfaces behaves more robust for most workflows as the surfaces of revolution.
my wish - allow planar sections in a _revolve to be build / replaced with a cap like surface like above example. (right)
my guess - most users will profit from a cap-like surface, so it might be the default option.
if this is adressed in _revovle it should also be addressed in _railrevolve
technically a completely different topic:
but similar aspect for “toNurbs” in subD …
maybe it would be nice to address both at the same time, this would make communication to the users easier:
“new features: handle planarity more carefully (toNurbs, revolve, railrevolve)”
I’ll add a note about RailRevolve to the linked issue since they belong to the same developer and are related. I’m happy you brought this up. I think we should keep the geometry we make as simple as possible.
Hi Tom -
I’ve pretty much always replaced planar surfaces that resulted from the Revolve command myself in my previous job as an engineer. But that was just because “it felt better” to me.
So I don’t necessarily object to your wish…
However, …
Can you post examples of where this is the case?
If that’s really something that goes on, that should be fixed.
-wim
same for me - at a certain level cad is a craft …
always use protection glasses …
as soon as you add details to the planar section - you have more hassle with the revolved surface.
project a curve will not result in a single line - it will have 2 segements - of course you can _simplifyCrv
setting autocplane will behave different
splolp, FlowAlongSrf, …
