fix coordinates merge

kermitt2 · May 5, 2024 · 9199170 · 9199170
1 parent 6336512
commit 9199170
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/grobid-core/src/main/java/org/grobid/core/engines/FundingAcknowledgementParser.java b/grobid-core/src/main/java/org/grobid/core/engines/FundingAcknowledgementParser.java
@@ -278,8 +278,10 @@ private static Nodes mergeSentencesFallingOnAnnotations(Nodes sentences, List<Pa
                     destCoordinates = destination.getAttribute("coords");
                     String coordinates = destCoordinates.getValue();
                     boundingBoxes = Arrays.stream(coordinates.split(";"))
+                        .filter(StringUtils::isNotBlank)
                         .map(BoundingBox::fromString)
                         .collect(Collectors.toList());
+                    destination.removeAttribute(destCoordinates);
                 }
 
                 for (int i = 1; i < toMerge.size(); i++) {
@@ -288,16 +290,16 @@ private static Nodes mergeSentencesFallingOnAnnotations(Nodes sentences, List<Pa
 
                     // Merge coordinates
                     if (needToMergeCoordinates) {
-                        Attribute coords = destination.getAttribute("coords");
+                        Attribute coords = ((Element) sentenceToMerge).getAttribute("coords");
                         String coordinates = coords.getValue();
                         boundingBoxes.addAll(Arrays.stream(coordinates.split(";"))
+                            .filter(StringUtils::isNotBlank)
                             .map(BoundingBox::fromString)
                             .toList());
 
                         List<BoundingBox> mergedBoundingBoxes = mergeBoxes(boundingBoxes);
                         String coordsAsString = String.join(";", mergedBoundingBoxes.stream().map(BoundingBox::toString).toList());
                         Attribute newCoords = new Attribute("coords", coordsAsString);
-                        destination.removeAttribute(coords);
                         destination.addAttribute(newCoords);
                     }