Improved docs and INI support at /interfaces/povray.py

[sagetrac-slosoi]

To improve docs of Povray, support for inifile.

Component: interfaces

Keywords: povray

Author: Sami Losoi

Reviewer: mhampton, boothby

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

[sagetrac-slosoi]

comment:3

There are some bugs apparently in the width and height parameters in the changed code.
The parameters F and P work differently as I expected.

[sagetrac-slosoi]

Attachment: povray.py.gz

[sagetrac-slosoi]

comment:4

Replying to @sagetrac-slosoi:

There are some bugs apparently in the width and height parameters in the changed code.
The parameters F and P work differently as I expected.

The patch -file is old one. I will submit a new one after the compilation of Sage.

[sagetrac-slosoi]

Attachment: povray_doc.2.patch.gz

[sagetrac-slosoi]

Attachment: povray_doc.patch.gz

[sagetrac-mhampton]

comment:6

Why are you removing the docstring from the class definition?

[boothby]

Reviewer: mhampton, boothby

[boothby]

comment:7

Somethings up with these patches. Both povray_doc.patch and povray_doc.2.patch are identical and won't apply on top of povray.py. Also, the file is missing class-level documentation and doctests, (as mhampton noted, you should move the file-level doc back) and nothing is doctested in the methods.