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

feat!: use vrs 2.0-alpha models in /normalize #140

Merged
merged 9 commits into from
Sep 27, 2023
Merged

feat!: use vrs 2.0-alpha models in /normalize #140

merged 9 commits into from
Sep 27, 2023

Conversation

korikuzma
Copy link
Member

@korikuzma korikuzma commented Sep 12, 2023

Close #135 . Going to wait to open until I confirm everything works in metakb. But would still like your review @jsstevenson now

Notes:

  • Update to pydantic v2
  • Remove ga4gh.vrsatile.pydantic depdency and use ga4gh.vrs for core/vrs 2.0-alpha models
  • Set response_model_exclude_none=True in FastAPI response

@korikuzma korikuzma added enhancement New feature or request priority:high High priority labels Sep 12, 2023
@korikuzma korikuzma self-assigned this Sep 12, 2023
disease/query.py Outdated Show resolved Hide resolved
@korikuzma korikuzma marked this pull request as ready for review September 27, 2023 13:49
@korikuzma korikuzma merged commit 37b56dd into staging Sep 27, 2023
24 checks passed
@korikuzma korikuzma deleted the issue-135 branch September 27, 2023 17:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority:high High priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants