DOCS: Remove boston dataset from the README

[dsgibbons]

Overview

Closes dsgibbons#8 and resolves the issues noted in dsgibbons#8 (comment)

Related to #2322

Updating the README required the following additional changes:

• Use relative paths for README images instead of URLs tied to master.
• HTML plots generated inside a notebook, rendered in Chrome, and then screenshotted. There is no obvious way to convert the HTML straight to an image, as it requires JS - unless @connortann or @thatlittleboy know of a better way to convert shap HTML to PNG? I've tried several different HTML to PNG converters, but I keep getting a shap error saying JS needs initialising.

Checklist

• All pre-commit checks pass. - no, dependabot is not happy?
• Unit tests added (if fixing a bug or adding a new feature)

[dsgibbons]

Tests are failing due to calls to shap.summary_plot. Something about constrained mode is breaking shap.summary_plot. I think shap.summary_plot is an outdated API that used to be a catchall term for the various plot types (beeswarm, violin, etc.). Is it worth me fixing the test, or should we remove shap.summary_plot as it probably shouldn't be used any more? The new plots in constrained mode all look fine in mpl.

[dsgibbons]

Never mind, I think I've fixed shap.summary_plot. It may still be worth removing shap.summary_plot any way to reduce the size of the codebase.

[dsgibbons]

Never mind, I think I've fixed shap.summary_plot. It may still be worth removing shap.summary_plot any way to reduce the size of the codebase.

Looks like this is a duplicate of #3156

[dsgibbons]

I've added a commit to update the mpl image baselines. We can revert this if references to the legacy summary_plot are removed.

[dsgibbons]

There's a discrepancy between how my setup generates the baselines and GitHub Actions. I'll need to alter my setup so that I produce the same outputs as GitHub Actions.

[dsgibbons]

I've removed the changes to SHAP's plotting functionality. I think these changes should be postponed pending the wider plotting API change proposed for 0.44 by #2363.

[thatlittleboy]

LGTM, @dsgibbons one minor comment, any reason why we're not removing the boston_xxx.png files from docs/artwork?

[dsgibbons]

LGTM, @dsgibbons one minor comment, any reason why we're not removing the boston_xxx.png files from docs/artwork?

Good point, removed!