Skip to content

Write NIfTI data in the byte order the header declares - #1061

Merged
effigies merged 1 commit into
nipreps:masterfrom
asaraog:fix/unsafe-write-endianness
Sep 18, 2026
Merged

effigies merged 1 commit into
nipreps:masterfrom
asaraog:fix/unsafe-write-endianness

Conversation

@asaraog

@asaraog asaraog commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

Closes #1029. array_to_file wrote the array in native byte order while the header kept its own, so a big-endian header produced little-endian data; passing header.get_data_dtype() keeps them consistent.

@effigies
effigies merged commit c2d86ef into nipreps:master Sep 18, 2026
12 of 13 checks passed
@effigies effigies added the backport candidate Change that may be backported to a maintenance branch label Sep 28, 2026
effigies added a commit that referenced this pull request Oct 2, 2026
1.14.5 (October 02, 2026)

Patch release in the 1.14.x series.

* FIX: Collect data interactions with sessions and bids filters (#1063)
* FIX: Write NIfTI data in the byte order the header declares (#1061)
* FIX: Drop bids_filters entries with no matching query (#1058)
* FIX: Disable colorbar after default switch (#1057)
* FIX: Use rigid affine initialization for BOLD mask (#1064)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport candidate Change that may be backported to a maintenance branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

DerivativesDataSink inconsistent byte order

2 participants