Opened 8 years ago

Closed 6 months ago

#4464 closed enhancement (wontfix)

[PATCH] Add -srcnodata to gdal_fillnodata.py

Reported by: olt Owned by: warmerdam
Priority: normal Milestone: closed_because_of_github_migration
Component: Utilities Version: unspecified
Severity: normal Keywords:
Cc: olt

Description

This patch adds a -srcnodata option to gdal_fillnodata.py, similar to the same option in gdalwarp. This can be used to fill nodata values in raster files where this value is not set.

Attachments (1)

gdal_fillnodata-srcnodata.patch (1.7 KB) - added by olt 8 years ago.

Download all attachments as: .zip

Change History (5)

Changed 8 years ago by olt

comment:1 Changed 8 years ago by olt

A note to the patch: The elif arg == '-mask' part is two times in the code, so i changed one of the statements to match the new option.

comment:2 Changed 4 years ago by gislars

I'm wondering why this patch didn't get picked up. Missing it is not a big deal, but it would be really nice to have this option.

comment:3 in reply to:  2 Changed 4 years ago by Even Rouault

Summary: Add -srcnodata to gdal_fillnodata.py[PATCH] Add -srcnodata to gdal_fillnodata.py

Replying to gislars:

I'm wondering why this patch didn't get picked up. Missing it is not a big deal, but it would be really nice to have this option.

srcband.SetNoDataValue?(src_nodata) will modify the source dataset (well, presumably only in the .aux.xml file, but still), which is generally not intended. A cleaner way would be to have a SRCNODATA=val option passed to gdal.FillNodata?() in the options argument.

comment:4 Changed 6 months ago by Even Rouault

Milestone: closed_because_of_github_migration
Resolution: wontfix
Status: newclosed

This ticket has been automatically closed because Trac is no longer used for GDAL bug tracking, since the project has migrated to GitHub?. If you believe this ticket is still valid, you may file it to https://github.com/OSGeo/gdal/issues if it is not already reported there.

Note: See TracTickets for help on using tickets.