Zulip Chat Archive

Stream: mathlib4

Topic: Missing library notes


Anne Baanen (Jul 26 2023 at 09:38):

I can't find the declaration of the "lower instance priority" library note. In fact, grep only counts 27 occurrences of library_note in the mathlib4 repo (excluding the command setup itself) while it counts 37 in mathlib3 (again excluding the command setup itself).

Eric Wieser (Jul 26 2023 at 09:43):

Maybe an old version of mathport didn't support them, and so they're missing from early files?

Eric Wieser (Jul 26 2023 at 09:44):

Or more concerningly, maybe we have 10 ad-hoc files at the root of the import tree that were not ported at all

Anne Baanen (Jul 26 2023 at 10:00):

Sadly the syntax is slightly different so here is my best attempt at a manual diff (< marks lines only found in Mathlib 3, > only in Mathlib 4):

McChouffe% diff <(sort ../library-note-3) <(sort ../library-note-4)
1,16c1,7
< library_note "category_theory universes"
---
> library_note "CategoryTheory universes"
25,62c16,31
< library_note "function coercion"
< library_note "implicit instance arguments"
< library_note "is_R_or_C instance"
< library_note "likely generated binder names"
< library_note "lower instance priority"
< library_note "nolint_ge"
< library_note "open expressions"
< library_note "open expressions"
< library_note "partially-applied ext lemmas"
< library_note "use has_coe_t"
< library_note "user attribute parameters"
---
> library_note "IsROrC instance"/--
> library_note "norm_num lemma function equality"/--

Probably most of these are correctly removed because they are Lean 3 specific and I spot two renames, but we should check that carefully.

Scott Morrison (Aug 06 2023 at 23:14):

While not forgetting to followup on checking these library notes carefully, let me note the two open issues about clickability of library notes:


Last updated: Dec 20 2023 at 11:08 UTC