Closed porink0424 closed 3 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 69.51%. Comparing base (
7baa0b3
) to head (ac20f25
). Report is 8 commits behind head on main.:exclamation: Current head ac20f25 differs from pull request most recent head de0accd. Consider uploading reports for the commit de0accd to get more accurate results
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@c-bata Thank you for your suggestions, I have modified them!
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
What does this implement/fix? Explain your changes.
vitest
+@testing-library/react
jsdom
for test environment@testing-library/jest-dom
fortoBeInTheDocument
methodtypescript-tests.yml
loadJournalStorage
in@optuna/storage
:NaN
/Infinity
values with strings when parsing fails when they cannot be parsed withJSON.parse()
JournalOperation.SET_TRIAL_STATE_VALUES
, changed implementation so that, when the value is null, it is replaced with undefined according to the original type definitionReference
@testing-library/jest-dom
:Plotly
: