Skip to content

do elo calculation on a different thread#74

Open
owobred wants to merge 1 commit intolive-elofrom
live-elo-seperate-thread-calculator
Open

do elo calculation on a different thread#74
owobred wants to merge 1 commit intolive-elofrom
live-elo-seperate-thread-calculator

Conversation

@owobred
Copy link
Collaborator

@owobred owobred commented Dec 20, 2024

Changes

Currently elo calculation is done in an async task and is massively blocking, which can upset tokio if we have a small number of cpu cores and could cause the program to lock up until elo calculation is done.

Checklist

  • My code compiles
  • I have committed all the files needed to build the project (check if your file is found in .gitignore)
  • If I'm introducing a new step in the build process, I have documented / automated it
  • I have tested my changes (minimally with one Twitch VOD)

To-do list

idk i think everything is fine, you might want to check though

@owobred owobred self-assigned this Dec 20, 2024
@owobred owobred marked this pull request as ready for review December 20, 2024 12:02
@owobred owobred requested a review from jameshi16 December 20, 2024 12:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant