Skip to content

Align advertised Metropolis stats_dtypes with changes from 1e7d91f #5882

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

Merged

Conversation

michaelosthege
Copy link
Member

In pymc-devs/mcbackend#27 I tried to upgrade to 4.0.0 but the CI failed.

The reason for the failure is an incorrect stats_dtypes["accepted"] which was not updated with the changes in 1e7d91f.


  • No breaks
  • right before it's ready to merge, mention the PR in the RELEASE-NOTES.md

@michaelosthege michaelosthege added this to the v4.0.1 milestone Jun 12, 2022
@michaelosthege michaelosthege self-assigned this Jun 12, 2022
@codecov
Copy link

codecov bot commented Jun 12, 2022

Codecov Report

Merging #5882 (456a463) into main (575454f) will decrease coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #5882      +/-   ##
==========================================
- Coverage   89.48%   89.48%   -0.01%     
==========================================
  Files          73       73              
  Lines       13225    13225              
==========================================
- Hits        11835    11834       -1     
- Misses       1390     1391       +1     
Impacted Files Coverage Δ
pymc/step_methods/metropolis.py 83.36% <ø> (ø)
pymc/step_methods/hmc/base_hmc.py 89.68% <0.00%> (-0.80%) ⬇️

@michaelosthege michaelosthege marked this pull request as ready for review June 12, 2022 12:10
@ricardoV94
Copy link
Member

Should we be testing this somewhere here?

@michaelosthege
Copy link
Member Author

Should we be testing this somewhere here?

We should refactor the whole sampler stats API and test should be part of that..

@michaelosthege michaelosthege merged commit 78b1f17 into pymc-devs:main Jun 12, 2022
@michaelosthege michaelosthege deleted the fix-metropolis-stats-dtypes branch June 12, 2022 12:46
@ricardoV94
Copy link
Member

Should we be testing this somewhere here?

We should refactor the whole sampler stats API and test should be part of that..

Can you open an issue for remembering to test this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants