address CR

yangchen-MS · yangchen-MS · commit bda8fcbca024 · 2019-07-22T17:24:42.000-07:00
diff --git a/Source/CNTKv2LibraryDll/proto/onnx/CNTKToONNX.cpp b/Source/CNTKv2LibraryDll/proto/onnx/CNTKToONNX.cpp
@@ -5291,11 +5291,11 @@ bool CNTKToONNXHelper::ProcessLoopsAndCheckCNTKNodeContinueCreate(const Function
                         {
                             if (futureValueCustomAttrStr == "")
                             {
-                                futureValueCustomAttrStr = "{custom_attributes:" + SerializeDictionaryToString(dict) + "}";
+                                futureValueCustomAttrStr = "{\"custom_attributes\":" + SerializeDictionaryToString(dict) + "}";
                             }
                             else
                             {
-                                std::string attrStr = "{custom_attributes:" + SerializeDictionaryToString(dict) + "}";
+                                std::string attrStr = "{\"custom_attributes\":" + SerializeDictionaryToString(dict) + "}";
                                 if (attrStr != futureValueCustomAttrStr)
                                 {
                                     CNTK::LogicError("Scan node has multiple FutureValue custom attributes from state: %s",

Original file line number	Diff line number	Diff line change
`@@ -5291,11 +5291,11 @@ bool CNTKToONNXHelper::ProcessLoopsAndCheckCNTKNodeContinueCreate(const Function`
`5291`	`5291`	`{`
`5292`	`5292`	`if (futureValueCustomAttrStr == "")`
`5293`	`5293`	`{`
`5294`		`- futureValueCustomAttrStr = "{custom_attributes:" + SerializeDictionaryToString(dict) + "}";`
	`5294`	`+ futureValueCustomAttrStr = "{\"custom_attributes\":" + SerializeDictionaryToString(dict) + "}";`
`5295`	`5295`	`}`
`5296`	`5296`	`else`
`5297`	`5297`	`{`
`5298`		`- std::string attrStr = "{custom_attributes:" + SerializeDictionaryToString(dict) + "}";`
	`5298`	`+ std::string attrStr = "{\"custom_attributes\":" + SerializeDictionaryToString(dict) + "}";`
`5299`	`5299`	`if (attrStr != futureValueCustomAttrStr)`
`5300`	`5300`	`{`
`5301`	`5301`	`CNTK::LogicError("Scan node has multiple FutureValue custom attributes from state: %s",`