Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

small fix for ReadXenium #8298

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

alikhuseynov
Copy link

Hi, this is just a quick fix for ReadXenium to always return dataframes when sapply is used.
..see issue #8265
thanks

jsicherman and others added 2 commits January 10, 2024 13:39
fix: don't load unassigned codewords when there are none
@jsicherman
Copy link
Contributor

@AustinHartman any update on this? Would be awesome to get it merged when you've got some cycles. Also about to tag you on another Xenium-related PR

@alikhuseynov
Copy link
Author

@jsicherman I was thinking, probably it would be better to make reading transcript coords optional, such that users can decide themselves if to load molecules or not. I added add.molecules for readVizgen some time ago. Since those coords are not used at all in any analysis in Seurat, only for visualization.

@jsicherman
Copy link
Contributor

@jsicherman I was thinking, probably it would be better to make reading transcript coords optional, such that users can decide themselves if to load molecules or not. I added add.molecules for readVizgen some time ago. Since those coords are not used at all in any analysis in Seurat, only for visualization.

Added in #8605 :)

@pmarks
Copy link
Contributor

pmarks commented Mar 27, 2024

Hi @dcollins15 and @Gesmira. We have a few PRs to improve Xenium support in flight -- this one and #8605. Austin seems to be offline these days. Who should we ping for help getting these merged?

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.

None yet

3 participants