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

docs: Updated ICE/PDP doc with PDP-based feature importance #1441

Merged
merged 13 commits into from
Mar 16, 2022

Conversation

ezherdeva
Copy link
Contributor

Summary

Added documentation about PDP-based feature importance option for kind == feature. Added API calls and visualizations.

Tests

N/A

Dependency chances

N/A

@yalaudah
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@codecov-commenter
Copy link

codecov-commenter commented Mar 15, 2022

Codecov Report

Merging #1441 (b87bc8f) into master (a1781d2) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1441   +/-   ##
=======================================
  Coverage   84.65%   84.65%           
=======================================
  Files         288      288           
  Lines       14335    14335           
  Branches      701      701           
=======================================
  Hits        12135    12135           
  Misses       2200     2200           
Impacted Files Coverage Δ
...crosoft/azure/synapse/ml/io/http/HTTPClients.scala 68.18% <0.00%> (-3.04%) ⬇️
.../execution/streaming/continuous/HTTPSourceV2.scala 92.80% <0.00%> (+0.71%) ⬆️

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 a1781d2...b87bc8f. Read the comment docs.

@mhamilton723
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Collaborator

@mhamilton723 mhamilton723 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hit you with a few suggestions! Awesome work :)

ezherdeva and others added 7 commits March 15, 2022 21:27
…CE explainer.ipynb

Co-authored-by: Mark Hamilton <mhamilton723@gmail.com>
…CE explainer.ipynb

Co-authored-by: Mark Hamilton <mhamilton723@gmail.com>
…CE explainer.ipynb

Co-authored-by: Mark Hamilton <mhamilton723@gmail.com>
…CE explainer.ipynb

Co-authored-by: Mark Hamilton <mhamilton723@gmail.com>
…CE explainer.ipynb

Co-authored-by: Mark Hamilton <mhamilton723@gmail.com>
@ezherdeva
Copy link
Contributor Author

Thank you @memoryz and @mhamilton723 for the review! Sorry for my bad phrasing and grammar :)

@mhamilton723 mhamilton723 enabled auto-merge (squash) March 16, 2022 15:46
@memoryz
Copy link
Contributor

memoryz commented Mar 16, 2022

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@memoryz
Copy link
Contributor

memoryz commented Mar 16, 2022

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mhamilton723 mhamilton723 merged commit be81c20 into master Mar 16, 2022
@mhamilton723 mhamilton723 deleted the ezherdeva/pdp-based-feature-imp-docs branch March 16, 2022 19:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants