From 4f93e5586a421713614680710a7e78eeb4a7507a Mon Sep 17 00:00:00 2001 From: david-cortes Date: Tue, 21 Sep 2021 05:48:11 +0300 Subject: [PATCH] Improve wording for warning (#7248) This warning sounds a bit ungrammatical. Additionally, the second part of the warning is not clear. This PR changes the wording to make it clearer. --- src/learner.cc | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/learner.cc b/src/learner.cc index 59eaea3c6..8fd19091d 100644 --- a/src/learner.cc +++ b/src/learner.cc @@ -568,9 +568,9 @@ class LearnerConfiguration : public Learner { ss << "\"" << diff.back() << "\""; ss << R"W( } might not be used. - This may not be accurate due to some parameters are only used in language bindings but - passed down to XGBoost core. Or some parameters are not used but slip through this - verification. Please open an issue if you find above cases. + This could be a false alarm, with some parameters getting used by language bindings but + then being mistakenly passed down to XGBoost core, or some parameter actually being used + but getting flagged wrongly here. Please open an issue if you find any such cases. )W"; LOG(WARNING) << ss.str();