Ticket #2583 (closed defect: fixed)

Opened 5 years ago

Last modified 5 years ago

[with patch, positive review] Sage 2.11.a0: doctest failure in plot.py due to #2580

Reported by: mabshoff Owned by: rlm
Priority: blocker Milestone: sage-2.11
Component: graph theory Keywords:
Cc: Work issues:
Report Upstream: Reviewers:
Authors: Merged in:
Dependencies: Stopgaps:

Description

sage -t -long devel/sage/sage/plot/plot.py
----------------------------------------------------------------------
Total time for all tests: 965.6 seconds
mabshoff@sage:/scratch/mabshoff/release-cycle/sage-2.11.alpha0$ ./sage -t -long devel/sage/sage/plot/plot.py
sage -t -long devel/sage-main/sage/plot/plot.py             **********************************************************************
File "plot.py", line 3860:
    sage: networkx_plot(C._nxg, pos=C.get_pos(), edge_colors=edge_colors, vertex_labels=False, vertex_size=0)
Exception raised:
    Traceback (most recent call last):
      File "/scratch/mabshoff/release-cycle/sage-2.11.alpha0/local/lib/python2.5/doctest.py", line 1212, in __run
        compileflags, 1) in test.globs
      File "<doctest __main__.example_118[17]>", line 1, in <module>
        networkx_plot(C._nxg, pos=C.get_pos(), edge_colors=edge_colors, vertex_labels=False, vertex_size=Integer(0))###line 3860:
    sage: networkx_plot(C._nxg, pos=C.get_pos(), edge_colors=edge_colors, vertex_labels=False, vertex_size=0)
    AttributeError: 'Graph' object has no attribute '_nxg'
**********************************************************************
1 items had failures:
   1 of  18 in __main__.example_118
***Test Failed*** 1 failures.
For whitespace errors, see the file .doctest_plot.py
         [68.5 s]
exit code: 256

As a reminder: Ye Shall doctest the all of Sage before submitting any patch that changes fundamental things.

Cheers,

Michael

Attachments

2583.patch Download (934 bytes) - added by rlm 5 years ago.

Change History

Changed 5 years ago by rlm

comment:1 Changed 5 years ago by rlm

  • Summary changed from Sage 2.11.a0: doctest failure in plot.py due to #2580 to [with patch, needs review] Sage 2.11.a0: doctest failure in plot.py due to #2580

comment:2 Changed 5 years ago by mabshoff

  • Summary changed from [with patch, needs review] Sage 2.11.a0: doctest failure in plot.py due to #2580 to [with patch, positive review] Sage 2.11.a0: doctest failure in plot.py due to #2580

Patch looks good to me and fixes the issue.

Cheers,

Michael

comment:3 Changed 5 years ago by mabshoff

  • Status changed from new to closed
  • Resolution set to fixed

Merged in Sage 2.11.alpha0

comment:4 Changed 5 years ago by mabshoff

  • Status changed from closed to reopened
  • Resolution fixed deleted
  • Summary changed from [with patch, positive review] Sage 2.11.a0: doctest failure in plot.py due to #2580 to [with patch, positive review, reopened] Sage 2.11.a0: doctest failure in plot.py due to #2580

But it causes a new doctest failure:

mabshoff@sage:/scratch/mabshoff/release-cycle/sage-2.11.alpha0$ ./sage -t -long devel/sage/sage/plot/plot.py
sage -t -long devel/sage-main/sage/plot/plot.py             File "plot.py", line 3513:
    sage: plot(x^(1/3), (x,-1,1))
Expected nothing
Got:
    WARNING: When plotting, failed to evaluate function at 99 points.
    Last error message: 'negative number cannot be raised to a fractional power'
    <BLANKLINE>
**********************************************************************
1 items had failures:
   2 of  28 in __main__.example_111
***Test Failed*** 2 failures.
For whitespace errors, see the file .doctest_plot.py
         [70.3 s]
exit code: 256

----------------------------------------------------------------------
The following tests failed:

so I am reopening it.

Cheers,

Michael

comment:5 Changed 5 years ago by mabshoff

  • Status changed from reopened to closed
  • Resolution set to fixed

This is unrelated to this patch and hence we will close it and open another ticket for it.

Cheers,

Michael

comment:6 Changed 5 years ago by mabshoff

  • Summary changed from [with patch, positive review, reopened] Sage 2.11.a0: doctest failure in plot.py due to #2580 to [with patch, positive review] Sage 2.11.a0: doctest failure in plot.py due to #2580
Note: See TracTickets for help on using tickets.