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

Error handling EPA-NG jplace output file #20

Closed
erikrikarddaniel opened this issue Sep 23, 2022 · 3 comments
Closed

Error handling EPA-NG jplace output file #20

erikrikarddaniel opened this issue Sep 23, 2022 · 3 comments

Comments

@erikrikarddaniel
Copy link

Hi,

Sorry to bother you again. I get the following exception when running gappa examine graft --jplace-path NrdDmprooted.epa_result.broken.jplace.gz (input file attached):

Found 1 jplace file
terminate called after throwing an instance of 'std::runtime_error'
  what():  Invalid characters at 1:8543: '0.8590006000'.
Aborted (core dumped)

The input file was produced with EPA-NG, and contains labels at internal nodes. My first thought was the the "-" characters in the node labels was the problem, but removing them didn't help. When I replaced the input tree to one without the internal node names, I got a jplace file that works. The node names might not be the only difference though.

(One could imagine that two tools from the same group would use the same library to parse the files... ;-) )

NrdDmprooted.epa_result.broken.jplace.gz

/D

@erikrikarddaniel erikrikarddaniel changed the title Error handling EPA-NG tree Error handling EPA-NG jplace output file Sep 23, 2022
@lczech
Copy link
Owner

lczech commented Sep 23, 2022

Hey Daniel,

interesting, I cannot reproduce this error. I've tracked down the error message, and it should not occur with the file that you sent. It's a valid file, and I can parse it well, and gappa examine graft works for me with it... Sorry for that, "works for me" is not very helpful... Did you use the latest release of gappa? Which OS are you on?

One could imagine that two tools from the same group would use the same library to parse the files... ;-)

Haha yes of course, we want to achieve that, and so far, it has worked :-)

Cheers and so long
Lucas

@erikrikarddaniel
Copy link
Author

erikrikarddaniel commented Sep 24, 2022

Sorry to forget the version. I'm running the 0.8.0 Bioconda package on Ubuntu plus the biocontainer Docker image; same result with both. This seems to be the latest in Bioconda, but you're at 0.8.2 I see.

@lczech
Copy link
Owner

lczech commented Sep 25, 2022

Ah right, thanks, I was able to reproduce the issue with gappa v0.8.0, and traced back the change - it was indeed fixed in gappa v0.8.1. Unfortunately, for some reason, conda has stopped updating automatically, I will have to look into this. Hence, for now, please manually compile a more recent version of gappa to solve the issue.

@lczech lczech closed this as completed Sep 25, 2022
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

No branches or pull requests

2 participants