Skip to content

Pink Maridia shinesparks: split speed and general cleanup#2756

Merged
osse101 merged 3 commits intovg-json-data:masterfrom
kjbranch:ss-maridia-2
Mar 18, 2026
Merged

Pink Maridia shinesparks: split speed and general cleanup#2756
osse101 merged 3 commits intovg-json-data:masterfrom
kjbranch:ss-maridia-2

Conversation

@kjbranch
Copy link
Copy Markdown
Contributor

No description provided.

@kjbranch kjbranch added the Area:Maridia Stuff in Maridia label Mar 11, 2026
Comment thread region/maridia/inner-pink/Aqueduct.json Outdated
{"and": [
"HiJump",
"canSpeedyJump",
"canShinechargeMovementComplex",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

9 frames is about canShinechargeMovementTricky so 11 frames is a little nicer than that but not much.
If you really want to minimize damage without gravity jump or a snail, you can go 2->5->9. Just speed would have a shorter shinespark. Turning off gravity doesn't have to shinespark.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

honestly not sure how i feel about avoiding gravity jumps. We have made some effort to avoid them, but so inconsistently. I think I'll not worry about it as much now.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This just reads as adding complex logic when a gravity jump is easier and saves health.
It'd just be this branch that's excessive.

{"useFlashSuit": {}},
{"shinespark": {"frames": 1, "excessFrames": 1}}
{"shinespark": {"frames": 1, "excessFrames": 1}},
{"or": [
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reason for this addition? The flash suit is gone before you crouchjump.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it's just a thing you have to do. Should we assume that it's not needed?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It comes down to CrouchJump/DownGrab in a flash suit strat so it can be assumed you know how.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i think we have typically kept it, but it seems unnecessary to me. i can rip it out here.

@osse101 osse101 merged commit 6e1d165 into vg-json-data:master Mar 18, 2026
1 check passed
@kjbranch kjbranch deleted the ss-maridia-2 branch March 20, 2026 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area:Maridia Stuff in Maridia

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants