futureverse / futureverse/progressr
Enable progressr if output is connected to a TTY - a slight improvement to interactive mode
- Dominant language
- R
- Stars
- 299
- Forks
- 11
- PR merge metrics
- No merged PRs in 30d
Description
The default is to enable progress handler per:
```r
enable = getOption("progressr.enable", interactive())
```
The rationale is that you'll get progress updates when running R interactively, but not in batch mode (e.g. `Rscript main.R`).
However, it might be more natural to condition it on whether the output is connected to a TTY, e.g. `isatty(stdout())` or `isatty(stderr())`. However, does this logic apply to all progress handlers, or just those outputting to the terminal, or ...
Contributor guide
Research direction
Start by reviewing the default enable expression using interactive() and investigate R's isatty(stdout()) and isatty(stderr()) behavior. Determine whether TTY-based enabling should apply to every progress handler or only terminal-output handlers, then define the expected behavior for interactive and batch execution before adding coverage.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- r
- Domain
- tooling
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Needs clarification
- Newbie friendliness
- 45/100