-
Notifications
You must be signed in to change notification settings - Fork 87
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
Reword message when errors are returned from data checks in search #982
Conversation
Codecov Report
@@ Coverage Diff @@
## main #982 +/- ##
==========================================
+ Coverage 99.66% 99.85% +0.19%
==========================================
Files 171 179 +8
Lines 8771 9283 +512
==========================================
+ Hits 8742 9270 +528
+ Misses 29 13 -16
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@angela97lin Looks good to me!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
@@ -350,7 +350,7 @@ def search(self, X, y, data_checks="auto", feature_types=None, show_iteration_pl | |||
elif message.message_type == DataCheckMessageType.ERROR: | |||
logger.error(message) | |||
if any([message.message_type == DataCheckMessageType.ERROR for message in self._data_check_results]): | |||
raise ValueError("Data checks raised some warnings and/or errors. Please see `self.data_check_results` for more information or pass data_checks=EmptyDataChecks() to search() to disable data checking.") | |||
raise ValueError("Data checks raised some warnings and/or errors. Please see `self.data_check_results` for more information or pass data_checks='disabled' to search() to disable data checking.") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Rewords the error message that is returned when a user hits an error during
search
due to data checks per our new API :)