Feature Request: Support updating location geometry (latitude/longitude) in UpdateLocationRequest
@devsjc đang làm issue này rồi.
Từ ngày 9/7/2026.
Đánh giá
Issue này chưa được đánh giá.
Mô tả
Verification
- I have searched the existing issues to ensure this feature has not already been requested.
Provide a detailed description of the proposed feature.
In the quartz-api service, the PUT /sites/{site_uuid} endpoint allows users to update a site's properties, including its latitude and longitude coordinates.
However, the Data Platform (DP) gRPC service currently does not support updating coordinates for an existing location. The UpdateLocationRequest message only exposes fields to update capacity, name, and metadata, but has no geometry/WKT fields:
protobuf
// Current UpdateLocationRequest Schema
message UpdateLocationRequest {
string location_uuid = 1;
EnergySource energy_source = 2;
string new_location_name = 3;
int64 new_effective_capacity_watts = 4;
google.protobuf.Struct new_metadata = 5;
google.protobuf.Timestamp valid_from_utc = 6;
}
Because of this limitation:
quartz-api must ignore any coordinate updates passed during PUT requests for existing sites.
Coordinates are immutable after location creation.
Proposed Solution
Update the Data Platform gRPC API schema to support updating location geometry.
Specifically, we propose adding a new_geometry_wkt string field (or separate new_latitude/new_longitude fields) to UpdateLocationRequest:
protobuf
message UpdateLocationRequest {
string location_uuid = 1;
EnergySource energy_source = 2;
string new_location_name = 3;
int64 new_effective_capacity_watts = 4;
google.protobuf.Struct new_metadata = 5;
google.protobuf.Timestamp valid_from_utc = 6;
string new_geometry_wkt = 7; // <--- ADD THIS FIELD
}
This will allow the downstream database engines to execute geometry updates (e.g., UPDATE locations SET geometry = ST_GeomFromText(new_geometry_wkt) WHERE ...) and align the API client behavior.
What is the motivation for the feature?
Correcting Seeding Errors: Clients should be able to update the locations lat lon
Outline a potential approach or solution (optional).
- Ngôn ngữ chính
- Go
- Star
- 3
- Fork
- 1
- Merge trung bình
- 1 giờ 59 phút
- Pull request đã merge (30 ngày)
- 9
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Có mẫu pull request
- Không có hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của openclimatefix/data-platform
-
Reconstruct GSPs in DPĐang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 45/100
openclimatefix/data-platform#220 ·
-
GSP labelsĐang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 48/100
openclimatefix/data-platform#219 · 2 bình luận ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 45/100
openclimatefix/data-platform#218 · 1 bình luận ·
-
Save forecasts made in the pastĐang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 48/100
openclimatefix/data-platform#210 · 2 bình luận ·
-
Battery dataĐang mở
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
openclimatefix/data-platform#204 · 1 bình luận ·
Tất cả issue của openclimatefix/data-platform
Issue tương tự
-
VLAgent `spec.k8sCollector.includePodLabels: false` has no effectCó thể đã có người làm Có pull request liên kết đang mở hoặc đã được merge. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
VictoriaMetrics/operator#2772 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
Gaurav-Gosain/tuios#1316 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
needs: investigation
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Maintainer thường phản hồi trong vòng 1 ngày
-
area:auth documentation priority:low
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 86/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Mend: dependency security vulnerability
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
kube-tarian/sigrun#203 ·