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

Jump to bottom

Fix Python callback. #6320

Merged

trivialfis merged 2 commits into dmlc:master from trivialfis:fix-callback

Oct 29, 2020

Member

trivialfis commented Oct 29, 2020 •

edited

Copy the user provided callback parameter. Python is pass by reference, manipulating parameter directly has unexpected result.

Close #6319 .


          Fix Python callback.

08316da

hcho3 approved these changes

View reviewed changes


          Linter is happy.

d5c3feb

trivialfis merged commit 6ff331b into dmlc:master

trivialfis deleted the fix-callback branch

October 29, 2020 21:03

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment