busimus / cutelog

GUI for logging
MIT License
479 stars 45 forks source link

float type is also valid after deserialize #25

Closed ggajoch closed 2 years ago

ggajoch commented 2 years ago

Usecase: default python logging with SocketHandler. Used with remote debugging (hence lags between log creation and display).

Problem: Time column is not the time of log creation, but local cutelog reception time.

Without float type considered as valid, created dict field becomes string, later to be overwritten completely in logger_tab.py:155. This PR fixes this issue.

busimus commented 2 years ago

Thanks for the fix.

I think I didn't do this initially because it caused timestamps to be shown in scientific notation in the detail table, and didn't notice that it "broke" timestamps. So I added a commit to display values in the detail table as strings, hopefully this doesn't break something else.