Skip to content

refactor: Rename 'tio' to 'dynemo-run'#18

Merged
nnshah1 merged 2 commits into
mainfrom
gk-dynemo-run-2
Mar 5, 2025
Merged

refactor: Rename 'tio' to 'dynemo-run'#18
nnshah1 merged 2 commits into
mainfrom
gk-dynemo-run-2

Conversation

@grahamking

Copy link
Copy Markdown
Contributor

No description provided.

Comment thread launch/dynemo-run/README.md Outdated
Comment thread launch/dynemo-run/README.md
Comment thread launch/dynemo-run/README.md
Comment thread launch/dynemo-run/README.md
Comment thread launch/dynemo-run/src/lib.rs
Comment thread launch/dynemo-run/src/main.rs Outdated
Comment thread launch/dynemo-run/src/main.rs

@nnshah1 nnshah1 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

a few places with comments to change to dynemo - otherwise LGTM

@github-actions

github-actions Bot commented Mar 5, 2025

Copy link
Copy Markdown
Contributor

Test Results

 2 files   2 suites   52s ⏱️
77 tests 77 ✅ 0 💤 0 ❌
99 runs  98 ✅ 1 💤 0 ❌

Results for commit b44d70b.

@nnshah1 nnshah1 disabled auto-merge March 5, 2025 01:27
@nnshah1 nnshah1 merged commit 7dc246d into main Mar 5, 2025
@nnshah1 nnshah1 deleted the gk-dynemo-run-2 branch March 5, 2025 01:27
kylehh pushed a commit to kylehh/dynamo that referenced this pull request Apr 11, 2025
kylehh pushed a commit to kylehh/dynamo that referenced this pull request Apr 11, 2025
kangclzjc added a commit to kangclzjc/dynamo that referenced this pull request Jun 4, 2026
…oder reuse, doc fixes

Low-risk cleanup batch from the independent review (no decision-path change):

- ai-dynamo#4 chain_augment: add ``predicted_kv_hit_rate`` to ``_PREDICTION_FIELDS``
  so it participates in first-writer-wins partial merge like the other three
  predicted_* fields (was silently dropped in any 2+ plugin PREDICT chain,
  contradicting the proto/Pydantic contract). +2 chain_augment tests.
- #10 engine_adapter: add ``scale_down_capped_by_throughput`` to
  ``_aggregate_disagg_load_reason`` priority (PSM disagg emits it; placed
  between scale_up and scale_down to mirror PSM's _PRIORITY).
- ai-dynamo#11 dead code: drop ``contributing_plugin_ids`` (built, never read) in
  pipeline._run_fanout_stage; drop ``_set_enabled`` + ``_plugin_ids``
  (no caller in PR #1; would KeyError if reached).
- ai-dynamo#18 _encode_fpm: use the canonical
  ``dynamo.common.forward_pass_metrics.encode`` (shared module-level encoder)
  instead of allocating a fresh ``msgspec.msgpack.Encoder`` per tick and
  re-implementing the encoding. Byte-identical wire format; keeps FPM
  serialization in lock-step with the rest of dynamo.
- ai-dynamo#17 transport ABC docstring: timeout is enforced by the transport
  (``call()`` wraps ``asyncio.wait_for``), not the orchestrator — the
  pipeline uses a bare gather to avoid double-counting the deadline.
- ai-dynamo#20 scheduler docstring: note the heartbeat-eviction monitor is not wired
  in this PR (last_heartbeat_at is recorded but unread; monitor is follow-up).
- ai-dynamo#21 transport contract test: 7 inputs (not 8) → 14 cases (multi_pool fixture
  was removed with component_name; comments were stale).
- ai-dynamo#22 metrics test: remove the dead no-op ``pass`` loop in _sample_value.

828 planner tests pass (was 825; +3 chain-augment / merge tests).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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