Skip to content

Implement batch evaluation of RV shapes in get_plates#8092

Open
elc45 wants to merge 1 commit intopymc-devs:mainfrom
elc45:pr-8033
Open

Implement batch evaluation of RV shapes in get_plates#8092
elc45 wants to merge 1 commit intopymc-devs:mainfrom
elc45:pr-8033

Conversation

@elc45
Copy link
Contributor

@elc45 elc45 commented Feb 5, 2026

Description

Finishes up #8033 so to_graphviz can work with pm.CustomDist etc.

Related Issue

Checklist

Type of change

  • New feature / enhancement
  • Bug fix
  • Documentation
  • Maintenance
  • Other (please specify):

@github-actions github-actions bot added the bug label Feb 5, 2026
@codecov
Copy link

codecov bot commented Feb 5, 2026

Codecov Report

❌ Patch coverage is 94.73684% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 90.90%. Comparing base (11d0f1b) to head (4282a8c).
⚠️ Report is 3 commits behind head on main.

Files with missing lines Patch % Lines
pymc/pytensorf.py 90.90% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #8092      +/-   ##
==========================================
+ Coverage   90.89%   90.90%   +0.01%     
==========================================
  Files         123      123              
  Lines       19501    19535      +34     
==========================================
+ Hits        17726    17759      +33     
- Misses       1775     1776       +1     
Files with missing lines Coverage Δ
pymc/model_graph.py 84.74% <100.00%> (+0.20%) ⬆️
pymc/pytensorf.py 88.13% <90.90%> (+0.01%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@elc45
Copy link
Contributor Author

elc45 commented Feb 5, 2026

@ricardoV94 any idea why this random test is failing?

@ricardoV94
Copy link
Member

Rerunning

@elc45 elc45 marked this pull request as ready for review February 6, 2026 17:34
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.

BUG: to_graphviz breaks when the model has a pm.CustomDist

2 participants