Skip to content

Missing CI/CD and tests for contributed code in AutoML/census_income_prediction #2380

@gguuss

Description

@gguuss

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.api: automlIssues related to the AutoML API.priority: p2Moderately-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.

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions