Do not generate C code for BatchedDot when BLAS flags are missing

[ricardoV94]

Related Issue

• Closes New optimization Matmul -> BatchedDot causing problems in systems not properly linked to BLAS #549

Checklist

• Checked that the pre-commit linting/style checks pass
• Included tests that prove the fix is effective or that the new feature works
• Added necessary documentation (docstrings and/or example notebooks)
• If you are a pro: each commit corresponds to a relevant logical change

Type of change

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

[codecov-commenter]

Codecov Report

Merging #550 (3ac9efc) into main (c38eea0) will increase coverage by 0.00%.
Report is 2 commits behind head on main.
The diff coverage is 94.35%.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #550   +/-   ##
=======================================
  Coverage   80.90%   80.90%           
=======================================
  Files         162      162           
  Lines       46416    46418    +2     
  Branches    11354    11355    +1     
=======================================
+ Hits        37554    37556    +2     
  Misses       6639     6639           
  Partials     2223     2223           

Files	Coverage Δ
pytensor/breakpoint.py	55.00% <ø> (ø)
pytensor/compile/builders.py	77.17% <100.00%> (ø)
pytensor/graph/basic.py	89.08% <ø> (ø)
pytensor/graph/rewriting/kanren.py	100.00% <ø> (ø)
pytensor/ifelse.py	51.70% <100.00%> (ø)
pytensor/link/jax/dispatch/random.py	96.05% <100.00%> (ø)
pytensor/link/jax/dispatch/shape.py	87.09% <ø> (ø)
pytensor/link/jax/dispatch/subtensor.py	88.13% <ø> (ø)
pytensor/link/jax/dispatch/tensor_basic.py	91.96% <ø> (ø)
pytensor/link/numba/dispatch/random.py	98.83% <100.00%> (ø)
... and 36 more

[jessegrabowski]

So if BLAS isn't installed and you try to compile a graph with a BatchedDot to C it just errors out?

[ricardoV94]

No, when C code of an Op raises not ImplementedError PyTensor switches to the Python implementation. That's the API contract.

I added a test as well and you can see the thunk is no longer a cthunk (it will be a pythunk) but it still runs

[jessegrabowski]

Ah yeah I see that now!