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 193 ro injections #247

Merged
merged 4 commits into from
Jul 12, 2024
Merged

Feat 193 ro injections #247

merged 4 commits into from
Jul 12, 2024

Conversation

mekhlakapoor
Copy link
Contributor

@mekhlakapoor mekhlakapoor commented Jul 9, 2024

closes #193

  • creates container classes for RO injection info and Viral Info
  • If RO injection is requested, maps to RetroOrbitalInjection model
  • Uses prep lot id to extract and integrate viral info from TARS
  • Adds unit test examples
  • LAS has 3 options for injection eye: "left", "right", or either...the "either" option is being mapped to None

Notes:

  • The LAS sharepoint list contains a good amount of viral material (but it's not standardized) so for now, TARS data overwrites injectable material info. In the future, we could refactor to map LAS's viral info instead of defaulting to overwrite (will be useful in the case that TARS response is empty).

@mekhlakapoor mekhlakapoor requested review from abhandiwad, dyf, saskiad and jtyoung84 and removed request for abhandiwad July 9, 2024 23:11
@jtyoung84 jtyoung84 merged commit 61fdd66 into main Jul 12, 2024
2 checks passed
@jtyoung84 jtyoung84 deleted the feat-193-RO-injections branch July 12, 2024 22:35
mekhlakapoor added a commit that referenced this pull request Oct 25, 2024
* RO inj + injection materials

* fix tars database error
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Extract TARS identifiers from LabTracks RO injections and join with TARS metadata
2 participants