-
Notifications
You must be signed in to change notification settings - Fork 54
almost everywhere equality #1600
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
Conversation
9c1b05d
to
303b58b
Compare
In the interest of PR #1506 , I think that it is fine to merge ( |
303b58b
to
91b7937
Compare
Oops, mistake when force pushing |
Co-authored-by: Cyril Cohen <CohenCyril@users.noreply.github.com>
4d930a7
to
115b349
Compare
115b349
to
c5dbc61
Compare
I think this PR is almost ready for merging, and should be merged before #1606 Edit: I swapped notations in my original comment. |
I have a similar feeling about \near and {near , but not understood why these two are kept. I want to know a pointer to some previous discussion if any. |
I was about the make the same comment about |
I added a commit that removes the duplicated notation, and again a similar one on PR #1606 that removes its usage on the theory of essential supremum. |
This is not a duplication, please do not remove it... The purpose is the same as the It should be documented that the priority should be given to the |
548afb6
to
c5dbc61
Compare
You mean "the same as the |
I originally meant the "same as the But indeed we could also introduce |
OK, I understand the reasoning, thanks for the explanations. My main concern is that the notation is not stable and one (especially newcomers) may be surprised when they see something like this: Section test_notation.
Context d (T : measurableType d) (R : realType) (P : T -> Prop) (mu : measure T R).
Check \forall x \ae mu, P x.
(* 1: Prints: \forall x \ae mu, P x : Prop *)
Check {ae mu, forall x, P x}.
(* 2: Prints: \forall x \ae mu, P x : Prop *)
Definition Q := forall x, P x.
Check {ae mu, Q}.
(* 3: Prints: {ae mu, Q} : Prop *)
End test_notation. So test nr. 2 uses one notation and prints the other, which is undesirable. From a quick grep of the use of More long term, having an uniform |
On the one hand, non "stable" notations are pervasive in mathcomp, starting with |
The example of Shall we expand a bit this FAQ entry too then? |
I have adjusted the FAQ entry for |
Suggestions:
|
DONE. By the way, @t6s you don't have the rights to edit the wiki? |
Is it ok to merge? I think that none of us qualifies as an non-conflicting reviewer ^_^ but we did add documentation pieces where appropriate going beyond the scope of the PR. |
I suspect not. I could not find an edit button. |
--------- Co-authored-by: Cyril Cohen <CohenCyril@users.noreply.github.com> Co-authored-by: Reynald Affeldt <reynald.affeldt@aist.go.jp>
Motivation for this change
Adds notations and generalization for almost everywhere equality and related lemmas.
This is part of the split of PR #1506
Checklist
CHANGELOG_UNRELEASED.md
Reference: How to document
Merge policy
As a rule of thumb:
all compile are preferentially merged into master.
Reminder to reviewers