Skip to content

Guard missing DaCe configuration keys - #14

Open
guanLeTea wants to merge 1 commit into
mainfrom
dace-config-compat
Open

Guard missing DaCe configuration keys#14
guanLeTea wants to merge 1 commit into
mainfrom
dace-config-compat

Conversation

@guanLeTea

Copy link
Copy Markdown
Collaborator

Avoid raising "KeyError" when running against DaCe versions that do not declare all configuration keys.

The change:

  • skips missing configuration keys instead of failing
  • reports skipped keys once per process
  • preserves existing behavior when the keys are present

This restores compatibility with upstream DaCe while leaving the extended fork unchanged.

Validation:

  • Tested on both upstream main and the extended DaCe branch.
  • Upstream DaCe no longer fails during pin_build_caching.
  • Existing behavior is unchanged when the configuration keys exist.

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.

1 participant