Zulip Chat Archive

Stream: mathlib4

Topic: speeding up linting


Jeremy Tan (Mar 16 2023 at 04:29):

I have a related question. How can we speed up linting? I think much of the linting we do is unnecessary, and I've observed that now times for linting are routinely in the 15-min range rather than ~10 min

Jeremy Tan (Mar 16 2023 at 04:38):

For simple PRs like a plain mathlib port, right now linting takes up a majority of the time needed

Jeremy Tan (Mar 16 2023 at 04:39):

Would it be possible to cache linting results like we already do for the build?

Scott Morrison (Mar 16 2023 at 05:12):

Caching linting is quite hard. The most time consuming part is the simp normal form linter, which really is a global test of the simp-set. There's nothing really to cache.

Notification Bot (Mar 16 2023 at 08:40):

4 messages were moved here from #mathlib4 > mathlib4 speedcenter by Sebastian Ullrich.

Yaël Dillies (Mar 16 2023 at 08:53):

In mathlib3, for comparison, building takes ~6h and linting 40min.

Sebastian Ullrich (Mar 16 2023 at 09:00):

I moved this conversation as it's not about how to measure performance, but do we want to track the global linter in the speedcenter as well? The benchmarking machine is roughly twice as fast as the CI runner, so it would add some 5 minutes to each run.

Eric Wieser (Mar 16 2023 at 09:03):

Yaël Dillies said:

In mathlib3, for comparison, building takes ~6h and linting 40min.

I'm pretty sure linting is more like 1hr now

Yaël Dillies (Mar 16 2023 at 09:11):

Good old times...

Gabriel Ebner (Mar 16 2023 at 14:22):

@Sebastian Ullrich that would be great.


Last updated: Dec 20 2023 at 11:08 UTC