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

Individual feature importance interpret QA - part1 #2167

Merged
merged 66 commits into from
Jul 10, 2023

Conversation

vinuthakaranth
Copy link
Contributor

@vinuthakaranth vinuthakaranth commented Jul 5, 2023

Description

Individual feature importance interpret QA - part1

image

Initial PR was from master student Yuanjian Gao. I have updated to use isQA prop and made a few other changes.
#2146

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.

JarvisG495 and others added 30 commits June 5, 2023 14:12
temporally use a dropdown to select token
temporally using hardcode isQA flag as the prop for identifying qa from an upper level is TBD
fix left textbox importance data and add description for qa interpret
* changing dropdown to include class name

* prettier
…ock gates (#2106)

* Port over logic of `load_adult_income_dataset` from `dice-ml` to unblock gates

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

* Fix linting and add TODO

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

* Add fixture for dataset fetch

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

---------

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

* fix error when dropped_features contains cat_features

* fix lint

* add tests

* add tests

* fix UT error
)

* Update rai_test_utils in erroranalysis and responsibleai to 0.3.0

* Update requirements-dev.txt
Signed-off-by: vinutha karanth <[email protected]>
…on (#2097)

* bbox vott template ckpt

* canvastools image load ckpt

* loading image from scratch ckpt

* canvastools image loading support

* coordinate fixes

* regiondata call + path fix

* code cleanup

* image loading ckpt

* callback image loading support

* lint fixes

* disabling internal imports for vott

* file refactor lint fix

* auto lint fixes

* lint fixes

* image dims arg for frontend bbox

* lint fixes

* lint fixes

* image dimension support

* canvas module added

* comment fix

* async image loading support

* lint fixes

* lint fixes

* max-lines lint fixes

* lint fixes
…ools on Flyout (#2116)

* bbox vott template ckpt

* canvastools image load ckpt

* loading image from scratch ckpt

* canvastools image loading support

* coordinate fixes

* regiondata call + path fix

* code cleanup

* image loading ckpt

* callback image loading support

* lint fixes

* bounding box working ckpt for fridge

* disabling internal imports for vott

* freeze rough ckpt

* file refactor lint fix

* auto lint fixes

* freeze cleanup

* lint fixes

* image dims arg for frontend bbox

* lint fixes

* lint fixes

* image dimension support

* imagedim support comments

* image dimension support

* image dimension support

* canvas module added

* image dimension working ckpt

* comment fix

* cleanup

* async image loading support

* lint fixes

* lint fixes

* reverted max line override

* code cleanups

* lint fixes

* auto lint fixes

* comment fixes
* add error analysis for QA model
Added regex based question type feature extractor

---------

Co-authored-by: Jarvis Gao <[email protected]>
Co-authored-by: Jarvis Gao <[email protected]>
Signed-off-by: vinutha karanth <[email protected]>
1 similar comment
@codecov-commenter
Copy link

codecov-commenter commented Jul 5, 2023

Codecov Report

Merging #2167 (ee0353f) into main (d5643ae) will decrease coverage by 9.92%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #2167      +/-   ##
==========================================
- Coverage   88.09%   78.18%   -9.92%     
==========================================
  Files         129       26     -103     
  Lines        7277     2136    -5141     
==========================================
- Hits         6411     1670    -4741     
+ Misses        866      466     -400     
Flag Coverage Δ
unittests 78.18% <100.00%> (-9.92%) ⬇️

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

Impacted Files Coverage Δ
...t/responsibleai_text/managers/explainer_manager.py 91.23% <100.00%> (+0.09%) ⬆️

... and 103 files with indirect coverage changes

@vinuthakaranth vinuthakaranth enabled auto-merge (squash) July 6, 2023 21:48
@imatiach-msft imatiach-msft merged commit 5828c21 into main Jul 10, 2023
91 checks passed
@imatiach-msft imatiach-msft deleted the vikarant/textUpdatesQA branch July 10, 2023 18:36
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.