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.

Sign up for GitHub

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

Jump to bottom

Add validation rules to PrevalenceByLocationAndTimeHandler #72

Open
flaneuse opened this issue Feb 14, 2023 · 0 comments
Open

Add validation rules to PrevalenceByLocationAndTimeHandler #72

flaneuse opened this issue Feb 14, 2023 · 0 comments
Assignees
@remoteeng00
Labels
guru

Comments

@flaneuse
Copy link
Contributor

Refactor PrevalenceByLocationAndTimeHandler to add validation:

  1. As before, create a copy of the handler to enable easy testing between the new version of the handler and the existing one
  2. Refactor the structure of the parameters to be a BioThings-style kwargs format
  3. Add validation rules for the parameters:
  • location_id: str
  • pangolin_lineage: str
  • mutations: str
  • cumulative: bool
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@remoteeng00 remoteeng00

Labels
guru
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@flaneuse @remoteeng00