Add tooltip link to navigate to traces tab with time range filter by serena-ruan · Pull Request #20466 · mlflow/mlflow

serena-ruan · 2026-01-30T10:25:41Z

🥞 Stacked PR

Use this link to review incremental changes.

stack/trace_requests_tooltip_filter [Files changed]
- stack/usage_tab_charts_tooltips [Files changed]
  - stack/assessments_distribution_tooltip [Files changed]
    - stack/assessments_null_backend_support [Files changed]
      - stack/assessment_filters_update [Files changed]
        
        stack/assessments_tooltip [Files changed]

Related Issues/PRs

#xxx

What changes are proposed in this pull request?

Add 'view traces' link in tooltip so that we can jump to the traces view filtered by the current timestamp. This currently only works for a single bar, if users want an easy way to filter a certain range I think we should support changing time range based on zoom in & out behavior instead later.

The selected time range is preserved between overview tab and traces tab, so when we jump back to overview tab the charts are automatically updated

Screen.Recording.2026-01-30.at.6.27.45.PM.mov

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.

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-02-02T09:40:00Z

🛠 DevTools 🛠

Install mlflow from this PR

# mlflow
pip install git+https://github.com/mlflow/mlflow.git@refs/pull/20466/merge
# mlflow-skinny
pip install git+https://github.com/mlflow/mlflow.git@refs/pull/20466/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/20466/merge

Copilot

Pull request overview

Adds a “View traces for this period” link inside overview chart tooltips to navigate to the Traces tab with a time-range filter derived from the hovered bucket.

Changes:

Add timestampMs to trace requests/latency chart datapoints so tooltips can navigate based on the hovered bucket.
Extend ScrollableTooltip to optionally render a “View traces for this period” link and navigate to the Traces tab with startTimeLabel=CUSTOM, startTime, and endTime.
Add unit tests for the URL builder and basic tooltip/link rendering behavior.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
mlflow/server/js/src/lang/default/en.json	Adds localized message entry for the “View traces for this period” link text.
mlflow/server/js/src/experiment-tracking/pages/experiment-overview/hooks/useTraceRequestsChartData.ts	Includes `timestampMs` in requests chart datapoints for navigation.
mlflow/server/js/src/experiment-tracking/pages/experiment-overview/hooks/useTraceLatencyChartData.ts	Includes `timestampMs` in latency chart datapoints for navigation.
mlflow/server/js/src/experiment-tracking/pages/experiment-overview/components/TraceRequestsChart.tsx	Passes `linkConfig` into `ScrollableTooltip` and memoizes tooltip content.
mlflow/server/js/src/experiment-tracking/pages/experiment-overview/components/TraceLatencyChart.tsx	Passes `linkConfig` into `ScrollableTooltip` for latency chart tooltips.
mlflow/server/js/src/experiment-tracking/pages/experiment-overview/components/OverviewChartComponents.tsx	Implements traces URL builder and adds optional traces navigation link in tooltip.
mlflow/server/js/src/experiment-tracking/pages/experiment-overview/components/OverviewChartComponents.test.tsx	Adds tests for traces URL builder and tooltip link visibility.

Copilot · 2026-02-02T09:44:40Z

...rc/experiment-tracking/pages/experiment-overview/components/OverviewChartComponents.test.tsx

@@ -1,12 +1,138 @@
 import { describe, it, expect } from '@jest/globals';
+import { render, screen, fireEvent } from '@testing-library/react';
 import { renderHook } from '@testing-library/react';


fireEvent is imported but not used. Either remove the import to keep the test file tidy, or use it in a click-navigation test for the new "View traces for this period" link.

Suggested change

import { renderHook } from '@testing-library/react';

.../js/src/experiment-tracking/pages/experiment-overview/components/OverviewChartComponents.tsx

...rc/experiment-tracking/pages/experiment-overview/components/OverviewChartComponents.test.tsx

github-actions · 2026-02-02T09:49:04Z

Documentation preview for 3e2871a is available at:

https://pr-20466--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.

hubertzub-db · 2026-02-02T14:15:14Z

...server/js/src/experiment-tracking/pages/experiment-overview/components/TraceLatencyChart.tsx

+                    linkConfig={{
+                      experimentId,
+                      timeIntervalSeconds,
+                      componentId: 'mlflow.overview.usage.latency.view_traces_link',


in managed codebase, there's a lint rule enforcing use of static componentIds so it will have to be addressed anyway. I think we can use static componentId="mlflow.overview.usage.latency.view_traces_link" prop on Typography.Link instead

Signed-off-by: Serena Ruan <serena.rxy@gmail.com>

serena-ruan force-pushed the stack/trace_requests_tooltip_filter branch from fe8ca16 to 9a0a822 Compare February 2, 2026 06:29

This was referenced Feb 2, 2026

Support is null/is not null for assessments filters #20500

Merged

Add tooltip linking to traces for scorers distribution charts #20499

Merged

serena-ruan force-pushed the stack/trace_requests_tooltip_filter branch from 9a0a822 to 53c395e Compare February 2, 2026 09:37

serena-ruan mentioned this pull request Feb 2, 2026

Add tooltip linking to traces for moving average charts of scorers #20503

Merged

29 tasks

serena-ruan force-pushed the stack/trace_requests_tooltip_filter branch from 53c395e to 56945db Compare February 2, 2026 09:39

serena-ruan marked this pull request as ready for review February 2, 2026 09:39

Copilot AI review requested due to automatic review settings February 2, 2026 09:39

github-actions bot added the rn/feature Mention under Features in Changelogs. label Feb 2, 2026

Copilot started reviewing on behalf of serena-ruan February 2, 2026 09:40 View session

Copilot AI reviewed Feb 2, 2026

View reviewed changes

serena-ruan mentioned this pull request Feb 2, 2026

Support null operators for assessments filters #20506

Merged

29 tasks

serena-ruan requested a review from hubertzub-db February 2, 2026 11:00

hubertzub-db approved these changes Feb 2, 2026

View reviewed changes

github-actions bot assigned hubertzub-db Feb 2, 2026

serena-ruan added 2 commits February 3, 2026 12:22

Add tooltip link to navigate to traces tab with time range filter

908ad69

Signed-off-by: Serena Ruan <serena.rxy@gmail.com>

comment

3e2871a

Signed-off-by: Serena Ruan <serena.rxy@gmail.com>

serena-ruan force-pushed the stack/trace_requests_tooltip_filter branch from 56945db to 3e2871a Compare February 3, 2026 04:35

serena-ruan added this pull request to the merge queue Feb 3, 2026

Merged via the queue into master with commit e798d29 Feb 3, 2026
51 checks passed

serena-ruan deleted the stack/trace_requests_tooltip_filter branch February 3, 2026 05:16

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add tooltip link to navigate to traces tab with time range filter#20466

Add tooltip link to navigate to traces tab with time range filter#20466
serena-ruan merged 2 commits intomasterfrom
stack/trace_requests_tooltip_filter

serena-ruan commented Jan 30, 2026 •

edited

Loading

Uh oh!

github-actions bot commented Feb 2, 2026

Install mlflow from this PR

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Feb 2, 2026

Uh oh!

Uh oh!

Uh oh!

github-actions bot commented Feb 2, 2026 •

edited

Loading

Uh oh!

hubertzub-db Feb 2, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

serena-ruan commented Jan 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🥞 Stacked PR

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 Feb 2, 2026

Install mlflow from this PR

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Feb 2, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

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

Uh oh!

Uh oh!

hubertzub-db Feb 2, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

serena-ruan commented Jan 30, 2026 •

edited

Loading

github-actions bot commented Feb 2, 2026 •

edited

Loading