Fix inconsistencies between --hgvs and --hgvsg #1750
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
ENSVAR-3174
Changelog
N
s as reference sequence. The correct behaviour would be to mimic what is done with HGVS and error out.--hgvs
, the message is printed three times:As
--hgvsc
and--hgvsp
are internal parameters (not VEP arguments), they should be omitted from the message.Testing
vep --hgvsg --offline --cache $vep_cache --id "1 230710048 rs699 A G" --force
should raise an error:vep --hgvsg --cache $vep_cache --id "1 230710048 rs699 A G" --force
should warn that the database will be accessed to retrieve the sequence:vep --hgvs --cache $vep_cache --id "1 230710048 rs699 A G" --force
should only warn that the database is accessed to retrieve the sequence for--hgvs
.