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

Update main.lua #5

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update main.lua #5

wants to merge 1 commit into from

Conversation

ownlyme
Copy link

@ownlyme ownlyme commented Apr 13, 2022

  • Caching the changed skill pre-event to increase the experience post-event.
    When doing it pre-event, the client adds the default spellcast experience afterwards, which is not saved on the server until GetSkillProgress is called, thus increasing the delta experience and increasing exponentially with each subsequent spellcast
  • Added a treshold to prevent rounding errors at the skillchange detection
  • Removed the ceil from the bonus experience, which gave a lot more bonus experience than intended

- Caching the changed skill pre-event to increase the experience post-event.
When doing it pre-event, the client adds the default spellcast experience afterwards, which is not saved on the server until GetSkillProgress is called, thus increasing the delta experience and increasing exponentially with each subsequent spellcast
- Added a treshold to prevent rounding errors at the skillchange detection
- Removed the ceil from the bonus experience, which gave a lot more bonus experience than intended
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.

None yet

1 participant