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

Cardinality Validation View to Include Identifiers to Offenders #987

Open
chrisage opened this issue Aug 3, 2023 · 1 comment
Open

Cardinality Validation View to Include Identifiers to Offenders #987

chrisage opened this issue Aug 3, 2023 · 1 comment
Assignees
Labels
enhancement Improvements to an existing feature rack-ui Python web rack UI

Comments

@chrisage
Copy link
Contributor

chrisage commented Aug 3, 2023

Background:
In the RACK UI Cardinality Explorer Tab cardinality validations are listed in a tree hierarchy down to the entity type. To quicken debugging the cardinality issue we should expose the specific identifiers to the entities.

image

Acceptance Criteria:
Individual errors should be associated with specific identifiers to offending entities

Implementation Details:

Related: #986

@chrisage chrisage added enhancement Improvements to an existing feature needs-details Unactionnable until we get more information rack-ui Python web rack UI and removed needs-details Unactionnable until we get more information labels Aug 3, 2023
@cuddihyge
Copy link
Contributor

SemTK is not RACK specific. How do we propose that SPARQLgraph be told that "identifier" is the important property?
I'm not saying it can't be done (Franz Gruff) has a similar feature.
But it isn't an obvious or quick enhancement.
The RACK model doesn't even specify that "identifier" is special.
To mimic the functionality in Gruff, perhaps the user could set "identifier" as some kind of key property and SPARQLgraph could store the choice in a cookie and then use that information when executing and displaying the results of all CONSTRUCT queries. We'd need a mechanism to handle the cases where a CONSTRUCT query on the query tab does not include identifier. I don't care for Franz' solution of automatically modifying a query to add additional returns, so I don't think we should add identifier in those cases.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improvements to an existing feature rack-ui Python web rack UI
Projects
None yet
Development

No branches or pull requests

3 participants