topologicpy icon indicating copy to clipboard operation
topologicpy copied to clipboard

Memory corrupt error in 7.15

Open martinvrielink opened this issue 1 year ago • 2 comments

Hi Wassim,

By updating to topologicpy 7.15 i encountered a "memory error" error in dynamo. This error accurs after getting a list of faces and then using Topology.Geometry. I do this to reconstruct the faces in Dynamo: Schermafbeelding 2024-06-12 093354

I went ahead to recreating this issue in jupyter notebook to see if i could find out where the error comes from. Thus far I can get the vertices from the faces. But as soon I ask for the Face.Wire or Topology.Geometry the script crashes. Schermafbeelding 2024-06-12 094910

I have attached the BREPS and notebook. Situation 07 - Face memory error.zip

Kind regards,

Martin Vrielink

martinvrielink avatar Jun 12 '24 08:06 martinvrielink

Dear @martinvrielink, I believe I have fixed this in version 0.7.18. Can you please test and report back? Please note that the method Face.NormalAtParameters and Face.NormalAsEdgeAtParameters have been removed since topologicpy only deals with planar faces. As such, always use Face.Normal(aFace, outputType="xyz", mantissa=6)

wassimj avatar Jun 13 '24 07:06 wassimj

Also, when doing Topology.Geometry. It is recommended that you first Triangulate the topology before outputing to geometry. Many other 3D modelling software assume a mesh made of triangles.

wassimj avatar Jun 13 '24 07:06 wassimj

I am closing this issue since I have not heard back from you.

wassimj avatar Aug 25 '24 09:08 wassimj