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

Version 6.0 - merge from Develop #164

Merged
merged 166 commits into from
Oct 22, 2018
Merged

Version 6.0 - merge from Develop #164

merged 166 commits into from
Oct 22, 2018

Conversation

kbjarkefur
Copy link
Contributor

@kbjarkefur kbjarkefur commented Oct 22, 2018

PR to merge the updates for ietoolkit version 6.0

Part of checklist in #163

kbjarkefur and others added 30 commits May 3, 2018 17:29
Show how we want to first store all stats in a result matrix, and then pass that matrix to commands that create outputt according to the options used by user
- The first difference are here calcualted properly
- The N are included
- se also for mean instead of sd
- All stats are restricted to the same sample
This is rare in SCTO data as at least KEY or submission date is different, but in case it is not then this line creates a bug. Force is only used when dropping based on a subset of the variables, but this is only sued when all vars are duplicates
This segment list the commands needed for this project, and installs them if not installed. there was a `cap` on the row that installed the command not already installed. It is better if there is an error thrown if the command could not be installed, or if there was a type in the command name listed as required.

I discussed this with @eatorange
We do not want to force this as users might need an older version in another project
Variables not passed to the sub-command
bbdaniels and others added 25 commits October 19, 2018 13:13
So that the values in the matrix can be used to error solving
Added people in the test session that weren't there yet.
@kbjarkefur kbjarkefur changed the title Develop Version 6.0 - merge from Develop Oct 22, 2018
@kbjarkefur kbjarkefur merged commit e4ee5f4 into master Oct 22, 2018
@kbjarkefur kbjarkefur deleted the develop branch October 22, 2018 18:04
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.

4 participants