galaxyproject / usegalaxy-playbook

Ansible Playbook for usegalaxy.org
Academic Free License v3.0
30 stars 24 forks source link

Update Circos tool suite at usegalaxy.org #346

Closed jennaj closed 2 years ago

jennaj commented 3 years ago

Workaround for End-Users:

Please try the latest version of the tool at usegalaxy.eu: Circos visualizes data in a circular layout (Galaxy Version 0.69.8+galaxy8 0.69.8+galaxy10). Avoid earlier versions if using "Circos Presets" for Karyotype.

Problem reported/confirmed 10/27/21 Galaxy Help post: https://help.galaxyproject.org/t/circos-functional-on-usegalaxy-eu-but-not-usegalaxy-org/6972

Test histories (GTN test data)

cc @davebx

MTS: https://toolshed.g2.bx.psu.edu/view/iuc/circos/7f13dc906cb9

More context

jennaj commented 2 years ago

Current version is now: 0.69.8+galaxy10

MTS: https://toolshed.g2.bx.psu.edu/view/iuc/circos/290a11559985

Note: Includes at least one other distinct bug fix ("Chrom presets" wasn't functional anywhere, any version version 0.69.8+galaxy7 or later)

circos_Presets
jennaj commented 2 years ago

Fails tests: https://github.com/mvdbeek/usegalaxy-tests/issues/26

@bgruening @hexylena -- what version of the tool should we update to?

Also asked here: https://matrix.to/#/!RbhnsnsUHtkKuXuMDM:gitter.im/$XW7_xwK1nnUKLIjTKVLwBfdHY3qjii8A35bPkHDtZVI?via=gitter.im&via=matrix.org

.org is at 0.69.8+galaxy7 (known issue re karyotype index, but doesn't impact GTN tutorial afaik) .eu is at 0.69.8+galaxy9 mts is at 0.69.8+galaxy10

cc @davebx

hexylena commented 2 years ago

please update to galaxy10 @jennaj

For circos you can always just wait for my answer, I promise I'll answer whenever I'm awake. I'm still maintaining that tool and can offer the best commentary about which version :)

jennaj commented 2 years ago

Thank you and apologies for posting in the matrix chat about the same question!!

We can try get this updated before Monday if possible. If more delays, I think we should add to the visualisation_circos's tutorial channel in the pinned FAQ to run at eu or au instead.

jennaj commented 2 years ago

Didn't actually install. Not sure why.

Oh, unmerged commits -- https://github.com/galaxyproject/usegalaxy-tools/pull/482

jennaj commented 2 years ago

Not sure what we need to do. Can you help @hexylena?

The version sorting isn't working correctly when loaded from the tool panel (defaults to the 7 version, not the new 10 version). And the tool is failing for what looks like the original karyotype issue.

https://usegalaxy.org/u/jen/h/test-history-circos

dannon commented 2 years ago

@jennaj The tool version sorting is being looked into by @natefoo; he has some work in progress that will resolve this.

jennaj commented 2 years ago

Great, thank you!

I'm going to close out the open tickets -- the tool runs whoohoo

hexylena commented 2 years ago

@jennaj do you have a still failing circos history, I think I've recently seen one potential cause of the specific failure and I want to check.