GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Provide an alternative to lena.png for two examples that use it. by ericstalbot · Pull Request #1581 · matplotlib/matplotlib

Conversation

@ericstalbot

@ericstalbot ericstalbot commented Dec 10, 2012

Copy link
Copy Markdown

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.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
ericstalbot
Provided mandrill.png and mandrill.jpg as alternatives to lena.png an…
ca63b48
…d lena.jpg. Updated the two examples that appear in the gallery to use the new png
@WeatherGod

WeatherGod commented Dec 10, 2012

Copy link
Copy Markdown
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.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@ericstalbot

ericstalbot commented Dec 10, 2012

Copy link
Copy Markdown
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.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@efiring

efiring commented Dec 10, 2012

Copy link
Copy Markdown
Member

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.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@dmcdougall

dmcdougall commented Dec 10, 2012

Copy link
Copy Markdown
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.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@pelson

pelson commented Dec 11, 2012

Copy link
Copy Markdown
Member

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,

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@mdboom

mdboom commented Dec 11, 2012

Copy link
Copy Markdown
Member

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.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@mdboom

mdboom commented Dec 11, 2012

Copy link
Copy Markdown
Member

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

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@WeatherGod

WeatherGod commented Dec 11, 2012

Copy link
Copy Markdown
Member

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

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

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@efiring

efiring commented Dec 11, 2012

Copy link
Copy Markdown
Member

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.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@efiring

efiring commented Dec 11, 2012

Copy link
Copy Markdown
Member

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

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@ddale

ddale commented Dec 14, 2012

Copy link
Copy Markdown
Contributor

I vote for a variant of http://news.bbc.co.uk/olmedia/1785000/images/_1788845_statue300ap.jpg

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@WeatherGod

WeatherGod commented Dec 14, 2012

Copy link
Copy Markdown
Member

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

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@ivanov ivanov mentioned this pull request Dec 15, 2012
Ada Lovelace and Grace Murray Hopper images in place of Lena #1599
Merged
@efiring

efiring commented Jan 10, 2013

Copy link
Copy Markdown
Member

#1599 is merged, so I am closing this.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@efiring efiring closed this Jan 10, 2013
@pelson

pelson commented Jan 11, 2013

Copy link
Copy Markdown
Member

👍

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@hugovk hugovk mentioned this pull request Sep 4, 2014
Replace Lena with free image python-pillow/Pillow#884
Closed
@jenshnielsen jenshnielsen mentioned this pull request Aug 15, 2015
Remove lena.jpg/png, it is undistributable and violating copyright #4927
Closed
This was referenced Jul 31, 2016
License problem: please remove doc/image/lena.jpg torch/nn#854
Closed
License: please remove non-free lena image torch/image#172
Closed
@davide125 davide125 mentioned this pull request Jan 7, 2023
Consider replacing lena.tif with a free image Mathics3/mathics-core#728
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

v1.2.x

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

7 participants

@ericstalbot @WeatherGod @efiring @dmcdougall @pelson @mdboom @ddale
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.