Update Python bindings README.md

[JavClaude]

tokenizer.train(["./path/to/dataset/1.txt"], trainer=trainer)

Fix the "PyTokenizer.train() got multiple values for argument: trainer" error when the "files" parameter is missing

[n1t0]

I'm not sure to understand. The files parameter is not optional so it can't not be there.

[JavClaude]

The parameter is not mentioned in the documentation. What induces the error when trying the piece of code.
This does not fix the error in itself, just the document induces an error. Sorry for the incomprehension

[marcglobality]

I just came here because was having the mentioned error when running the example in the docs. Any reason not to merge this @n1t0 ?

[marcglobality]

I think the error appeared since #544

[n1t0]

The README reflects the most up to date state of the library, not the latest stable. If you try the 0.10.0rc1, I'm pretty sure this error does not exist.
Maybe we should make this clearer in the readme though.

[marcglobality]

The README reflects the most up to date state of the library, not the latest stable. If you try the 0.10.0rc1, I'm pretty sure this error does not exist.
Maybe we should make this clearer in the readme though.

For the future, I think the README should be updated to the latest stable, not the latest rc. Given that eventually the RC will becomes stable and the example in the README will work, I think @JavClaude can close this PR then?

If you feel motivated, it'd be great to get in the README the example here 723ee7c#diff-810f2de0965b0a169a00cee33465f4165347b006b5360ae3b1001723a6e9ebd3R19 to train with in-memory data