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

Query diagnostics #59

Open
nsbgn opened this issue Nov 8, 2021 · 6 comments
Open

Query diagnostics #59

nsbgn opened this issue Nov 8, 2021 · 6 comments

Comments

@nsbgn
Copy link
Contributor

nsbgn commented Nov 8, 2021

For real-life examples, it is often hard to see why a particular query does not find the corresponding workflow we would expect. An explanation module would help: it would trace every step and tell you why a query does not find a result for a particular graph, that is, show you the first point at which the two diverge.

@nsbgn
Copy link
Contributor Author

nsbgn commented Dec 9, 2021

We will need diagnostics also: how many types occur in each workflow? How many of those are in the right order? How many operators occur in each? Etcetera.

@nsbgn nsbgn changed the title Explanation module Query diagnostics Dec 11, 2021
@nsbgn nsbgn added this to the Version 0.2: RDF support milestone Dec 11, 2021
@nsbgn
Copy link
Contributor Author

nsbgn commented Jan 13, 2022

@nsbgn
Copy link
Contributor Author

nsbgn commented Jan 21, 2022

If you query for [A, f], e.g. a type A that eventually results from operation
f, then if you find no f on the branch before A, clearly it makes no sense to
search for another A on that branch that occurs before the other A --- there
will still not be an A. This should be optimized, but a test should make sure
that other branches are still inspected:

A <-- ... <- g
    `- ... <- f

nsbgn added a commit that referenced this issue Jan 23, 2022
This will allow us to change the query code much faster and allow us to
address issues #46 and #59. It immediately solves issue #60.
@nsbgn
Copy link
Contributor Author

nsbgn commented Feb 7, 2022

Cf commit e9ed04c.

nsbgn added a commit that referenced this issue Feb 10, 2022
@nsbgn
Copy link
Contributor Author

nsbgn commented Feb 11, 2022

Every node should get a unique label so it's easier to see what happens.

nsbgn added a commit that referenced this issue Feb 11, 2022
This will make it easier to refer to specific nodes to diagnose
problems. See issue #59.
nsbgn added a commit that referenced this issue Feb 11, 2022
@nsbgn nsbgn removed this from the Version 0.2: RDF support milestone Nov 2, 2022
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

1 participant