Skip to content

Conversation

@arcaldwell49
Copy link
Owner

Adding ANCOVA_analytic, ANCOVA_contrast, and power_oneway_ancova. Also, label_list argument added to ANOVA_design

First round of tests working
Working on factorial ANCOVA solution. Idea is there just need to fix the implementation
Multi-factor ANCOVA started.
Working for factorial ANOVAs. Just need to get contrasts and contrast functions working. Then I can focus on the vignettes, documentation, etc.
aov_list and con_list now provide power.htest objects in the list. Hmisc dependency also removed.
Need to get tests and vignette finished.
Almost ready for pull request. Passing all tests with 90%+ coverage.
Green Lantern is good
Fixing spelling errors
@codecov
Copy link

codecov bot commented Dec 9, 2021

Codecov Report

Merging #31 (b485b94) into master (5624ae3) will decrease coverage by 2.29%.
The diff coverage is 88.38%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #31      +/-   ##
==========================================
- Coverage   96.63%   94.34%   -2.30%     
==========================================
  Files          24       30       +6     
  Lines        2321     3111     +790     
==========================================
+ Hits         2243     2935     +692     
- Misses         78      176      +98     
Impacted Files Coverage Δ
R/alpha_standardized.R 100.00% <ø> (ø)
R/methods.ancova_power.R 0.00% <0.00%> (ø)
R/misc_functions.R 59.09% <0.00%> (-40.91%) ⬇️
R/p_standardized.R 100.00% <ø> (ø)
R/ANOVA_design.R 97.67% <20.00%> (-1.73%) ⬇️
R/oneway_ancova.R 64.13% <64.13%> (ø)
R/ANCOVA_contrast.R 93.38% <93.38%> (ø)
R/ANCOVA_analytic.R 99.22% <99.22%> (ø)
R/ANCOVA_factorial_power.R 100.00% <100.00%> (ø)
R/ANOVA_exact.R 98.46% <100.00%> (ø)
... and 3 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5624ae3...b485b94. Read the comment docs.

@arcaldwell49
Copy link
Owner Author

All checks running. Need to add some more tests before pushing to CRAN.

@arcaldwell49 arcaldwell49 merged commit 62e0b5c into master Dec 9, 2021
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