8000 Add executor `get_description` function by upsj · Pull Request #1615 · ginkgo-project/ginkgo · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add executor get_description function #1615

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

Merged
merged 5 commits into from
Jun 29, 2024
Merged

Add executor get_description function #1615

merged 5 commits into from
Jun 29, 2024

Conversation

upsj
Copy link
Member
@upsj upsj commented May 19, 2024

This adds a function that prints a textual representation of the device used by an executor, and uses it in tests/benchmarks

@upsj upsj added the 1:ST:ready-for-review This PR is ready for review label May 19, 2024
@upsj upsj requested a review from a team May 19, 2024 00:17
@upsj upsj self-assigned this May 19, 2024
@ginkgo-bot ginkgo-bot added reg:testing This is related to testing. reg:benchmarking This is related to benchmarking. type:solver This is related to the solvers type:preconditioner This is related to the preconditioners type:matrix-format This is related to the Matrix formats type:factorization This is related to the Factorizations type:reordering This is related to the matrix(LinOp) reordering type:multigrid This is related to multigrid type:stopping-criteria This is related to the stopping criteria mod:all This touches all Ginkgo modules. labels May 19, 2024
@@ -1427,6 +1432,8 @@ class ReferenceExecutor : public OmpExecutor {
return {this, 0};
}

std::string get_description() const override { return "ReferenceExecutor"; }
Copy link
Member

Choose a reason for hiding this comment

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

I guess at some point we should also move the reference exec implementations into a .cpp file.

@upsj upsj force-pushed the executor_description branch from 11a9491 to ade3329 Compare June 28, 2024 11:00
@upsj upsj added 1:ST:ready-to-merge This PR is ready to merge. and removed 1:ST:ready-for-review This PR is ready for review labels Jun 28, 2024
@upsj upsj force-pushed the executor_description branch from b8a9398 to ed5623d Compare June 28, 2024 20:07
@upsj upsj force-pushed the executor_description branch from ed5623d to bf09bd2 Compare June 28, 2024 20:45
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
B Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

@upsj upsj merged commit ea7802e into develop Jun 29, 2024
13 of 15 checks passed
@upsj upsj deleted the executor_description branch June 29, 2024 11:29
MarcelKoch pushed a commit to MarcelKoch/ginkgo that referenced this pull request Dec 2, 2024
Adds a function printing a textual representation of the executor's device

Related PR: ginkgo-project#1615
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1:ST:ready-to-merge This PR is ready to merge. mod:all This touches all Ginkgo modules. reg:benchmarking This is related to benchmarking. reg:testing This is related to testing. type:factorization This is related to the Factorizations type:matrix-format This is related to the Matrix formats type:multigrid This is related to multigrid type:preconditioner This is related to the preconditioners type:reordering This is related to the matrix(LinOp) reordering type:solver This is related to the solvers type:stopping-criteria This is related to the stopping criteria
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0