Robert Parini

LGTM.com - false positive for py/uninitialized-local-variable

3

**Description of the false positive** The error says that a variable may be used before it is initialized but the variable is defined on the line above. I cannot see...

Empty response body causes error or incorrect content-type inference/validation

4

### Description If an endpoint returns a response with no body without specifying the content-type and the API spec for the endpoint has other, non-empty, responses as options then Connexion...

Derivative with numpy 1.25.2 errors with complex valued function

2

This happens on the latest version of numpy (1.25.2), I guess something changed about the `percentile` function that numdifftools is using. It works fine on numpy 1.24.4. ```python >>> import...

The current method of building wheels is `python setup.py sdist bdist_wheel` but gives ``` /opt/hostedtoolcache/Python/3.10.6/x64/lib/python3.10/site-packages/setuptools/installer.py:27: SetuptoolsDeprecationWarning: setuptools.installer is deprecated. Requirements should be satisfied by a PEP 517 installer. ``` and...

`count_roots` is [there](https://rparini.github.io/cxroots/contours.html#cxroots.contour.Contour.count_roots) but the parameters aren't documented. `.roots(...)` seems to be entirely missing

Eg, what [scipy](https://github.com/scipy/scipy) has:

Improving derivative-free case ------- When `df` is not provided we currently approximate it using a finite difference method https://github.com/rparini/cxroots/blob/1b0c859736f301d61c85ca2e31d3c8c5b6406157/cxroots/root_counting.py#L197-L204 Is there a better way of doing this? I feel like...

It's a very long generator with several functions defined within it so hard to read. Might want to be a class with methods instead. There are also some special cases...