-
Notifications
You must be signed in to change notification settings - Fork 3
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
The "compiler" doesn't seem to be complete #54
Comments
Very strange! Here's the loop which does the inference: https://github.com/gtfierro/brick-tq-shacl/blob/main/brick_tq_shacl/topquadrant_shacl.py#L81 It should just be running the SHACL inference until the output file doesn't change in size (# of triples), but I will dig in more |
Isn't line 81 the problem?
...and _MAX_EXTERNAL_LOOPS is set to 3. |
Aha! Good find. Let me change that and see how it affects things |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
@gtfierro, I was testing our Pritoni file (NIST Example Building 1 Model 1) by downloading the "compiled" version, and was surprised to see that it does not contain all the inferred relations. Here's a screenshot in my environment after I ran the inference engine again. The purple ones were not present in the downloaded compiled file. It took two iterations beyond what was in the compiled file, so are you maybe stopping the iteration too soon? The "uncompiled" version takes 5 iterations.
The text was updated successfully, but these errors were encountered: