| 1 |
[quote]a patched CleanOrder() http://pastebin.com/vWALKF6Y to fix it don't remove Queue that overlap over building. You need to re-cleanup this patch.[/quote]
|
1 |
[quote]a patched CleanOrder() http://pastebin.com/vWALKF6Y to fix it don't remove Queue that overlap over building. You need to re-cleanup this patch.[/quote]
|
| 2 |
\n
|
2 |
\n
|
| 3 |
Ah thanks, I accidentally deleted the myQueue[key] = nil lines, probably before I understood what they actually did. I also incorporated your changes for keeping started buildings on the queue. I'll need to test that later to make sure it's working properly.
|
3 |
Ah thanks, I accidentally deleted the myQueue[key] = nil lines, probably before I understood what they actually did. I also incorporated your changes for keeping started buildings on the queue. I'll need to test that later to make sure it's working properly.
|
| 4 |
\n
|
4 |
\n
|
| 5 |
[quote]Airplane don't have moveID and "access a nil index" meant your index is NIL.[/quote]
|
5 |
[quote]Airplane don't have moveID and "access a nil index" meant your index is NIL.[/quote]
|
| 6 |
\n
|
6 |
\n
|
| 7 |
I still don't understand this. AFAIK 'nil' is a valid (although not necessarily sane) value in lua, so moveID = nil should be valid. The error isn't even occurring from the usage of moveID, but rather just from assigning it using UnitDefs[udid].moveDef.id. I already implemented a check for "if (moveID)" in isTargetReachable so that shouldn't cause any problems either, but I haven't been able to get far enough to test that yet.
|
7 |
I still don't understand this. AFAIK 'nil' is a valid (although not necessarily sane) value in lua, so moveID = nil should be valid. The error isn't even occurring from the usage of moveID, but rather just from assigning it using UnitDefs[udid].moveDef.id. I already implemented a check for "if (moveID)" in isTargetReachable so that shouldn't cause any problems either, but I haven't been able to get far enough to test that yet.
|
| 8 |
\n
|
8 |
\n
|
| 9 |
Also I didn't have any air units at the time of testing, unless recom/jumps count. I still don't see any significant differences with the old widget's pathing code either.
|
9 |
Also I didn't have any air units at the time of testing, unless recom/jumps count. I still don't see any significant differences with the old widget's pathing code either.
|
| 10 |
\n
|
10 |
\n
|
|
|
11 |
EDIT: Strike com also causes this error, so it's a problem with the way I'm trying to access the movedef in general, even though there's literally zero difference with the old widget. :| wtf???
|
|
|
12 |
\n
|
| 11 |
[quote]good news!
|
13 |
[quote]good news!
|
| 12 |
I found old widget that demo "AREA COMMAND" that require no complex coding:
|
14 |
I found old widget that demo "AREA COMMAND" that require no complex coding:
|
| 13 |
https://github.com/ZeroK-RTS/Zero-K/blob/4c65c1246586c509f1654584867e378cf0f7ea64/LuaUI/Widgets/cmd_area_mex.lua
|
15 |
https://github.com/ZeroK-RTS/Zero-K/blob/4c65c1246586c509f1654584867e378cf0f7ea64/LuaUI/Widgets/cmd_area_mex.lua
|
| 14 |
\n
|
16 |
\n
|
| 15 |
Command added:
|
17 |
Command added:
|
| 16 |
https://github.com/ZeroK-RTS/Zero-K/blob/4c65c1246586c509f1654584867e378cf0f7ea64/LuaUI/Widgets/cmd_area_mex.lua#L197-L220
|
18 |
https://github.com/ZeroK-RTS/Zero-K/blob/4c65c1246586c509f1654584867e378cf0f7ea64/LuaUI/Widgets/cmd_area_mex.lua#L197-L220
|
| 17 |
\n
|
19 |
\n
|
| 18 |
Command read:
|
20 |
Command read:
|
| 19 |
https://github.com/ZeroK-RTS/Zero-K/blob/4c65c1246586c509f1654584867e378cf0f7ea64/LuaUI/Widgets/cmd_area_mex.lua#L88-L90
|
21 |
https://github.com/ZeroK-RTS/Zero-K/blob/4c65c1246586c509f1654584867e378cf0f7ea64/LuaUI/Widgets/cmd_area_mex.lua#L88-L90
|
| 20 |
\n
|
22 |
\n
|
| 21 |
HOPEFULLY YOU READ THIS BEFORE YOU TRIED THE HARD WAY![/quote]
|
23 |
HOPEFULLY YOU READ THIS BEFORE YOU TRIED THE HARD WAY![/quote]
|
| 22 |
\n
|
24 |
\n
|
| 23 |
Ah that does help. No I haven't started working on implementing that part yet since I want to get the core code actually working properly before I start adding other things.
|
25 |
Ah that does help. No I haven't started working on implementing that part yet since I want to get the core code actually working properly before I start adding other things.
|
| 24 |
\n
|
26 |
\n
|
| 25 |
[quote]I'm looking forward to this new CBAI USrankDudeaeonios, but think twice before blindly giving in to the Global Build Command.[/quote]
|
27 |
[quote]I'm looking forward to this new CBAI USrankDudeaeonios, but think twice before blindly giving in to the Global Build Command.[/quote]
|
| 26 |
\n
|
28 |
\n
|
| 27 |
I don't get it. :P
|
29 |
I don't get it. :P
|
Central Build Widget