Scheduling broken with Heap addition

ondras / rot.js

ROguelike Toolkit in JavaScript. Cool dungeon-related stuff, interactive manual, documentation, tests!

https://ondras.github.io/rot.js/hp/

BSD 3-Clause "New" or "Revised" License

2.33k stars 255 forks source link

Closed ondras closed 4 years ago

ondras commented 4 years ago

Hi @Syntacticlosure,

I am afraid your recent addition of min-heap completely broke the scheduling.

unit tests at tests/ are failing,
people are running into issues: https://twitter.com/mikemcmarvelous/status/1267221319478767616

Would you mind taking a look?

Syntacticlosure commented 4 years ago

Yes,i am looking for the problem.

Syntacticlosure commented 4 years ago

fix here : #182, sorry,it's all my fault to upload untested codes.

ondras commented 4 years ago

Great, thanks for the extra-prompt fix!