#18289: Make graph display configurable
-------------------------------------+-------------------------------------
Reporter: vbraun | Owner:
Type: defect | Status: needs_review
Priority: major | Milestone: sage-6.7
Component: graph theory | Resolution:
Keywords: | Merged in:
Authors: Volker Braun | Reviewers: Nathann Cohen
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/vbraun/graph_preference | 6f1f1fe014e16abb644ba521347f0cbf085dff4a
Dependencies: | Stopgaps:
-------------------------------------+-------------------------------------
Comment (by vdelecroix):
Hello,
Perhaps it would be cool to first decide once for all the semantic of all
our display functions and methods in sage? There was a thread open for
that purpose in Sage. I guess we could open a ticket only for deciding
what should we do with `show`, `plot`, `pretty_print` `print`, etc.
Concerning the current branch, I would prefer that the
{{{
is_small = (0 < self.num_verts() < 20)
plot_graph = display_manager.preferences.plot_graphs != 'never' and \
(display_manager.preferences.plot_graphs == 'always' or is_small)
}}}
actually belongs to the display manager itself. People implenting classes
should not have to worry about the display manager preferences. Don't you
think? I would separate:
- which methods the class could possibly implement (e.g. `_latex_`,
`_rich_latex_`, `_ascii_art_`, `plot`)
- and let the display manager decide when we do things like `object`,
`plot(object)` or `latex(object)`
Vincent
--
Ticket URL: <http://trac.sagemath.org/ticket/18289#comment:85>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
