Zulip Chat Archive

Stream: general

Topic: github notifications


view this post on Zulip Sean Leather (Feb 28 2018 at 06:23):

Does anyone (with the right repository permissions) want to create streams and bots for GitHub notifications from https://github.com/leanprover/lean and https://github.com/leanprover/mathlib ?

view this post on Zulip Mario Carneiro (Feb 28 2018 at 07:14):

You should already be receiving notifications from mathlib, on the #travis stream. I doubt I can set up hooks for the lean repo since it inexplicably requires modifying the travis build script

view this post on Zulip Sean Leather (Feb 28 2018 at 07:15):

I'm referring to issues, PRs, comments, and commits, not Travis CI.

view this post on Zulip Mario Carneiro (Feb 28 2018 at 07:18):

well, I certainly can't, you need to be admin on the repo to set it up I think

view this post on Zulip Sean Leather (Feb 28 2018 at 07:20):

Exactly. :smile: Are you not an admin on mathlib?

view this post on Zulip Mario Carneiro (Feb 28 2018 at 07:30):

Nope, this is part of the reason I would prefer to move development elsewhere

view this post on Zulip Mario Carneiro (Feb 28 2018 at 07:30):

I'm just a really active contributor with push access

view this post on Zulip Sean Leather (Feb 28 2018 at 07:32):

Nope, this is part of the reason I would prefer to move development elsewhere

It sounds like there's more context to this statement that I am not aware of.

view this post on Zulip Moses Schönfinkel (Feb 28 2018 at 07:35):

Mario is going rogue.

view this post on Zulip Mario Carneiro (Feb 28 2018 at 07:51):

Mario is trying to figure out how to fork lean without making life more horrible for everyone

view this post on Zulip Sean Leather (Feb 28 2018 at 07:58):

It would be a shame to have fewer collaborators on and contributions to such a fork, were it to exist.

view this post on Zulip Simon Hudon (Feb 28 2018 at 07:59):

Is it still because of the instance situation?

view this post on Zulip Mario Carneiro (Feb 28 2018 at 08:04):

It's mostly been a politics thing - I don't think Leo is cultivating a good environment for an OSS to develop with his approach to PRs and issues management. The instance thing caused some of these problems to come to the fore, and although that's no longer directly an issue the problems have been there for a while and have not gone away. As it is, we're in a bit of a holding pattern, and I have no wish to lose collaborators and the like so I don't know whether there is a good way forward that avoids this

view this post on Zulip Simon Hudon (Feb 28 2018 at 08:06):

Did he explain why he's doing things this way?

view this post on Zulip Simon Hudon (Feb 28 2018 at 08:07):

I mean, is there some sort of dialogue going on that might help mitigate the situation?

view this post on Zulip Mario Carneiro (Feb 28 2018 at 08:09):

No, that is of course part of the problem. I have traded less than a hundred words with Leo since summer (when I met him in person)

view this post on Zulip Simon Hudon (Feb 28 2018 at 08:11):

You're in Jeremy's team right?

view this post on Zulip Mario Carneiro (Feb 28 2018 at 08:12):

I know very well why he's doing this though - he has no interest in managing general maintenance stuff and definitely doesn't want to maintain mathlib (not that I've ever asked him to), I think he's shellshocked from having to do this in lean 2. He has his projects to complete, and the other stuff is sucking out his enjoyment and stressing him out

view this post on Zulip Mario Carneiro (Feb 28 2018 at 08:12):

yes

view this post on Zulip Simon Hudon (Feb 28 2018 at 08:13):

Your team seems like such a big part of Lean. Does Leo appreciate your contribution?

view this post on Zulip Sean Leather (Feb 28 2018 at 08:14):

I think the comments on leanprover/lean#1521 are indicative of Leo's approach.

view this post on Zulip Mario Carneiro (Feb 28 2018 at 08:19):

I will let Leo answer that question himself

view this post on Zulip Simon Hudon (Feb 28 2018 at 08:20):

I'll have to look at that closely. Thanks! Btw, I see you seemed interested in coinductive types. You might be interested in my current PR: https://github.com/leanprover/mathlib/pull/71

view this post on Zulip Simon Hudon (Feb 28 2018 at 08:22):

I feel like I must sound a bit like a marriage counsellor but it seems like if he doesn't value your work, it will be hard to get him to make any kind of concession. I wonder if there's something he wishes you'd do (beside drop mathlib :stuck_out_tongue_closed_eyes: )

view this post on Zulip Sean Leather (Feb 28 2018 at 08:22):

Me: :smile:

In case it wasn't clear, I didn't mean to pick these out because I wanted to know about them. They were merely examples.

And, yes, I'm aware of your PR. And I'm impressed by it.

view this post on Zulip Sean Leather (Feb 28 2018 at 08:24):

@Simon Hudon: I think that is an example of how you go against Leo's assumptions:

On the other hand, you should not hold your breath for coinductive datatypes. They will not happen in the following years, unless someone as strong as @gebner or @Kha shows up and decides to implement this feature.

view this post on Zulip Simon Hudon (Feb 28 2018 at 08:27):

Thanks :) I can't compare myself to @Gabriel Ebner or @Sebastian Ullrich but it looks to me that something that does the job 30% of the way is better than nothing (at least for me). So let's just see how far I can get with it.

view this post on Zulip Mario Carneiro (Feb 28 2018 at 08:28):

I'm not totally sure about Simon's approach as a final solution for all coinductive datatypes, including nested inductive/coinductives and other things that might be nice, so I'm not sure how much it is worth focusing on extending the capabilities of the package if we are going to move to something else later

view this post on Zulip Mario Carneiro (Feb 28 2018 at 08:28):

Specifically, I want to see if the BNF approach used in isabelle also works for lean

view this post on Zulip Mario Carneiro (Feb 28 2018 at 08:29):

That said, I am certainly happy to have a decent coverage of basic coinductive types and will merge Simon's PR when it stabilizes

view this post on Zulip Simon Hudon (Feb 28 2018 at 08:31):

Thanks for the comment. I believe you're referring to Johannes' paper. I'm still learning that stuff so when I wrap my head around that BNF stuff, I may well try it out.

view this post on Zulip Sebastian Ullrich (Feb 28 2018 at 10:00):

Decoupling Lean is certainly on both Leo's and my wish list. As Mario wrote in another topic, things like a command for coinductive types should be written purely in Lean in the future, and it shouldn't matter whether this happens in the core lib or a separate package. I'll get to decoupling users from the Lean master... sometime this week.

view this post on Zulip Sebastian Ullrich (Feb 28 2018 at 10:01):

To go back to the original topic, we can add commit RSS feeds for arbitrary repos. Personally, that would be sufficient for me since I already get e-mails about all the other things by following the repos on Github.

view this post on Zulip Kevin Buzzard (Feb 28 2018 at 10:38):

To go back to the original topic, we can add commit RSS feeds for arbitrary repos. Personally, that would be sufficient for me since I already get e-mails about all the other things by following the repos on Github.

Lean and mathlib, please :-)

view this post on Zulip Sebastian Ullrich (Feb 28 2018 at 11:23):

Ah, you need a Python bot running somewhere for that. Yeah, no.


Last updated: May 08 2021 at 10:12 UTC