Fix randint ValueError in windows

[hrntsm]

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.

• I agree this patch may be ported to Goptuna by other Goptuna contributors.

Reference Issues/PRs

What does this implement/fix? Explain your changes.

When I run PreferentialOptimization in a windows environment, I get the following error in the randint function.

Traceback (most recent call last):
  File "c:\Users\hrntsm\Desktop\potest\main.py", line 60, in <module>
    main()
  File "c:\Users\hrntsm\Desktop\potest\main.py", line 27, in main
    sampler=PreferentialGPSampler(),
  File "C:\Users\hrntsm\Desktop\potest\.venv\lib\site-packages\optuna_dashboard\preferential\samplers\gp.py", line 320, in __init__
    seed=self._rng.randint(2**32)
  File "numpy\\random\\mtrand.pyx", line 779, in numpy.random.mtrand.RandomState.randint
  File "numpy\\random\\_bounded_integers.pyx", line 2881, in numpy.random._bounded_integers._rand_int32
ValueError: high is out of bounds for int32

Fix this error by specifying the dtype explicitly.

env

• windows11
• python3.9
• optuna3.3.0
• optuna-dashboard0.13.0

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (a801b89) 63.77% compared to head (bf7c930) 63.77%.
Report is 3 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #711   +/-   ##
=======================================
  Coverage   63.77%   63.77%           
=======================================
  Files          35       35           
  Lines        2280     2280           
=======================================
  Hits         1454     1454           
  Misses        826      826           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[not522]

Thank you for pointing it out, LGTM!