enable dynamic renormalization in updateWDL.sh #153
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR prepares the update script for the future, when official-stockfish/Stockfish#4920 will be merged into SF.
Some comments:
NormalizeToPawnValue
not being present insrc/uci.h
for SF revisions that do the dynamic eval rescaling. Edit: It now relies onconst in NormalizeToPawnValue
not being present insrc/uci.h
andconstexpr int NormalizeToPawnValue
not being present insrc/uci.cpp
.NormalizeToPawnValue
is computed on the fly, by summing the coefficients inas
. The final sum is rounded, so that after merging use round not trunc for NormalizeToPawnValue #136 this will be consistent with the python script. Note also that for un-doing the dynamic rescaling, the valueNormalizeToPawnValue
serves only a cosmetic purpose now. It is used at the end of the update script to indicate to the user if that value has changed from SF to the new model, or not.--firstrev
and--lastrev
it allows for extracting the newly fittedNormalizeToPawnValue
for both static and dynamic renormalization revisions of SF. This may be useful when adapting WIP: update a range of commits #119For as long as official-stockfish/Stockfish#4920 is not merged into SF, this PR is nonfunctional. Once that PR is merged, the update script needs to be pointed to that new commit in
default_firstrev
, and it should then seamlessly work for the dynamic eval rescalings.