Code indexing in gitaly is broken and leads to code not being visible to the user. We work on the issue with highest priority.

Skip to content
Snippets Groups Projects

Resolve "Problem in Command DUMPEMFIELDS and DUMPFIELDS"

Merged gsell requested to merge 767-problem-in-command-dumpemfields-and-dumpfields into master
1 unresolved thread

Closes #767 (closed)

Edited by gsell

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
    • Author Owner

      @ext-calvo_p @ext-rogers_c we still have the division by zero error in Cyclotron::apply(). Any idea to solve this?

    • I fear that the code and tracking is written with the assumption that particles don't have zero radius. There might be serious errors in the tracking in this case. I think the proposed solution of throwing an exception is valid. Perhaps @adelmann can comment.

      One way to possibly fix it on the user side could be to change the input from

      Z_START=-0.02, Z_STEPS=41, DZ=0.001

      to

      Z_START=-0.0201, Z_STEPS=41, DZ=0.001 ?

    • I do not understand "particles don't have zero radius".

      We mostly deal with macro particles and indeed for tracking and field calculations thee particles are viewed as delta functions i.e. they have no spatial extend but indeed maybe a macro charge k*e.

      Does that make sense?

    • Sorry for the misunderstanding and the unclarity. What I meant is that in the issue, particles tracked in a cyclotron with OPAL-cycl have R = 0, causing divisions by zero. I wonder if the tracking is properly handled even for small R, or if certain assumptions are made.

    • Please register or sign in to reply
  • gsell added 1 commit

    added 1 commit

    • f18fba5f - getIndex() reviewed in FM3DH5BlockBase class

    Compare with previous version

  • snuverink_j
  • gsell added 3 commits

    added 3 commits

    Compare with previous version

  • gsell added 1 commit

    added 1 commit

    • fd5703e2 - Revert "Merge remote-tracking branch...

    Compare with previous version

  • gsell added 1 commit

    added 1 commit

    Compare with previous version

  • gsell added 1 commit

    added 1 commit

    • 5b070a55 - bugfix: max is num_gridX_m-2

    Compare with previous version

  • snuverink_j approved this merge request

    approved this merge request

  • gsell marked this merge request as ready

    marked this merge request as ready

  • adelmann approved this merge request

    approved this merge request

  • merged

  • gsell mentioned in commit 697b4afc

    mentioned in commit 697b4afc

  • gsell mentioned in issue #771 (closed)

    mentioned in issue #771 (closed)

  • ext-calvo_p changed milestone to %2024.1

    changed milestone to %2024.1

  • Please register or sign in to reply
    Loading