Skip to content
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

Sample links to registered files are not displaying the the Datafile samples table view #2052

Closed
AndrewWood94 opened this issue Nov 8, 2024 · 4 comments
Labels
Milestone

Comments

@AndrewWood94
Copy link
Collaborator

When extracting Samples which include a 'Registered Data File' attribute, the link to the associated datafile works when viewing the confirm extraction table (e.g. column 'ImageFile' here).

However, once the file has been extracted, the Datafile samples table displays [object Object] instead of the linked datafile
(example here)

@stuzart
Copy link
Member

stuzart commented Nov 11, 2024

thanks @AndrewWood94 . I think this a UI issue and case of it not being updated as the features have evolved.

The button to view samples was originally added just to show extracted samples, before there was the ability to link data files through a sample attribute. Since this was added, the related_samples has been updated, and does include the linked samples as well as the extracted ones, but clicking the link to view all takes you to data_files/n/samples which just displays the extracted ones.

I think it needs updating so that the button is clearer that it is to show only the extracted samples, with its own route, but then data_files/n/samples should show the usual filtered view and include all related samples.

@stuzart stuzart added this to the 1.16.0 milestone Nov 11, 2024
@stuzart stuzart moved this to In progress in SEEK 1.16.x Nov 11, 2024
@stuzart stuzart moved this from In progress to Ready in SEEK 1.16.x Nov 11, 2024
stuzart added a commit that referenced this issue Nov 12, 2024
stuzart added a commit that referenced this issue Nov 12, 2024
@stuzart stuzart moved this from In progress to In review in SEEK 1.16.x Nov 13, 2024
stuzart added a commit that referenced this issue Nov 14, 2024
stuzart added a commit that referenced this issue Nov 14, 2024
@AndrewWood94
Copy link
Collaborator Author

Is it possible to have samples related to a datafile which are not extracted from the datafile?
If not, I think it would be good for the ISA graph to link to extracted samples rather than related samples, so users can see the sample data, not just metadata

@stuzart
Copy link
Member

stuzart commented Nov 15, 2024

yes, it is possible to have other samples related to a data file. Those that have been linked through the Registered DataFile sample attribute. So, data_file.related_samples can contain a mixture of those extracted and others that are linked through this attribute.

@stuzart
Copy link
Member

stuzart commented Nov 15, 2024

As mentioned on wednesday, fixing this raised some other issues so there is an issue for future improvements #2055 . But closing this as the underlying problem reported has been fixed.

@stuzart stuzart closed this as completed Nov 15, 2024
@github-project-automation github-project-automation bot moved this from In review to Done in SEEK 1.16.x Nov 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

No branches or pull requests

2 participants