search

wkearn / RasterIO.jl

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

Suggestion: extend fetch #19

Closed marciolegal closed 9 years ago

marciolegal commented 9 years ago

Hi,

That way one could simply use fetch(raster) instead of RasterIO.fetch(raster). At the moment, using fetch(raster) doesn't error out and returns the same Raster object which is not very helpful and can possibly create confusion by users in the future. If you agree with this I will try to make this work for all fetch methods and make a PR this weekend.

yeesian commented 9 years ago

Exporting names is easy. But we haven't yet decided whether we really want to settle on the method names fetch and update, or to disambiguate them from base julia methods by renaming them to fetchraster and updateraster (like the other IO methods instead).

marciolegal commented 9 years ago

If not planing future special functionality for these base functions, renaming them Base.fetch and Base.update sounds like a good option. Better integration with the Language, somewhat like R works. Does it make sense in Julia?