#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 | 4e6fd6bf921744ee94eef610ab3069b55e0d7e41
Dependencies: | Stopgaps:
-------------------------------------+-------------------------------------
Comment (by ncohen):
Helloooooooo,
> Basically because our 3d graphics sucks too hard right now.
Oh.
> The names don't capture the behavior of "graphics if it can be displayed
unobtrusively"
Should the name of the parameter capture it? Right now it can be set to
'always'. It is the value that `plot_graphs` takes that should carry the
meaning of "if it is small".
What about renaming the parameter to `graphical_rich_repr`, taking values
like `never`, `always`, `when_unobtrusive` (I am not a smart man™). I also
believe that it would be better to work with `if value == 'always' or
value == 'when_unobtrusive'` rather than `if value != 'never'`. We don't
know what will happen with this parameter.
> Plotting graphs was the default in SageNB, its
> also the default in Mathematica
> (https://reference.wolfram.com/language/ref/Graph.html).
Well, this branch is not about SageNB but about the notebook, and the
notebook behaves like Sage does on this respect.
> Nobody objected to plotting graphs in the notebook,
> they all mistakenly assumed that you were talking about the
> commandline interface.
Even if that were true, that would not mean that their 'no' should be
taken as a "yes, let the notebook plot small objects automatically". If
you think that they don't mind this change, mail to sage-devel and ask
them again.
I just don't want to give a positive review to a change when a question
was asked and people said "don't do it". That's all.
I added a small commit at public/18289 for a `_rich_repr_` in Posets and
Polyhedra. I was not able to find the Quiver class [1]. It is mostly a
copy/paste of your code for graphs.
Nathann
[1] By the way what is this?
http://www.sagemath.org/doc/reference/quivers/index.html
http://www.sagemath.org/doc/reference/combinat/sage/combinat/cluster_algebra_quiver/quiver.html
--
Ticket URL: <http://trac.sagemath.org/ticket/18289#comment:103>
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.
