Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Integration request to master #77

Merged
merged 10 commits into from
Sep 28, 2024
Merged

Integration request to master #77

merged 10 commits into from
Sep 28, 2024

Conversation

dpasukhi
Copy link
Member

Integration request to collect a sequence of patches as a periodical branch.
Will be available for short time only during development circle.

Fixed crash in STEPConstruct_Styles::GetColors() due to nullptr
  dereferencing when source step file has missing
  FILL_AREA_STYLE_COLOUR entities.
dpasukhi and others added 9 commits September 25, 2024 08:23
Cleaning repo from not maintaining functionality - genproj and wok
Only 'gendoc' functionality kept.
TODO: migrate gendoc to cmake
…sing an invalid file

fclose() call is added before early return statement in igesread() function.
… Write procedure

Implementation of common logic for OCCT formats.
…very different results under small change in deflection
ReadMe was moved to markdown format.
Updated a links to the resources
…is skipped on write

Processing of complex_triangulated_surface_set is added to
  STEPCAFControl_GDTProperty::GetTessellation().
Processing of tessellated_curve_set is refactored and moved to a
  separate function.

Problems related to normals list are fixed in
RWStepVisual_RWComplexTriangulatedSurfaceSet::WriteStep():
  Normals list is now checked for nullptr. It is a valid situation that
  occurs when complex_triangulated_surface_set in STEP file has no
  normals.
  Traversing of normals list is performed in correct order. Previously
  rows and columns were switched, which led to crashes or incorrect data.
Crash in TopoDSToStep_MakeStepFace::Init() that occured due to attempt
to copy null Geom2d_Curve is fixed.
… Write procedure

Fixed inconsistency of tests.
'IGES.OCC.write.unit no more' used.
@dpasukhi dpasukhi self-assigned this Sep 28, 2024
@dpasukhi dpasukhi merged commit 3462ea5 into master Sep 28, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants