Carlo de Franchis comments

Results 23 comments of


                                            Carlo de Franchis

Pull requests to disparity estimation codes to expose methods through a shared library (while maintaining the binary interface).

We've made a first minimal working version for `mgm_multi` here: https://github.com/gfacciol/mgm/tree/ctypes

Support black format for all Python files

Hi @lionlai1989, thanks for contributing! Is there a way to configure black so that the code style stays closer to that of the original codebase? I think that in its...

s2p ValueError: Number of columns or rows must be non-negative - with small tile_size (~<2000)

Hi @Vierod, this error usually happens when the input ROI is not entirely contained in the reference input image. Could you please share your input `config.json` and the dimensions of...

Resulting DSM has black tiles

Thank you @PratyushTripathy for reporting this issue. I'll reproduce the problem (I have these two Cartosat images) to see what's happening.

Out of Memory for big geotiffs

@kanishk-aidash to speed things up while keeping the memory usage as low as possible during the stereo matching step, you can remove the `max_processes` parameter from the input json file,...

Out of Memory for big geotiffs

@kanishk-aidash could you please add `"use_srtm": true` the input config json file? This should help

Run on collection of images? MVS example?

Hi @dbonafilia, these angles are computed with the rpcm package (`pip install rpcm`). For example using the function `rpcm.angle_between_views`: https://github.com/centreborelli/rpcm/blob/master/rpcm/__init__.py#L196

localization_iterative max number of iterations 100 is too small

Hi @drzhouq, could you please provide an example? Thanks

localization_iterative max number of iterations 100 is too small

No need to share an image, a list of RPC coefficients would be enough. I'd like to reproduce your issue. In general, the loop converges in one or two iterations.

--satellite Landsat-7 option actually downloads Landsat-8 images

Thank you @chlsl for reporting this issue. For Landsat-7 and older Landsats the only available catalog is GCloud, which you can use by adding the option `--api gcloud` to your...