microsoft / knossos-ksc

Compiler with automatic differentiation
Other
45 stars 10 forks source link

Add optional logging for pybind ks calls #839

Closed dcrc2 closed 3 years ago

dcrc2 commented 3 years ago

Extracted from #762, with some simplifications from our meeting earlier.

dcrc2 commented 3 years ago

@awf I've reimplemented this because my original version changed the signature of the entry function(s) in the pybind module, which several tests were relying on. Having had a look at those tests, I don't think we really want to change the signature. So I've written an alternative which avoids this by moving the global flag into the pybind module.