Context Navigation

← Previous Change
Ticket History
Next Change →

Warning: Can't synchronize with repository "(default)" (/home/git/ome.git does not appear to be a Git repository.). Look in the Trac log for more information.

Changes between Initial Version and Version 5 of Ticket #2597

Timestamp:: 11/01/12 10:16:47 (12 years ago)
Author:: wmoore
Comment:

Legend:

: Unmodified
: Added
: Removed
: Modified

Ticket #2597
- Property Owner set to jmoore
- Property Status changed from new to closed
- Property Resolution changed from to fixed
- Property Remaining Time changed from 0.25 to 0

Ticket #2597 – Description

-                      initial
+                      v5
 Depending on how PIL was installed, it may should be imported as:
+Depending on how PIL was installed, it may should be imported as {{{import Image}}} or {{{ from PIL import Image}}}.
+{{{import Image}}}
+However, in some cases, BOTH may work and cause failures: See #9790.
+Therefore it is essential that they are done IN THE SAME ORDER every time.
+or
+In all OMERO uses, both should be attempt in a try/except block in this order:
+{{{ from PIL import Image}}}
+{{{
+try:
+    from PIL import Image, ImageDraw, ImageFont     # see ticket:2597
+except: #pragma: nocover
+    try:
+        import Image, ImageDraw, ImageFont          # see ticket:2597
+    except:
+        logger.error('No PIL installed')
+}}}
 In all OMERO uses, both should be attempt in a try/except block. Searching for any import of PIL should also show this ticket number:
+Searching for any import of PIL should also show this ticket number:
 {{{
+grep -r PIL | grep import
+$ git grep "import Image"
+}}}
+Therefore, this should show NOTHING:
+{{{
+:OmeroPy will$ git grep "import Image" | grep -v ticket:2597 | grep -vE "(ImageI|ImageAnnotationLinkI|ImageWrapper|ImageColumn|ImageData)"
 }}}