-
-
Notifications
You must be signed in to change notification settings - Fork 4.5k
heavy_compute bench has bad parameters #4798
Copy link
Copy link
Closed
Labels
A-TasksTools for parallel and async workTools for parallel and async workC-BugAn unexpected or incorrect behaviorAn unexpected or incorrect behaviorC-PerformanceA change motivated by improving speed, memory usage or compile timesA change motivated by improving speed, memory usage or compile times
Description
bevy 0.8-dev e5eb837
What happened
The current heavy_compute benchmark is testing 1000 entities / 128 batch = 8 batches and using the default task pool. This makes the test overly sensitive to the number of cores on a machine.
Desired Fix
Not sure what the best fix is here. Playing with the batch size, I was seeing the best perf at around 8 to 16. We could change the batch size or increase the number of entities or fix the number of threads to a lower number. Or vary the number of threads, Or vary the batch size.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
A-TasksTools for parallel and async workTools for parallel and async workC-BugAn unexpected or incorrect behaviorAn unexpected or incorrect behaviorC-PerformanceA change motivated by improving speed, memory usage or compile timesA change motivated by improving speed, memory usage or compile times