Skip to content

Testing/load testing#107

Draft
tensor-ninja wants to merge 5 commits intomainfrom
testing/load-testing
Draft

Testing/load testing#107
tensor-ninja wants to merge 5 commits intomainfrom
testing/load-testing

Conversation

@tensor-ninja
Copy link
Collaborator

No description provided.

tensor-ninja and others added 5 commits February 26, 2026 12:31
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@dipeshbabu
Copy link
Contributor

Nice direction overall: having a built-in load testing harness (waves, SSE tracking, optional host monitor, result artifacts) is super useful for Nightshift ops. I do have a few merge blockers to address first:

  • several files look minified into single-line blobs, which makes review and future maintenance hard; please restore normal formatting and run the formatter.
  • this PR removes uv.lock; please revert that here or split it into a separate PR so we can discuss lockfile policy.
  • visualize.py appears incomplete and the README references it; either implement it or remove the claim for now.
  • After that, I have a few non-blocking suggestions around avoiding sys.path hacks and making sudo iptables monitoring optional/graceful.

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.

2 participants