Zulip Chat Archive
Stream: PR reviews
Topic: !3#18789
Scott Morrison (Jul 25 2023 at 04:53):
@Michael Stoll, are you still interested in having your mathlib3 PR !3#18789 merged. It is marked as not-too-late
, but also marked WIP
. We'd like to clear out this backlog so it would be good to know your intentions here.
Michael Stoll (Jul 25 2023 at 13:11):
I think it would make sense to integrate this with the more general Matroid Theory development, but I can't really tell how much work that would be (but this would probably mean not to merge the mathlib3 PR). @Peter Nelson what do you think?
What the current PR is missing is a refactor of Hall's Theorem using the more general result, but that would not be a lot of work.
However, I'm at a workshop this week that I am organizing, and so I won't have time to work on this now.
Peter Nelson (Jul 25 2023 at 13:24):
I agree, and I think it'd be doable, though I haven't looked at the proof in detail. One would have to define a constructor for Matroid
as per rank_fn
in your code, and then to prove it gives a Matroid
that is Finitary
, whose independent sets are precisely the finite sets with rank equal to cardinality. Then it would be a matter of using the existing Matroid
API to make the proof go through, which possibly involves adding a few lemmas for Finitary
.
The relevant (lean4) code is here : https://github.com/apnelson1/Matroid. I'm planning to PR the first Data.Matroid
contribution imminently (review is almost complete for the last prereq), but the rank
stuff is a few PRs away. So if there's no hurry, it will fit in smoothly in (I imagine) a month or two.
Michael Stoll (Jul 25 2023 at 13:38):
OK, good. That time frame would fit quite well for me (I'll be back from vacation).
Scott Morrison (Jul 26 2023 at 23:08):
@Michael Stoll, would it be okay with you, in that case, if we closed !3#18789 for now, and plan to integrate this work later? It's also okay to leave it open: we do want to clear out the mathlib3 queue so we can stick a fork in it and say we're done, but not at the expense of losing appealing contributions.
Michael Stoll (Jul 27 2023 at 07:18):
@Scott Morrison It is OK with me to close it (unless that would mean that the code gets deleted eventually; I would like to keep it on record at least until a version of it makes it into mathlib(4)).
Scott Morrison (Jul 27 2023 at 07:32):
No, there are no plans that anything will even be deleted from the mathlib3 repository!
Scott Morrison (Jul 27 2023 at 07:33):
I've closed.
Last updated: Dec 20 2023 at 11:08 UTC