OPEN-NEXT / wp2.2_dev

Initial proof-of-concept of open source development (OSD) status dashboard with data-mining & visualisation components
https://opennext.eu/
GNU Affero General Public License v3.0
3 stars 1 forks source link

Generate interaction graphs based on file co-edition #23

Closed jbon closed 4 years ago

penyuan commented 4 years ago

I have also been working on the issue participation part of this. Should we split this work somehow?

jbon commented 4 years ago

Sorry I meant only file co-editon and not issue participation. I should remove "and issue participation" from the title of this issue. I think we can first generate graphs separately and later see how we can combine them.

penyuan commented 4 years ago

Sorry I meant only file co-editon and not issue participation. I should remove "and issue participation" from the title of this issue. I think we can first generate graphs separately and later see how we can combine them.

Understood, thanks! I'll continue to work hard on this.

jbon commented 4 years ago

branch iss23 is dedicated to this issue. First commit fa189ad0

jbon commented 4 years ago

The basic functionality is now implemented as of commit b8f38f70, with some remaining TODOs we can convert into issues later on. @penyuan can you review, test, eventually merge the branch iss23 back to master and close the issue?

penyuan commented 4 years ago

Thanks so much. Yes, I will take a look at this branch and merge as appropriate. I've been working on issue #14 which will propagate some changes through other files in this repository, including this one.

penyuan commented 4 years ago

I see that this has been merged into the vis-js branch. I've tested it on the wp2.2_dev and wp2.2_reference repositories with no errors, and I can view the GraphML results in yEd.

I will close this issue and merge the changes in this issue and vis-js into the master branch.

jbon commented 4 years ago

ok i didn't expect you to to merge vis-js as well, but right, why not. I'll continue to commit on this branch though (as I just did).