-
Notifications
You must be signed in to change notification settings - Fork 276
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
RFC- 赛题八:paddleviz能力扩展——Tensor信息展示 #600
Conversation
std::stringstream ss; | ||
ss << this; | ||
std::string this_pointer = ss.str(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这段逻辑应该是不需要
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
好的 这边修改下
{} | ||
// Create Grad Node | ||
{} | ||
VLOG(11) << \"Finish AD API GRAD: {}, gradnode_ptr: \"+ this_pointer; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这里改成
VLOG(11) << "Finish AD API GRAD: {}, gradnode_ptr: " << this;
就行了
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这里原来是不是VLOG(4) << "Finish AD API GRAD: {}"; 啊?原来的代码不要动。
你新加一行:
VLOG(11) << "gradnode_ptr = " << this;
吧
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这里原来是不是VLOG(4) << "Finish AD API GRAD: {}"; 啊?原来的代码不要动。 你新加一行: VLOG(11) << "gradnode_ptr = " << this; 吧
确定是修改了原来代码, 那再增加一行,尽量不修改之前代码
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
No description provided.