Deprecate parameter "web_optimized" of cogeo.cog_translate Python function.
Notes: I would like guidelines about:
Deprecation syntax in apidoc: I am not very familiar with Python docstring, so I have tried to apply Sphinx deprecated directive. But I am not sure if it was the right move.
I also tried to update the cli:
I have not deprecated web-optimized option there, because user cannot pass a tms id (like "WebMercatorQuad"), so I suppose web-optimized will remain prominent there.
However, I tried to rework internals to depend less on it. But by doing so, I stumbled upon a conflict: it looks like overview-blocksize is silently ignored in case tms or web-optimized is specified. I have let it as is, but maybe there is something to change here (echo a warning to the user ? change some logic ?)
Changes discussed in #277
Deprecate parameter "web_optimized" of
cogeo.cog_translate
Python function.Notes: I would like guidelines about:
web-optimized
option there, because user cannot pass a tms id (like "WebMercatorQuad"), so I supposeweb-optimized
will remain prominent there.overview-blocksize
is silently ignored in casetms
orweb-optimized
is specified. I have let it as is, but maybe there is something to change here (echo a warning to the user ? change some logic ?)