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

Applied size reduction effect to neut/nos #684

Merged
merged 6 commits into from
Jul 24, 2016
Merged

Conversation

Ebag333
Copy link
Contributor

@Ebag333 Ebag333 commented Jul 17, 2016

Relevant code is in the effects file. Basically grabs the ships rig size, and applies the small/medium/large modifiers as a straight up percentage.

Example, ship on the left with the legacy code, ship on the right with the new code.

http://puu.sh/q4qsc/83975220d9.png

Todo:
*Could figure out how CCP handles unrigged ships (freighters, shuttles, pods). This is an edge case, so we may simply choose not to cover it.
*Not sure how CCP determines the actual size, they may use a different formula than rig size.
*Need to handle the sig radius reduction for capital sized neuts. Still not sure on formula.
*Need to verify that stacking behavior works with cap batteries. Assumption here is that the module size reduction applies without a stacking penalty and prior to the cap battery reduction.

@Ebag333
Copy link
Contributor Author

Ebag333 commented Jul 17, 2016

Need to handle the sig radius reduction for capital sized neuts.

This one is tricky, because I know from testing that a capital neut basically does nothing to a frigate. But if we only apply the size reduction, it still instacaps the frigate (according to Pyfa).

@blitzmann if you want to merge this PR, we can always open a new one specific to the carrier neuts, if/when we ever figure out how that works.

@Ebag333
Copy link
Contributor Author

Ebag333 commented Jul 17, 2016

From Reddit:

Signature Radius works as per the bomb damage formula.
https://www.reddit.com/r/evecapitalfocusgroup/comments/489zj7/feedback_wanted_capital_neuts_nos/

And now looking up the bomb sig radius formula, turns up this:

Damage = Base_Damage * MIN(sig/Er,1)
Where
sig = ship's signature
Er = Explosion Radius of bomb

https://oldforums.eveonline.com/?a=topic&threadID=1505798

No idea if this is the correct formula or not but it looks legit.

@Ebag333
Copy link
Contributor Author

Ebag333 commented Jul 17, 2016

The relevant bit of code for the sig reduction:

        energyNeutralizerSignatureResolution = module.getModifiedItemAttr("energyNeutralizerSignatureResolution")
        signatureRadius = fit.ship.getModifiedItemAttr("signatureRadius")

        #Signature reduction, uses the bomb formula as per CCP Larrikin
        if energyNeutralizerSignatureResolution:
            sigRatio = signatureRadius/energyNeutralizerSignatureResolution

            sigReductionList = [1, sigRatio]
            amount = amount*min(sigReductionList)

So a couple things on this.

  1. I'm not sure that the is the actual formula used. (See post above)
  2. I'm not sure how this stacks. Testing in game seems to show it does both (for example there is virtually no effect against frigates )

@Ebag333 Ebag333 mentioned this pull request Jul 18, 2016
6 tasks
@blitzmann
Copy link
Collaborator

This all looks pretty good to me. I will wait until I can sit down and test things myself on SISI for expected values before merging. Great work @Ebag333!

@Ebag333
Copy link
Contributor Author

Ebag333 commented Jul 18, 2016

Confirmed by sources that the sig size is correct.

@blitzmann blitzmann merged commit 7a715fc into pyfa-org:master Jul 24, 2016
@Ebag333 Ebag333 deleted the Neuts branch July 25, 2016 01:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants