Skip to content
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

Problems with Harmony k-means calculation #239

Open
wlason opened this issue Mar 20, 2024 · 2 comments
Open

Problems with Harmony k-means calculation #239

wlason opened this issue Mar 20, 2024 · 2 comments

Comments

@wlason
Copy link
Contributor

wlason commented Mar 20, 2024

It's a problem with harmonypy so I opened an issue in their repo.
slowkow/harmonypy#10 (comment)

I wonder if adding nclust parameter in pipeline.yaml in the integration workflow might help?

@bio-la
Copy link
Collaborator

bio-la commented Mar 21, 2024

@wlason can you put in a Pull request with this feature added?
the integration CI will pick it up and test if harmony runs with the additional parameter.

@olivermccallion
Copy link

Quick comment to say I ran into this same issue. For me, I had an empty category in my protein modality. Filtering the unused category fixed the issue for me (varying nclust and using Kmeans2 v kmeans didn't)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants