Skip to content

feat[next]: include dace build_folder_mode in gt4py config - #2933

Merged
edopao merged 6 commits into
mainfrom
dace_build_folder_mode
Oct 5, 2026
Merged

edopao merged 6 commits into
mainfrom
dace_build_folder_mode

Conversation

@edopao

@edopao edopao commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

This pull request updates the mechanism for controlling whether DaCe build files are kept after compilation, switching from the DACE_compiler_build_folder_mode environment variable to a new flag, GT4PY_KEEP_DACE_BUILD_FILES.

This change was needed to comply with the new behavior of DaCe version 2.0.0a10. The DaCe PR spcl/dace#2602 gives highest priority to the configuration set inside the application code (dace.Config.set()), which overrides whatever is set through the DaCe environment variables.

  • Introduced the KEEP_DACE_BUILD_FILES flag in gt4py/next/config.py, controlled by the GT4PY_KEEP_DACE_BUILD_FILES environment variable, to determine if DaCe build files are retained after compilation.

  • Updated set_dace_config in src/gt4py/next/program_processors/runners/dace/workflow/common.py to use KEEP_DACE_BUILD_FILES for setting dace.Config['compiler', 'build_folder_mode'] (development if true, otherwise production).

@edopao
edopao added this pull request to stack #2935 October 5, 2026 06:19
Comment thread scripts/python/dace_determinism.py Outdated
Comment thread scripts/python/dace_determinism.py Outdated
Comment thread scripts/python/dace_determinism.py Outdated
Comment thread scripts/tests/python/test_dace_determinism.py Outdated

@philip-paul-mueller philip-paul-mueller left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

Comment thread src/gt4py/next/program_processors/runners/dace/workflow/common.py Outdated
Co-authored-by: Philip Müller <147368808+philip-paul-mueller@users.noreply.github.com>
@edopao
edopao merged commit 37c02a8 into main Oct 5, 2026
28 checks passed
@edopao
edopao deleted the dace_build_folder_mode branch October 5, 2026 13:24
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.

2 participants