search

ukaea / UDA

Universal Data Access library to provide data over the network in a unified data object.
https://ukaea.github.io/UDA/
Apache License 2.0
11 stars 4 forks source link

Update accAPI.cpp to allow multiple uda servers connection #19

Closed DorisBDR closed 5 months ago

DorisBDR commented 11 months ago

retrieve the server host and port from environment to have a proper comaprison

jholloc commented 11 months ago

I've incorporated these changes in the the release/2.8.0 branch. Please use this branch for any future testing.

DorisBDR commented 11 months ago

i found another issue...this one is a little bit tricky to solve but very annoying for us. if we have a new client (e.g. MINT) and two uda servers one new and one old then we can't get data because there is an issue with the protocol_version...which is not correct (as it assumes that the protocol_version is the same for everyone...it is for some reason a global static variable on the client side).

DorisBDR commented 10 months ago

commits done on our side to support multiple servers (including different versions), waiting for the build to check that things are ok

olivhoenen commented 8 months ago

@jholloc can you please look at bringing in https://github.com/ukaea/UDA/pull/19/commits/7e6d5c065ac3f877d018cbe79dba62e7dbb9995e and https://github.com/ukaea/UDA/pull/19/commits/5255a1bf8ed49963ce8b77f4edc85574d5337a00 so we can get closer to a 2.8.0 release?

I get that this PR may no target the correct branch (main, shall probably be develop or release/2.8.0 instead, --> will need to be clarified in guidelines for developers in the future).

jholloc commented 5 months ago

This has been merged into release/2.8.0