Hi, I appear to have run into a bug in AddRevSurf. Here is how to reproduce: open badRevolveExample.3dm (28.1 KB). Then run the following script, which creates (what should be) the same revolved surface via a macro and via AddRevSrf, and a simple sphere. Now if you try to eg BooleanSplit the spher…

Interesting. SelDup selects the two revolved surfaces as identical, but Properties>Details says that the surface created by AddRevSrf is a “surface of revolution” and the surface created by the macro is a “rational NURBS surface”. The intersector finds a valid intersection curve in both cases, but …

Rhino’s Revolve command creates Nurbs surface due to some difficulty in calculating closest points on surfaces of revolution. The rs.AddRevSrf function should do the same - I’ve reported this as a bug. This version of rs.AddRevSrf should work better for you. import math import Rhino import rhinosc…

@ZmeiGorynych , the problem in your link seems to be fixed already. Do you see this not working on trimmed surfaces ? Below example is identical to the code rs syntax is using: import Rhino import rhinoscriptsyntax as rs import scriptcontext def OffsetSrf(distance, tolerance=None, both_sides=False,…

@clement , here is the issue I see, working with badOffsetExample.3dm (475.5 KB) The call to OffsetSurface returns None, while the macro works fine. I got Rhino off the website less than a month ago, so the code there should be up to date. import rhinoscriptsyntax as rs srf=rs.ObjectsByName("tes…

@ZmeiGorynych , is understand now. You`re offsettings a polysurface instead of a surface. Usually this is done using rs.OffsetBrep() but this is not implemented yet in RhinoScript syntax. c.

If you are using the Rhino WIP, you can write your own version of RhinoScript’s OffsetBrep method by calling Rhino.Geometry.Brep.CreateOffsetBrep.

Dale, thanks! How can I download the WIP? I found this link , but when I click on the big Download button there, I end up here , where I can’t see any link to the WIP. Best, E.

A bug in AddRevSrf?

Scripting

ZmeiGorynych October 31, 2015, 12:49pm 4

Dale,

thanks a lot, that one works!

Would you by any chance also have a code snippet to fix this issue? Ran into it myself and also having to workaround via a macro…

Thanks a lot,
E.

Topic		Replies	Views
Latest Rhino6 WIP seems to have fixed AddRevSrf problem Serengeti (Rhino BETA)	1	562	April 25, 2017
Wish new Python function AddRailRevSrf like VbScript Scripting	8	1745	October 16, 2014
CreateRailRevolvedSurface Replay Turns PolySrf Into Srf Rhino Developer windows	4	181	June 26, 2024
Surface of revolution objects--can't Boolean split Serengeti (Rhino BETA)	12	1743	October 10, 2017
ObjectTable.AddSurface() can add multiple surfaces Scripting	6	1341	August 6, 2016

A bug in AddRevSrf?

Related topics