{"action":"created","issue":{"url":"https://api.github.com/repos/official-stockfish/Stockfish/issues/759","repository_url":"https://api.github.com/repos/official-stockfish/Stockfish","labels_url":"https://api.github.com/repos/official-stockfish/Stockfish/issues/759/labels{/name}","comments_url":"https://api.github.com/repos/official-stockfish/Stockfish/issues/759/comments","events_url":"https://api.github.com/repos/official-stockfish/Stockfish/issues/759/events","html_url":"https://github.com/official-stockfish/Stockfish/pull/759","id":172260143,"number":759,"title":"Let SF to be ONE_PLY value independent","user":{"login":"mcostalba","id":1099265,"avatar_url":"https://avatars.githubusercontent.com/u/1099265?v=3","gravatar_id":"","url":"https://api.github.com/users/mcostalba","html_url":"https://github.com/mcostalba","followers_url":"https://api.github.com/users/mcostalba/followers","following_url":"https://api.github.com/users/mcostalba/following{/other_user}","gists_url":"https://api.github.com/users/mcostalba/gists{/gist_id}","starred_url":"https://api.github.com/users/mcostalba/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mcostalba/subscriptions","organizations_url":"https://api.github.com/users/mcostalba/orgs","repos_url":"https://api.github.com/users/mcostalba/repos","events_url":"https://api.github.com/users/mcostalba/events{/privacy}","received_events_url":"https://api.github.com/users/mcostalba/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":8,"created_at":"2016-08-20T08:07:21Z","updated_at":"2016-08-22T07:57:36Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/official-stockfish/Stockfish/pulls/759","html_url":"https://github.com/official-stockfish/Stockfish/pull/759","diff_url":"https://github.com/official-stockfish/Stockfish/pull/759.diff","patch_url":"https://github.com/official-stockfish/Stockfish/pull/759.patch"},"body":"This non-functional change patch series is a deep work to allow SF to be independent from the actual value of ONE_PLY (currently set to 1).\r\n\r\nI have verified SF is now independent for ONE_PLY values 1, 2, 4, 8 above that there is still some change to be investigated (I think is some rounding error).\r\n\r\nI will run a test to verify there is no regression and then I will open a PR because I think this work is important and gives consistency to the code. I think is also the base above which to rewrite the heterogeneous formulas involving depth to allow better interpolation and avoiding of rounding errors.\r\n\r\nDiving in the low level detail of the code I found many issues, especially with relatively new code and I realized SF starts to suffer from continues add-on tweaks more or less bolted on existing code. It is time for some deep consistency improvement to allow future work to be based on solid grounds.\r\n\r\nI will run some regression test as soon as fishtest (now is down) returns online.\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/official-stockfish/Stockfish/issues/comments/241339154","html_url":"https://github.com/official-stockfish/Stockfish/pull/759#issuecomment-241339154","issue_url":"https://api.github.com/repos/official-stockfish/Stockfish/issues/759","id":241339154,"user":{"login":"mcostalba","id":1099265,"avatar_url":"https://avatars.githubusercontent.com/u/1099265?v=3","gravatar_id":"","url":"https://api.github.com/users/mcostalba","html_url":"https://github.com/mcostalba","followers_url":"https://api.github.com/users/mcostalba/followers","following_url":"https://api.github.com/users/mcostalba/following{/other_user}","gists_url":"https://api.github.com/users/mcostalba/gists{/gist_id}","starred_url":"https://api.github.com/users/mcostalba/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mcostalba/subscriptions","organizations_url":"https://api.github.com/users/mcostalba/orgs","repos_url":"https://api.github.com/users/mcostalba/repos","events_url":"https://api.github.com/users/mcostalba/events{/privacy}","received_events_url":"https://api.github.com/users/mcostalba/received_events","type":"User","site_admin":false},"created_at":"2016-08-22T07:57:36Z","updated_at":"2016-08-22T07:57:36Z","body":"@Stefano80 No LTC is not needed becuase is a non functional change. So there are no scalability issues.\r\n\r\n@zamar Joona, I would like to commit this one, I think gives consistency to seacrh code and enables future work opening the door to safely tweaking the ONE_PLY value for any reason.\r\n"}}
{"actor":{"display_login":"mcostalba"}}