Skip to content

feat(io): optional z-coordinate passthrough (include_z, default False)#57

Open
EliHei2 wants to merge 1 commit into
mainfrom
feat/include-z
Open

feat(io): optional z-coordinate passthrough (include_z, default False)#57
EliHei2 wants to merge 1 commit into
mainfrom
feat/include-z

Conversation

@EliHei2

@EliHei2 EliHei2 commented Jun 1, 2026

Copy link
Copy Markdown
Collaborator

Add an include_z preprocessor parameter (threaded through get_preprocessor); when enabled the transcript z coordinate is carried into the standardized output. Defaults to False so base behaviour is unchanged. Independent of the QV change.

Add an include_z preprocessor parameter (threaded through get_preprocessor);
when enabled the transcript z coordinate is carried into the standardized
output. Defaults to False so base behaviour is unchanged. Independent of the
QV change.
@Tobiaspk

Tobiaspk commented Jun 2, 2026

Copy link
Copy Markdown
Collaborator

It looks like rename_map is defined in the Xenium preprocessor but used in CosMX. Probably should be switched. Just two questions:

@Tobiaspk

Tobiaspk commented Jun 8, 2026

Copy link
Copy Markdown
Collaborator

A couple more considerations: how does including z impact accuracy vs. memory and time cost? My concern is that kNN etc. might explode, we should avoid that.

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