Skip to content

h_avoidScrewStuck#2843

Merged
osse101 merged 3 commits intovg-json-data:masterfrom
osse101:ScrewStuck
Apr 18, 2026
Merged

h_avoidScrewStuck#2843
osse101 merged 3 commits intovg-json-data:masterfrom
osse101:ScrewStuck

Conversation

@osse101
Copy link
Copy Markdown
Contributor

@osse101 osse101 commented Apr 15, 2026

Not sure how helpful this helper will be since the player is just going to take screw attack to the breakable blocks without thinking, but its something.

@osse101 osse101 added Area:Lower Norfair Stuff in Lower Norfair Area:Crateria Stuff in Crateria Area:Brinstar Stuff in Brinstar labels Apr 15, 2026
Copy link
Copy Markdown
Contributor

@kjbranch kjbranch left a comment

Choose a reason for hiding this comment

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

It all looks correct.

Comment thread region/brinstar/pink/Dachora Room.json Outdated
"h_destroyBombWalls",
{"and": [
"h_destroyBombWalls",
"h_avoidScrewStuck"
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.

I worry a bit about the readability of this. I wouldve thought the helper would include screw.

"canCarefulJump",
"ScrewAttack"
"ScrewAttack",
"h_avoidScrewStuck"
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.

I think having this include screw would be nicer for this one as well - because this strat already included a careful jump, so it feels a bit weird to have this helper which is technically just careful jump or morph

Copy link
Copy Markdown
Contributor

@kjbranch kjbranch Apr 18, 2026

Choose a reason for hiding this comment

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

The careful jump should remain here, as it wouldnt be required if you have Morph.

Edit: (A little confused, the preview here shows it remains, but the Files Changed shows it removed)

"ScrewAttack",
"h_avoidScrewStuck",
{"or": [
"canTwoTileSqueeze",
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 took me a minute to figure out how to do this and I wasn't sure if it was reasonable, or if it should need a description. But this strat is technically a walljumpless strat, so maybe it doesnt matter as much.

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 find its really different from the other 2high stand up strats so I upped the difficulty, calling it a tunnel crawl.

@osse101 osse101 marked this pull request as draft April 16, 2026 22:40
@osse101
Copy link
Copy Markdown
Contributor Author

osse101 commented Apr 16, 2026

I think this is acting a lot like a new tech.
Pillars room should be written differently if you have to morph out.
Mixing it with h_destroyBombWalls isn't so clean so it may need another helper to make that read nicer.

@osse101 osse101 marked this pull request as ready for review April 18, 2026 06:10
@osse101
Copy link
Copy Markdown
Contributor Author

osse101 commented Apr 18, 2026

canCarefulJump aligns too well with what a new canAvoidScrewStuck tech would look like so I left it at that.
And Pillar room is tough right to left, so I only made it an option left to right. And even then rolling out isn't actually costly most of the time unless you fail multiple times.

kjbranch
kjbranch previously approved these changes Apr 18, 2026
"canCarefulJump",
"ScrewAttack"
"ScrewAttack",
"h_avoidScrewStuck"
Copy link
Copy Markdown
Contributor

@kjbranch kjbranch Apr 18, 2026

Choose a reason for hiding this comment

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

The careful jump should remain here, as it wouldnt be required if you have Morph.

Edit: (A little confused, the preview here shows it remains, but the Files Changed shows it removed)

@osse101 osse101 merged commit 3c46a6c into vg-json-data:master Apr 18, 2026
1 check passed
@osse101 osse101 deleted the ScrewStuck branch April 18, 2026 06:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area:Brinstar Stuff in Brinstar Area:Crateria Stuff in Crateria Area:Lower Norfair Stuff in Lower Norfair

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants