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

Develop #409

Merged
merged 8 commits into from
Aug 16, 2023
Merged

Develop #409

merged 8 commits into from
Aug 16, 2023

Conversation

jreps
Copy link
Collaborator

@jreps jreps commented Aug 16, 2023

No description provided.

jreps and others added 8 commits June 22, 2023 20:59
- supporting new ShinyAppBuilder version (but keeping backwards compatible for older version)
- updating reticulate minimum package version
- adding spline feature engineering
- added function to impute based on age/sex
Change execution_date_time to varchar(100)
- added feature engineering functions
@codecov
Copy link

codecov bot commented Aug 16, 2023

Codecov Report

Merging #409 (5261519) into main (76cbe81) will decrease coverage by 0.12%.
Report is 5 commits behind head on main.
The diff coverage is 80.42%.

❗ Current head 5261519 differs from pull request most recent head 39aabd1. Consider uploading reports for the commit 39aabd1 to get more accurate results

@@            Coverage Diff             @@
##             main     #409      +/-   ##
==========================================
- Coverage   89.50%   89.39%   -0.12%     
==========================================
  Files          49       49              
  Lines        9653     9824     +171     
==========================================
+ Hits         8640     8782     +142     
- Misses       1013     1042      +29     
Files Changed Coverage Δ
R/ViewShinyPlp.R 0.00% <0.00%> (ø)
R/FeatureEngineering.R 94.69% <91.27%> (-5.31%) ⬇️
R/CovariateSummary.R 95.97% <100.00%> (+0.25%) ⬆️
R/RunPlp.R 87.28% <100.00%> (+0.22%) ⬆️

... and 2 files with indirect coverage changes

@jreps jreps merged commit f6bda23 into main Aug 16, 2023
9 checks passed
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.

2 participants