-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Give us all the SF main PV plies. To include the evaluated leaf position. #11452
Comments
I don't think the SF issue you're describing (#2423) has anything to do with the cap in the UI. One was about showing more, the other is about showing less. pvs are currently capped at 16: https://github.com/lichess-org/lila/blob/master/ui/ceval/src/view.ts#L388 Long pvs eventually get quite inaccurate though and it does clutter the UI so not sure how much sense increasing or removing the limit makes. |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
puzzle starting position. 1WR5Uusing the puzzle source game FEN and putting in analysis: I still get less than 16 or 16. It could still be transposition table for shorter than 16 and I do not see the old behavior of all 3 PVs having the same cut off as last year. But what we see with the extra ply in the solo analysis tool (not in puzzle context, with its own parameters), That the PV has changed with deeper input depth AND that the PV display depth, there, did not budge. Those observations alone are not enough to show the possible capping at user end of using lichess interface with SF executable used. I Need another equivalent experiment such as command line SF for info output with full PV length, where we can control the hash table too. That is WIP. However, this has been done last year, as can be read in the following posts from lichess forum, Todos
The last entry is WIP. see last comment for more. The software repository refers to libraries from lichess and chessground, in the readme file. The interface does not seem completed in terms of saving local files. The log can be copy pasted. And I can join screenshots to show the PV lengths exhibited in the PV box. Using SF12 does not matter, the question is about getting the leaf position at the source of the root score for any analysis. Trying the other engines, too, just to see if they also show Full PV. |
Getting off track here. Giving the full PV to the user is possible (though I believe that additional information is completely useless). It can't hurt anything except steal 15ish pixels per expanded PV from the move list, so maybe we should just label this "Good First Issue" and let someone do it. Then thibault could either merge it or not. |
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
Not a bug, but a real improvement suggestion for the lichess users doing engine analysis routinely and even talking about the root score, never really seeing the position that gave those values at the basis of their chess analysis and discussions. Help. How does one add a label of "improvement" to an issue? If this is in my jurisdiction as non contributor and visitor. why 16? why not 10. why not 5. why even display the score of the PV, if what only matters is the "accuracy" for the move candidate found best after the long inaccurate tree search (where does the inaccuracy creep in, exactly?). The TT keeps all recent evaluations. I would hope it would still have the hash of the position that gave the score value. If that is inaccurate. why clutter the display with root position score? There are real questions. I would like someone to give a try to answering them. It might either unclog some sinuses. or make me understand the common sense if any behind post #2, second sentence unsupported opinions, as can be read here. |
Liground interface. Please consider the right hand side. I will use this software, which is in part based on lichess codebase, to show that no it is possible to not have that mystifying cap put at 16.
6k1/7p/2q1r3/p1P2r2/P1Q5/5PP1/7P/5RK1_w_-_-_7_36 |
Liground. journalling WIP. problem in analysis mode. it does go depth infinite.
that box is under the engine log output (info) console. UCI talking. so i can do go depth 20.. that should be enough proof of concept based on software referring to lichess modules. Perhaps they PV display code is not. In liground, when choosing SF12 I get the following credits:
So I wonder what the other engine names multivariant might be: Answer same authors older version. scratch that (unless it shows some PV printout evolution. Finally maybe bad idea... this liground. although the UCI console shows that even those version output long PVs. the other is "Fairy-Stockfish 13 LB" by Fabian Fichter. I am confused as to which of those is lichess sending to our browsers.. I thought it was the Fairy version.. anyway will test all. |
Maybe there is a pîece of information missing.. that I take for granted.
so I might require some good will for the proof of concept. or do the whole command line thing with corresponding version once known. in the meantime. i just put the links and get the results from there, with leafs according to UCI info output for depth 17. also did 16.. but best would be to find a depth that jumps the score so people get the obvious screaming point. put that in the web guide too. all the PV leaves for known engine.. into the web guide unknown version (maybe it got fixed).
(might also try the command line on known local version of SF), in spite of past eval mode syntax ambiguities... does the web guide apply the same exact evaluation function as in SF on lichess or my command line. does it matter for proof of concept. The point is that one can look at the only full position information that is input to the static evaluatoin (e.g. web guide) and gives the root score value.. i don't get what is so cluttery or meaningless about that. wrong universe probably. |
This comment was marked as resolved.
This comment was marked as resolved.
At this point, you're just spamming. I think 16 is a good enough limit. |
This comment was marked as spam.
This comment was marked as spam.
could someone delete this whole issue. Having worked hard on it so uselessly given the self-satified convictions.. is a waste of everybodies times and nerves. got the message about some cultures expectations. my bad. too much expections about lichess. |
The real meaningless PV main sequence is the one that is arbitrarily
truncated. none of the position shown there have had the evaluation that
the root position get made.. (if any evaluation was made). /
We should care about where the meanings come from.. A lot of users are
in the fog, and will keep there and keep thinking of engine as ultimate
unscrutable source of truth. Having that last position might unveiled a
corner, at the population scale. Forcing everbody to go commanline to
get that basic information seems a bit outdated thinking to me./
Also there would be no additional clutter, as the PVs are already
collapsible to one liners. It is up to the user to have it take the full
space..
That was point #3 in first post. I am already not very concise. Having
to untangle the past issues on both lichess and SF, to support my point
of lichess not having updated its design choices for the news that now
SF does provide the full PV depths in output, and that not being the
case before, was the conclusion keeping 16 from being updated, is a
recipe, for lots of rambling, not very efficient, on my part. Best
consider the facts are they are now, perhaps than the historical
decision logic being possibly moot.
Le 29/08/2022 à 16:52, Dboingue a écrit :
… long PVs getting inaccurate? the main PV of 22 or 50 in puzzles
creation, being inaccurate?
The score attribute to current root search comes exactly from the exact
position information all the way down the PV.
I am not talking about the PV itself as important, but the last posiiton
which evaluation is exaclty and accurately what the variation first move
received as score being printed on the UI. This is about understanding
the point of view of the engine. Yes the path will change if updating
root. but the best move advertised at single root search is exactly the
accurate result of the min-max optmization. I will wait for more
voices. to show why the cap is at 16 having been based on SF not giving
it in the past.
It may have been linked to the other direction of how many PVs, and is
often the source of confusion, in the spaghetti of issues. I wish I did
not have the burden of untangling again. But the fact is that now SF
gives it all. And it did not in the past. Maybe read about min-max
basics. (I can't believe i am the one resorting to such burden of proof
transfer, but I hope others have read the first post)... issue
archeleogy, I wanted to avoid.. but maybe i should not even have
mentioned their existence.. my bad. i offered a distraction
counter-argument.
Le 29/08/2022 à 16:34, Benedikt Werner a écrit :
>
> I don't think the SF issue you're describing (#2423
> <#2423>) has anything to do
> with the cap in the UI. One was about showing more, the other is about
> showing less.
>
> pvs are currently capped at 16:
>
https://github.com/lichess-org/lila/blob/master/ui/ceval/src/view.ts#L388
>
> Long pvs eventually get quite inaccurate though and it does clutter
> the UI so not sure how much sense increasing or removing the limit
makes.
>
> —
> Reply to this email directly, view it on GitHub
>
<#11452 (comment)>,
> or unsubscribe
>
<https://github.com/notifications/unsubscribe-auth/ADNEZIFKR6LQ2ANZAOZJD5DV3UNFDANCNFSM576V4RMQ>.
> You are receiving this because you authored the thread.Message ID:
> ***@***.***>
>
—
Reply to this email directly, view it on GitHub
<#11452 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADNEZICVYSNGFWT37HRYNLTV3UPKXANCNFSM576V4RMQ>.
You are receiving this because you are subscribed to this
thread.Message ID: ***@***.***>
|
- I hereby ask that lichess gives us all the plyes or plies be it 22, 23, or more, not 16, or less, as is now the usual case.
We should be able to see the leaf position that gave a score we read in any engine analysis output.
There were related issues in the past, but they ultimately relied on the SF issue which was fixed. Long story. SF now gives it all.
**and sometimes less, not always about transposition table, as even many PV setting would adopt the same cap, which would be unlikely SF search results.
The text was updated successfully, but these errors were encountered: