Closed y0z closed 5 months ago
@keisuke-umezawa Could you review this PR?
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 69.60%. Comparing base (
f8650a8
) to head (e82d3b9
). Report is 1 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Thank you for your comment. I've fixed this.
Also, I found a minor difference of UI. Could you fix it?
LGTM! It also worked in my env. Could you resolve conflicts?
The conflicts have been resolved :)
Contributor License Agreement
This repository (
optuna-dashboard
) and Goptuna share common code. This pull request may therefore be ported to Goptuna. Make sure that you understand the consequences concerning licenses and check the box below if you accept the term before creating this pull request.What does this implement/fix? Explain your changes.
It is known that the Optuna dashboard seriously slows down to show the list of trials when the number of trials is too large (e.g., N > 100,000). This PR virtualizes the list of trials to accelerate rendering in the
dashboard/studies/:id/trials
page.react-virtual
.