Skip to content

Conversation

tomicapretto
Copy link
Contributor

@tomicapretto tomicapretto commented Apr 28, 2024

This PR adds a test that complements the bug fix in #7284

It finally closes #7258


📚 Documentation preview 📚: https://pymc--7287.org.readthedocs.build/en/7287/

@ricardoV94
Copy link
Member

You can remove the previous test (the one above), since this covers everything that is tested there. You can reuse the old name that's also shorter

@ricardoV94 ricardoV94 added tests no releasenotes Skipped in automatic release notes generation labels Apr 28, 2024
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.33%. Comparing base (a74c03f) to head (538c3a5).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #7287   +/-   ##
=======================================
  Coverage   92.33%   92.33%           
=======================================
  Files         102      102           
  Lines       17018    17018           
=======================================
  Hits        15713    15713           
  Misses       1305     1305           

@ricardoV94 ricardoV94 merged commit 29fd732 into pymc-devs:main Apr 28, 2024
@tomicapretto tomicapretto deleted the add_test_var_names branch April 28, 2024 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no releasenotes Skipped in automatic release notes generation tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unexpected behavior with pm.sample(var_names=...)

3 participants