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

--print-clusters bug with --sphere option #18

Closed
nlubock opened this issue Feb 1, 2018 · 2 comments
Closed

--print-clusters bug with --sphere option #18

nlubock opened this issue Feb 1, 2018 · 2 comments
Assignees
Labels

Comments

@nlubock
Copy link

nlubock commented Feb 1, 2018

Hello,

There appears to be a bug when using the --print-clusters flag with the --sphere clustering option. The resulting clusters are just repeats of the centroid. Here's a reproducible example from the latest master e125759

> cat test.tsv
GGGGGGGGGGGGGGGGGGGG	50
AGGGGGGGGGGGGGGGGGGG	10
AAGGGGGGGGGGGGGGGGGG	5
AAAGGGGGGGGGGGGGGGGG	20
TGGGGGGGGGGGGGGGGGGG	20
TTTTTTTTTTTTTTTTTTTT	100

> starcode -d1 --sphere --print-clusters -i test.tsv
running starcode with 1 thread
reading input files
raw format detected
sorting
progress: 100.00%
spheres clustering
TTTTTTTTTTTTTTTTTTTT	100	TTTTTTTTTTTTTTTTTTTT
GGGGGGGGGGGGGGGGGGGG	80	GGGGGGGGGGGGGGGGGGGG,GGGGGGGGGGGGGGGGGGGG,GGGGGGGGGGGGGGGGGGGG
AAAGGGGGGGGGGGGGGGGG	25	AAAGGGGGGGGGGGGGGGGG,AAAGGGGGGGGGGGGGGGGG

Hope this helps!
Nate

@ezorita
Copy link
Collaborator

ezorita commented Feb 2, 2018

Indeed, there seems to be a bug in sequence transfer/canonical annotation. Thanks for reporting.

@ezorita ezorita self-assigned this Feb 2, 2018
@ezorita ezorita added the bug label Feb 2, 2018
@ezorita ezorita closed this as completed in 6ea3f13 Feb 2, 2018
@ezorita
Copy link
Collaborator

ezorita commented Feb 2, 2018

This seems to solve the issue. Please pull the last commit to master.

Thanks!

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

No branches or pull requests

2 participants