added mission nod4a

[42foobar42]

After a break, here we go again.

I think there are a few points that can be discussed on this mission.

So happy testing and I am waiting for your comments.

[Mailaender]

This should make use of the new Lua API function from #7463.

[Mailaender]

The building husks don't hide the owner row:

[Mailaender]

Winning and Loosing scenario seem to work properly.

[42foobar42]

I keep in mind to change the the discover trigger as soon as it is merged.

Winning scenario is killing all civilains in the north west village. It does not matter if you killed all GDI infantry.

Loosing scenrario is if you loose all your own infantry or all civilians in the south east village are killed.

[Mailaender]

#7492 has been merged.

[penev92]

I noted this on IRC at the time, but I was really close to asking about progress here because I had forgotten myself: the Has dependencies label was for #7463, not #7492. (or maybe both; point is #7463 isn't merged, so the label should come back).
Anyway, have you made any progress on this, @42foobar42 ?

[obrakmann]

Typo: "[...] that support Nod."

[obrakmann]

Gameplay-wise this is fine, minus the mentioned nits. Knowing our audience, it's probably a bit hard. I don't have any good idea how to implement difficulty levels for it though, given how the mission works.

I'll look at the code when this has been rebased against #7463.

[obrakmann]

Oh, I almost forgot to mention: there's this graphical glitch, although that's most likely our fault:

[42foobar42]

Hi there,
I fixed the mention points.
But the trigger OnPlayerDiscovered did not work. There is no error or something. I could not figure out what the problem is. I try to do it the same way like gdi mission 5. Obviously I overlook something.

[abcdefg30]

Could you please remove those Health: 1 values here and below?

[abcdefg30]

Did you add EnemyWatcher: and AnnounceOnSeen:?

[Mailaender]

Oh, I almost forgot to mention: there's this graphical glitch, although that's most likely our fault:

This can be fixed by re-ordering the actors in the map.yaml listing.

[42foobar42]

Thanks @abcdefg30 ! EnemyWatcher: and AnnounceOnSeen fixed the problem.
The other points are also fixed.

[abcdefg30]

Note: This is ok for now, but should be removed once #7984 is merged (or the other way round).

[penev92]

To summarize: this has dependencies on #7463 and #7984. Sorry @42foobar42

[abcdefg30]

#7984 is merged now.

[42foobar42]

I removed the NotificationInterval, but I think it does not work as expected.

When my units arrive the village the trigger does not work.

[abcdefg30]

When my units arrive the village the trigger does not work.

Indeed it does, but it takes a while. To fix this,
you probably want add ScanInterval: 10 below this line (or maybe even a minor number).

And: Unintentional tab after the :.

[penev92]

What is the problem here? The default rescan interval should be ~1 second, which is a compromise between discovery speed and performance, but generally should be good enough for this.

[abcdefg30]

Hm, yeah it is really not the rescan interval. I guess I'll send a patch myself, and clean some other things up as well.

[abcdefg30]

This unit will stay at its spawn. Would be nice if you could do something agianst that.

[penev92]

#7463 has been merged and I believe there are no more dependencies now.

[abcdefg30]

I think, I fixed everything in 42foobar42#4.

[42foobar42]

Thanks @abcdefg30 . I merged your request.

Looks good to me now.

[abcdefg30]

Damn it. I'm that silly.
We need an IsDead check here.

Utils.Do(GcivActors, function(actor)
    if not actor.IsDead then
        actor.AttackMove(waypoint7.Location)
        actor.AttackMove(waypoint8.Location)
        IdleHunt(actor)
    end
end)

[abcdefg30]

Otherwise 👍 / ✅

[42foobar42]

no problem. I fixed it.

[Mailaender]

Works fine (I won, jay). ✅

[Mailaender]

Thanks!

[Mailaender]

Changelog