Zulip Chat Archive
Stream: mathlib4
Topic: GroupTheory.Divisible !4#2149
Ruben Van de Velde (Feb 07 2023 at 15:05):
I got some annoying issues with to_additive; fixed most of them by adding explicit names, but haven't figured out the last error. If someone could take a look, that would be great
Floris van Doorn (Feb 07 2023 at 15:13):
For the naming: you should add rootable
to
https://github.com/leanprover-community/mathlib4/blob/59ce0e8641506f646405607f92d84b300a2d0d67/Mathlib/Tactic/ToAdditive.lean#L676
Floris van Doorn (Feb 07 2023 at 15:14):
i.e. | "rootable" => ["divisible"]
Floris van Doorn (Feb 07 2023 at 15:17):
For the other error, please make sure A
is the first variable in all lemma statements, i.e. reorder variables {α A B : Type*}
. That should fix the error.
Floris van Doorn (Feb 07 2023 at 15:20):
The fact that it worked in Lean 3 is probably a regression in the behavior of docs4#ToAdditive.firstMultiplicativeArg. I will investigate, but you make life easier for to_additive
if you change the argument order (and that shouldn't break anything down the line, but you could add a porting note).
Ruben Van de Velde (Feb 07 2023 at 15:21):
Thanks, will try
Last updated: Dec 20 2023 at 11:08 UTC