#18972: twographs and Seidel switching
-------------------------------------+-------------------------------------
Reporter: dimpase | Owner:
Type: enhancement | Status: needs_work
Priority: major | Milestone: sage-6.9
Component: graph theory | Resolution:
Keywords: | Merged in:
Authors: | Reviewers: Nathann Cohen
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/dimpase/seidelsw | 6bc5cbd678d6d9db6e99ff23489e0e1a720e8569
Dependencies: #18960, #18948, | Stopgaps:
#18988, #18991, #18986, #19018, |
#19019 |
-------------------------------------+-------------------------------------
Comment (by ncohen):
> > - There are still broken links. If you can't be bothered to read the
doc, at
> > least compile it with `--warn-links`.
>
> I normally build docs with make. Does it mean I better do an explicit
`-docbuild`?
With make? It must take a lot of time! I compile the doc with `sage -b &&
sage -docbuild reference/combinat html`. You can add a `--warn-links` flag
to it to get a report of broken links.
> no, for two-graphs it's the way it is. Should I rename this
`.complement()` as
> `twograph_complement()`?
HMmmmmm `:-/`
It is true that the "right" notion of "complement" really depends on what
you do when it comes to hypergraphs. I made the other one the default
because it applies to all hypergraphs, and not only to uniform ones.
I thought for a while of your proposition to make it a `new method`, but
then why don't you just use `.complement(uniform=True)`? If you are ready
to leave `.complement` as it is and to create a new function, then do you
really need that function?
> well, I don't like inplace things. I don't even have an inplace option
for this function.
> I hope it's OK.
It's surprising, at least for me. Would you object if I added a commit
that makes it an inplace function, with an optional `inplace=False` to get
a copy?
> I'm going to fix the rest and add a construction of two-graphs (more
precisely, certain Seidel adj.matrices, from which the two-graph can be
constructed, if needed) from doubly-transitive permutation groups, i.e.
Taylor two-graphs.
Nice. Could you please do that in another ticket, please? This one is
already dozens of commits long.
Nathann
--
Ticket URL: <http://trac.sagemath.org/ticket/18972#comment:83>
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.
