Feature Request: GPUOptions for Go binding #22926
Comments
Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks. |
This problem appears to be broader than just specifying See, for example, the test case func TestSessionConfig(t *testing.T) {
// Exercise SessionOptions.
// Arguably, a better API would be for SessionOptions.Config to be the
// type generated by the protocol buffer compiler. But for now, the
// tensorflow package continues to be independent of protocol buffers
// and this test exercises the option since the implementation has a
// nuanced conversion to C types.
//
// Till then, the []byte form of Config here was generated using a toy
// tensorflow Python program:
/*
import tensorflow
c = tensorflow.ConfigProto()
c.intra_op_parallelism_threads = 1
print c.SerializeToString()
*/
graph := NewGraph()
c, err := Const(graph, "Const", int32(14))
if err != nil {
t.Fatal(err)
}
opts := SessionOptions{Config: []byte("(\x01")}
[...] Would the TensorFlow maintainers accept a non-Google contribution that added the ability to specify session options using pure Go code? This functionality would require generating instances of the |
@asimshankar Can you please take a look? Thanks! |
@frreiss we'll definitely accept a Go-only API to configure tensorflow. |
I am looking into this. |
Created PR #26682 with a Go API to create ConfigOptions protocol buffer messages. I made the changes as narrow in scope as I could. |
#26682 is reverted 6e9cb40#diff-dcd28ad951bd17e9c512d3b564640bab |
is this issue open? Can i work on it? @mattn |
Yes please. |
May I know how to reproduce the issue? |
Sorry, I don't know why the changes was reverted. |
ok no problem I will try some other issues |
Hi,Adesh I am beginner i don't where start Can anybody help me????? |
Hello! |
can someone guide me what to do? i will be really thankful |
Kindly close this issue. It's confusing for first time contributors who come here for solving their first Issue. |
I don't mind to close this but no one explain why #26682 was reverted. |
is this still open? |
is it still open ? |
@mattn is this issue closed? |
Assign me |
relax people! if u are here to solve these tasks, u are a beginner anyway. |
can any one guide me to this issue |
Current implementation of Go binding can not specify options.
GPUOptions struct is in internal package. And
go generate
doesn't work for protobuf directory. So we can't specify GPUOptions forNewSession
.The text was updated successfully, but these errors were encountered: