Change folder of "execution" about rio-cogeo HOT 11 CLOSED

the origin bug should be fixed in 3.5.1 (which should be on pypi in couple minutes). I'll open another PR later to add a new option for user to pass a temporary directory

from rio-cogeo.

@thaarhoff is your source_file a local file?

maybe you can try to change CPL_TMPDIR environment variable to point to a specific directory

is. target_file in /tmp/....

by default we will create a temporary file in the same directory as the target destination https://github.com/cogeotiff/rio-cogeo/blob/main/rio_cogeo/cogeo.py#L28-L40

from rio-cogeo.

Thanks for the quick response.
rio is executed in /opt/images. The source file is also in that folder. The target file would be in a (already existing) subfolder.

/opt/images < execution
/opt/images/source_file
/opt/images/result/target_file

I'll set the environment variable and come back.

from rio-cogeo.

Before rio execution

/# du --max-depth=1
68028460        ./opt
8       ./tmp

echo $CPL_TMPDIR > /opt/geoimages/result

Starting rio
Reading input: /opt/geoimages/input.tif
and tmp starts rising again.

/# du --max-depth=1
68028460        ./opt
918124  ./tmp

from rio-cogeo.

And what's in tmp ?

from rio-cogeo.

Before, tmp is empty.
There is a tmp-tif file being generated

And that is not the filename of the target_file

from rio-cogeo.

Interesting So the file is created in tmp

Can you make sure that --allow-intermediate-compression is used

from rio-cogeo.

I added the option. I mean - it raises slower. Still the file is created in tmp.
3.4GB at the moment.

And evicted at 12GB, since /vda1 is full. While the PVC with 400GB is untouched :D

from rio-cogeo.

🤦 I see!
I guess in https://github.com/cogeotiff/rio-cogeo/blob/main/rio_cogeo/cogeo.py#L33, is_file() will only return True when we pass a valid (existing) file.

I'll fix this and release a new version 🙏

from rio-cogeo.

Thank you for investigating!
I'll look forward to it.

from rio-cogeo.

Works. Thank you for the quick fix <3

from rio-cogeo.