-
Notifications
You must be signed in to change notification settings - Fork 375
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
[WIP] Pyro cell2location #930
Conversation
merge scvi-tools release
merge main master branch
merge master
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems like a lot of comments but I don't think the model is that far off from being ready to go.
Thanks a lot for the feedback! It will take a while to address (because I also want to keep things consistent with main cell2location pymc3 repo) and some of it I think needs discussion - but good to have feedback! |
Codecov Report
@@ Coverage Diff @@
## master #930 +/- ##
===========================================
- Coverage 90.64% 76.57% -14.08%
===========================================
Files 91 103 +12
Lines 6908 8434 +1526
===========================================
+ Hits 6262 6458 +196
- Misses 646 1976 +1330
Continue to review full report at Codecov.
|
Moved the model to https://github.com/BayraktarLab/cell2location/tree/master/cell2location/models and closing this PR (accidentally closed and reopened a few times) |
No description provided.