search

dynamicslab / pysindy

A package for the sparse identification of nonlinear dynamical systems from data
https://pysindy.readthedocs.io/en/latest/
Other
1.42k stars 310 forks source link

Hotfix to enable intersphinx mapping to sklearn #363

Closed Jacob-Stevens-Haas closed 1 year ago

Jacob-Stevens-Haas commented 1 year ago

Sphinx reads the public methods of all classes, and if one inherits from sklearn, it reads that method's docstring. If that method contains a relative link in the rst, sphinx needs to know where to link it.

This will fix the current CI failure, new as of scikit-learn 1.3. I'm going to give this a suspense time of a day in case anyone wants to read it, but its a hotfix blocking any PR's CI so I want to move it fast.

Jacob-Stevens-Haas commented 1 year ago

There appear to be more failures from scikit-learn-1.3.0, released June 30th, dealing with sparse matrices.