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

percentVJ all 0 #415

Closed
jiw181 opened this issue Oct 8, 2024 · 4 comments
Closed

percentVJ all 0 #415

jiw181 opened this issue Oct 8, 2024 · 4 comments
Labels
more info needed Waiting on more information

Comments

@jiw181
Copy link

jiw181 commented Oct 8, 2024

Hello,

For the percentVJ function, I got all 0 in the output for 2 of the samples. However, I am sure the V and J are present for the 2 samples in the Seurat object as shown in the attached file.
unknown_error.xlsx

If I only run the 2 samples, I got error: Error in combine_vars():
! Faceting variables must have at least one value

If I only run 1 sample, I got error:
Error in View : 'names' attribute [1] must be the same length as the vector [0]

Any suggestions?

Many thanks

@ncborcherding
Copy link
Owner

Hey @jiw181,

Thanks for reaching out.

Please make a reproducible example for this issue so I know what you are experiencing.

Also with the example, please make sure to include the output of sessionInfo()

Thanks,
Nick

@ncborcherding ncborcherding added the more info needed Waiting on more information label Oct 8, 2024
@jiw181
Copy link
Author

jiw181 commented Oct 8, 2024

Hello Nick @ncborcherding,

Thank you for the prompt response! Please find the .html with sessionInfo and intermediate files attached.
rstudio-export (47).zip

Thank you!

ncborcherding added a commit that referenced this issue Oct 8, 2024
Fix issue #415
No longer assumes > 1 group for faceting
@ncborcherding
Copy link
Owner

Found the issue - there was an assumption that there would be more than 1 sample during the plotting phase. It looks like subseting it to one specific orig.ident was what was causing it to break.

Got it fixed and in the dev branch. I will need to do some more testing this week and get it merged to the main branch.

For now, you can install the dev branch with:

devtools::install_github("ncborcherding/scRepertoire@dev")

Thanks,
Nick

@jiw181
Copy link
Author

jiw181 commented Oct 8, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
more info needed Waiting on more information
Projects
None yet
Development

No branches or pull requests

2 participants