[Bug 2274250] Review Request: katago - GTP engine and self-play learning in Go

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=2274250



--- Comment #8 from Davide Cavalca <davide@xxxxxxxxxxxx> ---
(In reply to Tom Rix from comment #7)
> When i do
> 
> # rpm -ihv review-katago/results/katago-1.14.1-1.fc41.x86_64.rpm 
> error: Failed dependencies:
> 	katago-backend = 1.14.1-1.fc41 is needed by katago-1.14.1-1.fc41.x86_64
> 
> Then i have to figure out which to use 'eigen' or 'opencl' 
> I think it would be easier on the user if the 'eigen' subpackage was
> collapsed to the main package.

This is a side effect of using rpm -i. If the packages are all in a repo and
one does dnf install katago, dnf should automatically pull in the eigen
subpackage.

> The setup assumes the user is going to read the README.fedora.
> I did not the first time.
> 
> The nn url in the readme could be improved to
> https://katagotraining.org/networks/

Will fix, thanks

> Could a katago-setup setup script be added to automate the instructions ?

Not easily, as there isn't a stable URL to the "last good network". We'd have
to scrape it from that web page, which doesn't seem worth the trouble.

> Some other nits from the review
> katago.spec:40: W: unversioned-explicit-provides bundled(clblast)           
> 
> katago.spec:43: W: unversioned-explicit-provides bundled(sha2)              
> 
> Take a guess on the versions.

These don't really _have_ a version unfortunately :(

> katago.spec:60: W: mixed-use-of-spaces-and-tabs (spaces: line 12, tab: line
> 60)

Will fix, thanks.

> Is there going to be a follow on gui package to go with this engine ?

Yes, I have a half-done package for q5go I'm planning to submit for review.


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
https://bugzilla.redhat.com/show_bug.cgi?id=2274250

Report this comment as SPAM: https://bugzilla.redhat.com/enter_bug.cgi?product=Bugzilla&format=report-spam&short_desc=Report%20of%20Bug%202274250%23c8
--
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux