galaxyproject / galaxy

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

Collections don't indicate deleted datasets, changing HDAs doesn't propagate up to DCEs #17735

Open jennaj opened 7 months ago

jennaj commented 7 months ago

edit Start with https://github.com/galaxyproject/galaxy/issues/17735#issuecomment-1999297437

Describe the bug The datasets included in a collection appear to be a mix of the original datasets and the datasets copied into the collection depending on the view -- Specifically: which you view nested inside the collection when expanded in the history (original files added to the collection) differs from which metadata is inherited/displayed on the collapsed collection (actual files inside the collection).

In practical use: if any of the hidden datasets are manipulated (active/deleted state changed, datatype changed) what can be viewed in the history panel is a mix of those states! This can impact how users curate data in the hidden panel (it happens..), how tool forms consume the collections, and using some of the state change functions (in particular, when clicking on the "eye" icon for a dataset in an expanded collection via the active tab, the link to "undelete" a datasets flips away too quickly to use (seems to be referencing one of the copies), since it is quickly replaced the file content (seems to be referencing a different copy?).

See screenshots.

Galaxy Version and/or server at which you observed the bug UseGalaxy.org The Galaxy Server is running version 24.0.rc1 , and the web client was built on Thursday Mar 14th 3:00:16 2024 GMT-7 .

Browser and Operating System Operating System: Windows, Linux, macOS Browser: Firefox, Chrome, Chrome-based, Safari

To Reproduce Steps to reproduce the behavior:

  1. Load up a few datasets, and put those into a collection
  2. Go to the hidden tab, notice that each of the original datasets has been duplicated (expected)
  3. Set up some changes in the second set (those that are "in" the collection)
  4. Go back to the top level of the history, and view just active datasets, then layer in deleted and hidden
  5. Notice that the collection has been updated with the new datatype assigned to the datasets (good!)
  6. Click to expand the collection, and notice that the datasets still have the old datatype (seem to be referencing the originally added datasets?, not so good...)
  7. Try to view the datasets in the collection using the "eye" icon to figure out the content (users do this, as do I, for first pass troubleshooting, say from a job error reporting "datasets deleted before job could run"), and notice that the dataset that was deleted in Step 2 above quickly flips between a) red bar with an undelete dataset link and b) regular file display of the contents (link is not usable, see the Ghelp ticket above for more about this situation).
  8. The collection cannot be used as an input (but can be selected on some tools forms and the run fails, and some bounce as containing deleted content directly on the form -- the latter is better).
  9. Big picture: how to recover is not obvious.

Expected behavior Reference/display the datasets actually inside the collection in all collection related views, including display views. As part of this, if a dataset is in a deleted state, leave the "click to undelete" view as a final state and not display the file contents (even if available under the hood). When undeleted, then it can be displayed.

Screenshots These are screenshots showing everything after Step 3 above .. .. Screen Shot 2024-03-14 at 10 55 27 AM .. .. Screen Shot 2024-03-14 at 10 55 52 AM .. .. Screen Shot 2024-03-14 at 10 56 23 AM .. .. Screen Shot 2024-03-14 at 10 56 52 AM .. .. Screen Shot 2024-03-14 at 10 57 13 AM .. .. Screen Shot 2024-03-14 at 12 01 40 PM

Additional context Test history screenshots are from. https://usegalaxy.org/u/jen-galaxyproject/h/test-ux-behavior-when-a-collection-contains-deleted-elements

mvdbeek commented 7 months ago

There are 2 independent issues here: