plotly / plotly/plotly.py

Allow arbitrary `Mapping` types when setting properties, rather than only `dict`

Ouverte
#4,212 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

feature P3
Langage dominant
Python
Étoiles
18.8k
Forks
2.8k
Merge moyen
16 h 26 min
PR mergées (30 j)
21

Description

Currently, it seems that only dict is allowed when setting properties. It would be better if an arbitrary Mapping type were allowed, since that grants the user the flexibility to control properties with alternative data structures such as a ChainMap.

As far as I can tell, this could be done just by changing the validate_coerce() logic to check isinstance(v, Mapping) instead of isinstance(v, dict), e.g. on this line.

Example

This raises a ValueError:

from collections import ChainMap

import plotly.graph_objects as go


layout = ChainMap(
    dict(width=1000),
    dict(width=500, height=500),
)
go.Figure(layout=layout)

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez dans packages/python/plotly/_plotly_utils/basevalidators.py, au niveau de la logique validate_coerce() mentionnée dans l’issue. Vérifiez comment l’exemple Figure(layout=ChainMap(...)) est validé, puis vérifiez que des entrées Mapping arbitraires sont acceptées sans modifier le comportement existant de dict.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
python
Domaine
data-visualization
Type d'issue
Fonctionnalité
Difficulté
2/5
Temps estimé
1-3 heures
Activité
À l'abandon
Clarté
Clairement spécifiée
Accessibilité débutants
52/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.