Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Validity of run-time comparisons #52

Open
7 tasks
rsuchecki opened this issue Nov 22, 2019 · 0 comments
Open
7 tasks

Validity of run-time comparisons #52

rsuchecki opened this issue Nov 22, 2019 · 0 comments
Labels

Comments

@rsuchecki
Copy link
Collaborator

rsuchecki commented Nov 22, 2019

⚠️

Run times may not be comparable between tools/runs if we don't ensure that the underlying computational conditions were comparable. For that, the evaluation would probably have to be executed on a dedicated server with a task having exclusive access to that machine and input/output files being placed on local storage (e.g. using nextflow's scratch true).

  1. Cluster execution could be acceptable if we can ensure
  • homogeneity of the nodes (explicit partition spec?)
  • exclusive use of nodes
  • use of local scratch space
  1. Cloud (awsbatch) execution could be acceptable if we can ensure
  • homogeneity of the nodes
  • exclusive use of nodes
  • use of local scratch space

In addition, we must capture more of the task information via

trace.fields = 'task_id,name,status,exit,realtime,%cpu,rss'
  • which should include requested resources cpus,memory,time - more here

The cpu details can be easily picked-up in the mapping process
e.g. beforeScript 'cat /proc/cpuinfo > cpuinfo' which can be parsed downstream.
It is of limited value on its own for serious speed benchmarking,
but may be useful for the indicative use of speed in reports.

rsuchecki added a commit to rsuchecki/repset that referenced this issue Nov 22, 2019
rsuchecki added a commit that referenced this issue Nov 22, 2019
* bug fix

* params not captured in meta

* timestamp in results release tag

* date in release tag, release body update

* removed beers remnants

* release body update

* iso date/time in release tag

* release body formatting

* tidy-up release

* update

* updated docu

* added --subset option

* cleanup

* nf trace populated later #52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant