idk what the issue is but do you really need to use yml for this. variables seem like they would be more appropriate in this situation
(you really shouldnt use yml at all https://forums.skunity.com/threads/dont-use-yaml.3850/)

 

I've had the same issues and seen other servers with the same issue, last time I fixed this by just looping all items in all players inventory then checking for names of chest items and removing them from loop-player's inventory, I don't know any other fixes.

 

@Donut {plain-variables} ended up being too slow as that file is now over 1.5mb large! I flattened out each user to a basic set of 5 variables in the .yml file so it's much faster now and avoided any lag problems before using plain global variables.

@BoatXD can you share that inventory check code please? That sounds like a pain to write from scratch! Thanks for your help, this issue is making me very sad for my users.

 

That's not true, YML will never be faster than variables (assuming vars are working correctly)

 

@Pikachu Can you please explain how vars are faster, are they kept in memory always? Why would I see a performance increase switching to a series of tiny YML files vs a huge 1.5MB variables file with tens of thousands of lines? I'm trying to understand what I'm missing, so thanks for helping to clarify!

 

This is exactly how variables work, which is why they'll always be faster than YAML (in Skript, at least).

 

How often does Skript write vars to disk? I was seeing lag issues before I switched to YAML and now these issues are gone, so I'm not sure how to measure variable performance beyond these tests.