Fix supplying (base_)compiledir as str

[Armavica]

Description

Fixes the bug signaled in https://discourse.pymc.io/t/changing-the-base-compiled-in-pytensor/14764

Related Issue

• Closes #
• Related to Remove unused config options #806

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]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.38%. Comparing base (a6b9585) to head (418fdf8).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #927      +/-   ##
==========================================
+ Coverage   81.37%   81.38%   +0.01%     
==========================================
  Files         170      170              
  Lines       46845    46862      +17     
  Branches    11422    11423       +1     
==========================================
+ Hits        38118    38139      +21     
+ Misses       6545     6542       -3     
+ Partials     2182     2181       -1     

Files	Coverage Δ
pytensor/configdefaults.py	72.67% <100.00%> (+1.24%)	⬆️

... and 1 file with indirect coverage changes

[maresb]

Looks good to me. I committed a few minor changes to test_config.py:

• use the tmp_path fixture instead of mkdtemp
• separate test functions for separate cases

Feel free to revert if you don't like it.

I wonder if we could actually add tests where we set compiledir and then actually compile something, but if it's just for the C backend then it's probably not worthwhile. Also we're not testing ~-expansion, but also probably not worth it.

Thanks @Armavica!