Zulip Chat Archive
Stream: mathlib4
Topic: Basic porting questions
Anatole Dedecker (Nov 13 2022 at 10:35):
Hi everyone! After successfully porting a rather trivial file in mathlib4#582, I started working on a port of control.basic
(yes, I'm starting from files low in the priority queue in order not to block too much people while I learn how to port efficiently), and I have a few small questions:
- if I understand correctly, simp attributes can no longer be used in the file where they are declared, is that right? If so, what is the expected solution for that? To be more explicit, I'm thinking about docs#simp_attr.functor_norm, which is used multiple times in
control.basic
(note: it also seems to be used incontrol.functor
, so it looks like some transitive import throughtactic
is ignored by theport_status
script). Should I create an intermediate file just for introducing the simp attribute? - what is the current attitude regarding small mismatches in the order of implicit arguments? Most of the
dubious translation
messages frommathport
seem to be reporting this sort of inconsistencies, and I'm not sure if we want to fix them or consider that we only need to ensure the order of explicit arguments - more specific to
control.basic
: the docs#fish operator is already defined in the core library as docs4#Bind.KleisliRight, and the precedence issue mentioned in this comment seems to no longer apply, so I'll just be removing the definition. Should I change the names of the following lemmas to replacefish
byKleisliRight
?
Anatole Dedecker (Nov 13 2022 at 10:50):
I opened a WIP PR as mathlib4#588 if anyone wants to have a look
Heather Macbeth (Nov 14 2022 at 18:20):
After a porting PR is merged, should I as the author update the port status from
No: PR mathlib4#585
to
Yes: PR mathlib4#585
? Or is this something that should be left to the porting leaders?
Same question about how to "verify" that the file is ported.
Scott Morrison (Nov 14 2022 at 20:41):
Yes, it's helpful if you check that, although I'm trying to remember to do it as I bors merge
as well. It's still a bit of a mess, sadly.
Scott Morrison (Nov 14 2022 at 20:41):
The verification step has so far not been happening at all. :-(
Heather Macbeth (Nov 14 2022 at 20:53):
OK, I made the update for that one, and I will keep an eye out for any announcements about changes to the workflow :)
Last updated: Dec 20 2023 at 11:08 UTC