Skip to content

Display UTC/GPS time on p1_display X axis and hover text. - #419

Merged
adamshapiro0 merged 47 commits into
masterfrom
utc-time
Jul 29, 2026
Merged

Display UTC/GPS time on p1_display X axis and hover text.#419
adamshapiro0 merged 47 commits into
masterfrom
utc-time

Conversation

@adamshapiro0

@adamshapiro0 adamshapiro0 commented Jul 29, 2026

Copy link
Copy Markdown
Collaborator

New Features

  • Added options for displaying UTC (default) or GPS time on all plots
  • Include UTC and GPS time in hover text on all plots
  • Added bulk data (post-processing) support to Python TimeProvider class
  • Added new --time-type={utc,gps,p1,relative} option to control X axes

Changes

  • Deprecated --time-axis option in favor of new --time-type
  • Removed Plotly text= arrays in favor of more efficient customdata= + custom JS hover callback implementation
  • Removed deprecated t0 and system_t0 class members in favor of new resolve functions, with direct access to self.reader where needed
  • Removed deprecated absolute time plot from time scale figure (information now contained in hover text)
  • Grouped all GNSS plots together consistently

Fixes

  • Fixed packing/unpacking errors in Python CalibrationStatus messages
  • Added new custom hover tooltip implementation to resolve Plotly race condition causing intermittent hover failures

Examples

image image image

@adamshapiro0
Added post-process bulk load support to TimeProvider.
f9e527a
@adamshapiro0
Auto-detect P1 == GPS time.
511d2b9
@adamshapiro0
wip! Added --time-type argument.
43c2787
@adamshapiro0
Fixed CalibrationStatus.pack() array index error.
7e6b2dc
@adamshapiro0
Documentation/comment cleanup.
2365b6f
@adamshapiro0
Exposed GPS to datetime function.
f2b1a22
@adamshapiro0
Cache GPS/POSIX offset.
be34308
@adamshapiro0
Fixed CalibrationStatus.pack() fixed-point percent rounding.
7588f64
@adamshapiro0
wip! Add UTC/GPS timescale support to calibration plot.
7b2aea5
@adamshapiro0
Use existing sec/week constant for GPS string generation.
96c92a0
@adamshapiro0
Include title in axis layout, and separate layout function from data …
8f760b4 
…resolution.
@adamshapiro0
Add UTC/GPS timescale support to GNSS plots.
b477b78
@adamshapiro0
Fixed off-by-1 in GNSS signals plot hover text.
56f024c
@adamshapiro0
Fixed a typo in latency plot title.
29b807c
@adamshapiro0
Add UTC/GPS timescale support to pose plots.
5ec351c
@adamshapiro0
Add UTC/GPS timescale support to latency/status profiling plots.
d63be0f
@adamshapiro0
Add UTC/GPS timescale support to measurement plots.
00078df
@adamshapiro0
Add UTC/GPS timescale support to displacement plots.
53d586f
@adamshapiro0
Increased hover text update speed.
1c2becc
@adamshapiro0
Add UTC/GPS timescale support to map.
a341a42
@adamshapiro0
Added system time support to _resolve_x_axis().
f96ce9c
@adamshapiro0
Use _resolve_x_axis() for reset timing plot.
9c6fcd3
@adamshapiro0
Use event system time numpy array directly in time scale plot.
d11d545
@adamshapiro0
Use system time hover JS for reset timing plot.
3108948
@adamshapiro0
Updated time scale plot to use hover JS.
715486b
@adamshapiro0
Removed self.time_axis.
b55ae50
@adamshapiro0
Removed self.*_time_label.
d440573
@adamshapiro0
Use _resolve_x_axis() for all nav engine trace for all wheel speed ti…
2c6ca09 
…me options.
@adamshapiro0
Use self.reader.t0 directly where appropriate.
34975e7
@adamshapiro0
Use _resolve_x_axis() for time scale plot.
6079f57
@adamshapiro0
Removed unused self.t0.
d0680cf
@adamshapiro0
wip! Custom tooltip.
5305085
@adamshapiro0
Apply custom tooltip to all plots.
51b89bc
@adamshapiro0
Let name, value label, and value be hidden.
df0d092
@adamshapiro0
Hide label and name in stationary status plot.
e71b483
@adamshapiro0
Display text instead of value for custom y tick axes.
d1795cf
@adamshapiro0
Removed unused self.system_t0.
de5dc63
@adamshapiro0
Group all GNSS plots together.
7bb4f3a
@adamshapiro0
Fixed system status axis label.
fe6f51a
@adamshapiro0
Removed deprecated absolute time subplot from time scale plot.
23d223e 
The hover text now includes all times (except system time).
@adamshapiro0
Use the custom hover tooltip by default.
d9c3d6b
@adamshapiro0
Combined --time-type and deprecated --time-axis.
af3aabb
@adamshapiro0
Fixed missing type hint import.
85480e9
@adamshapiro0
Use p1_time explicitly for dP1 time scale plot.
676f7d9
@adamshapiro0
Formatting cleanup.
0b49885
@adamshapiro0
Plot time scale against UTC/GPS time too.
634c456
@adamshapiro0
Updated custom tooltip comment.
92beaf5
@adamshapiro0
adamshapiro0 requested a review from bhers4 July 29, 2026 17:18
@adamshapiro0 adamshapiro0 self-assigned this Jul 29, 2026
bhers4
bhers4 approved these changes Jul 29, 2026
View reviewed changes

@bhers4 bhers4 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

@adamshapiro0
adamshapiro0 merged commit c33a8a7 into master Jul 29, 2026
27 checks passed
@adamshapiro0
adamshapiro0 deleted the utc-time branch July 29, 2026 23:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bhers4 bhers4 bhers4 approved these changes

Assignees

@adamshapiro0 adamshapiro0

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@adamshapiro0 @bhers4