Zulip Chat Archive

Stream: mathlib4

Topic: Kinds of induction' to be removed


Jeremy Tan (Mar 29 2025 at 00:25):

@Peter Nelson on #23324:

I am not sure that 5 is a large enough number for this to be a no-brainer. I would personally rather see opinions on scope of induction' removal actively solicited on Zulip (perhaps in the form of a poll) and/or a clear statement from maintainers on what their plan is for induction' before I am happy with this kind of PR being merged.

What kinds of induction' would you like to have removed?

Eric Wieser (Mar 29 2025 at 00:28):

Choose a number n such that the number of lines to review is less than 20?

Eric Wieser (Mar 29 2025 at 00:29):

Though I feel like there are probably more useful maintenance tasks than cleaning up induction'.

Kevin Buzzard (Mar 29 2025 at 00:52):

e.g. cleaning up erw

Eric Wieser (Mar 29 2025 at 00:53):

Or if you're set on doing things around induction, replacing ugly refine some_induct (motive := ..) with induction ... with

Jeremy Tan (Mar 29 2025 at 01:08):

Kevin Buzzard said:

e.g. cleaning up erw

I actually have #23389 cleaning up a few erw

Jeremy Tan (Mar 29 2025 at 01:16):

Nevertheless my choice of 5 was inspired by @Jireh Loreaux's comment on my prior PR about induction' with many variables being hard to figure out. I did not choose 4 because one of the targets I included in my automated removal PR (#23211) had 4 variables

Jeremy Tan (Mar 29 2025 at 01:17):

As you can see in #23324, some uses of induction' also named more variables than necessary. The extra variables are ignored and only cause more confusion

Jeremy Tan (Mar 29 2025 at 01:20):

Ignored silently, I must emphasise

Eric Wieser (Mar 29 2025 at 01:21):

That's indeed bad, but if only _some_ of them are ignored, it would be easier to review a PR consisting of just those

Eric Wieser (Mar 29 2025 at 01:22):

Or really, a smaller PR of any form; 68 files is a lot to touch, especially when some are much more justifiable changes than others

Jeremy Tan (Mar 29 2025 at 01:51):

@Eric Wieser #23419

Jeremy Tan (Apr 05 2025 at 00:23):

After the removal of 5+ variable induction' (#23324, #23419, #23650) the next class I want to remove is those generating only one subgoal, starting with #23676

Jeremy Tan (Apr 05 2025 at 00:23):

Please get #23324 merged by the way

Eric Wieser (Apr 05 2025 at 00:42):

Thanks for splitting things. I think this most recent split is precisely the one where the gain is most marginal


Last updated: May 02 2025 at 03:31 UTC