Skip to content

Commit

Permalink
add mypy test requirement to fix known mypy bug in python/mypy#4117
Browse files Browse the repository at this point in the history
  • Loading branch information
chanedwin committed Sep 20, 2020
1 parent fed82f0 commit 7dea27d
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
2 changes: 2 additions & 0 deletions requirements-test.txt
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,8 @@ pytest
coverage<5
codecov
pytest-mypy>=0.7.0
# this is because mypy had an issue where singledispatch _ usage resulted in errors (https://github.com/python/mypy/issues/2904)
mypy>=0.761
pytest-cov
pytest-black
nbval
Expand Down
6 changes: 3 additions & 3 deletions src/pandas_profiling/model/summary.py
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ def describe_1d(series, summarizer: BaseSummarizer, typeset) -> dict:
raise NotImplementedError("Method is not implemented for dtype")


@describe_1d.register
@describe_1d.register(pd.Series)
def _(series: pd.Series, summarizer: BaseSummarizer, typeset) -> dict:
"""
if a pd.Series is provided, we should wrap it in the pandas series wrapper.
Expand All @@ -54,7 +54,7 @@ def _(series: pd.Series, summarizer: BaseSummarizer, typeset) -> dict:
return describe_1d(PandasSeries(series), summarizer, typeset)


@describe_1d.register
@describe_1d.register(PandasSeries)
def _(series: PandasSeries, summarizer: BaseSummarizer, typeset) -> dict:
"""Describe a series (infer the variable type, then calculate type-specific values).
Expand All @@ -76,7 +76,7 @@ def _(series: PandasSeries, summarizer: BaseSummarizer, typeset) -> dict:
return summarizer.summarize(series, dtype=vtype)


@describe_1d.register
@describe_1d.register(SparkSeries)
def _(series: SparkSeries, summarizer: BaseSummarizer, typeset) -> dict:
"""Describe a series (infer the variable type, then calculate type-specific values).
Expand Down

0 comments on commit 7dea27d

Please sign in to comment.