{"action":"created","issue":{"url":"https://api.github.com/repos/official-stockfish/Stockfish/issues/340","labels_url":"https://api.github.com/repos/official-stockfish/Stockfish/issues/340/labels{/name}","comments_url":"https://api.github.com/repos/official-stockfish/Stockfish/issues/340/comments","events_url":"https://api.github.com/repos/official-stockfish/Stockfish/issues/340/events","html_url":"https://github.com/official-stockfish/Stockfish/pull/340","id":71106952,"number":340,"title":"Replace MVV/LVA by MVV","user":{"login":"Stefano80","id":8758991,"avatar_url":"https://avatars.githubusercontent.com/u/8758991?v=3","gravatar_id":"","url":"https://api.github.com/users/Stefano80","html_url":"https://github.com/Stefano80","followers_url":"https://api.github.com/users/Stefano80/followers","following_url":"https://api.github.com/users/Stefano80/following{/other_user}","gists_url":"https://api.github.com/users/Stefano80/gists{/gist_id}","starred_url":"https://api.github.com/users/Stefano80/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Stefano80/subscriptions","organizations_url":"https://api.github.com/users/Stefano80/orgs","repos_url":"https://api.github.com/users/Stefano80/repos","events_url":"https://api.github.com/users/Stefano80/events{/privacy}","received_events_url":"https://api.github.com/users/Stefano80/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-04-26T19:10:38Z","updated_at":"2015-04-26T21:52:53Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/official-stockfish/Stockfish/pulls/340","html_url":"https://github.com/official-stockfish/Stockfish/pull/340","diff_url":"https://github.com/official-stockfish/Stockfish/pull/340.diff","patch_url":"https://github.com/official-stockfish/Stockfish/pull/340.patch"},"body":"Passed STC\r\n\r\nLLR: 3.71 (-2.94,2.94) [-3.00,1.00]\r\nTotal: 64363 W: 12299 L: 12214 D: 39850\r\n\r\nand LTC\r\n\r\nLLR: 2.95 (-2.94,2.94) [-3.00,1.00]\r\nTotal: 69976 W: 11056 L: 11011 D: 47909\r\n\r\nReasons to commit the patch:\r\n\r\n1) In the MVV/LVA paradigm the LVA part does add very little ELO\r\n2) Removing LVA reduces both code line count and code complexity\r\n\r\nRegarding 1)\r\n- Simplification test passed with positive score, both in at [STC] (http://tests.stockfishchess.org/tests/view/55320f6d0ebc596367ee9a85) and [LTC] (http://tests.stockfishchess.org/tests/view/5535e1740ebc596367ee9b55). Comparing with results with different sprt simulators it should be something between -1 and -0.5 ELO.\r\n- [Tabled] (http://tests.stockfishchess.org/tests/view/552d33d80ebc596367ee990b) and [tuned] (http://tests.stockfishchess.org/tests/view/5530a47b0ebc596367ee9a03) MVV/LVA failed STC with identical run length, tuned MVV/LVA being practically identical to MVV/LVA + some random walk of parameters\r\n- LVA alone failed STC [miserably] (http://tests.stockfishchess.org/tests/view/5535582c0ebc596367ee9b32)\r\n- Nevertheless, there must be some part of LVA which is useful (or which correlates with something useful), since [MVV/MVA] (http://tests.stockfishchess.org/tests/view/553113590ebc596367ee9a42) failed more clearly than tabled or tuned MVV/LVA. "},"comment":{"url":"https://api.github.com/repos/official-stockfish/Stockfish/issues/comments/96438846","html_url":"https://github.com/official-stockfish/Stockfish/pull/340#issuecomment-96438846","issue_url":"https://api.github.com/repos/official-stockfish/Stockfish/issues/340","id":96438846,"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":"2015-04-26T21:52:53Z","updated_at":"2015-04-26T21:52:53Z","body":"@Stefano80 please update also the comments of the scoring function."}}