Split convert observed data

[michaelosthege]

Description

convert_observed_data is a messy do-it-all conversion function that was recently refactored in #7299.

This PR takes the next step and splits it such that generator data is treated separately.

Related Issue

• Makes it easier to work on BUG: pm.Data does not accept dtype argument #7277.

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):
  • Generator data is no longer supported with pm.Data.
  • Generator data of integer type is no longer converted to floatX, but only to intX.

---

📚 Documentation preview 📚: https://pymc--7334.org.readthedocs.build/en/7334/

[ricardoV94]

I'm still of the opinion we should drop generators altogether. I don't find any documentation on how to use it, and I can't really think of all the implications this Op means inside PyTensor, which is supposed to be side-effects free unless told otherwise (it's not being told so here). It's also only applicable in the default backend / and even here only if you have models without Deterministics (or any workflow where separate functions are compiled from the same underlying graph).

If this is useful in VI, the machinery that creates and call functions can use regular Python code to work with generators (such as setting a shared variable to the next value in every iteration of the loop). I don't see why we need this magic at the PyTensor level.

[michaelosthege]

I'll add a depreaction warning to the conversion of generator data, asking people to get in touch if this is relevant.

[michaelosthege]

• Rebased
• Added deprecation warning
• Hopefully fixed the tests by defaulting to floatX conversion when user-provided data does not carry dtype information.

[ricardoV94]

I would rather not add these new methods to all as I prefer we to get rid of them in the future, so the less discoverable they are the better.

For reference the whole intX is pretty questionable: #7331

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.45%. Comparing base (a197b19) to head (d9bd20c).
Report is 2 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #7334      +/-   ##
==========================================
- Coverage   92.47%   92.45%   -0.03%     
==========================================
  Files         102      102              
  Lines       17188    17180       -8     
==========================================
- Hits        15894    15883      -11     
- Misses       1294     1297       +3     

Files	Coverage Δ
pymc/data.py	89.44% <100.00%> (+0.13%)	⬆️
pymc/pytensorf.py	90.38% <100.00%> (-0.53%)	⬇️

... and 20 files with indirect coverage changes