Reduce number of `try()`, `suppressMessages()`, `suppressWarnings()` used in the package
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
調査の方向性
まず、scoringutils のコードベースで try()、suppressMessages()、suppressWarnings() を検索します。提案されている内部関数と外部関数の境界を理解するために、以前の NAMESPACE に関する議論と pull request 791 のフィードバックを読みます。影響を受けるパッケージコード全体で動作を維持しつつ、無関係な条件を隠すことなくこれらのパターンを減らせれば完了です。
索引モデルが issue の本文から書いたものです。
説明
scoringutils codebase contains a lot of try(), suppressMessages(), suppressWarnings(). This is likely the symptoms that a refactoring or redesign is necessary. In particular, one potential cause can be the lack of delineation between internal and external functions, as discussed in the earlier NAMESPACE section.
try(), suppressMessages(), suppressWarnings() should usually be a last resort, and not a common pattern in the codebase. Since they affect all conditions, not just conditions created by scoringutils, they can also hide deeper issues, and make debugging harder.
Originally posted by @Bisaloo in https://github.com/epiforecasts/scoringutils/pull/791#pullrequestreview-2003401135
- 主要言語
- R
- スター
- 65
- フォーク
- 24
- 平均マージ
- 1時間 24分
- マージ済み PR（30日）
- 2
環境構築
このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
epiforecasts/scoringutils のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
epiforecasts/scoringutils#1264 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
epiforecasts/scoringutils#1265 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
epiforecasts/scoringutils#1260 ·
メンテナーはふだん 1 日以内に返信
-
Incorrect guidance in `NEWS.md` re `add_relative_skill()`?対応中かも @nikosbosse が 3 日前に担当しました。 オープンbug
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
epiforecasts/scoringutils#1226 · コメント 1 件 · リアクション 2 件 ·
メンテナーはふだん 1 日以内に返信
-
get_pit_histogram(): unsorted breaks produce negative densities; num_bins = 1 returns an empty table対応中かも @nikosbosse が 3 日前に担当しました。 オープンbug
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
epiforecasts/scoringutils#1219 ·
メンテナーはふだん 1 日以内に返信
epiforecasts/scoringutils の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
waldronlab/bugphyzz#301 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
r-lib/tree-sitter-r#209 ·
-
難易度 1/5 1時間未満 初心者へのやさしさ 67/100
-
難易度 1/5 1時間未満 初心者へのやさしさ 79/100
cran-task-views/CausalInference#52 · コメント 1 件 ·