This repository has been archived by the owner on Jul 24, 2024. It is now read-only.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
[Merged by Bors] - refactor(data/set/finite): reorganize and put emphasis on fintype instances #14136
[Merged by Bors] - refactor(data/set/finite): reorganize and put emphasis on fintype instances #14136
Changes from 20 commits
5835bcf
0c3ecf3
a4693f5
2e5e43a
b76df6d
eae560a
d8ffb5a
41e6bd9
b572e0b
8774203
ec15466
feed464
4b25d90
84ba4e1
ab44747
b69ec3e
ad7e533
cdcf54b
023cfa5
5799e88
000351e
494939c
5e3d59b
5409e54
c2078a1
0b4b83b
92a8b56
31812fd
6c2898a
9c12a57
3854af7
aaeac4c
1e3c1a8
0497014
a68b38d
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
Similarly what happened here?
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.
This one is because I changed
card_subtype_compl
fromto
and the rewrite is somehow not able to find the third fintype instance. If you give it the predicate directly, then it works fine. In the commit I pushed, I modified it to give the predicate directly.
I'm not sure what the best thing to do here is, and I don't really understand what's causing the instance lookup failure... (Maybe it's
ne
vs not-equals in whateverp
it infers?)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.
What's the reason for some of these to be
simp
and not others?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'm not sure, this is how they were, and I didn't want to see if mathlib would compile with them as
simp
lemmas.