Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Changelog for v0.17.0-rc1 #273

Merged
merged 13 commits into from
Aug 19, 2018
Merged

Conversation

mooskagh
Copy link
Member

No description provided.

@mooskagh mooskagh requested review from dubslow and borg323 August 19, 2018 09:50
changelog.txt Outdated
* Added --out-of-order-eval flag (default is off).
Switching it on makes cached/terminal nodes higher priority, which increases
nps.
* OpenCL backend now supports batches (~5x speedup!)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is up to 5x

changelog.txt Outdated

Other changes:
* Release .zip package now contains NVdia CUDA and cuDNN .dlls.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Release windows executables are built with "whole program optimization".

changelog.txt Outdated


Other changes:
* Release .zip package now contains NVdia CUDA and cuDNN .dlls.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mention full lc0-windows-cuda.zip name?

changelog.txt Outdated

syzygy-paths=/path/to/syzygy/

* In uci info, depth is not average depth rather than full depth
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess this should be now average depth

changelog.txt Outdated
* Tablebases are supported now (only WDL probe for now).
Command line parameter is
--syzygy-paths=/path/to/syzygy/
* Old smart pruning flag is gone. Insted there is
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo for instead

changelog.txt Outdated
not possible.
* Fixed the bug which caused --cache-history-length for values 2..7 work the
same as --cache-history-length=1.
This is fixed, but default is temporary changed to --cache-history-length=1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be temporarily :)

changelog.txt Outdated
@@ -21,7 +21,7 @@ New visible features:

syzygy-paths=/path/to/syzygy/

* In uci info, depth is not average depth rather than full depth
* In uci info, "depth" is now average depth rather than full depth
(which was 4 all the time).
Also, depth values do include reused tree, only nodes visited during the
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Er uh this should say "do not include reused tree"

changelog.txt Outdated
@@ -48,7 +48,7 @@ Bugfixes:
not possible.
* Fixed the bug which caused --cache-history-length for values 2..7 work the
same as --cache-history-length=1.
This is fixed, but default is temporary changed to --cache-history-length=1
This is fixed, but default is temporariry changed to --cache-history-length=1
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Uh... another typo? should be "temporarily" I think

@mooskagh mooskagh merged commit c72d6d1 into LeelaChessZero:master Aug 19, 2018
borg323 pushed a commit to borg323/lc0 that referenced this pull request Sep 6, 2018
* Changelog for v0.17.0-rc1

* More details in changelog.

* Code review fixes.

* Mistype fix.

* Clang-format all files.

* Fix comment.

* Add "for FPU reduction" to changelog.

* Fix typo.

* Typo.

* typo

* Typo LeelaChessZero#5.

* Fix number_out_of_order.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants