Skip to content

Restore the old mypy primer blocklist#2713

Closed
stroxler wants to merge 2 commits intomainfrom
export-D95741691
Closed

Restore the old mypy primer blocklist#2713
stroxler wants to merge 2 commits intomainfrom
export-D95741691

Conversation

@stroxler
Copy link
Contributor

@stroxler stroxler commented Mar 8, 2026

Summary:
Mypy primer is pretty heavyweight, and github CI is underpowered.

I was looking at runs from before it started timing out, and realized that a lot of things
that take 1-2s to check internally take much longer.

I think it's possible that a few projects (e.g. pandas, sympy) might just be too big to
type check massively in parallel on a tiny, underpowered server.

Working off that assumption, let's try restoring the old "determinism blocklist" - these
projects tend to be some of the biggest, just because the things that trigger nondeterminism
also make type check expensive.

Differential Revision: D95741691

stroxler added 2 commits March 8, 2026 16:00
Summary:
We started using pyrefly init to set up, which required moving the builds out of
mypy primer's control.

In the process, we accidentally switched the build mode from dev to release; release
builds are too slow.

It probably won't fix the problem alone since we already tested this, but without this
it probably won't work even if any other problems are fixed

Reviewed By: rchen152

Differential Revision: D95741090
Summary:
Mypy primer is pretty heavyweight, and github CI is underpowered.

I was looking at runs from before it started timing out, and realized that a lot of things
that take 1-2s to check internally take much longer.

I think it's possible that a few projects (e.g. pandas, sympy) might just be too big to
type check massively in parallel on a tiny, underpowered server.

Working off that assumption, let's try restoring the old "determinism blocklist" - these
projects tend to be some of the biggest, just because the things that trigger nondeterminism
also make type check expensive.

Differential Revision: D95741691
@meta-cla meta-cla bot added the cla signed label Mar 8, 2026
@meta-codesync
Copy link

meta-codesync bot commented Mar 8, 2026

@stroxler has exported this pull request. If you are a Meta employee, you can view the originating Diff in D95741691.

Copy link
Contributor

@rchen152 rchen152 left a comment

Choose a reason for hiding this comment

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

Review automatically exported from Phabricator review in Meta.

meta-codesync bot pushed a commit that referenced this pull request Mar 9, 2026
Summary:

Mypy primer is pretty heavyweight, and github CI is underpowered.

I was looking at runs from before it started timing out, and realized that a lot of things
that take 1-2s to check internally take much longer.

I think it's possible that a few projects (e.g. pandas, sympy) might just be too big to
type check massively in parallel on a tiny, underpowered server.

Working off that assumption, let's try restoring the old "determinism blocklist" - these
projects tend to be some of the biggest, just because the things that trigger nondeterminism
also make type check expensive.

Reviewed By: rchen152

Differential Revision: D95741691
meta-codesync bot pushed a commit that referenced this pull request Mar 9, 2026
Summary:

Mypy primer is pretty heavyweight, and github CI is underpowered.

I was looking at runs from before it started timing out, and realized that a lot of things
that take 1-2s to check internally take much longer.

I think it's possible that a few projects (e.g. pandas, sympy) might just be too big to
type check massively in parallel on a tiny, underpowered server.

Working off that assumption, let's try restoring the old "determinism blocklist" - these
projects tend to be some of the biggest, just because the things that trigger nondeterminism
also make type check expensive.

Reviewed By: rchen152

Differential Revision: D95741691
@meta-codesync meta-codesync bot closed this in ee128e8 Mar 9, 2026
@meta-codesync
Copy link

meta-codesync bot commented Mar 9, 2026

This pull request has been merged in ee128e8.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants