plotly / plotly/plotly.py

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

オープン
#4,212 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

feature P3
主要言語
Python
スター
18.8k
フォーク
2.8k
平均マージ
16時間 26分
マージ済み PR(30日)
21

説明

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)

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

packages/python/plotly/_plotly_utils/basevalidators.py の、issue で参照されている validate_coerce() ロジックから始めてください。Figure(layout=ChainMap(...)) の例がどのように検証されるかを確認し、そのうえで、既存の dict の動作を変更せずに任意の Mapping 入力が受け入れられることを確認してください。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
python
領域
data-visualization
issue の種類
機能追加
難易度
2/5
見積もり時間
1〜3時間
活発さ
停滞
明瞭さ
明確に書かれている
初心者へのやさしさ
52/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。