Skip to content

Commit c1ef460

Browse files
author
Dylan Huang
authored
Fix local UI (#295)
* vite build * Add subscription and event handling methods to LazyEventBus This fixed code from not starting event bus listening loop - Introduced `subscribe`, `unsubscribe`, `emit`, `start_listening`, and `stop_listening` methods to the _LazyEventBus class. - Enhanced type hinting for better code clarity and usability.
1 parent 42d5ecf commit c1ef460

9 files changed

Lines changed: 156 additions & 144 deletions

File tree

‎eval_protocol/event_bus/__init__.py‎

Lines changed: 15 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
# Global event bus instance - uses SqliteEventBus for cross-process functionality
2+
from typing import Any, Callable
23
from eval_protocol.event_bus.event_bus import EventBus
34

45

@@ -18,8 +19,20 @@ def _get_event_bus(self):
1819
self._event_bus = _get_default_event_bus()
1920
return self._event_bus
2021

21-
def __getattr__(self, name):
22-
return getattr(self._get_event_bus(), name)
22+
def subscribe(self, callback: Callable[[str, Any], None]) -> None:
23+
return self._get_event_bus().subscribe(callback)
24+
25+
def unsubscribe(self, callback: Callable[[str, Any], None]) -> None:
26+
return self._get_event_bus().unsubscribe(callback)
27+
28+
def emit(self, event_type: str, data: Any) -> None:
29+
return self._get_event_bus().emit(event_type, data)
30+
31+
def start_listening(self) -> None:
32+
return self._get_event_bus().start_listening()
33+
34+
def stop_listening(self) -> None:
35+
return self._get_event_bus().stop_listening()
2336

2437

2538
event_bus: EventBus = _LazyEventBus()

‎vite-app/dist/assets/index-BGlGI2LH.css‎

Lines changed: 1 addition & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎vite-app/dist/assets/index-BnDJont9.css‎

Lines changed: 0 additions & 1 deletion
This file was deleted.

‎vite-app/dist/assets/index-CnGlFAnP.js‎

Lines changed: 137 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎vite-app/dist/assets/index-CnGlFAnP.js.map‎

Lines changed: 1 addition & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎vite-app/dist/assets/index-Cu9t0G5i.js‎

Lines changed: 0 additions & 137 deletions
This file was deleted.

‎vite-app/dist/assets/index-Cu9t0G5i.js.map‎

Lines changed: 0 additions & 1 deletion
This file was deleted.

‎vite-app/dist/index.html‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,8 @@
55
<meta name="viewport" content="width=device-width, initial-scale=1.0" />
66
<title>EP | Log Viewer</title>
77
<link rel="icon" href="/assets/favicon-BkAAWQga.png" />
8-
<script type="module" crossorigin src="/assets/index-Cu9t0G5i.js"></script>
9-
<link rel="stylesheet" crossorigin href="/assets/index-BnDJont9.css">
8+
<script type="module" crossorigin src="/assets/index-CnGlFAnP.js"></script>
9+
<link rel="stylesheet" crossorigin href="/assets/index-BGlGI2LH.css">
1010
</head>
1111
<body>
1212
<div id="root"></div>

‎vite-app/src/GlobalState.tsx‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -268,7 +268,6 @@ export class GlobalState {
268268

269269
// Apply query params to global configuration
270270
applyQueryParams(queryParams: Record<string, string>) {
271-
debugger;
272271
const partialConfig = queryParamsToPartialConfig(queryParams);
273272

274273
// Apply each section of the partial config

0 commit comments

Comments
 (0)