Adam Crume

Results 68 comments of Adam Crume

It should definitely be sealed. This would be a great easy improvement for someone in the community to work on.

Definitely! I look forward to seeing the PR.

There isn't anything like that at the moment, although it sounds like a good idea. Do you have any specific questions that I could use as a starting point?

There's an example in `examples/regression_savedmodel.rs`. Short answer: use `Session::from_saved_model`. If it still doesn't work, share your code, and we'll help. By the way, we encourage people to ask "How do...

Sorry, I was out for a week for family reasons. That seems like a reasonable way to load the ops. I can't find any handy shared library of all contrib...

This is expected. If it doesn't find TensorFlow installed locally, it will download a binary. If you want to build without internet access, you'll need to install TensorFlow locally and...

It does support a way to specify the location of libraries manually, and this is through `pkg-config`. The instructions for this are in https://github.com/tensorflow/rust/blob/master/tensorflow-sys/README.md, specifically the `generate-pc.sh` step.

Hm, it looks like I've been ignoring tensorflow-internal-macros during releases. I'll update the release instructions to fix that. The tensorflow-internal-macros crate is entirely internal and should not be depended on...

I'm unable to reproduce the issue, even running under valgrind. Can you verify which TensorFlow library is getting pulled in? You can try building with `-v` if it's not clear....

It'd be easy to look for a `.dll` in addition to a `.lib`. I have little Windows experience, though, so I don't know whether that fully fixes the linking problem...