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

tensorflow op Fill is not supported #79

Closed
pengwa opened this issue Jul 18, 2018 · 1 comment
Closed

tensorflow op Fill is not supported #79

pengwa opened this issue Jul 18, 2018 · 1 comment

Comments

@pengwa
Copy link
Collaborator

pengwa commented Jul 18, 2018

When I tried to convert dcgan inference model, TensorFlow Fill operator can not be converted well.

node {
  name: "generator/ones_1/shape_as_tensor"
  op: "Const"
  attr {
    key: "dtype"
    value {
      type: DT_INT32
    }
  }
  attr {
    key: "value"
    value {
      tensor {
        dtype: DT_INT32
        tensor_shape {
          dim {
            size: 4
          }
        }
        tensor_content: "@\000\000\000\016\000\000\000\016\000\000\000\n\000\000\000"
      }
    }
  }
}
node {
  name: "generator/ones_1/Const"
  op: "Const"
  attr {
    key: "dtype"
    value {
      type: DT_FLOAT
    }
  }
  attr {
    key: "value"
    value {
      tensor {
        dtype: DT_FLOAT
        tensor_shape {
        }
        float_val: 1.0
      }
    }
  }
}
node {
  name: "generator/ones_1"
  op: "Fill"
  input: "generator/ones_1/shape_as_tensor"
  input: "generator/ones_1/Const"
  attr {
    key: "T"
    value {
      type: DT_FLOAT
    }
  }
  attr {
    key: "index_type"
    value {
      type: DT_INT32
    }
  }
}

While on the other hand, the corresponding operator in ONNX ConstantFill is in experimental stage.

Since the mapping is pretty straight forward, I will create a pull request for fixing this.

@guschmue
Copy link
Collaborator

This should be fixed.

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

Successfully merging a pull request may close this issue.

2 participants