Incorrect PIECE_VALUE used for multiplication of knights on the evaluate function

[tolumide-ng]

Hello,
Thank you for your hard work and dedication to this project! I really appreciate the effort that goes into maintaining it.
I encountered an issue I think might have been an unintended typo on the evaluate function in position.rs here:

carp/engine/src/position.rs

Line 153 in 91a609f

self.board.knights().count_bits() as Eval * PIECE_VALUES[Piece::WK as usize] +

It appears that the PIECE_VALUE of a king was used to multiply the knights count.

Thanks again for your time.

[dede1751]

Thanks a lot for pointing this out, this is indeed a bug!
Sadly, Carp is not currently in development and although this is a bug standard chess engine development practice is to test every change. I will leave this issue open and try to fire up a test when I have the time.