Skip to content
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

Create vlog_guide.md #5121

Closed

Conversation

tonyyang-svail
Copy link

This is a follow up for switching logging to VLOG. I feel we need a VLOG level guide for the PaddlePaddle developers. Here is what I have in mind.

@helinwang
Copy link
Contributor

Maybe add a small intro for what is VLOG, such as (taken from the link you provided):

Verbose Logging

When you are chasing difficult bugs, thorough log messages are very useful. However, you may want to ignore too verbose messages in usual development. For such verbose logging, glog provides the VLOG macro, which allows you to define your own numeric logging levels.

@helinwang
Copy link
Contributor

I think it's important to use the vmodule flag:

vmodule (string, default="")

Per-module verbose level.

--vmodule=mapreduce=2,file=1,gfs*=3 --v=0

@@ -0,0 +1,13 @@
This is a follow up for switching logging to VLOG. I feel we need a VLOG level guide for the PaddlePaddle developers. Here is what I have in mind
Copy link
Collaborator

@wangkuiyi wangkuiyi Oct 27, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We are going to follow the convention adopted by Chromium to use VLOG instead of LOG(INFO) to generate debug information. Please be aware that usually log messages are still via LOG.

For more information about VLOG, please refer to the official document of glog.

VLOG requires a verbose level parameter, like that LOG requires a severity level. This document explains the verbose level value for various kinds of VLOG messages.

@tonyyang-svail
Copy link
Author

VLOG guide has been added to CONTRIBUTING.md.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants