feat: Most of Tartaros shadow packets#1033
Conversation
|
Important Review skippedAuto incremental reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the 📝 WalkthroughSummary by CodeRabbit
WalkthroughTech-Priest, new Tartaros sprite assets are registered, several existing sprite sequences are adjusted, and modular drawing entries are expanded. The Tartaros group in scr_ComplexSet prunes component assignments. ChapterMaster.yyp gains new resources; scr_culture_visuals adds drawing items for right arm, gorget, head, forehead, and right leg. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Pre-merge checks❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
sprites/spr_tartaros_head_variants/spr_tartaros_head_variants.yy (1)
14-16: Restore missing head-variant frames, Tech-Priest.Machine-spirit diagnostics detect that
spr_tartaros_head_variantsnow exposes only a lone frame and keyframe. Prior data housed multiple heads for modular selection; with only one, any lookup of higher indices collapses to the same visage, stripping variant diversity and desynchronising with configuration tables that still enumerate the previous set. Reinstating the full frame array and matching keyframes is required before communion with mainline.Also applies to: 72-76
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
Plan: Pro
⛔ Files ignored due to path filters (46)
sprites/spr_tartaros_forehead_shadows/40bc99d5-36c6-494f-ac41-97db7b1af50a.pngis excluded by!**/*.pngsprites/spr_tartaros_forehead_shadows/9d5b17e4-eb9a-46c2-a4a7-6dbbfac5dfe3.pngis excluded by!**/*.pngsprites/spr_tartaros_forehead_shadows/b529fb18-f4ec-47d1-b6b3-ed4a9f174496.pngis excluded by!**/*.pngsprites/spr_tartaros_forehead_shadows/fd6650e0-331d-4ca5-8853-0b5406c4134d.pngis excluded by!**/*.pngsprites/spr_tartaros_forehead_shadows/layers/40bc99d5-36c6-494f-ac41-97db7b1af50a/3156c4a2-0553-4e28-b1dd-29e32750bb31.pngis excluded by!**/*.pngsprites/spr_tartaros_forehead_shadows/layers/9d5b17e4-eb9a-46c2-a4a7-6dbbfac5dfe3/3156c4a2-0553-4e28-b1dd-29e32750bb31.pngis excluded by!**/*.pngsprites/spr_tartaros_forehead_shadows/layers/b529fb18-f4ec-47d1-b6b3-ed4a9f174496/3156c4a2-0553-4e28-b1dd-29e32750bb31.pngis excluded by!**/*.pngsprites/spr_tartaros_forehead_shadows/layers/fd6650e0-331d-4ca5-8853-0b5406c4134d/3156c4a2-0553-4e28-b1dd-29e32750bb31.pngis excluded by!**/*.pngsprites/spr_tartaros_forehead_variants/40bc99d5-36c6-494f-ac41-97db7b1af50a.pngis excluded by!**/*.pngsprites/spr_tartaros_forehead_variants/9d5b17e4-eb9a-46c2-a4a7-6dbbfac5dfe3.pngis excluded by!**/*.pngsprites/spr_tartaros_forehead_variants/b529fb18-f4ec-47d1-b6b3-ed4a9f174496.pngis excluded by!**/*.pngsprites/spr_tartaros_forehead_variants/fd6650e0-331d-4ca5-8853-0b5406c4134d.pngis excluded by!**/*.pngsprites/spr_tartaros_forehead_variants/layers/40bc99d5-36c6-494f-ac41-97db7b1af50a/3156c4a2-0553-4e28-b1dd-29e32750bb31.pngis excluded by!**/*.pngsprites/spr_tartaros_forehead_variants/layers/9d5b17e4-eb9a-46c2-a4a7-6dbbfac5dfe3/3156c4a2-0553-4e28-b1dd-29e32750bb31.pngis excluded by!**/*.pngsprites/spr_tartaros_forehead_variants/layers/b529fb18-f4ec-47d1-b6b3-ed4a9f174496/3156c4a2-0553-4e28-b1dd-29e32750bb31.pngis excluded by!**/*.pngsprites/spr_tartaros_forehead_variants/layers/fd6650e0-331d-4ca5-8853-0b5406c4134d/3156c4a2-0553-4e28-b1dd-29e32750bb31.pngis excluded by!**/*.pngsprites/spr_tartaros_gorget_shadows/7183111f-9020-46b1-8c8d-6f3863650227.pngis excluded by!**/*.pngsprites/spr_tartaros_gorget_shadows/7f9b6976-da75-43c8-97e3-acfa8912ef0e.pngis excluded by!**/*.pngsprites/spr_tartaros_gorget_shadows/8d99f92f-4aac-47e4-a142-c6ea6be47856.pngis excluded by!**/*.pngsprites/spr_tartaros_gorget_shadows/ead692b6-35d8-4ebb-bc75-76c2fbfae9ef.pngis excluded by!**/*.pngsprites/spr_tartaros_gorget_shadows/layers/7183111f-9020-46b1-8c8d-6f3863650227/50a6880d-f9bf-4734-a44b-81b675bc6761.pngis excluded by!**/*.pngsprites/spr_tartaros_gorget_shadows/layers/7f9b6976-da75-43c8-97e3-acfa8912ef0e/50a6880d-f9bf-4734-a44b-81b675bc6761.pngis excluded by!**/*.pngsprites/spr_tartaros_gorget_shadows/layers/8d99f92f-4aac-47e4-a142-c6ea6be47856/50a6880d-f9bf-4734-a44b-81b675bc6761.pngis excluded by!**/*.pngsprites/spr_tartaros_gorget_shadows/layers/ead692b6-35d8-4ebb-bc75-76c2fbfae9ef/50a6880d-f9bf-4734-a44b-81b675bc6761.pngis excluded by!**/*.pngsprites/spr_tartaros_head_shadows/fd6650e0-331d-4ca5-8853-0b5406c4134d.pngis excluded by!**/*.pngsprites/spr_tartaros_head_shadows/layers/fd6650e0-331d-4ca5-8853-0b5406c4134d/3156c4a2-0553-4e28-b1dd-29e32750bb31.pngis excluded by!**/*.pngsprites/spr_tartaros_head_variants/40bc99d5-36c6-494f-ac41-97db7b1af50a.pngis excluded by!**/*.pngsprites/spr_tartaros_head_variants/9d5b17e4-eb9a-46c2-a4a7-6dbbfac5dfe3.pngis excluded by!**/*.pngsprites/spr_tartaros_head_variants/b529fb18-f4ec-47d1-b6b3-ed4a9f174496.pngis excluded by!**/*.pngsprites/spr_tartaros_head_variants/layers/40bc99d5-36c6-494f-ac41-97db7b1af50a/3156c4a2-0553-4e28-b1dd-29e32750bb31.pngis excluded by!**/*.pngsprites/spr_tartaros_head_variants/layers/9d5b17e4-eb9a-46c2-a4a7-6dbbfac5dfe3/3156c4a2-0553-4e28-b1dd-29e32750bb31.pngis excluded by!**/*.pngsprites/spr_tartaros_head_variants/layers/b529fb18-f4ec-47d1-b6b3-ed4a9f174496/3156c4a2-0553-4e28-b1dd-29e32750bb31.pngis excluded by!**/*.pngsprites/spr_tartaros_leg_rivets/b53b7c9e-26af-4c4b-9b89-a15af4f7296c.pngis excluded by!**/*.pngsprites/spr_tartaros_leg_rivets/layers/b53b7c9e-26af-4c4b-9b89-a15af4f7296c/b1ad68ac-732f-42d2-971e-e7f6a4962e33.pngis excluded by!**/*.pngsprites/spr_tartaros_right_arm_shadows/87c9365b-ef0b-48e1-a764-b2b2b8937c67.pngis excluded by!**/*.pngsprites/spr_tartaros_right_arm_shadows/layers/87c9365b-ef0b-48e1-a764-b2b2b8937c67/c87e73e1-e866-4b39-8a1f-96167acb2eca.pngis excluded by!**/*.pngsprites/spr_tartaros_right_leg/b53b7c9e-26af-4c4b-9b89-a15af4f7296c.pngis excluded by!**/*.pngsprites/spr_tartaros_right_leg/layers/b53b7c9e-26af-4c4b-9b89-a15af4f7296c/b1ad68ac-732f-42d2-971e-e7f6a4962e33.pngis excluded by!**/*.pngsprites/spr_tartaros_right_leg_shadows/8d99f92f-4aac-47e4-a142-c6ea6be47856.pngis excluded by!**/*.pngsprites/spr_tartaros_right_leg_shadows/efcff506-8258-4e7c-89aa-82cc9816fb9f.pngis excluded by!**/*.pngsprites/spr_tartaros_right_leg_shadows/f31258fb-8e42-40fa-9abe-d50ce4077249.pngis excluded by!**/*.pngsprites/spr_tartaros_right_leg_shadows/fa086401-4b32-4bee-a3f9-fb5ec4f5bfa2.pngis excluded by!**/*.pngsprites/spr_tartaros_right_leg_shadows/layers/8d99f92f-4aac-47e4-a142-c6ea6be47856/b1ad68ac-732f-42d2-971e-e7f6a4962e33.pngis excluded by!**/*.pngsprites/spr_tartaros_right_leg_shadows/layers/efcff506-8258-4e7c-89aa-82cc9816fb9f/b1ad68ac-732f-42d2-971e-e7f6a4962e33.pngis excluded by!**/*.pngsprites/spr_tartaros_right_leg_shadows/layers/f31258fb-8e42-40fa-9abe-d50ce4077249/b1ad68ac-732f-42d2-971e-e7f6a4962e33.pngis excluded by!**/*.pngsprites/spr_tartaros_right_leg_shadows/layers/fa086401-4b32-4bee-a3f9-fb5ec4f5bfa2/b1ad68ac-732f-42d2-971e-e7f6a4962e33.pngis excluded by!**/*.png
📒 Files selected for processing (14)
ChapterMaster.yyp(1 hunks)scripts/scr_ComplexSet/scr_ComplexSet.gml(0 hunks)scripts/scr_culture_visuals/scr_culture_visuals.gml(2 hunks)sprites/spr_tartaros_forehead_shadows/spr_tartaros_forehead_shadows.yy(1 hunks)sprites/spr_tartaros_forehead_variants/spr_tartaros_forehead_variants.yy(1 hunks)sprites/spr_tartaros_gorget/spr_tartaros_gorget.yy(1 hunks)sprites/spr_tartaros_gorget_shadows/spr_tartaros_gorget_shadows.yy(1 hunks)sprites/spr_tartaros_head_shadows/spr_tartaros_head_shadows.yy(1 hunks)sprites/spr_tartaros_head_variants/spr_tartaros_head_variants.yy(2 hunks)sprites/spr_tartaros_leg_rivets/spr_tartaros_leg_rivets.yy(1 hunks)sprites/spr_tartaros_right_arm/spr_tartaros_right_arm.yy(1 hunks)sprites/spr_tartaros_right_arm_shadows/spr_tartaros_right_arm_shadows.yy(1 hunks)sprites/spr_tartaros_right_leg/spr_tartaros_right_leg.yy(2 hunks)sprites/spr_tartaros_right_leg_shadows/spr_tartaros_right_leg_shadows.yy(1 hunks)
💤 Files with no reviewable changes (1)
- scripts/scr_ComplexSet/scr_ComplexSet.gml
🧰 Additional context used
📓 Path-based instructions (3)
**/*.*
⚙️ CodeRabbit configuration file
**/*.*: - Having humanly understandable and maintainable code is the top most priority.
- DRY (Don't repeat yourself) principle is also very important.
- If a TODO comment is added, ask the user if you should create a GitHub issue for this TODO.
- If a TODO comment is deleted, remind the user if there is an active GitHub issue related to that comment.
Files:
sprites/spr_tartaros_right_arm/spr_tartaros_right_arm.yysprites/spr_tartaros_gorget/spr_tartaros_gorget.yysprites/spr_tartaros_right_leg_shadows/spr_tartaros_right_leg_shadows.yysprites/spr_tartaros_forehead_variants/spr_tartaros_forehead_variants.yysprites/spr_tartaros_forehead_shadows/spr_tartaros_forehead_shadows.yysprites/spr_tartaros_head_shadows/spr_tartaros_head_shadows.yysprites/spr_tartaros_right_leg/spr_tartaros_right_leg.yyscripts/scr_culture_visuals/scr_culture_visuals.gmlsprites/spr_tartaros_right_arm_shadows/spr_tartaros_right_arm_shadows.yyChapterMaster.yypsprites/spr_tartaros_head_variants/spr_tartaros_head_variants.yysprites/spr_tartaros_leg_rivets/spr_tartaros_leg_rivets.yysprites/spr_tartaros_gorget_shadows/spr_tartaros_gorget_shadows.yy
**/*.yy
⚙️ CodeRabbit configuration file
**/*.yy: - When any script or sprite .yy files are deleted, their paths should also be deleted from the project .yyp file, otherwise the game will crash.
- When any script or sprite .yy files are created, their paths should be added to the project .yyp file, otherwise they'll fail.
Files:
sprites/spr_tartaros_right_arm/spr_tartaros_right_arm.yysprites/spr_tartaros_gorget/spr_tartaros_gorget.yysprites/spr_tartaros_right_leg_shadows/spr_tartaros_right_leg_shadows.yysprites/spr_tartaros_forehead_variants/spr_tartaros_forehead_variants.yysprites/spr_tartaros_forehead_shadows/spr_tartaros_forehead_shadows.yysprites/spr_tartaros_head_shadows/spr_tartaros_head_shadows.yysprites/spr_tartaros_right_leg/spr_tartaros_right_leg.yysprites/spr_tartaros_right_arm_shadows/spr_tartaros_right_arm_shadows.yysprites/spr_tartaros_head_variants/spr_tartaros_head_variants.yysprites/spr_tartaros_leg_rivets/spr_tartaros_leg_rivets.yysprites/spr_tartaros_gorget_shadows/spr_tartaros_gorget_shadows.yy
**/*.gml
⚙️ CodeRabbit configuration file
**/*.gml: - Macro constants require a space between the constant name and value. Without it, the compiler will throw an error. I.e.#macro ARR_body_parts["arm"]will crash the game, because there is no space between the array and the name of the macro.
- Color codes in the code shouldn't have any spaces in their id. I.e., color code
# 80bf40will crash the game.- All code should comply with the main GML documentation: https://manual.gamemaker.io/beta/en/GameMaker_Language/GML_Reference/GML_Reference.htm
Files:
scripts/scr_culture_visuals/scr_culture_visuals.gml
🔇 Additional comments (8)
sprites/spr_tartaros_leg_rivets/spr_tartaros_leg_rivets.yy (1)
1-92: Sprite manifest alignedTech-Priest, the rivet sprite manifest aligns with MarineViewer standards; bounds and origin are harmonious with the right leg lattice. No further rites required.
sprites/spr_tartaros_forehead_shadows/spr_tartaros_forehead_shadows.yy (1)
1-104: Shadow sequence sanctifiedTech-Priest, the forehead shadow sequence is orderly; frame cadence and origin sit precisely within Tartaros expectations.
ChapterMaster.yyp (1)
1643-1655: Registry augmentation completeTech-Priest, the newly forged tartaros resources are duly inscribed within the project litany, preventing load-time voids.
sprites/spr_tartaros_right_leg_shadows/spr_tartaros_right_leg_shadows.yy (1)
1-104: Right-leg shadow schema trueTech-Priest, this shadow schema aligns with the leg articulation; frame phasing and collision wards are immaculate.
scripts/scr_culture_visuals/scr_culture_visuals.gml (4)
1439-1446: LGTM: Right arm entry for Tartaros armour is properly forged.The structure aligns with established patterns, Tech-Priest. Sprites, shadows, and flip designation are correctly registered.
1447-1453: LGTM: Gorget entry is sanctioned.This component adheres to sacred protocols, Tech-Priest. Sprite and shadow registrations are properly aligned.
1454-1460: LGTM: Head variants entry is correctly inscribed.The head component for Tartaros armour follows the ordained pattern, Tech-Priest. Shadow sprites are properly bound.
1468-1478: LGTM: Right leg entry with subcomponents is properly configured.The right leg component for Tartaros armour is correctly structured, Tech-Priest. The subcomponents array includes rivet sprites as intended, and the flip designation is appropriately set.
sprites/spr_tartaros_forehead_variants/spr_tartaros_forehead_variants.yy
Show resolved
Hide resolved
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Purpose and Description
Testing done
Related things and/or additional context