|
Notes |
|
|
(0004319)
|
|
Ivan
|
|
2014-01-12 20:33
|
|
It looks that this bug may lead to interesting results: http://forum.vcmi.eu/viewtopic.php?p=11332#11332 [^]
In previous mission Gem had art that gives +1 to scouting radius. And it looks that artifacts got transferred but only on server side (server assumes that that row of tiles is visible for player and haven't included them in FoW netpack). |
|
|
|
(0004395)
|
|
Tow
|
|
2014-02-02 14:34
|
|
Desync, that's serious issue. :(
Do you have some easy way to reproduce this issue? |
|
|
|
(0004396)
|
|
Ivan
|
|
2014-02-02 14:54
|
|
After a few restarts I got nice artifact in vicinity - everflowing crystal cloak. Check attached saves:
a) gemtestbefore - made just before picking up artifact. Load game, pick art, vcmisilmaril, end turn
b) gemtestafter - already in next scenario, just end turn and you'll get +1 to crystals despite that Gem no longer have the cloak.
Another problem is that in these missions artifacts should not transfer to begin with. So this desync may not be directly connected to this bug. |
|
|
|
(0004397)
|
|
Tow
|
|
2014-02-02 17:52
|
|
Thanks, that helps.
Interestingly, I can't reproduce the issue using gemtestafter savegame. Whatever it is, it doesn't survive save/load.
Otherwise, using the before savegame, issue is perfectly reproducible. I'll investigate it. |
|
|
|
(0004398)
|
|
Tow
|
|
2014-02-02 17:59
|
|
|
The artifact is not transferred. The issue is that bonus tree dependencies are transferred. |
|
|
|
(0004402)
|
|
Tow
|
|
2014-02-02 21:46
|
|
I see two issues:
1) The CGameState::prepareCrossoverHeroes function does not properly remove things from hero (it goes around the bonus system). That's easy to fix.
2) Desync. What really puzzles me is why the hero on client doesn't have old stacks/artifacts from the beginning, while on server they are removed during init. That should not happen.
I'll leave 1) until I investigate properly 2). |
|
|
|
(0004414)
|
|
Tow
|
|
2014-02-08 16:49
|
|
The source of this desync (and likely the others) is that server, when entering the next campaign mission goes like this:
1. newGame -> gh::init -> gs::init -> placeCampaignHeroes -> prepareCrossoverHeroes <= modifies the heroes that are part of start info
2. send start info (that has been already modified) all the clients, so they can perform their inits
The start info fed to the clients and server is different and that results in desync. I have even found TODO note in "prepareCrossoverHeroes" saying that heroes needs to be deep copied. |
|
|
|
(0004430)
|
|
Tow
|
|
2014-02-08 23:08
|
|
|
Fixed all the issues in r3745. |
|
|
|
(0004741)
|
|
Kantor
|
|
2014-06-02 11:28
|
|
|
gemtest.zip (1,151,387) 2014-02-02 14:55