Give correct name in xlsx importer to segmentation token.

korpling · Sep 5, 2024 · 4bc6089 · 4bc6089
1 parent 943136d
commit 4bc6089
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -5,6 +5,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Fixed
+
+- `xlsx` importer did not give the correct node name to segmentation token. Due
+  to this inconsistency, span annotations on segmentation nodes where not
+  connected to the segmentation token.
+
 ## [0.16.0] - 2024-09-02
 
 ### Added

diff --git a/src/importer/xlsx.rs b/src/importer/xlsx.rs
@@ -231,7 +231,7 @@ impl ImportSpreadsheet {
                         let overlapped_base_tokens: &[String] =
                             &base_tokens[base_token_start..base_token_end]; // TODO check indices
 
-                        let node_name = if token_annos.contains(name) {
+                        let node_name = if name == tok_name || token_annos.contains(name) {
                             format!("{}#{}_{}-{}", &doc_path, tok_name, start_row, end_row_excl)
                         } else {
                             format!(