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

explain "compiled" models #35

Closed
VladimirAlexiev opened this issue Aug 30, 2024 · 3 comments · Fixed by #55
Closed

explain "compiled" models #35

VladimirAlexiev opened this issue Aug 30, 2024 · 3 comments · Fixed by #55

Comments

@VladimirAlexiev
Copy link
Contributor

VladimirAlexiev commented Aug 30, 2024

https://models.open223.info/examples/bdg1-1.html (and most other models) have two variants:

Turtle file (compiled) (original)

By eyeballing, "Compiled" is much worse since it uses generated prefixes (eg ns2:) and doesn't use instance data prefixes.
So please explain in the intro what are Compiled models for and whether they include more info than Original models.

Maybe the only benefit is that ValidationReports print instance URLs in full, so it's a bit easier to look them up in the Compiled model?

ns2:commandedByProperty is not defined for instance <http://data.ashrae.org/standard223/1.0/data/pd-sr-mp-pritoni#VAVBox2ReheatValveActuator>.

@VladimirAlexiev
Copy link
Contributor Author

#54 by @steveraysteveray
suggests that "compiling" makes iterative inference, as much as 5 iterations.

I've seen presentations by @gtfierro that show how 233 has a lot more nodes (details) than Bricks and Haystack, yet you can derive one from the other.

Is compilation related to this?

@VladimirAlexiev
Copy link
Contributor Author

#10 and https://github.com/open223/models.open223.info/blob/main/tools/compile.py suggest that SHACL rules are applied

@steveraysteveray
Copy link
Contributor

Yes, they are.

VladimirAlexiev added a commit to VladimirAlexiev/models.open223.info that referenced this issue Sep 18, 2024
gtfierro pushed a commit that referenced this issue Sep 23, 2024
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 a pull request may close this issue.

2 participants