4eyebiped lootgen updates #2 #3160
Conversation
|
Requesting review from @harliq and @ziang4891 |
| }; | ||
|
|
||
| public static readonly int[,] TwoHandedWeaponDamageTable = | ||
| { // 0|250|300|325|350|370|400|420|430 | ||
| { 13, 17, 22, 26, 30, 35, 39, 42, 45 }, // Cleaving | ||
| { 14, 19, 23, 28, 33, 37, 42, 45, 48 } // Spears | ||
| { 13, 20, 23, 27, 30, 33, 39, 42, 45 }, // 0 - Cleaving |
There was a problem hiding this comment.
Can you provide the data for these changes? You are creating a pretty big gap from 370 to 400 wield Cleavers. That is uncharacteristic of the pattern that Turbine used. Basically the changes from 250-370 wields are in question.
There was a problem hiding this comment.
The information you see comes from two sources. Mag-Tools, which lacks a lot of data 370 and below, with the lower tiers being almost non existent. The other source is from a recent mining of pcaps done by morosity. We both compared data, discuss a couple of anomalies and sorted through the numbers. The changes look to be the most accurate to date for both EMUs. Feel free to message me on discord about any of this for further info
| new float[] { 2.3f, 2.3f, 2.4f, 2.5f, 2.6f, 2.6f, 2.6f, 2.6f, 2.6f } // Thrown | ||
| { // 0| 250| 270| 290| 315| 335| 360| 375| 385| | ||
| new float[] { 2.1f, 2.2f, 2.3f, 2.4f, 2.4f, 2.4f, 2.4f, 2.4f, 2.4f }, // Bow | ||
| new float[] { 2.4f, 2.5f, 2.55f, 2.65f, 2.65f, 2.65f, 2.65f, 2.65f, 2.65f }, // Crossbow |
There was a problem hiding this comment.
You will have to provide specific data for these changes, as I painstakingly verified this data.
|
I still do not agree with some of the changes for damage. No hard proof was given for those changes. Just data from others that can not be verified. |
To explain further, I first examined the information in Combined_Mag-Tools_Inventory_Loot_Generation.xlsm. I gathered what information was available from there, then I compared it to Morosity’s very recent pcap mining to verify my information and to check for anomalies on both our sides. You mentioned several issues you have with the changes I presented for two handers. All that information is in the magtools/virindi export. That is my hard proof, which I thought we all have access to and were using? I didn’t need further sources for two handers, though I still compared all information against Morosity’s data. The more sources along with multiple people gathering it in a different way, the better. Seeing as he gathered his data separately and through a different resource, it made me feel more confident in my numbers. I will list below the number of examples that can be found in the magtools/virindi export to support my numbers, as I fully understand you dismissing Morosity’s data, since you don’t have direct access to it. I do have a link to the raw PCAPs that he used (Which I assume both projects are using but perhaps he has additional ones), but we would have to mine the data ourselves from it. Which I would happily dig through afterwards as I lack coding skills. 30 examples to support 39 damage 400 wield If I am somehow misunderstanding you on why you do not accept this data, let me know so we can discuss it. If Combined_Mag-Tools_Inventory_Loot_Generation.xlsm is considered inaccurate, let me know. I got this file from either Hells or OptimShi and was told this is what you guys were using. EDIT: The original 35 damage currently listed for 370 wield, there isn't even 1 example of that in the magtool export. |
Includes Hammer wcids from ACEmulator/ACE-World-16PY-Patches#603
From 4eyebiped: "Weapon damage overhaul and other corrections"