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

Re-running @dickgroenenberg's files #22

Closed
wants to merge 1 commit into from
Closed

Conversation

rvosa
Copy link
Member

@rvosa rvosa commented Aug 29, 2024

Some records had a 'full' sequence length of zero. The argument check previously threw an exception when length was <=0.

@rvosa
Copy link
Member Author

rvosa commented Aug 29, 2024

🤖 - Hi! This is an automated message from the barcode validation robot. I'm going to validate the FASTA files in your request. Please wait while I process the files. This takes about two minutes per sequence.

@rvosa
Copy link
Member Author

rvosa commented Aug 29, 2024

🤖 - Error processing PR 22: Command 'blastn' returned non-zero exit status 3.
Stack trace:
Traceback (most recent call last):
File "/data/rutger.vos/barcode_validator/barcode_validator/daemon.py", line 263, in process_pr
results = run_validation(config, pr_number, branch, validator)
File "/data/rutger.vos/barcode_validator/barcode_validator/daemon.py", line 110, in run_validation
results = validator.validate_fasta(file['filename'])
File "/data/rutger.vos/barcode_validator/barcode_validator/core.py", line 40, in validate_fasta
result = self.validate_record(process_id, record)
File "/data/rutger.vos/barcode_validator/barcode_validator/core.py", line 71, in validate_record
result.obs_taxon = run_localblast(record, self.ncbi_tree, self.bold_tree, self.config)
File "/data/rutger.vos/barcode_validator/barcode_validator/taxonomy.py", line 153, in run_localblast
raise subprocess.CalledProcessError(return_code, 'blastn')
subprocess.CalledProcessError: Command 'blastn' returned non-zero exit status 3.

@rvosa rvosa closed this Aug 29, 2024
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

Successfully merging this pull request may close these issues.

1 participant