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

easy enhancement #270

Open
ArizonaCowboy opened this issue Oct 9, 2024 · 5 comments
Open

easy enhancement #270

ArizonaCowboy opened this issue Oct 9, 2024 · 5 comments
Labels
enhancement New feature or request

Comments

@ArizonaCowboy
Copy link

ArizonaCowboy commented Oct 9, 2024

Could you display positive multi-PV evals in green and negative evals in red? - with only a + or - sign separating the positive and negative eval numbers, it is easy to accidentally misread them, which causes confusion. Also, how to we donate funds to this project?

@rooklift
Copy link
Owner

Hmm. It's possible to do. However Nibbler isn't really under development these days.

As for donating, as I say, there is no development, so you'd only be donating for work already done. But anyway there is no such process.

@rooklift rooklift added the enhancement New feature or request label Oct 11, 2024
@ArizonaCowboy
Copy link
Author

how about $100 for this enhancement and previous work - and for incremental tiny improvements in the future, it is near perfect, so don't mess with it much! - It would be sad to see the world's best chess analysis UCI for both Stockfish and Leela and other top engines go completely dormant.

@rooklift
Copy link
Owner

Hmm - as I say, there is no such process.

Anyway, I dunno if I want this feature myself, but you can try editing 82_infobox.js in your files, and replace this line:

substrings.push(`<span class="blue">${value_string} </span>`);

With:

substrings.push(`<span class="${info.q > 0 ? "blue" : "pink"}">${value_string} </span>`);

It should be line 170 if you have the most up to date version.

@ArizonaCowboy
Copy link
Author

ArizonaCowboy commented Nov 3, 2024

OK, I made the following changes, and it is now, IMHO, perfect!

  170       substrings.push(`<span class="${info.q > 0 ? "green" : "red"}">${value_string} </span>`);
  184       spanclass = colour === "w" ? "white" : "gray";
  222       --> this line of code was deleted <--  (don't need WDL)
  255       substrings.push(`<span class="yellow">(${extra_stat_strings.join(', ')})</span>`);

if you want the $100 (or £80), please either post or DM me your paypal or venmo address where I can send the support - a bargain in my eyes for the perfect analysis UCI interface for Stockfish! - Thank you sooo much!

@ArizonaCowboy
Copy link
Author

ArizonaCowboy commented Nov 3, 2024

nibbler example

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants