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

Feature Request: Adding in install buffer to estimation times #77

Closed
Eltord opened this issue Jul 13, 2023 · 6 comments
Closed

Feature Request: Adding in install buffer to estimation times #77

Eltord opened this issue Jul 13, 2023 · 6 comments
Assignees
Labels
enhancement New feature or request waiting feedback Waiting on feedback from requestor

Comments

@Eltord
Copy link
Contributor

Eltord commented Jul 13, 2023

Problem:

Sometimes users look at the estimates for the download and go "well how come it says an estimated 2 minutes 20 seconds, but it took 15 minutes to complete?" Well, they aren't grasping that its download time and not install time being presented to them.

Solution:

Add in a buffer to the block of configuration variables that we can add in a set amount of seconds to apply to the estimates. Made it individual so to give full customizability if certain configurations take longer than others due to the packages being installed.

configurationInstallBuffer="480"     # Buffer time added to estimates to include installation time of packages, in seconds. Set to 0 to disable.

configurationOneInstallBuffer="480"     # Buffer time added to estimates to include installation time of packages, in seconds. Set to 0 to disable.

configurationTwoInstallBuffer="600"     # Buffer time added to estimates to include installation time of packages, in seconds. Set to 0 to disable. 

configurationThreeInstallBuffer="720"     # Buffer time added to estimates to include installation time of packages, in seconds. Set to 0 to disable.

configurationOneEstimatedSeconds=$( echo "scale=2; ((((( $configurationOneSize / $mbps ) * 60 ) * 60 ) * $correctionCoefficient ) + $configurationOneInstallBuffer)" | bc | sed 's/\.[0-9]*//' )

configurationTwoEstimatedSeconds=$( echo "scale=2; ((((( $configurationTwoSize / $mbps ) * 60 ) * 60 ) * $correctionCoefficient ) + $configurationTwoInstallBuffer)" | bc | sed 's/\.[0-9]*//' )

configurationThreeEstimatedSeconds=$( echo "scale=2; ((((( $configurationThreeSize / $mbps ) * 60 ) * 60 ) * $correctionCoefficient ) + $configurationThreeInstallBuffer)" | bc | sed 's/\.[0-9]*//' )

configurationCatchAllEstimatedSeconds=$( echo "scale=2; ((((() $configurationCatchAllSize / $mbps ) * 60 ) * 60 ) * $correctionCoefficient ) + configurationInstallBuffer)" | bc | sed 's/\.[0-9]*//' )

@Eltord Eltord added the enhancement New feature or request label Jul 13, 2023
@dan-snelson
Copy link
Collaborator

Happy Thursday, @karatetrigger!

Thanks for the FR; please confirm that the already available correctionCoefficient does NOT meet your use-case.

@dan-snelson dan-snelson added the waiting feedback Waiting on feedback from requestor label Jul 13, 2023
@Eltord
Copy link
Contributor Author

Eltord commented Jul 13, 2023

Howdy howdy @dan-snelson

It does not as the different configurations would at least require different "fudge factors" in my case, as the factor would need to increase with more packages requiring to be installed. Modifying the correctionCoefficient didn't seem to give me the results and times I was looking for across the board, so it felt easier to just after a few times running through it on a machine, get a good estimate of time it takes to run through the configuration, add that in seconds to the estimate, and gave a much better result.

@Eltord
Copy link
Contributor Author

Eltord commented Jul 13, 2023

I did leave that in there in the math for the estimates in seconds, as even leaving it at 1.01 or slightly more, accounts for the smallest factors in kbps download speeds, but in total it did not answer the full problem it was trying to solve.

@dan-snelson
Copy link
Collaborator

Thanks for the additional information.

@dan-snelson
Copy link
Collaborator

Thanks again, @Eltord!

@dan-snelson
Copy link
Collaborator

Addressed in Setup Your Mac 1.12.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request waiting feedback Waiting on feedback from requestor
Projects
None yet
Development

No branches or pull requests

2 participants