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

Change Progress Updates from Counter to Time based #1504

Closed
wants to merge 1 commit into from

Conversation

toshski
Copy link
Contributor

@toshski toshski commented Nov 17, 2023

This is a quality-of-life change. It changes the various Progress Messages to be time based (15 seconds) rather than count based.

e.g. after scraping sites, indexing scenes currently updates every 100 scenes, you can get 200 progress messages in a few seconds, because most of your scenes are already indexed. This also clogs up the logs. However, if you clear you your search index and rebuild, then every 100 scenes is more appropriate as it is a lot longer to process as every scenes has to be index.

Time based also better allows for a wide range of machine with different processing speeds.

Progress updates will happen at a default rate of every 15 seconds. The setting can be changed in the database, via the kvs table in the config record where most other settings. Look for the progressTimeInterval entry under the advanced group, it is an existing setting, as time-based updates were used in the Actors mod. I haven't made configurable through the UI, as I assume 15 seconds should suit most and not really need to be changed, but I can add it into the Options UI if needed.

@toshski toshski closed this May 31, 2024
@toshski toshski deleted the Time_Based_Progress_Alerts branch May 31, 2024 18:37
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.

None yet

1 participant