Closed schristley closed 2 years ago
Hey @schristley, I apologize for the oversight. I started a new job and it's been quite a while since I've been actively working on TCRMatch and I just got back to it - I didn't mean to forget our VDJ friends :-)
Your PR looks fantastic and I'll go ahead and merge it after I get a chance to make sure nothing looks strange when I run it on my end - thanks for the help.
It looks like with the rewrite that AIRR TSV support was lost. I think it should be easy add back, just need to parse the TSV and extract the
junction_aa
column, or maybecdr3_aa
because tcrmatch does not want the conserved residues. I suppose the only issue there iscdr3_aa
is an optional field in AIRR TSV whilejunction_aa
is required, so may be better to usejunction_aa
and trim.I will try to submit a PR.