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

module_degree_zscore directed version wrong? #98

Open
silverbullet1472 opened this issue May 10, 2021 · 1 comment
Open

module_degree_zscore directed version wrong? #98

silverbullet1472 opened this issue May 10, 2021 · 1 comment

Comments

@silverbullet1472
Copy link

Here is the MATLAB version comments:

 flag,   0, undirected graph (default)
%                       **1, directed graph: out-degree**
%                       **2, directed graph: in-degree**
%                       3, directed graph: out-degree and in-degree

This is bctpy version:

 # flag : int
    #     Graph type. 0: undirected graph (default)
    #                 **1: directed graph in degree**
    #                 **2: directed graph out degree**
    #                 3: directed graph in and out degree

Notice that they use different flag. And for py version, it transposes connection matrix for OUT DEGREE.
I look up the code and found we use NP.SUM AXIS=1 which means it applies row-wise summation, so it's made for OUT DEGREE, so maybe do not need transposition?

@silverbullet1472
Copy link
Author

I test 2 version using same dataset. I think it is wrong, the comment for py version should be changed.

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