Handle compute_map=None in Scan

[zaxtax]

Description

This should also fix some regressions in the recent release. Add a None check for compute_map in make_thunk

Related Issue

Closes #1425

Type of change

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

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.12%. Comparing base (92eef5e) to head (914e6e4).
⚠️ Report is 131 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1435   +/-   ##
=======================================
  Coverage   82.12%   82.12%           
=======================================
  Files         211      211           
  Lines       49751    49752    +1     
  Branches     8817     8818    +1     
=======================================
+ Hits        40857    40859    +2     
  Misses       6714     6714           
+ Partials     2180     2179    -1     

Files with missing lines	Coverage Δ
pytensor/scan/op.py	84.83% <100.00%> (+0.01%)	⬆️

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[ricardoV94]

It's not a hack, compute map for inner graphs is useless.

Instead of creating a compute_map make an if/else below where rval is returned and return an rval that doesn't try to assign to compute_map if it was None

[zaxtax]

@ricardoV94 is this change alright?

[ricardoV94]

Thanks @zaxtax