CubeCoders / AMP

Issue tracking and documentation for AMP
https://cubecoders.com/AMP
207 stars 38 forks source link

Scheduler Timezone doesn't care about DST #1160

Open BroOtti opened 3 weeks ago

BroOtti commented 3 weeks ago

Bug Report

System Information

┏━━━━━━━━━━━━━━━━━━━━━━┳━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┓
┃ Key                  ┃ Value                                 ┃
┣━━━━━━━━━━━━━━━━━━━━━━╋━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┫
┃ Operating System     ┃ Linux - Debian GNU/Linux 12 on x86_64 ┃
┃ Product              ┃ AMP 'Callisto' v2.5.1.8 (Mainline)    ┃
┃ Virtualization       ┃ Docker                                ┃
┃ Application          ┃ Palworld                              ┃
┃ Module               ┃ GenericModule                         ┃
┃ Running in Container ┃ Yes                                   ┃
┃ Current State        ┃ Ready                                 ┃
┗━━━━━━━━━━━━━━━━━━━━━━┻━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┛

I confirm:

Symptoms

Reproduction

  1. Set Scheduler Timezone to a timezone which has DST
  2. Create a scheduler running at 3am
  3. While DST is active, the scheduler is one hour off and runs at 4am instead of 3am
PhonicUK commented 3 weeks ago

Workaround for now is to change the timezone when DST changes. I believe this is a Linux-specific issue. No resolution at this time until after the .Net 8 port is completed.

p0t4t0sandwich commented 2 weeks ago

I can see some use cases for a lack of DST (example here: https://discord.com/channels/266012086423912458/266012086423912458/1285646467289317481) where there's various players spread across different timezones, making it so that the offset itself matters more than DST itself (if that makes sense). That's my long winded way of saying there should be a setting for the offset, then another setting to toggle DST.