Skip to content

add option to check less often for level-ups#2

Open
katepzrp wants to merge 3 commits into
MaxWasUnavailable:masterfrom
katepzrp:master
Open

add option to check less often for level-ups#2
katepzrp wants to merge 3 commits into
MaxWasUnavailable:masterfrom
katepzrp:master

Conversation

@katepzrp

@katepzrp katepzrp commented May 1, 2024

Copy link
Copy Markdown
Contributor

As part of an effort to reduce server load, I'd like to propose this change to make it an option to run the skill limiter less often.

@MaxWasUnavailable

Copy link
Copy Markdown
Owner

Have you profiled this and noticed any significant difference in performance? (Preferably with some actual stats)

The original system already only runs (anything remotely impactful) once every 30 ticks.

@katepzrp

katepzrp commented May 1, 2024

Copy link
Copy Markdown
Contributor Author

Have you profiled this and noticed any significant difference in performance? (Preferably with some actual stats)

The original system already only runs (anything remotely impactful) once every 30 ticks.

Let me get together with our server staff and discuss how we can get you these stats.

@katepzrp

katepzrp commented May 1, 2024

Copy link
Copy Markdown
Contributor Author

Have you profiled this and noticed any significant difference in performance? (Preferably with some actual stats)
The original system already only runs (anything remotely impactful) once every 30 ticks.

Let me get together with our server staff and discuss how we can get you these stats.

We don't have specific statistics, but we have noticed a drop in performance since we added the mod. It wasn't the only mod added at the time, but it was the only one to loop into the EveryTick event, trivially or no.

If you'd like us to test before updating the mod, would you allow us to repack the updated version into our server code? Lack of permission to do so is what's kept us from seeing whether this is the specific issue (without removing the mod entirely, which we'd like to avoid).

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.

2 participants