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

Add linkouts to CellGuide for curated/validated defs #2216

Open
Tracked by #2081
dosumis opened this issue Nov 7, 2023 · 6 comments
Open
Tracked by #2081

Add linkouts to CellGuide for curated/validated defs #2216

dosumis opened this issue Nov 7, 2023 · 6 comments

Comments

@dosumis
Copy link
Contributor

dosumis commented Nov 7, 2023

For all terms in https://docs.google.com/spreadsheets/d/13NJsDH0fBZEFYuGZYkJ2QUah9WV9731zliiW-9uOTuA/edit#gid=2059655613

xrefs → CL for all reviewed definitions
Bioregistry needs a cell Guide entry - base https://cellxgene.cziscience.com/cellguide/
or
https://cellxgene.cziscience.com/cellguide/CL_
CL terms get xrefs: cellguide:CL_0002623 OR cellguide:0002623

Once released this should => links to CellGuide on OLS4

@dosumis
Copy link
Contributor Author

dosumis commented Nov 15, 2023

@anitacaron - just checking this is on agenda for tech team. Thanks!

@anitacaron
Copy link
Contributor

Requested new prefix in Bioregistry biopragmatics/bioregistry#1001

@dosumis
Copy link
Contributor Author

dosumis commented Dec 4, 2023

@anitacaron - Given discussion on bioregistry and linked OLS tickets, I think we should change plan and add URLs directly to term using seeAlso as predicate. I would also like to annotate this axiom with some link text so that consumers of the ontology don't have to display raw URLs. Not sure of best predicate for that. Maybe rdfs:label?

@dosumis
Copy link
Contributor Author

dosumis commented Jan 15, 2024

@hkir-dev - can you add CellGuide URLs as part of the process for adding subset tags for CxG Census? We will also need CellGuide URLs for ancestor classes in CL for all terms in census. These should be rdfs:SeeAlso AP + typed IRI. Could you also investigate whether there is a standard way add hyperlink text to links in seeAlso?

Copy link

This issue has not seen any activity in the past 6 months; it will be closed automatically in one year from now if no action is taken.

@github-actions github-actions bot added the Stale label Jul 14, 2024
@dosumis
Copy link
Contributor Author

dosumis commented Sep 10, 2024

Closing ticket as we are now we are adding xrefs with https - as part of this template https://github.com/obophenotype/cell-ontology/blob/master/src/patterns/dosdp-patterns/ExtendedDescription.yaml#L35

(Maybe better to switch to seeAlso? )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants