-
-
Notifications
You must be signed in to change notification settings - Fork 292
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
Display estimate as a range #130
Comments
What would the different assumptions be? |
Magic constants for compression ratio and influence of quality. They are typical/guessed fudge factors, so it's valid to vary them. |
From #205 (comment):
What would those constants/assumptions be? This issue is a bit vague on how exactly it should be solved. |
let frameCount = duration * frameRateSlider.doubleValue
let dimensions = resizableDimensions.changed(dimensionsType: .pixels).currentDimensions.value
var fileSize = (Double(dimensions.width) * Double(dimensions.height) * frameCount) / 3
fileSize = fileSize * (qualitySlider.doubleValue + 1.5) / 2.5 The |
The estimate could be displayed as a range, e.g.: 0.5MB-2MB. This could be done simply by calculating estimate twice, with different assumptions (fudge factors).
Displaying it as a range would communicate to users that it's not an actual expected file size, and would communicate how imprecise the estimate is, so they would be less surprised when it's off.
The text was updated successfully, but these errors were encountered: