Skip to content

Conversation

@devmotion
Copy link
Member

This PR fixes the test error revealed by #91 (hopefully): The MCMCChains error seems to be a downstream problem caused by Plots or UnicodePlots but AbstractMCMC's tests failed as well (https://github.com/TuringLang/AbstractMCMC.jl/runs/4870654544?check_suite_focus=true).

@codecov
Copy link

codecov bot commented Jan 20, 2022

Codecov Report

Merging #93 (ede9854) into master (56431bd) will increase coverage by 0.11%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #93      +/-   ##
==========================================
+ Coverage   97.57%   97.68%   +0.11%     
==========================================
  Files           7        7              
  Lines         206      216      +10     
==========================================
+ Hits          201      211      +10     
  Misses          5        5              
Impacted Files Coverage Δ
src/stepper.jl 100.00% <0.00%> (ø)
src/transducer.jl 100.00% <0.00%> (ø)
src/sample.jl 97.97% <0.00%> (+0.04%) ⬆️
src/logging.jl 94.44% <0.00%> (+0.32%) ⬆️

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 56431bd...ede9854. Read the comment docs.

@devmotion devmotion merged commit fe972e8 into master Jan 20, 2022
@delete-merged-branch delete-merged-branch bot deleted the dw/fix_ci branch January 20, 2022 23: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.

3 participants