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

Default 1D Domain Decomposition: z #203

Open
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

ax3l
Copy link
Member

@ax3l ax3l commented Aug 11, 2022

Change the default domain decomposition from x to z.

To Do

  • check if that is smarter
  • check if that is faster for most of our long-ish bunches

@ax3l ax3l added component: core Core ImpactX functionality Performance optimization labels Aug 11, 2022
@ax3l ax3l force-pushed the topic-1d-domain-decomp-in-z branch from a23c3b2 to 9a72e60 Compare August 11, 2022 18:12
@ax3l ax3l requested a review from n01r September 9, 2022 17:52
@ax3l ax3l assigned n01r Sep 9, 2022
@ax3l ax3l force-pushed the topic-1d-domain-decomp-in-z branch from 9a72e60 to 0c2db3d Compare September 23, 2022 01:48
Keep the ASCII file open as we write diagnostics for a step. This is
providing a decent speedup for this preliminary output method.
@n01r
Copy link
Member

n01r commented Jan 25, 2024

Picking this up again after a very long time:

When I compare the current suite of CTests there is no difference.

For a case like cyclotron which seems to have a rather long bunch duration, the z-decomposed case is actually about 100ms slower. But this runs within a second.

All_CTest_decomp_comparison.zip

Perhaps I would need to run something larger, and with more particles?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants