Skip to content

Conversation

@shaneahmed
Copy link
Member

  • Add mypy checks to image.py

@codecov
Copy link

codecov bot commented Dec 11, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (a4463ee) 99.85% compared to head (4af2c06) 99.85%.
Report is 1 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #753   +/-   ##
========================================
  Coverage    99.85%   99.85%           
========================================
  Files           65       65           
  Lines         7532     7541    +9     
  Branches      1460     1461    +1     
========================================
+ Hits          7521     7530    +9     
  Misses           4        4           
  Partials         7        7           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@shaneahmed shaneahmed added the dev tools Changes/Updates in Development tools label Dec 11, 2023
@shaneahmed shaneahmed self-assigned this Dec 11, 2023
@shaneahmed shaneahmed added this to the Release v1.5.0 milestone Dec 11, 2023
Signed-off-by: Shan E Ahmed Raza <[email protected]>
Signed-off-by: Shan E Ahmed Raza <[email protected]>
@shaneahmed shaneahmed requested a review from Jiaqi-Lv December 11, 2023 17:09
@Jiaqi-Lv
Copy link
Collaborator

Strangely when I run mypy locally it says there are 10 errors. I don't know why this is happening.

tiatoolbox/utils/image.py:283: error: No overload variant of "pad" matches argument types "Any", "ndarray[Any, Any]", "str", "Union[int, Tuple[Any, ...]]" [call-overload]
tiatoolbox/utils/image.py:283: note: Possible overload variants:
tiatoolbox/utils/image.py:283: note: def [_SCT <: generic] pad(array: Any, pad_width: Any, mode: Literal['constant', 'edge', 'linear_ramp', 'maximum', 'mean', 'median', 'minimum', 'reflect', 'symmetric', 'wrap', 'empty'] = ..., *, stat_length: Optional[Any] = ..., constant_values: Any = ..., end_values: Any = ..., reflect_type: Literal['odd', 'even'] = ...) -> Any
tiatoolbox/utils/image.py:283: note: def pad(array: Any, pad_width: Any, mode: Literal['constant', 'edge', 'linear_ramp', 'maximum', 'mean', 'median', 'minimum', 'reflect', 'symmetric', 'wrap', 'empty'] = ..., *, stat_length: Optional[Any] = ..., constant_values: Any = ..., end_values: Any = ..., reflect_type: Literal['odd', 'even'] = ...) -> Any
tiatoolbox/utils/image.py:283: note: def [_SCT <: generic] pad(array: Any, pad_width: Any, mode: _ModeFunc, **kwargs: Any) -> Any
tiatoolbox/utils/image.py:283: note: def pad(array: Any, pad_width: Any, mode: _ModeFunc, **kwargs: Any) -> Any
tiatoolbox/utils/image.py:284: error: No overload variant of "pad" matches argument types "Any", "ndarray[Any, Any]", "str" [call-overload]
tiatoolbox/utils/image.py:284: note: Possible overload variants:
tiatoolbox/utils/image.py:284: note: def [_SCT <: generic] pad(array: Any, pad_width: Any, mode: Literal['constant', 'edge', 'linear_ramp', 'maximum', 'mean', 'median', 'minimum', 'reflect', 'symmetric', 'wrap', 'empty'] = ..., *, stat_length: Optional[Any] = ..., constant_values: Any = ..., end_values: Any = ..., reflect_type: Literal['odd', 'even'] = ...) -> Any
tiatoolbox/utils/image.py:284: note: def pad(array: Any, pad_width: Any, mode: Literal['constant', 'edge', 'linear_ramp', 'maximum', 'mean', 'median', 'minimum', 'reflect', 'symmetric', 'wrap', 'empty'] = ..., *, stat_length: Optional[Any] = ..., constant_values: Any = ..., end_values: Any = ..., reflect_type: Literal['odd', 'even'] = ...) -> Any
tiatoolbox/utils/image.py:284: note: def [_SCT <: generic] pad(array: Any, pad_width: Any, mode: _ModeFunc, **kwargs: Any) -> Any
tiatoolbox/utils/image.py:284: note: def pad(array: Any, pad_width: Any, mode: _ModeFunc, **kwargs: Any) -> Any
tiatoolbox/utils/image.py:424: error: No overload variant of "pad" matches argument types "ndarray[Any, Any]", "List[Tuple[Any, Any]]", "str", "Dict[Any, Any]" [call-overload]
tiatoolbox/utils/image.py:424: note: Possible overload variants:
tiatoolbox/utils/image.py:424: note: def [_SCT <: generic] pad(array: Any, pad_width: Any, mode: Literal['constant', 'edge', 'linear_ramp', 'maximum', 'mean', 'median', 'minimum', 'reflect', 'symmetric', 'wrap', 'empty'] = ..., *, stat_length: Optional[Any] = ..., constant_values: Any = ..., end_values: Any = ..., reflect_type: Literal['odd', 'even'] = ...) -> Any
tiatoolbox/utils/image.py:424: note: def pad(array: Any, pad_width: Any, mode: Literal['constant', 'edge', 'linear_ramp', 'maximum', 'mean', 'median', 'minimum', 'reflect', 'symmetric', 'wrap', 'empty'] = ..., *, stat_length: Optional[Any] = ..., constant_values: Any = ..., end_values: Any = ..., reflect_type: Literal['odd', 'even'] = ...) -> Any
tiatoolbox/utils/image.py:424: note: def [_SCT <: generic] pad(array: Any, pad_width: Any, mode: _ModeFunc, **kwargs: Any) -> Any
tiatoolbox/utils/image.py:424: note: def pad(array: Any, pad_width: Any, mode: _ModeFunc, **kwargs: Any) -> Any
tiatoolbox/utils/image.py:597: error: Too many arguments for "find_overlap" [call-arg]
tiatoolbox/utils/image.py:601: error: "find_overlap" gets multiple values for keyword argument "image_size" [misc]
tiatoolbox/utils/image.py:626: error: Too many arguments for "find_overlap" [call-arg]
tiatoolbox/utils/image.py:646: error: "find_padding" gets multiple values for keyword argument "image_size" [misc]
tiatoolbox/utils/image.py:648: error: Too many arguments for "find_padding" [call-arg]
tiatoolbox/utils/image.py:656: error: No overload variant of "pad" matches argument types "Any", "Any", "str", "Dict[Any, Any]" [call-overload]
tiatoolbox/utils/image.py:656: note: Possible overload variants:
tiatoolbox/utils/image.py:656: note: def [_SCT <: generic] pad(array: Any, pad_width: Any, mode: Literal['constant', 'edge', 'linear_ramp', 'maximum', 'mean', 'median', 'minimum', 'reflect', 'symmetric', 'wrap', 'empty'] = ..., *, stat_length: Optional[Any] = ..., constant_values: Any = ..., end_values: Any = ..., reflect_type: Literal['odd', 'even'] = ...) -> Any
tiatoolbox/utils/image.py:656: note: def pad(array: Any, pad_width: Any, mode: Literal['constant', 'edge', 'linear_ramp', 'maximum', 'mean', 'median', 'minimum', 'reflect', 'symmetric', 'wrap', 'empty'] = ..., *, stat_length: Optional[Any] = ..., constant_values: Any = ..., end_values: Any = ..., reflect_type: Literal['odd', 'even'] = ...) -> Any
tiatoolbox/utils/image.py:656: note: def [_SCT <: generic] pad(array: Any, pad_width: Any, mode: _ModeFunc, **kwargs: Any) -> Any
tiatoolbox/utils/image.py:656: note: def pad(array: Any, pad_width: Any, mode: _ModeFunc, **kwargs: Any) -> Any
tiatoolbox/utils/image.py:663: error: No overload variant of "pad" matches argument types "Any", "Any", "str" [call-overload]
tiatoolbox/utils/image.py:663: note: Possible overload variants:
tiatoolbox/utils/image.py:663: note: def [_SCT <: generic] pad(array: Any, pad_width: Any, mode: Literal['constant', 'edge', 'linear_ramp', 'maximum', 'mean', 'median', 'minimum', 'reflect', 'symmetric', 'wrap', 'empty'] = ..., *, stat_length: Optional[Any] = ..., constant_values: Any = ..., end_values: Any = ..., reflect_type: Literal['odd', 'even'] = ...) -> Any
tiatoolbox/utils/image.py:663: note: def pad(array: Any, pad_width: Any, mode: Literal['constant', 'edge', 'linear_ramp', 'maximum', 'mean', 'median', 'minimum', 'reflect', 'symmetric', 'wrap', 'empty'] = ..., *, stat_length: Optional[Any] = ..., constant_values: Any = ..., end_values: Any = ..., reflect_type: Literal['odd', 'even'] = ...) -> Any
tiatoolbox/utils/image.py:663: note: def [_SCT <: generic] pad(array: Any, pad_width: Any, mode: _ModeFunc, **kwargs: Any) -> Any
tiatoolbox/utils/image.py:663: note: def pad(array: Any, pad_width: Any, mode: _ModeFunc, **kwargs: Any) -> Any
Found 10 errors in 1 file (checked 1 source file)

@shaneahmed
Copy link
Member Author

Strangely when I run mypy locally it says there are 10 errors. I don't know why this is happening.

tiatoolbox/utils/image.py:283: error: No overload variant of "pad" matches argument types "Any", "ndarray[Any, Any]", "str", "Union[int, Tuple[Any, ...]]" [call-overload]
tiatoolbox/utils/image.py:283: note: Possible overload variants:
tiatoolbox/utils/image.py:283: note: def [_SCT <: generic] pad(array: Any, pad_width: Any, mode: Literal['constant', 'edge', 'linear_ramp', 'maximum', 'mean', 'median', 'minimum', 'reflect', 'symmetric', 'wrap', 'empty'] = ..., *, stat_length: Optional[Any] = ..., constant_values: Any = ..., end_values: Any = ..., reflect_type: Literal['odd', 'even'] = ...) -> Any
tiatoolbox/utils/image.py:283: note: def pad(array: Any, pad_width: Any, mode: Literal['constant', 'edge', 'linear_ramp', 'maximum', 'mean', 'median', 'minimum', 'reflect', 'symmetric', 'wrap', 'empty'] = ..., *, stat_length: Optional[Any] = ..., constant_values: Any = ..., end_values: Any = ..., reflect_type: Literal['odd', 'even'] = ...) -> Any
tiatoolbox/utils/image.py:283: note: def [_SCT <: generic] pad(array: Any, pad_width: Any, mode: _ModeFunc, **kwargs: Any) -> Any
tiatoolbox/utils/image.py:283: note: def pad(array: Any, pad_width: Any, mode: _ModeFunc, **kwargs: Any) -> Any
tiatoolbox/utils/image.py:284: error: No overload variant of "pad" matches argument types "Any", "ndarray[Any, Any]", "str" [call-overload]
tiatoolbox/utils/image.py:284: note: Possible overload variants:
tiatoolbox/utils/image.py:284: note: def [_SCT <: generic] pad(array: Any, pad_width: Any, mode: Literal['constant', 'edge', 'linear_ramp', 'maximum', 'mean', 'median', 'minimum', 'reflect', 'symmetric', 'wrap', 'empty'] = ..., *, stat_length: Optional[Any] = ..., constant_values: Any = ..., end_values: Any = ..., reflect_type: Literal['odd', 'even'] = ...) -> Any
tiatoolbox/utils/image.py:284: note: def pad(array: Any, pad_width: Any, mode: Literal['constant', 'edge', 'linear_ramp', 'maximum', 'mean', 'median', 'minimum', 'reflect', 'symmetric', 'wrap', 'empty'] = ..., *, stat_length: Optional[Any] = ..., constant_values: Any = ..., end_values: Any = ..., reflect_type: Literal['odd', 'even'] = ...) -> Any
tiatoolbox/utils/image.py:284: note: def [_SCT <: generic] pad(array: Any, pad_width: Any, mode: _ModeFunc, **kwargs: Any) -> Any
tiatoolbox/utils/image.py:284: note: def pad(array: Any, pad_width: Any, mode: _ModeFunc, **kwargs: Any) -> Any
tiatoolbox/utils/image.py:424: error: No overload variant of "pad" matches argument types "ndarray[Any, Any]", "List[Tuple[Any, Any]]", "str", "Dict[Any, Any]" [call-overload]
tiatoolbox/utils/image.py:424: note: Possible overload variants:
tiatoolbox/utils/image.py:424: note: def [_SCT <: generic] pad(array: Any, pad_width: Any, mode: Literal['constant', 'edge', 'linear_ramp', 'maximum', 'mean', 'median', 'minimum', 'reflect', 'symmetric', 'wrap', 'empty'] = ..., *, stat_length: Optional[Any] = ..., constant_values: Any = ..., end_values: Any = ..., reflect_type: Literal['odd', 'even'] = ...) -> Any
tiatoolbox/utils/image.py:424: note: def pad(array: Any, pad_width: Any, mode: Literal['constant', 'edge', 'linear_ramp', 'maximum', 'mean', 'median', 'minimum', 'reflect', 'symmetric', 'wrap', 'empty'] = ..., *, stat_length: Optional[Any] = ..., constant_values: Any = ..., end_values: Any = ..., reflect_type: Literal['odd', 'even'] = ...) -> Any
tiatoolbox/utils/image.py:424: note: def [_SCT <: generic] pad(array: Any, pad_width: Any, mode: _ModeFunc, **kwargs: Any) -> Any
tiatoolbox/utils/image.py:424: note: def pad(array: Any, pad_width: Any, mode: _ModeFunc, **kwargs: Any) -> Any
tiatoolbox/utils/image.py:597: error: Too many arguments for "find_overlap" [call-arg]
tiatoolbox/utils/image.py:601: error: "find_overlap" gets multiple values for keyword argument "image_size" [misc]
tiatoolbox/utils/image.py:626: error: Too many arguments for "find_overlap" [call-arg]
tiatoolbox/utils/image.py:646: error: "find_padding" gets multiple values for keyword argument "image_size" [misc]
tiatoolbox/utils/image.py:648: error: Too many arguments for "find_padding" [call-arg]
tiatoolbox/utils/image.py:656: error: No overload variant of "pad" matches argument types "Any", "Any", "str", "Dict[Any, Any]" [call-overload]
tiatoolbox/utils/image.py:656: note: Possible overload variants:
tiatoolbox/utils/image.py:656: note: def [_SCT <: generic] pad(array: Any, pad_width: Any, mode: Literal['constant', 'edge', 'linear_ramp', 'maximum', 'mean', 'median', 'minimum', 'reflect', 'symmetric', 'wrap', 'empty'] = ..., *, stat_length: Optional[Any] = ..., constant_values: Any = ..., end_values: Any = ..., reflect_type: Literal['odd', 'even'] = ...) -> Any
tiatoolbox/utils/image.py:656: note: def pad(array: Any, pad_width: Any, mode: Literal['constant', 'edge', 'linear_ramp', 'maximum', 'mean', 'median', 'minimum', 'reflect', 'symmetric', 'wrap', 'empty'] = ..., *, stat_length: Optional[Any] = ..., constant_values: Any = ..., end_values: Any = ..., reflect_type: Literal['odd', 'even'] = ...) -> Any
tiatoolbox/utils/image.py:656: note: def [_SCT <: generic] pad(array: Any, pad_width: Any, mode: _ModeFunc, **kwargs: Any) -> Any
tiatoolbox/utils/image.py:656: note: def pad(array: Any, pad_width: Any, mode: _ModeFunc, **kwargs: Any) -> Any
tiatoolbox/utils/image.py:663: error: No overload variant of "pad" matches argument types "Any", "Any", "str" [call-overload]
tiatoolbox/utils/image.py:663: note: Possible overload variants:
tiatoolbox/utils/image.py:663: note: def [_SCT <: generic] pad(array: Any, pad_width: Any, mode: Literal['constant', 'edge', 'linear_ramp', 'maximum', 'mean', 'median', 'minimum', 'reflect', 'symmetric', 'wrap', 'empty'] = ..., *, stat_length: Optional[Any] = ..., constant_values: Any = ..., end_values: Any = ..., reflect_type: Literal['odd', 'even'] = ...) -> Any
tiatoolbox/utils/image.py:663: note: def pad(array: Any, pad_width: Any, mode: Literal['constant', 'edge', 'linear_ramp', 'maximum', 'mean', 'median', 'minimum', 'reflect', 'symmetric', 'wrap', 'empty'] = ..., *, stat_length: Optional[Any] = ..., constant_values: Any = ..., end_values: Any = ..., reflect_type: Literal['odd', 'even'] = ...) -> Any
tiatoolbox/utils/image.py:663: note: def [_SCT <: generic] pad(array: Any, pad_width: Any, mode: _ModeFunc, **kwargs: Any) -> Any
tiatoolbox/utils/image.py:663: note: def pad(array: Any, pad_width: Any, mode: _ModeFunc, **kwargs: Any) -> Any
Found 10 errors in 1 file (checked 1 source file)

Same here. Need to check why.

@shaneahmed
Copy link
Member Author

@Jiaqi-Lv this should pass now locally.

@shaneahmed shaneahmed merged commit 3afb52a into develop Dec 15, 2023
@shaneahmed shaneahmed deleted the dev-fix-mypy-image branch December 15, 2023 10:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dev tools Changes/Updates in Development tools

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants