{"action":"created","issue":{"url":"https://api.github.com/repos/dropbox/pyston/issues/574","labels_url":"https://api.github.com/repos/dropbox/pyston/issues/574/labels{/name}","comments_url":"https://api.github.com/repos/dropbox/pyston/issues/574/comments","events_url":"https://api.github.com/repos/dropbox/pyston/issues/574/events","html_url":"https://github.com/dropbox/pyston/pull/574","id":83791046,"number":574,"title":"Add a section-ordering script","user":{"login":"kmod","id":271916,"avatar_url":"https://avatars.githubusercontent.com/u/271916?v=3","gravatar_id":"","url":"https://api.github.com/users/kmod","html_url":"https://github.com/kmod","followers_url":"https://api.github.com/users/kmod/followers","following_url":"https://api.github.com/users/kmod/following{/other_user}","gists_url":"https://api.github.com/users/kmod/gists{/gist_id}","starred_url":"https://api.github.com/users/kmod/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kmod/subscriptions","organizations_url":"https://api.github.com/users/kmod/orgs","repos_url":"https://api.github.com/users/kmod/repos","events_url":"https://api.github.com/users/kmod/events{/privacy}","received_events_url":"https://api.github.com/users/kmod/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-06-02T02:10:26Z","updated_at":"2015-06-02T09:49:45Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/dropbox/pyston/pulls/574","html_url":"https://github.com/dropbox/pyston/pull/574","diff_url":"https://github.com/dropbox/pyston/pull/574.diff","patch_url":"https://github.com/dropbox/pyston/pull/574.patch"},"body":"I think this lets us specify that certain functions should be\r\nput together at the end of the text segment.\r\n\r\nI picked some random functions that were showing up in the perf report\r\nand listed those.  I think there's a lot we can do here for improving\r\nperformance, but my main goal for now is to improve performance consistency.\r\n\r\n```\r\n              pyston django_migrate.py                  :    2.1s tp_hash: 2.2 (-1.1%)\r\n              pyston virtualenv_bench.py                :    5.7s tp_hash: 5.8 (-1.1%)\r\n              pyston django-template.py                 :   10.9s tp_hash: 11.4 (-3.8%)\r\n              pyston interp2.py                         :    5.4s tp_hash: 5.5 (-1.7%)\r\n              pyston raytrace.py                        :    6.5s tp_hash: 6.4 (+1.0%)\r\n              pyston nbody.py                           :    8.4s tp_hash: 8.4 (+0.9%)\r\n              pyston fannkuch.py                        :    7.3s tp_hash: 7.5 (-3.0%)\r\n              pyston chaos.py                           :   20.5s tp_hash: 19.9 (+2.9%)\r\n              pyston fasta.py                           :    5.4s tp_hash: 5.5 (-1.4%)\r\n              pyston pidigits.py                        :    5.7s tp_hash: 5.7 (-0.9%)\r\n              pyston richards.py                        :    2.4s tp_hash: 2.5 (-1.4%)\r\n              pyston deltablue.py                       :    1.8s tp_hash: 1.8 (-1.5%)\r\n              pyston (geomean-1a6f)                     :    5.5s tp_hash: 5.6 (-1.7%)\r\n```"},"comment":{"url":"https://api.github.com/repos/dropbox/pyston/issues/comments/107899535","html_url":"https://github.com/dropbox/pyston/pull/574#issuecomment-107899535","issue_url":"https://api.github.com/repos/dropbox/pyston/issues/574","id":107899535,"user":{"login":"kmod","id":271916,"avatar_url":"https://avatars.githubusercontent.com/u/271916?v=3","gravatar_id":"","url":"https://api.github.com/users/kmod","html_url":"https://github.com/kmod","followers_url":"https://api.github.com/users/kmod/followers","following_url":"https://api.github.com/users/kmod/following{/other_user}","gists_url":"https://api.github.com/users/kmod/gists{/gist_id}","starred_url":"https://api.github.com/users/kmod/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kmod/subscriptions","organizations_url":"https://api.github.com/users/kmod/orgs","repos_url":"https://api.github.com/users/kmod/repos","events_url":"https://api.github.com/users/kmod/events{/privacy}","received_events_url":"https://api.github.com/users/kmod/received_events","type":"User","site_admin":false},"created_at":"2015-06-02T09:49:45Z","updated_at":"2015-06-02T09:49:45Z","body":"Ok I tried to be more careful with the building and measurement, and this is what I got:\r\n```\r\n              pyston django_migrate.py                  :    2.1s baseline: 2.2 (-2.3%)\r\n              pyston virtualenv_bench.py                :    5.7s baseline: 5.7 (-1.3%)\r\n              pyston django-template.py                 :   10.8s baseline: 11.1 (-2.4%)\r\n              pyston interp2.py                         :    5.4s baseline: 5.4 (-0.7%)\r\n              pyston raytrace.py                        :    6.4s baseline: 6.2 (+3.1%)\r\n              pyston nbody.py                           :    8.4s baseline: 8.4 (+0.4%)\r\n              pyston fannkuch.py                        :    7.2s baseline: 7.7 (-6.0%)\r\n              pyston chaos.py                           :   20.2s baseline: 20.1 (+0.5%)\r\n              pyston fasta.py                           :    5.4s baseline: 5.5 (-1.5%)\r\n              pyston pidigits.py                        :    5.6s baseline: 5.7 (-1.3%)\r\n              pyston richards.py                        :    2.5s baseline: 2.5 (-1.0%)\r\n              pyston deltablue.py                       :    1.8s baseline: 1.8 (-1.2%)\r\n              pyston (geomean-1a6f)                     :    5.5s baseline: 5.6 (-1.2%)\r\n```\r\nI messed with the ordering some more and got some different results.\r\n\r\nI guess this gives a decent sense of how much text layout (which is all this change affects) can affect overall perf.  It also says to me that our previous crop of benchmarks is not very good and motivates me to replace it -- our newer benchmarks show much more consistent and understandable changes.\r\n\r\nBut anyway, I'm going to push this for now, and see if it can help with the next set of perf investigations."}}