Skip to content

Add tags for experiments#2052

Merged
deep1401 merged 12 commits into
mainfrom
add/experiment-tags-design
May 15, 2026
Merged

Add tags for experiments#2052
deep1401 merged 12 commits into
mainfrom
add/experiment-tags-design

Conversation

@deep1401

Copy link
Copy Markdown
Member

No description provided.

deep1401 and others added 6 commits May 12, 2026 07:52
Filesystem-stored tags in index.json, new tag CRUD endpoints under
the experiment router, CLI add/remove/list-by-tag commands, and
read-with-inline-edit chips in the experiments manager modal.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@netlify

netlify Bot commented May 12, 2026

Copy link
Copy Markdown

Deploy Preview for transformerlab canceled.

Name Link
🔨 Latest commit 1074c58
🔍 Latest deploy log https://app.netlify.com/projects/transformerlab/deploys/6a077ee155e3760008c67aa6

@polarityinc

polarityinc Bot commented May 12, 2026

Copy link
Copy Markdown

Paragon Review Skipped

Hi @deep1401! Your Polarity credit balance is insufficient to complete this review.

Please visit https://app.paragon.run to finish your review.

@sentry

sentry Bot commented May 12, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 64.28571% with 35 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...pi/transformerlab/routers/experiment/experiment.py 21.42% 22 Missing ⚠️
api/transformerlab/services/experiment_service.py 81.42% 9 Missing and 4 partials ⚠️

📢 Thoughts on this report? Let us know!

@greninja

Copy link
Copy Markdown
Contributor

Thoughts:

  1. after typing out a tag and clicking on "add", it takes a good amount of time for the tag to show up, sometimes 3-4 seconds
  2. the x button to remove a tag doesnt seem to be working
  3. would prefer "Edit tags (experiment name)" instead of "Edit tags — experiment name" as modal title
  4. all cli tags stuff worked fine

@deep1401

Copy link
Copy Markdown
Member Author

Thoughts:

  1. after typing out a tag and clicking on "add", it takes a good amount of time for the tag to show up, sometimes 3-4 seconds
  2. the x button to remove a tag doesnt seem to be working
  3. would prefer "Edit tags (experiment name)" instead of "Edit tags — experiment name" as modal title
  4. all cli tags stuff worked fine

Fixed 1 by making adding as a fire-and-forget op and storing locally for a brief time. remove button works now. Fixed the modal

@deep1401 deep1401 merged commit 56cd7ec into main May 15, 2026
24 checks passed
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