[DiscordArchive] Do you have an example of somewhere that's doing it correctly?
[DiscordArchive] Do you have an example of somewhere that's doing it correctly?
Archived author: Deleted User • Posted: 2024-07-22T19:32:33.787000+00:00
Original source
Do you have an example of somewhere that's doing it correctly?
Archived author: jackpoz • Posted: 2024-07-22T19:35:50.781000+00:00
Original source
oh more like https://github.com/TrinityCore/TrinityCo...r.cpp#L689 and https://github.com/TrinityCore/TrinityCo...#L658-L667
[Embed: TrinityCore/src/server/game/Movement/PathGenerator.cpp at c22d1ce93...]
TrinityCore Open Source MMO Framework (master = 10.2.7.55664, 3.3.5 = 3.3.5a.12340, wotlk_classic = 3.4.3.54261, cata classic = 4.4.0.55262) - TrinityCore/TrinityCore
https://github.com/TrinityCore/TrinityCo...erator.cpp
Archived author: jackpoz • Posted: 2024-07-22T19:36:43.261000+00:00
Original source
they check 2 different datasets
Archived author: jackpoz • Posted: 2024-07-22T19:50:07.373000+00:00
Original source
and this is wrong, because it checks maps/vmaps
Archived author: jackpoz • Posted: 2024-07-22T19:50:17.577000+00:00
Original source
but path finding uses mmaps
Archived author: jackpoz • Posted: 2024-07-22T19:50:25.714000+00:00
Original source
and that determines if something is accessible or not
Archived author: jackpoz • Posted: 2024-07-22T20:04:54.867000+00:00
Original source
this comes back to the fact that we use detour wrong: we don't store the current poly of each unit but we look it up every time we need a path
Archived author: jackpoz • Posted: 2024-07-22T20:05:27.710000+00:00
Original source
if we stored the poly, we could just check if the poly is ground or water (or any other allowed/disallowed type)
Archived author: Deleted User • Posted: 2024-07-22T20:27:41.341000+00:00
Original source
Hmm wouldn't that be quite expensive to look up on update? Assuming you'd need to for players too.
Archived author: jackpoz • Posted: 2024-07-22T20:48:36.396000+00:00
Original source
if we would track the poly, then the lookup would be instant as we have it already