Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat[next]: Remove dace_iterator backend and pass_manager_legacy #1753

Merged
merged 3 commits into from
Dec 3, 2024

Conversation

edopao
Copy link
Contributor

@edopao edopao commented Nov 27, 2024

The dace orchestration tests are temporarily skipped until #1742 is merged.
The dace backend with SDFG optimization is temporarily disabled in unit tests until #1639 is merged.
A second PR will reorganize the files in dace backend module.

Copy link
Contributor

@havogt havogt left a comment

Choose a reason for hiding this comment

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

🥳

@edopao edopao merged commit a2551ac into GridTools:main Dec 3, 2024
31 checks passed
@edopao edopao deleted the dace-remove_iterator branch December 3, 2024 20:51
edopao added a commit that referenced this pull request Jan 22, 2025
PR #1753 removed the `dace_iterator` backend, that before existed by
side of `dace_fieldview` backend. Code common to these two backends was
placed in `dace_common` module.

The current GTIR-DaCe backend supports both iterator view and field view
flavors of the GTIR, so there is no need for a distinction at the top
level of the backend module.

This PR applies the following cleanup tasks:
```
runners/dace_common/dace_backend -> runners/dace_fieldview/sdfg_callable
runners/dace_common/utility -> runners/dace_fieldview/utils
runners/dace_common/workflow, runners/dace_fieldview/workflow -> runners/dace_fieldview/workflow
runners/dace_fiedview/utility -> runners/dace_fieldview/gtir_sdfg_utils
runners/dace_fieldview -> runners/dace
```

The module `runners/dace/workflow` was also split into sub-modules. A
doc string was added to `runners/dace/workflow/__init__.py`
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