-
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
Refactor neighbors, leiden UMAP into separate workflow. #942
Conversation
Code LGTM, but the integration tests are still failing atm |
They worked after merging main: |
Aren't those the unit tests? There's some other integration tests failing here: https://github.com/openpipelines-bio/openpipeline/actions/runs/12655368361 |
You're right, I was thinking about another PR 🙄 I'm running the tests now |
Integration tests are green now: https://github.com/openpipelines-bio/openpipeline/actions/runs/12689751983 |
Changelog
Merge #941 first!Refactor neighbors, leiden UMAP into separate workflow.
Issue ticket number and link
Closes #xxxx (Replace xxxx with the GitHub issue number)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!