Skip to content

Conversation

@Mc-andan
Copy link

No description provided.

@FakEEE14
Copy link
Contributor

FakEEE14 commented Oct 4, 2025

it's possible as blitz sayes to add a option like updateitem: bool so it's check for the itemstack, so it's like [refresh] but for single item

@Mc-andan Mc-andan closed this Oct 5, 2025
@BlitzOffline
Copy link
Member

@Mc-andan I wasn't saying it has poor performance, I was asking if it was tested for this. In the past this was the case but maybe with the many improvements we've added to the codebase it could be better now.

@Mc-andan
Copy link
Author

Mc-andan commented Oct 5, 2025

@Mc-andan I wasn't saying it has poor performance, I was asking if it was tested for this. In the past this was the case but maybe with the many improvements we've added to the codebase it could be better now.

I know, but I’m currently on vacation. The code I submitted is running on the server and I haven’t noticed any issues so far, but I haven’t done detailed performance testing yet. I’ll run tests after my vacation, and if there are performance problems, I’ll change the implementation and resubmit.

@Mc-andan Mc-andan reopened this Oct 8, 2025
@Mc-andan
Copy link
Author

Mc-andan commented Oct 8, 2025

@BlitzOffline I've already tested this change, and there are no performance or energy consumption issues. It can be merged.
The plugin has been running on the server for over a week since I submitted the pull request, and no issues have occurred.
image

@Mc-andan Mc-andan requested a review from BlitzOffline October 10, 2025 09:47
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.

4 participants