Skip to content

Declare ndim_supp on value transforms#8331

Open
ricardoV94 wants to merge 1 commit into
pymc-devs:mainfrom
ricardoV94:add_ndim_supp_to_transforms
Open

Declare ndim_supp on value transforms#8331
ricardoV94 wants to merge 1 commit into
pymc-devs:mainfrom
ricardoV94:add_ndim_supp_to_transforms

Conversation

@ricardoV94

@ricardoV94 ricardoV94 commented Jun 16, 2026

Copy link
Copy Markdown
Member

This is useful for knowing whether a transformed variable can retain the dims of the original natural view. Elemwise transforms (ndim=0) trivially can, and these are the most common cases.

@ricardoV94 ricardoV94 force-pushed the add_ndim_supp_to_transforms branch from 0b0ae9d to 286925f Compare June 16, 2026 11:26
@read-the-docs-community

Copy link
Copy Markdown

Documentation build overview

📚 pymc | 🛠️ Build #33162804 | 📁 Comparing 286925f against latest (9d24260)

  🔍 Preview build  

3 files changed
± glossary.html
± _modules/pymc/distributions/transforms.html
± _modules/pymc/logprob/transforms.html

@codecov

codecov Bot commented Jun 16, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.71%. Comparing base (9d24260) to head (286925f).

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##             main    #8331       +/-   ##
===========================================
+ Coverage   79.80%   91.71%   +11.91%     
===========================================
  Files         125      125               
  Lines       20526    20545       +19     
===========================================
+ Hits        16380    18843     +2463     
+ Misses       4146     1702     -2444     
Files with missing lines Coverage Δ
pymc/distributions/transforms.py 100.00% <100.00%> (ø)
pymc/logprob/transforms.py 95.75% <100.00%> (+0.31%) ⬆️

... and 42 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jessegrabowski

Copy link
Copy Markdown
Member

Looks good, don't understand how it managed to break tests

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.

2 participants