-
Notifications
You must be signed in to change notification settings - Fork 352
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
MultiVI with Poisson/NB likelihood for ATAC? #2257
Comments
Hi @vitkl I'm linking @marianogabitto here as he developed multiVI and was also working on a different implementation of PoissonPeakVI. What's your opinion on this request? |
Yes, I also think it should be relatively straightforward and also remember that @marianogabitto already had an implementation |
Thanks for getting back to me about this! If you @marianogabitto have an implementation of this - would be great if you could share it or add it to scvi-tools (if makes sense for @canergen) |
Happy to review the PR. |
I don't understand the implementations well enough to easily try implementing this - but I can do a PR if I see the example implementation from @marianogabitto |
I'm sorry for the confusion that was a comment to @marianogabitto to assign me the PR if he creates one. |
Hi all, I am very interested in this implementation and I'm curious to know if there is any updates on it? |
@lauradmartens congrats on the new manuscript: https://www.biorxiv.org/content/10.1101/2024.09.19.613754v2. Any plan to continue with the Poisson MultiVI branch. |
Hi @martinkim0 and all
Similarly to #2249 by @lauradmartens would be great to have a MultiVI version with Poisson or even NB likelihood for ATAC data. How hard do you @martinkim0 think it is to add such option to MultiVI?
The text was updated successfully, but these errors were encountered: