Zulip Chat Archive

Stream: mathlib4

Topic: Data.Set.Lattice


Chris Hughes (Dec 21 2022 at 10:22):

An issue came up in this file. One is that the simp attribute on unionᵢ_interᵢ_ge_nat_add triggered the linter Left-hand side does not simplify, when using the simp lemma on itself. This usually means that it will never apply. However, it does seem to apply when used on itself. Maybe a linter bug?

Patrick Massot (Dec 21 2022 at 10:24):

This sounds similar to https://leanprover.zulipchat.com/#narrow/stream/287929-mathlib4/topic/complete_lattice.20and.20has_sup/near/316497982

Kevin Buzzard (Dec 21 2022 at 11:39):

And as Eric points out, both this example and the other example have ge in.

Kevin Buzzard (Dec 21 2022 at 15:23):

@Chris Hughes your example is essentially the same lemma as Patrick's and in that thread Gabriel is suggesting that it's a bad simp lemma for other reasons; you'll understand them better than I do and if they also apply in your case then you can just remove the simp, add a porting note and at least you're unblocked, even though we don't actually understand what's going on.

Kevin Buzzard (Dec 21 2022 at 15:43):

I've recorded the issue here but clearly the short term solution is just to remove the simp and move on.

Eric Wieser (Dec 21 2022 at 16:49):

Note that the forward-port at mathlib4#1109 will conflict with the port of this file; which takes priority here? Do we make the forward-porter redo the forward-port, or make the porter redo the port?

Patrick Massot (Dec 21 2022 at 16:58):

This is really a mess. I think that each day brings us closer to the day where touching a mathlib3 file this already ported will simply be totally forbidden (maybe with some very rare exceptions discussed among maintainers).

Eric Wieser (Dec 21 2022 at 16:58):

I think probably things that touch a mix of ported and partially-ported files are the really nasty cases

Patrick Massot (Dec 21 2022 at 16:59):

I'm not sure what bors is doing with #1109 anyway.

Eric Wieser (Dec 21 2022 at 16:59):

If you mean the mathlib3 version of mathlib4#1109, it's about 20 minutes from hitting master

Patrick Massot (Dec 21 2022 at 17:00):

Yes, I meant #17882, sorry.

Eric Wieser (Dec 21 2022 at 17:01):

Ideally the porting warning bot would detect files with in-progress ports too

Yuyang Zhao (Dec 21 2022 at 19:33):

Just merge mathlib4#1121 first. It's easy to deal with the conflict.


Last updated: Dec 20 2023 at 11:08 UTC