-
Notifications
You must be signed in to change notification settings - Fork 14
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
Add densmap component #748
Conversation
… 739-add-densmap-component
… 739-add-densmap-component
… 739-add-densmap-component
…bio/openpipeline into 739-add-densmap-component
@jakubmajercik Seems like something went wrong after you recent updates, care to have a look? 😃 |
The tests fail because |
Of course! I keep forgetting to update the test resources 👀 |
Changelog
densMAP dimensionality reduction component implemented. Neighbor computing updated to output KNN results as input to densMAP. Test resources need to be updated to include these extra fields in the mudata/anndata objects.
Issue ticket number and link
Closes #739
Checklist before requesting a review
I have performed a self-review of my code
Conforms to the Contributor's guide
Check the correct box. Does this PR contain:
Proposed changes are described in the CHANGELOG.md
CI tests succeed! - failing due to modified neighbor computing component.