[GitHub] [incubator-mxnet] samskalicky commented on issue #17034: Dynamic subgraph property

2020-01-08 Thread GitBox
samskalicky commented on issue #17034: Dynamic subgraph property URL: https://github.com/apache/incubator-mxnet/pull/17034#issuecomment-572313680 @wkcn ready to go! This is an automated message from the Apache Git Service. To

[GitHub] [incubator-mxnet] samskalicky commented on issue #17034: Dynamic subgraph property

2020-01-07 Thread GitBox
samskalicky commented on issue #17034: Dynamic subgraph property URL: https://github.com/apache/incubator-mxnet/pull/17034#issuecomment-571781708 apologies @eric-haibin-lin my mistake on the comments. Thanks for the suggestions, ive gone through and changed the variable names to use the

[GitHub] [incubator-mxnet] samskalicky commented on issue #17034: Dynamic subgraph property

2020-01-07 Thread GitBox
samskalicky commented on issue #17034: Dynamic subgraph property URL: https://github.com/apache/incubator-mxnet/pull/17034#issuecomment-571722683 @mxnet-label-bot update [pr-awaiting-merge] This is an automated message from

[GitHub] [incubator-mxnet] samskalicky commented on issue #17034: Dynamic subgraph property

2020-01-07 Thread GitBox
samskalicky commented on issue #17034: Dynamic subgraph property URL: https://github.com/apache/incubator-mxnet/pull/17034#issuecomment-571485298 Thanks @TaoLv I created this issue: #17236 This is an automated message from

[GitHub] [incubator-mxnet] samskalicky commented on issue #17034: Dynamic subgraph property

2020-01-06 Thread GitBox
samskalicky commented on issue #17034: Dynamic subgraph property URL: https://github.com/apache/incubator-mxnet/pull/17034#issuecomment-571397143 Thanks for the review @eric-haibin-lin! ive made changes based on your feedback, updated the PR description with "Next Steps" for some todo

[GitHub] [incubator-mxnet] samskalicky commented on issue #17034: Dynamic subgraph property

2019-12-31 Thread GitBox
samskalicky commented on issue #17034: Dynamic subgraph property URL: https://github.com/apache/incubator-mxnet/pull/17034#issuecomment-569983621 Hey @pengzhao-intel @ZhennanQin @TaoLv @ptrendx @szha this PR is complete and ready for review. Would really appreciate your feedback. Thanks

[GitHub] [incubator-mxnet] samskalicky commented on issue #17034: Dynamic subgraph property

2019-12-30 Thread GitBox
samskalicky commented on issue #17034: Dynamic subgraph property URL: https://github.com/apache/incubator-mxnet/pull/17034#issuecomment-569743014 @mxnet-label-bot add [pr-awaiting-review] This is an automated message from the