Closed TomDonoghue closed 1 year ago
Thank you! Looks good.
Can you please make sure your original branch is up-to-date?
My fork says it's up to date with amueller:master
thank you for checking! I'm confused why the test are failing then. But this PR is clearly not the cause.
The README is a bit out of date with the current tested versions, and in the current version implies that the module still is tested on / supports Python2, which I think is no longer the case (?). This update lists the current set of tested Python versions, drawing from the Github actions file.
In addition, there is a minor update to the install section of the README to note that matplotlib is now a required dependency (it previously stated it was optional, but I believe this is outdated).