search

assisi / assisipy

Python API for the ASSISI|bf project.
Other
0 stars 3 forks source link

Spawning non-sim arenas in the simulator seems like a bad idea #85

Open dmiklic opened 8 years ago

dmiklic commented 8 years ago

The CASUs get spawned, but nobody can connect to them, so they're pretty useles. Maybe split .arena files into two parts (physical layout and connection details?)

Remove default='tcp://localhost:5556' and code relying on that from sim.py

rmm-fcul commented 8 years ago

To use spawn_array it expects homogeneous addresses. I suppose the right thing to do is either a) enforce that all casus in a layer have the same sub/pub addr (in the .arena file) or b) at least add as a validation, either pre-experiment config checker or during the sim.py deployment. Then we could use the sub_addr from the first component in the layer for the sim.Control initialisation (line 204, sim.py).

On 19 August 2016 at 13:35, Damjan Miklic notifications@github.com wrote:

Remove default='tcp://localhost:5556' and code relying on that from sim.py

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/larics/assisi-python/issues/85, or mute the thread https://github.com/notifications/unsubscribe-auth/AHa-l4KGizGidC2dCACJYjzf8rVL7fW0ks5qhaMWgaJpZM4Jobse .