8000 CVS versions. by bhuffaker · Pull Request #1 · CAIDA/topostats · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

CVS versions. #1

New issue

Have a question about this project? Sign up for a fr 8000 ee 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
Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

CVS versions. #1

wants to merge 3 commits into from

Conversation

bhuffaker
Copy link
Member

I have updated with the CVS versions

@avinen
Copy link
Collaborator
avinen commented Feb 9, 2018

In normalize_centrality() in betweenness.c, you have the following lines:

printf("num_nodes: %lu factor:%e\n",num_nodes, d); // debug
return;

That cuts short the function and looks like temporary debugging code. Is that right?

@avinen
Copy link
Collaborator
avinen commented Feb 9, 2018

Also, some code in distance.c for printing out the progress of the run was lost when you imported from CVS (that is, the git repository had some commits that the CVS tip didn't). Could you please review the diff and make sure all changes are ones you made?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bhuffaker @avinen
0