-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Duplicate HIDs on usegalaxy.org #3818
Comments
@yvanlebras has a history where a history can not be extracted. The only thing that I can find in our logs is:
I tried to find it on sentry, but had no luck. If I can help with more details, let me please know. |
Triskel made 3 histories. One with raw data. Then 2 different histories where input data (data collections) were "copied" from the first history. Then, on the 2 last histories, she apply almost same tools to have results. One history is ok when extraction of workflow, the other not. One possibility, is maybe that for creating the third history, she copied the 2 data collections from the second history and not the first one, so on the third history, input data collections are copied from the second history who are copied from the first.... My 2 cents for now |
Triskel just tried to copy the history who doesn't work for workflow extraction, then delete all analytical steps (keeping only input data collections), then apply a tool on it, and here, workflow extraction works. So it seems this is maybe more related to an analytical step than input datasets... |
ok, it seems this is due to the qualimap bam tool https://ecology.usegalaxy.eu/root?tool_id=toolshed.g2.bx.psu.edu/repos/iuc/qualimap_bamqc/qualimap_bamqc/2.2.2c+galaxy1. Deleting this step allow Triskel to extract a workflow! |
It's an unfortunate design choice, but implicit conversions create datasets with the same hid as the source, this isn't problematic per se. Extraction from histories could surely use a good overhaul, in the meantime it would help to share a record of a problematic history, so we can track that in a separate issue. |
Here is an history: https://ecology.usegalaxy.eu/u/ylebras/h/copy-of-poolfrancoiserfassembles |
See @bgruening comment and sample history here #3816 (comment). It would seem to be unrelated to the original issue to me so I wanted to track it here. We don't and have never relied on only one thread generating HIDs to ensure uniqueness - so it doesn't seem related to me. It should be enforced by Postgres though so this is very troublesome for sure - xref https://github.com/galaxyproject/galaxy/blob/dev/lib/galaxy/model/mapping.py#L2528.
The text was updated successfully, but these errors were encountered: