Skip to content

fix: correct statement about execution time(Event loop) #3422

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

JeraldVin
Copy link
Contributor

@JeraldVin JeraldVin commented Apr 1, 2023

fix: correct statement about execution time(Event loop)

Current text:
The notable thing is that both variants -- with and without splitting the job by setTimeout -- are comparable in speed. There's not much difference in the overall counting time.

Actual behaviour:
The execution time increases when using setTimeout. ~1.5secs without and ~8secs with setTimeout(desktop). ~3.5secs without and ~15secs with setTimeout(mobile). After the scheduling improvement, the setTimeout variant takes ~4secs(desktop and mobile)

chore: improvement in starting sentence grammar

chore: fix conflicting statement in Event loop(added 'even')
chore: improvement in starting sentence grammar
when using setTimeout, the execution time actually increases
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