- 
                Notifications
    You must be signed in to change notification settings 
- Fork 2.1k
Fix MCMC non-deterministic seeding with Generators #7637
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
Conversation
deca034    to
    7ba67b6      
    Compare
  
    7ba67b6    to
    d3fb9a7      
    Compare
  
    There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The failing test from the mac env actually happens because your second commit doesn't have all of the changes to pymc.step_methods.hmc.quadpotential that I had over in my PR
| 
 So there was a broken intermediate commit? | 
| I guess this is missing: 2913e4a | 
| It's strange though, the test passes locally for me | 
| Ah must be the other mp_ctx that Mac defaults to. Should we be worried though that is not being passed? | 
e008367    to
    475c04a      
    Compare
  
    | 
 Yes, it's that commit. 
 It's not super important at the moment I think. The parallel sampling should still work because sampling state is not passed around there, but if you cherry-pick that other commit, then things should pass. | 
475c04a    to
    611220f      
    Compare
  
    | It was the sequential sampling (on the CI we only have 1 core). I had to reorder so we handle the default values before we do the fix on sequential sampling | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
611220f    to
    60d6845      
    Compare
  
    Co-authored-by: ricardoV94 <[email protected]>
60d6845    to
    9dcd50a      
    Compare
  
    | Codecov ReportAll modified and coverable lines are covered by tests ✅ 
 Additional details and impacted files@@            Coverage Diff             @@
##             main    #7637      +/-   ##
==========================================
+ Coverage   92.83%   92.85%   +0.01%     
==========================================
  Files         106      106              
  Lines       17748    17771      +23     
==========================================
+ Hits        16477    16501      +24     
+ Misses       1271     1270       -1     
 🚀 New features to boost your workflow:
 | 
Closes #7612
Picked up the fix from #7540
📚 Documentation preview 📚: https://pymc--7637.org.readthedocs.build/en/7637/