Skip to content

fix(bec-dispatcher): add disconnect_owner method to manage widget slo…#1166

Draft
wakonig wants to merge 2 commits into
mainfrom
fix/cleanup_dispatcher
Draft

fix(bec-dispatcher): add disconnect_owner method to manage widget slo…#1166
wakonig wants to merge 2 commits into
mainfrom
fix/cleanup_dispatcher

Conversation

@wakonig
Copy link
Copy Markdown
Member

@wakonig wakonig commented May 20, 2026

Description

add disconnect_owner method to manage widget slot disconnections

Related Issues

[Cite any related issues or feature requests that are addressed or resolved by this pull request. Link the associated issue, for example, with fixes #123 or closes #123.]

Type of Change

  • Change 1
  • Change 2

How to test

  • Run unit tests
  • Open [widget] in designer and play around with the properties

Potential side effects

[Describe any potential side effects or risks of merging this PR.]

Screenshots / GIFs (if applicable)

[Include any relevant screenshots or GIFs to showcase the changes made.]

Additional Comments

[Add any additional comments or information that may be helpful for reviewers.]

Definition of Done

  • Documentation is up-to-date.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 20, 2026

Benchmark comparison

Threshold: 20% (lower is better).
Result: 0 regression(s), 0 improvement(s) beyond threshold.

No benchmark regression exceeded the configured threshold.

No benchmark improvement exceeded the configured threshold.

All benchmark results
Benchmark Baseline Current Change Status
BEC IPython client with companion app 5.7537 s 5.93222 s +3.10% ok
BEC IPython client without companion app 2.13192 s 2.01379 s -5.54% ok
Import bec_widgets 0.0132106 s 0.01365 s +3.33% ok
tests/unit_tests/benchmarks/test_dock_area_benchmark.py::test_add_waveform_to_dock_area 0.156049 s 0.142393 s -8.75% ok

@wakonig wakonig force-pushed the fix/cleanup_dispatcher branch from 0872b09 to 269228e Compare May 20, 2026 15:48
@codecov
Copy link
Copy Markdown

codecov Bot commented May 20, 2026

Codecov Report

❌ Patch coverage is 85.00000% with 6 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
bec_widgets/utils/ui_loader.py 79.31% 4 Missing and 2 partials ⚠️

📢 Thoughts on this report? Let us know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant