Anders Fredrik Kiær

Results 60 comments of Anders Fredrik Kiær

I see this PR has gone through a first review by @joshmgross already. Will it be merged in the near future, or should use cases needing to skip the save...

We investigated this a bit further: - The vectors in `.keys()` but not in `EclSumKeyWordVector` ends with `:INTEGER`, where `INTEGER` is probably an integer representing total grid cell index. Corresponding...

> Another Observation (may or may not be related): When a previously selected value in a dropdown becomes invalid (not contained in the options anymore after update), it would disappear...

We see the same thing. Strangely enough the following snippet ```python #pylint: disable=missing-module-docstring,pointless-statement import pandas as pd df = pd.read_csv("some.csv") df.columns df.columns ``` gives `Instance of 'TextFileReader' has no 'columns'...

Investigated a bit further. The false positive appears to have been introduced between `astroid==2.5.7`and `astroid==2.5.8`, in https://github.com/PyCQA/astroid/pull/1009. For the specific snippet above it looks like increasing [`max_inferred`](https://github.com/PyCQA/astroid/blob/b48b370b34c07be6b41bc669f0d65cddf1bfeb89/astroid/context.py#L45) to `>=166` removes...

Based on the findings in https://github.com/PyCQA/pylint/issues/4577#issuecomment-871694490 we use the workaround below which might be useful for others here facing this issue. Basically we for now increase `astroid.context.InferenceContext.max_inferred` to a higher...

While waiting on dropping both Python 3.6 and 3.7 support (in quite some time), it probably is cleaner to use this back-port https://pypi.org/project/typing-inspect/ on Python `< 3.8`. Would also be...

We should ideally also support overloading (e.g. that plugin authors use https://www.python.org/dev/peps/pep-0484/#function-method-overloading to define different "combinations of arguments"). This could then also be used to further improve both automatic documentation...

We can out source the majority of this work to `pydantic` - see https://github.com/samuelcolvin/pydantic/issues/2127.

See also https://github.com/plotly/dash/pull/1707.