Skip to content

Intercept Wind deflection#5704

Closed
dedmen wants to merge 8 commits into
acemod:masterfrom
dedmen:interceptWindDeflection
Closed

Intercept Wind deflection#5704
dedmen wants to merge 8 commits into
acemod:masterfrom
dedmen:interceptWindDeflection

Conversation

@dedmen
Copy link
Copy Markdown
Contributor

@dedmen dedmen commented Oct 28, 2017

Rewrite of #3222 utilizing the latest Intercept features.

Performance at 200 bullets active
FPS baseline 50fps
Script: 10-12ms per simulation, FPS drop to 35fps
Intercept measured with script: 0.97-1ms per simulation(actually limited by diag_tickTime precision), FPS drops to 48 fps.
Intercept measured by Intercept: 0.474ms per simulation, FPS drops to 45 fps.

The FPS numbers are ofcause not really a reliable measurement.
The Intercept from Intercept measurement ignores the call from SQF->Intercept which should be about 3-20us.

Currently requires Intercept's develop branch because this uses features that I just implemented because they were useful for this :D

As you can see I have kept focus on this being fully optional. There is only one isNil check inside PREP and nothing else was changed to make this possible. The isNil check isn't even needed but otherwise it would log errors about trying to overwrite a compileFinal'ed function.

CMake requires Intercept to be at P:\z\intercept IMO not the nicest solution. But the old Intercept implementation did that and I didn't have a better idea.

Open question. Where put the DLL?
Jonpas and I suggested to put it into intercept folder right in repository main directory. Next to tools/optionals/addons.

Here is a full build including Intercept Core:
InterceptWindDef.zip

@jonpas jonpas added status/experimental kind/enhancement Release Notes: **IMPROVED:** kind/optimization Release Notes: **IMPROVED:** labels Oct 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/enhancement Release Notes: **IMPROVED:** kind/optimization Release Notes: **IMPROVED:** status/experimental

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants