Fix unexpected recursions in geometry splitting#13
Draft
Conversation
Some minor fixes to the build system
Geometry transformations
Enable automatic builds
Checks if alternate, faster linkers like mold, lld, or gold are supported by the compiler and enables the best one found.
Some minor improvements to the build system
Methods that return inplace shall use a verb, methods that return a modified object shall use the matching adjective.
The mathematical notation with operators might be correct, but isn't that easy to read, and even harder to memorize: Is it "matrix * object", or rather "object * matrix"? "Why the former!? The later reads better!".
Modify containers in place, and reserve sufficient space early.
Some minor housekeeping
This will allow reuse once we add more tests.
It's just more pleasant read without the `QVector3D::` noise. More importantly this will give the chance to replace the fast, but imprecise QVector3D tpye by more precise vector types.
The new name is parseGeometry() for symmetry with loadGeometry().
Make a few functions freestanding that should have always been
hasselmm
force-pushed
the
bugfix/unexpected-recursions
branch
from
4397a49 to
69cab6e
Compare
The shaders are verbatimly taken from "Wireframe QML Example" of Qt3D. The material and effect are my adoption of the QML code to C++.
So that QtCreator and others shows them in their project overview.
Changed the shaders to Qt Coding Style: I really like consistency.
The wireframe material from Qt3D examples used a fixed light source instead of the real lighting model from Qt3D to keep its adsModel() function trivial. Luckily Qt3D modularized its shaders some time ago. Therefore we can use use the real thing.
…demo Add wireframes to the demo application
…ithub Hopefully make Github understand license info
…ithub Another attempt to fix license information
…ithub Fix broken license link
Task-Id: #12
This interface shall help visualizing, understanding, and debugging the boolean geometry operations.
hasselmm
force-pushed
the
bugfix/unexpected-recursions
branch
from
69cab6e to
cd2659e
Compare
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
1 participant
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.