Skip to content

rust(feat): hdf5 and tdms preview shows time stamp channels#605

Merged
Brandon-Shippy merged 13 commits into
mainfrom
rust/tdms-and-hdf5-timestamp-preview
Jun 3, 2026
Merged

rust(feat): hdf5 and tdms preview shows time stamp channels#605
Brandon-Shippy merged 13 commits into
mainfrom
rust/tdms-and-hdf5-timestamp-preview

Conversation

@Brandon-Shippy

@Brandon-Shippy Brandon-Shippy commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

Description

🎫 ENG-11824 🎫

For HDF5 and TDMS import previews we show the timestamp and properly surface that to the user.
HDF5 1D we show each groups timestamp
HDF5 2D we show the time index for each 2D array
HDF5 Compound we show the group timestamp
TDMS shows waveform/detected timestamp

HDF5 1D:

Asset: hdf5-time-check
Run: time-check
Channels: {
  CHANNEL_DATA_TYPE_INT_64    timestamp
    description [time]
  CHANNEL_DATA_TYPE_DOUBLE    cpu_usage
  CHANNEL_DATA_TYPE_INT_64    disk_io
  CHANNEL_DATA_TYPE_DOUBLE    memory_usage
  CHANNEL_DATA_TYPE_INT_64    group1.timestamp
    description [time]
  CHANNEL_DATA_TYPE_INT_64    group1.current
  CHANNEL_DATA_TYPE_DOUBLE    group1.pressure
  CHANNEL_DATA_TYPE_DOUBLE    group1.temperature
  CHANNEL_DATA_TYPE_DOUBLE    group1.voltage
  CHANNEL_DATA_TYPE_INT_64    group2.timestamp
    description [time]
  CHANNEL_DATA_TYPE_DOUBLE    group2.acceleration
  CHANNEL_DATA_TYPE_DOUBLE    group2.altitude
  CHANNEL_DATA_TYPE_INT_64    group2.thrust
  CHANNEL_DATA_TYPE_DOUBLE    group2.velocity
  CHANNEL_DATA_TYPE_INT_64    group2.group3.timestamp
    description [time]
  CHANNEL_DATA_TYPE_DOUBLE    group2.group3.battery_current
  CHANNEL_DATA_TYPE_DOUBLE    group2.group3.battery_temp
  CHANNEL_DATA_TYPE_DOUBLE    group2.group3.battery_voltage
  CHANNEL_DATA_TYPE_DOUBLE    group2.group3.group4.cell_voltage
}

HDF5 2D:

Asset: hdf5-time-check
Run: time-check
Channels: {
  CHANNEL_DATA_TYPE_INT_64    group1.current.0
    description [time]
  CHANNEL_DATA_TYPE_DOUBLE    group1.current.1
  CHANNEL_DATA_TYPE_INT_64    group1.pressure.0
    description [time]
  CHANNEL_DATA_TYPE_DOUBLE    group1.pressure.1
  CHANNEL_DATA_TYPE_INT_64    group1.temperature.0
    description [time]
  CHANNEL_DATA_TYPE_DOUBLE    group1.temperature.1
  CHANNEL_DATA_TYPE_INT_64    group1.voltage.0
    description [time]
  CHANNEL_DATA_TYPE_DOUBLE    group1.voltage.1
}

HDF5 Compound:

Asset: hdf5-time-check
Run: time-check
Channels: {
  CHANNEL_DATA_TYPE_INT_64    sensors.timestamp
    description [time]
  CHANNEL_DATA_TYPE_DOUBLE    sensors.temperature
  CHANNEL_DATA_TYPE_DOUBLE    sensors.pressure
  CHANNEL_DATA_TYPE_DOUBLE    sensors.voltage
  CHANNEL_DATA_TYPE_INT_64    sensors.current
}

TDMS:

Asset: asset
Run: run
Channels: {
  CHANNEL_DATA_TYPE_INT_64    group_a_time_channel.Time
    description [time]
  CHANNEL_DATA_TYPE_DOUBLE    group_a_time_channel.voltage
    units V
    description Sine wave 1 Hz
  CHANNEL_DATA_TYPE_DOUBLE    group_a_time_channel.current
    units A
    description Cosine wave 1 Hz
  CHANNEL_DATA_TYPE_DOUBLE    group_b_waveform.voltage
    units V
    description Sine wave 1 Hz (waveform)
  CHANNEL_DATA_TYPE_DOUBLE    group_b_waveform.current
    units A
    description Cosine wave 1 Hz (waveform)
  CHANNEL_DATA_TYPE_INT_64    group_c_named_timestamp.Timestamp
    description [time]
  CHANNEL_DATA_TYPE_UINT_64   group_c_named_timestamp.counter
    units ticks
    description Monotonic counter
}

Verification

Manual testing end to end.

Added unit tests for auto detection.

@Brandon-Shippy Brandon-Shippy marked this pull request as ready for review June 1, 2026 22:12
@Brandon-Shippy Brandon-Shippy changed the title hdf5 and tdms preview shows time stamp channels rust(feat): hdf5 and tdms preview shows time stamp channels Jun 2, 2026
Comment thread rust/crates/sift_cli/src/cmd/import/hdf5/detect_hdf5_schema.rs Outdated
Comment thread rust/crates/sift_cli/src/cmd/import/hdf5/detect_hdf5_schema.rs Outdated
Comment thread rust/crates/sift_cli/src/cmd/import/hdf5/tests.rs Outdated
Comment thread rust/crates/sift_cli/src/cmd/import/hdf5/tests.rs Outdated
Comment thread rust/crates/sift_cli/src/cmd/import/hdf5/tests.rs
Comment thread rust/crates/sift_cli/src/cmd/import/tdms/detect_tdms_config.rs Outdated
Comment thread rust/crates/sift_cli/src/cmd/import/tdms/detect_tdms_config.rs Outdated
@Brandon-Shippy Brandon-Shippy requested a review from solidiquis June 3, 2026 18:39
@Brandon-Shippy Brandon-Shippy merged commit 34fb871 into main Jun 3, 2026
21 checks passed
@Brandon-Shippy Brandon-Shippy deleted the rust/tdms-and-hdf5-timestamp-preview branch June 3, 2026 19:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants