Possible Bug?

[Mihhaelo]

The game always ends up crashing after about 30 seconds with the code at the bottom. Changing the Owner of the Asteroid to the computer player prevents the bug (With no tree's or tree-slots). Any idea's of how to prevent the bug? The map i'm working on requires enemy seedlings to be spawned onto player asteroids.

(click to show/hide)

function LevelLogic()
    while GameRunning() do
   
       Timer = GetGameTime()
                                coroutine.yield()
      
      if LoopTime < Timer then
          GetAsteroid(0):AddSeedlings(1,2,1,1,1)
          LoopTime = LoopTime + 1
       end

    end
end

[Pilchard123]

need to move the coroutine.yield() to the end of the while loop, like this: (I think)

Code: [Select]

function LevelLogic()
     while GameRunning() do
          Timer = GetGameTime()                 
          if LoopTime < Timer then
               GetAsteroid(0):AddSeedlings(1,2,1,1,1)
               LoopTime = LoopTime + 1
          end
     coroutine.yield()
     end
end

[annikk.exe]

Pilchard is correct.  :>

[Mihhaelo]

Naw, still doesn't work, it's still doing what it done last night. Thanks for the suggestion though.

The map runs for about 30 seconds and then crashes. I think the AI code might be trying to make the seedlings do something they can't actually do?

[Pilchard123]

I think I know the problem...just checking if I'm right.

[Pilchard123]

Could you post the whole level code, or PM me with it? I won't leak it, I promise! I think I've worked it out, but need to see the whole level to be certain.

[Mihhaelo]

That's the whole of the example level i made to show the bug.

(click to show/hide)

function LevelSetup()

   -- Set Global Values
   Globals.G.Asteroids=(0)
   Globals.G.EnemyFactionsMin=(0)
   Globals.G.EnemyFactionsMax=(0)


   -- Asteroid 0 - starting asteroid
   a = AddAsteroidWithAttribs(0,0, 0.7,0.6,0.5)
   a.Owner = 1
   a.TreeCap = 1
   a:SetRadius(250)
   a.SendDistance = 2500

   a:AddSeedlings(90,1,1,1,1)

LoopTime = 0
   
end



function LevelLogic()
     while GameRunning() do
          Timer = GetGameTime()                 
          if LoopTime < Timer then
               GetAsteroid(0):AddSeedlings(1,2,1,1,1)
               LoopTime = LoopTime + 1
          end
     coroutine.yield()
     end
end

[Mihhaelo]

You can also induce the bug, by planting a tree onto a planet that does not belong to you. You can use the following code to explore that.

(click to show/hide)

function LevelSetup()

   -- Set Global Values
   Globals.G.Asteroids=(0)
   Globals.G.EnemyFactionsMin=(1)
   Globals.G.EnemyFactionsMax=(1)
   Globals.AI.GraceTimer = (9999)


   -- Asteroid 0 - starting asteroid
   a = AddAsteroidWithAttribs(0,0, 0.7,0.6,0.5)
   a.Owner = 1
   a.TreeCap = 1
   a:SetRadius(250)
   a.SendDistance = 2500
   
   a:AddSeedlings(90)
   
    a = a:AddDysonTree()
    a:LevelUp()
    a:LevelUp()
    a:LevelUp()
   
   -- Asteroid 1
   a = AddAsteroidWithAttribs(2000,0, 0.3,0.3,0.3)
   a.Owner = 2
   a.TreeCap = 2
   a:SetRadius(450)
   a.SendDistance = 2500
   a:Reveal(1)
   
   a:AddSeedlings(100)
   
end



function LevelLogic()

end

[Alex]

Thanks for this, I've fixed a bug in the army code when the enemy AI tries to organise an army on one of its zero owned asteroids.

I will look into the new one you just posted. edit: Ah it's probably the same thing - after your tree takes over the asteroid the enemy has no asteroids left but still tries to set up an army onto one.

BTW your example above is great for getting one of the new achievements

[Pilchard123]

(click to show/hide)

Code: [Select]

function LevelSetup()

   -- Set Global Values
   Globals.G.Asteroids=(0)
   Globals.G.EnemyFactionsMin=(0)
   Globals.G.EnemyFactionsMax=(0)


   -- Asteroid 0 - starting asteroid
   a = AddAsteroidWithAttribs(0,0, 0.7,0.6,0.5)
   a.Owner = 1
   a.TreeCap = 1
   a:SetRadius(250)
   a.SendDistance = 2500

   a:AddSeedlings(90,1,1,1,1)

LoopTime = 0
  
end



function LevelLogic()
     while GameRunning() do
          Timer = GetGameTime()                
          if LoopTime < Timer then
               GetAsteroid(0):AddSeedlings(1,2,1,1,1)
               LoopTime = LoopTime + 1
          end
     coroutine.yield()
     end
end

Won't work. There isn't even a second empire, so that may be the problem.

EDIT: The second one gives me no trouble at all. Are you sure it causes the bug?

[Alex]

Working fine here. Seems solved.

[annikk.exe]

I always put my coroutine.yield() right before the "end" statement that closes off the While loop.
Would suggest that as a best practice, rather than slipping it in haphazardly somewhere inside the loop.