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

Wip features #67

Draft
wants to merge 137 commits into
base: master
Choose a base branch
from
Draft

Wip features #67

wants to merge 137 commits into from

Conversation

rvyuha
Copy link
Contributor

@rvyuha rvyuha commented Aug 18, 2020

Contains Deprecated/unfinished functions

-apply_missing_tagged_na was replaced by a step that performs the same function
-create_variable_details_template General skeleton for variable details creation however a better approach using DDI is in the making
-PopulateVariableDetails This function relies on DDI which has been dropped for the time being
-ProcessDDIVariableDetails a variation of populate however also dropped for time being
-AddDDIToList internal function used for DDI reading
-bll-flow ddi support section was dropped due to lack of ddi
-clean.R was replaced with filter
-create_bll_model_object was dropped due to small use case
-ddi-utils was dropped due to lack of ddi contains helper ddi reading functions
-LogFunctionActivity has been dropped for now due to existance of modules
-print.metaDataLog was tied to LogFunctionActivity
-recipy-utility.R contains helper functions skeletons that would be nice to have
-check_small_cells dropped due to changes in bllflow structure
-util-funcs contains old long table utility long table has been reworked to work with RDF

rvyuha and others added 30 commits April 1, 2019 12:21
[Feature] Added strata handling
Merge remote-tracking branch 'origin/model-example' into create-reference

# Conflicts:
#	.RData
…ence

Merge stacey's changes into create reference
…ot in var details and added missing label for tableone print
@DougManuel DougManuel requested review from DougManuel and removed request for DougManuel August 18, 2020 17:43
@DougManuel
Copy link
Contributor

DougManuel commented Aug 18, 2020

Looks good to clear out this WIP.

Some of this can be moved to xFlow.

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.

3 participants