Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Replace boston dataset in Torch unit tests #19

Merged
merged 1 commit into from
May 10, 2023
Merged

Replace boston dataset in Torch unit tests #19

merged 1 commit into from
May 10, 2023

Conversation

connortann
Copy link
Collaborator

@connortann connortann commented May 10, 2023

Replaces the use of the Boston dataset in some Torch unit tests. These tests don't seem to rely on any particular feature of the dataset used, so switching the dataset should be safe.

Should help fix the test suite as per #4. Does not address the use of the Boston dataset in SHAP examples, as per #8.

Copy link
Owner

@dsgibbons dsgibbons left a comment

Choose a reason for hiding this comment

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

Looks good - thank you!

@dsgibbons dsgibbons merged commit c1d68b7 into dsgibbons:master May 10, 2023
@connortann connortann deleted the fix/boston branch May 10, 2023 13:04
@connortann connortann mentioned this pull request May 11, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants