{"action":"created","issue":{"url":"https://api.github.com/repos/official-stockfish/Stockfish/issues/428","labels_url":"https://api.github.com/repos/official-stockfish/Stockfish/issues/428/labels{/name}","comments_url":"https://api.github.com/repos/official-stockfish/Stockfish/issues/428/comments","events_url":"https://api.github.com/repos/official-stockfish/Stockfish/issues/428/events","html_url":"https://github.com/official-stockfish/Stockfish/pull/428","id":106489014,"number":428,"title":"MinimumSplitDepth=2 Max Slaves 7(fixed for crashes)","user":{"login":"ajithcj","id":5998883,"avatar_url":"https://avatars.githubusercontent.com/u/5998883?v=3","gravatar_id":"","url":"https://api.github.com/users/ajithcj","html_url":"https://github.com/ajithcj","followers_url":"https://api.github.com/users/ajithcj/followers","following_url":"https://api.github.com/users/ajithcj/following{/other_user}","gists_url":"https://api.github.com/users/ajithcj/gists{/gist_id}","starred_url":"https://api.github.com/users/ajithcj/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ajithcj/subscriptions","organizations_url":"https://api.github.com/users/ajithcj/orgs","repos_url":"https://api.github.com/users/ajithcj/repos","events_url":"https://api.github.com/users/ajithcj/events{/privacy}","received_events_url":"https://api.github.com/users/ajithcj/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":14,"created_at":"2015-09-15T06:24:52Z","updated_at":"2015-09-19T22:50:41Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/official-stockfish/Stockfish/pulls/428","html_url":"https://github.com/official-stockfish/Stockfish/pull/428","diff_url":"https://github.com/official-stockfish/Stockfish/pull/428.diff","patch_url":"https://github.com/official-stockfish/Stockfish/pull/428.patch"},"body":"Reduce minimumSplitDepth to 2. But use fewer slaves while splitting at lower depths. Test passed\r\n\r\nSTC 3 Threads:\r\nLLR: 2.97 (-2.94,2.94) [0.00,5.00]\r\nTotal: 11351 W: 2008 L: 1838 D: 7505\r\n\r\nSTC 7 Threads:\r\nLLR: 2.95 (-2.94,2.94) [0.00,5.00]\r\nTotal: 2587 W: 485 L: 353 D: 1749\r\n\r\nLTC 7 Threads:\r\nLLR: 2.95 (-2.94,2.94) [0.00,5.00]\r\nTotal: 20538 W: 2769 L: 2584 D: 15185\r\n\r\nCrashes were reported on the original branch. \r\nTwo additional lock acquire/release pairs were added to fix the occasional crash.\r\nAfter this a two-day crash test passed on a 20-core machine.\r\n\r\nWith fixed code, test again passed STC 7 Threads:\r\nLLR: 2.96 (-2.94,2.94) [0.00,5.00]\r\nTotal: 50470 W: 8033 L: 7720 D: 34717\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/official-stockfish/Stockfish/issues/comments/141712330","html_url":"https://github.com/official-stockfish/Stockfish/pull/428#issuecomment-141712330","issue_url":"https://api.github.com/repos/official-stockfish/Stockfish/issues/428","id":141712330,"user":{"login":"AlexandreMasta","id":5086048,"avatar_url":"https://avatars.githubusercontent.com/u/5086048?v=3","gravatar_id":"","url":"https://api.github.com/users/AlexandreMasta","html_url":"https://github.com/AlexandreMasta","followers_url":"https://api.github.com/users/AlexandreMasta/followers","following_url":"https://api.github.com/users/AlexandreMasta/following{/other_user}","gists_url":"https://api.github.com/users/AlexandreMasta/gists{/gist_id}","starred_url":"https://api.github.com/users/AlexandreMasta/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/AlexandreMasta/subscriptions","organizations_url":"https://api.github.com/users/AlexandreMasta/orgs","repos_url":"https://api.github.com/users/AlexandreMasta/repos","events_url":"https://api.github.com/users/AlexandreMasta/events{/privacy}","received_events_url":"https://api.github.com/users/AlexandreMasta/received_events","type":"User","site_admin":false},"created_at":"2015-09-19T22:50:41Z","updated_at":"2015-09-19T22:50:41Z","body":"2 was also a speculative cap due to the assumption that above 7 SF was entering unknown waters.\r\n1 cannot be committed without 3 because the fix gutted the original patch. "}}