-
Notifications
You must be signed in to change notification settings - Fork 299
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(set_theory/game/pgame): Add missing basic API #13744
Conversation
vihdzp
commented
Apr 27, 2022
@[simp] theorem half_left_moves : half.left_moves = punit := rfl | ||
@[simp] theorem half_right_moves : half.right_moves = punit := rfl | ||
@[simp] lemma half_move_left (x) : half.move_left x = 0 := rfl | ||
@[simp] lemma half_move_right (x) : half.move_right x = 1 := rfl |
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.
I think it would be worth trying to train simps
to generate these
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.
How does simps
work?
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.
If pgame
were a structure, rather than an inductive type, we would use initialize_simps_projections
.
However as it is an inductive type I don't think we can do anything.
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.
Co-authored-by: Eric Wieser <wieser.eric@gmail.com>
bors merge |
Pull request successfully merged into master. Build succeeded: |