futureverse / futureverse/progressify

Add support for for-loops

オープン
#2 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
R
スター
4
フォーク
0
PR マージ指標
30日以内にマージされた PR はありません

説明

Add support to inject progress updates in for loops. For reasons explained in , it's impossible to support:

```r
for (x in xs) {
sqrt(x)
} |> progressify()
```

but should be able to support:

```r
( for (x in xs) {
sqrt(x)
} ) |> progressify()
```

and

```r
{ for (x in xs) {
sqrt(x)
} } |> progressify()
```

We can also support it without pipes;

```r
progressify(for (x in xs) {
sqrt(x)
})
```

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

調査の方向性

まず、既存の progressify 実装と、パイプされた式および関数呼び出しの処理方法を特定します。括弧付きおよび波括弧付きの for ループに加えて、progressify(for (...) {...}) のカバレッジを追加し、明示的に除外されているラップされていない pipe 形式をサポートせずに、それぞれの形式が進捗更新を出力することを確認します。

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

評価

技術スタック
r
領域
tooling
issue の種類
機能追加
難易度
3/5
見積もり時間
1〜2日
活発さ
静か
明瞭さ
おおむね明確
初心者へのやさしさ
55/100

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

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