Fixes#2
Open
magistermaks wants to merge 2 commits into
Open
Conversation
Fixes many configuration, compile and link errors. Splits the rendering into a separate library so that the physics doesn't depend on graphics components. Added a new headless demo. Broken the style a bit but more refactoring is needed anyway so this is fine for now.
Adds positon correction to collisions, fixes demos being inconsistent due to varying window-creation time. Renames all files to be lower case.
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
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.
I found your repo when i was looking for a minimal C++ physics library for my project. In the current state it didn't really work at all so before I could test it I needed to make some changes and decided to try sharing them here. The changes I made:
std::tuple<int, int, int>having no hash function, switched toglm::ivec3./srcinto theCMAKE_BUILD_DIR, but that was more work and I wanted to focus on physics.physics/with a tiny bit of graphics code (seephysics/mesh.hpp). I wanted to make the separation better in the future but at least it works right now as-is.pheV3_graphicstargetbroadPhaseFilterfromPhysicsWorld::updateas it did not work (and I had no need for this optimization in my use-case)There where probably some other changes I missed here,
Kind regards.