arviz-devs / arviz-devs/arviz

Consider changing `rcParams` default for `log_likelihood` to `False`

Open
#2,155 3 comments 0 reactions 0 assignees View on GitHub
Check Status Discussion
Dominant language
TeX
Stars
1.9k
Forks
508
Avg merge
2d 20h
Merged PRs (30d)
3

Description

https://github.com/arviz-devs/arviz/blob/2d886383019eb95ea9174ea39070c15d03ffdb7e/arvizrc.template#L10

Paraphrasing from the PyMC issue: https://github.com/pymc-devs/pymc/issues/6266

> This evaluation can be quite expensive and require too much memory. I think it's safe to say most users don't make use of this information making it pretty wasteful.

Would require coordinating with other PPLs that use this variable to see if they agree and can help users make the transition.

CC @OriolAbril

Contributor guide

Open the contributing guide

Research direction

Start with arvizrc.template at line 10 and read the linked PyMC issue to understand the cost and transition concerns. Confirm which other PPLs use this variable and whether they agree on a coordinated change; done means an agreed migration plan and an updated default.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
data
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.