Opened 8 years ago

Closed 8 years ago

#6664 closed defect (fixed)

Update cubic OpenCL calculations

Reported by: sprice Owned by: warmerdam
Priority: normal Milestone:
Component: default Version: 2.1.1
Severity: normal Keywords:
Cc:

Description

The changes here: ​https://trac.osgeo.org/gdal/changeset/26492/ Should be applied to the OpenCL kernel also. A diff would look like this: 

$ diff gdal-2.1.1/alg/gdalwarpkernel_opencl.c gdal-2.1.1.new/alg/gdalwarpkernel_opencl.c
938,941c938,941
<     "return   (  -f0 +    f1  - f2 + f3) * dist3\n"
<            "+ (2.0f*(f0 - f1) + f2 - f3) * dist2\n"
<            "+ (  -f0          + f2     ) * dist1\n"
<            "+             f1;\n"
---
>      "return (  f1\n"
>       "+ 0.5f * (dist1*(f2 - f0)\n"
>               "+ dist2*(2.0f*f0 - 5.0f*f1 + 4.0f*f2 - f3)\n"
>               "+ dist3*(3.0f*(f1 - f2) + f3 - f0)));\n"

Change History (1)

comment:1 by Even Rouault, 8 years ago

Resolution: → fixed
Status: new → closed

In 35531:

OpenCL warper: update cubicConvolution to use same formula as CPU case (patch by Seth Price, fixes #6664)

Note: See TracTickets for help on using tickets.