Claude finished @from2001's task in 2m 31s —— View job • claude/issue-319-20260128-1359 • Create PR ➔

Client simulator improvements completed

Summary

I've successfully implemented all three requested changes to the client simulator:

Removed --enable-recv option: The receiving behavior is now enabled by default. The enable_receive parameter defaults to True throughout the codebase.
Changed server address format: The default server address is now localhost instead of tcp://localhost. The code automatically adds the tcp:// prefix if it's not present (unless using ipc://), maintaining backward compatibility with the old format.
Added --transform-send-rate option: New command-line option to control the transform update send rate in Hz (default: 10.0 Hz). The update interval is now calculated dynamically as 1.0 / transform_send_rate.

Changes Made

Updated SimulatedClient.__init__() to accept transform_send_rate parameter with default of 10.0 Hz
Modified SimulatedClient.run() to calculate update interval from the send rate
Updated ClientSimulator.__init__() to accept and store transform_send_rate
Modified CLI parser to add --transform-send-rate argument
Changed default --server value from tcp://localhost to localhost
Removed --enable-recv argument
Added address normalization logic to support both formats

Note on Code Quality Checks

I was unable to run the standard pre-commit checks (black, ruff, mypy, pytest) because installing dev dependencies requires permission approval. However, the code changes are syntactically valid and follow the existing patterns in the file. Please run these checks locally before merging:

cd STYLY-NetSync-Server
pip install -e ".[dev]"
black src/ tests/
ruff check src/ tests/
mypy src/
pytest

--- | Branch: claude/issue-319-20260128-1359

Client simulator option #319

Description

Metadata

Metadata

Assignees

Labels

Projects

Milestone

Relationships

Development

Issue actions