python-visualization / folium

Python Data. Leaflet.js Maps.
https://python-visualization.github.io/folium/
MIT License
6.87k stars 2.22k forks source link

Closember participation #1638

Closed ocefpaf closed 1 year ago

ocefpaf commented 1 year ago

@Conengmo what do you think? Take a look at https://closember.org/ and let me know.

I believe there are some stale PRs, already resolved or no longer relevant issues that could use a clean-up!

Conengmo commented 1 year ago

Love it! We could definitely use some cleaning up, and merging good PRs we got waiting. Setting a goal for this month sounds like a good idea to gain some momentum. And fortunately I have some time to spend as well.

Few things we could focus on:

For PRs:

I'm not sure about closing old issues. I think many of them are still valid, it's more that what they propose is not getting done. Maybe it's a good idea to close them since they didn't get traction, and offer anybody that reads the closed issue to reopen if they are actually able to work on it.

To be done: a shortlist of what would be good to have done first.

Conengmo commented 1 year ago

How users can help

Let us know in this issue thread!

Close your own stale issues or PRs

If you have an open issue or PR on this repo that you don't support anymore, you can close it yourself. Please include a closing message with a (one sentence) explanation.

Test open PR's

Look at the open PR's, pick one you are interested in, pull that code and test any or all of the following:

Then include your test report in the PR! The goal is for us maintainers to able to merge or close the PR with less effort.

Help triage issues

We have many open issues without any replies yet. You could help by flagging issues that can be closed:

Conengmo commented 1 year ago

Ambitions

Conengmo commented 1 year ago

Look at that, no open issues or PRs on branca 🎉

Conengmo commented 1 year ago

Branca got a mention from the closember twitter account: https://nitter.net/ClosemberDev/status/1589522298742128641#m

Conengmo commented 1 year ago

I also liked this message, congratulating us with reaching zero on branca and 27 closed PRs on folium: https://nitter.net/ClosemberDev/status/1590774786745307136#m

ocefpaf commented 1 year ago

We should schedule a call for a virtual toast to celebrate Closember ;-p

Conengmo commented 1 year ago

We have the most closed PRs on the board, which is a nice reflection of the amount of work we did!

https://nitter.net/ClosemberDev/status/1592651419059789824#m

afbeelding

ocefpaf commented 1 year ago

which is a nice reflection of the amount of work we did!

You mean you did! I'm pretty much watching from the sidelines. Thanks for the effort you put into this!! We need a new release soon to ship all the new features and bugfixes.

Conengmo commented 1 year ago

Well we made the top of the list, so we can cross off that goal as well. Let's see how long we can stay there.

afbeelding

ocefpaf commented 1 year ago

@Carreau is the mastermind behind closember. I'm pretty sure he has a "mission accomplished" feeling right now.

Carreau commented 1 year ago

😍 If you loved it and it put you at ease, then you are the winner ! I likely need to tweak the code of the website to only count on the third closeboard projects that actually had opened PRs so that you are at the top of two closeboard.

If you don't mind once this is over I'd love to send you some questions and write a blog post on how you liked focusing on finishing things.

Conengmo commented 1 year ago

Thanks for organizing this @Carreau! It's much appreciated.

If you don't mind once this is over I'd love to send you some questions and write a blog post on how you liked focusing on finishing things.

Sure thing, I'd be happy to help!

Conengmo commented 1 year ago

Only 7 unlabeled open issues remaining! https://github.com/python-visualization/folium/issues?q=is%3Aopen+is%3Aissue+no%3Alabel I'm confident we can resolve those before the month is over. We closed around 150 issues in the past few weeks :sweat_smile:

Conengmo commented 1 year ago

We got all unlabeled open issues! I'll tick off the goal to triage and label all open issues. Remaining open issues contain valid feature ideas/requests, still to be solved bugs and valid requests for better documentation in certain areas.

Conengmo commented 1 year ago

Looks like there is no more low hanging fruit left at the moment. I'll go ahead and compile a changelog for the upcoming release. It's going to be a big one!

Conengmo commented 1 year ago

I added the changelog for the upcoming release: https://github.com/python-visualization/folium/blob/main/CHANGES.txt. Looks like we are good to go with this huge release! @ocefpaf could you bake a 0.14.0 release? I'll not merge new stuff until then.

Conengmo commented 1 year ago

It's still some hours for this last day of Closember, also depending on your timezone, but since I'm busy coming days I'm going to call it now already and say folium made the top of the closeboard:

afbeelding

So I'll go ahead and mark that final goal as well :) Feels really good to have the project in a much better state now.

ocefpaf commented 1 year ago

I'm freezing this here for posterity:

Screenshot from 2022-12-01 09-48-55

Conengmo commented 1 year ago

I'm freezing this here for posterity:

That looks great, thanks for sharing!