pymc-devs/pymc

Introduce `pm.config` or `rcParams` for global & local config settings

Fermée

#4 657 ouverte le 21 avr. 2021

Voir sur GitHubGuide contributeur
 (8 commentaires) (0 réaction) (0 personne assignée)Python (1 902 forks)batch import
enhancementsfeature requesthelp wanted

Métriques du dépôt

Stars
 (7 926 étoiles)
Métriques de merge PR
 (Merge moyen 11j) (7 PRs mergées en 30 j)

Description

Let me begin with saying that global variables are evil and nobody should be using them.

For things like #4372 it would be really useful to have a place where to make global config changes. Matplotlib does it and ArviZ too. Aesara also has a quite powerful config implementation that I refactored quite a bit a few months back. Or maybe we don't even need/want a place for global configuration settings?

Anyways it would be great to discuss and figure out what we want!

Update 2022-03-30 + ToDo

We have a general consensus that we want to do this. (I'm updating the title+labels in a second.)

Whoever wants to work on this should

  1. Read this thread
  2. Specifically investigate the options here
  3. Decide which route to go for
  4. Open a PR

Guide contributeur

Direction de recherche
Lisez toute la discussion, en particulier le commentaire à issue#4657comment 826064715 qui décrit les approches possibles. Étudiez comment les rcParams de matplotlib et la config d'Aesara sont implémentés. Ensuite, proposez une conception et implémentez la dans un nouveau module. L'issue fournit une liste claire de TODO : lire le fil de discussion, étudier les options, décider de la voie, ouvrir une PR.
Stack technique
python
Domaine
backend
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
PythonConfiguration patterns
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60