From 8d9c93db727d12174b6706d6cdf476d7c9442457 Mon Sep 17 00:00:00 2001 From: Rob Court Date: Tue, 4 Nov 2025 17:20:22 +0000 Subject: [PATCH] Update validation in CurationWriter to use short_form for unrestricted fields in entity checks --- src/vfb/curation/curation_writer.py | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/vfb/curation/curation_writer.py b/src/vfb/curation/curation_writer.py index 0933332..eda345e 100644 --- a/src/vfb/curation/curation_writer.py +++ b/src/vfb/curation/curation_writer.py @@ -278,12 +278,13 @@ def write_rows(self, verbose=False, start='100000', allow_duplicates=False): if verbose: print(f" ✗ {entity_name}: {yaml_value} not in class_lookup (will fail per-row)") else: - # Fields without restriction: validate by label directly (e.g., DataSet) - # These are passed as-is to add_anatomy_image_set() + # Fields without restriction: validate by short_form (e.g., DataSet) + # These are passed as short_forms to add_anatomy_image_set() + # add_anatomy_image_set() defaults to match_on='short_form' self.pattern_writer.ec.roll_entity_check( labels=labels, query=yaml_value, - match_on='label' + match_on='short_form' ) # Check if it exists - only cache if validation succeeds