Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Basic Analytics Tools #132

Merged

Conversation

Doppe1g4nger
Copy link
Contributor

This PR adds a functioning Analytics object that can take a fit MatPipe object and perform operations for interpreting, visualizing, and using the produced model. Currently only supports model agnostic feature importance and single feature partial dependency plotting.

@ardunn
Copy link
Contributor

ardunn commented Nov 18, 2018

one quick comment: when making plot methods, don’t save the fig for the time being, return the plot object. then the user can decide what to do with the plot object (see pymatgen plotting)

overall i’m very stoked on this tho

@codecov
Copy link

codecov bot commented Nov 18, 2018

Codecov Report

Merging #132 into master will increase coverage by 2.81%.
The diff coverage is 27.5%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #132      +/-   ##
==========================================
+ Coverage   66.79%   69.61%   +2.81%     
==========================================
  Files          23       23              
  Lines        1262     1211      -51     
==========================================
  Hits          843      843              
+ Misses        419      368      -51
Impacted Files Coverage Δ
mslearn/analytics/core.py 27.5% <27.5%> (+15.41%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 59f54e3...cb44351. Read the comment docs.

@ardunn
Copy link
Contributor

ardunn commented Nov 18, 2018

btw don’t worry about failing codecov check

@Doppe1g4nger Doppe1g4nger merged commit 2556689 into hackingmaterials:master Nov 18, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants