Skip to content

feat: add DashServing HTTP adapter - #270

Closed
Yunnglin wants to merge 5 commits into
mainfrom
feat/dashserving-adapter
Closed

Yunnglin wants to merge 5 commits into
mainfrom
feat/dashserving-adapter

Conversation

@Yunnglin

Copy link
Copy Markdown
Collaborator

Summary

  • add a standalone DashServing Native HTTP adapter with POST /api and GET /health
  • forward generic HTTP JSON requests to a fixed internal runtime for both Twinkle and Tinker APIs
  • expose a dedicated adapter launcher and declare server-side HTTP dependencies
  • add an end-to-end mock covering Client -> ModelScope -> DashServing -> Adapter -> runtime

Validation

  • conda run -n twinkle python -m pytest -vv -s tests/dashserving/test_mock_e2e.py (5 passed)
  • pre-commit run for all changed files (all hooks passed)

Draft follow-up

  • validate body passthrough and the 600-second request path against a real DashServing runtime

@Yunnglin
Yunnglin force-pushed the feat/dashserving-adapter branch from 5cf1136 to 9707325 Compare August 26, 2026 02:38
@Yunnglin
Yunnglin force-pushed the feat/dashserving-adapter branch 2 times, most recently from bf79177 to 6371b30 Compare August 26, 2026 03:38
@Yunnglin
Yunnglin marked this pull request as ready for review August 26, 2026 09:44
Copilot AI lite review requested due to automatic review settings August 26, 2026 09:44

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@Yunnglin Yunnglin closed this Aug 26, 2026
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