Opened 11 years ago
Closed 7 years ago
#2171 closed enhancement (fixed)
rfe r.in.gdal: auto-calc memory= value
Reported by: | hamish | Owned by: | |
---|---|---|---|
Priority: | normal | Milestone: | 7.4.0 |
Component: | Raster | Version: | svn-trunk |
Keywords: | r.in.gdal, GDAL_CACHEMAX | Cc: | |
CPU: | All | Platform: | All |
Description
Hi, a r.in.gdal wish refined with hints from EvenR,
it would be nice if the r.in.gdal memory= option allowed the user to specify memory=auto, in which case the value was automatically calculated from "tile_height * raster_width * band_count * sizeof_datatype" rounded up to the next megabyte or so for good luck.
This will require changing the parser check to manual testing, but that shouldn't be too difficult. The default should remain at the current gdal-internal 40MB for low-resource users who want to import very large files.
for now I've put a hint in the wiki:
thanks, Hamish (back soon)
Attachments (1)
Change History (8)
comment:1 by , 11 years ago
comment:2 by , 10 years ago
r.in.gdal: "memory" (cache) parameter added in trunk r61362/r61364 and in relbr7 r61363/r61458. Leads to notable (!) speed-up. Ticket could be closed.
Concerning extra wish:
Attached a patch for r.out.gdal. For smaller files no improvement, perhaps useless? Needs to be tested with large file export.
by , 10 years ago
Attachment: | r.out.gdal.memparm.diff added |
---|
patch to add memory parameter to r.out.gdal
comment:3 by , 9 years ago
Milestone: | 7.0.0 → 7.0.5 |
---|
comment:5 by , 8 years ago
Milestone: | 7.0.5 → 7.3.0 |
---|
comment:7 by , 7 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
No activity last 3 years, closing. Feel free to reopen if needed.
Replying to hamish:
As a side note: wouldn't a memory parameter also be beneficial for r.out.gdal ?
Moritz