Closes #107, #132, #134 pharmaverseadam updates for pk/pd

[zdz2101]

No description provided.

[github-actions]

Unit Tests Summary

  1 files  111 suites   3m 6s ⏱️
243 tests 213 ✅  30 💤 0 ❌
476 runs  235 ✅ 241 💤 0 ❌

Results for commit b4f2300.

♻️ This comment has been updated with latest results.

[github-actions]

Unit Test Performance Difference

Test Suite	$Status$	Time on main	$±Time$	$±Tests$	$±Skipped$	$±Failures$	$±Errors$
table_ent	💔	$14.04$	$+1.01$	$0$	$0$	$0$	$0$
table_lbt14	💔	$28.62$	$+1.90$	$0$	$0$	$0$	$0$

Additional test case details

Test Suite	$Status$	Time on main	$±Time$	Test Case
table_pkpt04	💀	$0.24$	$-0.24$	PKPT04_is_produced_correctly_for_Drug_Y
table_pkpt05	💀	$0.35$	$-0.35$	PKPT05_Drug_Y_is_produced_correctly

Results for commit 154550d

♻️ This comment has been updated with latest results.

[edelarua]

Looks good!! I'm not familiar with the specs for each table (i.e. if it's ok to drop the columns you did) but I'm sure you know better than I do. Just a few comments/suggestions from me :)

[zdz2101]

Looks good!! I'm not familiar with the specs for each table (i.e. if it's ok to drop the columns you did) but I'm sure you know better than I do. Just a few comments/suggestions from me :)

I didn't really drop columns as much as it was the dataset isn't as "rich" as the previous, we only have 1 analyte for urine where the doses are just different in the pharamverseadam data so there's no "Drug Y" equivalent to look at

@shajoezhu do you have thoughts how this has been handled?

[edelarua]

Lgtm! Thanks Zelos

[shajoezhu]

lgtm! Thanks @zdz2101 dont worry for drug y for now