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
fix: fix ExplanationDashboard interpretability failure #1966
base: master
Are you sure you want to change the base?
Conversation
Hey @imatiach-msft 👋! We use semantic commit messages to streamline the release process. Examples of commit messages with semantic prefixes:
To test your commit locally, please follow our guild on building from source. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Summary by GPT-4
The changes in the code are related to the installation of required packages and dependencies. In the updated version, %pip install
is used instead of os.system("pip install ...")
to install the packages. Additionally, some new packages are installed, such as raiwidgets
, itsdangerous
, interpret-community
, and protobuf
. These packages are related to responsible AI and interpretability features in SynapseML.
Suggestions
No suggestions are needed as the changes in this PR are clear and well-organized.
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
Codecov Report
@@ Coverage Diff @@
## master #1966 +/- ##
==========================================
- Coverage 87.01% 86.98% -0.04%
==========================================
Files 305 305
Lines 15993 15993
Branches 839 839
==========================================
- Hits 13917 13911 -6
- Misses 2076 2082 +6 |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
cc8b592
to
c375d85
Compare
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
5a3d2e1
to
0d19aab
Compare
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
0d19aab
to
ac05d60
Compare
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
ac05d60
to
6b0ac86
Compare
/azp run |
289d15c
to
d3bd34a
Compare
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
0599a0a
to
5bd36a0
Compare
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
5bd36a0
to
b689587
Compare
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
2184f46
to
64afbce
Compare
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
64afbce
to
0a83fee
Compare
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
Related Issues/PRs
#xxx
What changes are proposed in this pull request?
Fix the ExplanationDashboard notebook failure in e2e tests.
How is this patch tested?
Does this PR change any dependencies?
Does this PR add a new feature? If so, have you added samples on website?
website/docs/documentation
folder.Make sure you choose the correct class
estimators/transformers
and namespace.DocTable
points to correct API link.yarn run start
to make sure the website renders correctly.<!--pytest-codeblocks:cont-->
before each python code blocks to enable auto-tests for python samples.WebsiteSamplesTests
job pass in the pipeline.