MantisBT - VCMI
View Issue Details
0001818VCMIMechanics - Adventure Mappublic2014-06-29 15:072022-02-10 11:10
Ivan 
SXX 
lowmajoralways
resolvedfixed 
0.95c 
1.next 
0001818: Extreme slowdown on hero movement after revealing entire map (pathfinder?)
To reproduce:
- Load map "Thousand Islands"
- Move your hero around island (everything OK)
- reveal entire map via vcmieagles
- Try moving hero again. At this moment movement becomes extremely slow consuming all CPU power.
- Board ship at south-east corner of island, using vcminahar if needed
- Hero movement seem so be back to normal

Probably pathfinder does not reacts well to huge water spaces when there is available ship in vicinity.
No tags attached.
? billions of billions islands.h3m (11,524) 2014-12-21 15:39
https://bugs.vcmi.eu/file_download.php?file_id=2038&type=bug
Issue History
2014-06-29 15:07IvanNew Issue
2014-06-29 18:08WarmongerNote Added: 0004810
2014-06-29 18:09WarmongerAssigned To => Warmonger
2014-06-29 18:09WarmongerStatusnew => assigned
2014-12-20 10:52WarmongerNote Added: 0005220
2014-12-20 10:52WarmongerStatusassigned => feedback
2014-12-21 15:39KantorNote Added: 0005246
2014-12-21 15:39KantorFile Added: billions of billions islands.h3m
2015-11-25 07:08SXXNote Added: 0006113
2022-02-10 11:10PovelitelNote Added: 0008056
2022-02-10 11:10PovelitelStatusfeedback => resolved
2022-02-10 11:10PovelitelFixed in Version => 1.next
2022-02-10 11:10PovelitelResolutionopen => fixed
2022-02-10 11:10PovelitelAssigned ToWarmonger => SXX

Notes
(0004810)
Warmonger   
2014-06-29 18:08   
It may be similiar to already fixed bottleneck in adventure AI, will check.
(0005220)
Warmonger   
2014-12-20 10:52   
Is this still an issue?
(0005246)
Kantor   
2014-12-21 15:39   
I've compared both versions(0.95c and 0.97b) and it seems that issue has gone. In VCMI 0.95c moving hero was doing small delays between tiles, and in current version he doesn't. But there still is something like small freeze every time hero is ending his move. That slowdowns are quite well visible on the map which I'm including, just repeat the same steps in description. I'm not sure if it can be considered as a new issue, if yes, I'll create one.
(0006113)
SXX   
2015-11-25 07:08   
Updated pathfinder I just merged should already be much faster as we fixed multiple bottlenecks, but there also work on separate thread:
https://github.com/vcmi/vcmi/pull/138 [^]

If threading implemented pathfinder won't affect client responsiveness even on super-slow devices.
(0008056)
Povelitel   
2022-02-10 11:10   
Another outdated ticket. Not repeated in the current develop