Velothi Temple and Siltstrider Port Improvements#62
Open
BethOfDeath wants to merge 2 commits intomop-org:masterfrom
Open
Velothi Temple and Siltstrider Port Improvements#62BethOfDeath wants to merge 2 commits intomop-org:masterfrom
BethOfDeath wants to merge 2 commits intomop-org:masterfrom
Conversation
- Simplified, split, combined, and scaled collision mesh to reduce calculation volume and somewhat correct minor inaccuracy relative to visual mesh. - Added LOD reducing from 5 drawcalls to 1 - Previously: 47407 bytes; new: 67788. (+43%) ex_velothi_temple_01: - Added two LODs, reducing from 7 drawcalls to 4, then to 3, and simplifying geometry. - Previously: 70453 bytes; new: 126832. (+80%) ex_velothi_striderport_01: - Collision mesh did not cover entirety of visual mesh, limiting the height at which it could be used even beyond what the relative height of a silt strider dictates. Adjusted collision posts and chopped into sufficient pieces to minimize computation volume. -- Also split platforms' collision meshes, as they tended to reach to the ground or nearly so. -- New collision meshes total 266 tris in 33 pieces instead of the former 94 in 9, but I'm hoping to find less of an FPS impact moving around underneat and atop it. - Added, or replaced, edgeloops on platforms and walkway, to improve vertex lighting. - Removed sharp edges from tx_wood_docks mesh as they were not helpful at causing better normals and may have been an artifact of importing anyway. _Added_ some to the actually-flat platform, where they _do_help - Reweighted normals. - Retopologized triangle-fan endgrain to max-area, saving 32 tris and possibly a little overdraw... somewhere. - Added LODs, reducing from 6 drawcalls to 4, then 2. - Previously 111238 bytes; new: 201005. (+81%)
C3pa
reviewed
Feb 19, 2026
Contributor
There was a problem hiding this comment.
Could you fix a UV seam in this model that was carried over from vanilla:
Contributor
Author
There was a problem hiding this comment.
Dunno how I forgot to do my usual UV islands combining which would've caught that, but it's fixed now.
Contributor
There was a problem hiding this comment.
The collision on the dome isn't split in the best way possible:
Contributor
Author
There was a problem hiding this comment.
Odd. I'll try to remember to look into it, thanks.
Contributor
There was a problem hiding this comment.
The changes in 01b5aae are already a massive improvement over vanilla.
Do you think it would be possible to also remove this seam?
Contributor
Author
There was a problem hiding this comment.
Maybe? I'll try to remember to look at it sometime tomorrow.
- Combined UV islands to fix seams I missed, due to focus on collision mesh and LODing. - Now: 66924 bytes.
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.
ex_velothi_temple_02:
ex_velothi_temple_01:
ex_velothi_striderport_01: