-
Notifications
You must be signed in to change notification settings - Fork 14
Add STACAS as new method component #58
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
Conversation
|
@JGarnica22 Could you merge upstream/main into this PR so the github actions can run? 🙇 |
thanks, done! |
|
Hi, It looks like the check failure is related to reticulate and its Python discovery/installation mechanism. From the logs, the STACAS package installs fine, but when the test is run, reticulate tries to provision its own Python: This suggests that the temporary Python environment created by reticulate is not persisted or is being erased/moved between build/test phases inside the container. I found this same problem when I run the test locally but I could solve it by not installing dependencies via This makes me think the problem is container-specific:
Would you recommend a preferred way forward here?
Thanks for the guidance! |
Co-authored-by: Michaela Müller <51025211+mumichae@users.noreply.github.com>
…ontainer baser_r:1 Co-authored-by: Luke Zappia <lazappi@users.noreply.github.com>
|
Hi, thanks for your support, I merged the changes. |
|
Added changes into a new PR: |
Describe your changes
Add
methods/stacasnew metric component.Add non-supervised version of STACAS tool for integration of single-cell transcriptomics data.
This functionality enables correction of batch effects while preserving biological variability without requiring prior cell type annotations.
More information at:
Checklist before requesting a review
I have performed a self-review of my code
Check the correct box. Does this PR contain:
Proposed changes are described in the CHANGELOG.md
CI Tests succeed and look good!