jiayuasu commented on code in PR #801:
URL: https://github.com/apache/sedona/pull/801#discussion_r1140550058
##########
zeppelin/index.js:
##########
@@ -109,21 +114,33 @@ export default class LeafletMap extends Visualization {
map.invalidateSize(true)
var imageBounds = null
- const markers = chartDataModel.rows.map(
+ const markers = chartDataModel.rows.flatMap(
row => {
const {image, boundary, info} = row;
+ var markers = [];
+
// throw new Error(image);
var jsts = require("jsts");
// Read WKT string from Sedona
var reader = new jsts.io.WKTReader();
- var obj = reader.read(boundary)
+ var obj = reader.read(boundary);
// Collect the centroid point of the
input geometry
var centroid = obj.getCentroid()
Review Comment:
This looks nice! Thank you very much. I think for polygons and linestrings,
we don't need to print their centroid any more and we can just plot their
shapes. Could you modify the logic here a little bit?
--
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: dev-unsubscr...@sedona.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
