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

Error: Unbound record field Data.snapshot_file #5

Closed
hmedina opened this issue Oct 10, 2022 · 3 comments
Closed

Error: Unbound record field Data.snapshot_file #5

hmedina opened this issue Oct 10, 2022 · 3 comments

Comments

@hmedina
Copy link
Collaborator

hmedina commented Oct 10, 2022

Trying to re-build the TQL engine after KaSim updates, I'm running into the below error:

~/Kappa-TQL$ make
ocamlbuild -use-ocamlfind main.native
+ ocamlfind ocamlopt -c -g -bin-annot -safe-string -open Kappa_generic_toolset -open Kappa_mixtures -open Kappa_terms -open Kappa_runtime -package kappa-library.runtime -package biniou -I src -o src/measures.cmx src/measures.ml
File "src/measures.ml", line 57, characters 8-26:
57 |         Data.snapshot_file = file ;
             ^^^^^^^^^^^^^^^^^^
Error: Unbound record field Data.snapshot_file
Hint: Did you mean snapshot_time?
Command exited with code 2.
Compilation unsuccessful after building 56 targets (0 cached) in 00:00:06.
make: *** [Makefile:4: all] Error 10

https://github.com/jonathan-laurent/Kappa-TQL/blob/502f7e1b046129af0374452ed694c4171c2e8c74/src/measures.ml#L57

@jonathan-laurent
Copy link
Owner

Nice to hear from you Hector!
I'll be looking at this in the next few days.

More generally, I think there is a case to be made that the TQL should be included in the main KaSim repo. This way, it won't get broken at each major update.

@hmedina
Copy link
Collaborator Author

hmedina commented Oct 12, 2022

Hi Jonathan! I think it would be perfectly acceptable for the TQLe to be one of the agents in the Kappa Tools umbrella, like the ODE generator (KaDe), static analyzer (KaSa) or the story thing (KaStor). Replace the "query" for an "inquiry", and you're halfway to a pronounceable acronym! Perhaps the Kappa Trace Inquiry Engine (KaTie) ?

FYI, I'm running into issues compiling KaSim (Kappa-Dev/KappaTools#644), so I'll be taking a closer look at its github actions file in the coming days...

@jonathan-laurent
Copy link
Owner

I just pushed a fix and switched the build system to dune in passing. Enjoy!

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

2 participants