Change Progress Updates from Counter to Time based #1504
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.