Adding Wilcoxon Rank-Sum Tests

[ddsjoberg]

What changes are proposed in this pull request?

• Added ard_wilcoxtest() and ard_paired_wilcoxtest()

Related #42
Related #39

---

Reviewer Checklist (if item does not apply, mark is as complete)

• Ensure all package dependencies are installed: devtools::install_dev_deps()
• PR branch has pulled the most recent updates from master branch: usethis::pr_merge_main()
• If a bug was fixed, a unit test was added.
• Run pkgdown::build_site(). Check the R console for errors, and review the rendered website.
• Code coverage is suitable for any new functions/features: devtools::test_coverage()
• usethis::use_spell_check() runs with no spelling errors in documentation

When the branch is ready to be merged:

• Update NEWS.md with the changes from this pull request under the heading "# cards (development version)". If there is an issue associated with the pull request, reference it in parentheses at the end update (see NEWS.md for examples).
• Increment the version number using usethis::use_version(which = "dev")
• Run usethis::use_spell_check() again
• Approve Pull Request
• Merge the PR. Please use "Squash and merge".

[github-actions]

Unit Tests Summary

 1 files   4 suites   1s ⏱️
 4 tests  4 ✅ 0 💤 0 ❌
12 runs  10 ✅ 2 💤 0 ❌

Results for commit 0c4da80.

♻️ This comment has been updated with latest results.

[github-actions]

Code Coverage Summary

Filename              Stmts    Miss  Cover    Missing
------------------  -------  ------  -------  ---------
R/ard_ttest.R            78       0  100.00%
R/ard_wilcoxtest.R       83       0  100.00%
TOTAL                   161       0  100.00%

Diff against main

Filename              Stmts    Miss  Cover
------------------  -------  ------  --------
R/ard_ttest.R           -10       0  +100.00%
R/ard_wilcoxtest.R      +83       0  +100.00%
TOTAL                   +73       0  +100.00%

Results for commit: 0c4da80

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

[github-actions]

Unit Test Performance Difference

Test Suite	$Status$	Time on main	$±Time$	$±Tests$	$±Skipped$	$±Failures$	$±Errors$
ard_wilcoxtest	👶		$+0.00$	$+3$	$0$	$0$	$0$

Additional test case details

Test Suite	$Status$	Time on main	$±Time$	Test Case
ard_wilcoxtest	👶		$+0.00$	ard_paired_wilcoxtest_works
ard_wilcoxtest	👶		$+1.02$	ard_wilcoxtest_works

Results for commit fc63f9c

♻️ This comment has been updated with latest results.

[Melkiades]

This does not allow wrapping in docs, maybe it should be on the same line somehow?

[ddsjoberg]

When the help files are rendered, it doesn't break the line mid-code. But if you make the window wide enough it does all appear on one line.

Am I addressing your comment?

[Melkiades]

just wondering if it was intentional that the first breaks mid-code while the second does not

[Melkiades]

Only minor questions from my side. Generally, it is good to go! Thanks Daniel ^^

ps: I really appreciate testing the output directly with the stats:: function. I was suggesting doing something similar some time ago to start walking the long road of addressing double programming and QCing. Going beyond this would be testing stats itself...

[Melkiades]

Just found this. It should be wilcox test @ddsjoberg

[ddsjoberg]

oh goodness, thank you!

[Melkiades]

missing stats::

[ddsjoberg]

I think it's ok to keep it like this, since it's essentially base R