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

Target painter not calculated in a DPS graph #137

Closed
cyb34 opened this issue Jun 22, 2014 · 3 comments
Closed

Target painter not calculated in a DPS graph #137

cyb34 opened this issue Jun 22, 2014 · 3 comments

Comments

@cyb34
Copy link

cyb34 commented Jun 22, 2014

Fitting 2x Phased Weapon Navigation Array Generation Extron on a Raven (Arbalest Cruise Launcher I) gives no extra DPS against targets with lower signature radius.

pyfa dps

@cyb34 cyb34 closed this as completed Jun 22, 2014
@cyb34 cyb34 reopened this Jun 22, 2014
@blitzmann
Copy link
Collaborator

Thanks for this. It's been reported by quite a few people in the last few weeks. Looking at the code, it seems that it completely ignores ewar effects on the target and simply calculates turret / missile damage as is.

The problem here is that we supply 4 simple values for the target, but this is NOT considered a fit in pyfa. Since it's not a fit, we cannot do a simple projection of attacker to target and get the values. Therefore, to support EWAR, we have to manually check for TPs and webs and apply them manually to the target. I'm sure this can be added with a bit of thought, tho it is not the best solution.

The best solution would be to overhaul the graphing support. Allow us to pick a fit of aggressor and target (kinda like how EFT does it). That way we can use the already included projection functions of Pyfa to easily do this. But that would require much more time. We can also look into making a blank fit with only the user-defined values for the ship attributes. I have no idea if that would even work or not.

I'll continue to look into this.

@blitzmann
Copy link
Collaborator

I put together something that would include target painters and webs factoring into the user-supplied values.

tpgraph

Graph does not consider TP or web drones. I can add these later if there is demand for them (shouldn't be too difficult), but I'd rather not patch this thing up - complete rewrite would be best option.

@cyb34
Copy link
Author

cyb34 commented Jul 24, 2014

Awesome! Thank you :)

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

No branches or pull requests

2 participants