-
Notifications
You must be signed in to change notification settings - Fork 623
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
Bugfix isolated nodes #296
base: bridging-centrality-plugin
Are you sure you want to change the base?
Bugfix isolated nodes #296
Conversation
Fix pull request problems gephi#200
public static boolean isIsolatedNode(Graph graph, Node node) { | ||
for (Node s : graph.getNodes()) { | ||
if (graph.isAdjacent(node, s)) { | ||
return false; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will provoke NOT releasing the underlying read lock and therefore hang Gephi.
The following will make sure to release the read lock as well:
NodeIterable iterable = graph.getNodes();
for (Node s : iterable) {
if (graph.isAdjacent(node, s)) {
iterable.doBreak();
return false;
}
}
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi! I made the changes as you requested, can you make a check, please? Any other suggestion?
New plugin or plugin update?
What is the purpose of this plugin?
Generate Bridging Centrality of a Graph
How to test your plugin in Gephi?
Checklist before submission
master
branch to get the latest updates?pom.xml
file