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

Fetching global causal effects and global policy from SDK backend #1753

Merged
merged 38 commits into from
Dec 10, 2022

Conversation

gaugup
Copy link
Contributor

@gaugup gaugup commented Sep 28, 2022

Description

Currently the causal functionality shows causal effects and policy for the entire data. Selecting a cohort doesn't have an impact of the causal tab. This change adds the functionality to fetch the global causal effects and policy from the SDK backend.

Screenshot of global causal effects requested from SDK backend.
image

Screenshot of global causal policy requested from SDK backend.
image

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.

Signed-off-by: Gaurav Gupta <gaugup@microsoft.com>
Signed-off-by: Gaurav Gupta <gaugup@microsoft.com>
Signed-off-by: Gaurav Gupta <gaugup@microsoft.com>
Signed-off-by: Gaurav Gupta <gaugup@microsoft.com>
Signed-off-by: Gaurav Gupta <gaugup@microsoft.com>
Signed-off-by: Gaurav Gupta <gaugup@microsoft.com>
Signed-off-by: Gaurav Gupta <gaugup@microsoft.com>
Signed-off-by: Gaurav Gupta <gaugup@microsoft.com>
Signed-off-by: Gaurav Gupta <gaugup@microsoft.com>
Signed-off-by: Gaurav Gupta <gaugup@microsoft.com>
Signed-off-by: Gaurav Gupta <gaugup@microsoft.com>
Signed-off-by: Gaurav Gupta <gaugup@microsoft.com>
Signed-off-by: Gaurav Gupta <gaugup@microsoft.com>
Signed-off-by: Gaurav Gupta <gaugup@microsoft.com>
@codecov-commenter
Copy link

codecov-commenter commented Sep 29, 2022

Codecov Report

Merging #1753 (0f454aa) into main (86c66a4) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1753   +/-   ##
=======================================
  Coverage   89.32%   89.32%           
=======================================
  Files         118      118           
  Lines        5927     5927           
=======================================
  Hits         5294     5294           
  Misses        633      633           
Flag Coverage Δ
unittests 89.32% <ø> (ø)

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

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

gaugup and others added 2 commits October 5, 2022 12:44
* Add model wrapper for wrapping predictions and test data

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

* Fix failing tests

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

Signed-off-by: Gaurav Gupta <gaugup@microsoft.com>
@gaugup gaugup changed the base branch from postga to main November 21, 2022 18:22
@gaugup gaugup requested a review from hawestra as a code owner November 21, 2022 18:22
Signed-off-by: Gaurav Gupta <gaugup@microsoft.com>
@gaugup gaugup enabled auto-merge (squash) December 10, 2022 01:58
@gaugup gaugup merged commit 0b89381 into main Dec 10, 2022
@gaugup gaugup deleted the gaugup/GlobalCausalCohortSupport branch December 10, 2022 03:28
gaugup added a commit that referenced this pull request Jan 6, 2023
)

* Add callback for global causal effects

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

* Query SDK backend to get global causal effects

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

* Fix bug in causal serialization

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

* Adding callback for causal ploicy

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

* propagate global and local effects only

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

* Add support for getting global causal effects from SDK backend

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

* lint fix

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

* Fix lint

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

* Fix

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

* Fix bug

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

* Modify the cohort message

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

* Move to state

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

* Fix imports

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

* Remove console.log

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

* Change the if condition

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

* Add postga build trigger (#1755) (#1756)

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

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

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

* Add model wrapper for wrapping predictions and test data (#1762)

* Add model wrapper for wrapping predictions and test data

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

* Fix failing tests

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

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

* Fix lint

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

* Remove force update

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

* Fix lint

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

Signed-off-by: Gaurav Gupta <gaugup@microsoft.com>
gaugup added a commit that referenced this pull request Jan 6, 2023
* Add missing property 'id' in ICounterfactualData

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

* Update some sample notebooks to include dropped_features (#1836)

* update some notebooks to include dropped_features

* update some notebooks to include dropped_features

* update some notebooks to include dropped_features

* remove unused changes

* remove unused changes

* address comments

* upgrade setup-python github action from v2 to v4 and pin to older ubuntu image for python 3.6 to fix build error (#1838)

* Gate call to probability distribution end point (#1839)

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

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

* Revert "Update some sample notebooks to include dropped_features (#1836)" (#1837)

This reverts commit d89b03a.

* fix cf error on unable to update text (#1842)

* Disable partial dependence plots for large data scenarios (#1843)

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

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

* Remove redundant call to serialize_json_safe() in raiwidgets causal tests (#1840)

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

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

* Fetching global causal effects and global policy from SDK backend (#1753)

* Add callback for global causal effects

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

* Query SDK backend to get global causal effects

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

* Fix bug in causal serialization

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

* Adding callback for causal ploicy

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

* propagate global and local effects only

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

* Add support for getting global causal effects from SDK backend

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

* lint fix

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

* Fix lint

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

* Fix

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

* Fix bug

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

* Modify the cohort message

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

* Move to state

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

* Fix imports

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

* Remove console.log

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

* Change the if condition

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

* Add postga build trigger (#1755) (#1756)

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

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

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

* Add model wrapper for wrapping predictions and test data (#1762)

* Add model wrapper for wrapping predictions and test data

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

* Fix failing tests

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

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

* Fix lint

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

* Remove force update

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

* Fix lint

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

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

* Update requirements-linting.txt (#1847)

* Remove typo casual and correcting it to causal (#1846)

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

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

* Disable local causal chart for large data support (#1845)

* Disable local causal chart for large data support

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

* Fix lint

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

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

* add multilabel text example and fix correct and incorrect groups in data analysis (#1848)

* Add number of cohort samples inside generateMetrics() (#1849)

* Add number of cohort samples inside generateMetrics()

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

* Yarn lintfix

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

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

* add multilabel RAI vision example for fridge multilabel dataset (#1844)

* fix install command for speech to text example (#1856)

* Send the correct column name to SDK backend for model overview distribution charts (#1835)

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

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

* Derive fairness metrics from metric stats instead of cohort stats (#1854)

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

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

* add upper bounds to greenlet and gevent in rai-core-flask as 5.0 release fails until they are more stable (#1852)

* fix loading older versions of RAIInsights due to _FEATURE_METADATA key error (#1857)

* Update maintainers (#1860)

* Update requirements.txt in `raiwidgets` and `responsibleai` to pin numpy upper bound (#1864)

* Update requirements.txt

* Update requirements.txt

* Port `cohort.py` from `raiwidgets` to `raiutils` (#1859)

* Port cohort.py from raiwidgets to raiutils

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

* Add more constants

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

* Add constants.py

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

* Make constants public

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

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

* Add common APIs for model overview metrics (#1862)

* Add metric SDK APIs for Model Overview metrics

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

* remove console.log()

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

* Port `cohort.py` from `raiwidgets` to `raiutils` (#1859)

* Port cohort.py from raiwidgets to raiutils

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

* Add more constants

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

* Add constants.py

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

* Make constants public

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

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

* Fix lint

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

* lint fix

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

* Fix lint

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

* Fix lint

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

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

* Update requirements-linting.txt (#1861)

* update raiwidgets to rai-core-flask v0.5.0 (#1851)

* fix error in erroranalysis heatmap caused by max value over right side of max bin (#1866)

* remove dead space in aggregate feature importance (#1869)

* display tree branching decisions (#1870)

* Remove warning for 'dropped_features are not in use currently.' (#1871)

* display tree branching decisions

* Remove warning for 'dropped_features are not in use currently.'

* turn off fail-fast for python tests (#1878)

* add multilabel model overview metrics (#1882)

* Show branching decisions on hover over on unselected tree nodes (#1881)

* show tree label on hover over

* address comments

* Update some sample notebooks to include dropped_features (#1836)

* update some notebooks to include dropped_features

* update some notebooks to include dropped_features

* update some notebooks to include dropped_features

* remove unused changes

* remove unused changes

* address comments

* Revert "Update some sample notebooks to include dropped_features (#1836)" (#1837)

This reverts commit d89b03a.

* Update requirements-linting.txt (#1847)

* Add number of cohort samples inside generateMetrics() (#1849)

* Add number of cohort samples inside generateMetrics()

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

* Yarn lintfix

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

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

* Update requirements-linting.txt (#1861)

* add multilabel model overview metrics (#1882)

* Update `raiutils` in `raiwidgets`, `responsibleai`, `erroranalysis` and `nlp_feature_extractors` (#1873)

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

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

* Fix test

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

* Undo changes

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

* Fix lint

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

Signed-off-by: Gaurav Gupta <gaugup@microsoft.com>
Co-authored-by: tongy-msft <91754176+tongyu-microsoft@users.noreply.github.com>
Co-authored-by: Ilya Matiach <ilmat@microsoft.com>
Co-authored-by: Roman Lutz <romanlutz13@gmail.com>
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.

4 participants