Opened 15 years ago

Last modified 11 years ago

#57 closed task

PIL vs imagemagick — at Version 10

Reported by: Will Kahn-Greene Owned by: Christopher Webber
Priority: trivial Milestone:
Component: documentation Keywords:
Cc: Parent Tickets:

Description (last modified by Will Kahn-Greene)

We're having some issues with getting PIL installed into the development environment. Chris talked with the deviantart folks and discovered they're not using PIL--they're using imagemagick.

This issue is for the following:

  1. researching the pros/cons of switching
  2. checking to see if there are other options
  3. if it seems like a good idea and the other options aren't enticing, looking at switching GNU MediaGoblin over to using imagemagick
  4. if 3 happens, then updating all the docs

Change History (14)

comment:1 by Christopher Allan Webber, 15 years ago

If we are doing imagemagick (good idea) we'll probably need to implement http://bugs.foocorp.net/issues/303 for when the queuing storage systems are remote.

comment:2 by Elrond, 15 years ago

Milestone: → 0.0.3
Owner: set to Christopher Webber

meeting result: [C cwebber] Imagemagick support

comment:3 by Christopher Allan Webber, 15 years ago

Priority: Normal → Low

Moving this to low because PIL seems to be working okay for now.

The one issue seems to be that Windows users have some problem with PIL?

comment:4 by Christopher Allan Webber, 15 years ago

Milestone: 0.0.3 → 0.0.4

comment:4 by Will Kahn-Greene, 15 years ago

Milestone: 0.0.4 → 0.0.5

We release 0.0.4, so I'm bumping this to 0.0.5.

comment:5 by Christopher Allan Webber, 15 years ago

Component: → Documentation
Status: New → Closed

Updated to suggest installing PIL. Easier than dealing with development header rigamarule.

comment:6 by Christopher Allan Webber, 15 years ago

Owner: set to Christopher Webber
Status: Closed → New

Arg! Updated the wrong ticket!

comment:7 by Christopher Allan Webber, 15 years ago

Milestone: 0.0.5 → 0.1.0

comment:8 by Christopher Allan Webber, 15 years ago

Milestone: 0.1.0 → 0.2.0

comment:9 by Christopher Allan Webber, 15 years ago

Milestone: 0.2.0 → 0.2.1

comment:7 by Christopher Allan Webber, 15 years ago

Imagemagick might still be a good idea. Before we can move forward with this though, we have to figure out a safe way to do it. Options afaict are:

  • Use one of the wrapper libraries. There are a few of them, I don't have them listed here offhand. But they seem like they might be fragile... I'm not sure.
  • Use the command line using subproces.Popen. Supposedly safe because the args are in a list and also it doesn't run a shell. Is it safe enough?
  • Use ctypes to bind to imagemagick on our own.

We also want to see: just how much faster is it? Fast enough to switch? I suspect yes...

comment:8 by Will Kahn-Greene, 14 years ago

The original url for this bug was ​http://bugs.foocorp.net/issues/332 .
Relations:
#140: related

comment:9 by Elrond, 14 years ago

Milestone: 0.2.1

As this is low prio, we likely wont handle this for the next release.

comment:10 by Will Kahn-Greene, 14 years ago

Description: modified (diff)

There's another package called "pillow" which is PIL, but with better packaging. We could also switch to that.

Note: See TracTickets for help on using tickets.