Skip to content

Commit 3c07790

Browse files
authored
Add batched publishing to pubsub (#281)
* Make pubsub system buffer-based to reduce syscall overhead in the hot-loop * Remove service shutdown timeout, add debug logging for in-flight info in MetricsAggregator service * Add python logging configuration to subscriber services * Remove msync temporarily for ARM bottleneck * PR comments, documentation cleanup and fixes, remove stray files * Fix load generator flow graph, rename saturation to warmup * Move file structure section up, remove mentioning of removed features * Add non-first chunks to load generator data flow diagram
1 parent 68c8846 commit 3c07790

24 files changed

Lines changed: 1329 additions & 1570 deletions

File tree

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -196,3 +196,4 @@ examples/03_BenchmarkComparison/vllm_venv/
196196
# User-specific local rules (local Docker dev); do not commit
197197
.cursor/rules/local-docker-dev.mdc
198198
CLAUDE.local.md
199+
.cursor_artifacts/

AGENTS.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -351,6 +351,8 @@ Known failure modes when AI tools generate code for this project. Reference thes
351351
- **Importing removed or renamed modules**: After refactors, AI (working from stale context) may import old module paths. Always verify imports resolve to actual files.
352352
- **Over-documenting**: AI generates verbose docstrings, inline comments explaining obvious code, and type annotations on trivial variables. This project prefers minimal comments — only where the _why_ isn't obvious from the code.
353353
- **Adding backwards-compatibility shims**: If something was renamed or removed, AI may add re-exports, aliases, or deprecation wrappers. In this project, just delete the old thing and update all call sites.
354+
- **Empty except blocks**: Every `except` block must contain either a comment explaining why the exception is ignored, or a logging statement. Bare `except: pass` without explanation is disallowed. AI often generates empty handlers — always add the reason.
355+
- **No lazy imports**: All imports must be at the top of the file. Imports inside functions, methods, or conditional blocks (other than `TYPE_CHECKING`) are disallowed. The only exceptions are: (1) circular import avoidance with a documenting comment, (2) optional dependencies with a top-level try/except that sets the import to `None`, (3) security sandboxing code that intentionally restricts imports.
354356

355357
### Dependency & Environment
356358

accuracy_only_support.md

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

0 commit comments

Comments
 (0)