-
Notifications
You must be signed in to change notification settings - Fork 2
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
Support non-null DRS URIs in file descriptors #3631
Comments
@hannes-ucsc to move the PR along. |
Updated list of reviewers on HumanCellAtlas/dcp2#55 and notified straggler on Slack. |
@hannes-ucsc to merge PR on 10/20/22 unless Amnon request changes by then. |
HumanCellAtlas/dcp2#55 was merged. |
Follow up with CCHMC for timeline for new LungMAP snapshots with these DRS URIs. |
LungMAP is working on a new release with updates to the existing datasets. |
Apparently, |
Commit 58556e0 for #5824 disables the |
For an example of what one of these external DRS URIs looks like see #5769 (comment) |
Assignee to work with LungMap team to restart progress on the schema PR. |
This should probably be moved to backlog. |
Assignee to consider next steps. |
The workaround (58556e0 causes these external DRS URIs to be ignored by Azul and omitted from the index. There really isn't a point of putting them into the metadata if Azul simply ignores them. These DRS URIs are compact, e.g. In order to determine how much work we put into this, it would be good to know, what the expected user journey is for these files, according to the LungMAP project leadership. It's fine to not support direct downloads for them, but surely we should at least include their DRS URIs in the manifest. |
Assignee to discuss with LungMAP leadership. |
Needed for LungMAP managed access data files stored in BDCat.
Pre-requisite is a metadata schema change to support DRS URIs - to be proposed by Andrew Herbst (Broad).
@hannes-ucsc:
The text was updated successfully, but these errors were encountered: