Skip to content

Add capability to serialize and de-serialize dice-ml explainers#289

Merged
gaugup merged 7 commits intomasterfrom
gaugup/SerializeDeserializeExplainers
May 9, 2022
Merged

Add capability to serialize and de-serialize dice-ml explainers#289
gaugup merged 7 commits intomasterfrom
gaugup/SerializeDeserializeExplainers

Conversation

@gaugup
Copy link
Copy Markdown
Collaborator

@gaugup gaugup commented Apr 22, 2022

Signed-off-by: Gaurav Gupta gaugup@microsoft.com

Signed-off-by: Gaurav Gupta <gaugup@microsoft.com>
@gaugup gaugup requested a review from amit-sharma as a code owner April 22, 2022 21:50
Signed-off-by: Gaurav Gupta <gaugup@microsoft.com>
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 22, 2022

Codecov Report

Merging #289 (4e8f4c2) into master (edc5415) will increase coverage by 0.08%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #289      +/-   ##
==========================================
+ Coverage   71.26%   71.34%   +0.08%     
==========================================
  Files          26       26              
  Lines        3438     3448      +10     
==========================================
+ Hits         2450     2460      +10     
  Misses        988      988              
Flag Coverage Δ
unittests 71.34% <100.00%> (+0.08%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
dice_ml/explainer_interfaces/explainer_base.py 89.56% <100.00%> (+0.29%) ⬆️

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 edc5415...4e8f4c2. Read the comment docs.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 22, 2022

Unit Test Results

    1 files  ±0      1 suites  ±0   9m 26s ⏱️ - 1m 59s
247 tests ±0  242 ✔️ ±0  5 💤 ±0  0 ±0 

Results for commit d19a916. ± Comparison against base commit 6b35253.

♻️ This comment has been updated with latest results.

gaugup added 2 commits May 6, 2022 13:49
@gaugup gaugup changed the title [WIP] Add capability to serialize and de-serialize dice-ml explainers Add capability to serialize and de-serialize dice-ml explainers May 6, 2022
Signed-off-by: Gaurav Gupta <gaugup@microsoft.com>
Copy link
Copy Markdown
Collaborator

@amit-sharma amit-sharma left a comment

Choose a reason for hiding this comment

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

Looks good, just one minor typo in docstring.

Comment thread dice_ml/explainer_interfaces/explainer_base.py Outdated
@gaugup gaugup merged commit 5e70ef4 into master May 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants