Opened 6 years ago

Closed 4 years ago

#4379 closed defect (fixed)

Raster unit tests failing with GDAL master

Reported by: Algunenano Owned by: dustymugs
Priority: medium Milestone: PostGIS 3.1.0
Component: raster Version: master
Keywords: Cc:

Description

I'm using both GDAL and PROJ master.

Run Summary:    Type  Total    Ran Passed Failed Inactive
              suites     12     12    n/a      0        0
               tests     65     65     64      1        0
             asserts  45905  45905  45904      1      n/a
./test/cunit/cu_tester test_raster_compute_skewed_raster

Running test 'test_raster_compute_skewed_raster' in suite 'raster_misc'.

    FAILED - asserts -   4 passed,   1 failed,   5 total.


  1. cu_raster_misc.c:199  - CU_ASSERT_EQUAL(rt_raster_get_width(rast),2)
199             CU_ASSERT_EQUAL(rt_raster_get_width(rast), 2);
(gdb) p rast
$1 = (rt_raster) 0x555555799b70
(gdb) p *rast
$2 = {size = 0, version = 0, numBands = 0, scaleX = 1, scaleY = -1, ipX = -0.5, ipY = 0, skewX = 0.25, skewY = 0.25, srid = 0, width = 3, 
  height = 3, bands = 0x0}
(gdb) quit

This works fine with PROJ 5.2 and GDAL 2.3.2 in the same environment.

Change History (9)

comment:1 by Algunenano, 5 years ago

This now works fine with current master and also with gdal 3.0.0 + PROJ 6.1.0

comment:2 by komzpa, 5 years ago

Resolution: worksforme
Status: newclosed

Travis does not confirm either.

comment:3 by Algunenano, 5 years ago

Resolution: worksforme
Status: closedreopened

I've rebuilt with gdal release/3.0 and I reproduce it again:

  Test: test_raster_compute_skewed_raster ...FAILED
    1. cu_raster_misc.c:199  - CU_ASSERT_EQUAL(rt_raster_get_width(rast),2)

Using:

I have a feeling that this might depend on GDAL compilation options (CFLAGS / CXXFLAGS) since I couldn't reproduce it with the distribution packages but I do when building the package myself.

  C compiler:                gcc -DHAVE_AVX_AT_COMPILE_TIME -DHAVE_SSSE3_AT_COMPILE_TIME -march=native -mtune=native -O3 -pipe -fno-plt -Wall -Wextra -Wno-unused-parameter -g3 -gdwarf-4 -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer
  C++ compiler:              g++ -DHAVE_AVX_AT_COMPILE_TIME -DHAVE_SSSE3_AT_COMPILE_TIME -march=native -mtune=native -O3 -pipe -fno-plt -Wall -Wextra -Wno-unused-parameter -g3 -gdwarf-4 -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer

comment:5 by robe, 5 years ago

Note this might be related to #4499 both are failing gdalwarp. Though I don't have CUnit configured on Bessie yet.

They both have commonality of running Proj 6.1.0, however bessie is running GDAL 2.4.1 so may be more to do with proj than GDAL.

comment:6 by algunenano, 5 years ago

In 17813:

Disable raster failing tests

References #4379

comment:7 by Algunenano, 5 years ago

I've disabled that test until the root cause is investigated.

comment:8 by robe, 5 years ago

Milestone: PostGIS 3.0.0PostGIS 3.1.0

comment:9 by pramsey, 4 years ago

Resolution: fixed
Status: reopenedclosed
Note: See TracTickets for help on using tickets.