Should `score()`/`apply_metrics()` return warning about failed metrics in a batch?

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

メンテナーはふだん 1 日以内に返信

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

  • #1085 @nikosbosse による — マージされずにクローズ

評価

難易度
5/5
見積もり時間
1週間以上
初心者へのやさしさ
20/100
issue の種類
機能追加
明瞭さ
説明が足りない
活発さ
停滞
技術スタック
r
領域
data

調査の方向性

まず pull request #803 の議論と、エラーの延期についてのリンク先のブログ記事を確認します。成功したスコアが出力されるまで、batch 実行で failed-metric の警告を延期すべきかどうかを判断し、実装箇所とテスト箇所を特定する前に、期待される動作を定義します。

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

説明

enhancement

Should we also print the warnings in a batch instead of eagerly at the time of occurrence? They could easily get lost in the other warnings as the functions are getting executed.

I imagine a setup where at the time of printing the successful scores dt, we also print "In addition, the following metrics failed with the following warning". This is nicely discussed in this blog post about deferring errors. Worth considering.

Originally posted by @jamesmbaazam in https://github.com/epiforecasts/scoringutils/pull/803#discussion_r1592175575

主要言語
R
スター
65
フォーク
24
平均マージ
1時間 24分
マージ済み PR(30日)
2

環境構築

Codespaces で開く

このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。

はじめの一歩

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

epiforecasts/scoringutils のほかの issue

epiforecasts/scoringutils の issue をすべて見る

似ている issue

R の issue をもっと見る

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

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