Opened 8 years ago

Closed 3 years ago

#53033 closed enhancement (wontfix)

saga doesn't open LAS files

Reported by: dershow Owned by: Veence (Vincent)
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc: ryandesign (Ryan Carsten Schmidt)
Port: saga

Description

The main reason that I wanted to install saga was to open and work with las LiDAR files. But, for some reason there is no I/O tool in the MacPorts version of saga that imports LAS. This tool is present on the Windows version, and it would be great if it becomes available on the macports version as well.

Change History (11)

comment:1 Changed 8 years ago by mf2k (Frank Schima)

Cc: vince@… removed
Owner: set to Veence
Status: newassigned

comment:2 Changed 7 years ago by Veence (Vincent)

Hmmm… I think I recently added LASTools. Is that satisfactory?

comment:3 Changed 7 years ago by dershow

Yes, I would think so.

comment:4 Changed 7 years ago by Veence (Vincent)

Resolution: fixed
Status: assignedclosed

Please reopen if it still doesn’t work with the new Saga version I just plugged in.

comment:5 Changed 7 years ago by dershow

Resolution: fixed
Status: closedreopened

I just built the new version of Saga that you updated (thanks for doing this!). But, I still don't see an option to import an LAS file. I still don't see any option in the tool set related to LAS files. I think that it should be in Import/Export->LAS. I just don't see any LAS related tools.

comment:6 Changed 7 years ago by Veence (Vincent)

Okay, I’m going to have a look this WE

comment:7 Changed 7 years ago by Veence (Vincent)

I have LIBLAS installed but it seems SAGA doesn’t look for an existing function:

checking for LAS_GetVersion in -llas... no

Since:

nm /opt/local/lib/liblas.dylib | grep LAS_GetVersion

but:

nm /opt/local/lib/liblas.dylib | grep GetVersion | awk '{print $3}' | xargs c++filt
liblas::GetVersion()
liblas::Header::GetVersionMajor() const
liblas::Header::GetVersionMinor() const

So, I’m a bit skeptical here. I will have to file a bug report upstream to get more information. Sorry.

comment:8 Changed 7 years ago by Veence (Vincent)

Resolution: invalid
Status: reopenedclosed

Ok. The LAS support has been removed in SAGA. Check http://www.saga-gis.org/saga_tool_doc/6.1.0/index.html w/r to http://www.saga-gis.org/saga_tool_doc/2.2.1/index.html, for example, and you’ll see the LAS I/O module has been removed.

comment:9 Changed 7 years ago by dershow

Thanks much for looking into it. That is strange, but clearly not a port issue.

comment:10 in reply to:  8 Changed 4 years ago by ryandesign (Ryan Carsten Schmidt)

Cc: ryandesign added
Resolution: invalid
Status: closedreopened

Replying to Veence:

The LAS support has been removed in SAGA.

I'm not sure about that. The configure script is still checking for LAS_GetVersion in -llas (and not finding it), so some liblas support is still in there.

I found this discussion stating that saga only supports liblas 1.2. Here is another discussion saying the same thing. We have liblas 1.8.1+ in MacPorts so that probably explains why saga is unable to find the (presumably obsolete) symbol it's looking for.

The saga port still declares a dependency on the liblas port. I suggest removing the liblas dependency. Also, please add configure arguments or whatever else is necessary to prevent the configure script from attempting to access liblas files even if it is installed.

comment:11 Changed 3 years ago by Veence (Vincent)

Resolution: wontfix
Status: reopenedclosed

I’m closing all old tickets, since I’m not sure they’re relevant at all. Please reopen a new one in case the issue is still at hand.

Note: See TracTickets for help on using tickets.