Skip to content

fix(config): errore su chiavi sconosciute in raw.sources[] (#514) - #516

Open
Andreacnt wants to merge 1 commit into
mainfrom
fix/raw-sources-unknown-keys
Open

Andreacnt wants to merge 1 commit into
mainfrom
fix/raw-sources-unknown-keys

Conversation

@Andreacnt

Copy link
Copy Markdown

Problema

RawSourceConfig.from_dict leggeva solo le chiavi note e ignorava le altre: un refuso come extractpr: (o extractor: con toolkit < 1.52) veniva scartato senza alcun avviso, lasciando il dataset in silenzio senza l'opzione voluta.

Fix

  • reject_unknown_keys() in toolkit/core/config.py: ValueError con path completo, chiavi non riconosciute e chiavi ammesse.
  • Applicato a raw.sources[i] e a raw.sources[i].client.
  • Messaggio di esempio: raw.sources[0]: unknown key(s) ['extractpr']; allowed keys: [...].

Nessun dataset.yml del lab usa chiavi fuori standard (verificati 157 file locali), quindi il cambio non rompe config esistenti.

Test

  • 2 nuovi test in tests/test_config_loading.py (source e client).
  • Suite locale: 1324 passed, stessi 15 failure/16 error preesistenti su Windows (bash assente, path backslash, rename file) verificati identici su main.

Closes #514

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

config: chiavi sconosciute in raw.sources[] scartate in silenzio (caso extractor, 1.51.0)

1 participant