New issue
Advanced search Search tips

Issue 606052 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug



Sign in to add a comment

Bisect results table sometimes formatted wrong

Project Member Reported by dtu@chromium.org, Apr 22 2016

Issue description

The table entries are sometimes extra long, and not enough space is provided, causing the table to jumble together.

Here's an example:
https://bugs.chromium.org/p/chromium/issues/detail?id=595331#c5

It happens with not just the revision, as in this case, but also the means and standard deviations.
 
Thank you thank you thank you for taking a look at this.

But is it a dupe of https://github.com/catapult-project/catapult/issues/2093 ?

Comment 2 by dtu@chromium.org, Apr 22 2016

Another example with the mean:
https://bugs.chromium.org/p/chromium/issues/detail?id=599945#c5

Comment 3 by dtu@chromium.org, Apr 22 2016

Annie: oh, you're right! Does the table need to be limited to 80 characters?
That's a really good question! I think the important thing is how it displays in monorail, and monorail doesn't have a set line length as far as i can tell--on my current window the first line of this bug is way over 80 chars, but then i make the window smaller and it wraps.

So I can only think of a hand-wavey answer like "make it as concise as possible while still wrapping if it must to show all the data", which probably doesn't help much.
Project Member

Comment 5 by bugdroid1@chromium.org, Apr 26 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/26ef26441a318284b93d7522239aa2ae29e38597

commit 26ef26441a318284b93d7522239aa2ae29e38597
Author: catapult-deps-roller <catapult-deps-roller@chromium.org>
Date: Tue Apr 26 20:22:02 2016

Roll src/third_party/catapult/ 89ac5c3cc..7744caa3a (9 commits).

https://chromium.googlesource.com/external/github.com/catapult-project/catapult.git/+log/89ac5c3cccf7..7744caa3a0f0

$ git log 89ac5c3cc..7744caa3a --date=short --no-merges --format='%ad %ae %s'

BUG=605972, 606052 , 606706 

TBR=catapult-sheriff@chromium.org

Review URL: https://codereview.chromium.org/1923583002

Cr-Commit-Position: refs/heads/master@{#389872}

[modify] https://crrev.com/26ef26441a318284b93d7522239aa2ae29e38597/DEPS

Comment 6 by dtu@chromium.org, Apr 28 2016

Status: Fixed (was: Started)
Components: Speed>Bisection

Sign in to add a comment