Opened 16 years ago
Closed 16 years ago
#2378 closed enhancement (fixed)
GDAL driver for FARSITE LCP format
Reported by: | ctoney | Owned by: | Even Rouault |
---|---|---|---|
Priority: | normal | Milestone: | 1.6.0 |
Component: | GDAL_Raster | Version: | unspecified |
Severity: | normal | Keywords: | farsite lcp driver |
Cc: |
Description
FARSITE v. 4 landscape file (LCP) is a multi-band raster format used by wildland fire behavior and fire effect simulation models such as FARSITE, FLAMMAP, and FBAT (www.fire.org). More information on the LCP format is provided in the attached format description document. The driver source and a test LCP file are also attached. This driver can be contributed under the terms of an MIT License. Thanks for considering it as an addition to the GDAL drivers set.
Chris
Attachments (4)
Change History (16)
by , 16 years ago
Attachment: | LCP_format_description.html added |
---|
by , 16 years ago
Attachment: | lcpdataset.cpp added |
---|
comment:1 by , 16 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
comment:2 by , 16 years ago
Mateusz,
a quick glance at the code shows that the main issue will be with endianness. I guess that Intel-order is assumed in the code.
comment:3 by , 16 years ago
Even,
Thanks for pointing that.
I've been in contact with Chris off-list guiding him how to submit the diver. I'm going to review it, but not today (actually, I've not got really back to work still).
If you like to work on it, feel free to take over the ticket (please, reassign). Anyway, ATM I'm not able to discuss internals of LCP driver before I jump to the code.
comment:4 by , 16 years ago
Component: | default → GDAL_Raster |
---|---|
Keywords: | farsite lcp driver added |
Milestone: | → 1.6.0 |
Owner: | changed from | to
Status: | assigned → new |
I'm going to try to integrate that.
comment:6 by , 16 years ago
Even, Mateusz-
Yes, I thought that could be an issue. Intel order is assumed. I can work on it if you like, but feel free to take it on if you prefer. I would need to study some sample code to see how to handle the byte-swapping. Thanks!
comment:7 by , 16 years ago
Chris,
I've commited the driver in trunk in r14473. I've dealt with the endianness issue. Are the 2 attached files appropriate for including in the GDAL autotest suite (from the licence point of view) ?
comment:8 by , 16 years ago
Even,
Thanks very much! The two attached files are appropriate for including from the license point of view.
comment:10 by , 16 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Closing : autotest runs fine on all the buildbot slaves
comment:11 by , 16 years ago
Resolution: | fixed |
---|---|
Status: | closed → reopened |
Even, There seems to be a problem with CPL_LSBINT32PTR in cpl_port.h. The LCP driver reports invalid values for certain min or max values by band. For example, ELEVATION_MIN=67109928 is incorrect for the test file "test_FARSITE_UTM12.LCP". The correct value is ELEVATION_MIN=1064. Similarly affected metadata fields are: ELEVATION_MAX, ASPECT_MAX, CANOPY_HT_MAX. I cannot tell what the problem is in the CPL_LSBINT32PTR macro.
comment:12 by , 16 years ago
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
LCP format description