Skip to content

Stop merging deprecated terms into the released OWL; keep the archive repo-side only #600

Description

@turbomam

METPO should keep its record of obsolete terms in the repo but stop shipping it in the released OWL, so obsolete terms no longer appear in OLS and other downstream browsers.

This reverses the decision in #378 Decide whether deprecated entities should be in the released OWL, closed 2026-06-08, which resolved to include them.

Current state. src/templates/deprecated.tsv holds 1217 data rows and is the declared source of truth for burned IDs. INCLUDE_OBSOLETE (default true) merges it into the build. The current build emits 1216 owl:deprecated classes; OLS4 reports 1455 terms for version 2026-06-12, so most of what OLS shows is obsolete.

Proposed change. Flip the INCLUDE_OBSOLETE default to false. Keep deprecated.tsv committed and hand-maintained, unchanged. Regenerate and commit the affected artifacts in the same PR.

Archive format. deprecated.tsv already satisfies "a TSV that ROBOT can convert to OWL", so no new committed artifact is strictly needed. Recommendation is to keep the TSV as the only committed form and generate the OWL on demand, because a committed generated OWL is a second copy that drifts. If a consumable OWL is wanted, attach metpo-obsolete.owl to tagged releases with its own ontology IRI (https://w3id.org/metpo/metpo-obsolete.owl, not the main one) rather than committing it.

Cost to weigh before merging. kg-microbe reads metpo.owl from main. Once obsolete terms leave the release, kg-microbe and other consumers can no longer detect that a referenced ID is obsolete. That was the argument that carried #378, and it does not go away. Decide deliberately whether the cleaner OLS presence is worth it.

Do not couple this to deprecation cleanup. #521 Deprecation metadata review found that 643 of 1217 obsolete terms are cross-era duplicates with mechanically assigned reasons. For OLS presentation that is a defect; for an ID-burn archive it is not, because every burned ID belongs on the record including duplicates. Fix presentation there, not here.

Check before merging. make audit-ids and reports/id-allocation-audit.md must still see the full burned-ID set from the TSV once the merge step is off.

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