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

blastp: ignoring client and choosing incorrect defaults? #10

Closed
aurreco-uga opened this issue Apr 3, 2022 · 3 comments · Fixed by #12
Closed

blastp: ignoring client and choosing incorrect defaults? #10

aurreco-uga opened this issue Apr 3, 2022 · 3 comments · Fixed by #12
Assignees

Comments

@aurreco-uga
Copy link
Member

aurreco-uga commented Apr 3, 2022

from error emails
B6BFA154DFD985F41240B6CF5E156DE3

when the user changes the matrix to PAM250, we seem to ignore client values and select incorrect defaults for -gapopen":11,"-gapextend":1,
it ends in 500, error is clear to the user

LOGS
A secondary issue is that the logs show 200, all good????? (check with ellie)

VALIDATOR
2022/04/03 14:13:27 Input JSON: {"-outfmt":{"type":15},"-max_target_seqs":100,"-word_size":6,"-gapopen":11,"-gapextend":1,"-matrix":"PAM250"}

@aurreco-uga aurreco-uga changed the title blastp: choosing incorrect defaults blastp: ignoring client and choosing incorrect defaults? Apr 3, 2022
@Foxcapades
Copy link
Member

The config in the DB is:

{
  "tool": "blastp",
  "-outfmt": {
    "type": 15
  },
  "-max_target_seqs": 100,
  "-db": "//db/PlasmoDB/build-57/Pyoeliiyoelii17X/blast/Pyoeliiyoelii17XAnnotatedProteins",
  "-word_size": 6,
  "-gapopen": 11,
  "-gapextend": 1,
  "-matrix": "PAM250"
}

This means either there is a bug in the translation of the input format to the internal format or there is a client issue.

@aurreco-uga
Copy link
Member Author

yes this might be a client issue.. with previously stored param values...
Screen Shot 2022-04-05 at 5 09 09 PM

@aurreco-uga aurreco-uga assigned jtlong3rd and unassigned Foxcapades Apr 6, 2022
@aurreco-uga aurreco-uga transferred this issue from VEuPathDB/service-multi-blast Apr 6, 2022
@aurreco-uga
Copy link
Member Author

(from jamie) This fix both (1) disables the submit button and (2) prevents the form from being submitted by pressing “Enter” while dependent params are loading. (edited)

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 a pull request may close this issue.

3 participants