jgabry commented 3 years ago

Closes #120

Doesn't assume draw column is present when detecting number of columns to decide which plot to make
Removes draw column from fake data used in test (this was preventing the test from failing when it should have failed)

codecov-commenter commented 3 years ago

Codecov Report

Merging #122 (700a8cc) into master (8e5cf79) will not change coverage. The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #122   +/-   ##
=======================================
  Coverage   84.67%   84.67%           
=======================================
  Files           5        5           
  Lines         522      522           
=======================================
  Hits          442      442           
  Misses         80       80

Impacted Files	Coverage Δ
R/SurveyFit.R	`90.00% <100.00%> (ø)`

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 8e5cf79...700a8cc. Read the comment docs.

lauken13 commented 3 years ago

Super speedy! Thanks @jgabry! :)

lauken13 / mrpkit

Fix violin plot #122

Codecov Report