Skip to content

Conversation

@lucamar
Copy link
Contributor

@lucamar lucamar commented Jun 14, 2021

By mistake OMP_NUM_THREADS was set equal to str(self.num_cpus_per_task) (which is 16), rather than 8: in this way the performance of the job is almost half than expected. Defining self.num_cpus_per_task = 16 and then OMP_NUM_THREADS = 8 for some reason gives a slightly better performance with respect to setting directly self.num_cpus_per_task = 8.

@codecov-commenter
Copy link

codecov-commenter commented Jun 14, 2021

Codecov Report

Merging #2015 (2c6dc9b) into master (52a313e) will not change coverage.
The diff coverage is n/a.

❗ Current head 2c6dc9b differs from pull request most recent head 039f40c. Consider uploading reports for the commit 039f40c to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2015   +/-   ##
=======================================
  Coverage   87.48%   87.48%           
=======================================
  Files          50       50           
  Lines        8848     8848           
=======================================
  Hits         7741     7741           
  Misses       1107     1107           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 52a313e...039f40c. Read the comment docs.

@teojgo
Copy link
Contributor

teojgo commented Jun 15, 2021

@jenkins-cscs retry pilatus

@teojgo teojgo merged commit bc59cb9 into reframe-hpc:master Jun 15, 2021
@lucamar lucamar deleted the fix-qe branch July 9, 2021 12:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants