8000 add sequence read in for 10x json by MiTPenguin · Pull Request #619 · scverse/scirpy · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

add sequence read in for 10x json #619

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

MiTPenguin
Copy link

Previously, full nt or aa tcr sequences from 10x contigs are not being read in by read_10x_vdj. This PR adds those attributes when read from 10x json output.

Copy link
Collaborator
@grst grst left a comment

Choose a reason for hiding this comment

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

Thanks for adding this.

Two comments:

  • I think one field name is incorrect
  • Does the same information also exist in the corresponding .csv file? I.e. should we also add support for these fields there?

@@ -130,6 +130,9 @@ def _read_10x_vdj_json(
chain["productive"] = contig["productive"]
chain["is_cell"] = contig["is_cell"]
chain["high_confidence"] = contig["high_confidence"]
chain["sequence"] = contig["sequence"]
chain["aa_sequence"] = contig["aa_sequence"]
Copy link
Collaborator

Choose a reason for hiding this comment

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

Copy link
Author

Choose a reason for hiding this comment

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

you are correct. I can also look into adding support for csv load on this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0