[Python] IPConnection und mehrere Verbindungen

[bparzella]

Hey,

 

ich möchte die python Bindings mit zwei Verbindungen nutzen (WLAN + lokaler brickd). Jedoch bekomme ich sowohl unter Linux als auch unter OSX nach einer zufälligen Zeit die folgende Exception:

Exception in thread Thread-2:
Traceback (most recent call last):
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/threading.py", line 552, in __bootstrap_inner
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/threading.py", line 505, in run
  File "/Users/bparzella/Developer/HomeManager/tinkerforge/ip_connection.py", line 247, in callback_loop
    device = self.devices[stack_id]
KeyError: 4

 

Bei der Suche nach der Ursache bin ich darauf gestoßen, das die callback_queue als Klassenvariable angelegt ist. Das heißt das alle Instanzen sich diese Variable teilen. Das würde heißen das der Eintrag der Queue zufällig an eine der Instanzen verteilt würde. Wenn nun ein Callback für einen Stack mit 4 Bricklets an die Instanz eines Stacks mit nur 2 Bricklets verteilt wird, kann das unter Umständen die oben genannte Exception auftreten (Callback für stack_id > 2).

 

Um es kurz zu machen sollte in der ip_connection.py die Variable als Objektvariable anstatt als Klassenvariable benutzt werden.

 

Alt:

    PLUGIN_CHUNK_SIZE = 32

    callback_queue = Queue()

    def __init__(self, host, port):
        """
        Creates an IP connection to the Brick Daemon with the given *host*
        and *port*. With the IP connection itself it is possible to enumerate the
        available devices. Other then that it is only used to add Bricks and
        Bricklets to the connection.
        """

        self.pending_add_device = None

 

Neu:

    PLUGIN_CHUNK_SIZE = 32

    def __init__(self, host, port):
        """
        Creates an IP connection to the Brick Daemon with the given *host*
        and *port*. With the IP connection itself it is possible to enumerate the
        available devices. Other then that it is only used to add Bricks and
        Bricklets to the connection.
        """

        self.callback_queue = Queue()

        self.pending_add_device = None

 

Ich haben den Patch angehängt.

 

Grüße

Benjamin

py_ipcon_multiple_connections.patch

[photron]

Problem erkannt und richtig behoben, danke

 

Ich hab deinen Patch gerade in unsere github Repository gepushed.

[Nic]

Habe die Bindings gerad nicht im Kopf, aber könnte das auch bei Delphi bzw. C# ein Problem werden ?

[photron]

Nein, das Problem war nur in Python.