Skip to content

kg-pipeline recommended revisions #263

Description

@anngvu

Took a quick look, three things I'd do first:

  • Use Synapse IDs directly in the instance IRIs. Rather than minting a parallel w3id identifier (.../cckp-portal/data/Dataset/syn77073588), point at the canonical one: <https://www.synapse.org/Synapse:syn77073588> a cckp:Dataset. These entities already have stable identity in Synapse and a second identifier just means two things to keep in sync.
  • Dual-type with Biolink so we can query everything more easily. <instance_uri> a cckp:Dataset, biolink:Dataset. Once Neptune supports reasoning we can drop the additional materialization and just have a subclass axiom.
  • Emit manifest.ttl as part of the build; it doesn't need much, just have a prov statement about the graph build. Tom's auto-loader uses that file as its trigger.

Activity

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions