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

Remove verbose parameter from all models #183

Closed
nicrie opened this issue Aug 19, 2024 · 0 comments · Fixed by #216
Closed

Remove verbose parameter from all models #183

nicrie opened this issue Aug 19, 2024 · 0 comments · Fixed by #216
Labels
design Internal code structure

Comments

@nicrie
Copy link
Contributor

nicrie commented Aug 19, 2024

Initially, I thought having a quick on/off switch to track the progress of dask computations would be useful. However, I realize now that I've never actually used the verbose parameter in any model. The two main reasons are:

  1. In its current form, it's not very informative. It shows the progress of each computation process, but users typically don't know how many processes there are, making the information incomplete.
  2. You can simply import from dask.diagnostics import ProgressBar and run the computation within a with context for similar progress tracking.

If you're okay with it, @slevang, I'd suggest adding a DeprecationWarning to indicate that we plan to remove this parameter in the future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design Internal code structure
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant