galaxyproject / galaxy

Data intensive science for everyone.
https://galaxyproject.org
Other
1.41k stars 1.01k forks source link

History "hastag" improvements/issues #3730

Open dannon opened 7 years ago

dannon commented 7 years ago

#hashtags are a tag that is propagated to derived datasets and displayed alongside history items even when collapsed.

In history collections should display tags just like individual datasets. When collection is expanded:

coltags

we need to display tags somewhere around a list of paired datasets element.

dannon commented 7 years ago

Fixed same-name different-value tag issue preventing tagging in the UI with #3760

dannon commented 7 years ago

Tagging for collections in https://github.com/galaxyproject/galaxy/pull/3868

jxtx commented 7 years ago

@dannon It looks like tags on collections do not propagate through tool executions

dannon commented 7 years ago

Yeah, it looks like where I added the tag addition code, handle_output only touches the datasets. I can add it to collections too pretty easily though, it looks like. Good catch!

dannon commented 7 years ago

Redraw also handled in #3868 now.

nekrut commented 7 years ago

Testing TAGs on test.galaxyproject.org getting this: screen shot 2017-04-15 at 3 32 15 pm

Datasets 2 - 5 are labelled with tags. Tags propagate correctly to 6 - 9. In step 10 I merge datasets 6 - 9 into one. Tag A gets duplicated. This history can be accessed here: https://test.galaxyproject.org/u/anton/h/hiv-r01-example-tagged

Clicking on dataset 10 does not show duplicated tag within tag list:

screen shot 2017-04-15 at 3 36 53 pm

dannon commented 7 years ago

Hashtags in #3979

dannon commented 7 years ago

@nekrut Was the recent addition (sometimes not propagating tags) yours? Can you elaborate? I'd lean towards doing this in the job execution context (at runtime) rather than doctoring the tool XMLs, if that'd handle your use-case.

nekrut commented 7 years ago

@dannon: I can't come up with an example for this

dannon commented 7 years ago

Sort/unique done in https://github.com/galaxyproject/galaxy/pull/4139

chambm commented 7 years ago

Are workflow-set hashtags supposed to overwrite (reset) the dataset-derived hashtags? If so, is there a way to combine both? I'd like the workflow to propagate which tools and reference genome was used to derived datasets, while the original input dataset hashtags propagate the sample identifier.

dannon commented 7 years ago

Resultant tag should be the combination of both, my guess is a bug in the way PJA (workflow set) tags are set is overwriting. I'll check and get a fix out shortly.

yvanlebras commented 7 years ago

Hi everyone, I see another potential issue related to tagging workflow step in "configure output" / "Tags". If at a step X I add a tag #lovegalaxy and that in the step x+1 I add #loveiuc, then executing the wf on a history will result for the x+1 output step on a uniq tag (here #loveiuc). It seems that the previous one is erased...

dannon commented 7 years ago

(#4188 fixed @yvanlebras problem)

martenson commented 6 years ago

Nothing here seems pressing, removing milestone for now so we do not have to keep pushing it back.