Fix bug in compute_log_likelihood when variable has dims without coords

[jaharvey8]

Bugfixes

Fixes #6820

---

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

[codecov]

Codecov Report

Merging #6882 (6e96bf4) into main (f77372c) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #6882      +/-   ##
==========================================
- Coverage   92.16%   92.16%   -0.01%     
==========================================
  Files         100      100              
  Lines       16841    16839       -2     
==========================================
- Hits        15521    15519       -2     
  Misses       1320     1320              

Files Changed	Coverage Δ
pymc/model/core.py	91.29% <ø> (ø)
pymc/backends/arviz.py	96.51% <100.00%> (+0.03%)	⬆️
pymc/sampling/jax.py	98.27% <100.00%> (-0.03%)	⬇️
pymc/stats/log_likelihood.py	100.00% <100.00%> (ø)

[ricardoV94]

Thanks @jaharvey8. Can you add a small test that was triggering the error before this fix?

[jaharvey8]

Sure, I hate to ask an annoying question, but what's the best format for uploading a test? Or is there documentation somewhere on uploading tests?

[ricardoV94]

Sure, I hate to ask an annoying question, but what's the best format for uploading a test? Or is there documentation somewhere on uploading tests?

Not annoying at all. There should some tests in tests/stats/test_log_likelihood.py (or something similar). You can add a separate test function for this bug there.

To run the tests locally check this guide: https://www.pymc.io/projects/docs/en/latest/contributing/running_the_test_suite.html

If you have any questions, just ask away!

[ricardoV94]

@jaharvey8 I refactored a bit the code to avoid duplication and added a test. Let me know if it looks good on your end.