`not isinstance(float_arg, float)` should narrow to int; `not isinstance(complex_arg, complex)` should narrow to float
オープン
まだ誰も着手していません。
bug
topic-runtime-semantics
topic-type-narrowing
- 主要言語
- Python
- スター
- 20.6k
- フォーク
- 3.3k
- PR マージ指標
- PR 指標を取得中
説明
The typing specs require that a float typed argument also accept int values, and that a complex type argument accept both int and complex values. That implies that, if we have a val typed complex, and isinstance(val, complex) returns false, it can still be an int or float. So,
def test(x: float):
if not isinstance(x, float):
reveal_type(x) # Should reveal int
and,
def test(x: complex):
if not isinstance(x, complex):
reveal_type(x) # Should reveal int | float (or possibly just float)
But they actually get treated as unreachable code and don't reveal anything.
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
まず、issue にある 2 つの snippet を再現し、isinstance チェックを処理する既存の型の絞り込みテストまたは実装を調べます。完了条件は、最初の例で int が、2 番目の例で int | float またはドキュメントに記載された許容される絞り込みが明らかになり、分岐が到達不能として扱われないことです。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- python
- 領域
- devtools
- issue の種類
- バグ
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 45/100