-
Notifications
You must be signed in to change notification settings - Fork 1
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
Quiver timeline - add sorting #46
Comments
Regarding filtering by year: each gt has a
-> which one should be considered "older"?
But some people might consider the 2nd one of the second example to be older. |
Thank you for the explanation! |
Should only the average of the latest runs be used for filtering by metric or the average of all runs? I assume only the latest runs but want to be sure. |
Yes, only the latest run. And consider only the average (this might noch be optimal, but for this view fine enough; there is still the table view for more specific sorting [coming soon]). |
Okay, thanks! I think ideally the sorting should only consider the average of all filtered workflows, but for this to be consistend with the average timeline chart the average timeline chart should also only consider the filtered workflows. So this switch might be better as a seperate task. For now I will implement it to consider the average of the latest runs of all workflows. |
This is almost ready for pull request, i just need to wait for #81 to be merged so i can use these changes. |
Add sorting
The text was updated successfully, but these errors were encountered: