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

merge postga branch into main #1815

Merged
merged 32 commits into from
Nov 18, 2022
Merged

merge postga branch into main #1815

merged 32 commits into from
Nov 18, 2022

Conversation

gaugup
Copy link
Contributor

@gaugup gaugup commented Nov 17, 2022

Description

merge postga branch into main

Checklist

  • I have added screenshots above for all UI changes.
  • I have added e2e tests for all UI changes.
  • Documentation was updated if it was needed.

gaugup and others added 30 commits September 29, 2022 16:27
Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>
* Add model wrapper for wrapping predictions and test data

Signed-off-by: Gaurav Gupta <[email protected]>

* Fix failing tests

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>
…ortance (#1770)

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>
* Port tests

Signed-off-by: Gaurav Gupta <[email protected]>

* Fix rai_insights

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>
* Simply tests in test_cohort_filter.py

Signed-off-by: Gaurav Gupta <[email protected]>

* Change test name

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>
* Move cohort re-labelling logic to Cohort.ts

Signed-off-by: Gaurav Gupta <[email protected]>

* Lint fix

Signed-off-by: Gaurav Gupta <[email protected]>

* Lint fix

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>
…nceView (#1775)

* Refactor DataAnalysisTab into Chartview/DataAnalysisView and DataBalanceView

Signed-off-by: Gaurav Gupta <[email protected]>

* Fix lint

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>
… highcharts (#1783)

* enable csv xls download in hightcharts

* localization
* Add rai_test_utils package

Signed-off-by: Gaurav Gupta <[email protected]>

* Lint fix and namespace changes

Signed-off-by: Gaurav Gupta <[email protected]>

* Fix duplicated cohorts not retaining the selection in combobox (#1785)

* fix and e2e

Signed-off-by: vinutha karanth <[email protected]>

* update filter

Signed-off-by: vinutha karanth <[email protected]>

Signed-off-by: vinutha karanth <[email protected]>

* release raiwidgets and responsibleai 0.23.0 (#1787)

* fix release github action for raiwidgets and responsibleai by making it same as in gated build (#1788)

* Add e2e for dependency plot Avg of abs value option (#1768)

* fix dep plot

Signed-off-by: vinutha karanth <[email protected]>

* add e2e

Signed-off-by: vinutha karanth <[email protected]>

* lintfix

Signed-off-by: vinutha karanth <[email protected]>

* lintfix

Signed-off-by: vinutha karanth <[email protected]>

Signed-off-by: vinutha karanth <[email protected]>

* update e2e visit function (#1792)

* Fix code review comments

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>
Signed-off-by: vinutha karanth <[email protected]>
Co-authored-by: Vinutha Karanth <[email protected]>
Co-authored-by: Ilya Matiach <[email protected]>
Co-authored-by: tongy-msft <[email protected]>
Signed-off-by: vinutha karanth <[email protected]>

Signed-off-by: vinutha karanth <[email protected]>
…portance, counterfactual and casual (#1802)

* update checkbox to toggle in axis config panel

* support log toggle for counterfactual

* enable log scaling for feature importance

* enable log scaling for casual and data explorer

* address comment
…s not have any title or aria-label (#1806)

* add aria label

Signed-off-by: vinutha karanth <[email protected]>

* address comment

Signed-off-by: vinutha karanth <[email protected]>

Signed-off-by: vinutha karanth <[email protected]>
* Update erroranalysis tests to use new rai_test_utils package

Signed-off-by: Gaurav Gupta <[email protected]>

* Remove shap and xgboost from test requirements

Signed-off-by: Gaurav Gupta <[email protected]>

* Pin rai_test_utils

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>
* fix tooltip color

Signed-off-by: vinutha karanth <[email protected]>

* fix theme

Signed-off-by: vinutha karanth <[email protected]>

* fix

Signed-off-by: vinutha karanth <[email protected]>

* rev

Signed-off-by: vinutha karanth <[email protected]>

Signed-off-by: vinutha karanth <[email protected]>
Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>
…1804)

* [WIP] Add APIs for indicating large data scenario in ModelAssessmentContext

Signed-off-by: Gaurav Gupta <[email protected]>

* Change API name

Signed-off-by: Gaurav Gupta <[email protected]>

* Add unit test

Signed-off-by: Gaurav Gupta <[email protected]>

* Fix lint

Signed-off-by: Gaurav Gupta <[email protected]>

* Fix lint again

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>
tongyu-microsoft and others added 2 commits November 11, 2022 15:31
* Add RAI version number to oss serialize result

* update json file namg

* add test
@codecov-commenter
Copy link

codecov-commenter commented Nov 17, 2022

Codecov Report

Merging #1815 (990e086) into main (0727d58) will increase coverage by 0.99%.
The diff coverage is 92.24%.

@@            Coverage Diff             @@
##             main    #1815      +/-   ##
==========================================
+ Coverage   87.53%   88.52%   +0.99%     
==========================================
  Files          96       96              
  Lines        5390     5472      +82     
==========================================
+ Hits         4718     4844     +126     
+ Misses        672      628      -44     
Flag Coverage Δ
unittests 88.52% <92.24%> (+0.99%) ⬆️

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

Impacted Files Coverage Δ
...is/erroranalysis/_internal/surrogate_error_tree.py 87.85% <78.57%> (-0.89%) ⬇️
...ranalysis/erroranalysis/_internal/matrix_filter.py 96.47% <85.71%> (+0.01%) ⬆️
...sibleai/responsibleai/rai_insights/rai_insights.py 93.51% <97.14%> (+14.77%) ⬆️
...ranalysis/erroranalysis/analyzer/error_analyzer.py 95.18% <100.00%> (+0.18%) ⬆️
responsibleai/responsibleai/_interfaces.py 100.00% <100.00%> (ø)
...ai/responsibleai/rai_insights/rai_base_insights.py 87.83% <100.00%> (+0.25%) ⬆️
...eai/responsibleai/managers/data_balance_manager.py 97.60% <0.00%> (+1.59%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@gaugup gaugup merged commit 6e0a391 into main Nov 18, 2022
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.

6 participants