Skip to content

[Benches] Fix too long bench names in charts legends#21417

Open
PatKamin wants to merge 10 commits intointel:syclfrom
PatKamin:whole-torch-names
Open

[Benches] Fix too long bench names in charts legends#21417
PatKamin wants to merge 10 commits intointel:syclfrom
PatKamin:whole-torch-names

Conversation

@PatKamin
Copy link
Contributor

@PatKamin PatKamin commented Mar 2, 2026

No description provided.

@PatKamin PatKamin requested a review from a team as a code owner March 2, 2026 22:27
Copy link
Contributor

@lukaszstolarczuk lukaszstolarczuk left a comment

Choose a reason for hiding this comment

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

as this is "breaking change" for our results, maybe we should group this PR with other "breaking" ones...? like changing the json format we talked about...
EDIT: it only applies to torch benches, so it's ok to merge it now.

@PatKamin PatKamin force-pushed the whole-torch-names branch 2 times, most recently from 92f719e to 1156267 Compare March 5, 2026 08:52
@PatKamin PatKamin marked this pull request as draft March 5, 2026 08:59
@PatKamin PatKamin marked this pull request as ready for review March 5, 2026 09:02
@PatKamin PatKamin force-pushed the whole-torch-names branch from 1156267 to d28e238 Compare March 5, 2026 09:15
@PatKamin PatKamin force-pushed the whole-torch-names branch 2 times, most recently from 2c33911 to 20afd4b Compare March 9, 2026 09:52
@PatKamin PatKamin marked this pull request as draft March 10, 2026 08:52
@PatKamin PatKamin force-pushed the whole-torch-names branch from a565456 to 4bc554b Compare March 10, 2026 08:55
@PatKamin PatKamin marked this pull request as ready for review March 10, 2026 08:55
@PatKamin PatKamin force-pushed the whole-torch-names branch from 4bc554b to 56f5484 Compare March 10, 2026 08:56
@PatKamin PatKamin force-pushed the whole-torch-names branch from 63e3243 to efd6bef Compare March 10, 2026 19:43
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