New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Tiberian Dawn Nod Campaign 10a #15870

Merged
merged 1 commit into from Dec 27, 2018

Conversation

Projects
None yet
6 participants
@dtluna
Copy link
Contributor

dtluna commented Dec 1, 2018

Add the 10a mission a.k.a. "Terminate Doctor Wong" for the Tiberian Dawn Nod campaign.
_2018-12-01_21-05-16

Playthrough of the original mission

This mission is a little bit different from the original:

  1. the Transport Helicopter comes in when you actually discover the Tech Center
  2. it comes from the side where your forces were originally spawned, not from the upper right as in the original

Important note: there is nothing in the docs on the EnemyWatcher trait and OnDiscover Lua trigger about the AnnounceOnSeen trait being required.

@dtluna dtluna force-pushed the dtluna:feature/nod-campaign-10 branch 2 times, most recently from 6400e59 to a04f38a Dec 1, 2018

@Muh-Muh

This comment has been minimized.

Copy link
Contributor

Muh-Muh commented Dec 2, 2018

Woohoo, more campaign :)

I found a couple of minor things,

(1) the map preview is not aligned correctly.
(2) You can cause a lua exception if you kill the R&D-center before revealing it. (Repro-case: use the visibility cheat and fire with the artilleries from the south; In principle this is of course also possible without cheating, but you would have to remember where to shoot.)
(3) In general I feel like the artilleries are too powerful here. I believe (also I didn't make sure) they had much less range in the original. Point in case: you can kill the R&D basically from the starting location. The helicopter also serves little purpose since your artilleries will do all the killing anyway. I tentatively suggest changing the artillery for another unit with less range. That might make it less true to the original but more fun to play.

@dtluna dtluna force-pushed the dtluna:feature/nod-campaign-10 branch 2 times, most recently from b4edebc to 7a28b7e Dec 9, 2018

@dtluna

This comment has been minimized.

Copy link
Contributor

dtluna commented Dec 11, 2018

@Muh-Muh I replaced artillery with flame tanks, removed the "Locate R&D Center" objective and realigned the map preview. Please review once again :)

@Muh-Muh

This comment has been minimized.

Copy link
Contributor

Muh-Muh commented Dec 11, 2018

@dtluna The image is still aligned differently than all the other maps. I tried to do it myself but mine also looks misaligned ^_°.

I can't get this to crash anymore and as far as I can tell it plays as it should. I have two more suggestions.

(1) Perhaps you could add a lose-condition: all infantry killed before the helicopter arrives. That happened to me while I was playing around.

(2) I feel this mission is still a bit easy. I think on normal and hard (or at least on hard) the player should start with less units. But for the exact balancing it would be best if someone else who knows more about that could chime in

@dtluna

This comment has been minimized.

Copy link
Contributor

dtluna commented Dec 12, 2018

@Muh-Muh This mission was easy in the original campaign as well, to be honest.

@pchote

This comment has been minimized.

Copy link
Member

pchote commented Dec 12, 2018

IMO it would be better to focus on the technical issues or game-breaking balance issues now, but defer general difficulty-level tweaks to a followup - better to merge something that is too easy than to merge nothing at all.

@dtluna dtluna force-pushed the dtluna:feature/nod-campaign-10 branch from 7a28b7e to 0fd0f36 Dec 16, 2018

@obrakmann

This comment has been minimized.

Copy link
Contributor

obrakmann commented Dec 19, 2018

There's a tiling error on this map which leads OpenRA to believe that there's a bridge where there isn't one:

Can you please replace that single tile with a normal desert tile?

@dtluna

This comment has been minimized.

Copy link
Contributor

dtluna commented Dec 21, 2018

@obrakmann Yes, sure

@obrakmann
Copy link
Contributor

obrakmann left a comment

👍 This works well enough for my taste, unless @dtluna wants to use his newly-learned powers to add more to this :)

@dtluna dtluna force-pushed the dtluna:feature/nod-campaign-10 branch 4 times, most recently from 115343c to 356caa6 Dec 23, 2018

@reaperrr

This comment has been minimized.

Copy link
Contributor

reaperrr commented Dec 26, 2018

Works and plays fine, only some really minor things left:

  • One of the northern oil derricks in front of the GDI base is GDI-owned, which looks a bit weird (might have been carried over from original, but there it would have been unnoticable since Neutral used GDI colors). Please either make that one neutral as well, or make all of them GDI.
  • The two(?) minigunners below that oil field are on the same sub-cell.
  • The map image in the selection screen is still a bit horizontally misaligned, too far to the left (vertically it's fine).

@dtluna dtluna force-pushed the dtluna:feature/nod-campaign-10 branch 2 times, most recently from eed29c3 to be7022a Dec 27, 2018

@dtluna

This comment has been minimized.

Copy link
Contributor

dtluna commented Dec 27, 2018

@reaperrr The minigunner and neutral structure issues are definitely fixed and the map preview is probably fixed too (I tried aligning it, but I'm not sure I did it correctly).

@reaperrr reaperrr force-pushed the dtluna:feature/nod-campaign-10 branch from be7022a to cd965b8 Dec 27, 2018

@reaperrr

This comment has been minimized.

Copy link
Contributor

reaperrr commented Dec 27, 2018

The alignment was better, but not perfect yet, and there were some strange gaps in der green borders that the other previews didn't have.

Fixed with GIMP and pushed it over.

@reaperrr reaperrr merged commit 7608922 into OpenRA:bleed Dec 27, 2018

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@reaperrr

This comment has been minimized.

Copy link
Contributor

reaperrr commented Dec 27, 2018

@dtluna dtluna deleted the dtluna:feature/nod-campaign-10 branch Dec 27, 2018

@obrakmann obrakmann referenced this pull request Jan 1, 2019

Open

Add C&C:TD campaign #4988

6 of 8 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment