-
Notifications
You must be signed in to change notification settings - Fork 514
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
[bug-fix] avoid paramsSolver::{n_rows,n_cols} shadowing their base class counterparts #4130
[bug-fix] avoid paramsSolver::{n_rows,n_cols} shadowing their base class counterparts #4130
Conversation
Signed-off-by: Yitao Li <yitao@rstudio.com>
Can one of the admins verify this patch? |
ok to test |
Codecov Report
@@ Coverage Diff @@
## branch-21.10 #4130 +/- ##
===============================================
Coverage ? 85.91%
===============================================
Files ? 232
Lines ? 18377
Branches ? 0
===============================================
Hits ? 15788
Misses ? 2589
Partials ? 0
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
@gpucibot merge |
…rparts (rapidsai#4130) This looks to me like a typo, and may be problematic and confusing if the `n_rows` and `n_cols` members from the base class instead of the ones from the derived class are accessed. Signed-off-by: Yitao Li <yitao@rstudio.com> Authors: - Yitao Li (https://github.com/yitao-li) Approvers: - Micka (https://github.com/lowener) - Dante Gama Dessavre (https://github.com/dantegd) URL: rapidsai#4130
This looks to me like a typo, and may be problematic and confusing if the
n_rows
andn_cols
members from the base class instead of the ones from the derived class are accessed.Signed-off-by: Yitao Li yitao@rstudio.com