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

construct "fuzzy ball" graphs #9142

Closed
jasongrout opened this issue Jun 4, 2010 · 10 comments
Closed

construct "fuzzy ball" graphs #9142

jasongrout opened this issue Jun 4, 2010 · 10 comments

Comments

@jasongrout
Copy link
Member

The Fuzzy Ball graphs are cospectral with respect to the normalized laplacian matrix. This patch makes a function to construct such graphs. I will be adding a reference in a separate patch once we publish our paper :).

CC: @nathanncohen @rlmill

Component: graph theory

Author: Jason Grout

Reviewer: Nathann Cohen

Merged: sage-4.4.4.alpha0

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

@jasongrout
Copy link
Member Author

Attachment: trac-9142-fuzzy-ball.patch.gz

@nathanncohen
Copy link
Mannequin

nathanncohen mannequin commented Jun 4, 2010

comment:2

Positive review to this patch, which will be nicer with a reference :-)

I add a small patch with minor corrections : some math formulas were written plain text, which reflects badly on the indices. "Anyone but me can review this patch", as Minh says !

Nathann

@nathanncohen
Copy link
Mannequin

nathanncohen mannequin commented Jun 4, 2010

comment:5

Please forget about my patch, as it makes no difference. I can not see why, though O_o

@nathanncohen
Copy link
Mannequin

nathanncohen mannequin commented Jun 4, 2010

comment:6

Only because dvipng was not installed on my machine. Please consider my patch anew, and my excuses for this mistake :-)

Nathann

@nathanncohen
Copy link
Mannequin

nathanncohen mannequin commented Jun 4, 2010

comment:7

I had forgotten to add the "r" in front of """. Updated !

Nathann

@jasongrout
Copy link
Member Author

Attachment: trac_9142-reviewer.patch.gz

apply on top of previous patches

@jasongrout
Copy link
Member Author

comment:8

Attachment: trac-9142-small-doc-fix.patch.gz

Positive review to your patch. Can you look at my minor change in the explanation, to make it clear that you pass a list instead of a sum in for the partition?

@nathanncohen
Copy link
Mannequin

nathanncohen mannequin commented Jun 4, 2010

comment:9

Agreed :-)

Nathann

@mwhansen
Copy link
Contributor

mwhansen commented Jun 6, 2010

Merged: sage-4.4.4.alpha0

@mwhansen
Copy link
Contributor

mwhansen commented Jun 6, 2010

Reviewer: Nathann Cohen

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

2 participants