Skip to content

(strict) typing for __init__.py, mpi.py, graph.py - #176

Open
matthiasdiener wants to merge 9 commits into
inducer:mainfrom
matthiasdiener:typing-record
Open

(strict) typing for __init__.py, mpi.py, graph.py#176
matthiasdiener wants to merge 9 commits into
inducer:mainfrom
matthiasdiener:typing-record

Conversation

@matthiasdiener

Copy link
Copy Markdown
Contributor

No description provided.

@matthiasdienermatthiasdiener changed the title Typing recordTyping for RecordMar 11, 2023
@matthiasdiener
matthiasdienerforce-pushed the typing-record branch 3 times, most recently from 71cadf4 to 94b706bCompareMarch 14, 2023 01:53
@matthiasdienermatthiasdiener changed the title Typing for Record(strict) typing for RecordMar 14, 2023
@matthiasdienermatthiasdiener changed the title (strict) typing for Record(strict) typing for __init__.py, mpi.py, graph.pyMar 14, 2023
@matthiasdiener
matthiasdiener marked this pull request as ready for review March 14, 2023 23:03
@matthiasdiener

Copy link
Copy Markdown
ContributorAuthor

This is ready for review. I can break the changes into smaller pieces if you prefer.

Comment threadpytools/__init__.py
return result
comb = MovedFunctionDeprecationWrapper(math.comb, deadline=2023)

def perm(n: SupportsIndex, # type: ignore[misc]

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I removed this function since pytools requires Python 3.8+ currently.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@matthiasdiener