{"action":"closed","issue":{"url":"https://api.github.com/repos/ESMCI/cime/issues/1506","repository_url":"https://api.github.com/repos/ESMCI/cime","labels_url":"https://api.github.com/repos/ESMCI/cime/issues/1506/labels{/name}","comments_url":"https://api.github.com/repos/ESMCI/cime/issues/1506/comments","events_url":"https://api.github.com/repos/ESMCI/cime/issues/1506/events","html_url":"https://github.com/ESMCI/cime/issues/1506","id":227690140,"number":1506,"title":"Logger Strings aren't Suppose to be Formatted in Place","user":{"login":"erichlf","id":1392578,"avatar_url":"https://avatars3.githubusercontent.com/u/1392578?v=3","gravatar_id":"","url":"https://api.github.com/users/erichlf","html_url":"https://github.com/erichlf","followers_url":"https://api.github.com/users/erichlf/followers","following_url":"https://api.github.com/users/erichlf/following{/other_user}","gists_url":"https://api.github.com/users/erichlf/gists{/gist_id}","starred_url":"https://api.github.com/users/erichlf/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/erichlf/subscriptions","organizations_url":"https://api.github.com/users/erichlf/orgs","repos_url":"https://api.github.com/users/erichlf/repos","events_url":"https://api.github.com/users/erichlf/events{/privacy}","received_events_url":"https://api.github.com/users/erichlf/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":{"login":"erichlf","id":1392578,"avatar_url":"https://avatars3.githubusercontent.com/u/1392578?v=3","gravatar_id":"","url":"https://api.github.com/users/erichlf","html_url":"https://github.com/erichlf","followers_url":"https://api.github.com/users/erichlf/followers","following_url":"https://api.github.com/users/erichlf/following{/other_user}","gists_url":"https://api.github.com/users/erichlf/gists{/gist_id}","starred_url":"https://api.github.com/users/erichlf/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/erichlf/subscriptions","organizations_url":"https://api.github.com/users/erichlf/orgs","repos_url":"https://api.github.com/users/erichlf/repos","events_url":"https://api.github.com/users/erichlf/events{/privacy}","received_events_url":"https://api.github.com/users/erichlf/received_events","type":"User","site_admin":false},"assignees":[{"login":"erichlf","id":1392578,"avatar_url":"https://avatars3.githubusercontent.com/u/1392578?v=3","gravatar_id":"","url":"https://api.github.com/users/erichlf","html_url":"https://github.com/erichlf","followers_url":"https://api.github.com/users/erichlf/followers","following_url":"https://api.github.com/users/erichlf/following{/other_user}","gists_url":"https://api.github.com/users/erichlf/gists{/gist_id}","starred_url":"https://api.github.com/users/erichlf/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/erichlf/subscriptions","organizations_url":"https://api.github.com/users/erichlf/orgs","repos_url":"https://api.github.com/users/erichlf/repos","events_url":"https://api.github.com/users/erichlf/events{/privacy}","received_events_url":"https://api.github.com/users/erichlf/received_events","type":"User","site_admin":false}],"milestone":null,"comments":2,"created_at":"2017-05-10T14:00:30Z","updated_at":"2017-05-10T16:58:26Z","closed_at":"2017-05-10T16:58:25Z","body":"When using logging in python, string formatting is suppose to be done by the logger, e.g.\r\n```\r\nlogger.info(\"doing an %s %s initial test with restart file at %s %s with pio type %s\"\r\n                   .format(str(self._stop_n), stop_option, str(rest_n), stop_option, pio_type))\r\n```\r\nshould be \r\n```\r\nlogger.info(\"doing an %s %s initial test with restart file at %s %s with pio type %s\",\r\n                   str(self._stop_n), stop_option, str(rest_n), stop_option, pio_type)\r\n```\r\nhttps://docs.python.org/2/library/logging.html#logging.debug\r\nhttp://stackoverflow.com/questions/34619790/pylint-message-logging-format-interpolation\r\nhttp://stackoverflow.com/questions/20780608/how-to-log-two-variables-values-in-a-single-logging-in-python"}}
{"actor":{"display_login":"erichlf"}}