Skip to content
This repository has been archived by the owner on Mar 8, 2022. It is now read-only.

Use double math until CSV output time in Interop 2022 metrics #90

Closed
wants to merge 1 commit into from

Conversation

foolip
Copy link
Collaborator

@foolip foolip commented Feb 21, 2022

No description provided.

@foolip
Copy link
Collaborator Author

foolip commented Feb 21, 2022

This will undo the 0.1% changes to the output caused by #88, but it's not a plain revert since lots of comments are updated.

@jgraham @gsnedders would you like to review? I personally prefer having 0-1000 integers at every level, from here and in the frontend. It increases the accumulated error, but is easy to understand.

Aside: I think that if we add support in wpt.fyi for showing percentage scores at test and directory levels, we wouldn't want to put doubles on the wire because of the size, it's more likely we'd work with 0-100 or 0-1000 integers there.

@foolip
Copy link
Collaborator Author

foolip commented Mar 8, 2022

I have moved this to web-platform-tests/results-analysis#67 ahead of archiving this repo.

@foolip foolip closed this Mar 8, 2022
@foolip foolip deleted the interop-2022-double branch March 8, 2022 12:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant