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 missing explainers to Explainers enum #34

Merged
merged 1 commit into from
Jan 10, 2023
Merged

Conversation

adamwawrzynski
Copy link
Collaborator

@adamwawrzynski adamwawrzynski commented Jan 10, 2023

Description

This PR adds missing explainer classes to Explainers enum class to enable thier usage in context manager AutoXaiExplainer.

How Has This Been Tested?

Please describe the tests that you ran to verify your changes.

  • Test A

Checklist:

  • I have updated all related pyproject.toml files.
  • I have updated lock files.
  • I have set an Assignee and Reviewers.
  • I have labeled the PR correctly.
  • I have performed a self-review of my own code.
  • I have covered my code and changes with unit tests.
  • I have annotated types as extensively as possible.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no new warnings.
  • All code references and dependencies will work.

Signed-off-by: Adam Wawrzyński <adam.wawrzynski@reasonfieldlab.com>
@adamwawrzynski adamwawrzynski self-assigned this Jan 10, 2023
@adamwawrzynski adamwawrzynski added the bug Something isn't working label Jan 10, 2023
@adamwawrzynski adamwawrzynski merged commit e3f8dec into main Jan 10, 2023
@kamilrzechowski kamilrzechowski deleted the fix/missing_enums branch January 10, 2023 11:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants