{"action":"created","issue":{"url":"https://api.github.com/repos/official-stockfish/Stockfish/issues/341","labels_url":"https://api.github.com/repos/official-stockfish/Stockfish/issues/341/labels{/name}","comments_url":"https://api.github.com/repos/official-stockfish/Stockfish/issues/341/comments","events_url":"https://api.github.com/repos/official-stockfish/Stockfish/issues/341/events","html_url":"https://github.com/official-stockfish/Stockfish/pull/341","id":72548762,"number":341,"title":"Prepare PSQT for tuning","user":{"login":"lucasart","id":2278294,"avatar_url":"https://avatars.githubusercontent.com/u/2278294?v=3","gravatar_id":"","url":"https://api.github.com/users/lucasart","html_url":"https://github.com/lucasart","followers_url":"https://api.github.com/users/lucasart/followers","following_url":"https://api.github.com/users/lucasart/following{/other_user}","gists_url":"https://api.github.com/users/lucasart/gists{/gist_id}","starred_url":"https://api.github.com/users/lucasart/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lucasart/subscriptions","organizations_url":"https://api.github.com/users/lucasart/orgs","repos_url":"https://api.github.com/users/lucasart/repos","events_url":"https://api.github.com/users/lucasart/events{/privacy}","received_events_url":"https://api.github.com/users/lucasart/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2015-05-01T23:40:06Z","updated_at":"2015-05-02T00:52:05Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/official-stockfish/Stockfish/pulls/341","html_url":"https://github.com/official-stockfish/Stockfish/pull/341","diff_url":"https://github.com/official-stockfish/Stockfish/pull/341.diff","patch_url":"https://github.com/official-stockfish/Stockfish/pull/341.patch"},"body":"The way the code is designed currently is not very tuning friendly. In order to\r\ntune the PSQT, one first has to rearrange it and write some boiler plate code,\r\nvery much like this patch.\r\n\r\nNo functional change."},"comment":{"url":"https://api.github.com/repos/official-stockfish/Stockfish/issues/comments/98271896","html_url":"https://github.com/official-stockfish/Stockfish/pull/341#issuecomment-98271896","issue_url":"https://api.github.com/repos/official-stockfish/Stockfish/issues/341","id":98271896,"user":{"login":"lucasart","id":2278294,"avatar_url":"https://avatars.githubusercontent.com/u/2278294?v=3","gravatar_id":"","url":"https://api.github.com/users/lucasart","html_url":"https://github.com/lucasart","followers_url":"https://api.github.com/users/lucasart/followers","following_url":"https://api.github.com/users/lucasart/following{/other_user}","gists_url":"https://api.github.com/users/lucasart/gists{/gist_id}","starred_url":"https://api.github.com/users/lucasart/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lucasart/subscriptions","organizations_url":"https://api.github.com/users/lucasart/orgs","repos_url":"https://api.github.com/users/lucasart/repos","events_url":"https://api.github.com/users/lucasart/events{/privacy}","received_events_url":"https://api.github.com/users/lucasart/received_events","type":"User","site_admin":false},"created_at":"2015-05-02T00:52:05Z","updated_at":"2015-05-02T00:52:05Z","body":"Can you explain what is the logic of:\r\n* putting the PSQT namespace in position.*, when the PSQT array is in psqtab.h ?\r\n* putting the psq array in the anonymous namespace of position.cpp, instead of in PSQT namespace where it belongs ?\r\n\r\nAlso, can you also introduce the file symmetry in your patch?"}}