search

PolymerLabs / arcs

Arcs
BSD 3-Clause "New" or "Revised" License
57 stars 35 forks source link

Run some reasonable number of speculations in parallel. #705

Open shaper opened 6 years ago

shaper commented 6 years ago

From chat discussion:

There is a TODO to fix this: https://github.com/PolymerLabs/arcs/blob/master/runtime/planner.js#L139

It was earlier changed from running all speculative executions in parallel:

https://github.com/PolymerLabs/arcs/commit/98b62f12f557dbb8dddcdc9ab1afd342e8f1807d#diff-aa20163f4eb8b98dfd65800032bad5f3

because IIRC it was consuming too much RAM.

navigator.hardwareConcurrency looks helpful.

shaper commented 6 years ago

Seems could be fun and out of near term critical runtime path. FYI @dstockwell @mmandlis

shaper commented 6 years ago

Will open anew if we determine rework needed per comment in above issue.