{"action":"closed","issue":{"url":"https://api.github.com/repos/astropy/halotools/issues/564","repository_url":"https://api.github.com/repos/astropy/halotools","labels_url":"https://api.github.com/repos/astropy/halotools/issues/564/labels{/name}","comments_url":"https://api.github.com/repos/astropy/halotools/issues/564/comments","events_url":"https://api.github.com/repos/astropy/halotools/issues/564/events","html_url":"https://github.com/astropy/halotools/issues/564","id":162349533,"number":564,"title":"Add documentation note for optimization flags","user":{"login":"aphearin","id":6951595,"avatar_url":"https://avatars.githubusercontent.com/u/6951595?v=3","gravatar_id":"","url":"https://api.github.com/users/aphearin","html_url":"https://github.com/aphearin","followers_url":"https://api.github.com/users/aphearin/followers","following_url":"https://api.github.com/users/aphearin/following{/other_user}","gists_url":"https://api.github.com/users/aphearin/gists{/gist_id}","starred_url":"https://api.github.com/users/aphearin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aphearin/subscriptions","organizations_url":"https://api.github.com/users/aphearin/orgs","repos_url":"https://api.github.com/users/aphearin/repos","events_url":"https://api.github.com/users/aphearin/events{/privacy}","received_events_url":"https://api.github.com/users/aphearin/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/astropy/halotools/labels/mock-observables","name":"mock-observables","color":"c7def8"},{"url":"https://api.github.com/repos/astropy/halotools/labels/performance-enhancement","name":"performance-enhancement","color":"84b6eb"}],"state":"closed","locked":false,"assignee":{"login":"aphearin","id":6951595,"avatar_url":"https://avatars.githubusercontent.com/u/6951595?v=3","gravatar_id":"","url":"https://api.github.com/users/aphearin","html_url":"https://github.com/aphearin","followers_url":"https://api.github.com/users/aphearin/followers","following_url":"https://api.github.com/users/aphearin/following{/other_user}","gists_url":"https://api.github.com/users/aphearin/gists{/gist_id}","starred_url":"https://api.github.com/users/aphearin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aphearin/subscriptions","organizations_url":"https://api.github.com/users/aphearin/orgs","repos_url":"https://api.github.com/users/aphearin/repos","events_url":"https://api.github.com/users/aphearin/events{/privacy}","received_events_url":"https://api.github.com/users/aphearin/received_events","type":"User","site_admin":false},"assignees":[{"login":"aphearin","id":6951595,"avatar_url":"https://avatars.githubusercontent.com/u/6951595?v=3","gravatar_id":"","url":"https://api.github.com/users/aphearin","html_url":"https://github.com/aphearin","followers_url":"https://api.github.com/users/aphearin/followers","following_url":"https://api.github.com/users/aphearin/following{/other_user}","gists_url":"https://api.github.com/users/aphearin/gists{/gist_id}","starred_url":"https://api.github.com/users/aphearin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aphearin/subscriptions","organizations_url":"https://api.github.com/users/aphearin/orgs","repos_url":"https://api.github.com/users/aphearin/repos","events_url":"https://api.github.com/users/aphearin/events{/privacy}","received_events_url":"https://api.github.com/users/aphearin/received_events","type":"User","site_admin":false}],"milestone":{"url":"https://api.github.com/repos/astropy/halotools/milestones/6","html_url":"https://github.com/astropy/halotools/milestones/v0.3","labels_url":"https://api.github.com/repos/astropy/halotools/milestones/6/labels","id":1771806,"number":6,"title":"v0.3","description":"This feature will be part of Halotools v0.3, the first stable release of the code, and the version associated with the code release paper, http://arxiv.org/abs/1606.04106.","creator":{"login":"aphearin","id":6951595,"avatar_url":"https://avatars.githubusercontent.com/u/6951595?v=3","gravatar_id":"","url":"https://api.github.com/users/aphearin","html_url":"https://github.com/aphearin","followers_url":"https://api.github.com/users/aphearin/followers","following_url":"https://api.github.com/users/aphearin/following{/other_user}","gists_url":"https://api.github.com/users/aphearin/gists{/gist_id}","starred_url":"https://api.github.com/users/aphearin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aphearin/subscriptions","organizations_url":"https://api.github.com/users/aphearin/orgs","repos_url":"https://api.github.com/users/aphearin/repos","events_url":"https://api.github.com/users/aphearin/events{/privacy}","received_events_url":"https://api.github.com/users/aphearin/received_events","type":"User","site_admin":false},"open_issues":2,"closed_issues":6,"state":"open","created_at":"2016-05-17T14:55:09Z","updated_at":"2016-06-26T23:16:47Z","due_on":null,"closed_at":null},"comments":0,"created_at":"2016-06-26T21:48:11Z","updated_at":"2016-06-26T23:16:47Z","closed_at":"2016-06-26T23:16:47Z","body":"Forcing certain compiler optimization flags can lead to Halotools being less robust to install, especially on older machines, as pointed on #561. In the short term, rather than determining the cython compiler flags in a fancy automated fashion, the simple way forward is just to add a note to installation section of the documentation informing performance-hungry users to build their own version from source, throwing their own flags as necessary. This is probably a pretty small thing to ask of a typical user who is interested enough in optimizing performance a the ~25% level. And anyway, there are at least factors of 2-3x speedups that can be gained simply from grinding down the source code implementation of the pair-counting algorithm, so labor this automation should play second fiddle to those changes. *After* the next iteration on the pair counting engines (which will probably not be utill v0.5), revisit the issue of automated compiler flag optimization. "}}
{"actor":{"display_login":"aphearin"}}