EconML icon indicating copy to clipboard operation
EconML copied to clipboard

Adapt default values in docstrings to supported format

Open kbattocchi opened this issue 5 years ago • 0 comments

Sphinx 3.2 checks default argument specifications more stringently (see e.g. https://github.com/sphinx-doc/sphinx/issues/8088), so the build was failing with that version of Sphinx. These changes should address the issue, except that our classes that derive from classes in external libraries can still have problems with inherited method docstrings (e.g. StatsModelsLinearRegression), so we either need to wait for sklearn to update their own classes or for Sphinx to relax the new restrictions.

kbattocchi avatar Aug 10 '20 18:08 kbattocchi