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

Per-file progress for uninstall and upgrade #4193

Merged
merged 1 commit into from
Sep 24, 2024

Conversation

HebaruSan
Copy link
Member

@HebaruSan HebaruSan commented Sep 24, 2024

Motivation

As of #4055, progress bar updates during installation are more granular and reflective of the actual progress of unpacking mods, but removal and upgrades still treat each mod as a single unit and all mods as the same size, which makes the progress bar nearly useless.

Changes

Now for removal, we treat each file to remove as a unit of work, and mods with more files take up more progress bar space.

Now for upgrading, the removal part is treated as above and the installation part is treated as in #4055. The proportion of the progress bar devoted to removal vs installation is weighted by the number of mods in each group, so for most upgrades we will switch from removal to installation at the 50% mark, and the installation piece will expand when we're pulling in new dependencies.

@HebaruSan HebaruSan added Enhancement New features or functionality Easy This is easy to fix Core (ckan.dll) Issues affecting the core part of CKAN labels Sep 24, 2024
@HebaruSan HebaruSan merged commit 219f799 into KSP-CKAN:master Sep 24, 2024
3 checks passed
@HebaruSan HebaruSan deleted the feature/uninstall-progress branch September 24, 2024 18:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Core (ckan.dll) Issues affecting the core part of CKAN Easy This is easy to fix Enhancement New features or functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant