Skip to content

Fix telemetry handling for local model paths - #1273

Merged
Yue Sun (KayMKM) merged 1 commit into
release/v0.3.0from
fix/release-v0.3.0-telemetry-path
Aug 3, 2026
Merged

Fix telemetry handling for local model paths#1273
Yue Sun (KayMKM) merged 1 commit into
release/v0.3.0from
fix/release-v0.3.0-telemetry-path

Conversation

@KayMKM

Copy link
Copy Markdown
Contributor

Summary

  • Normalize PathLike model references before telemetry string scrubbing.
  • Prevent successful commands using local model paths from exiting with a Path.replace TypeError during telemetry reporting.
  • Add regression coverage for Click options parsed with path_type=Path.
  • Cherry-picked as a single focused commit onto release/v0.3.0.

Validation

  • uv run pytest tests/unit/telemetry/test_click_group.py tests/unit/telemetry/test_utils_scrubbing.py -q (63 passed)
  • uv run ruff check src/winml/modelkit/telemetry/utils.py tests/unit/telemetry/test_click_group.py

@KayMKM
Yue Sun (KayMKM) merged commit 4df4a04 into release/v0.3.0 Aug 3, 2026
9 checks passed
@KayMKM
Yue Sun (KayMKM) deleted the fix/release-v0.3.0-telemetry-path branch August 3, 2026 08:46
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