search

noctjs / ecs-benchmark

ECS benchmark comparison
MIT License
110 stars 23 forks source link

Add new benchmark #57

Open EnderShadow8 opened 2 years ago

EnderShadow8 commented 2 years ago

I added the new benchmark as suggested by Sander.

They say the hardest problem in programming is naming things.

ooflorent commented 2 years ago

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'
EnderShadow8 commented 2 years ago

Ah. Better add .vscode to .gitignore.

ooflorent commented 2 years ago

I think something weird happened during the rebase 🤔

EnderShadow8 commented 2 years ago

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

Everything should be ok to merge now

ooflorent commented 2 years ago

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