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

Feature/explainability comparison #103

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

seemab24
Copy link

This Pull Request adds the following features:

  • Scaled the training and test data using StandardScaler.
  • Added SHAP and LIME comparison functionalities for model explainability.
  • Updated the model training process to use the scaled data.

Please review the code and provide feedback. Let me know if any adjustments are needed!

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉 Thank you for your contribution! Your pull request has been submitted successfully. A maintainer will review it as soon as possible. We appreciate your support in making this project better

Copy link
Owner

@ombhojane ombhojane left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hey please have tis project under /examples dir. up there create a meaningful project name as /explainability_comparison. then have all your codes and notebooks in that dir. Readme file would be a plus explaining about brief about project.

Thanks for contributing, looking to merge this PR

@seemab24
Copy link
Author

Hi @ombhojane , I have updated the README and added the LIME and SHAP explanations, along with new model comparison steps. Please review the changes. Thank you!

@seemab24 seemab24 requested a review from ombhojane October 31, 2024 12:17
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.

2 participants