alteryx / alteryx/evalml

Investigate better way to make integer grid values for partial dependence

Open
#4,100 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
850
Forks
96
PR merge metrics
No merged PRs in 30d

Description

With https://github.com/alteryx/evalml/issues/4095, we made the handling of integer grid values for partial dependence explicit, rounding the fractional values produced in `_grid_from_X` to allow IntegerNullable types to be used for partial dependence.

One downside of this method for determining grid values is that it's possible for us to end up with duplicate grid values if, say, two grid values were calculated to be `4.1` and `4.4` and then both got rounded down to `4`. We should investigate if there is a better way to get integer grid values for partial dependence calculations or if there is a way for us to maintain the fractional grid values and pass those into our pipelines (this is currently not possible because of our `_schema_is_equal` check when transforming on new data).

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.