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

CAROs cyclicity prevents it being displayed correctly in OLS #21

Open
matentzn opened this issue Mar 17, 2020 · 3 comments
Open

CAROs cyclicity prevents it being displayed correctly in OLS #21

matentzn opened this issue Mar 17, 2020 · 3 comments

Comments

@matentzn
Copy link
Contributor

See: EBISPOT/OLS#324

If you want CARO to be displayed correctly, someone needs to go and figure out what has happened. This is often not trivial. Also, fixing this may also fixing some underlying issues with the modelling, so its probably worth it.

@matentzn
Copy link
Contributor Author

matentzn commented Apr 16, 2020

Some help please @balhoff @dosumis

Seems the problem is caused by "multi-modal cycles", i.e. cycles across different relations.

BFO_0000004 (independent continuant)
	BFO_0000040 (material entity)
		CARO_0001010 (organism or virus or viroid)
			CARO_0010004 (Cellular organism)
	CARO_0030000 (biological entity) --> part of CARO_0010004 (Cellular organism)
		CARO_0000000 (anatomical entity)
			CARO_0000006 (Material anatomical Entity)
				CARO_0000003 (Connected anatomical structure)
					CARO_0010004 (Cellular organism)

As you can see

  • CARO_0010004 ---------------->CARO_0030000 and
  • CARO_0030000 ----[part of]--->CARO_0010004

Can you comment on whether this is even conceptually sensible?

@balhoff
Copy link
Member

balhoff commented Apr 16, 2020

I think it's totally valid logically. I'm surprised it's not a problem for OLS elsewhere; I would expect this kind of situation to show up in various ontologies.

But in this case I don't think the part_of relation is correct. It seems like there are biological entities that are not parts of cellular organisms (like virus structures).

@matentzn
Copy link
Contributor Author

Yeah of course, these kinds of cycles are logically valid.. No I meant that particular part-of relation.

@udp is working on dealing with the OLS issue - that is just a bug after all.

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

2 participants