i.sentinel.import: wrong resolution set for latlong locations about grass-addons

[veroandreo]

It could work, did you try if you addons works properly?

Yes, It works as expected. It's r.import who does the resolution estimation for latlong locations.

[neteler]

Probably we need a change here for LongLat:

grass-addons/grass7/imagery/i.sentinel/i.sentinel.import/i.sentinel.import.py

Line 229 in f5b90b7

args['resolution'] = 'value'

[veroandreo]

ping @landam :)
any chance to have a look at this?

[lucadelu]

Probably we need a change here for LongLat:

grass-addons/grass7/imagery/i.sentinel/i.sentinel.import/i.sentinel.import.py

Line 229 in f5b90b7

args['resolution'] = 'value'

The change should be done in _raster_resolution function, the problem it is to get the right value in degree according to the position on the Earth surface.

Could geodesic lenght be useful?

[neteler]

... the problem it is to get the right value in degree according to the position on the Earth surface.

Could geodesic lenght be useful?

We have length_geodesic(), maybe interesting?

• doc: https://grass.osgeo.org/grass79/manuals/libpython/pygrass.vector.html?highlight=length_geodesic#pygrass.vector.geometry.Line.length_geodesic
• code: https://github.com/OSGeo/grass/blob/90634b76e3bcf9695a844cf3f6581975fa7283d9/lib/python/pygrass/vector/geometry.py#L848

[veroandreo]

In this commit in the i.landsat.import addon I propose a simple approach, i.e., using r.import's resolution = estimated when the target location is longlat. What do you think @neteler and @lucadelu ?

[lucadelu]

It could work, did you try if you addons works properly?