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

nf-nomad #11

Open
wants to merge 575 commits into
base: master
Choose a base branch
from
Open

nf-nomad #11

wants to merge 575 commits into from

Conversation

abhi18av
Copy link
Owner

No description provided.

pditommaso and others added 29 commits October 28, 2023 08:35
Signed-off-by: Paolo Di Tommaso <[email protected]>


Signed-off-by: Ben Sherman <[email protected]>
Signed-off-by: Paolo Di Tommaso <[email protected]>
Co-authored-by: Paolo Di Tommaso <[email protected]>

Signed-off-by: Ben Sherman <[email protected]>
Co-authored-by: Marcel Ribeiro-Dantas <[email protected]>
Co-authored-by: Paolo Di Tommaso <[email protected]>
Signed-off-by: Paolo Di Tommaso <[email protected]>
This commit add the `cpu_model` in the Nextflow trace record. 

Note: the field is not included in the default trace file, but it can be added by adding 
in the nextflow config the setting 

```
trace.fields = '..,cpu_model'
```


Signed-off-by: Sabrina Krakau <[email protected]>
Co-authored-by: Paolo Di Tommaso <[email protected]>

Signed-off-by: Ben Sherman <[email protected]>
Signed-off-by: Paolo Di Tommaso <[email protected]>
Co-authored-by: Paolo Di Tommaso <[email protected]>

Signed-off-by: Fritjof Lammers <[email protected]>
Signed-off-by: Fritjof Lammers <[email protected]>
Signed-off-by: Ben Sherman <[email protected]>
Co-authored-by: Ben Sherman <[email protected]>
nxf_parallel runs a number of processes in parallel, distributing them
across a limited number of workers.

Prior to PR nextflow-io#1884, errors were not being checked.  PR#1884 was
intended to remedy this but effectively only checked the _last_
command to exit (which was, unfortunately the case that was tested
for).

Processes that exit while there is still work going on were not being
"waited" on, so their exit status was not being checked, and failures
were escaping unnoticed.

This commit:

- updates the loop that's tracking the work so that it waits on
  processes that have exited; and
- updates the test cases so that the failing command is _not_ the
  last in the list and ensures that the templated text is as expected.


Signed-off-by: George Hartzell <[email protected]>
Co-authored-by: Paolo Di Tommaso <[email protected]>
Signed-off-by: Paolo Di Tommaso <[email protected]>
Signed-off-by: Paolo Di Tommaso <[email protected]>
This commit remove the weblog as a core feature and move
it into a plugin maintained separately. See the link below

https://github.com/nextflow-io/nf-weblog

Signed-off-by: Paolo Di Tommaso <[email protected]>
pditommaso and others added 25 commits October 28, 2023 08:35
Signed-off-by: Paolo Di Tommaso <[email protected]>
Signed-off-by: Paolo Di Tommaso <[email protected]>
Signed-off-by: Paolo Di Tommaso <[email protected]>
Signed-off-by: Paolo Di Tommaso <[email protected]>
Signed-off-by: Paolo Di Tommaso <[email protected]>
Signed-off-by: Ben Sherman <[email protected]>
Signed-off-by: Paolo Di Tommaso <[email protected]>
Co-authored-by: Paolo Di Tommaso <[email protected]>
Signed-off-by: Paolo Di Tommaso <[email protected]>
Signed-off-by: Paolo Di Tommaso <[email protected]>

Signed-off-by: Ben Sherman <[email protected]>
Signed-off-by: Paolo Di Tommaso <[email protected]>
Co-authored-by: Paolo Di Tommaso <[email protected]>

Signed-off-by: Ben Sherman <[email protected]>
Signed-off-by: Paolo Di Tommaso <[email protected]>
Co-authored-by: Phil Ewels <[email protected]>
Co-authored-by: Paolo Di Tommaso <[email protected]>
Signed-off-by: Paolo Di Tommaso <[email protected]>
Signed-off-by: Paolo Di Tommaso <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.