search

wkearn / RasterIO.jl

Simple Raster Formats for Julia
Other
18 stars 5 forks source link

Remove image structure from REPL display #17

Closed yeesian closed 9 years ago

yeesian commented 9 years ago

@marciolegal was right afterall, I don't think we should require them:

julia> raster = RasterIO.openraster(filename)
RasterIO.Raster (Driver: PNG/Portable Network Graphics)
Metadata: Error showing value of type RasterIO.Raster:
ERROR: Unable to find key AREA_OR_POINT
 in getmetadataitem at /Users/yeesian/.julia/v0.4/RasterIO/src/gdal/utilities.jl:108
 in show at /Users/yeesian/.julia/v0.4/RasterIO/src/raster_types.jl:25
 in anonymous at show.jl:1278
 in with_output_limit at /Applications/Julia-0.4.0.app/Contents/Resources/julia/lib/julia/sys.dylib
 in showlimited at show.jl:1277
 in writemime at replutil.jl:4
 in display at REPL.jl:114
 in display at REPL.jl:117
 in display at multimedia.jl:151
 in print_response at REPL.jl:134
 in print_response at REPL.jl:121
 in anonymous at REPL.jl:624

We should also return empty strings rather than error-ing out, but that's left for another time

yeesian commented 9 years ago

@visr okay to merge this, before I tag a new version in METADATA?

visr commented 9 years ago

Yes go ahead, definitely don't want errors on showing valid rasters.