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

Add RFC 108 text: driver removals for GDAL 3.11 #11862

Merged
merged 3 commits into from
Mar 2, 2025

Conversation

rouault
Copy link
Member

@rouault rouault commented Feb 18, 2025

No description provided.

@rouault rouault added this to the 3.11.0 milestone Feb 18, 2025
@rouault rouault added the funded through GSP Work funded through the GDAL Sponsorship Program label Feb 18, 2025
The decision of which driver to removal comes from a mix of "intuition" and
community feedback from
https://docs.google.com/spreadsheets/d/12yk0p8rRK4rAYO_VwXboatWkiILAjE2Cepu32lsHhM8/edit?gid=0#gid=0

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why not consider available data such as traffic to documentation pages? This would show e.g. ~90 hits/month to the documentation for the TIGER driver, which is more than retained drivers such as JSONFG.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TIGER may be a special case and hit statistics contain mishits because most TIGER data are not in that format.

JSONFG is a format of the future, not yet accepted as an OGC standard, but it will be soon. TIGER is a format from the past. The choise which one to keep is clear.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not suggesting that we remove JSONFG because it has less web traffic than TIGER. But I don't think we should disregard that traffic either. Given relatively recent discussion or usage of data in this format, I would not write off the site traffic as accidental.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In the "usage" link they have used Data: County subdivision lines from US Census Bureau 2018 TIGER/line files, thus shapefiles, but it does not mean that some other researcher wants and needs to use historical TIGER data for a good reason, like in the "discussion" link.

whole codebase enhancements.
The decision of which driver to removal comes from a mix of "intuition" and
community feedback from
https://docs.google.com/spreadsheets/d/12yk0p8rRK4rAYO_VwXboatWkiILAjE2Cepu32lsHhM8/edit?gid=0#gid=0
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The community was solicited for feedback about removing write support for certain drivers, with a link to this spreadsheet. The spreadsheet is more broad in its scope and indicates complete removal of certain drivers. The spreadsheet shows Interlis as the only vector driver being considered for removal, but this RFC proposes to remove 5 drivers (of which Interlis is not one?). It's not easy to understand the connection between the community feedback and the list in this RFC.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Interlis write support removal has already been removed prior to this RFC, given blessing from his maintainer.

It's not easy to understand the connection between the community feedback and the list in this RFC.

Don't try too hard to find logic ;-)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is pretty-much DOGE based. We wipe off everything and finds out later what is actually needed.

@rouault rouault merged commit c94c42e into OSGeo:master Mar 2, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
funded through GSP Work funded through the GDAL Sponsorship Program
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants