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.

Sign up for GitHub

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

Jump to bottom

Tiff: add support to read RAT stored in ArcGIS style .tif.vat.dbf auxiliary file #11475

Merged
merged 2 commits into from
Dec 12, 2024
Merged

Tiff: add support to read RAT stored in ArcGIS style .tif.vat.dbf auxiliary file #11475

merged 2 commits into from
Dec 12, 2024
+340 −4

Conversation

rouault
Copy link
Member

@rouault rouault commented Dec 10, 2024

CC @elpaso

@rouault rouault added this to the 3.11.0 milestone Dec 10, 2024
@rouault rouault force-pushed the vat_dbf branch 2 times, most recently from 018809f to 705471d Compare December 10, 2024 20:51
@dbaston
Copy link
Member

dbaston commented Dec 10, 2024

Just tested with a CORINE 2018 file I had sitting around, works nicely 👍

@coveralls
Copy link
Collaborator

coveralls commented Dec 10, 2024 •
edited
Loading

Coverage Status

coverage: 69.904% (-0.002%) from 69.906%
when pulling 475edfc on rouault:vat_dbf
into a72535d on OSGeo:master.

@elpaso
Copy link
Collaborator

elpaso commented Dec 12, 2024

CC @elpaso

Nice! This will allow to remove from QGIS the DBF parsing logic and delegate the whole RAT handling to GDAL.

@rouault rouault merged commit 87f83c8 into OSGeo:master Dec 12, 2024
38 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
3.11.0
Development

Successfully merging this pull request may close these issues.
4 participants
@rouault @dbaston @coveralls @elpaso