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 e2e tests for Housing decision making and multiclass dnn notebooks #1212

Merged
merged 11 commits into from
Feb 11, 2022

Conversation

vinuthakaranth
Copy link
Contributor

Add e2e tests for Housing decision making and multiclass dnn notebooks

Description

Add e2e tests for https://github.com/microsoft/responsible-ai-toolbox/blob/main/notebooks/responsibleaidashboard/responsibleaidashboard-housing-decision-making.ipynb and https://github.com/microsoft/responsible-ai-toolbox/blob/main/notebooks/responsibleaidashboard/responsibleaidashboard-multiclass-dnn-model-debugging.ipynb notebooks

image

Areas changed

npm packages changed:

  • responsibleai/causality
  • responsibleai/core-ui
  • responsibleai/counterfactuals
  • responsibleai/dataset-explorer
  • responsibleai/fairness
  • responsibleai/interpret
  • responsibleai/localization
  • responsibleai/mlchartlib
  • responsibleai/model-assessment

Python packages changed:

  • raiwidgets
  • responsibleai
  • erroranalysis
  • rai_core_flask

Tests

  • No new tests required.
  • New tests for the added feature are part of this PR.
  • I validated the changes manually.

Screenshots (if appropriate):

Documentation:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@codecov-commenter
Copy link

codecov-commenter commented Feb 9, 2022

Codecov Report

Merging #1212 (57001af) into main (21a5657) will decrease coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1212      +/-   ##
==========================================
- Coverage   67.19%   67.17%   -0.03%     
==========================================
  Files          91       91              
  Lines        4393     4393              
==========================================
- Hits         2952     2951       -1     
- Misses       1441     1442       +1     
Flag Coverage Δ
unittests 67.17% <ø> (-0.03%) ⬇️

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

Impacted Files Coverage Δ
...onsibleai/responsibleai/managers/causal_manager.py 95.41% <0.00%> (-0.77%) ⬇️

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 21a5657...57001af. Read the comment docs.

@vinuthakaranth vinuthakaranth enabled auto-merge (squash) February 10, 2022 23:16
@vinuthakaranth vinuthakaranth merged commit f8e5d5a into main Feb 11, 2022
@vinuthakaranth vinuthakaranth deleted the vikarant/addmoreTestsToNotebook branch February 11, 2022 20:26
gaugup pushed a commit that referenced this pull request Feb 27, 2022
#1212)

* add tests

* update

* add tests

* lintfix

* fix

* fix
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.

3 participants