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

Remove remaining usages of the boston dataset. #20

Merged
merged 1 commit into from
May 10, 2023

Conversation

simonangerbauer
Copy link

@simonangerbauer simonangerbauer commented May 10, 2023

There are a few remaining usages of the boston dataset in the tests that are not covered by #19.

As the california dataset is much bigger (20k samples) than the boston dataset (500 samples), I only used 500 samples of the california dataset to not increase test runtime too much (same as slundberg#2501).

Related to #4 and #8

@dsgibbons
Copy link
Owner

LGTM - thank you for your contribution!

@dsgibbons dsgibbons merged commit 1d9224e into dsgibbons:master May 10, 2023
@simonangerbauer simonangerbauer deleted the remove_boston_dataset branch May 10, 2023 12:47
# 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