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

Update annotSeg_7102f1c.pl #471

Open
wants to merge 19 commits into
base: develop
Choose a base branch
from
Open

Update annotSeg_7102f1c.pl #471

wants to merge 19 commits into from

Conversation

tizatt
Copy link

@tizatt tizatt commented Sep 26, 2022

AnnotSeg creates a window from which genes are selected from the ensembl gtf. This window was found to be too large and is annotating beyond the actual region of genes. Because it was doing a division/multiplication before a subtraction - this caused windows to be thousands of bp larger than what they actually should be. This fix takes care of making the window size more accurate by doing the subtraction/addition before multiplication/division.

PedalheadPHX and others added 19 commits August 27, 2020 09:50
Lancet changes to sync with other pipelines
AnnotSeg creates a window from which genes are selected from the ensembl gtf.  This window was found to be too large and is annotating beyond the actual region of genes.  Because it was doing a division/multiplication before a subtraction - this caused windows to be thousands of bp larger than what they actually should be.  This fix takes care of making the window size more accurate by doing the subtraction/addition before multiplication/division.
@PedalheadPHX
Copy link
Member

@tizatt can you update teh pull request to be to develop, this update will require a release update. @bryce-turner is this code running in tempe?

@tizatt
Copy link
Author

tizatt commented Sep 26, 2022

Ok yes I can do it for the develop

@tizatt tizatt changed the base branch from main to develop September 26, 2022 19:39
@bryce-turner
Copy link
Member

@tizatt can you update teh pull request to be to develop, this update will require a release update. @bryce-turner is this code running in tempe?

Currently not in tempe, avoiding a perl container for an output not many people use. Plus the implementation is theoretical.

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

Successfully merging this pull request may close these issues.

3 participants