-
Notifications
You must be signed in to change notification settings - Fork 2
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
Reproducibility issue of XGBoostSampler
results for Windows and Linux
#49
Comments
marcofavoritobi
pushed a commit
that referenced
this issue
Mar 18, 2023
We had to workaround issue #49 by providing different expected results for Windows.
marcofavoritobi
pushed a commit
that referenced
this issue
Mar 18, 2023
We had to workaround issue #49 by providing different expected results for Windows.
marcofavoritobi
pushed a commit
that referenced
this issue
Mar 18, 2023
We had to workaround issue #49 by providing different expected results for Windows.
marcofavoritobi
pushed a commit
that referenced
this issue
Mar 18, 2023
We had to workaround issue #49 by providing different expected results for Windows.
marcofavoritobi
pushed a commit
that referenced
this issue
Mar 18, 2023
We had to workaround issue #49 by providing different expected results for Windows.
marcofavoritobi
pushed a commit
that referenced
this issue
Mar 18, 2023
We had to workaround issue #49 by providing different expected results for Windows.
marcofavoritobi
pushed a commit
that referenced
this issue
Mar 20, 2023
We had to workaround issue #49 by providing different expected results for Windows.
marcofavoritobi
pushed a commit
that referenced
this issue
Mar 20, 2023
We had to workaround issue #49 by providing different expected results for Windows.
marcofavoritobi
pushed a commit
that referenced
this issue
Mar 20, 2023
We had to workaround issue #49 by providing different expected results for Windows.
marcofavoritobi
pushed a commit
that referenced
this issue
Mar 21, 2023
We had to workaround issue #49 by providing different expected results for Windows.
marcofavoritobi
pushed a commit
that referenced
this issue
Sep 20, 2023
To reproduce the errors: ``` ruff check --select "NPY" tests ``` This commit updates the tests code so to make them to use np.random.default_rng with a certain seed, rather than relying on global random seed handling, i.e. using np.random.seed. To do so, a new fixture, 'rng', has been added so to avoid the tests code to initialize a np.random.Generator instance manually. This allowed to remove the special case for macOS platforms in TestCalibrate.test_calibrator_calibrate tests. Fix #49.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Subject of the issue
There is a reproducibility issue for the
XGBoostSampler
across platforms. In particular, when I useXGBoostSampler
on Linux and on Windows, I get different results even ifrandom_state
is set correctly.Your environment
Steps to reproduce
Run the following script on both Windows and Ubuntu:
On Ubuntu, the value of
new_params
is:On Windows, the value of
new_params
is:Expected behaviour
I expected to get the same result.
Actual behaviour
Different result, see above.
Furhter comments
I understand that the provided details might not be enough for the exact reproduction of the bug. This issue is more a way to point out the reproducibility issue when using the
XGBoostSampler
.On the other hand, I suspect this does not depends on
black-it
, but rather on the underlying implementation of XGBoost, provided by the packagexgboost
. The following links already document reproducibility issues for that package:The text was updated successfully, but these errors were encountered: