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.
[x] I agree this patch may be ported to Goptuna by other Goptuna contributors.
Reference Issues/PRs
Continued from #851
What does this implement/fix? Explain your changes.
As the continuation of #851, separated UI components into @optuna/react to make codes common
In this PR, the following components were moved:
PlotImportance
PlotIntermediateValues
Tests for components above are also added.
Add importance prop in PlotImportance so that importance can be calculated outside
Because if @optuna/react has a dependency for wasm, it would destroy the merit that who developing the main body of dashboard did not have to build the rust env.
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.Reference Issues/PRs
Continued from #851
What does this implement/fix? Explain your changes.
@optuna/react
to make codes commonPlotImportance
PlotIntermediateValues
importance
prop inPlotImportance
so that importance can be calculated outside@optuna/react
has a dependency for wasm, it would destroy the merit that who developing the main body of dashboard did not have to build the rust env.