Skip to content
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

Add plot at BinningProcess level #320

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bmreiniger
Copy link
Contributor

@bmreiniger bmreiniger commented Jun 10, 2024

Closes #315

See the three new png test results, and the last cell of the modified doc/...FICO... notebook.

This draft adds the multi-plot to BinningProcess, and modifies the BinningTable.plot method to suit. We need to add the same sort of thing to the other single-var binners, ContinuousBinningTable.plot and MulticlassBinningTable, and reconcile any differences in how those three operate.

I'm not so familiar with the multidimensional, piecewise, or streaming binners; maybe something needs/should be done there too.

Other TODOs:

  • write something to select the subplots grid shape; there's probably something we can borrow from pandas for this
  • I'd really like to be able to share the y-axis of the response aggregates (WoE etc.); this would surface a quick feature importance.

(I wonder whether we can refactor a bit to make the various flavors share more code, and make this easier?...)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
1 participant