BridgesUNCC / bridges-java

JAVA Client library for Bridges
http://bridgesuncc.github.io
GNU General Public License v2.0
4 stars 12 forks source link

David changes #106

Closed squeetus closed 7 years ago

squeetus commented 7 years ago

KR,

A setServer() method is added to the bridges and connector classes. You can call bridges.setServer(server), bridges will pass the server argument to the constructor version of the method, and it will set either localhost, live (bridges-cs), or dev (bridges-clone) as the server_URL of the connector. It only accepts the following arguments: ['live', 'dev', 'local']. I also modified the output URL from bridges.visualize() to get the appropriate server name from connector.

However, I was unable to install astyle on my machine, so I have not been able to fix the formatting issues this introduces. If it's working on your machine, can you pull this branch, run the astyle make command, then merge these changes into the version2.0 branch?

krs-world commented 7 years ago

I have pushed a version of the Bridges to version2.0 which should be pretty decent. I also put in the setServer() function in Bridges. Pull from version 2.0. This version will do all data structures except Array

Still to do:

update changes to the Array JSON creation.

go through the process of removing all unused code in Bridges and 

delink ADTVisualizer from Bridges. or it will be there but wont be used

Remove generic params from Bridges class.

Then pray that the whole thing works!

 -- krs

On 6/12/17 5:54 PM, David Burlinson wrote:

KR,

A setServer() method is added to the bridges and connector classes. You can call bridges.setServer(server), bridges will pass the server argument to the constructor version of the method, and it will set either localhost, live (bridges-cs), or dev (bridges-clone) as the server_URL of the connector. It only accepts the following arguments: ['live', 'dev', 'local']. I also modified the output URL from bridges.visualize() to get the appropriate server name from connector.

However, I was unable to install astyle on my machine, so I have not been able to fix the formatting issues this introduces. If it's working on your machine, can you pull this branch, run the astyle make command, then merge these changes into the version2.0 branch?


    You can view, comment on, or merge this pull request online at:

https://github.com/krs-world/bridges/pull/106

    Commit Summary

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/krs-world/bridges/pull/106, or mute the thread https://github.com/notifications/unsubscribe-auth/AFZSOPeAk6-vvV4QP0ZTbyoSyCPfYD6rks5sDbN8gaJpZM4N3vEi.

-- Kalpathi Subramanian Ph: 704 687 8579 Associate Professor Email: krs@uncc.edu Dept of Computer Science Web:http://webpages.uncc.edu/krs The University of North Carolina Charlotte, NC 28202-0001