Merge pull request #443 from LAAC-LSCP/438-eaf-builder-import-annotat…

…ions-speaker_id-na solving the issue speaker_id NA #438
LAAC-LSCP · Aug 18, 2023 · 1b62f62 · 1b62f62
2 parents e284d2b + 343aa4d
commit 1b62f62
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -7,6 +7,7 @@ All notable changes to this project will be documented in this file.
 ### Fixed
 
 - zooniverse chunks upload was failing if the dataset column was missing in the csv
+- eaf_builder with speaker_id NA no longer fails (#438)
 
 ## [0.1.1] 2023-05-11
 

diff --git a/ChildProject/pipelines/eafbuilder.py b/ChildProject/pipelines/eafbuilder.py
@@ -65,7 +65,7 @@ def create_eaf(
         for segment in speech_segments.to_dict(orient="records"):
             speaker_id = None
 
-            if "speaker_id" in segment:
+            if "speaker_id" in segment and not pd.isnull(segment["speaker_id"]):
                 speaker_id = segment["speaker_id"]
             elif "speaker_type" in segment:
                 speaker_id = segment["speaker_type"]