Replace outdated stumpy.mplstyle URL in notebooks#1141
Conversation
|
Found 17 changed notebooks. Review the changes at https://app.gitnotebooks.com/stumpy-dev/stumpy/pull/1141 |
|
@strangertomycode Thank you for your PR. It appears that there are big changes in several of the notebooks that make it impossible to review. However, the issue should simply require opening the |
c40828c to
de5c128
Compare
|
@seanlaw Thanks for the feedback. I've cleaned up the notebook diffs and updated the PR so that it only contains the requested |
seanlaw
left a comment
There was a problem hiding this comment.
@strangertomycode Thank you for making the changes. They all look great!
I was wondering if you would be willing to do the same thing and replace all references of:
Fixes #1139
Summary
Replaced outdated
stumpy.mplstyleURL references in the listed notebooks:From:
https://raw.githubusercontent.com/TDAmeritrade/stumpy/main/docs/stumpy.mplstyleTo:
https://raw.githubusercontent.com/stumpy-dev/stumpy/main/docs/stumpy.mplstyleVerification
blackflake8pytest-covblack --exclude=".*\.ipynb" --extend-exclude=".venv" --diff ./flake8 --extend-exclude=.venv ./Notes
./setup.sh dev && ./test.shwas attempted. Black, isort, and flake8 checks completed successfully. The test suite could not be completed locally due to missing optional test dependencies (tornado, followed bydask.distributed) during test collection. No source code functionality was modified; this PR only updates notebook URL references.