Set default progress_bar option to None #2389
Conversation
This change sets the default value of `bigframes.options.display.progress_bar` to `None`.
This means progress bars will be disabled by default, instead of automatically detecting the environment ("auto").
Users can still enable them by setting the option to "auto", "terminal", or "notebook".
This change also adds a unit test to verify the default value and the behavior of the option context manager.
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
unit_tests_output.txt
Outdated
| @@ -0,0 +1,231 @@ | |||
| ============================= test session starts ============================== | |||
There was a problem hiding this comment.
Done. I have removed the unit_tests_output.txt file.
This change sets the default value of `bigframes.options.display.progress_bar` to `None`.
This means progress bars will be disabled by default, instead of automatically detecting the environment ("auto").
Users can still enable them by setting the option to "auto", "terminal", or "notebook".
This change also adds a unit test to verify the default value and the behavior of the option context manager.
Set default progress_bar option to None
PR created automatically by Jules for task 9809612783716536441 started by @tswast