Add geo types support #30

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

Merged

chandr-andr merged 31 commits into psqlpy-python:main from Mernus:feature/add-geo-types

Aug 14, 2024

Contributor

Mernus commented Apr 9, 2024 •

edited

Loading

Added geo types support from PostgreSQL and geo_types module in Rust.

Description

Motivation and Context

How has this been tested?

Screenshots (if appropriate):

Types of changes

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

My code follows the code style of this project.
My change requires a change to the documentation.
I have updated the documentation accordingly.

AGAEV Denis E added 2 commits

April 8, 2024 09:42


          Add base geo types

3a4c7b5


          Add another geo types

ad12537

chandr-andr reviewed

View reviewed changes

src/geometric_types.rs Outdated Show resolved Hide resolved

chandr-andr reviewed

View reviewed changes

src/lib.rs Outdated Show resolved Hide resolved

AGAEV Denis E added 7 commits

April 12, 2024 10:50


          Move geo types to extra

227cecc


          Add macro for geo types

32f1954


          Merge branch 'main' into feature/add-geo-types

107d9f6


          Add additional types

1d20769


          Update

d438483


          Temp changes

4c71e7b


          Added build_point realization

e5146e1

chandr-andr reviewed

View reviewed changes

src/additional_types.rs Outdated Show resolved Hide resolved

chandr-andr reviewed

View reviewed changes

src/additional_types.rs Outdated Show resolved Hide resolved

chandr-andr reviewed

View reviewed changes

src/additional_types.rs Show resolved Hide resolved

chandr-andr reviewed

View reviewed changes

src/additional_types.rs Outdated Show resolved Hide resolved

chandr-andr reviewed

View reviewed changes

src/value_converter.rs Outdated Show resolved Hide resolved

chandr-andr reviewed

View reviewed changes

src/value_converter.rs Show resolved Hide resolved

AGAEV Denis E added 2 commits

May 7, 2024 10:25


          Added fixes

cec3ff6


          Added fixes

acfd743

chandr-andr reviewed

View reviewed changes

src/additional_types.rs Outdated Show resolved Hide resolved

chandr-andr reviewed

View reviewed changes

src/value_converter.rs Outdated Show resolved Hide resolved

Member

chandr-andr commented May 8, 2024

Btw. Looks great! Hope to see the final version soon!

AGAEV Denis E added 5 commits

May 9, 2024 08:36


          Merge branch 'main' into feature/add-geo-types

2a191f5


          Merge branch 'main' into feature/add-geo-types


          Fix errors

fa4a4f0


          Add builders for rust geo types

97615ab


          Merge branch 'main' into feature/add-geo-types

5f6eead

Contributor Author

Mernus commented May 13, 2024

Today will update on using exceptions that changed after main branch merge

AGAEV Denis E added 2 commits

May 17, 2024 11:17


          Add ToSQL traits for geo types

0d596ed


          Add geo types to PythonDTO and py_to_rust method

1296b19

chandr-andr reviewed

View reviewed changes

src/value_converter.rs Show resolved Hide resolved

Member

chandr-andr commented May 19, 2024

Add tests, please

AGAEV Denis E added 6 commits

May 21, 2024 12:20


          Add converting postgres to python for geo types

6a1a37e


          Add values for serialization tests

18fb41e


          Merge branch 'main' into feature/add-geo-types

e560d7c


          Fix working of all geo types

010ad47


          Remove prints

20348f3


          Merge remote-tracking branch 'origin/main' into feature/add-geo-types

46be94d

Contributor Author

Mernus commented Jun 25, 2024

Added fixes, now all geo types except polygon are working well!
Have some troubles with writing polygon to databases, trying to figure out how to do it:

DriverError: Database engine exception: db error: ERROR: invalid number of points in external "polygon" value.

AGAEV Denis E and others added 6 commits

August 3, 2024 23:23


          Merge remote-tracking branch 'origin/main' into feature/add-geo-types

3581dc9


          Remove polygon usage and fix some errors

402d23a


          Added docs and small fixes

09e5531


          Fix deriving serde and fix annotated usage

c29d400


          Fix tests

9c5edd1


          Merge branch 'main' into feature/add-geo-types

027f01f

chandr-andr approved these changes

View reviewed changes


          Merge branch 'main' into feature/add-geo-types

4d8085a

chandr-andr merged commit cad337e into psqlpy-python:main

11 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet