-
Notifications
You must be signed in to change notification settings - Fork 153
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
Track LPNF vs. OPNF performance #65
Comments
For propnet size as measured by the number of components, even small propnets are better coming out of the OPNF. (These measurements are before any post-optimizations are done.) Games where the number of components from the OPNF is greater than the number of components from the LPNF:
And then there are the eight games listed above where OPNF creation failed and LPNF creation did not. Again, some of these games have been superseded. |
And measuring link counts instead of component counts:
|
The goal of these is to determine if we can get rid of the LegacyPropNetFactory entirely in favor of the OptimizingPropNetFactory.
Results from a recent round of testing that focused on time taken:
The LPNF is faster on smaller games, where the exact time for computation doesn't really matter. The OPNF consistently takes at least 200 ms to run, while the LPNF can finish in under 20 ms. Instead, we look for games where the OPNF takes several seconds to run (at least) or fails while the LPNF is significantly faster.
Games where the OPNF takes at least 5 seconds to build a propnet and the LPNF is significantly faster:
Games where the OPNF times out (120 second limit) or fails and the LPNF does not:
Note that games listed above may be earlier versions of games that have since been modified. (For some of these games, at least, adding bases and inputs makes the OPNF faster.) I need to go through and weed these out. Numbers in square brackets are game IDs from a personal database, for my reference (and so my output code doesn't conflate different versions of a game).
The text was updated successfully, but these errors were encountered: