sfa-siard / SiardGui

SIARD Suite - SiardGui
Other
17 stars 3 forks source link

Loading .siard in SiardGui with javaw -jar siardgui.jar -Duser.timezone=GMT still creates timezone shift #30

Closed ibbenz closed 2 years ago

ibbenz commented 2 years ago

Time in Date is shifted by hours after uploading data. It does not matter if: javaw -jar siardgui.jar -Duser.timezone=GMT or javaw -jar siardgui.jar is used.

Picture of the original Dates+Time Original_Time

Picture of the Dates+Time after uploading it to SiardSuite TimeShift_AfterUpload

The .siard-file (uploaded as .zip) 0_1-3_Northwind_simple_2022_07_10_V04.zip

mburri commented 2 years ago

@ibbenz Thanks for your detailed report and the great finding. As always - dealing with timezones and daylight savings is quite painful...

I think the SIARD Suite uses UTC per default - If I start the application with java -jar -Duser.timezone=UTC ./lib/siardgui.jar the dates/ times are shown correctly.

Hope that helps - but I think we should look into this in more detail for future releases.

image

ibbenz commented 2 years ago

Thank you very much for your fast reply. I mixed up a bit the order of the commands.

mburri commented 2 years ago

You're welcome - yes, I noticed that too - the options should be placed in front of the jar file path.

mburri commented 2 years ago

closing this for now since the issue also depends on the users system settings - but we will have to look into how we should handle dates/ times and timezone to get the best user expecerience...