Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: official-stockfish/Stockfish
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: 54f8a9cb13
Choose a base ref
...
head repository: official-stockfish/Stockfish
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: adeded29fb
Choose a head ref
  • 8 commits
  • 12 files changed
  • 4 contributors

Commits on May 31, 2014

  1. Restore development version

    bench: 8732553
    mcostalba committed May 31, 2014
    Configuration menu
    Copy the full SHA
    f5622cd View commit details
    Browse the repository at this point in the history
  2. Retire polyglot.ini

    All actively maintained GUI are able
    to understand UCI protocol, so there
    is no more need for this obsolete file.
    
    No functional change.
    mcostalba committed May 31, 2014
    Configuration menu
    Copy the full SHA
    07a525c View commit details
    Browse the repository at this point in the history

Commits on Jun 1, 2014

  1. Retire eval weights UCI options

    There is really little that user can achieve (apart
    from a weakened engine) tweaking these parameters
    that are already tuned and have no immediate or visible
    effect.
    
    So better do not expose them to the user and avoid the
    typical "What is the best setup for my machine?" kind of
    question (by far the most common, by far the most useless).
    
    No functional change.
    mcostalba committed Jun 1, 2014
    Configuration menu
    Copy the full SHA
    e4fc9d8 View commit details
    Browse the repository at this point in the history
  2. Retire Polyglot Book management

    Book handling belongs to GUI, we kept this code
    for historical reasons, but nowdays there is
    really no need of this old, (mostly) unused
    and especially incorrect designed functionality.
    
    It is up to the GUI to choose the book (far easier for
    the user) and to select the book parameters. In no
    place, including fishtest, TCEC, rating lists, etc.
    the "own book" is used, moreover currently SF is
    released without any book and even if in the future we
    bundle a book in the release package, it will be the GUI
    that will take care of it.
    
    This corrects a wrong design decision that Galurung
    and later Stockfish inherited from what was common
    practice many yeas ago.
    
    No functional change.
    mcostalba committed Jun 1, 2014
    Configuration menu
    Copy the full SHA
    f4dcec0 View commit details
    Browse the repository at this point in the history

Commits on Jun 3, 2014

  1. Decrease reduction for moves that escape a capture

    Passed both STC
    LLR: 2.96 (-2.94,2.94) [-1.50,4.50]
    Total: 10094 W: 1833 L: 1704 D: 6557
    
    and LTC
    LLR: 2.95 (-2.94,2.94) [0.00,6.00]
    Total: 27738 W: 4147 L: 3928 D: 19663
    
    bench : 8599236
    R-Peleg authored and mcostalba committed Jun 3, 2014
    Configuration menu
    Copy the full SHA
    83a574f View commit details
    Browse the repository at this point in the history
  2. Add some const qualifier

    No functional change.
    scchess authored and mcostalba committed Jun 3, 2014
    Configuration menu
    Copy the full SHA
    6f48367 View commit details
    Browse the repository at this point in the history
  3. Use see() instead of see_sign() in previous patch

    Before it was working by accident in case of
    see_sign() and failing with see() due to how
    castle moves are coded (king captures the rook).
    
    Better to explicitly filter out castling moves
    and use see() without any surprise/trick.
    
    No functional case.
    mcostalba committed Jun 3, 2014
    Configuration menu
    Copy the full SHA
    187a9fe View commit details
    Browse the repository at this point in the history
  4. Symmetric King Safety: take 2

    Another attempt at retiring current asymmetric
    king evaluation and use a much simpler symmetric
    one. As a good side effect we can avoid recalculating
    eval after a null move.
    
    Tested in no-regression mode and passed
    
    STC
    LLR: 2.96 (-2.94,2.94) [-3.00,1.00]
    Total: 21580 W: 3752 L: 3632 D: 14196
    
    LTC
    LLR: 2.96 (-2.94,2.94) [-3.00,1.00]
    Total: 18253 W: 2593 L: 2469 D: 13191
    
    And a LTC regression test against SF DD to
    verify we don't have regression against
    weaker engines due to some kind of 'contempt'
    effect:
    
    ELO: 54.69 +-2.1 (95%) LOS: 100.0%
    Total: 40000 W: 11072 L: 4827 D: 24101
    
    bench: 8205159
    lucasart authored and mcostalba committed Jun 3, 2014
    Configuration menu
    Copy the full SHA
    adeded2 View commit details
    Browse the repository at this point in the history
Loading