-
Notifications
You must be signed in to change notification settings - Fork 16
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
Add new merge_clin()
function for export
#96
Add new merge_clin()
function for export
#96
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #96 +/- ##
==========================================
+ Coverage 75.30% 75.51% +0.21%
==========================================
Files 41 42 +1
Lines 1478 1491 +13
==========================================
+ Hits 1113 1126 +13
Misses 365 365 ☔ View full report in Codecov by Sentry. |
958e58c
to
efad5e7
Compare
51ae661
to
458ac0f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! Just a few minor grammatical changes suggested.
458ac0f
to
4db0496
Compare
- new function wrapper to allow users to merge in clinical variables to `soma_adat` objects easily - closes SomaLogic#80
4db0496
to
3100b70
Compare
- vignettes triggered a warning about the missing favicon in the articles footer - the path spec was a level off - this fixes the issue; no more warnings and the logo is on all articles
Summary
soma_adat
objects easilymerge_clin.R
functionality from Rscript to an exported function #80