-
Notifications
You must be signed in to change notification settings - Fork 143
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
Add tracing kernel #1309
base: develop
Are you sure you want to change the base?
Add tracing kernel #1309
Conversation
Your Note how the Moscow ML build fails. You can get Moscow ML to work with complicated mixtures of functors, structures and signatures all in the one file, but it’s unnecessary work in cases like this. |
Is there a way to make |
Hm, I'm not really sure how to do the ifdef in this situation. I would like to have a function which is enabled only for polyml+trknl, and which is not assumed to compile on any other combination. It is unclear to me why references to the |
There's an example of this sort of dance in the way
Moscow ML is happy with this "style". |
Is there an easy fix for the regression here? Would be nice to resolve the PR if possible. |
This project has gone to the backburner for me but I think I might be better equipped to understand the issues and get it to build now. |
Adds a new kernel type,
--trknl
. This is mostly the same as the standard kernel, but it uses the OT kernel for theThm
type. There is also one additional step during theexport_theory
command, in which the export data is pickled directly into a byte blob (preserving sharing) and written out. The result is postprocessed withgzip
to avoid the traces from getting too large.This depends on my fork of PolyML for the
exportSmall
function, which was added for a very similar use case in Isabelle.