search

VEuPathDB / service-multi-blast

Async BLAST+ query service and supporting libraries.
Apache License 2.0
0 stars 0 forks source link

query sequence should allow spaces and numbers as in https://www.ncbi.nlm.nih.gov/protein/1790350 #84

Closed aurreco-uga closed 2 years ago

aurreco-uga commented 3 years ago

the query sequence should allow spaces…and should be stripped out like a newline or an empty line

Foxcapades commented 2 years ago

Relates to #91 and #104

aurreco-uga commented 2 years ago

i skipped validating spaces https://github.com/VEuPathDB/service-multi-blast/commit/137692d28b2b2aa5d4d4bfacd1f7ebe3a670544d and it works, the query in teh result sshows without spaces though.. possibly part of the hash generation which was removing empty lines..

the commit triggered a build: https://ci.apidb.org/view/containers/job/multi-blast/job/master/

will patch qa and close this issue.

aurreco-uga commented 2 years ago

we need to enable also numbers as in https://www.ncbi.nlm.nih.gov/protein/1790350

Foxcapades commented 2 years ago

This should already be fixed?