Vlek/roll-cli

Convert parser to Cython

Vlek opened this issue · 2 comments

Vlek commented

Attempt to increase the speed of the script by converting the underlying parser/evaluator to Citron for a hopefully decent speed boost. Current tests take ~0.50ms to complete. I wonder if I can halve that time.

Vlek commented

I think we should take a good look at our speed and memory usage statistics in our speed metric tests in #41 and really consider whether this is the bottleneck. I believe that it is pretty fair to say that this begins to become a bottleneck when it's the case that we have thousands of dice rolls, but with regular usage it is unlikely to be the case.