Skip to content
Snippets Groups Projects

Threadpoolize all the things

Merged Pedro Gonnet requested to merge threadpoolize_all_the_things into master

These are all the remaining function from the list in #338 (closed). I've tested this as far as possible on my laptop, i.e. MPI and gravity should work.

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
  • With this merged in I was going to:

    • Up the version number to 0.6.0 as it is becoming untracktable and we have done a lot of scaling improvements since 0.5.0
    • Run the usual scaling test
    • Make task and threadpool plots on 16 cores.
  • OK, will set the EAGLE_25 check away.

  • Added 1 commit:

    • 7dd3495a - Pointer arithmetic fix for icc warning. Documentation fixes.
  • Matthieu Schaller Status changed to merged

    Status changed to merged

  • mentioned in commit 6dff7c9e

  • mentioned in issue #338 (closed)

  • Here are some new threadpool plots:

    If you search hard you'll find the now more colourful many tasked steps...

  • Does exactly what it was intended to do! Excellent.

  • Author Developer

    Sweet! So do we know what the two remaining white gaps are?

  • Author Developer

    Also, the super-cell fix I have in mind should reduce the fraying in the smaller steps...

    Edited by Pedro Gonnet
  • Please register or sign in to reply
    Loading