Skip to content

Conversation

@j-or
Copy link
Contributor

@j-or j-or commented Jan 21, 2026

  • Add backwards compatible, optional UI label to score vars
  • Add initialStateMultiObjective and button to create a multiobjective experiment/project
Screenshot 2026-01-21 at 10 10 22 Screenshot 2026-01-21 at 10 10 00 Screenshot 2026-01-21 at 10 31 50

@j-or j-or requested a review from Copilot January 21, 2026 09:14
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR implements multiobjective experiment support by adding an optional UI label field to score variables and creating a new multiobjective experiment initialization option.

Changes:

  • Added optional label field to score variables for UI display (defaults to name if not provided)
  • Added initialStateMultiObjective state with two score variables and a button to create multiobjective experiments
  • Updated table headers and cells to display label ?? name for score variables

Reviewed changes

Copilot reviewed 13 out of 13 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
sample-app/src/experiment-container.tsx Reads multiObjective query parameter and passes it to ExperimentProvider
sample-app/src/components/home/home.tsx Adds new button and handler to create multiobjective experiments
packages/ui/src/features/data-points/useDataPoints.ts Updates score names to include label field and propagates it through data structures
packages/ui/src/features/core/editable-table/types.ts Adds optional label field to TableDataPoint type
packages/ui/src/features/core/editable-table/editable-table.tsx Updates table headers/footers to display label instead of name
packages/ui/src/features/core/editable-table/editable-table-expanded-row.tsx Updates expanded row headers to display label instead of name
packages/core/src/context/experiment/store.ts Adds initialStateMultiObjective with two score variables and updates default score variable structure
packages/core/src/context/experiment/experiment-reducers.ts Updates score variable creation to use label field
packages/core/src/context/experiment/experiment-context.tsx Adds isMultiObjective prop to conditionally use multiobjective initial state
packages/core/src/common/util/migration/schemas/17.json Adds optional label field to score variable schema
packages/core/src/common/types/common.ts Defines scoreNames and scoreLabels constants
README.md Adds note about building packages with watch mode
.changeset/nervous-points-move.md Documents the multiobjective feature addition

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@j-or j-or requested a review from langdal January 21, 2026 09:19
@j-or j-or requested a review from Copilot January 21, 2026 14:59
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 22 out of 22 changed files in this pull request and generated 6 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@j-or j-or requested a review from Copilot January 22, 2026 15:02
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 27 out of 27 changed files in this pull request and generated 4 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@j-or j-or requested a review from Copilot January 23, 2026 09:42
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 26 out of 26 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@j-or j-or merged commit e58e6cb into main Jan 23, 2026
1 check passed
@j-or j-or deleted the implement-multiobjective branch January 23, 2026 09:56
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.

3 participants