-
Notifications
You must be signed in to change notification settings - Fork 49
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
[Suggestions] Suggest UTC Now as standard Time Format, confused by logging times atm #639
Comments
There should not be a time difference between the time zones you mention. @RGrabichler could you check that you call time synch in your program for the context? Something like this: |
You're right. It should be aligned... will have a look on Monday. |
/cib |
We are unable to reproduce. |
for me it is UTC+1, Berlin Time. |
Same as me! 😊 You can try running it with the second argument, specifying the AMS ID of your server as follows: |
ok, in this specific scenario, where i tinker with the Diagnosis Component, i use UmRT and blazor runs on local Dev Pc/ |
It looks like that might be an issue with UmRT.
? |
@TomKovac any comment on this? @RGrabichler sorry if you answered this question before, but what would be your correct local time in the example above? @peterbarancek what time do we set to the TimeStamp in the messenger? |
in this case, 21.44 was the right time. |
In the Issues #519 ]New-Feature] Extended Diagnostics in Blazor, i tinker around with the TcoMessages, i i noticed a TimeStamp Issue.
this is Timestamp logged in Console
this is the Timestamp in the Plc
this is my local Timestamp in Germany and Summertime
this is UTC
And this is a complete mess in my opinion.
I would suggest, to allways log in UTC, and everybody can display whatever he wants.
I only assume, the 15.50 timestamp comes from local Slovenia Time? UTC+2 ? In difference to Berlin +1 ?
So in my opinion, the Logger should use per default UtcNow, the Connector should use UtcNow, and
the Wpf or Blazor View can render whatever they want for the User, probably Utc with corresponding TimeZone.
The text was updated successfully, but these errors were encountered: