TheDreadedAndy / SkyrimAEUncapper-Rust

Rust continuation of the Skyrim AE Uncapper.
10 stars 0 forks source link

Dubious skill multiplication. #3

Closed TheDreadedAndy closed 1 year ago

TheDreadedAndy commented 1 year ago

As per this conversation, it seems the skill multiplier may be applied incorrectly. Possibly delaying when the multiplier is applied to until after the offset is retrieved could resolve this, though that may not be in the same section of the code.

TheDreadedAndy commented 1 year ago

It seems moving the skill level increase patch to ID: 41561 + 0xF1 (16 byte size) will allow for this issue to be addressed AND for the improve_skill_by_training patch to be completely removed, as well as the net/og call for improve_player_skill().