Don't close connection after each pass

Previously, the sim would close the connection and then start a new run upon re-connecting.
That wasn't compatible with libertem-live-server prototypes and didn't seem to correspond to the behavior
of the real detector software.

With this change, the sim will keep starting new acquisitions on the same connection
without closing it. This seems to work better.

Discusson with @sk1p: We should see if and when we want to simulate
the detector closing a connection. Effects of this change in the wild to be monitored!

Co-authored-by: Alexander Clausen <a.clausen@fz-juelich.de>

Author: uellue

docs/source/changelog/bugfix/mib-simulator-fixes.rst

@@ -1,7 +1,7 @@
 [Bugfix] Make the MIB simulator a bit more realistic
 ====================================================
 
-* Allow multiple acquisitions per connection in triggered mode (:pr:`159`).
+* Allow multiple acquisitions per connection in triggered and untriggered mode (:pr:`159`, :pr:`191`).
 * Fix sequence number in continuous mode; fix generated acquisition header for
   the continuous case
 * Fix control socket handling

src/libertem_live/detectors/merlin/sim.py

@@ -580,11 +580,6 @@ def handle_conn(self, connection: socket.socket):
                     self.trigger_event.clear()
                 self._sim.handle_acquisition(connection)
                 self.finish_event.set()
-                if not (self._wait_trigger or self._manual_trigger) and not self._continuous:
-                    logger.info(
-                        "not continuous and not triggering, stopping sim for this connection"
-                    )
-                    return
         finally:
             connection.close()