Skip to content

[optimizing] float32 and small refactoring#53

Merged
davidfischer-ch merged 2 commits intosmartcrop:masterfrom
derVedro:float32
Mar 1, 2026
Merged

[optimizing] float32 and small refactoring#53
davidfischer-ch merged 2 commits intosmartcrop:masterfrom
derVedro:float32

Conversation

@derVedro
Copy link
Contributor

Using float32 for the computation should speed up calculations a little when dealing with larger arrays. Unfortunately, the results are almost imperceptible and can be considered statistical errors. However another change - the factoring out a subtraction out of score method - give a minor boost for larger crops and a bigger amount of crops (for example for other min_scale, max_scale and scale_step values besides the defaults).

@derVedro derVedro mentioned this pull request Feb 20, 2026
@davidfischer-ch davidfischer-ch merged commit d14d490 into smartcrop:master Mar 1, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants