-
Notifications
You must be signed in to change notification settings - Fork 632
Twitter Stream Plugin #84
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
Conversation
This is awesome, Matthieu. I think the Gephi community is going to find this plugin super helpful---I know it is something I will highlight when teaching introduction sessions. I'm going to make a few comments/suggestions on the code to help improve it. I am not a member of the core development team; so, these are only my suggestions. However, since I have developed a few plugins for Gephi I've learned some of the common issues and hopefully this will help your plugin get launched more quickly. Thank you again for writing this and sharing it. |
* User Network : Do a weighter User to User network with parallel edges for RT and Mentions | ||
* Full Smart Netowrk : Do a full representation of User, Tweet, Hashtag, Url, Media & Symbol | ||
|
||
This README supports Markdown, see [syntax](https://help.github.com/articles/markdown-basics/) |
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 line that the readme supports markdown can be deleted.
One general comment about your calls to Also, it would be awesome if you could attach one or two screenshots to your plugin. It's really simple, just put images into the |
|
||
CTL_SimpleAction=Twitter Stream | ||
CTL_SimpleTopComponent=Twitter Stream | ||
HINT_SimpleTopComponent=WHEEEEEEE |
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.
That's what appears in the tooltip in the component and the copy can be improved I think :)
Cool, not an issue but a suggestion regarding edge types. I think there's a nice opportunity to leverage the multi-graph capabilities. Right now all edges are created equal but you could have typed edges (mention, follow etc..). You can do so by providing a type integer to the |
Hi Totet, Thanks, |
hi, |
Hi Totet, pretty awesome work! Can I suggest a few small improvements before merge? One would be adding a 'Type' string column or similar to nodes, so data can be filtered more easily. I created a pull request in your branch with these changes + a few details totetmatt#1 |
Implement some improvements for pull request gephi#84
Plugin available, congrats! |
No description provided.