Skip to content

Bridging Centrality Plugin #200

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

Merged
merged 8 commits into from
Jan 4, 2019
Merged

Conversation

santosardr
Copy link

Bridging Centrality (BC) depicts nodes connected to immediate neighbors that, by turn, are connecting clusters of nodes. BC is the result of Betweenness Centrality multiplied by Bridging Node Centrality. BC can be useful to identify links for groups of nodes. For instance, considering a protein network, BC could point up proteins acting as a bridge between complementary molecular functions. This plugin appends the new data columns to the Gephi data laboratory tab: Betweenness, Bridging Node and Bridging Centrality.

@eduramiba eduramiba self-assigned this Nov 13, 2018
@eduramiba
Copy link
Member

Hi, thanks for the pull request.

You should actually base your branch on our master branch, and create a pull request into our master-forge branch.

@santosardr
Copy link
Author

santosardr commented Nov 13, 2018 via email

@eduramiba
Copy link
Member

Sorry, I was confused, I misread the pull request target being our master. Your request is correct.

@santosardr
Copy link
Author

santosardr commented Nov 13, 2018 via email

@eduramiba
Copy link
Member

Hi,
I don't know much about publications, but you can check some existing Gephi citations: https://gephi.org/users/publications/

https://scholar.google.fr/scholar?cites=692893633449340225

@santosardr
Copy link
Author

santosardr commented Nov 13, 2018 via email

@santosardr
Copy link
Author

santosardr commented Nov 14, 2018 via email

@eduramiba
Copy link
Member

Mmm I think data laboratory limits precision, can you try with BigDecimal.class? Anyway, on double click it should show the full value.

@santosardr
Copy link
Author

santosardr commented Nov 14, 2018 via email

@eduramiba
Copy link
Member

I guess we should increase the shown precision to 5 or 6 decimals, and show full precision with bigdecimal, but that will need to wait for a next release.

Anyway, I advise you to use BigDecimal if you want some real precision, as floats and doubles have inherent precision loss, that's why we round it.

@santosardr
Copy link
Author

santosardr commented Nov 14, 2018 via email

@santosardr
Copy link
Author

santosardr commented Nov 14, 2018 via email

@santosardr
Copy link
Author

santosardr commented Nov 14, 2018 via email

…g the problem of limited numerical precision
…us the column with the same name created by this plugin; plotting in the VERTICAL orientation; dividing the domain into ten tips according to the maximum value
eduramiba added a commit that referenced this pull request Jan 4, 2019
@eduramiba eduramiba merged commit 2d5e813 into gephi:master-forge Jan 4, 2019
@eduramiba
Copy link
Member

Hi Anderson,
Finally your plugin is merged and should be available soon. Thank you very much for the effort and the patience for this to be merged.

I made some small corrections to your code, so please be sure to merge the following pull request into your codebase so you can later update the plugin: santosardr#1

@eduramiba
Copy link
Member

Hi,
Just noticed that you changed the root pom.xml in your branch and I merged it by accident. To clarify, plugins request should never touch other poms that their own module poms.

eduramiba added a commit that referenced this pull request Jan 5, 2019
@eduramiba
Copy link
Member

Your plugin is now available at https://gephi.org/plugins and in Gephi - Tools - Plugins menu

@santosardr
Copy link
Author

santosardr commented Jan 7, 2019 via email

santosardr added a commit to santosardr/gephi-plugins that referenced this pull request Mar 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants