ecs-benchmark icon indicating copy to clipboard operation
ecs-benchmark copied to clipboard

Add new benchmark

Open EnderShadow8 opened this issue 3 years ago • 5 comments

I added the new benchmark as suggested by Sander.

They say the hardest problem in programming is naming things.

EnderShadow8 avatar Jan 29 '22 13:01 EnderShadow8

Thanks for the new benchmark! Regarding the suite's name, what about query_stress?

Could you fix the formatting? You can automate this using:

node ./node_modules/.bin/prettier --write 'src/cases/wolf-ecs/*.js'

ooflorent avatar Jan 29 '22 13:01 ooflorent

Ah. Better add .vscode to .gitignore.

EnderShadow8 avatar Jan 29 '22 13:01 EnderShadow8

I think something weird happened during the rebase 🤔

ooflorent avatar Jan 29 '22 13:01 ooflorent

Fixed the git history, after I nuked it with rebase.

Everything should be ok to merge now

EnderShadow8 avatar Jan 29 '22 13:01 EnderShadow8

There is an error in the benchmark:

  query_stress    ERROR
Error: maximum entity limit reached
    at ECS.createEntity (file:///home/runner/work/ecs-benchmark/ecs-benchmark/node_modules/wolf-ecs/wolf-ecs.js:260:23)
    at default (file:///home/runner/work/ecs-benchmark/ecs-benchmark/src/cases/wolf-ecs/query_stress.js:70:19)
    at file:///home/runner/work/ecs-benchmark/ecs-benchmark/src/bench_worker.js:9:16

ooflorent avatar Jan 29 '22 15:01 ooflorent