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

Output cluster metadata in separate file #259

Merged
merged 2 commits into from
Nov 15, 2023
Merged

Output cluster metadata in separate file #259

merged 2 commits into from
Nov 15, 2023

Conversation

jakobnissen
Copy link
Member

@jakobnissen jakobnissen commented Nov 10, 2023

The overall idea here is that we output more metadata about generated clusters, allowing our users to more easily get an idea of the cluster quality.

I would like to ping off ideas around this with you guys.

See #240

Based on #251, though the changes are small enough that they can be cherry-picked to existing master if necessary

@jakobnissen jakobnissen added the DO NOT SQUASH Each commit in this PR should be kept individually label Nov 10, 2023
@jakobnissen jakobnissen changed the base branch from master to binsplit November 10, 2023 13:16
Base automatically changed from binsplit to master November 14, 2023 10:28
Each Cluster object now also stores the actual observed peak/valley ratio for
the given cluster, where it previously only stored the maximally allows PVR.
This metadata can be valuable, as clusters with low PVR are usually of higher
quality.
This file contains information about the clusters, which can be useful for users,
especially in determining which contigs might be good or bad.
@jakobnissen jakobnissen changed the title [WIP] Output cluster metadata in separate file Output cluster metadata in separate file Nov 15, 2023
@jakobnissen jakobnissen marked this pull request as ready for review November 15, 2023 08:48
@jakobnissen jakobnissen merged commit e0960b9 into master Nov 15, 2023
@jakobnissen jakobnissen deleted the cluster_info branch November 15, 2023 08:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DO NOT SQUASH Each commit in this PR should be kept individually
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant