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

scvi doublet predictions seem off in comparison to four alternative tools #3120

Open
robertzeibich opened this issue Jan 2, 2025 · 2 comments

Comments

@robertzeibich
Copy link

robertzeibich commented Jan 2, 2025

scvi doublet predictions seem off (too many cells that should not be doublets) in comparison to four alternative tools.

Image

    adata.obs.index.name = None
    scvi.model.SCVI.setup_anndata(adata)
    vae = scvi.model.SCVI(adata)
    vae.train() # batch_size=130

    solo = scvi.external.SOLO.from_scvi_model(vae)
    solo.train(batch_size=130) # batch_size=130
    # vae.train(batch_size=130) and solo.train(batch_size=130) # ValueError: Expected more than 1 value per channel when training, got input size torch.Size([1, 128])
    # change batch_size when error is thrown
    
    doublet_prediction = solo.predict()
    doublet_prediction['prediction'] = solo.predict(soft = False)

    adata.obs = adata.obs.reset_index().merge(doublet_prediction.reset_index(),on='index',how='left').set_index('index')
    adata.obs.index.name = None
[Paste the error output produced by the above code here]

Versions:

1.2.0

@canergen
Copy link
Member

canergen commented Jan 2, 2025

Hi, can you show the history of the solo model? Did it converge? I assume the data is from a single batch?

@canergen canergen added enhancement and removed bug labels Jan 2, 2025
@robertzeibich
Copy link
Author

robertzeibich commented Jan 10, 2025

Thank you for getting back to me.

I am not entirely sure what you mean with converge.
Using solo on a single batches only.

I have attached one output file:
log-3713343_20_scvi_doublets.txt

Is there a way to increase the number of workers?

I suspect that this is the same tool (https://github.com/calico/solo/tree/master). Is that correct?

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

No branches or pull requests

2 participants