VariantEffect / mavedb-api

MaveDB API
GNU Affero General Public License v3.0
8 stars 2 forks source link

Expand Target Label Allowed Characters #194

Closed bencap closed 5 months ago

bencap commented 5 months ago

Fixes #190

When using target labels on sequences to validate score and count data, we split variants on the same line by spaces and we split the fully qualified variant on ':' to infer the label and hgvs string. See: https://github.com/VariantEffect/mavedb-api/blob/fa1ce7d0bfd57f0e178719f462273b764a467212/src/mavedb/lib/validation/dataframe.py#L421-L424

Spaces and colons are the only characters that need to be reserved from our perspective. We can raise a validation error if a user tries to include a colon in their target label and sanitize the label of any whitespace during validation.

Adds view model tests for target sequences.