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

model optimizer failure with different dilation parameters for width and height #65

Closed
xonobo opened this issue Jan 10, 2019 · 4 comments
Labels
bug Something isn't working category: MO Model Optimizer question Further information is requested

Comments

@xonobo
Copy link

xonobo commented Jan 10, 2019

I can get an IR model of a caffe model by the model optimizer but when I look at the xml output I see that the repeated dilation paramaters for height and width are not parsed by the model optimizer.

Here is the convolutional layer in caffe prototxt:

layer {
  name: "conv1"
  type: "Convolution"
  bottom: "data"
  top: "conv1"
  convolution_param {
    num_output: 32
    kernel_w: 5
    kernel_h: 5
    stride: 1
    pad_w: 4
    pad_h: 0
    dilation: 1
    dilation: 2
  }
}

and this is the corresponding xml output of the MO:

<layer id="1" name="conv1" precision="FP16" type="Convolution">
			<data dilations="1,1" group="1" kernel="5,5" output="32" pads_begin="0,4" pads_end="0,4" strides="1,1"/>
			<input>
				<port id="0">
					<dim>1</dim>
					<dim>1</dim>
					<dim>32</dim>
					<dim>256</dim>
				</port>
			</input>
			<output>
				<port id="3">
					<dim>1</dim>
					<dim>32</dim>
					<dim>28</dim>
					<dim>260</dim>
				</port>
			</output>
			<blobs>
				<weights offset="0" size="1600"/>
				<biases offset="1600" size="64"/>
			</blobs>
		</layer>

The dilations in the xml should be "1,2". Is there any work around or solution to this case?

@xonobo
Copy link
Author

xonobo commented Jan 23, 2019

Correcting the xml file manually solves the issue but it is tedious and error prone. Need to correct the parser of the optimizer.

@lazarevevgeny lazarevevgeny added bug Something isn't working category: MO Model Optimizer labels May 25, 2020
@lazarevevgeny
Copy link
Contributor

@xonobo, is this issue still actual?

@lazarevevgeny lazarevevgeny added the question Further information is requested label May 25, 2020
@Adrianf1980
Copy link

Adrianf1980 commented May 25, 2020 via email

@AnastasiaKazantaeva
Copy link
Contributor

It seems that the issues is not actual anymore as no response. Closing it. Feel free to reopen it or create a new one.

ceciliapeng2011 referenced this issue in ceciliapeng2011/openvino May 25, 2021
eshoguli pushed a commit to eshoguli/openvino that referenced this issue Jun 1, 2021
* Azure CI: Add "ref: releases/2021/3"

* Fix Klocwork issues in Java

* Updated arm plugin build script (openvinotoolkit#65)

* Fix some Klocwork issues

* Update README.md (openvinotoolkit#76)

* Fixed Convolution fusion (openvinotoolkit#84)

* Remove MO deps from mo_pytorch module (openvinotoolkit#93)

Co-authored-by: azhogov <[email protected]>
Co-authored-by: Dmitry Kurtaev <[email protected]>
Co-authored-by: Aleksandr Voron <[email protected]>
Co-authored-by: Liubov Batanina <[email protected]>
slyalin added a commit to mvafin/openvino that referenced this issue Dec 20, 2022
…append_listunpack_transform

Add transformations for `prim::ListUnpack`
akashchi pushed a commit to akashchi/openvino that referenced this issue Dec 6, 2023
…factor

Rename getPrecision() to getElementType()
dmatveev added a commit to dmatveev/openvino that referenced this issue Jun 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working category: MO Model Optimizer question Further information is requested
Projects
None yet
Development

No branches or pull requests

4 participants