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

"Generating key + certificate" message #74

Closed
wlandau opened this issue Jan 23, 2025 · 4 comments · Fixed by #75
Closed

"Generating key + certificate" message #74

wlandau opened this issue Jan 23, 2025 · 4 comments · Fixed by #75
Labels
question Further information is requested

Comments

@wlandau
Copy link

wlandau commented Jan 23, 2025

It's a small thing, but rnng_write_cert() prints "Generating key + certificate" messages when the R session is interactive. Example:

if (interactive) REprintf("Generating key + certificate [ ]");

I don't think the time delay (~1s) is burdensome enough to require a progress message, and I find messages from REprintf() difficult to suppress without also suppressing errors.

@shikokuchuo
Copy link
Owner

shikokuchuo commented Jan 23, 2025

Sometimes it can take ~2 seconds and is noticeable. Without a prompt, the user won't know that it's doing what it's supposed to be doing rather than just being slow.

Why do you need to suppress it for interactive sessions? Would something else be more helpful?

@shikokuchuo shikokuchuo added the question Further information is requested label Jan 24, 2025
@shikokuchuo
Copy link
Owner

Actually it's a good suggestion for nanonext and I'm merging a PR which removes it from here. However I will re-instate the user prompt at mirai unless you have a better suggestion. Thanks.

@wlandau
Copy link
Author

wlandau commented Jan 27, 2025

Moving the message to mirai sounds great, especially if you plan on using message() so messages are easy to suppress (c.f. https://devguide.ropensci.org/pkg_building.html#console-messages). Like the dev guide says, it would also be great to turn off the message with a function argument.

@shikokuchuo
Copy link
Owner

Since when does my code adhere to any style guide? ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants