-
Notifications
You must be signed in to change notification settings - Fork 52
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Frobenius elements #18
Conversation
@jjaassoonn @jouglasheen can you clarify why there are two files here? My instinct is to merge this PR (but ideally with the file in |
@kbuzzard @jjaassoonn CrazyFrobenius.lean is an obsolete version of ExistsFrobenius.lean. We can delete CrazyFrobenius.lean. |
Can you delete it and bump to current |
FLT/CrazyFrobenius.lean
Outdated
to an element in the orbit of `Q` under the action of the Galois group `L ≃ₐ[K] L`. | ||
We will consider the orbit to be a 'Fintype', as used in the statement of | ||
the Chinese Remainder Theorem. -/ | ||
noncomputable def f : |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jouglasheen, to delete the file, do it on the command line: |
You can just do:
If the merge works, then just |
[IsDedekindDomain A] [IsDedekindDomain B] | ||
|
||
/-- re-definition of `Ideal B` in terms of 'AKLB setup'. -/ | ||
@[nolint unusedArguments] abbrev Ideal' (A K L B : Type*) [CommRing A] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Naming things with '
is rarely a good idea. Can you think up a name that gives the human a hint? Even AKLB_Ideal
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alternatively can you make it private
??
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The file compiles fine with
@[nolint unusedArguments] abbrev Ideal' (A K L B : Type*) [CommSemiring A]
[CommSemiring B] [Algebra A B] [CommSemiring K] [Semiring L]
[Algebra A K] [Algebra B L]
[Algebra K L] [Algebra A L] [IsScalarTower A B L]
[IsScalarTower A K L] := Ideal B
which confirms that Ideal' B
only needs to know "it's in an A K L B
set-up" which might be code for a commutative square (with the diagonal filled in) in the same way that three Algebra
instances and an IsScalarTower
is code for a commutative triangle.
FLT/CrazyFrobenius.lean
Outdated
erw[map_prod] | ||
simp only [map_sub, coe_mapRingHom, map_X, map_C] | ||
|
||
lemma quotient_F_is_root_iff_is_conjugate (x : (B ⧸ Q)) : |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems to me that many lemmas and definitions until here would apply to any element in B
instead of the carefully chosen β
. It may be worth generalizing the construction of F
, etc, in that generality.
obsolete version of ExistsFrobenius.lean
@semorrison @joelriou Thank you very much for the reviews and help with github. I have succeeded in deleting the obsolete file and I think I have succeeded in merging to the main branch. Please, excuse some delay in responding to your reviews (I am currently studying for exams). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
[IsDedekindDomain A] [IsDedekindDomain B] | ||
|
||
/-- re-definition of `Ideal B` in terms of 'AKLB setup'. -/ | ||
@[nolint unusedArguments] abbrev Ideal' (A K L B : Type*) [CommRing A] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The file compiles fine with
@[nolint unusedArguments] abbrev Ideal' (A K L B : Type*) [CommSemiring A]
[CommSemiring B] [Algebra A B] [CommSemiring K] [Semiring L]
[Algebra A K] [Algebra B L]
[Algebra K L] [Algebra A L] [IsScalarTower A B L]
[IsScalarTower A K L] := Ideal B
which confirms that Ideal' B
only needs to know "it's in an A K L B
set-up" which might be code for a commutative square (with the diagonal filled in) in the same way that three Algebra
instances and an IsScalarTower
is code for a commutative triangle.
Frobenius elements