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

Cutscenes being skipped leads to incorrect time of day in some cases #1902

Open
ETR-BTF opened this issue Feb 25, 2023 · 6 comments
Open

Cutscenes being skipped leads to incorrect time of day in some cases #1902

ETR-BTF opened this issue Feb 25, 2023 · 6 comments
Labels
Type: Bug Something isn't working

Comments

@ETR-BTF
Copy link

ETR-BTF commented Feb 25, 2023

This is not a new bug. I'm just opening this issue for documentation purposes.

Some events that normally set the time in vanilla due to a cutscene either set it to the incorrect time or don't set it at all in OoTR due to cutscenes being skipped.

all blue warps should also set the time, although water ends up setting it to morning rather than noon
talking to zelda and going adult should both set it to noon since the normal cutscene sequences go through the cs map and chamber of sages, respectively
getting suns should advance it based on the current time of day (this is the natural in-game way of getting dampe time easily)
and probably a few others im forgetting

Additionally, the "Default (10:00)" starting time option in the gui is a bit of a misnomer since in vanilla you don't get control over Link until noon. This is because by the time this setting was added, people were already used to 10:00 being the "default" starting time of day in Randomizer. So "default" doesn't refer to vanilla, but OoTR.

@r0bd0g
Copy link

r0bd0g commented Feb 25, 2023

Sometimes it can be a bit weird to handle this since it might depend on the speed that you advance through the cutscene.

IMO the most egregious case currently in the release (and it's been this way for forever) is Zelda's Lullaby not setting it to noon.

The one for going adult is a bit weird since nothing really happens when you go adult for the first time. And what if you start as adult, should it still make it noon the first time you swap or the first time you go adult again? But if there's ever a light medallion check in ToT I might suggest getting that make it noon.

@Cuphat Cuphat added the Type: Bug Something isn't working label Mar 6, 2023
@Cuphat Cuphat changed the title [Bug] Cutscenes being skipped leads to incorrect time of day in some cases Cutscenes being skipped leads to incorrect time of day in some cases Mar 6, 2023
@r0bd0g
Copy link

r0bd0g commented Mar 14, 2023

I want to point out also that beating dungeons setting time of day might actually cause some non-repeatable access in rare cases.

@fenhl
Copy link
Collaborator

fenhl commented Jul 9, 2023

It's worth noting that #2034 currently contains code that explicitly sets time of day in blue warps to emulate the vanilla behavior. We may want to consider removing that code to address the nonrepeatable access.

@fenhl
Copy link
Collaborator

fenhl commented Sep 18, 2024

With #2287, we have eliminated the last instance of nonrepeatable ToD access that could cause softlocks, by making blue warp ToD changes repeatable. We should avoid reintroducing nonrepeatable ToD access, so changing ToD when going adult for the first time or checking Song from Royal Familys Tomb should not be added. Therefore, the remaining things to do here are:

  • Reintroduce Song from Impa setting time to noon. This is safe because the cutscene takes you either to Hyrule Field or the Hyrule Castle grounds, depending on ER settings, both of which are locations where ToD passes.
  • Decide what to do with the starting ToD options.
    • I could see simply changing the default value of the setting to noon for accuracy with respect to vanilla, but keeping the 10:00 option for the purpose of racing presets, since the racing community would most likely want to stay on that option. Would that make sense? Is fixing the inaccuracy worth the confusion the change would cause?

@r0bd0g
Copy link

r0bd0g commented Sep 18, 2024

I would like to be able to select any starting time of day.

(I've heard there are some weird edge cases right around the transition between days and nights. Does anybody know what the specifics of those are? There might need to be some logic adjustments to be able to handle those.)

@fenhl
Copy link
Collaborator

fenhl commented Sep 18, 2024

That could be a decent way of solving this. Maybe just changing the dropdown to a 0–23 slider to allow selecting any full hour would work without having to exhaustively check if these edge cases exist.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants