Failure in generated op.h in version 1.3.1

[Bumblebee1964]

Note: Providing complete information in the most concise form is the best way to get help. This issue template serves as the checklist for essential information to most of the technical issues and bug reports. For non-technical issues and feature requests, feel free to present the information in what you believe is the best form.

For Q & A and discussion, please start a discussion thread at https://discuss.mxnet.io

Description

When compiling MXNET with CPP package, an op.h header is generated with syntax errors. Compared to version 1.3.0, some operators have been added to have this syntax error (missing type specifier in argument list of inline function)

Environment info (Required)

WINDOWS 10 Pro, X64
MXNET 1.3.1.
VS2017
MKL
OPENCV 3.4.5
CMake 3.13.4
CUDA 9.2
CUDNN 9.2

Build info (Required if built from source)

Compiler: VS2017

MXNet commit hash:
(Paste the output of git rev-parse HEAD here.)

Build config:
1>------ Build started: Project: cpp_package_op_h, Configuration: Release x64 ------
1>"Running: OpWrapperGenerator.py"
1>D:/Projects/MXNet-versions/MxNet1-3-1/build/Release/libmxnet.dll
1>argument "lrs" of operator "multi_sgd_update" has unknown type ", required"
1>argument "wds" of operator "multi_sgd_update" has unknown type ", required"
1>argument "lrs" of operator "multi_sgd_mom_update" has unknown type ", required"
1>argument "wds" of operator "multi_sgd_mom_update" has unknown type ", required"
1>argument "lrs" of operator "multi_mp_sgd_update" has unknown type ", required"
1>argument "wds" of operator "multi_mp_sgd_update" has unknown type ", required"
1>argument "lrs" of operator "multi_mp_sgd_mom_update" has unknown type ", required"
1>argument "wds" of operator "multi_mp_sgd_mom_update" has unknown type ", required"
========== Build: 1 succeeded, 0 failed, 3 up-to-date, 0 skipped ==========

Problem in generated code (the parameters lrs and wds):
inline Symbol multi_sgd_update(const std::string& symbol_name, const std::vector<Symbol>& data, lrs, wds, mx_float rescale_grad = 1, mx_float clip_gradient = -1, int num_weights = 1) { return Operator("multi_sgd_update") .SetParam("lrs", lrs) .SetParam("wds", wds) .SetParam("rescale_grad", rescale_grad) .SetParam("clip_gradient", clip_gradient) .SetParam("num_weights", num_weights) (data) .CreateSymbol(symbol_name); }

The same for the inline functions:
multi_sgd_mom_update
multi_mp_sgd_update
multi_mp_sgd_mom_update
..and overloaded functions

Error Message:

Minimum reproducible example

Steps to reproduce

What have you tried to solve it?

1. op.h generation worked with same system / compiler settings for 1.3.0

[mxnet-label-bot]

Hey, this is the MXNet Label Bot.
Thank you for submitting the issue! I will try and suggest some labels so that the appropriate MXNet community members can help resolve it.
Here are my recommended labels: Bug

[Bumblebee1964]

Suggested labels: build, cpp
@mxnet-label-bot add [build, cpp]

[Bumblebee1964]

dug a bit deeper in the problem. It seems that the new inline functions have an argument type that is not listed as known argument types in OpWrapper.py. I guess the argument type is nnvm::Tuple but not sure.

[smissan]

this is a critical issue that breaks CPP interface. Please fix

[bj5546]

release 1.40 has the same erro

[Bumblebee1964]

Please fix, it is annoying to work around this issue especially when trying to understand cpp samples as these won't build out of the box.

[dbsxdbsx]

@Bumblebee1964 , and is especially annoying when it pops up after 1 hours compiling! I got it when compiling mxnet1.5.

[ausk]

Rebuild more than 5 times, building failed when run OpWrapperGenerator.py, finally locate to this issue ...

[bj5546]

was this bug fixed？

[smissan]

No

…

On Mon., Mar. 25, 2019, 8:56 a.m. bj5546, ***@***.***> wrote: was this bug fixed？ — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#14116 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AA3wWnekCkMTGJz60hLQvUzkD_Yq1xBYks5vaLmEgaJpZM4az7n5> .

[Bumblebee1964]

It seems fixed in the latest version. I downloaded the code 2 weeks ago, and it seemed to be resolved.

[smissan]

The script did not change in 2 months and is still not working

…

On Tue., Mar. 26, 2019, 5:21 a.m. Fred Hugen, ***@***.***> wrote: It seems fixed in the latest version. I downloaded the code 2 weeks ago, and it seemed to be resolved. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#14116 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AA3wWgwsOJs7CWwiaFzRHSn_l_hfm0K_ks5vadh1gaJpZM4az7n5> .