Skip to content
Snippets Groups Projects

Ions

All threads resolved!
Merged GUBAIDULIN requested to merge ions into develop
All threads resolved!

Watanyu found some bugs in the ions implementation that even prevented the code from running. I've fixed them, and now the example should run.

  • Bugfixes with beam_ion_elements.py
  • renamed particles_electromagnetic_fields.py into emfields.py and changed all the imports
  • I've added a flag to MPI parallelization that should prevent print statements from appearing for every rank of MPI.

We should think about making a new release or at least of publishing the bugfix to the stable branch :)

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
  • Watanyu FOOSANG
  • GUBAIDULIN added 1 commit

    added 1 commit

    Compare with previous version

  • GUBAIDULIN reset approvals from @GAMELIN by pushing to the branch

    reset approvals from @GAMELIN by pushing to the branch

  • GUBAIDULIN resolved all threads

    resolved all threads

  • GUBAIDULIN added 1 commit

    added 1 commit

    Compare with previous version

  • Watanyu FOOSANG approved this merge request

    approved this merge request

  • I'll try to add a simple test, which checks that the code will run in a typical scenario, and then I would merge.

  • GUBAIDULIN mentioned in commit e27fb231

    mentioned in commit e27fb231

  • merged

  • Please register or sign in to reply
    Loading