futureverse / futureverse/progressr

Enable progressr if output is connected to a TTY - a slight improvement to interactive mode

Open
#197 0 comments 0 reactions 0 assignees View on GitHub
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.