Ml61127/remove experiment type selector inside experiment page by ispoljari · Pull Request #20161 · mlflow/mlflow

ispoljari · 2026-01-20T21:27:27Z

This should actually be a chained PR where this branch goes into this one: #20160

This PR is the complete ML61123 feature/epic.

Related Issues/PRs

#xxx

ML61127

What changes are proposed in this pull request?

This PR removes the experiment type selector from experiment pages and replaces experiment kind logic to use the global WorkflowType context instead of reading from experiment tags. When workflow-based navigation is enabled, experiment kind is now derived from the global workflow type rather than per-experiment tags.

Key changes:

Hide experiment kind selector in ExperimentView and ExperimentPageTabs when feature flag is enabled
Create useExperimentKind hook that returns experiment kind from WorkflowType context (when enabled) or falls back to tags
Update all React components to use useExperimentKind instead of getExperimentKindFromTags
Short-circuit useInferExperimentKind hook when feature flag is enabled
Auto-set experiment kind tag in CreateExperimentModal based on current workflow type

How is this PR tested?

Existing unit/integration tests
New unit/integration tests
Manual tests

Does this PR require documentation update?

Release Notes

Is this a user-facing change?

No. You can skip the rest of this section.
Yes. Give a description of this change to be included in the release notes for MLflow users.

The experiment type selector has been removed from experiment pages. Experiment type is now determined by the global workflow type selection instead of per-experiment tags.

What component(s), interfaces, languages, and integrations does this PR affect?

Components

How should the PR be classified in the release notes? Choose one:

rn/none - No description will be included. The PR will be mentioned only by the PR number in the "Small Bugfixes and Documentation Updates" section
rn/breaking-change - The PR will be mentioned in the "Breaking Changes" section
rn/feature - A new user-facing feature worth mentioning in the release notes
rn/bug-fix - A user-facing bug fix worth mentioning in the release notes
rn/documentation - A user-facing documentation change worth mentioning in the release notes

Should this PR be included in the next patch release?

Yes should be selected for bug fixes, documentation updates, and other small changes. No should be selected for new features and larger changes. If you're unsure about the release classification of this PR, leave this unchecked to let the maintainers decide.

What is a minor/patch release?

Minor release: a release that increments the second part of the version number (e.g., 1.2.0 -> 1.3.0).
Bug fixes, doc updates and new features usually go into minor releases.
Patch release: a release that increments the third part of the version number (e.g., 1.2.0 -> 1.2.1).
Bug fixes and doc updates usually go into patch releases.

Yes (this PR will be cherry-picked and included in the next patch release)
No (this PR will be included in the next minor release)

github-actions · 2026-01-20T21:27:48Z

🛠 DevTools 🛠

Install mlflow from this PR

# mlflow
pip install git+https://github.com/mlflow/mlflow.git@refs/pull/20161/merge
# mlflow-skinny
pip install git+https://github.com/mlflow/mlflow.git@refs/pull/20161/merge#subdirectory=libs/skinny

For Databricks, use the following command:

%sh curl -LsSf https://raw.githubusercontent.com/mlflow/mlflow/HEAD/dev/install-skinny.sh | sh -s pull/20161/merge

github-actions · 2026-01-20T21:27:49Z

@ispoljari Thank you for the contribution! Could you fix the following issue(s)?

⚠ DCO check

The DCO check failed. Please sign off your commit(s) by following the instructions here. See https://github.com/mlflow/mlflow/blob/master/CONTRIBUTING.md#sign-your-work for more details.

github-actions · 2026-01-20T21:36:45Z

Documentation preview for 8435609 is available at:

https://pr-20161--mlflow-docs-preview.netlify.app/docs/latest/

More info

Ignore this comment if this PR does not change the documentation.
The preview is updated when a new commit is pushed to this PR.
This comment was created by this workflow run.
The documentation was built by this workflow run.

mlflow/server/js/src/experiment-tracking/components/modals/CreateExperimentModal.tsx

.../src/experiment-tracking/pages/experiment-logged-models/ExperimentLoggedModelDetailsPage.tsx

mlflow/server/js/src/experiment-tracking/utils/ExperimentKindUtils.ts

[ML-61124]: update translation labels [ML-61124]: add feature flag Remove databricks serving endpoint adapter and redirect databricks managed models to litellm adapter (mlflow#20037) Signed-off-by: Xiang Shen <xshen.shc@gmail.com> Co-authored-by: Claude Sonnet 4.5 <noreply@anthropic.com> [ML-61125]: set shouldEnableWorkflowBasedNavigation to false [ML-61124]: sidebar width

[ML-61125]: remove models from top level sidebar if workflowType is genai [ML-61125]: rename feature flag [ML-61125]: fix acc. to comments [ML-61125]: simplify [ML-61125]: fix prettier [ML-61125]: fix conflict

[ML-61126]: fix routing [ML-61126]: fix routing

[ML-61127]: resolve acc. to comments

Signed-off-by: Daniel Lok <daniel.lok@databricks.com>

daniellok-db

lg

daniellok-db · 2026-01-22T08:01:20Z

merging, python should be unaffected

ispoljari requested review from daniellok-db and hubertzub-db January 20, 2026 21:27

github-actions bot added area/uiux Front-end, user experience, plotting, JavaScript, JavaScript dev server rn/feature Mention under Features in Changelogs. labels Jan 20, 2026

This was referenced Jan 20, 2026

[ML-61124]: add selector for workflow type in top level navbar #20158

Merged

Ml61125/implement different sidebar tab configs based on workflow type #20159

Merged

Ml61126/remove nested sidebars within gateway and experiments tab #20160

Merged

daniellok-db reviewed Jan 21, 2026

View reviewed changes

mlflow/server/js/src/experiment-tracking/components/modals/CreateExperimentModal.tsx Outdated Show resolved Hide resolved

daniellok-db reviewed Jan 21, 2026

View reviewed changes

.../src/experiment-tracking/pages/experiment-logged-models/ExperimentLoggedModelDetailsPage.tsx Outdated Show resolved Hide resolved

daniellok-db reviewed Jan 21, 2026

View reviewed changes

mlflow/server/js/src/experiment-tracking/utils/ExperimentKindUtils.ts Outdated Show resolved Hide resolved

github-actions bot assigned daniellok-db Jan 21, 2026

ispoljari force-pushed the ML61127/remove-experiment-type-selector-inside-experiment-page branch 3 times, most recently from 1e57be1 to c92ab1a Compare January 22, 2026 02:09

ispoljari requested a review from daniellok-db January 22, 2026 02:10

ispoljari force-pushed the ML61127/remove-experiment-type-selector-inside-experiment-page branch from c92ab1a to 4194b96 Compare January 22, 2026 02:12

ispoljari added 4 commits January 22, 2026 14:33

[ML-61125]: switch render sidebar tabs based on worflow type

7e9a847

[ML-61125]: remove models from top level sidebar if workflowType is genai [ML-61125]: rename feature flag [ML-61125]: fix acc. to comments [ML-61125]: simplify [ML-61125]: fix prettier [ML-61125]: fix conflict

[ML-61126]: remove nested sidebars within gateway and experiment tabs

202dcc5

[ML-61126]: fix routing [ML-61126]: fix routing

[ML-61127]: remove the code paths depending on the experimentKind tag

6339914

[ML-61127]: resolve acc. to comments

daniellok-db force-pushed the ML61127/remove-experiment-type-selector-inside-experiment-page branch from 4194b96 to 6339914 Compare January 22, 2026 06:34

daniellok-db added 2 commits January 22, 2026 15:02

fixes

b7b421e

Signed-off-by: Daniel Lok <daniel.lok@databricks.com>

fix test

8435609

Signed-off-by: Daniel Lok <daniel.lok@databricks.com>

daniellok-db approved these changes Jan 22, 2026

View reviewed changes

daniellok-db merged commit cacf9c0 into mlflow:master Jan 22, 2026
50 of 54 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Ml61127/remove experiment type selector inside experiment page#20161

Ml61127/remove experiment type selector inside experiment page#20161
daniellok-db merged 6 commits intomlflow:masterfrom
ispoljari:ML61127/remove-experiment-type-selector-inside-experiment-page

ispoljari commented Jan 20, 2026 •

edited by daniellok-db

Loading

Uh oh!

github-actions bot commented Jan 20, 2026

Install mlflow from this PR

Uh oh!

github-actions bot commented Jan 20, 2026

Uh oh!

github-actions bot commented Jan 20, 2026 •

edited

Loading

Uh oh!

Uh oh!

Uh oh!

Uh oh!

daniellok-db left a comment

Uh oh!

daniellok-db commented Jan 22, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

ispoljari commented Jan 20, 2026 • edited by daniellok-db Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Related Issues/PRs

What changes are proposed in this pull request?

How is this PR tested?

Does this PR require documentation update?

Release Notes

Is this a user-facing change?

What component(s), interfaces, languages, and integrations does this PR affect?

How should the PR be classified in the release notes? Choose one:

Should this PR be included in the next patch release?

Uh oh!

github-actions bot commented Jan 20, 2026

Install mlflow from this PR

Uh oh!

github-actions bot commented Jan 20, 2026

⚠ DCO check

Uh oh!

github-actions bot commented Jan 20, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

daniellok-db left a comment

Choose a reason for hiding this comment

Uh oh!

daniellok-db commented Jan 22, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

ispoljari commented Jan 20, 2026 •

edited by daniellok-db

Loading

github-actions bot commented Jan 20, 2026 •

edited

Loading