Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Log basic_RVs sampled in sample_*_predictive functions #6142

Merged
merged 2 commits into from
Sep 22, 2022

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
Contributor

@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

pymc/tests/test_sampling.py Outdated Show resolved Hide resolved
@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