Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dependency Updates - GEOS, then trickle on #5336

Closed
jlaura opened this issue Nov 14, 2023 · 7 comments · Fixed by #5350
Closed

Dependency Updates - GEOS, then trickle on #5336

jlaura opened this issue Nov 14, 2023 · 7 comments · Fixed by #5350
Assignees
Labels
enhancement New feature or request

Comments

@jlaura
Copy link
Collaborator

jlaura commented Nov 14, 2023

Description

The ISIS dependencies are beginning to be out of date enough as to be problematic when installing ISIS with other planetary science applications (e.g., ASP).

Currently, we are working to get PDAL2.6+ integrated into ASP. ASP depends on ISIS. Therefore the older ISIS dependencies are causing issues.

Specifically, the following packages need an update:

  • GEOS / GDAL to 3.12

Updating that then requires looser pinning or updates to (at least):

  • Boost
  • QT
  • OpenCV
  • Protobuf
@jlaura jlaura added the enhancement New feature or request label Nov 14, 2023
@acpaquette
Copy link
Collaborator

Something to note is that this will fast forward our MacOS support to 10.9 (released October 22, 2013) and up, I don't think it will be a problem but something to note. We would also update glibc to 2.17 still supporting Ubuntu 14 but may drop other Linux systems

@jlaura
Copy link
Collaborator Author

jlaura commented Nov 15, 2023

@acpaquette That sounds like a net positive?

Here are the EOL dates for OS X. 10.9 service updates ended 10/1/2016. I look at ubuntu, but I suspect most OSes are in a similar place.

@acpaquette
Copy link
Collaborator

I tried creating an environment to see what else may need to updated and it seems like we can just updated geos to 3.12 without having to update the rest. What version of the other dependencies would be useful to update to?

@jlaura
Copy link
Collaborator Author

jlaura commented Nov 17, 2023

@oleg-alexandrov Are you able to weigh in here? I am not 100% sure what the dependency update tree looks like for PDAL beyond what I posted.

@oleg-alexandrov
Copy link
Contributor

As I recall, PDAL also needs recent versions of libxml2, GDAL, and tiledb. GDAL seems to need a newer boost. Otherwise I was getting a lot of conflicts.

The sure way to avoid problems later is to create an environment with the conda PDAL 2.6 package and install all other dependencies around it. Especially tiledb is problematic, as PDAL 2.6 fails to compile with older versions of it.

@dshean
Copy link

dshean commented Nov 17, 2023

https://pdal.io/en/2.6.0/development/compilation/dependencies.html

+1 for updating ASP/ISIS to more recent versions of GDAL and especially PROJ (hopefully 9+). We need to the much improved coordinate reference system transformation support, which should improve planetary CRS support as well.

@oleg-alexandrov
Copy link
Contributor

These are minimal dependencies, though. I've been using until now an approach where, due to ISIS being behind, I have to keep a handlful of custom-built conda packages that exist on conda-forge already, but are not compatible with ISIS.

With PDAL and tiledb, the list of custom packages I need to rebuild each time will just go up and and will get harder as other software moves forward.

I am not sure if it is feasible, but if ISIS has, by default, conda-level compatibility with PDAL and GDAL that will make it more convenient and easier for everybody.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Development

Successfully merging a pull request may close this issue.

4 participants