Copilot commented on code in PR #2683:
URL: https://github.com/apache/sedona/pull/2683#discussion_r2885917626


##########
docs/api/snowflake/vector-data/Geometry-Processing/ST_SimplifyPreserveTopology.md:
##########
@@ -20,6 +20,8 @@
 # ST_SimplifyPreserveTopology
 
 Introduction: Simplifies a geometry and ensures that the result is a valid 
geometry having the same dimension and number of components as the input,
+
+![ST_SimplifyPreserveTopology](../../../../image/ST_SimplifyPreserveTopology/ST_SimplifyPreserveTopology.svg
 "ST_SimplifyPreserveTopology")
 and with the components having the same topological relationship.

Review Comment:
   The image insertion breaks the Introduction sentence in half (comma at end 
of the line, image, then the sentence continues). Please move the image block 
after the full Introduction paragraph so the text reads as one continuous 
sentence/paragraph.



##########
docs/api/sql/Geometry-Processing/ST_SimplifyPreserveTopology.md:
##########
@@ -20,6 +20,8 @@
 # ST_SimplifyPreserveTopology
 
 Introduction: Simplifies a geometry and ensures that the result is a valid 
geometry having the same dimension and number of components as the input,
+
+![ST_SimplifyPreserveTopology](../../../image/ST_SimplifyPreserveTopology/ST_SimplifyPreserveTopology.svg
 "ST_SimplifyPreserveTopology")
 and with the components having the same topological relationship.

Review Comment:
   The image insertion breaks the Introduction sentence in half (comma at end 
of the line, image, then the sentence continues). Please move the image block 
after the full Introduction paragraph so the text reads as one continuous 
sentence/paragraph.



##########
docs/api/flink/Geometry-Processing/ST_LabelPoint.md:
##########
@@ -20,6 +20,8 @@
 # ST_LabelPoint
 
 Introduction: `ST_LabelPoint` computes and returns a label point for a given 
polygon or geometry collection. The label point is chosen to be sufficiently 
far from boundaries of the geometry. For a regular Polygon this will be the
+
+![ST_LabelPoint](../../../image/ST_LabelPoint/ST_LabelPoint.svg 
"ST_LabelPoint")
 centroid.

Review Comment:
   The image is inserted mid-sentence ("...will be the" then image, then 
"centroid."). Please move the image to after the complete Introduction 
paragraph to avoid splitting the sentence/paragraph.
   ```suggestion
   Introduction: `ST_LabelPoint` computes and returns a label point for a given 
polygon or geometry collection. The label point is chosen to be sufficiently 
far from boundaries of the geometry. For a regular Polygon this will be the 
centroid.
   
   ![ST_LabelPoint](../../../image/ST_LabelPoint/ST_LabelPoint.svg 
"ST_LabelPoint")
   ```



##########
docs/api/sql/Geometry-Processing/ST_LabelPoint.md:
##########
@@ -20,6 +20,8 @@
 # ST_LabelPoint
 
 Introduction: `ST_LabelPoint` computes and returns a label point for a given 
polygon or geometry collection. The label point is chosen to be sufficiently 
far from boundaries of the geometry. For a regular Polygon this will be the
+
+![ST_LabelPoint](../../../image/ST_LabelPoint/ST_LabelPoint.svg 
"ST_LabelPoint")
 centroid.

Review Comment:
   The image is inserted mid-sentence ("...will be the" then image, then 
"centroid."). Please move the image to after the complete Introduction 
paragraph to avoid splitting the sentence/paragraph.



##########
docs/api/flink/Geometry-Processing/ST_SimplifyPreserveTopology.md:
##########
@@ -20,6 +20,8 @@
 # ST_SimplifyPreserveTopology
 
 Introduction: Simplifies a geometry and ensures that the result is a valid 
geometry having the same dimension and number of components as the input,
+
+![ST_SimplifyPreserveTopology](../../../image/ST_SimplifyPreserveTopology/ST_SimplifyPreserveTopology.svg
 "ST_SimplifyPreserveTopology")
 and with the components having the same topological relationship.

Review Comment:
   The image insertion breaks the Introduction sentence in half (comma at end 
of the line, image, then the sentence continues). Please move the image block 
after the full Introduction paragraph so the text reads as one continuous 
sentence/paragraph.
   ```suggestion
   Introduction: Simplifies a geometry and ensures that the result is a valid 
geometry having the same dimension and number of components as the input, and 
with the components having the same topological relationship.
   
   
![ST_SimplifyPreserveTopology](../../../image/ST_SimplifyPreserveTopology/ST_SimplifyPreserveTopology.svg
 "ST_SimplifyPreserveTopology")
   ```



##########
docs/api/snowflake/vector-data/Geometry-Processing/ST_LabelPoint.md:
##########
@@ -20,6 +20,8 @@
 # ST_LabelPoint
 
 Introduction: `ST_LabelPoint` computes and returns a label point for a given 
polygon or geometry collection. The label point is chosen to be sufficiently 
far from boundaries of the geometry. For a regular Polygon this will be the
+
+![ST_LabelPoint](../../../../image/ST_LabelPoint/ST_LabelPoint.svg 
"ST_LabelPoint")
 centroid.

Review Comment:
   The image is inserted mid-sentence ("...will be the" then image, then 
"centroid."). Please move the image to after the complete Introduction 
paragraph to avoid splitting the sentence/paragraph.
   ```suggestion
   Introduction: `ST_LabelPoint` computes and returns a label point for a given 
polygon or geometry collection. The label point is chosen to be sufficiently 
far from boundaries of the geometry. For a regular Polygon this will be the 
centroid.
   
   ![ST_LabelPoint](../../../../image/ST_LabelPoint/ST_LabelPoint.svg 
"ST_LabelPoint")
   ```



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to