Skip to content

fix: experiment sessions can now be found on reruns#1804

Merged
alexgshaw merged 2 commits into
harbor-framework:mainfrom
samecrowder:sam_fix_experiment_session_reruns
Jun 2, 2026
Merged

fix: experiment sessions can now be found on reruns#1804
alexgshaw merged 2 commits into
harbor-framework:mainfrom
samecrowder:sam_fix_experiment_session_reruns

Conversation

@samecrowder

Copy link
Copy Markdown
Contributor

there was a bug in our plugin implementation that caused the langsmith experiment to not always be found on subsequent reruns of a task

Two changes:

  1. Append the job ID to the experiment name so each Harbor job creates a fresh session which avoids the 409 error I was seeing
  2. When a 409 does occur, look up the existing session by name via GET /sessions and use its actual ID (same pattern already used for datasets)

@github-actions

github-actions Bot commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

Enjoy a better diff viewing experience by clicking one of these URLs:

@alexgshaw alexgshaw merged commit 466fb98 into harbor-framework:main Jun 2, 2026
6 checks passed
Messimeimei added a commit to Messimeimei/bitfun-harbor that referenced this pull request Jul 7, 2026
81e9c27 Add job plugin support and refactor Harbor Hub upload (harbor-framework#1762)
a870a8b Add harbor-langsmith plugin package for LangSmith integration. (harbor-framework#1702)
284456b Add harbor-langsmith publish script and PyPI package metadata.
a7ea96b Fail fast on Harbor Hub auth errors when using --upload (harbor-framework#1781)
087217b claude_code: bundle one LLM turn into a single ATIF step (RFC-0001) (harbor-framework#1760)
5c7d0ea fix(openclaw): Use --retry-all-errors flag only when available (harbor-framework#1789)
67bcaea Add harbor leaderboard submit command (harbor-framework#1695)
466fb98 fix: experiment sessions can now be found on reruns (harbor-framework#1804)
298a27d Release harbor-langsmith 0.1.1
e23c4d5 ci: upgrade github actions to latest versions (harbor-framework#1819)
51f6007 v0.13.1

Co-authored-by: Cursor <cursoragent@cursor.com>
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