-
Notifications
You must be signed in to change notification settings - Fork 109
Add stats command #239
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
base: main
Are you sure you want to change the base?
Add stats command #239
Conversation
|
well.. lichess already shows some of these.. but the others seem cool |
|
Hmm , the imports should be at top ... |
fixed, torom let me know if you want any specific things in the stats (or if you want the command) |
|
I think it's overkill to download all the games of the day just to get daily statistics. I also hope that you understand your AI code and test it yourself. The RAM and CPU usage might be quite nice, we already have everything we need for that... |
I have fixed this, thank you.
I also understand the AI code and have tested it, thank you very much. This is an example of what it looks like currently: |
Doesn't look like it to me. |
@Torom |
|
Remove comments (lines starting with a checkmate symbol) |
@veerbot |
|
All of them CONMENTS DONT MAKE SENSE |
|
U can do it manually without ai |
|
Such as |
|
@veerbot |
@Torom I am waiting for your review. If you want I can add RAM and CPU usage, or other info in stats. EDIT: I have found the user activity endpoint, sorry for the inconvenience. Is it okay if I add colour to the stats? I can commit with new endpoint. |
|
What I wrote above still applies. I don't want to download all the games of a day. Please don't add color to keep the scope of the PR minimal. |
I can remove it, no problem.
We can use https://lichess.org/api#tag/Users/operation/apiUserActivity right? |
|
Yes, that's one of the endpoints your AI had already shown you... |
|
I am not sure but https://lichess.org/api/user/{username}/perf/{perf} is an endpoint |
|
@Torom
@suprateem-ux |
|
Tbh it feels out of scope of botli . Botli is basically an interface between engines , books and the lichess bot api . Also the user can cleanly see it via website and i am not a fan of downloading ideas in botli |
True, I guess you can see it in the browser. I am just waiting for @Torom to approve / disapprove it. |
|
I have already said twice that I don't want the game download in BotLi. I could imagine a command that reads system parameters such as CPU, RAM, and perhaps temperatures (?) and displays them to the user being useful. |
@Torom I can use that endpoint to make it look something like: Do you want that or do you want to change it a bit? |
|
@Torom This is an example of what it looks like now: I can start implementing RAM and CPU |
|
Sorry, but that's too much code for something you can see with three clicks on the Lichess website. |
@Torom |
No description provided.