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

Graph multiresolution: eps computation #2

Open
mys007 opened this issue Nov 1, 2016 · 1 comment
Open

Graph multiresolution: eps computation #2

mys007 opened this issue Nov 1, 2016 · 1 comment

Comments

@mys007
Copy link

mys007 commented Nov 1, 2016

The current code throws "GSP_GRAPH_SPARSIFY: Epsilon out of required range" if gsp_graph_multiresolution() comes to the point that the current graph has three nodes. I think L175 in gsp_graph_multiresolution.m should be if Gs{lev+1}.N>3 as 2/sqrt(3)>1, a condition in gsp_graph_sparsify(). Another useful condition would be sum(find(Gs{lev+1}.A,1))>0 as otherwise there is nothing to sparsify and this later leads to error in results=gendist(Pe',q,1);.
In addition a piggy-backed comment: gsp_graph_sparsify() should be documented that it is non-deterministic.

@mdeff
Copy link
Contributor

mdeff commented Sep 21, 2017

Thanks for reporting the issue. Note however that this toolbox is not developed nor maintained anymore. Please consider using the pygsp instead.

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

No branches or pull requests

2 participants