Skip to content

feat(cli): add SageMaker streaming support - #142

Draft
GregHolmes wants to merge 1 commit into
mainfrom
gh/sagemaker-cli
Draft

GregHolmes wants to merge 1 commit into
mainfrom
gh/sagemaker-cli

Conversation

@GregHolmes

Copy link
Copy Markdown
Contributor

Summary

  • add optional SageMaker HTTP/2 streaming transport support
  • support SageMaker-backed Listen and Aura Speak commands using AWS credentials
  • validate transport limitations and cover SageMaker command behavior with unit tests

Testing

  • uv run pytest packages/deepctl-core/tests/unit/test_base.py packages/deepctl-cmd-listen/tests/unit/test_ws_and_routing.py packages/deepctl-cmd-speak/tests/unit/test_speak_command.py -k sagemaker
  • uv run ruff check packages/deepctl-core/src/deepctl_core/sagemaker.py packages/deepctl-core/src/deepctl_core/base_command.py packages/deepctl-cmd-listen/src/deepctl_cmd_listen/command.py packages/deepctl-cmd-speak/src/deepctl_cmd_speak/command.py

Further testing is ongoing before this PR is ready for review.

@GregHolmes

Copy link
Copy Markdown
Contributor Author

Further testing is ongoing before I ask for this PR to be reviewed.

This branch has not been deployed

No deployments
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.

1 participant