Skip to content

perf(epic): back off idle download polling when no downloads are active - #149

Open
amir-rezaei wants to merge 1 commit into
mont127:mainfrom
amir-rezaei:perf/back-off-idle-download-polling
Open

perf(epic): back off idle download polling when no downloads are active#149
amir-rezaei wants to merge 1 commit into
mont127:mainfrom
amir-rezaei:perf/back-off-idle-download-polling

Conversation

@amir-rezaei

Copy link
Copy Markdown

Description

This PR resolves issue #125 by backing off background download polling in EpicLandingView.swift when no downloads are active.

Details

  • Previously, startGamesPolling continuously polled legendary_all_downloads every 3 seconds regardless of whether any download task was in progress.
  • This PR increases the polling interval to 10 seconds when backend.epicDownloads is empty (and maintains 3 seconds during active downloads), reducing CPU and IPC log churn.

@realmaitreal realmaitreal left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's making it better, but it would be even better we replace polling with an event from legendary directly

@realmaitreal realmaitreal added enhancement New feature or request Medium priority Medium priority Feat QoL Quality of life improvement, no new compatibility area:epic Epic Games integration labels Jul 27, 2026
@mont127

mont127 commented Jul 27, 2026

Copy link
Copy Markdown
Owner

@amir-rezaei whats your discord?

@amir-rezaei

Copy link
Copy Markdown
Author

Thanks @realmaitreal and @mont127! Good suggestion on integrating event notifications directly from Legendary when active downloads change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:epic Epic Games integration enhancement New feature or request Feat QoL Quality of life improvement, no new compatibility Medium priority Medium priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants