official-stockfish / nnue-pytorch

Stockfish NNUE (Chess evaluation) trainer in Pytorch
GNU General Public License v3.0
320 stars 99 forks source link

Add formatting #277

Open Disservin opened 6 months ago

Disservin commented 6 months ago

@Sopel97 what do you say about adding some formatters?

black is also used for the wdl repo https://github.com/official-stockfish/WDL_model/tree/master, and the clang-format is taken from stockfish master

Sopel97 commented 6 months ago

if we can get it to agree whether strings should be '' or "" I'm fine with it

Disservin commented 6 months ago

It uses double quotes " but some strings have double quotes inside them and use the opposite quotation characters. I.e. print('root_dir - the directory to "cleanup"'), this isn't automatically converted into a double quote because it'd need escaping.

vondele commented 4 months ago

I would also add a formatter, so if this can be rebased, let's merge.