Commit _just_ the Bloodmoon rugs from closed PR #34.#80
Open
BethOfDeath wants to merge 1 commit intomop-org:masterfrom
Open
Commit _just_ the Bloodmoon rugs from closed PR #34.#80BethOfDeath wants to merge 1 commit intomop-org:masterfrom
BethOfDeath wants to merge 1 commit intomop-org:masterfrom
Conversation
To restate the relevant portion of that PR's commit message: furn_colony_wolfrug01 - Simplified mesh. Was 48 tris, now 34. - Removed two NiZBufferProperties which (going by an old comment by Greatness7) appeared to be set to default behavior and thus did nothing - Original's NiAlphaProperty was set to 1-bit alpha sharp edge high filtration mode (15085) and which uses both alpha blending AND testing. Replaced with 13036 to disable blending and sorting, and set threshold to 168 to match appearance (at least in NiSkope). - Added NiStringExtraData NC to disable collision checking, restoring the function of the NCO which was removed from previous MOP version. - Vanilla was 2863 bytes, previous MOP version 2851, now 2481. (13% decrease from vanilla) furn_bearskin_rug - It appears this one was created with overmuch efficiency, by not enabling backface culling and just leaving the generated backfaces colocating with the visible frontfaces. To get rid of clipping, these backside faces have been sepearated, decimated, flattened, and shifted. - Shifted and merged some unneeded vertices with identical vcols on topside. - Bogus NiZBufferProperty removed. NiStringExtraData NCO replaced with NC. - Weighted normals - Was: 120 tris 222 verts, now: 74 tris 39 verts. - Was 5452 bytes, now 4019. (26% decrease) furn_redwolf_rug - Same issues as the bearskin rug, same solution. - Was 144 tris 265 verts, now 94 tris 52 verts. - Was 6287 bytes, now 5037. (20% decrease) furn_whitewolf_rug - Same issues as bearskin rug, same solution - Was 120 tris and 222 vertices, now 88 tris 46 verts. - Was 5484 bytes, now 4451. (19% decrease)
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.
To restate the relevant portion of that PR's commit message (ignore part about "clipping from co-located reversed faces" -- I hadn't internalized the fact such just get merged in the export. Probably still looks better when messed with in Perfect Placement?):
furn_colony_wolfrug01
furn_bearskin_rug
furn_redwolf_rug
furn_whitewolf_rug