Add TOTALVI integration method - Githubissues

openpipelines-bio / openpipeline

https://openpipelines.bio

MIT License

25 stars 11 forks source link

Add TOTALVI integration method #386

Closed VladimirShitov closed 10 months ago

VladimirShitov commented 1 year ago

Changelog

Add TOTALVI integration component according to the tutorial: https://docs.scvi-tools.org/en/stable/tutorials/notebooks/scarches_scvi_tools.html#Reference-mapping-with-TOTALVI"

Issue ticket number and link

Contributes to #340

Checklist before requesting a review

[x] I have performed a self-review of my code
[x] Conforms to the Contributor's guide
Check the correct box. Does this PR contain:
- [ ] Breaking changes
- [x] New functionality
- [ ] Major changes
- [ ] Minor changes
- [ ] Bug fixes
[ ] Proposed changes are described in the CHANGELOG.md
[x] CI tests succeed!

DriesSchaumont commented 11 months ago

@VladimirShitov could you merge main? :)

VladimirShitov commented 11 months ago

All done, please check :)

VladimirShitov commented 10 months ago

I believe everything is done :) Happy to hear your feedback @DriesSchaumont

Some test in dataflow/concat is failing but it should not be connected to this PR

ddemaeyer commented 10 months ago

@DriesSchaumont and @rcannood what is the current state of this pull request? Are there still outstanding issues?