GitHub PullRequestReviewCommentEvent at 2014-05-29 09:51:14
Repository: matplotlib
Actor: pelson
Payload: {"comment":{"url":"https://api.github.com/repos/matplotlib/matplotlib/pulls/comments/13174830","id":13174830,"diff_hunk":"@@ -0,0 +1,75 @@\n+'''\n+This example shows how to use MaxNPiLocator and PiFormatter in order to have \n+automatic pretty looking axes tick labels for PI numbres. This demo contains two subplots\n+that have PI axes with different options.\n+'''\n+\n+import numpy as np\n+import matplotlib as mpl\n+import matplotlib.pyplot as plt\n+from matplotlib.ticker import *\n+\n+# rendering area\n+plt.figure(figsize=(8,10), dpi=100)\n+\n+###########\n+#example 1#\n+###########\n+ax1 = plt.subplot(211)\n+\n+# range","path":"examples/ticks_and_spines/pi_labels.py","position":20,"original_position":20,"commit_id":"17dfc7d9ff57ff492fa2fdbefb1fbf1c7beb1d2d","original_commit_id":"17dfc7d9ff57ff492fa2fdbefb1fbf1c7beb1d2d","user":{"login":"pelson","id":810663,"avatar_url":"https://avatars.githubusercontent.com/u/810663?","gravatar_id":"a3a8307ffcd780e908d042bcc97cd2f8","url":"https://api.github.com/users/pelson","html_url":"https://github.com/pelson","followers_url":"https://api.github.com/users/pelson/followers","following_url":"https://api.github.com/users/pelson/following{/other_user}","gists_url":"https://api.github.com/users/pelson/gists{/gist_id}","starred_url":"https://api.github.com/users/pelson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pelson/subscriptions","organizations_url":"https://api.github.com/users/pelson/orgs","repos_url":"https://api.github.com/users/pelson/repos","events_url":"https://api.github.com/users/pelson/events{/privacy}","received_events_url":"https://api.github.com/users/pelson/received_events","type":"User","site_admin":false},"body":"Comments are good, but some of these are not that helpful - I'd suggest either removing them, or being a little more descriptive with the comments. ([Ideally](http://legacy.python.org/dev/peps/pep-0008/#comments), they should be a proper sentence with a capital letter and a full stop, but I'm not too worried about that here).","created_at":"2014-05-29T09:51:14Z","updated_at":"2014-05-29T09:51:14Z","html_url":"https://github.com/matplotlib/matplotlib/pull/3071#discussion_r13174830","pull_request_url":"https://api.github.com/repos/matplotlib/matplotlib/pulls/3071","_links":{"self":{"href":"https://api.github.com/repos/matplotlib/matplotlib/pulls/comments/13174830"},"html":{"href":"https://github.com/matplotlib/matplotlib/pull/3071#discussion_r13174830"},"pull_request":{"href":"https://api.github.com/repos/matplotlib/matplotlib/pulls/3071"}}}}