Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BUG: Number of results incorrectly showing n/a #1541

Open
DeaMariaLeon opened this issue Nov 27, 2023 · 1 comment
Open

BUG: Number of results incorrectly showing n/a #1541

DeaMariaLeon opened this issue Nov 27, 2023 · 1 comment

Comments

@DeaMariaLeon
Copy link
Contributor

DeaMariaLeon commented Nov 27, 2023

The results number is incorrect in most commits. Correctly showing the number of results in only 7 commits.

There are 7528 entries in commit 2b67593 for example, but the results number is "n/a".
These results numbers seem to be correct for a while (when the converter is posting data to the web API), but they change later to n/a.
Screenshot 2023-11-27 at 09 41 17

No results given after the first page:
Screenshot 2023-11-27 at 09 57 32

The first part of the Proof of Concept for pandas can be seen here for the time being: http://15.235.45.255:5000/
I set up the web app with the container on a server as if I were working locally. This is just temporary.
Here is my code: https://github.com/DeaMariaLeon/conbench_toy
I need to add tests etc.

@austin3dickey
Copy link
Member

Thanks, this is exciting!

Regarding the bug: this is a known limitation of populating these numbers from the server's cache of recent benchmark results:

https://github.com/conbench/conbench/issues/977. However, the BMRT cache
(built for other purpose) knows a per-run result count estimate for many
of the most recent runs. Feed that per-run result count from there. For

But I think it's still worth trying to figure out a way to display these numbers correctly.

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

No branches or pull requests

2 participants