otb issueshttps://gitlab.orfeo-toolbox.org/orfeotoolbox/otb/-/issues2020-02-05T13:44:54Zhttps://gitlab.orfeo-toolbox.org/orfeotoolbox/otb/-/issues/1980Upgrade GDAL, PROJ and GEOTIFF in Superbuild2020-02-05T13:44:54ZCédric TraizetUpgrade GDAL, PROJ and GEOTIFF in SuperbuildAt some point we should upgrade GDAL version in the Superbuild to version 3.X.
Currently GDAL 3.X is supported (see !511), but the Superbuild version is 2.4.1. Upgrading GDAL to version 3.X also means we have to upgrade PROJ to version ...At some point we should upgrade GDAL version in the Superbuild to version 3.X.
Currently GDAL 3.X is supported (see !511), but the Superbuild version is 2.4.1. Upgrading GDAL to version 3.X also means we have to upgrade PROJ to version 6.0 and GEOTIFF to a recent version e.g. 1.5.1 (because GEOTIFF depends on PROJ). Note that we don't have to upgrade OSSIM (fortunately because there is no new version) because it depends on GEOTIFF, but not directly on PROJ.7.1.0Cédric TraizetCédric Traizethttps://gitlab.orfeo-toolbox.org/orfeotoolbox/otb/-/issues/1532Synchronized writers2020-01-16T15:14:35ZJulien MichelSynchronized writersIn Orfeo ToolBox, there are several filters that produces multiple output images. For now, writing all those output to separate files is inneficient, since the streaming mechanism will end up executing the filter on the whole dataset sev...In Orfeo ToolBox, there are several filters that produces multiple output images. For now, writing all those output to separate files is inneficient, since the streaming mechanism will end up executing the filter on the whole dataset several time.
The workaround is to pack all those outputs in a single output image using `ImageListToVectorImageFilter`, but this is not a good solution in many cases : the different ouptut might represent very different things that should not resid in a single `VectorImage`, and sometimes the different ouptuts have different types (for instance if their is an output mask along with a scalar image).
We need to enable synchronization between writer, so that once a stream is calculated by the filter, all corresponding output files are updated.7.1.0Cédric TraizetCédric Traizethttps://gitlab.orfeo-toolbox.org/orfeotoolbox/otb/-/issues/1931ExtractROI application testing2019-10-28T10:05:32ZCédric TraizetExtractROI application testingCurrently, the `ExtractROI` application is only tested in the case where the output is the same as the input, for example if its parameters are not compliant with the input image. The tests are:
- `apTvUtExtractROISizeTooBig` : `sizex` i...Currently, the `ExtractROI` application is only tested in the case where the output is the same as the input, for example if its parameters are not compliant with the input image. The tests are:
- `apTvUtExtractROISizeTooBig` : `sizex` is too big,
- `apTvUtExtractROISizeTooBigExtent` : `mode.extent.lrx` is not in the input image,
- `apTvUtExtractROISizeNULL` : `sizex` is 0,
- `apTvUtExtractROIRightInputFile` : no parameters given,
- `apTvUtExtractROIComplexInputFile` : no parameters given, complex input.
These tests are useful, but it is even more important to test the case where the application works correctly. I suggest to add the 9 following tests:
- Mode standard (test with `sizex` `sizey` `startx` and `starty`)
- Mode Fit:
- case with a reference image
- case with a reference vector
- Mode Extent (test with `mode.extent.ulx`, `mode.extent.uly` ,`mode.extent.lrx` and `mode.extent.lry`:
- case with unit = `pxl`
- case with unit = `phy`
- case with unit = `lonlat`
- Mode Radius (test with `radius.r` `radius.cs` and `radius.cy` :
- case with center unit `pxl` and radius unit `pxl`
- case with center unit `phy` and radius unit `phy`
- case with center unit `lonlat` and radius unit `pxl`
Also set the channel parameter (`cl`) for these tests, to also test this functionality.
This is a lot of new tests, but if we choose the parameters smartly we can use only one baseline for all tests!
`ExtractROI` is probably one of the most used application of the library, so I think it deserves to be tested exhaustively (radius and extent modes have been broken for a while before !537). This feature request is not about testing the `MultiChannelExtractROI` filter, but testing the code specific to the application (parameter management).7.1.0Cédric TraizetCédric Traizet