Skip to content

Spin push for ExactSbend#1448

Open
cemitch99 wants to merge 4 commits into
BLAST-ImpactX:developmentfrom
cemitch99:add_spin_exact_sbend
Open

Spin push for ExactSbend#1448
cemitch99 wants to merge 4 commits into
BLAST-ImpactX:developmentfrom
cemitch99:add_spin_exact_sbend

Conversation

@cemitch99
Copy link
Copy Markdown
Member

@cemitch99 cemitch99 commented May 2, 2026

This PR adds the spin push for the ExactSbend element. Consistent with the phase space push, which is analytically known in closed form, the spin map is analytically determined in closed form.

This is the most important and nontrivial example of an analytically-determined spin map with fully nonlinear dependence on the phase space variables.

  • add draft of spin push, including evaluation of the first spin map factor
  • add evaluation of the second spin map factor
  • add benchmarks

Suggestions: consistency comparisons against Sbend and ExactCFbend with spin.
In particular, scaling tests can be performed (with respect to the phase space deviation and the map stepsize, respectively for the two comparisons above).

Comment thread src/elements/ExactSbend.H Fixed
Comment thread src/elements/ExactSbend.H Fixed
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented May 3, 2026

Merging this PR will not alter performance

✅ 39 untouched benchmarks
⏩ 2 skipped benchmarks1


Comparing cemitch99:add_spin_exact_sbend (7de8217) with development (472938d)

Open in CodSpeed

Footnotes

  1. 2 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@cemitch99 cemitch99 added component: elements Elements/maps/external fields component: spin tracking Spin tracking labels May 4, 2026
@ax3l ax3l mentioned this pull request May 13, 2026
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component: elements Elements/maps/external fields component: spin tracking Spin tracking

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants