Code indexing in gitaly is broken and leads to code not being visible to the user. We work on the issue with highest priority.

Skip to content

Fix/auto updates

wakonig_k requested to merge fix/auto_updates into main

Description

After introducing docks, the auto update functionality didn't work anymore. This MR tries to bring it back to live. To also support live updates of the entire dock area, the auto update is now assigned to the dock area, not the BECFigure. As a result, a default update figure has to be declared, enabling the auto update to retrieved and updated it within the callback.

Related Issues

closes #180 (closed) #118 (closed) #120 (closed) #145 (closed)

Type of Change

  • Added support for docke area-based auto updates
  • Added e2e test to ensure the basic functionality works.

Potential side effects

  • Beamline repos need to be updated
  • BEC IPython client needs to assign the default bec figure upon startup
  • BECWidgets template for the plugin repo needs to be updated.

Additional Comments

I experienced quite a few problems whilst setting up the e2e test. As it turns out, the current implementation of the e2e tests share the bec client between client and server #192 (closed) . Somehow I doubt that this was done on purpose, at least we should discuss it. For now, I simply added a new fixture for the auto-updates test. Without it, the test would get stuck due to a blocking dispatcher callback.

Definition of Done

  • Documentation is up-to-date.
Edited by wakonig_k

Merge request reports