search

Crimso777 / Factorio-Access

An accessibility mod for the video game Factorio, making the game accessible to the blind and visually impaired.
MIT License
14 stars 13 forks source link

Bug: Time is reported incorrectly when you press T more than once #38

Closed LevFendi closed 1 year ago

EphDoering commented 2 years ago

I think it's reporting the in game time of day correctly such that it'll cycle through 24 hours for each day/night cycle. However, it's kinda like we're in a different timezone, because in vanilla the night day cycle goes as follows: dusk: 0.25 or 6:00 night: 0.45 or 10:48 morning: 0.55 or 13:12 dawn: 0.75 or 18:00

I imagine the purpose of this feature is to let people know when their solar panels will be producing, but since that cycle can vary with mods I'm not sure the best way to go about announcing it. Maybe something like "nighttime, morning in x minutes"?

LevFendi commented 2 years ago

If we end up reporting the solar panel yield from the electricity overview interface, perhaps this time feature would not be needed at all?

Or it can simply report "it is dawn/dusk/day/night time".

EphDoering commented 2 years ago

Yeah that's a good point. I usually get my information about how long the day is and where in the day night cycle it currently is from the graphs on the electricity grid. I think knowing how far along you are through the light cycle is pretty important for diagnosing if you need more accumulators or more solar. I'm trying to think of the best way to convey the information that I normally get from the solar production plot.

On Sat, Jul 30, 2022, 8:25 PM Fendi @.***> wrote:

If we end up reporting the solar panel yield from the electricity overview interface, perhaps this time feature would not be needed at all?

Or it can simply report "it is dawn/dusk/day/night time".

— Reply to this email directly, view it on GitHub https://github.com/Crimso777/Factorio-Access/issues/38#issuecomment-1200319518, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADA3Y6K57NGA44QNPMHVNYLVWXBY7ANCNFSM54SNNT4Q . You are receiving this because you commented.Message ID: @.***>