grafana / explore-profiles

Explore Profiles is a native Grafana application designed to integrate seamlessly with Pyroscope, the open-source continuous profiling platform, providing a smooth, query-less experience for browsing and analyzing profiling data.
GNU Affero General Public License v3.0
17 stars 1 forks source link

feat(CompareView): Implement new Comparison view with Scenes #119

Closed grafakus closed 2 weeks ago

grafakus commented 1 month ago

✨ Description

Related issue(s): is blocked by https://github.com/grafana/explore-profiles/pull/117

This PR introduces the new comparison view:

image

As well as the ability to select the same item for baseline and comparison in the "Labels" view:

image

📖 Summary of the changes

See diff tab for specific comments.

🧪 How to test?

github-actions[bot] commented 1 month ago

Unit test coverage

Lines Statements Branches Functions
Coverage: 10%
11.12% (464/4170) 8.6% (134/1558) 8.32% (107/1285)
ifrost commented 2 weeks ago

A few minor UI issues:

Issue Screenshot
Bottom border of the highlight is cut off Screenshot 2024-08-23 at 08 57 46
Missing padding at the bottom of comparison graph and label being positioned to high Screenshot 2024-08-23 at 08 58 59
grafakus commented 2 weeks ago

A few minor UI issues: Bottom border of the highlight is cut off

It's been fixed in fixed in https://github.com/grafana/explore-profiles/pull/119/commits/f9791ffabfd37281901b699dd92a718d72febeb6#diff-408e010c621ad2b07badfec78e18b0eac9453f9ad586e631f85d5fdc48dae0d7R424

Missing padding at the bottom of comparison graph and label being positioned to high

I can't see the issue, which browser do you use?

grafakus commented 2 weeks ago

A few minor UI issues: Bottom border of the highlight is cut off

It's been fixed in fixed in f9791ff#diff-408e010c621ad2b07badfec78e18b0eac9453f9ad586e631f85d5fdc48dae0d7R424

Missing padding at the bottom of comparison graph and label being positioned to high

I can't see the issue, which browser do you use?

It's actually an issue with the selector that removes the border from the panel. Fixed.