Conversation
- Renamed grid terminology and command/API surface from `inf`/“infinite” to `gpu`/“GPU”, plus `viewAdaptive` for camera-relative sizing. - Updated OpenGl grid rendering to compute adaptive bounds from the visible region and to fix axis coloring for rotated rectangular grids. - Refreshed Draw tests, help text, and API comments to match the new behavior.
- Redesign seam detection/pairing to be connectivity-derived (`SeamPair()` walks sibling coedges on the same face with opposite orientations), with seam halves now present in wire `CoEdgeRefIds` to match TopoDS iteration. - Move geometric continuity storage/round-trip to `BRepGraph_LayerRegularity` (including seam continuity), updating populate/reconstruct/editor APIs accordingly. - Add/adjust GTests to cover seam ordering, seam symmetry, layer regularity behavior, and split/compact/reconstruct interactions under the new model.
dpasukhi
added
the
2. IR
|
Performance Test Summary Commit: Title: Merge 6ba27a8 into a70427f Platform: macos-x64 Platform: windows-x64 Platform: linux-clang-x64 |
…ency (#1265) - Renames many example variables to align better with current OCCT naming/style conventions. - Updates documentation around deprecated 2D construction APIs, build options, resource paths, and environment variables. - Cleans up assorted wording, links, and outdated references across user guides, samples, and contributor docs.
|
Performance Test Summary Commit: Title: Merge 9281002 into a70427f Platform: macos-x64 Platform: windows-x64 Platform: linux-clang-x64 |
…AdditionalPointOrAdjust() (#1267) To input parameters of ShapeConstruct_ProjectCurveOnSurface::insertAdditionalPointOrAdjust() new arrays were assigned. The size of new arrays are guaranteed to be 1 element larger that original array. NCollection_Array1::Assign() is guaranteed to throw exception when arrays size mismatch. So, given that assignment code is reached, function will always throw. Fixed by calling move assignment instead - size mismatch is allowed in this case. Also it is more optimal.
|
Performance Test Summary Commit: Title: Merge 6f6be66 into a70427f Platform: macos-x64 Platform: windows-x64 Platform: linux-clang-x64 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Integration request to collect a sequence of patches as a periodical branch.
Will be available for short time only during development circle.