You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Since a file-based session watcher was introduced in #150 one and a half years ago, a lot of features has been implemented around it, and they work well in a wide range of scenarios. This PR changes the status of the session watcher from experimental to normal by enabling r.sessionWatcher by default in the setting as I suggested at #668 (comment).
(If you do not have screenshot) How can I check this pull request?
The setting of r.sessionWatcher is enabled by default.
Not sure if this fits the scope of the PR, but should we also also pass r.previewDataframe and r.previewEnvironment commands to session watcher by default?
Not sure if this fits the scope of the PR, but should we also also pass r.previewDataframe and r.previewEnvironment commands to session watcher by default?
Yes, I think this PR should address these commands that are introduced before sesssion watcher. It might make sense to check on r.sessionWatcher and if it is enabled, then r.previewDataframe just sends View(X) and r.previewEnvironment sends View(globalenv()) to terminal or otherwise keep the old behavior.
previewEnvironment and previewDataframe are updated so that if session watcher is enabled, then View(globalenv()) and View(symbol) are sent to terminal, respectively; otherwise, fallback to the old behavior.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What problem did you solve?
Since a file-based session watcher was introduced in #150 one and a half years ago, a lot of features has been implemented around it, and they work well in a wide range of scenarios. This PR changes the status of the session watcher from experimental to normal by enabling
r.sessionWatcherby default in the setting as I suggested at #668 (comment).(If you do not have screenshot) How can I check this pull request?
The setting of
r.sessionWatcheris enabled by default.