search

pyros-dev / pyros

Python interfacing for multiprocess software - A Python blanket for ROS to hide inside
BSD 3-Clause "New" or "Revised" License
24 stars 4 forks source link

Topic connection time #104

Closed asmodehn closed 7 years ago

asmodehn commented 7 years ago

We need to improve the way we ensure topics are connected...

Currently this code is making everything depending on that very slow : https://github.com/asmodehn/pyros/blob/master/pyros/rosinterface/topic.py#L266

But if we try to do it faster, topic communication is not established when returning from __init__(), and things start breaking, when running podium with connection cache...

When running podium without connection cache however, topic interfacing seems fine. So it is likely a race condition between the connection cache message frequency, and the time to establish a topic connection.

But when running tests there doesn't seem to be any difference if this sleep time is slow or fast, with or without cache.

More investigation needed...

asmodehn commented 7 years ago

This has now been improved after splitting pubs and subs. We check for at least a connection before returning from init()

asmodehn commented 7 years ago

done and 0.3.0 has been released.