Skip to content

Show % identical on crater#1673

Closed
rsheeter wants to merge 1 commit into
mainfrom
crhtml
Closed

Show % identical on crater#1673
rsheeter wants to merge 1 commit into
mainfrom
crhtml

Conversation

@rsheeter

@rsheeter rsheeter commented Oct 2, 2025

Copy link
Copy Markdown
Contributor

Show % identical, combine identical/target into one column. Makes the number driving toward 100 a little easier to see, particularly in the face of a volatile denominator.

image

JMM

@cmyr

cmyr commented Oct 2, 2025

Copy link
Copy Markdown
Member

Is your main interest in wanting to just see the identical %? I do find the combined column a bit information dense, and it does make it harder to see the changes in total targets.

Does #1674 give you the information you need?

@rsheeter

rsheeter commented Oct 2, 2025

Copy link
Copy Markdown
Contributor Author

Either works, perhaps unsurprisingly I prefer my own formatting. I think you should pick a winner, merge one and close the other. Both are an improvement over what we have.

I do find the combined column a bit information dense, and it does make it harder to see the changes in total targets.

fwiw I have almost exactly the reverse reaction, I can get the information much faster in the denser presentation. That said, I don't know if that generalizes.

Does #1674 give you the information you need?

Strictly speaking yes, but at least for me it's harder to read and also isn't in the format I want to copy/paste into things like status updates.

@cmyr cmyr closed this Oct 2, 2025
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