Skip to content

feat: add api placeholder#1547

Open
Nexisato wants to merge 12 commits intomainfrom
feat/api-placeholder
Open

feat: add api placeholder#1547
Nexisato wants to merge 12 commits intomainfrom
feat/api-placeholder

Conversation

@Nexisato
Copy link
Copy Markdown
Contributor

Description

  • feat: add core_python api placeholder

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request significantly expands the SwanLab SDK's internal API layer, introducing new modules for experiment management, project operations, user/API key handling, and self-hosted deployment features. It includes new functions for experiment heartbeats, state updates, and metrics retrieval, along with corresponding type definitions and utility functions. Review feedback highlights a discrepancy between the type hint and docstring for the experiment metrics key, as well as a missing return type hint in the self-hosted user retrieval function.

Comment thread swanlab/sdk/internal/core_python/api/experiment.py
Comment thread swanlab/sdk/internal/core_python/api/self_hosted.py
@Nexisato Nexisato force-pushed the feat/api-placeholder branch from d3503f3 to 85392d1 Compare April 20, 2026 02:18
@Nexisato Nexisato self-assigned this Apr 20, 2026
@Nexisato Nexisato marked this pull request as ready for review April 21, 2026 05:55
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