Non-exportable objects: Add support for future.globals.onReference=function(...) { }
まだ誰も着手していません。
評価
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 初心者へのやさしさ
- 35/100
- issue の種類
- 機能追加
- 明瞭さ
- 説明が足りない
- 活発さ
- 停滞
- 技術スタック
- r
調査の方向性
既存の future.globals.onReference の処理と future:::reference_filters() から始め、issue に示されている ignore_envirs フィルターも含めます。callback の戻り値の契約と、それが受け入れられる参照型とどのように相互作用するかを定義します。動作が仕様化され、一貫してサポートされれば作業は完了です。
索引モデルが issue の本文から書いたものです。
説明
Currently, R option future.globals.onReference can take values:
"ignore"- don't check for non-exportable objects among globals (fastest)"message"- produce a message when a non-exportable objects is detected"warning"- produce a warning when a non-exportable objects is detected"error"- produce an error when a non-exportable objects is detected
I think it could be useful to provide some further control to savvy users/developers out there. More precisely, support for something like:
options(future.globals.onReference = function(ref) {
# accept / reject 'ref' based on inspect
# Q: What should be returned? Above `"ignore"`, ..., `"error"`, or TRUE/FALSE?
})
Having this in place would simplify enabling future.globals.onReference = "error" by default. For instance, if it turns out there are references that can indeed be exported (yet to be heard of), then we can fall back to above filter.
OTH, there's alteady an internal future:::reference_filters() function that allows us to set, add, drop, reset the list of accepted reference types, e.g. (the following is already set)
future:::reference_filters("append", ignore_envirs = function(ref, typeof, class, ...) {
typeof != "environment"
})
Maybe this is enough for now?
- 主要言語
- R
- スター
- 1k
- フォーク
- 93
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
futureverse/future のほかの issue
-
The first code example in the README does not work対応中かも @sean0407 が 4 日前に担当しました。 オープンbug
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
futureverse/future#838 · コメント 1 件 · リアクション 1 件 ·
-
bug
難易度 4/5 3〜5日 初心者へのやさしさ 48/100
futureverse/future#833 ·
-
feature/resources RNG
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
futureverse/future#832 ·
-
RNG
難易度 5/5 1週間以上 初心者へのやさしさ 38/100
futureverse/future#831 · コメント 1 件 ·
-
feature request
難易度 5/5 1週間以上 初心者へのやさしさ 30/100
futureverse/future#822 · リアクション 1 件 ·
futureverse/future の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
HelenaLC/spatialdataR#284 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 71/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 71/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
-
Hackathon: Tier 1
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
pharmaverse/logrx#309 ·