Archived issue #0022592
Data Exchange - Corrected processing of parameter read.iges.bspline.continuity
Description
RLN_fix405 (contribution).
- Summary: Corrected processing of parameter read.iges.bspline.continuity set to 0
- Detailed description: When read.iges.bspline.continuity set to 0 (with
Interface_Static::SetCVal()), the BSplineSurface is not enforced to become
periodic if it is closed what is done when it's set to 1. In some cases (e.g.
www.firstcadlibrary.com/seatmotor2.igs, entity 397) this leads to wrong
resulting shape after Shape Healing. Instead of added missing seam the face is
split into a compound of 2 invalid faces.
NOTE: See comments in the code, for consistency between TransferSplineSurface()
and TransferBSplineSurface() the former must have a code to convert to periodic
surfaces, and the latter - to IGESConvGeom for enforcing continuity.
It appeared that OCC IGES converter never enforced surface continuity though it
was expected to. It is up to the OCC team to verify if addition of these calls
will not break regressions.
====================================================
Branch OCC22592_IgestobrepSurfaceContinuity is created.
- Summary: Corrected processing of parameter read.iges.bspline.continuity set to 0
- Detailed description: When read.iges.bspline.continuity set to 0 (with
Interface_Static::SetCVal()), the BSplineSurface is not enforced to become
periodic if it is closed what is done when it's set to 1. In some cases (e.g.
www.firstcadlibrary.com/seatmotor2.igs, entity 397) this leads to wrong
resulting shape after Shape Healing. Instead of added missing seam the face is
split into a compound of 2 invalid faces.
NOTE: See comments in the code, for consistency between TransferSplineSurface()
and TransferBSplineSurface() the former must have a code to convert to periodic
surfaces, and the latter - to IGESConvGeom for enforcing continuity.
It appeared that OCC IGES converter never enforced surface continuity though it
was expected to. It is up to the OCC team to verify if addition of these calls
will not break regressions.
====================================================
Branch OCC22592_IgestobrepSurfaceContinuity is created.
Public activity
1 archived note
Participants are labeled by their role within this record.
> Branch OCC22592_IgestobrepSurfaceContinuity is created.
Note that referred branch has been lost (probably it is in SVN).
Note that referred branch has been lost (probably it is in SVN).