Skip to content

Conversation

pratikvn
Copy link
Member

@pratikvn pratikvn commented Aug 5, 2024

This PR adds benchmarking support for distributed preconditioners. It also fixes a few things that were causing increased allocations for distributed preconditioners.

@pratikvn pratikvn added the 1:ST:ready-for-review This PR is ready for review label Aug 5, 2024
@pratikvn pratikvn requested a review from a team August 5, 2024 08:45
@pratikvn pratikvn self-assigned this Aug 5, 2024
@ginkgo-bot ginkgo-bot added mod:core This is related to the core module. reg:benchmarking This is related to benchmarking. type:solver This is related to the solvers type:preconditioner This is related to the preconditioners labels Aug 5, 2024
GKO_ASSERT_EQ(level > 0, true);
auto last_mg_level = mg_level_list_.back();

using ws = workspace_traits<Multigrid>;
Copy link
Collaborator

Choose a reason for hiding this comment

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

unused?

Copy link
Collaborator

@greole greole left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@yhmtsai yhmtsai left a comment

Choose a reason for hiding this comment

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

could you separate the pr to two? benchmark and the rest.

#ifdef GINKGO_BUILD_MPI
", schwarz-jacobi, schwarz-ilu, schwarz-ic, schwarz-lu, dist-mg"
#endif
"");
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
"");
);

.on(exec));
return gko::solver::Multigrid::build()
.with_mg_level(gko::multigrid::Pgm<etype, itype>::build()
.with_deterministic(FLAGS_mg_deterministic))
Copy link
Member

Choose a reason for hiding this comment

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

these flag are not available yet

@MarcelKoch MarcelKoch added this to the Ginkgo 1.9.0 milestone Aug 26, 2024
@MarcelKoch MarcelKoch modified the milestones: Ginkgo 1.9.0, Ginkgo 1.10.0 Dec 9, 2024
@pratikvn
Copy link
Member Author

Closed in favour of #1844 and #1845

@pratikvn pratikvn closed this May 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1:ST:ready-for-review This PR is ready for review mod:core This is related to the core module. reg:benchmarking This is related to benchmarking. type:preconditioner This is related to the preconditioners type:solver This is related to the solvers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants