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

tempTimer does not fire at exactly the specified time #7444

Closed
deliberatelycomposed opened this issue Sep 20, 2024 · 1 comment · Fixed by #7528
Closed

tempTimer does not fire at exactly the specified time #7444

deliberatelycomposed opened this issue Sep 20, 2024 · 1 comment · Fixed by #7528
Labels

Comments

@deliberatelycomposed
Copy link

Brief summary of issue / Description of requested feature:

tempTimer does not fire at exactly the specified time, in my testing it is routinely off by up to a half of a second.

Steps to reproduce the issue / Reasons for adding feature:

  1. Capture the current time with getEpoch().
  2. Create a tempTimer.
  3. When the tempTimer fires, compare the current time with the captured timestamp. This should be pretty close to the duration specified for your tempTimer, within a few milliseconds I would expect?

Here is a short script I wrote to reproduce the issue.

I also tried capturing the timestamp in step 1 after creating a timer, but the discrepancies were about the same, so I don't think its the actual time spent creating the timer which is the issue here.

Error output / Expected result of feature

Example results from the script linked above.

Extra information, such as the Mudlet version, operating system and ideas for how to solve / implement:

Linux, Mudlet 4.18.5

@deliberatelycomposed deliberatelycomposed changed the title tempTimer does not fire at exactly specified time tempTimer does not fire at exactly the specified time Sep 20, 2024
@ZookaOnGit
Copy link
Collaborator

discord discussion, need to test if a more accurate timer will not significantly affect performance
https://discord.com/channels/283581582550237184/283582068334526464/1286718111135043605

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants