-
Notifications
You must be signed in to change notification settings - Fork 537
Occt reconstruct #946
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
Open
caimingzhi
wants to merge
81
commits into
Open-Cascade-SAS:master
Choose a base branch
from
caimingzhi:occt_reconstruct
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Occt reconstruct #946
Conversation
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
Fixed issue when folder name contains .c or .h as a part of the name.
Updated missed components leaded to the issue with VCPKG
Refactor CMake configuration to remove patch handling and simplify includes
Added checking for file in package or toolkit. Removed not-existed files from the list
…ade-SAS#419 Update the detection Qt5 folder to search for bin folder. That will affected for the starting DRAW or samples as a part of env.
Updated VCPKG workflow to follow the type of linking with the triplet. Removed symbolic linker flag. As a result downgrade of performance. The symbolic flag has high risk of instability, better to disable.
Add CMAKE_POLICY_VERSION_MINIMUM to TInspector configuration for Windows and Linux
…ade-SAS#496 Refactor TBB CMake configuration for checking configuration type. The solution will be reorganized to math release/debug version. Prepared only for the hot-fix reason.
…n-Cascade-SAS#523) Update target_link_libraries to use PUBLIC and PRIVATE visibility for dependencies. Now all 3rd-party is hidden for the external linking, no more issues to invalid path to 3rd-party
…-Cascade-SAS#393) Fix configuration loading to correctly use FileCS parameter Add stability test for configuration changes Update documentation's de wrapping names
Refactor axis handling with correct positioning and handling exceptions.
…properly Open-Cascade-SAS#422 Fixed issue with bounding box cleaning algorithm that was causing the bounding box to be only increased. Now the bounding box is increased and decreased to fit the shape.
Fix schema format for AP242_MANAGED_MODEL_BASED_3D_ENGINEERING_MIM_LF in StepAP214_Protocol.cxx
Fixed issue when gethostbyname returns a nullptr, because the host can't resolve itself
…n Linux Open-Cascade-SAS#458 Simply disabling the code which enables floating point exceptions is probably wrong, but I don't have a replacement for the non-posix functions fegetexcept(3) and feenableexcept(3).
Fix memory usage reporting for GLIBC compatibility in OSD_MemInfo
…en-Cascade-SAS#470 In some cases face without natural bound can be not have edges on curve
…#468 Update BndLib_Add3dCurve to check the periodic case u1-u2 matching with period
Enhance periodic curve handling in BndLib_Add3dCurve::Add method. Checks the periodic BSpline on 3 directions of period for tolerance upgrade.
…pen-Cascade-SAS#483 Check if the curve was already created and use it. Works for shared edges cases.
…e translation of roots reference (Open-Cascade-SAS#529) Add processing of roots, which are transformed references of parts/assemblies.
…pen-Cascade-SAS#532 Added null checks for TopoDS_Face in ChFi3d_Builder_2 and BRepAdaptor_Surface. Added tests to check for crash.
…e-SAS#514 Adding a null check for the triangulation in BRep_Tool::IsClosed. Simplifying index lookup logic in BOPDS_DS. Introducing helper functions (IsPlaneFF and IsClosedFF) and updating iteration loops in BOPAlgo_PaveFiller_6 for improved clarity and robustness.
…nverted (Open-Cascade-SAS#699) Make the degenerated torus bounded in two values of parameters processed as a regular to insert a seam edge properly.
…thod (Open-Cascade-SAS#703) Fixed a critical indexing bug in IntAna_IntQuadQuad::NextCurve where the method incorrectly used nextcurve[I] instead of nextcurve[I-1] for determining the theOpposite parameter. This mismatch between 1-indexed API parameters and 0-indexed array access could lead to out-of-bounds memory access and incorrect curve connectivity determination. Changes: - Fix IntAna_IntQuadQuad::NextCurve to use consistent I-1 indexing for both condition check and return value - Add comprehensive GTests covering NextCurve functionality, edge cases, and performance - Ensure proper error handling for invalid curve indices
Removed overrides for jemalloc, now use standard port
Introduces a new "tcltk" feature in vcpkg.json with relevant dependencies. Updates tcl portfiles and patches to incorporate Tk building, installation, and cleanup. Modifies CMake and batch environment templates to align with the new TclTk support.
- Updating Qt project files to use C++17 standard instead of GNU++11 - Configuring Visual Studio project files to use C++17 language standard - Setting CMake projects to require C++17 standard
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.