ECS Part 7 | entity Model#2680
Merged
IntegratedQuantum merged 87 commits intoPixelGuys:masterfrom Apr 11, 2026
Merged
Conversation
also renamed entity.zig -> clientEntity.zig to make it easier to distingues it from the already existing Entity.zig in server/
… into ECS_United7
…ityLoadsComponents
…eLoadingFromAsset
… into ECS_United7
… into ECS_Protocol
IntegratedQuantum
requested changes
Apr 11, 2026
…immediately where it's created.
…iled if the model didnt load
IntegratedQuantum
requested changes
Apr 11, 2026
IntegratedQuantum
requested changes
Apr 11, 2026
IntegratedQuantum
approved these changes
Apr 11, 2026
IntegratedQuantum
added a commit
that referenced
this pull request
Apr 11, 2026
Requires #2733 and #2680 Progress towards #2419 and #87 progress towards #2777 fixes #2830 So far only works in blockbench coordinate system (right handed, y-up, z-forward) converting it to our system, so for now you have to be sure you export your mesh in the right system if its from another program. Doesn't load textures so they are still stored in game assets. This is a really simple start that i will build up on in future PRs. A few questions still remain: - ~~Do we want to keep the legacy .obj loading?~~ (no, because in a later pr when i add joints i wont be able to add them and it will brake stuff, and thats generally useless) - ~~Do we want to load the file data ourselves and then feed it into the cgltf or just let cgltf handle that~~ (turns out there is no problem with that so its best to do that so that we can have local allocations) - Do we want to override cgltf allocation functions? - ~~Do we want to try handle other possible edge cases that might happen when exporting from other applications? (like different coordinate systems and stuff)~~ (yes #2777) --------- Co-authored-by: till++ <tillplusplus@gmail.com> Co-authored-by: careeoki <122191047+careeoki@users.noreply.github.com> Co-authored-by: tillpp <tillpp@users.noreply.github.com> Co-authored-by: IntegratedQuantum <43880493+IntegratedQuantum@users.noreply.github.com>
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
5 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.
Sixth step of splittig up the 3000 lines big PR: #2652
(requires Part 1 to 6, except 4)
Adds EntityModels