Hunters for 2.2.25 and moving forward

Topics: Rawr.Hunter
Developer
Nov 6, 2009 at 10:04 PM

Any Hunters with access to compile the unreleased code for 2.2.25 please download the current revision and test your Character against the model named HunterSE. This will be a major overhaul to the Hunter model for next release and I want to make sure there aren't any serious issues from you guys before I deactivate the current model. In my testing the model remains in line with the other theory-crafting sources (E.g.- A char doing ~3k dps still does ~3k dps in this model) and I don't see any issues myself with the handful of test characters I have been working with.

You have 24 hours to post your major concerns here before I release it. NO CONCERNS = NEW MODEL POSTED and OLD ONE GONE.

The purpose of these changes are to separate Rawr from the Spreadsheets and make them utilize standard features available to Rawr, such as the SpecialEffects methods we have in place for all proc effects. I've also resolved several of the issues posted in the Issue Tracker, such as wrong ArP formula, hit issues, etc. I will be handling maintenance for the model until we get a new full-time Hunter Dev and will be taking some time to get it up to speed with some of the newer features that have come out in the last 5 or so versions.

Editor
Nov 6, 2009 at 10:26 PM

I'll give it a cursory glance-over (as a total non-Hunter) when I get back later tonight, to see if I can find anything glaringly wrong.

Nov 7, 2009 at 7:14 PM

The updates I made to the 2.2.24 code base does not seem to have been carried over to the Hunter SE model. Some probably shouldn't since the trinket procs and similar now uses central functionality as you stated (Banner of Victory and heroic Death's verdict addition).

But there are other updates that I would say would still be valid in the SE model. Most notably is probably the Piercing shot talent. In hunter SE that talent does not give any dps while in Hunter it gives ~800-900 extra dps for a ~T9.5 fully buffed MM hunter (wolf pet).

Not sure what the differences are with the new new module from the old one, and since the spreadsheet no longer can be used to validate things I don't know how to test it.

I currently have a difference between ~8750 dps (Hunter) and ~6500 (Hunter SE) where the piercing talent, as mentioned above, probably can account for 800-900 of the delta. My guess is that the 2pT9 problem could account for some as well.

Developer
Nov 7, 2009 at 7:18 PM

I actually did go thru and try to apply your last 2 patches to HunterSE but like you said, some of it doesn't carry over. Would you be able to write a new patch for me for that talent based on SE? I'll try to look at it while I'm working as well.

Coordinator
Nov 7, 2009 at 7:42 PM
Edited Nov 7, 2009 at 7:43 PM

How quickly can you two get this worked out? Please contact each other directly!

Developer
Nov 7, 2009 at 8:26 PM

I double-checked the Piercing Shots work, you were right it didn't carry over like I had intended. I've added that work and now the talent gives value (on my main dummy character it's givig 191.15 for thre 3rd point so ~573 DPS overall).

Nov 7, 2009 at 8:32 PM

Well, I've identified some differences. In general rows following where the comments are // 29-10-2009 Drizz: Added for Piercing Shots.

For example:

//29-10-2009 Drizz: Added for PiercingShots
            double steadyShotAvgNonCritDamage = steadyShotDamageNormal * steadyShotDamageAdjust*armorReductionDamageAdjust;
            double steadyShotAvgCritDamage = steadyShotAvgNonCritDamage * (1 + steadyShotCritAdjust);
            //021109 Drizz: Have to add the Mangle/Trauma buff effect.
            double steadyShotPiercingShots = (1+ statsBuffs.BonusBleedDamageMultiplier)*(character.HunterTalents.PiercingShots * 0.1)*steadyShotCritChance*steadyShotAvgCritDamage;

            //Drizz: Add the piercingShots effect
            calculatedStats.steadyShot.damage = steadyShotDamageReal + steadyShotPiercingShots;

But the armorReductionDamageAdjust addition have been removed. ( at row 1779 in HunterSE vs 2058 in Hunter) My guess is that the armor reduction is meant to be taken care of somewhere else, not sure where though.

 

Developer
Nov 7, 2009 at 8:36 PM

Drizz, double-check against last commit

Nov 7, 2009 at 8:37 PM

....piercing shot damage need to be added to the Steady Shot, Aimed and Chimera shot section.

 

Nov 7, 2009 at 8:38 PM
Edited Nov 7, 2009 at 8:39 PM

Hehe... you managed to post while I was writing my comment :)

If there are big differences left I will create a ticket so that I can attach the character file I'm currently using.

 

 

Nov 7, 2009 at 8:53 PM
Edited Nov 7, 2009 at 8:54 PM

I created a ticket and attached the character file.

The dps reacts to changes on the talent now... but there must be something else.

Oh. and the Interface on the Optionspanel looks strange. The grouping window cuts of some of the options so they can not be seen.

 

 

Coordinator
Nov 7, 2009 at 10:23 PM

I'm trying to test the latest build, r38904 and it's giving me a Error message when compiling in the Program.cs file where it says "Application.Run(new FormMain());"

"Could not load file or assembly 'Rawr.Baase, Version=2.2.24.0, Culture=neutral, PublicKeyToken=null' or one of it sdependencies. The system cannot find the file specified."

Not sure it this is because of the addition of the HunterSE module or not. Else I would test the module itself.

Granted I'm working with the free version of Microsoft C# 2008 Express Edition so that maybe a factor there.

Developer
Nov 7, 2009 at 10:36 PM

Hina try doing a clean checkout.

Drizz, I've fixed the interface hiding stuff for being too small issue

Nov 7, 2009 at 11:20 PM

Jothay, did you check the 2pT9 bonus? I had to update the name of the buff check for it to react correctly when I made the updates to 2.2.24. (i.e. the check was for the alliance version ... but it needed the Horde version of the gear... or vice versa.) I know I've seen code for "normalizing" those names but I'm not sure if that was only for ending up with one name in the core buff code...

The 2pT9 bonus increases the serpent sting damage (since the serpent sting damage can now be a crit) and one part of Chimera SHot's damage is based on the serpent sting damage.

 

 

Developer
Nov 7, 2009 at 11:44 PM

Serpent Sting had a bad modifier on it (causing it to zero out) for Chimera Shot, I found it and fixed it. Then I saw that I broke Auto-shot but I fixed that and it's within 1% now.

So now the only major difference is Steady Shot, which I know to be a Frequency issue, still diving it to that.

Nov 7, 2009 at 11:50 PM

Man... you are checking in new versions so fast I almost only have the time to download/unpack and rebuild :)

Not sure if you have done it already, but it is a good idea to test with some good gear and close to full raid buffs (could use the char file I posted in the ticket). I ran into the trouble when I tested last time that everything looked fairly ok when I had not added any buffs... then, full buffs... the small errors got amplified.

A possibility to save the buff configuration similar to what you can do with talents would help out testing greatly. But I think there is a ticket somewhere posted about that already.  :)

 

 

Nov 7, 2009 at 11:51 PM

...oh and one more thing. Must say that I'm very happy to see some active dev on the Hunter module. :)

/bow

 

Developer
Nov 8, 2009 at 12:20 AM

I am using your char file except for the settings I have to change so that both can match as closely as possible, and taking off the trinkets (cuz those are no way gonna be the same as before).

Developer
Nov 8, 2009 at 12:35 AM

Last commit should be good to go, see the comparison data I got (total DPS was off by 0.63% though the individual parts more-so)

http://rawr.codeplex.com/WorkItem/View.aspx?WorkItemId=14684