Allow arbitrary `Mapping` types when setting properties, rather than only `dict`
オープン
まだ誰も着手していません。
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)
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- 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