add exe graph physical shape check msg

oneflow/core/common/error_util.cpp

@@ -16,6 +16,7 @@ limitations under the License.
 #include <sstream>
 #include "oneflow/core/common/error_util.h"
 #include "oneflow/core/common/util.h"
+#include "oneflow/core/job/graph_scope_vars.h"
 
 namespace oneflow {
 
@@ -97,7 +98,9 @@ std::string FormatFunctionOfStackFrame(const std::string& function) {
 
 // msg in stack frame
 Maybe<std::string> FormatMsgOfStackFrame(std::string error_msg, bool is_last_stack_frame) {
-  if (!is_last_stack_frame) { error_msg = *JUST(ShortenMsg(error_msg)); }
+  const bool debug_mode = GetGraphDebugMode();
+  // only shorten the message if it is not the last stack frame AND not in debug mode
+  if (!is_last_stack_frame && !debug_mode) { error_msg = *JUST(ShortenMsg(error_msg)); }
   // error_msg of last stack frame come from "<<"
   if (is_last_stack_frame) { error_msg = StripSpace(error_msg); }
   std::stringstream ss;

oneflow/core/graph/exec_graph.cpp

@@ -14,6 +14,8 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 #include "oneflow/core/graph/exec_graph.h"
+#include <sstream>
+#include "oneflow/core/common/just.h"
 #include "oneflow/core/graph/op_graph.h"
 
 namespace oneflow {
@@ -92,9 +94,9 @@ Maybe<void> CheckPhysicalBlobDesc(
       continue;
     }
     if (*JUST(op.GetParallelDesc4BnInOp(bn)) == *op_parallel_desc) {
-      JUST(CheckPhysicalBlobDesc(*JUST(GetLogicalBlobDesc(bn)),
+      JUST_MSG(CheckPhysicalBlobDesc(*JUST(GetLogicalBlobDesc(bn)),
                                  nd_sbp_signature->bn_in_op2nd_sbp().at(bn), *op_parallel_desc,
-                                 parallel_ctx, *physical_blob_desc));
+                                 parallel_ctx, *physical_blob_desc), std::stringstream() << " check physical shape failed, op name " << op.op_loc());
     }
   }
   return Maybe<void>::Ok();