search

IEDB / TCRMatch

Other
26 stars 12 forks source link

output is a 3 column file #40

Closed crisjs closed 1 year ago

crisjs commented 1 year ago

Hi, When I run the code to generate the initial scores, I get a file with only the first three columns mentioned in the readme. Shouldn't I get a file with seven columns? Is there any additional step I should do? I'm running TCRMatch in unbuntu. Many thanks, Cristina

acrinklaw commented 1 year ago

Hi Cristina, Are you passing in your own database with -d? Currently the additional annotations for matches is only supported if you are using the default IEDB database. Please post the command you ran, if possible

crisjs commented 1 year ago

Hi, Thank you for your answer. I'm using the default IEDB database and the command: ./tcrmatch -i myinputfile -t num_threads [-s threshold] [-d /path/to/database] [-a] > output_file TCRMatch version was 0.1.1. I tried also with version 1.0.3 but this one didn't work at all.

acrinklaw commented 1 year ago

For version 1.0.3 what exactly happens when you run it? Here is my output running the test sequences: image

I would suggest just running it as ./tcrmatch -i input.txt -t num_threads > output_file just to verify that it is working correctly. The other parameters are optional

crisjs commented 1 year ago

When I run version 1.0.3, it doesn't do anything, it just says: _terminate called after throwing an instance of 'std::invalidargument' what(): stoi Aborted

With version 0.1.1, I tried the command you suggested (./tcrmatch -i input.txt -t num_threads > output_file), and again, the output file only contains the first three columns. Can you think of anything else I can try? Thanks

acrinklaw commented 1 year ago

I believe version 0.1.1 does not support the full output. Can you please copy and paste the exact command you ran? stoi() is only called when parsing the number of threads. An example command would be: tcrmatch -i input.txt -t 15 > output.txt. It sounds like you are passing a string instead of an integer for number of threads? In this case I am specifying 15 as my CPU has 16 cores available. I apologize if the documentation is unclear about this, I will update to make it more apparent

crisjs commented 1 year ago

It finally worked! You were right, I wasn't specifying a number for -t. Thanks so much for your help!