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
Salmon reports library info to General Stats table #1485
Salmon reports library info to General Stats table #1485
Conversation
Thanks a lot for the contribution - and sorry that it took that long to review this PR. Finally going through the backlog! Merged with master and slightly cleaned, looks good to me :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! Small suggestion but that's all..
"max": 1.0, | ||
"scale": "YlGn", | ||
}, | ||
"strand_mapping_bias": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think that we have any test data with this key yet..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(Not a blocker, but would be nice to get some if we can).
Co-authored-by: Phil Ewels <phil.ewels@seqera.io>
CHANGELOG.md
has been updatedSalmon module reports the following to the General Stats table:
libary_types
fromaux_info/meta_info.json
(hidden if all values are the same across all samples)compatible_fragment_ratio
andstrand_mapping_bias
from 'lib_format_counts.json'