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

Non-graphical IRIs in the project #288

Merged
merged 8 commits into from
Nov 20, 2024

Conversation

mnamici
Copy link
Collaborator

@mnamici mnamici commented Nov 18, 2024

Changes related to the support of non-graphical IRIs in a graphol project, such as the ones
that can result by the inclusion of an external RDF dataset.

This was done to allow viewing and editing assertions about subjects
that are not part of the graphical graphol project, but can be seen
as arbitrary RDF triples representing additional metadata for subjects
not mentioned graphically.
No need to have a reference to avoid new instance creation
as python does this automatically.
Datatype and lang combos have the following behaviour now:
* If there is a datatype selected, disable the lang selector
* If there is a lang tag entered, disable the datatype selector

In RDF there cannot be a datatype and a lang string for any literal,
even if OWL 2 allows the rdf:PlainLiteral to have one, but this is
implicit, so stop the user from asserting that.
We already set the UI properly in the constructor, so there is
no need for such slot.
This includes datatypes, facets, and top/bottom IRIs since we want to
enable support for managing non-graphical IRI in the project.
Note that due to this, when selecting a subset of the diagrams you stll
get the IRIs of those, as there is no way to distinguish between those
coming for external sources and those from excluded diagrams.
@mnamici mnamici marked this pull request as ready for review November 19, 2024 18:58
@mnamici mnamici merged commit 5709b67 into obdasystems:develop Nov 20, 2024
14 of 15 checks passed
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.

1 participant