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

Trying to resolve a version issue with GDAL and TensorFlow, cuXFilter, etc. #533

Merged
merged 3 commits into from
Sep 19, 2022

Conversation

thomcom
Copy link
Contributor

@thomcom thomcom commented Sep 14, 2022

cuspatial has GDAL pinned to 3.4.x which is causing a conflict with rapidsai/cuspatial#674 . I'm updating here and cuspatial to use 3.*. Contributes to rapidsai/cuspatial#674

@ajschmidt8
Copy link
Member

@thomcom, we need to keep the following CVE in mind when editing the GDAL version. The affected GDAL versions are 3.3.0-3.4.0. Ideally we'd bump to newer versions. Some of the older versions caused conda conflicts in our images (see #521 and rapidsai/cuspatial#623)

@thomcom
Copy link
Contributor Author

thomcom commented Sep 19, 2022

Thanks for that reminder @ajschmidt8 I think that we have GDAL pinned specifically to 3.4.3

@thomcom
Copy link
Contributor Author

thomcom commented Sep 19, 2022

I'm still experimenting with a good GDAL version to support, but it'll probably be set to 3.5 which is what I was going with on this one.

@ajschmidt8 ajschmidt8 merged commit c624bc3 into rapidsai:branch-22.10 Sep 19, 2022
rapids-bot bot pushed a commit to rapidsai/cuspatial that referenced this pull request Sep 19, 2022
@beckernick report's a GDAL version conflict with cuda. We're pinned at GDAL 3.4 and TensorFlow and other libs have already updated to GDAL 3.5. Let's update that here.

Fixes #674 and depends on rapidsai/integration#533

Authors:
  - H. Thomson Comer (https://github.com/thomcom)

Approvers:
  - Mark Harris (https://github.com/harrism)
  - AJ Schmidt (https://github.com/ajschmidt8)

URL: #675
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants