Skip to content

add sv2a pet atlas from NRU to examples#552

Open
bendhouseart wants to merge 2 commits intomasterfrom
atlas-sv2a-nru-pet
Open

add sv2a pet atlas from NRU to examples#552
bendhouseart wants to merge 2 commits intomasterfrom
atlas-sv2a-nru-pet

Conversation

@bendhouseart
Copy link
Copy Markdown
Contributor

Adds PET atlas to examples, will fail validator for version v1.11.1, fix coming in minor release v1.11.2.

@effigies
Copy link
Copy Markdown
Contributor

You're going to add hemi for volumetric data? Or are you using the FS-NIfTI hack for surfaces?

Either way, just use GIFTI.

@CPernet
Copy link
Copy Markdown
Contributor

CPernet commented Apr 16, 2026

I am quite certain the extension should be .mimap and not pet since the values are not PET values anymore (discussed many times in many different threads @mnoergaard

@effigies
Copy link
Copy Markdown
Contributor

If we want to use mimap, we need to finish off bids-standard/bids-specification#2262.

@CPernet
Copy link
Copy Markdown
Contributor

CPernet commented Apr 16, 2026

I agree - we cannot push a PET atlas with the wrong extension

@bendhouseart
Copy link
Copy Markdown
Contributor Author

I agree - we cannot push a PET atlas with the wrong extension

We need to update the spec then as any suffix is allowed. I'd rather change the schema than try to meddle with the spec directly, but maybe that's just me. This PR works with these examples and complies with the spec, so it is possible to have both.

@bendhouseart bendhouseart self-assigned this Apr 17, 2026
@bendhouseart bendhouseart added WIP Work in Progress PET labels Apr 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PET WIP Work in Progress

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants