Skip to content

Add alpha argument to mcmc_hist and mcmc_dens - #244

Merged
jgabry merged 5 commits into
masterfrom
mcmc-hist-dens-alpha
Oct 5, 2021
Merged

jgabry merged 5 commits into
masterfrom
mcmc-hist-dens-alpha

Conversation

@jgabry

@jgabry jgabry commented Oct 23, 2020

Copy link
Copy Markdown
Member

As requested in #237, this PR adds argument alpha to mcmc_hist() and mcmc_dens() so that users can control the fill transparency. Also added visual tests.

@jgabry
jgabry requested a review from tjmahr October 23, 2020 19:29
@codecov-io

codecov-io commented Oct 23, 2020

Copy link
Copy Markdown

Codecov Report

Merging #244 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #244   +/-   ##
=======================================
  Coverage   98.34%   98.34%           
=======================================
  Files          32       32           
  Lines        4112     4116    +4     
=======================================
+ Hits         4044     4048    +4     
  Misses         68       68           
Impacted Files Coverage Δ
R/mcmc-distributions.R 99.44% <100.00%> (+0.01%) ⬆️

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 8786298...255f31d. Read the comment docs.

@jgabry

jgabry commented Oct 5, 2021

Copy link
Copy Markdown
Member Author

Merging this as part of my effort to clear out small old PRs that have been piling up.

@jgabry
jgabry merged commit e36f040 into master Oct 5, 2021
@jgabry
jgabry deleted the mcmc-hist-dens-alpha branch October 5, 2021 19:43
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