Skip to content
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

Fix invalid param value "use_gpu":"true" #383

Merged
merged 1 commit into from
May 22, 2024

Conversation

NvTimLiu
Copy link
Collaborator

@NvTimLiu NvTimLiu commented May 21, 2024

After installing latest Jupyter notebooke, the json format check is strick, due to passing a string value "true"

for the "use_gpu":"true", "use_gpu": True is required.

params = {
    "tree_method": "gpu_hist",
    "grow_policy": "depthwise",
    "num_workers": 1,
    "use_gpu": "true", -->  "device": "cuda"
}

TypeError: Invalid param value given for param "use_gpu".

Boolean Param requires value of type bool. Found <class 'str'>.

More over, "use_gpu": True is deprecated in dmlc/XGBoost, change to user "device": "cuda" instead

After installing latest Jupyter notebooke, the json format check is strick, due to passing a string value "true"

for the "use_gpu":"true", "use_gpu": True is required.

```
params = {
    "tree_method": "gpu_hist",
    "grow_policy": "depthwise",
    "num_workers": 1,
    "use_gpu": "true",
}
```
TypeError: Invalid param value given for param "use_gpu".

Boolean Param requires value of type bool. Found <class 'str'>.

More over, "use_gpu": True is deprecated in dmlc/XGBoost, change to user "device": "cuda" instead

Signed-off-by: Tim Liu <[email protected]>
@NvTimLiu NvTimLiu self-assigned this May 21, 2024
@NvTimLiu NvTimLiu added the enhancement New feature or request label May 21, 2024
@NvTimLiu NvTimLiu requested review from wbo4958 and nvliyuan May 21, 2024 12:40
@NvTimLiu NvTimLiu merged commit 3627263 into NVIDIA:branch-24.06 May 22, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants