Skip to content
This repository has been archived by the owner on Jun 19, 2021. It is now read-only.

MapPalette lag #468

Open
luizsusin opened this issue Apr 21, 2021 · 5 comments
Open

MapPalette lag #468

luizsusin opened this issue Apr 21, 2021 · 5 comments

Comments

@luizsusin
Copy link

Expected behavior
Plugin work flawlessly.

Observed behavior
Huge freeze.

Steps to reproduce (if known)
Load the plugin. After that, it works normally.
Random freezes can also be seen.

Installed plugins
Plugins (65): AdvancedNMotd, AsyncWorldEdit, AsyncWorldEditBossBar, AuthMe, AutoMessage, BeastWithdraw, BetterShop, BetterSpawners, BetterStackMob, BlockLocker, BlockVersion*, ChestSort, CommandHook, ConsoleSpamFix, CreativeCrates, CustomHeads*, eco, EcoEnchants, EntityDetection, EntityTrackerFixer, Essentials, EssentialsSpawn, FastLogin, GoldenCrates, GriefPrevention, GriefPreventionFlags, GSit, HolographicDisplays, ImageOnMap, IronElevators*, LuckPerms, MarriageMaster, MaxBans, mcMMO, MegaArena, Minepacks, Multiverse-Core, MyPet, NexEngine, NoNetherRoof, PlaceholderAPI, PlugMan, PlusProtection, PowerVIP, ProtocolLib, ProtocolSupport, PvPManager, RevampClans, SamAliases, SimplePortals, SkinsRestorer, sleep-most, spark, SusinEvents*, TAB, Vault, VentureChat, VillagerOptimiser, Vote*, Votifier, Webhooker, Wild, WorldEdit, WorldGuard, zEconomy

Version
Yatopia version git-Yatopia-ver/1.16.5-81 (MC: 1.16.5) (Implementing API version 1.16.5-R0.1-SNAPSHOT)

Additional context
This might have to do with PR #308, although it's WIP, I'm not sure if it was implemented afterall.
Thread dump can be seen here: https://paste.ubuntu.com/p/fDswhkS9X6/

@Titaniumtown
Copy link
Collaborator

This is due to having too many or too big of maps.

@luizsusin
Copy link
Author

That's the point: We don't let our players use it, so they can't create it. The maps are majorly 1x1, we probably don't have more than 30 or 40 1x1 and a few 4x4 (maybe 3 or 4).

@metabrixkt
Copy link
Contributor

metabrixkt commented Apr 21, 2021

Not sure if it's Yatopia bug. Bukkit's MapPalette implementation is very laggy, it can cause tps drop.

edit: accidentally closed, sorry

@metabrixkt metabrixkt reopened this Apr 21, 2021
@Titaniumtown Titaniumtown changed the title ImageOnMap Incompatibility MapPalette lag May 6, 2021
@luizsusin
Copy link
Author

I won't close this issue although now I'm sure it isn't a Yatopia bug, this is just an update on the situation.
I've rewritten the ImageOnMap render code using BKCommonLib's MapColorPalette. The result was a massive improvement in performance.
I hope @epserv can get his PR to work.

@toinouH
Copy link
Contributor

toinouH commented May 25, 2021

Thank you for your comment. MrBoom took note of this.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants