-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Add Matplotlib Journey online course to external resources #30032
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
Add Matplotlib Journey online course to external resources #30032
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for opening your first PR into Matplotlib!
If you have not heard from us in a week or so, please leave a new comment below and that should bring it to our attention. Most of our reviewers are volunteers and sometimes things fall through the cracks.
You can also join us on gitter for real-time discussion.
For details on testing, writing docs, and our review process, please see the developer guide
We strive to be a welcoming and open project. Please follow our Code of Conduct.
This comment was marked as outdated.
This comment was marked as outdated.
I think spam bots have existed ever since online forums have existed. On GitHub, you can report them using a link in their profile (at the bottom of the left-hand panel). |
First time I see some on Github, but thanks for the info. |
Is this a paid course? If so has anyone on the Matplotlib team taken it so we know it is reasonable? While we have linked to paid resources in the past, I think that is mostly for people we know from the project. |
Yes it is. @story645 has access to it, and I can give access to anyone in the matplotlib team (I only need an email), as I already said to @tacaswell by email. |
I think we should be cautious about linking commercial resources, including books, as it gives the impression we think these resources are "good". I'm not at all saying that this particular contribution is not "good" or not "worth the money". However, I think we should reconsider linking to commercial resources, or if we do, the "Resources" page should have a disclaimer that we are listing these for informational purposes, and don't endorse any commercial links on the page. |
@JosephBARBIERDARNAL has a bunch of third party packages and his collaborator (Yan Holtz) is the creator of the python graph gallery which is listed on the external resources page.
What about a paid resources section with a disclaimer there? Though we've never explicitly endorsed the free resources either. I'm also gonna loop in the @matplotlib/steering-council since I sent 'em an email about this as a follow up to last weeks call discussion. |
Fundamentally, listing something in "External resources" is a kind of endorsement. We would not add random stuff without knowing it is any good. There's no difference in whether it's free or paid content. What we should do is
|
The third party packages page has:
Maybe a mash up:
We should also have that for the third party page - I think the current standard is more or less that it's a documented and tested installable package. |
I'd be fine with that, if the members who approve actually view the material to vet it. However, I won't be watching 30 h of commercially available material unless I am also paid, and I wouldn't expect anyone else to do that - even textbook reviewers typically get honorariums for their time. I'd also probably be OK if we wanted to have "sponsored content" where someone pays the project a nominal fee to include an advertisement that has clearly has no endorsement from us |
So skimming through mpl-journey, I think they've got a clearer definition of figure and axes than we do in our docs 😅 and they've got a clean scaffolded and chunked way of unrolling the information about mpl. I think for their audience, it's nicely motivated - axes has methods, want to make chart x? here's the methods on axes for that (with lots of interactivity that we can't have on our docs) - while still being concept first and wrapping up each section with quizzes and small modules. They also weave in visualization concepts/advice/best practices in a way I'd find out of scope for our docs. Which they're covering:
tldr: I think it's a solid well crafted resource that I could see being especially good for folks who aren't scientists. ETA: I guess I should add the disclaimer that I have no financial stake/conflict of interest w/ this resource, I've just interacted with both Yan and Joseph on socials b/c they've been sharing a lot of "how to make pretty mpl" for I think years now. |
The original offer was for an affliate link/coupon code but that was shut down b/c of concerns about tracking. |
Thanks @story645 ! Some other things that might be relevant:
|
You don't have to do a tutorial / read a book completely to get a sense of quality. In fact, I believe that the assessment can often be done quite quickly by looking at a few passages. Therefore, I'd claim that paying for review in this specific context is barking up the wrong tree. It would come with it's own set of challenges. I would start with the base assumption that the topic can be handled like any other review in the sense that everybody is free to decide what they want to review. Only if such topics (of which we'll still have very few), then we'd need to discuss other approaches. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We are over-thinking this.
If we want to follow up and add (paid) markers to everything paid in this list, we should make a new issue and have the discussion there.
thanks! |
I'll merge based on two positive reviews. |
PR summary
Following discussions with @tacaswell and @story645, I have added Matplotlib Journey, an online course on matplotlib, in the external resources.
Please let me know if any changes need to be made (for example, to make it clear that this is a paying resource).
PR checklist