Opened 11 years ago

Closed 9 years ago

Last modified 7 years ago

#57 closed task (wontfix)

PIL vs imagemagick

Reported by: Will Kahn-Greene Owned by:
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



Subtickets

Change History (15)

comment:1 Changed 11 years ago by Christopher Allan Webber

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



comment:2 Changed 11 years ago by Elrond

Milestone: 0.0.3
Owner: set to Christopher Webber
meeting result:
[C cwebber] Imagemagick support



comment:3 Changed 10 years ago by Christopher Allan Webber

Priority: NormalLow
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 Changed 10 years ago by Christopher Allan Webber

Milestone: 0.0.30.0.4

comment:4 Changed 10 years ago by Will Kahn-Greene

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



comment:5 Changed 10 years ago by Christopher Allan Webber

Component: Documentation
Status: NewClosed
Updated to suggest installing PIL. Easier than dealing with
development header rigamarule.



comment:6 Changed 10 years ago by Christopher Allan Webber

Owner: set to Christopher Webber
Status: ClosedNew
Arg! Updated the wrong ticket!



comment:7 Changed 10 years ago by Christopher Allan Webber

Milestone: 0.0.50.1.0

comment:8 Changed 10 years ago by Christopher Allan Webber

Milestone: 0.1.00.2.0

comment:9 Changed 10 years ago by Christopher Allan Webber

Milestone: 0.2.00.2.1

comment:7 Changed 10 years ago by Christopher Allan Webber

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 Changed 10 years ago by Will Kahn-Greene

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

comment:9 Changed 10 years ago by Elrond

Milestone: 0.2.1

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

comment:10 Changed 9 years ago by Will Kahn-Greene

Description: modified (diff)

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

comment:11 Changed 9 years ago by Will Kahn-Greene

Resolution: wontfix
Status: acceptedclosed

I'm going to close this out since no one is interested in taking it on and the project has done fine without this work. If it's an issue, we can open up a new one.

Note: See TracTickets for help on using tickets.