Skip to content

Conversation

@ricardoV94
Copy link
Member

@ricardoV94 ricardoV94 commented Sep 21, 2022

Closes #5973

Major / Breaking Changes

  • ...

Bugfixes / New features

  • ...

Docs / Maintenance

  • Log name of basic_RVs sampled in prior and posterior predictive

@ricardoV94 ricardoV94 force-pushed the log_resampled_varnames branch from 3e00114 to 1b83522 Compare September 21, 2022 10:24
@ricardoV94 ricardoV94 changed the title Log sampled basic_RVs sample_*_predictive functions Log basic_RVs sampled in sample_*_predictive functions Sep 21, 2022
@codecov
Copy link

codecov bot commented Sep 21, 2022

Codecov Report

Merging #6142 (6479ad4) into main (4c92adf) will increase coverage by 1.30%.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #6142      +/-   ##
==========================================
+ Coverage   91.28%   92.58%   +1.30%     
==========================================
  Files         101      101              
  Lines       20945    20948       +3     
==========================================
+ Hits        19119    19395     +276     
+ Misses       1826     1553     -273     
Impacted Files Coverage Δ
pymc/sampling.py 82.19% <100.00%> (+12.18%) ⬆️
pymc/smc/smc.py 74.80% <0.00%> (-22.45%) ⬇️
pymc/smc/sample_smc.py 77.85% <0.00%> (-5.37%) ⬇️
pymc/backends/base.py 86.44% <0.00%> (+0.73%) ⬆️
pymc/step_methods/slicer.py 96.20% <0.00%> (+1.26%) ⬆️
pymc/backends/report.py 91.04% <0.00%> (+4.47%) ⬆️
pymc/step_methods/arraystep.py 94.32% <0.00%> (+7.80%) ⬆️
pymc/step_methods/metropolis.py 82.98% <0.00%> (+43.15%) ⬆️

@ricardoV94 ricardoV94 force-pushed the log_resampled_varnames branch from 1b83522 to feb72ae Compare September 21, 2022 10:51
@ricardoV94
Copy link
Member Author

Only flaky test failing: #6091

Copy link
Member

@lucianopaz lucianopaz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great @ricardoV94 ! There was only a typo in a comment, and I’m not sure which word you wanted to put

@ricardoV94 ricardoV94 force-pushed the log_resampled_varnames branch from feb72ae to 6479ad4 Compare September 22, 2022 10:50
@ricardoV94
Copy link
Member Author

Merging, flaky test is documented in #6143

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Log name of RVs that are being resampled in posterior predictive

2 participants