Conversation
| def bbox_to_wkt(bbox: List[float], srid: int = 4326) -> str: | ||
| """Return WKT representation of a BBOX.""" | ||
| poly = Polygon.from_bounds(*bbox) # type:ignore | ||
| return f"SRID={srid};{poly.wkt}" |
There was a problem hiding this comment.
This can go, we don't need to set the CRS anymore
| # ), | ||
| # logic.V(geometry_column.name), | ||
| # ) | ||
| # ) |
There was a problem hiding this comment.
TODO: Spatial filter
|
Note: for this PR to go through we will need to get rid of buildpg cc @bitner |
|
I'm just wondering if the switch has in its scope the validation of the CQL-TEXT/CQL-JSON in the request. If not, is this feature planned or of interest? |
|
|
||
| # default to cql2-text | ||
| return cql2_text_parser(query) | ||
| return Expr(query) |
There was a problem hiding this comment.
@francbartoli In this PR the validation should be done here.
There was a problem hiding this comment.
cool, the use of dependencies already makes the validation clean
Yeah it should be done when we switch to CQL2 library |
2 participants
@bitner I could need some help to finish this PR