DiscussionsIssue archiveOCCT:Data Exchange

Archived issue #0022592

Data Exchange - Corrected processing of parameter read.iges.bspline.continuity

CommunityOCCT:Data Exchangeassigned1 public note

Search issues

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.

Public activity

1 archived note

Participants are labeled by their role within this record.

01Commenter 1
> Branch OCC22592_IgestobrepSurfaceContinuity is created.
Note that referred branch has been lost (probably it is in SVN).