Integrate Synty vehicles and props#27
Merged
Merged
Conversation
Ship the static-mesh half of the Synty pack (handoff step 4), which needs none of the character retarget: two vehicles (armoured car, technical) and five props (barrel, barrel stack, ammo box, T-wall barrier, water tank). New converter tools/art/blender/import_synty_prop.py: applies the importer's metre-scale transform (the static FBX carries unit metadata; only the skeletal FBX is centimetres), recentres to ground, names material slots by mesh, welds, and decimates the vehicles. Textures stay cheap. The Land_Vehicles atlas ships as one desert variant (synty_vehicles.webp, 76 KB) bound to every vehicle, since the pack's ten variants share a UV layout. Props reuse the character atlas and add no texture at all. Vehicles decimate ~1.5 MB -> ~0.5 MB with no visible loss, verified by preview render. Shell budget 5.38 -> 6.33 MB, under the 9 MB guard. Placed as cosmetic set-dressing in the spawn zones and back corners, non- colliding and out of the three combat lanes, so world.ts rule 1 still holds for the play space. Promoting any to real cover is a separate map.ts change. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
vu1nz Security Review0 finding(s) in PR #? No security issues found. |
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.
Ships the static-mesh half of the Synty pack — the part that needs none of the character retarget that's still blocked.
Two vehicles (armoured car, technical) and five props (barrel, barrel stack, ammo box, T-wall barrier, water tank).
New converter
tools/art/blender/import_synty_prop.py: applies the importer's metre-scale transform (the static FBX carries unit metadata and is already in metres — only the skeletal FBX is centimetres), recentres to ground, names material slots by mesh, welds, and decimates the vehicles.Textures stay cheap. The Land_Vehicles atlas ships as one desert variant (76 KB) bound to every vehicle, since the pack's ten variants share a UV layout. Props reuse the character atlas and add no texture at all. Vehicles decimate ~1.5 MB → ~0.5 MB with no visible loss.
Shell budget 5.38 → 6.33 MB, under the 9 MB guard.
Also updates
docs/HANDOFF-synty.md, which still claimed 0 of 1,950 integrated.201 tests pass. Format, lint, typecheck clean; both apps build.
🤖 Generated with Claude Code