Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

clean generic_graph.py (part 6) #26637

Closed
dcoudert opened this issue Nov 4, 2018 · 8 comments
Closed

clean generic_graph.py (part 6) #26637

dcoudert opened this issue Nov 4, 2018 · 8 comments

Comments

@dcoudert
Copy link
Contributor

dcoudert commented Nov 4, 2018

Done here:

  • correct method eulerian_orientation for graphs without edge (was not using the right set of vertices)

    • Should we move this method to orientation.py ?
  • some care in min_spanning_tree, but we have remaining issues:

    • algorithm "Prim_fringe" compare vertex labels. I added hash(..) around, but this is certainly not enough.
    • the output of min spanning tree methods is sorted. As said in clean spanning_tree.pyx #26547, there is no reason for that.
  • avoid a call to .edges in cycle_basis

CC: @tscrim @fchapoton @seblabbe

Component: graph theory

Author: David Coudert

Branch/Commit: 7766d8e

Reviewer: Travis Scrimshaw

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

@dcoudert dcoudert added this to the sage-8.5 milestone Nov 4, 2018
@dcoudert
Copy link
Contributor Author

dcoudert commented Nov 4, 2018

Branch: public/26637_generic_graph_part_6

@dcoudert
Copy link
Contributor Author

dcoudert commented Nov 4, 2018

New commits:

7766d8etrac #26637: part 6

@dcoudert
Copy link
Contributor Author

dcoudert commented Nov 4, 2018

Commit: 7766d8e

@tscrim
Copy link
Collaborator

tscrim commented Nov 5, 2018

comment:2

LGTM.

@tscrim
Copy link
Collaborator

tscrim commented Nov 5, 2018

Reviewer: Travis Scrimshaw

@jhpalmieri
Copy link
Member

comment:3

Is there a metaticket for files in graphs? I'm particularly interested in more on generic_graph.py, especially is_isomorphic, and also base/sparse_graph.pyx, especially iterator_edges. Both of these try to sort vertices, which cause Python 3 doctest failures in homology/simplicial_complexes.py and homology/examples.py.

@dcoudert
Copy link
Contributor Author

dcoudert commented Nov 5, 2018

comment:4

Let us use #26640 as meta ticket. Should have been opened earlier. Feel free to modify it as I don't know how to organize it.

@vbraun
Copy link
Member

vbraun commented Nov 7, 2018

Changed branch from public/26637_generic_graph_part_6 to 7766d8e

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants