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 batch size default values in the tuning guide #2207

Merged
merged 1 commit into from
Apr 21, 2021

Conversation

rongou
Copy link
Collaborator

@rongou rongou commented Apr 20, 2021

Signed-off-by: Rong Ou rong.ou@gmail.com

All these values default to Integer.MAX_VALUE, which is 2147483647, not 2147483648.

@rongou rongou added bug Something isn't working documentation Improvements or additions to documentation labels Apr 20, 2021
@rongou rongou requested a review from jlowe April 20, 2021 19:24
@rongou rongou self-assigned this Apr 20, 2021
jlowe
jlowe previously approved these changes Apr 20, 2021
Copy link
Member

@jlowe jlowe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems like a bugfix appropriate for 0.5 but this is targeted to 0.6.

@rongou rongou changed the base branch from branch-0.6 to branch-0.5 April 20, 2021 19:28
@rongou rongou dismissed jlowe’s stale review April 20, 2021 19:28

The base branch was changed.

Signed-off-by: Rong Ou <rong.ou@gmail.com>
@jlowe
Copy link
Member

jlowe commented Apr 20, 2021

build

@revans2 revans2 merged commit d01f283 into NVIDIA:branch-0.5 Apr 21, 2021
@rongou rongou deleted the fix-tuning-guide-batch-sizes branch May 11, 2021 17:49
nartal1 pushed a commit to nartal1/spark-rapids that referenced this pull request Jun 9, 2021
Signed-off-by: Rong Ou <rong.ou@gmail.com>
nartal1 pushed a commit to nartal1/spark-rapids that referenced this pull request Jun 9, 2021
Signed-off-by: Rong Ou <rong.ou@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants