search

nextflow-io / nextflow

A DSL for data-driven computational pipelines
http://nextflow.io
Apache License 2.0
2.76k stars 629 forks source link

Render last tag along with the process name #1144

Closed welchwilmerck closed 5 years ago

welchwilmerck commented 5 years ago 
$ nnf info
  Version: 19.0.0.modules-draft10-SNAPSHOT build 5058
  System: Linux 4.4.41-36.55.amzn1.x86_64

Just run the example .nf in the docs for tag

N E X T F L O W  ~  version 19.0.0.modules-draft10-SNAPSHOT
Launching `tag.nf` [stoic_picasso] - revision: b3fea3d29c
executor >  local (3)
[bf/2119fc] process > foo [100%] 3 of 3 ✔
Completed at: 07-May-2019 13:25:06
Duration    : 2.3s
CPU hours   : (a few seconds)
Succeeded   : 3
N E X T F L O W  ~  version 19.01.0
Launching `tag.nf` [backstabbing_goodall] - revision: b3fea3d29c
[warm up] executor > local
[b9/736b4e] Submitted process > foo (aa alpha)
[40/0599b7] Submitted process > foo (aa gamma)
[9d/d4256e] Submitted process > foo (aa omega)
pditommaso commented 5 years ago

Duplicate of #1141?

welchwilmerck commented 5 years ago

Ahh, certainly appears to be.

pditommaso commented 5 years ago

Closing in favour of #1141.

pditommaso commented 5 years ago

Actually, the #1141 is proposing a more extended visualisation. I'm re-opening this to visualise the last along with the process name.

wjv commented 5 years ago

This is purely aesthetic, but…

If the tags are not all the same character length (e.g. if they are species names), this change leads to the right-hand column in the command line output (the column showing [100%] 82 of 82) continually jumping left and right while the pipeline is executing. While this doesn’t affect processing, but it’s visually distracting.

(I’m considering removing tagging from a project since my users thought the output jumping back and forth meant something is wrong.)

It might be better if the column containing processName (tag) is simply widened and never shrunk back down again if the length of the tag decreases.

pditommaso commented 5 years ago

@wjv please report it as separate issue

wjv commented 5 years ago

Thanks @pditommaso, I’ll do that. I’ll also add a little video I recorded to demonstrate what happens.