-
Notifications
You must be signed in to change notification settings - Fork 298
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
[Merged by Bors] - feat(group_theory/order_of_element): pow_eq_mod_card #8354
Conversation
ChrisHughes24
commented
Jul 17, 2021
Co-authored-by: Eric Wieser <wieser.eric@gmail.com>
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.
bors d+
It looks like some of the existing lemmas have bad argument explicitness, but fixing that is obviously out of scope. It's probably still worth adjusting the proofs to not need @
s though.
by rw [pow_eq_mod_order_of, @pow_eq_mod_order_of _ _ _ (_ % fintype.card G), | ||
nat.mod_mod_of_dvd _ order_of_dvd_card_univ] |
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.
You can avoid the @
here by writing in reverse:
by rw [pow_eq_mod_order_of, @pow_eq_mod_order_of _ _ _ (_ % fintype.card G), | |
nat.mod_mod_of_dvd _ order_of_dvd_card_univ] | |
by rw [pow_eq_mod_order_of, ←nat.mod_mod_of_dvd n order_of_dvd_card_univ, ←pow_eq_mod_order_of] |
by rw [gpow_eq_mod_order_of, @gpow_eq_mod_order_of _ _ _ (_ % fintype.card G), | ||
int.mod_mod_of_dvd _ (int.coe_nat_dvd.2 order_of_dvd_card_univ)] |
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.
Although it's less effective here.
by rw [gpow_eq_mod_order_of, @gpow_eq_mod_order_of _ _ _ (_ % fintype.card G), | |
int.mod_mod_of_dvd _ (int.coe_nat_dvd.2 order_of_dvd_card_univ)] | |
by rw [gpow_eq_mod_order_of, ←int.mod_mod_of_dvd n (int.coe_nat_dvd.2 order_of_dvd_card_univ), | |
←gpow_eq_mod_order_of] |
✌️ ChrisHughes24 can now approve this pull request. To approve and merge a pull request, simply reply with |
bors merge |
Co-authored-by: Chris Hughes <33847686+ChrisHughes24@users.noreply.github.com>
Pull request successfully merged into master. Build succeeded: |