-
Notifications
You must be signed in to change notification settings - Fork 6.6k
Closed
Labels
🚨This issue needs some love.This issue needs some love.api: automlIssues related to the AutoML API.Issues related to the AutoML API.priority: p2Moderately-important priority. Fix may not be included in next release.Moderately-important priority. Fix may not be included in next release.type: bugError or flaw in code with unintended results or allowing sub-optimal usage patterns.Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Description
It looks like someone is circumventing our style checks and avoiding contributing tests by putting code in Jupyter notebooks.
See this PR for additional details regarding the missing tests for AutoML/census_income_prediction. For all functions within the code, there should be an accompanying test and the contributed Python should meet the style standards for our repo.
There are definitely errors in the code that would be caught by lint, for example, most of the code uses 2 spaces where there should be 4 for indentations. Another example, on lines 288-289, it looks like the import order is wrong. There are probably a lot more but it's difficult to tell with the code obfuscated within Jupyter markdown.
Metadata
Metadata
Assignees
Labels
🚨This issue needs some love.This issue needs some love.api: automlIssues related to the AutoML API.Issues related to the AutoML API.priority: p2Moderately-important priority. Fix may not be included in next release.Moderately-important priority. Fix may not be included in next release.type: bugError or flaw in code with unintended results or allowing sub-optimal usage patterns.Error or flaw in code with unintended results or allowing sub-optimal usage patterns.