-
-
Notifications
You must be signed in to change notification settings - Fork 143
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
Issue209 #222
Issue209 #222
Conversation
… model. This way, we can check whether a user-defined keyword (e.g., the steps in a pipeline) does not shadow a model parameter
openml/flows/sklearn_converter.py
Outdated
if isinstance(o, tuple): | ||
assert(parent_model is not None) |
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.
I think this checks should live in _extract_information_from_model
as this function only distributes objects to the right serialization functions.
tests/test_flows/test_sklearn.py
Outdated
for case in cases: | ||
self.assertRaises(PyOpenMLError, sklearn_to_flow, case) | ||
|
||
def test_illegal_parameter_names_sklearn(self): |
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.
Could you repeat this test for FeatureUnion
.
Codecov Report
@@ Coverage Diff @@
## develop #222 +/- ##
===========================================
- Coverage 88.88% 88.82% -0.07%
===========================================
Files 23 23
Lines 1800 1807 +7
===========================================
+ Hits 1600 1605 +5
- Misses 200 202 +2
Continue to review full report at Codecov.
|
Fixes #209. Prevents users from spoofing parameter names