Skip to content

Convert metadata to fixed-length byte strings - #1522

Merged
tskisner merged 1 commit into
masterfrom
tsk_hdf5_meta
Jan 31, 2026
Merged

Convert metadata to fixed-length byte strings#1522
tskisner merged 1 commit into
masterfrom
tsk_hdf5_meta

Conversation

@tskisner

Copy link
Copy Markdown
Member

When loading context metadata or HK data into toast, convert any numpy unicode arrays into fixed-length bytes. Also add a timer around the HK loading, which is slow.

Note that this cannot be merged until hpc4cmb/toast#910 is merged and a toast tag is made.

@tskisner
tskisner requested a review from keskitalo January 15, 2026 04:37

@keskitalo keskitalo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Seems uncontroversial.

@tskisner
tskisner force-pushed the tsk_hdf5_meta branch 3 times, most recently from 3aef271 to 798b820 Compare January 23, 2026 06:30
@tskisner
tskisner force-pushed the tsk_hdf5_meta branch 2 times, most recently from 297b28f to 7015aa9 Compare January 30, 2026 17:50
- Support loading raw data as int32 DAQ units.

- When loading metadata, ensure that byte arrays are converted
  to strings as the starting point.  These get automatically
  converted to / from bytestrings during HDF5 save / load.

- The HKManager now excludes several large fields by default
  (WG / HWP encoders and ACU udp stream).

- HKManager converts all byte strings to unicode on load, and
  then separately converts to / from bytestrings during HDF5
  save / load.

- Small fix to always use "so3g compatibility mode" when
  converting boresight Az/El to RA/DEC on load.
@tskisner
tskisner merged commit 992dcf3 into master Jan 31, 2026
5 checks passed
@tskisner
tskisner deleted the tsk_hdf5_meta branch January 31, 2026 18:36
tskisner added a commit that referenced this pull request Feb 13, 2026
Found through use of some new integration tests in `sopipetest`,
PR will be opened soon with those.
tskisner added a commit that referenced this pull request Feb 13, 2026
Found through use of some new integration tests in `sopipetest`,
PR will be opened soon with those.
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