Refactor variants.get_snv into two or more functions - Githubissues

statgen / bravo_api

Server side data processing and retrieval endpoints for BRAVO

MIT License

1 stars 2 forks source link

Refactor variants.get_snv into two or more functions #11

Open grosscol opened 2 years ago

grosscol commented 2 years ago

Issue or current state

In practice, lookup of snv is by accession identifier (rsid) xor by position (chrom-pos)

variants.get_snv(args['variant_id'], None, None, args['full']):
variants.get_snv(None, args['chrom'], args['pos'], args['full']):
variants.get_snv(query, None, None, False):

Resolved when

Uses of get_snv are refactored into get_accession_snv and get_position_snv.
Tests illustrate expected use.
Return type and data format is documented.