Opened 12 years ago
Closed 5 years 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)
Change History (5)
by , 12 years ago
Attachment: | gdal_fillnodata-srcnodata.patch added |
---|
comment:1 by , 12 years ago
follow-up: 3 comment:2 by , 9 years ago
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 by , 9 years ago
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 by , 5 years ago
Milestone: | → closed_because_of_github_migration |
---|---|
Resolution: | → wontfix |
Status: | new → closed |
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.
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.