plot/plot3d/tri_plot.py is at 0% coverage: get it to 100%

[sagetrac-mhampton]

The adaptive refinement code in tri_plot.py has almost no docstrings or tests. However, there are a lot of helpful comments in the code, and the file is not very long, so it should be pretty easy to bring this up to 100% coverage.

CC: @sagetrac-wcauchois @boothby

Component: graphics

Keywords: plotting

Author: Marshall Hampton

Reviewer: Minh Van Nguyen, Tom Boothby

Merged: sage-4.4.1.alpha2

Issue created by migration from https://trac.sagemath.org/ticket/8394

[sagetrac-mhampton]

adds basic doctests to tri_plot.py

[sagetrac-mhampton]

comment:1

Attachment: trac_8394_v1.patch.gz

[sagetrac-mhampton]

comment:2

The patch formally gets coverage of tri_plot.py from 0 to 100%. Since I still don't completely understand this module, the tests and descriptions could be better but this is an improvement.

[sagetrac-mvngu]

Changed author from mhampton to Marshall Hampton

[sagetrac-mvngu]

Reviewer: Minh Van Nguyen

[sagetrac-mvngu]

comment:3

Attachment: trac_8394-reviewer.patch.gz

The documentation is OK by me. I have just one trivial patch to add, which requires another pair of eyes other than mine. So if that reviewer patch gets a positive review, the whole ticket is good to go.

When the module plot/plot3d/tri_plot.py is added to the reference manual, building the reference manual with that newly added module would result in some warnings. But fixing that is another ticket.

[boothby]

Changed reviewer from Minh Van Nguyen to Minh Van Nguyen, Tom Boothby

[boothby]

comment:4

Looks great! Thank you!

[williamstein]

Merged: 4.4.1.alpha2

[sagetrac-mvngu]

Changed merged from 4.4.1.alpha2 to sage-4.4.1.alpha2