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

return geometry in CRS84 by default #85

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

return geometry in CRS84 by default #85

wants to merge 2 commits into from

Conversation

3nids
Copy link
Member

@3nids 3nids commented Jun 2, 2023

use psqlextra to allow renaming a column

fixes #98

3nids and others added 2 commits June 2, 2023 08:27
@3nids 3nids closed this Sep 25, 2023
@3nids 3nids reopened this Sep 25, 2023
Comment on lines +120 to +121
queryset.filter(geom__intersects=my_bbox_polygon)
.annotate(geom2=Transform("geom", 4326))
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
queryset.filter(geom__intersects=my_bbox_polygon)
.annotate(geom2=Transform("geom", 4326))
queryset.filter(geom__intersects=my_bbox_polygon)
.defer("geom")
.annotate(geom=Transform("geom", 4326))

from django.http import HttpResponseBadRequest
from psqlextra.models import PostgresModel
Copy link
Collaborator

Choose a reason for hiding this comment

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

Ideally we remove this dependency.

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.

By default the extent should be returned in CRS84
3 participants