Opened 4 years ago

Last modified 3 years ago

#17815 needs_work defect

Remove verbose from the global namespace

Reported by: rws Owned by:
Priority: major Milestone: sage-6.6
Component: user interface Keywords:
Cc: Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description (last modified by vbraun)

Its confusing to users. Why are we exposing our logging system to end users?

Stretch goal: Use Python logging instead of reinventing the logging wheel.

Change History (5)

comment:1 Changed 4 years ago by rws

  • Milestone changed from sage-6.6 to sage-duplicate/invalid/wontfix

Wrong command! It should be set_verbose(-1).

comment:2 Changed 4 years ago by rws

  • Status changed from new to needs_review

comment:3 Changed 4 years ago by rws

  • Status changed from needs_review to positive_review

comment:4 Changed 4 years ago by vbraun

  • Description modified (diff)
  • Milestone changed from sage-duplicate/invalid/wontfix to sage-6.6
  • Status changed from positive_review to needs_work
  • Summary changed from cannot turn off plot warnings with verbose() to Remove verbose from the global namespace

I suggest to repurpose the ticket, see new description. The old description follows:

What use is sage.misc.misc.verbose("WARNING:... in plot.py when no possible verbosity level will turn off the output?

sage: verbose(level=-1)
verbose -1 (<module>) 
5.786
sage: plot(sqrt(x),(x,-1,3))
verbose 0 (2691: plot.py, generate_plot_points) WARNING: When plotting, failed to evaluate function at 51 points.
verbose 0 (2691: plot.py, generate_plot_points) Last error message: 'math domain error'

comment:5 Changed 3 years ago by dkrenn

See also #21349.

Note: See TracTickets for help on using tickets.