Skip to content
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

Optimizing Oengus schedule Import #129

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gaeldemarseille
Copy link
Contributor

  • calling twitch and speedrun.com api asynchronously
  • removed the 1s sleep from searchForUserData
  • prevent speedrun.com api rate limiting
  • no longer call speedrun.com api for runners info if we won't use any of them
  • added caching to searchForTwitchGame

Import duration difference from [Oengus Import] Started importing schedule to [Oengus Import] Successfully imported schedule from Oengus :

Old code

Importing ESAW22
with SRC (initial import)	4:39.070
with SRC (reimport)		1:15.961
without SRC			0:33.078

Importing GTAM21
with SRC (initial import)	3:19.769
with SRC (reimport)		0:30.514
without SRC			0:08.987

New Code

Importing ESAW22
with SRC (initial import)	2:02.394	(-56.1%)
with SRC (reimport)		0:01.978	(-97.4%)
without SRC			0:01.621	(-95.1%)

Importing GTAM21
with SRC (initial import)	1:00.885	(-69.5%)
with SRC (reimport)		0:01.753	(-94.3%)
without SRC			0:01.288	(-85.7%)

- calling twitch and speedrun.com api asynchronously
- removed the 1s sleep from searchForUserData
- prevent speedrun.com api rate limiting
- no longer call speedrun.com api for runners info if we won't use any of them
- added caching to searchForTwitchGame
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