Skip to content

Remove several fields used less than five times#2899

Merged
deepakunni3 merged 8 commits intoOBOFoundry:masterfrom
cthoyt:remove-more-fields
Apr 20, 2026
Merged

Remove several fields used less than five times#2899
deepakunni3 merged 8 commits intoOBOFoundry:masterfrom
cthoyt:remove-more-fields

Conversation

@cthoyt
Copy link
Copy Markdown
Collaborator

@cthoyt cthoyt commented Apr 17, 2026

As a follow-up to #2892, this PR removes the following fields, each of which was used less than five times:

  • canonical (not needed because of convention)
  • documentation (duplicated homepage)
  • integration_server (irrelevant)
  • label (duplicate of title field)
  • releases (trivally computable from repository field)
  • wasDerivedFrom (only in NCBITaxon)
  • wikidata_template (only in UBERON)
  • slack (only in UBERON)

This was done by running the script:

$ python -m obofoundry.remove_field canonical documentation integration_server label releases wasDerivedFrom wikidata_template slack

If you want to see which fields are not frequently used, run:

$ python utils/check_schema.py --max-cutoff 20
key count ontologies
twitter 10 go, hp, uberon, chebi, doid, ddpheno, po, cl, disdriv, ddanat
in_foundry 11 obcs, bco, epo, zfs, omit, stato, bcgo, flu, fix, uo, lipro
in_foundry_order 11 pbpko, go, pr, obi, chebi, doid, pato, po, bfo, zfa, xao
funded_by 4 nomen, uberon, eco, kisao
pull_request_added 4 clyh, t4fs, clao, ngbo
issue_requested 3 occo, t4fs, ngbo

@cthoyt cthoyt requested review from deepakunni3 and matentzn April 17, 2026 07:03
Copy link
Copy Markdown
Contributor

@matentzn matentzn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QC issues but the content is great from my perspective

@cthoyt
Copy link
Copy Markdown
Collaborator Author

cthoyt commented Apr 17, 2026

@matentzn fixed by adding uv installation to the workflows

@matentzn
Copy link
Copy Markdown
Contributor

@deepakunni3 I have no issues with these changes and approve of them. Can you sanity check?

Copy link
Copy Markdown
Member

@deepakunni3 deepakunni3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @cthoyt for removing less frequently used fields.
The PR looks good to me 👍

@deepakunni3 deepakunni3 merged commit 3a31f3a into OBOFoundry:master Apr 20, 2026
3 checks passed
@cthoyt cthoyt deleted the remove-more-fields branch April 20, 2026 09:19
@nataled
Copy link
Copy Markdown
Contributor

nataled commented Apr 20, 2026

I'm pretty sure in_foundry and in_foundry_order are no longer needed. The tip-off was the list of in_foundry_order, which mirrors the ontologies that passed review to become a full-fledged Foundry member (as opposed to Library). As the 'Foundry' and 'Library' distinctions are no longer made, these tags will not be used anywhere. Please double check against now-in-use processes to make sure.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants