Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feature]: Add code cell option to remove warning/error messages (warning: false and message: false) #104

Closed
p0bs opened this issue Nov 17, 2023 · 4 comments
Labels
p: medium Address issue before the next release if possible t: feature-request Request the addition of a new feature
Milestone

Comments

@p0bs
Copy link

p0bs commented Nov 17, 2023

Feature Description

First things first: many thanks for this wonderful repo!

I was looking through the old and new issues and couldn't find one that relates to using cell options.

At the moment, the 'hash-pipe' approach doesn't seem to work for me within webr-r code cells (using the v0.3.9 tag). For example, I don't seem to be able to remove messages in my output (when it runs the first time).


Screenshot 2023-11-17 at 09 16 06

Ultimately, my question is: Is this a standard case of PICNIC? And, in the unlikely case that it is not, would you be able to add this functionality?

Many thanks again.

@p0bs p0bs added the t: feature-request Request the addition of a new feature label Nov 17, 2023
@coatless
Copy link
Owner

Thanks for the issue! Right now, we do not receive formatted options from Quarto. So, we have to set up a custom listener on our end for each option that we want to support. With that in mind, there are certain Quarto options we cannot listen for due to where we are in the process, e.g. no support for echo: false. c.f. quarto-dev/quarto-cli#4761

So, is there a set of behaviors that you would find beneficial outside of disabling STDERR?

Minor note: We likely will not be able to differentiate between message and warning contexts without submitting an upstream change as the event types we read from the captureR setup are stdout and stderr.

@p0bs
Copy link
Author

p0bs commented Nov 17, 2023

From my perspective (that is only a sample of one), I'm just seeking a way to hide the red under a code chunk. As such, an option to suppress warnings/messages would be great.

Many thanks again.

@coatless
Copy link
Owner

@p0bs okidokie. I'll modify the title of the issue ticket to clarify the feature request.

We'll probably have a bit more support for this option after #93 is addressed in the next week or so.

@coatless coatless changed the title [Feature]: Add some cell options to webr-r code cells? [Feature]: Add code cell option to remove warning/error messages (warning: false and message: false) Nov 17, 2023
@coatless coatless added this to the 4.0 milestone Nov 17, 2023
@coatless coatless added the p: medium Address issue before the next release if possible label Nov 18, 2023
@coatless
Copy link
Owner

Addressed in #141

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
p: medium Address issue before the next release if possible t: feature-request Request the addition of a new feature
Projects
None yet
Development

No branches or pull requests

2 participants