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

Don't queue up more than a single save command #19

Closed
huntfx opened this issue Jun 29, 2017 · 1 comment
Closed

Don't queue up more than a single save command #19

huntfx opened this issue Jun 29, 2017 · 1 comment
Assignees

Comments

@huntfx
Copy link
Owner

huntfx commented Jun 29, 2017

If the saving is set to 5 seconds for example, by the time one save is finished, there may have been another two queued up. This causes the script to get further and further behind where it'll never get a chance to catch up.

@huntfx huntfx added this to the Initial Release milestone Jun 29, 2017
@huntfx huntfx self-assigned this Jun 29, 2017
@huntfx
Copy link
Owner Author

huntfx commented Jun 29, 2017

I think I've got it working. The next save is now calculated from the end of the saving, not the start. The background thread needs to send back a notification for this to happen.

@huntfx huntfx closed this as completed Jun 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant