Skip to content

Comments

lib/vector: new functions for setting trim of GEOS WKT output#4123

Merged
nilason merged 2 commits intoOSGeo:mainfrom
nilason:fix_geos312_regression
Aug 2, 2024
Merged

lib/vector: new functions for setting trim of GEOS WKT output#4123
nilason merged 2 commits intoOSGeo:mainfrom
nilason:fix_geos312_regression

Conversation

@nilason
Copy link
Contributor

@nilason nilason commented Aug 1, 2024

GEOS 3.12 switched the default setting of the trim option to true. To enable consistent behaviour to Vect_read_area_to_wkt() and Vect_line_to_wkt() across GEOS versions, they now call new 2nd generation functions, which set this trim option explicitly to false.

Closes: #4117 #4118

GEOS 3.12 switched the default setting of the trim option to true.
To enable consistent behaviour to Vect_read_area_to_wkt() and
Vect_line_to_wkt() across GEOS versions, they now call new
2nd generation functions, which set this trim option explicitly
to false.
@nilason nilason added this to the 8.5.0 milestone Aug 1, 2024
@nilason nilason self-assigned this Aug 1, 2024
@github-actions github-actions bot added vector Related to vector data processing C Related code is in C libraries labels Aug 1, 2024
echoix
echoix previously approved these changes Aug 1, 2024
@nilason nilason added the backport to 8.4 PR needs to be backported to release branch 8.4 label Aug 1, 2024
@nilason nilason merged commit 85498f4 into OSGeo:main Aug 2, 2024
@nilason nilason deleted the fix_geos312_regression branch August 2, 2024 15:02
Mahesh1998 pushed a commit to Mahesh1998/grass that referenced this pull request Sep 19, 2024
…4123)

GEOS 3.12 switched the default setting of the trim option to true.
To enable consistent behaviour to Vect_read_area_to_wkt() and
Vect_line_to_wkt() across GEOS versions, they now call new
2nd generation functions, which set this trim option explicitly
to false.
@neteler
Copy link
Member

neteler commented Sep 20, 2024

Shall I backport it, @nilason ?

@nilason
Copy link
Contributor Author

nilason commented Sep 20, 2024

Shall I backport it, @nilason ?

Please, do.

neteler pushed a commit that referenced this pull request Sep 20, 2024
GEOS 3.12 switched the default setting of the trim option to true.
To enable consistent behaviour to Vect_read_area_to_wkt() and
Vect_line_to_wkt() across GEOS versions, they now call new
2nd generation functions, which set this trim option explicitly
to false.
@neteler neteler modified the milestones: 8.5.0, 8.4.1 Sep 20, 2024
@neteler neteler removed the backport to 8.4 PR needs to be backported to release branch 8.4 label Sep 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C Related code is in C libraries vector Related to vector data processing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] pygrass/vector – test_pygrass_vector_doctests fails with GEOS 3.12 [Bug] pygrass/vector – test_geometry fails with GEOS 3.12

3 participants