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.

Sign up for GitHub

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

Jump to bottom

Sorting edges of a graph #9742

Closed
rbeezer mannequin opened this issue Aug 13, 2010 · 5 comments
Closed

Sorting edges of a graph #9742

rbeezer mannequin opened this issue Aug 13, 2010 · 5 comments
Assignees
@jasongrout @rlmill @nathanncohen
Labels
c: graph theory p: minor / 4 t: enhancement
Milestone
sage-4.6

Comments

@rbeezer
Copy link
Mannequin

rbeezer mannequin commented Aug 13, 2010

This patch adds a "key" argument to allow custom sorting of the output of the graph method edges(). It adds to the documentation to make it clear that vertices will not always have a default sort order and thus edges may not always sort properly or as expected.

See:

#9741

http://groups.google.com/group/sage-devel/browse_thread/thread/40ac90ee3f28d723/

http://groups.google.com/group/sage-devel/browse_thread/thread/5adbb850f787373c/

Component: graph theory

Author: Rob Beezer

Reviewer: Nathann Cohen

Merged: sage-4.6.alpha1

Issue created by migration from https://trac.sagemath.org/ticket/9742
@rbeezer rbeezer mannequin added this to the sage-4.6 milestone Aug 13, 2010
@rbeezer rbeezer mannequin added c: graph theory labels Aug 13, 2010
@rbeezer
Copy link
Mannequin Author

rbeezer mannequin commented Aug 13, 2010

comment:1

Attachment: trac_9742-graph-edges-sort.patch.gz

@rbeezer
Copy link
Mannequin Author

rbeezer mannequin commented Aug 13, 2010

Author: Rob Beezer

@rbeezer rbeezer mannequin added the s: needs review label Aug 13, 2010
@nathanncohen
Copy link
Mannequin

nathanncohen mannequin commented Sep 7, 2010

comment:2

Nothing to add ! Positive review :-)

Nathann

@qed777
Copy link
Mannequin

qed777 mannequin commented Sep 15, 2010

Reviewer: Nathann Cohen

@qed777
Copy link
Mannequin

qed777 mannequin commented Sep 15, 2010

Merged: sage-4.6.alpha1

@qed777 qed777 mannequin removed the s: positive review label Sep 15, 2010
@qed777 qed777 mannequin closed this as completed Sep 15, 2010
@rbeezer rbeezer mannequin mentioned this issue Feb 10, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jasongrout jasongrout

@rlmill rlmill

@nathanncohen nathanncohen

Labels
c: graph theory p: minor / 4 t: enhancement
Projects
None yet
Milestone
sage-4.6
Development

No branches or pull requests
1 participant
@jasongrout