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

extract_dollop_output #5

Open
kavithamallayaa opened this issue Oct 2, 2019 · 1 comment
Open

extract_dollop_output #5

kavithamallayaa opened this issue Oct 2, 2019 · 1 comment
Assignees
Labels

Comments

@kavithamallayaa
Copy link

Hi,

I was trying to use extract_dollop_output_sequences_v2-fast.pl on outfile from dollop. But my output files "*newstyle_report.txt" has same node number getting repeated several times.
When I ran the script, the following message was shown
"Argument "Cpe29" isn't numeric in addition (+) at extract_dollop_output_sequences_v2-fast.pl line 375, <$
[...]> line 248" for many lines.
Can please me help me with that.
[My data consist of 124 species and 87 states]
with regards
Kavitha

@guyleonard guyleonard self-assigned this Oct 2, 2019
guyleonard added a commit that referenced this issue Oct 2, 2019
hopefully this fixes a problem from issue #5 where leaf nodes with numbers in them are matched instead of only internal nodes that are solely numbers...
@guyleonard
Copy link
Owner

Hi Kavitha

I think this is caused by a lazy error on my part, the line above matches taxon names if they have a digit [0-9] in them - when it looks for internal node numbers, in retrospect this is a bad idea as sometimes people ( as in your case ) have digits in their taxon names. I have changed the regex match to specifically look for node numbers, so hopefully it should now work with your use case.

Let me know,
cheers Guy

@guyleonard guyleonard added the bug label Oct 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants