-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
mep12 on geo_demo.py #5030
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
mep12 on geo_demo.py #5030
Conversation
Looks like that worked. |
Indeed! |
How many pylab examples do you still need to convert? IMHO it would be faster (and also energy-saving as the # of travis CI runs gets reduced), if you grouped several examples into a single PR, but this has probably already been discussed in one of your PRs. |
Not many examples left before having to reclassify them. I estimate ~15-ish right now. I decided against multiple examples in a single PR earlier. Forgot the exact reason, but I think it had to do with simplifying each PR. |
I'd be happy with 3 examples per PR. Especially now that it seems like you By the way, in case it hasn't been said, we are really grateful for this On Fri, Sep 4, 2015 at 3:33 PM, Eric Ma [email protected] wrote:
|
Thanks a lot guys! 😄 I'm happy to help as well! Btw, some nice things I've observed so far about this - The task's level was just right for my level, it gave me the extra knowledge to make my own repos continuously-integrated, and oh my, it was a great release/positive distraction to work on open source having had my paper rejected from a 3rd journal 😆 |
@ericmjl Seconding what Ben says, thank you so much for doing this! Poor documentation and out of date examples are the soft dark underbelly of OSS. Also glad to hear that this work has made you a better developer in your other work. That has been my experience as well and is one of (in my view) the best things about our community. |
No description provided.