Skip to content
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

Provide an alternative to lena.png for two examples that use it. #1581

Closed
wants to merge 1 commit into from
Closed

Conversation

ericstalbot
Copy link

I realize that lena is the standard image in the image processing world. However, I would think that most matplotlib users (myself included) are not aware of and/or don't care about standard images; and that most people would object to the pornographic origins and nature of the image. The changes will allow these people to browse the gallery and not see lena. The original lena images remain in the example sample data directory for those who are interested.

…d lena.jpg. Updated the two examples that appear in the gallery to use the new png
@WeatherGod
Copy link
Member

Do note that getting rid of lena.png (which I don't advocate), constitutes a change that requires going through the deprecation process as other people's code-bases (or testing code) may have depended upon getting the lena image data from matplotlib's module.

@ericstalbot
Copy link
Author

Thanks for the warning. I have not deleted lena.png, so it is still there for code to use. I have simply added a second set of images, and changed the two example files to read the new images. As I understand it, as long as the lena images are still in the sample data directory, then nothing will be broken.

@efiring
Copy link
Member

efiring commented Dec 10, 2012

I think I have always found both standard images--lena and the mandrill--faintly irritating, perhaps because they are so standard. We aren't using the image as part of a standardized test, so I don't see any reason to use a standard image. A portrait of a flower, or an ocean wave, or perhaps a less intense-looking animal than that mandrill--a photogenic bird, for example--would work perfectly well. Perhaps someone could contribute such a photo, with full usage rights? I rarely take photos, so I doubt I could dig up anything myself; but surely someone active in mpl is an amateur photographer with a nice image to contribute.

@dmcdougall
Copy link
Member

I enjoy photography, but rarely get time to enjoy now it though. I can look through my collection to try and find something appropriate if there's interest for it.

Of course, if anybody else has an image they'd like to donate then please feel free. Most of my photos may still be on a hard drive in the UK that didn't make it in the move to the US.

@pelson
Copy link
Member

pelson commented Dec 11, 2012

I know a photographer who would love to be able to contribute - I will post something this evening which can be donated to the mpl project and put under the same mpl license. We can collate images and have a little vote on the mpl-users mailing list if there are images made available to mpl from several sources.

@ericstalbot - thank you for following through from #1580. If somebody feels passionate enough to put in a pull request to get something changed then you will find we can almost always accommodate it.

Cheers,

@mdboom
Copy link
Member

mdboom commented Dec 11, 2012

I'm marking this as 1.2.x -- not because it's necessarily a bugfix but because otherwise it won't get pushed to the main website for some time. As was pointed out, this shouldn't break backward compatibility because we aren't removing lena.png from the tree.

@mdboom
Copy link
Member

mdboom commented Dec 11, 2012

I think it's great that this is being worked on so close to Ada Lovelace's birthday.

@WeatherGod
Copy link
Member

I couldn't resist:
http://planetrenders.net/renders/albums/userpics/11123/ada.png

I'm going to hell, aren't I?

@efiring
Copy link
Member

efiring commented Dec 11, 2012

On 2012/12/11 8:33 AM, Michael Droettboom wrote:

I think it's great that this is being worked on so close to Ada
Lovelace's birthday.

Hmmm... Googling Ada Lovelace images turns up quite a few; perhaps one
of them is suitable and free of restrictions.

@efiring
Copy link
Member

efiring commented Dec 11, 2012

On 2012/12/11 8:33 AM, Michael Droettboom wrote:

I think it's great that this is being worked on so close to Ada
Lovelace's birthday.

http://en.wikipedia.org/wiki/File:Ada_Lovelace.jpg

@ddale
Copy link
Contributor

ddale commented Dec 14, 2012

@WeatherGod
Copy link
Member

I see who will be joining me in hell...

@efiring
Copy link
Member

efiring commented Jan 10, 2013

#1599 is merged, so I am closing this.

@efiring efiring closed this Jan 10, 2013
@pelson
Copy link
Member

pelson commented Jan 11, 2013

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants