Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Split heat frame reqs before/after shinesparks #1067

Open
blkerby opened this issue Sep 13, 2023 · 3 comments
Open

Split heat frame reqs before/after shinesparks #1067

blkerby opened this issue Sep 13, 2023 · 3 comments

Comments

@blkerby
Copy link
Contributor

blkerby commented Sep 13, 2023

The order in which shinespark requirements appear in relation to other energy requirements (for example heatFrames) can affect how the logic works. If a shinespark requirement is placed later than it actually occurs, then it can result in logic that is more lenient/conservative than it has to be, since it increases the possibility of hitting 29 energy and hence not being able to satisfy the shinespark requirement. On the other hand, with excessFrames it can also go the other way around. The ideal would be if we could put the requirements in chronological order of what happens in the strat.

The situations most likely to have problems is shinesparks in heated rooms. There are a number of heated rooms where the heat frames before and after the spark should possibly be split apart from each other in order to make the logic fully accurate. If the endpoint of the shinespark is in a part of the room that requires more than 29 energy afterward to reach safety (i.e., a door, a farm, or an item which could possibly be an ETank) then probably none of this matters. But if the endpoint is within 29 energy of reaching safety then it can make a difference.

@osse101
Copy link
Contributor

osse101 commented Sep 13, 2023

Here's what I think the new grouping would look like:

heatFrames used while shinesparking
{shinespark: frames, excess}
heatFrames used in crash animation and when running to next node

Most of the time excess is under 10 frames. And we can try to ignore the extra run distance on heatFrame count. But LN pillar room has some long ExcessShinesparkFrame counts. So it may need an additional {"or": shinespark, heatFrames} to cover running to the door after stopping early.

@kjbranch
Copy link
Contributor

kjbranch commented Jun 9, 2024

Closed #1101 as it was on the same topic.

With the complexity of excess frames, heat frames, and the possibility of adding energy free shinesparks, would it make more sense to add a heated shinespark option?

@osse101
Copy link
Contributor

osse101 commented Jun 17, 2024

HeatFrames
Shinespark
HeatFrames

Should cover most of the cases. Maybe we need to think about how to divide the heat frames. More initial heatFrames would trigger the excessFrames so you need to be sure that the secondary heatframes is a proper number for reaching the door.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants