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

Strand #91

Merged
merged 26 commits into from Apr 22, 2018

Conversation

Projects
None yet
2 participants
@yang-yangfeng
Contributor

yang-yangfeng commented Nov 15, 2017

two main things:
"-s" option which requires an int as argument to set strandness of rnaseq data: 0 = NONE (for unstranded), 1 = FR, 2 = RF

new function for obtaining strand info from bitwise flag, if rna prep was stranded
retains old XS function for setting strand, if rna prep was unstranded

Note: XS tags are required if data is unstranded. This is mainly an issue for STAR, which does not provide these by default. Need to mention this in documentation - if you want to use unstranded STAR bams, make sure you align with the "--outSAMstrandField intronMotif" option.

@coveralls

This comment has been minimized.

coveralls commented Nov 15, 2017

Coverage Status

Coverage decreased (-0.7%) to 88.951% when pulling a252a17 on strand into 9da076e on master.

@coveralls

This comment has been minimized.

coveralls commented Jan 6, 2018

Coverage Status

Coverage decreased (-0.6%) to 89.092% when pulling 5690304 on strand into 9da076e on master.

@coveralls

This comment has been minimized.

coveralls commented Jan 6, 2018

Coverage Status

Coverage increased (+0.2%) to 89.902% when pulling 93cd094 on strand into 9da076e on master.

@coveralls

This comment has been minimized.

coveralls commented Jan 7, 2018

Coverage Status

Coverage increased (+0.2%) to 89.932% when pulling d3b06a0 on strand into 9da076e on master.

@coveralls

This comment has been minimized.

coveralls commented Jan 7, 2018

Coverage Status

Coverage increased (+0.2%) to 89.932% when pulling d3b06a0 on strand into 9da076e on master.

yang-yangfeng added some commits Jan 7, 2018

@coveralls

This comment has been minimized.

coveralls commented Jan 7, 2018

Coverage Status

Coverage increased (+0.2%) to 89.911% when pulling 69f0d8a on strand into cacbb3b on master.

yang-yangfeng added some commits Apr 21, 2018

@yang-yangfeng yang-yangfeng merged commit 640a30b into master Apr 22, 2018

3 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.2%) to 89.911%
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment