Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Simulations API Performance Improvements #29

Open
damonhook opened this issue Feb 12, 2020 · 0 comments
Open

Simulations API Performance Improvements #29

damonhook opened this issue Feb 12, 2020 · 0 comments
Labels
api The issue relates to the Nodejs API (Backend)
Milestone

Comments

@damonhook
Copy link
Owner

Right now it does a complete set of simulations per save. This means that at a numSimulations of 5,000 it actually simulates 30,0000 attack sequences.

However, an attack sequence is the same up until the save roll phase. This means we could change the simulations to simulate once, then at the save phase, do 6 save rolls. Thus we would only do 5,000 attack sequences up until the divergent path.

This would dramatically improve response times and allow us to increase the number of simulations used

@damonhook damonhook added enhancement New feature or request api The issue relates to the Nodejs API (Backend) labels Feb 12, 2020
@damonhook damonhook added this to To do in v2.0.0 via automation Feb 12, 2020
@damonhook damonhook added this to the v1.2.0 milestone Feb 12, 2020
@damonhook damonhook removed the enhancement New feature or request label Feb 13, 2020
@damonhook damonhook removed this from To do in v2.0.0 Feb 18, 2020
@damonhook damonhook removed this from the v2.0.0 milestone Feb 18, 2020
@damonhook damonhook added this to the 3.0.0 milestone Jan 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api The issue relates to the Nodejs API (Backend)
Projects
None yet
Development

No branches or pull requests

1 participant