Skip to content
This repository was archived by the owner on Dec 16, 2025. It is now read-only.

Conversation

@mina-parham
Copy link

No description provided.

@mina-parham mina-parham marked this pull request as draft November 3, 2025 16:52
@mina-parham mina-parham marked this pull request as ready for review November 3, 2025 21:06
experiment_id: Optional[str] = Form(None),
job_name: Optional[str] = Form(None),
tlab_job_id: Optional[str] = Form(None),
tlab_parent_job_id: Optional[str] = Form(None),
Copy link
Member

Choose a reason for hiding this comment

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

Since both of these would always come together, is it possible to send things in a single param instead so we dont increase too many parameters on this route and its easier to port things later to other endpoints when necessary

@mina-parham mina-parham closed this Nov 3, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants