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

Add charpoly as an alias for graph characteristic polynomials #11711

Closed
rbeezer mannequin opened this issue Aug 20, 2011 · 6 comments
Closed

Add charpoly as an alias for graph characteristic polynomials #11711

rbeezer mannequin opened this issue Aug 20, 2011 · 6 comments
Assignees
@jasongrout @rlmill @nathanncohen
Labels
c: graph theory good first issue p: minor / 4 t: enhancement
Milestone
sage-4.7.2

Comments

@rbeezer
Copy link
Mannequin

rbeezer mannequin commented Aug 20, 2011

Throughout the linear algebra code, characteristic_polynomial() and charpoly() are aliases. Patch makes it so for graphs (their adjacency matrices, really).

See discussion at:
http://groups.google.com/group/sage-devel/browse_thread/thread/a7e9eba4a501326f/9b32e4bbdfa0975a

Apply:

  1. attachment: trac_11711-graph-charpoly.patch

Component: graph theory

Author: Rob Beezer

Reviewer: Nathann Cohen

Merged: sage-4.7.2.alpha3

Issue created by migration from https://trac.sagemath.org/ticket/11711
@rbeezer rbeezer mannequin added this to the sage-4.7.2 milestone Aug 20, 2011
@rbeezer rbeezer mannequin added c: graph theory labels Aug 20, 2011
@rbeezer

This comment has been minimized.

Sign in to view
@rbeezer
Copy link
Mannequin Author

rbeezer mannequin commented Aug 20, 2011

comment:1

Attachment: trac_11711-graph-charpoly.patch.gz

@rbeezer
Copy link
Mannequin Author

rbeezer mannequin commented Aug 20, 2011

Author: Rob Beezer

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

nathanncohen mannequin commented Sep 11, 2011

comment:2

Passes doctests and does nothing dangerous. The Graph class does not really need any new method right now (more than 250 already), but well :-D

This one's good to go :-)

Nathann

@nathanncohen
Copy link
Mannequin

nathanncohen mannequin commented Sep 11, 2011

Reviewer: Nathann Cohen

@nexttime
Copy link
Mannequin

nexttime mannequin commented Sep 12, 2011

Merged: sage-4.7.2.alpha3

@nexttime nexttime mannequin removed the s: positive review label Sep 12, 2011
@nexttime nexttime mannequin closed this as completed Sep 12, 2011
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 good first issue p: minor / 4 t: enhancement
Projects
None yet
Milestone
sage-4.7.2
Development

No branches or pull requests
1 participant
@jasongrout