Closed WouterSpekkink closed 8 years ago
Where does the mds.jar come from? Is it available in maven central?
If it's not, maybe we can upload it where we store thirdparty libraries: https://github.com/gephi/gephi/tree/mvn-thirdparty-repo but we should avoid uploading jars to master-forge branch.
Hi Eduardo,
I expect it is not in maven central. It was created a while ago by two scholars from an Austrian institute, and they uploaded it to a dedicated website. Can I upload it to the third-party repo, or do you prefer to do that yourself?
Best,
Wouter
2016-09-21 15:34 GMT+01:00 Eduardo Ramos notifications@github.com:
Where does the mds.jar come from? Is it available in maven central?
If it's not, maybe we can upload it where we store thirdparty libraries: https://github.com/gephi/gephi/tree/mvn-thirdparty-repo but we should avoid uploading jars to master-forge branch.
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/gephi/gephi-plugins/pull/127#issuecomment-248631156, or mute the thread https://github.com/notifications/unsubscribe-auth/AGgoXkN31ndVW8ciEacaMdT7WFi2Fn-Qks5qsUBwgaJpZM4JMXAQ .
Is the source code available? What's the license?
I am not sure if the source code itself is available. It is released with a Creative Commons License.
This is the website: http://algo.uni-konstanz.de/software/mdsj/
2016-09-21 15:47 GMT+01:00 Eduardo Ramos notifications@github.com:
Is the source code available? What's the license?
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/gephi/gephi-plugins/pull/127#issuecomment-248635074, or mute the thread https://github.com/notifications/unsubscribe-auth/AGgoXqIa4rT3ElMGHX5L6erGimHEwrz_ks5qsUOEgaJpZM4JMXAQ .
Hmm, giving this another look makes me realise I should make more explicit that the plugin should not be used for commercial purposes, right?
2016-09-21 15:49 GMT+01:00 Wouter Spekkink wouterspekkink@gmail.com:
I am not sure if the source code itself is available. It is released with a Creative Commons License.
This is the website: http://algo.uni-konstanz.de/software/mdsj/
2016-09-21 15:47 GMT+01:00 Eduardo Ramos notifications@github.com:
Is the source code available? What's the license?
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/gephi/gephi-plugins/pull/127#issuecomment-248635074, or mute the thread https://github.com/notifications/unsubscribe-auth/AGgoXqIa4rT3ElMGHX5L6erGimHEwrz_ks5qsUOEgaJpZM4JMXAQ .
Yeah, I guess the plugin description should contain the citation to the mds library, and the no commercial use thing. Also, it would be nice to have the Citation in the Gephi UI, like statistics algorithms do.
You can also contact the author just to make sure it's ok to upload the jar in https://github.com/gephi/gephi/tree/mvn-thirdparty-repo
Hi Eduardo,
I will make some changes somewhere in the coming days (not sure when I will have the time). The reference is already included in the UI (I indeed took the other statistics modules as an example), and I report in every online description of the plugin. However, I still need to add something about the commercial-use stuff, because I feel that is not sufficiently clear.
I will contact the authors of the plugin as well. I will send you a message once I have made all the necessary changes.
Thanks.
Best,
Wouter
2016-09-26 9:26 GMT+01:00 Eduardo Ramos notifications@github.com:
Yeah, I guess the plugin should contain the citation to the mds library, and the no commercial use thing. Also, it would be nice to have the Citation in the Gephi UI, like statistics algorithms do.
You can also contact the author just to make sure it's ok to upload the jar in https://github.com/gephi/gephi/tree/mvn-thirdparty-repo
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/gephi/gephi-plugins/pull/127#issuecomment-249509010, or mute the thread https://github.com/notifications/unsubscribe-auth/AGgoXsQb2qztXc2D1qh4JjzZeco_92W_ks5qt4GzgaJpZM4JMXAQ .
Great, thanks!
Both authors of the plugin (Christian Pich and Ulrik Brand) are okay with us uploading the MDSJ library to the repository. I will make some small changes to the plugin soon to highlight the license's conditions of use, and then I think I will be ready for another pull request.
2016-09-26 9:49 GMT+01:00 Eduardo Ramos notifications@github.com:
Great, thanks!
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/gephi/gephi-plugins/pull/127#issuecomment-249513860, or mute the thread https://github.com/notifications/unsubscribe-auth/AGgoXkf_J4uJ5WZKDDO_2zEp1msIgemmks5qt4cugaJpZM4JMXAQ .
Cool! You can just update this one by pushing more commits to your branch.
El 27 sept. 2016 10:19 a. m., "Wouter Spekkink" notifications@github.com escribió:
Both authors of the plugin (Christian Pich and Ulrik Brand) are okay with us uploading the MDSJ library to the repository. I will make some small changes to the plugin soon to highlight the license's conditions of use, and then I think I will be ready for another pull request.
2016-09-26 9:49 GMT+01:00 Eduardo Ramos notifications@github.com:
Great, thanks!
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <https://github.com/gephi/gephi-plugins/pull/127#issuecomment-249513860 , or mute the thread https://github.com/notifications/unsubscribe-auth/AGgoXkf_J4uJ5WZKDDO_ 2zEp1msIgemmks5qt4cugaJpZM4JMXAQ .
— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/gephi/gephi-plugins/pull/127#issuecomment-249798216, or mute the thread https://github.com/notifications/unsubscribe-auth/AAMCpere3Wgdsve4E-8sUtJAPb1NWUsvks5quNGugaJpZM4JMXAQ .
Hey Eduardo,
I took at look at the third party repo: https://github.com/gephi/gephi/tree/mvn-thirdparty-repo
I am not sure to properly add the MDSJ libary to it. Could you give me some advice?
Best,
Wouter
2016-09-27 9:21 GMT+01:00 Eduardo Ramos notifications@github.com:
Cool! You can just update this one by pushing more commits to your branch.
El 27 sept. 2016 10:19 a. m., "Wouter Spekkink" notifications@github.com escribió:
Both authors of the plugin (Christian Pich and Ulrik Brand) are okay with us uploading the MDSJ library to the repository. I will make some small changes to the plugin soon to highlight the license's conditions of use, and then I think I will be ready for another pull request.
2016-09-26 9:49 GMT+01:00 Eduardo Ramos notifications@github.com:
Great, thanks!
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <https://github.com/gephi/gephi-plugins/pull/127# issuecomment-249513860 , or mute the thread https://github.com/notifications/unsubscribe- auth/AGgoXkf_J4uJ5WZKDDO_ 2zEp1msIgemmks5qt4cugaJpZM4JMXAQ .
— You are receiving this because you commented. Reply to this email directly, view it on GitHub <https://github.com/gephi/gephi-plugins/pull/127#issuecomment-249798216 , or mute the thread https://github.com/notifications/unsubscribe-auth/AAMCpere3Wgdsve4E- 8sUtJAPb1NWUsvks5quNGugaJpZM4JMXAQ .
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/gephi/gephi-plugins/pull/127#issuecomment-249798559, or mute the thread https://github.com/notifications/unsubscribe-auth/AGgoXsYB54h8dJfIgXTy9wEK_n2yWrcPks5quNIKgaJpZM4JMXAQ .
Hi Wouter, Soon I will create the artifact and upload it so you can use it.
Great, thanks. I guess I just have to point to a certain address in the pom file right? I already made most other changes. Also cleaned up some things in the code.
2016-10-03 13:20 GMT+01:00 Eduardo Ramos notifications@github.com:
Hi Wouter, Soon I will create the artifact and upload it so you can use it.
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/gephi/gephi-plugins/pull/127#issuecomment-251092140, or mute the thread https://github.com/notifications/unsubscribe-auth/AGgoXkSwFAlQWa6-cWLbSo4ZlL73l36Dks5qwPL-gaJpZM4JMXAQ .
I uploaded the artifact uni-konstanz.de.algo:mdsj
at https://github.com/gephi/gephi/blob/mvn-thirdparty-repo/uni-konstanz/de/algo/mdsj/0.2
You will probably need to add the thirdparty repository as in https://github.com/gephi/gephi/blob/master/pom.xml
Hey Eduardo,
I made a new commit. It took me a while, because I had some issues trying to create the MDSJ dependency, but I think I worked it out in the end. Small summary:
I think that is about it. I hope I got the code style right this time.
Best,
Wouter
Great, I will review the code as soon as possible and if necessary ask for additional changes.
Hi, I created a pull request in your branch that fixes some parts of the code. Please accept it so we can merge this.
Also, I don't really understand the algorithm so I will assume it's correct :laughing:
Hi Eduardo,
Just a check to be sure. It seems like a lot of code was removed, including the code that adds and removes columns to the data laboratory based on the settings of the user. It also seems that nothing came in their place. Won't this break the plugin, or am I overlooking something?
Best,
Wouter
2016-10-03 19:37 GMT+01:00 Eduardo Ramos notifications@github.com:
Hi, I created a pull request in your branch that fixes some parts of the code. Please accept it so we can merge this.
Also, I don't really understand the algorithm so I will assume it's correct 😆
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/gephi/gephi-plugins/pull/127#issuecomment-251188277, or mute the thread https://github.com/notifications/unsubscribe-auth/AGgoXheYFDFlyyLveHFCQ-5Z5AqaXYBGks5qwUtWgaJpZM4JMXAQ .
Oh, right. Yes, I was overlooking something. I see that you condensed a few hundred lines of code in only a dozen. Nice. :)
2016-10-03 21:43 GMT+01:00 Wouter Spekkink wouterspekkink@gmail.com:
Hi Eduardo,
Just a check to be sure. It seems like a lot of code was removed, including the code that adds and removes columns to the data laboratory based on the settings of the user. It also seems that nothing came in their place. Won't this break the plugin, or am I overlooking something?
Best,
Wouter
2016-10-03 19:37 GMT+01:00 Eduardo Ramos notifications@github.com:
Hi, I created a pull request in your branch that fixes some parts of the code. Please accept it so we can merge this.
Also, I don't really understand the algorithm so I will assume it's correct 😆
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/gephi/gephi-plugins/pull/127#issuecomment-251188277, or mute the thread https://github.com/notifications/unsubscribe-auth/AGgoXheYFDFlyyLveHFCQ-5Z5AqaXYBGks5qwUtWgaJpZM4JMXAQ .
Hi, I replaced those long parts with two for loops, as they were very repetitive. The plugin runs the same but you can try it before merging.
El 3 oct. 2016 10:43 p. m., "Wouter Spekkink" notifications@github.com escribió:
Hi Eduardo,
Just a check to be sure. It seems like a lot of code was removed, including the code that adds and removes columns to the data laboratory based on the settings of the user. It also seems that nothing came in their place. Won't this break the plugin, or am I overlooking something?
Best,
Wouter
2016-10-03 19:37 GMT+01:00 Eduardo Ramos notifications@github.com:
Hi, I created a pull request in your branch that fixes some parts of the code. Please accept it so we can merge this.
Also, I don't really understand the algorithm so I will assume it's correct 😆
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <https://github.com/gephi/gephi-plugins/pull/127#issuecomment-251188277 , or mute the thread https://github.com/notifications/unsubscribe- auth/AGgoXheYFDFlyyLveHFCQ-5Z5AqaXYBGks5qwUtWgaJpZM4JMXAQ .
— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/gephi/gephi-plugins/pull/127#issuecomment-251220956, or mute the thread https://github.com/notifications/unsubscribe-auth/AAMCpW3n7bfAD1eAL3ck_ZVmd-ZFNAKXks5qwWjqgaJpZM4JMXAQ .
Looks much better, thanks. Perhaps the part of the algorithm that you were wondering about is the part with the minValue, maxVaue stuff? That is just used when the user uses edges that represent distances or similarities. To handle them correctly, I needed to normalize them, and all that stuff is basically an implementation of a commonly used normalization formula.
Thanks for all your work. Must be a pain to go through all that code.
2016-10-03 21:45 GMT+01:00 Eduardo Ramos notifications@github.com:
Hi, I replaced those long parts with two for loops, as they were very repetitive. The plugin runs the same but you can try it before merging.
El 3 oct. 2016 10:43 p. m., "Wouter Spekkink" notifications@github.com escribió:
Hi Eduardo,
Just a check to be sure. It seems like a lot of code was removed, including the code that adds and removes columns to the data laboratory based on the settings of the user. It also seems that nothing came in their place. Won't this break the plugin, or am I overlooking something?
Best,
Wouter
2016-10-03 19:37 GMT+01:00 Eduardo Ramos notifications@github.com:
Hi, I created a pull request in your branch that fixes some parts of the code. Please accept it so we can merge this.
Also, I don't really understand the algorithm so I will assume it's correct 😆
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <https://github.com/gephi/gephi-plugins/pull/127# issuecomment-251188277 , or mute the thread https://github.com/notifications/unsubscribe- auth/AGgoXheYFDFlyyLveHFCQ-5Z5AqaXYBGks5qwUtWgaJpZM4JMXAQ .
— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub <https://github.com/gephi/gephi-plugins/pull/127#issuecomment-251220956 , or mute the thread https://github.com/notifications/unsubscribe-auth/AAMCpW3n7bfAD1eAL3ck_ ZVmd-ZFNAKXks5qwWjqgaJpZM4JMXAQ .
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/gephi/gephi-plugins/pull/127#issuecomment-251221489, or mute the thread https://github.com/notifications/unsubscribe-auth/AGgoXuFhKY4OjtzCBiyMBcFq6JVToujDks5qwWlkgaJpZM4JMXAQ .
Merged in commit 3231f8518b7b4e97856eef614623ff1dc4e92e97
Your plugin is now available at the update center :smiley:
I reopened the pull request, after resolving an issue I had with including the mdsj.jar, on which the plugin depends. To build the plugin locally, the mdsj.jar needs to be added to the local maven repository of the builder to prevent unresolved dependencies. I am not sure how to otherwise make sure that the dependency is resolved. I used the accepted solution mentioned here to resolve the dependency issue: http://stackoverflow.com/questions/4955635/how-to-add-local-jar-files-in-maven-project