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

Remove BFO:processual entity #65

Open
wdduncan opened this issue Nov 21, 2019 · 10 comments
Open

Remove BFO:processual entity #65

wdduncan opened this issue Nov 21, 2019 · 10 comments
Assignees

Comments

@wdduncan
Copy link
Member

The term 'processual entity' should not be in the ontology. BFO now endorses the term 'process'.

@cmungall
Copy link
Member

is this an ecto problem?

@matentzn
Copy link
Contributor

processual entity is not a BFO class: it is an UBERON class.

We cant not use it :P If anything, you can make an UBERON ticket to ask to align processual entity with BFO:process.. Again, not an ECTO issue but feel free to re-open if you disagree.

@wdduncan wdduncan reopened this Nov 22, 2019
@wdduncan
Copy link
Member Author

Yes. I disagree :)
When you do your import from UBERON, you placed the entities under process. You do not have to be beholden to processual entity.

@matentzn
Copy link
Contributor

Oh ok.. I am afraid I dont quite understand yet; which entities have we placed under process? Can you give an example? I believe, but @diatomsRcool correct me if I am wrong, we have not actually added a single axiom to ECTO:

https://github.com/EnvironmentOntology/environmental-exposure-ontology/blob/master/src/ontology/ecto-edit.owl

everything is either generated by dosdp (no manual assertions) or coming from external ontologies..

@wdduncan
Copy link
Member Author

Under process (BFO_0000015) I see:

  • biological process
  • environmental system process
  • exposure event or process
  • molecular function
  • planned process
  • regulation of behavior

In regard to 'processual entity', this was originally a BFO 1.1 class.
@cmungall Are you able to use the BFO 'process' class instead of 'processual entity'? I am not familiar with how UBERON is built.

@matentzn
Copy link
Contributor

Ah you say that UBERON added processual entity as a replacement for BFO:processual entity, and your suggestion is to obsolete UBERON:processual entity in favour of BFO:process?

@wdduncan
Copy link
Member Author

Processual entity used to be included in BFO 1. In BFO 2, the authors created a process class that replaced processual entity.

UBERON, for reasons I am not aware of, retained processual entity. Processual entity has an UBERON IRI, but the definition is the same as the BFO 1 class. It would make more sense (IMHO) to move the entities under processual entity to be under process.

@matentzn
Copy link
Contributor

Ok, maybe move that ticket there then?

https://github.com/obophenotype/uberon/issues

@wdduncan
Copy link
Member Author

We should discuss with @cmungall

@diatomsRcool
Copy link
Contributor

Where are we with this? Is this an ECTO issue?

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

No branches or pull requests

4 participants