-
Notifications
You must be signed in to change notification settings - Fork 24
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
Trailing Slash for samples #52
Comments
Hi!
Thanks so much for debugging this.
I will test this with the devel branch and see whether we still have it, it
is a very annoying bug!
Thanks so much for your help!
Cheers
Lorena Pantano, P.h.D
Research Scientist
Biostatistics Department
Harvard T.H. Chan School of Public Health
https://mirtop.github.io
https://lpantano.github.io
I'll be out of office from Dec-17th to Jan9th
…On Tue, Jan 22, 2019 at 5:11 AM Jose Francisco Sanchez-Herrero < ***@***.***> wrote:
Dear mirtop developers,
I have noticed a bug in the last version of mirtop (0.3.17) when dealing
with back slashes.
I run mirtop gff conversion from sRNAbench data.
mirtop gff --sps hsa --hairpin ../sRNAtoolboxDB/libs/hairpin.fa --gtf
../hsa.gff3 --format srnabench -vd -o test1
isomiR_sRNAbenchtoolbox/sample_1/
I noticed that if I leave the backslash for the folder from sRNAbench data
( isomiR_sRNAbenchtoolbox/sample_1/), mirtop gff generates only a
mirtop.gff file with no COLDATA information.
*## mirGFF3. VERSION 0.9*
*## source-ontology: miRBasev22 doi:10.25504/fairsharing.hmgte8*
*## COLDATA:*
But if I removed the backslash, (isomiR_sRNAbenchtoolbox/sample_1) mirtop
gff would generate a mirtop.gff and a sample_1.gff file containing
information in the COLDATA section
*## mirGFF3. VERSION 0.9*
*## source-ontology: miRBasev22 doi:10.25504/fairsharing.hmgte8*
*## COLDATA: sample_1*
I know it is not a big issue, but I took a while to discover why mirtop
stats was failing after mirtop gff step.
Adding a back slash is a very common practice and some users might
experience the same problem. Adding a simple if folder.endswith("/") clause
should sort it out.
Thank you very much in advance,
Great code and program!
Greetings from IGTP
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#52>, or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABi_HDIKDzZ6VF0iC7iBaQqL2vPOXEBvks5vFuPTgaJpZM4aMVfU>
.
|
Hi, this is fixed in the devel version. Thanks! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Dear mirtop developers,
I have noticed a bug in the last version of mirtop (0.3.17) when dealing with back slashes.
I run mirtop gff conversion from sRNAbench data.
mirtop gff --sps hsa --hairpin ../sRNAtoolboxDB/libs/hairpin.fa --gtf ../hsa.gff3 --format srnabench -vd -o test1 isomiR_sRNAbenchtoolbox/sample_1/
I noticed that if I leave the backslash for the folder from sRNAbench data ( isomiR_sRNAbenchtoolbox/sample_1/), mirtop gff generates only a mirtop.gff file with no COLDATA information.
## mirGFF3. VERSION 0.9
## source-ontology: miRBasev22 doi:10.25504/fairsharing.hmgte8
## COLDATA:
But if I removed the backslash, (isomiR_sRNAbenchtoolbox/sample_1) mirtop gff would generate a mirtop.gff and a sample_1.gff file containing information in the COLDATA section
## mirGFF3. VERSION 0.9
## source-ontology: miRBasev22 doi:10.25504/fairsharing.hmgte8
## COLDATA: sample_1
I know it is not a big issue, but I took a while to discover why mirtop stats was failing after mirtop gff step.
Adding a back slash is a very common practice and some users might experience the same problem. Adding a simple if folder.endswith("/") clause should sort it out.
Thank you very much in advance,
Great code and program!
Greetings from IGTP
The text was updated successfully, but these errors were encountered: