pyro-ppl/pyro

[FR] Error messages should be more specific

Open

#1,453 opened on 2018年10月16日

GitHub で見る
 (6 comments) (0 reactions) (0 assignees)Python (981 forks)batch import
good first issuehelp wantedwarnings & errors

Repository metrics

Stars
 (8,211 stars)
PR merge metrics
 (平均マージ 10d 19h) (30d で 1 merged PR)

説明

Users at PROBPROG 2018 requested more-specific error messages when models are invalid.

Specific improvements include:

  • #1509 Print sample site information when Distribution inputs are invalid. Currently it requires a debugger to see what line of the model is affected.
  • #1534 On error print shapes of sites in the trace: value shape for params, and value,dist,log_prob shapes for sample sites.
  • On error, print contextual information: the current cond_indep_stack, the portion of stack trace under svi.step(), whether the model or guide is being executed, _DIM_ALLOCATOR sate, tracing state, etc. This could be accomplished via a try: except: raise block to add information to a caught error.
  • others?

コントリビューターガイド

調査方針
`pyro/infer/svi.py` および関連するトレースモジュールにおける既存のエラー処理を調査します。PR #1509 と #1534 の実装を研究し、そのパターンを採用します。残りのタスクでは、`svi.step()` を囲む try except ブロック内にコンテキスト情報（例：`cond indep stack`、トレース状態、` DIM ALLOCATOR` 状態）を追加する必要があります。内部状態変数を確認し、エラー時に出力すると役立つものを特定します。
技術スタック
pythonpytorch
領域
developer experience
Issue 種別
機能
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
4
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-2日
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
古い
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
おおむね明確
前提条件
PythonPyTorchPyro internalsError handling
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
35