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

Support for HCA duos_id #6196

Open
dsotirho-ucsc opened this issue Apr 23, 2024 · 10 comments
Open

Support for HCA duos_id #6196

dsotirho-ucsc opened this issue Apr 23, 2024 · 10 comments
Assignees
Labels
+ [priority] High code [subject] Production code demo [process] To be demonstrated at the end of the sprint orange [process] Done by the Azul team spike:1 [process] Spike estimate of one point

Comments

@dsotirho-ucsc
Copy link
Contributor

Factored out of #6131 due to the fact that current metadata does not contain this field as of today.

Index project.duos_id, expose in all /index/… responses.

@dsotirho-ucsc dsotirho-ucsc added the orange [process] Done by the Azul team label Apr 23, 2024
@achave11-ucsc achave11-ucsc added enh code [subject] Production code - [priority] Medium labels Apr 24, 2024
@achave11-ucsc
Copy link
Member

Assignee to periodically check for availability of duos_id in snapshots for pilot catalog, or dcpXY catalogs once manage access snapshots are available in prod.

@bvizzier-ucsc
Copy link

@dsotirho-ucsc When was the last time this was checked?

Let me know if it isn't there and I'll escalate it.

@dsotirho-ucsc
Copy link
Contributor Author

@bvizzier-ucsc This was last checked for in dcp42, but I somehow missed the managed access snapshot. Just re-checked and confirmed the duos_id field in project 35d5b057's snapshot.

@achave11-ucsc
Copy link
Member

@hannes-ucsc: "Spike to confirm with CC that it is sufficient to return this property in the hits part of the response (no filtering, no sorting, no faceting)"

@achave11-ucsc achave11-ucsc added the spike:1 [process] Spike estimate of one point label Oct 8, 2024
@dsotirho-ucsc
Copy link
Contributor Author

Received confirmation from CC that adding the duos_id field to the hits part of the response would be sufficient.

@nadove-ucsc
Copy link
Contributor

@hannes-ucsc: "At least for MA snapshots, the HCA metadata project entity has a DUOS Id property that we will use. For non-MA projects the DUOS Id may be absent, but getting it from TDR as we do for AnVIL would be much more complicated, as it would require a special bundle type, for which we have no support in HCA."

@bvizzier-ucsc
Copy link

The requirement for the DUOS ID is to request access for a project via DUOS. As a result, non-MA projects should not require it.

@bvizzier-ucsc
Copy link

@hannes-ucsc: "In this case it will be relatively easy to implement."

@bvizzier-ucsc bvizzier-ucsc added + [priority] High and removed - [priority] Medium labels Dec 12, 2024
@dsotirho-ucsc
Copy link
Contributor Author

Moved to Triage due to the merge of the blocking ticket #6620

@achave11-ucsc achave11-ucsc removed the enh label Feb 11, 2025
@hannes-ucsc hannes-ucsc added the demo [process] To be demonstrated at the end of the sprint label Feb 27, 2025
@hannes-ucsc
Copy link
Member

For demo, show new field in response. Show facets, filtering and sorting by new field.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
+ [priority] High code [subject] Production code demo [process] To be demonstrated at the end of the sprint orange [process] Done by the Azul team spike:1 [process] Spike estimate of one point
Projects
None yet
Development

No branches or pull requests

5 participants