-
Notifications
You must be signed in to change notification settings - Fork 2
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
short_long does not support gtf without gene line. #42
Comments
It is done exactly like you want. gene_ids are taken from transcript line if such a line exists. If not, they are taken from exon lines. |
My previous comment is for convert_to_ensembl function. You're right, get_shortest/longest_transcript functions don't work if there is no gene lines. I will try to fix that ... |
I just pushed a fixed version for this bug. Can you test it and tell me if it works ? |
This does not seem to fix the issue regarding #42.
|
This one is not fixed
|
If you add a "gtftk convert_ensembl" after select_by_key, there is no more error. I will try to do what you suggested before, that is to get transcript_ids from exon lines. |
I have made a push to try to fix this one. The changes are big so I made a copy of the function select_transcript that I called select_transcript2. Can you test your example in python after replacing the calls to select_transcript by the the new function ? |
I will try this asap. |
I end up with 61 failures out of 414 tests. The select_transcript2 function also seem to also have problems:
I have created a branch in gtftk containing your last libgtftk version (feature_libgtftk_bc072e5). Maybe it will help to debug. |
I can't see the select_transcript2 function in the new branch. Is it normal ? |
did you try a: git pull |
I did even more : And still no select_transcript2 function. |
This one seems to be fixed by the last push. |
To be tested. |
It seems that short_long does not support files without gene lines. I think the gene_ids should could taken from the transcript lines or even the exon line since it is very common to have no gene line. What do you think about this ?
The text was updated successfully, but these errors were encountered: