Skip to content
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 money crates to GDI08a and GDI09 #17881

Merged
merged 5 commits into from Apr 17, 2020
Merged

Add money crates to GDI08a and GDI09 #17881

merged 5 commits into from Apr 17, 2020

Conversation

abcdefg30
Copy link
Member

Closes #17877.

@abcdefg30 abcdefg30 added this to the Next Release milestone Mar 31, 2020
@pchote
Copy link
Member

pchote commented Mar 31, 2020

It looks like ImportTiberianDawnLegacyMapCommand is missing crates from its list of overlays to convert. Can we please fix that while we're here, and someone (I can do it if you don't have an easy way to check) grep the original mission inis to check for missing crates in the other missions?

@abcdefg30
Copy link
Member Author

Updated. I still need to find the TD mission ini's so I didn't check all missions so far, but I think at least nod07 is missing a crate.

@fruestueck
Copy link
Contributor

fruestueck commented Apr 4, 2020

scg08ea.zip INI

...
[OVERLAY]
...
2693=WCRATE

scg09ea.zip INI
Seach for *crate* without luck.

@pchote
Copy link
Member

pchote commented Apr 9, 2020

We have

paul@Ragnarok:missions$ grep CRATE *.ini
scb06ea.ini:2811=SCRATE
scb06eb.ini:1775=SCRATE
scb06ec.ini:1432=SCRATE
scb07ea.ini:3086=WCRATE
scb08ea.ini:807=SCRATE
scb08eb.ini:3480=SCRATE
scb09ea.ini:516=WCRATE
scb11ea.ini:3471=WCRATE
scb11eb.ini:3403=WCRATE
scb12ea.ini:301=SCRATE
scb13ec.ini:1222=WCRATE
scg04ea.ini:3737=SCRATE
scg04ea.ini:1528=WCRATE
scg04wa.ini:846=SCRATE
scg05wa.ini:2604=WCRATE
scg05wb.ini:2714=WCRATE
scg08ea.ini:2693=WCRATE
scg09ea.ini:3077=WCRATE
scg12ea.ini:2551=WCRATE
scg12eb.ini:1179=WCRATE

So for the missions we have ported, that is two of the three gdi04's (not sure which ones our a and b map to), both gdi05's, gdi09 and 08 (already added in this PR), all three nod06's, one of the nod07s, both nod08s, and nod09.

The map cell can be calculated by taking the number (e.g. 2811 for scb06ea), doing an integer division by 64 to get the y coordinate, and then taking the remainder (or taking the original number modulo 64) to get the x coordinate.

@reaperrr
Copy link
Contributor

So for the missions we have ported, that is two of the three gdi04's (not sure which ones our a and b map to), both gdi05's, gdi09 and 08 (already added in this PR), all three nod06's, one of the nod07s, both nod08s, and nod09.

Just to make sure this doesn't cause any confusion, only the WCRATEs are money crates, the SCRATEs are usually what you need to collect as mission objective.

@abcdefg30
Copy link
Member Author

Added all the remaining crates. Two notes:

  • The crate of gdi04a is outside of the map. You can't build anything in this mission so I left the crate there as 'easteregg'.
  • The crate of nod09 is supposed to spawn when the church is destroyed, but our civilian buildings leave husks when destroyed and there is no nearby space available to spawn the crate, so I placed it in the shadow of the tree instead.
    grafik

@abcdefg30
Copy link
Member Author

One more thing maybe. That is the original location the money crate of nod07a spawns at when the refinery is destroyed:
grafik
I chose to not offset the location to be where the building explodes, as the AI will otherwise build over the crate. (This is not an issue with the original location as seen in the screenshot.)

@reaperrr reaperrr merged commit 942dd0e into OpenRA:bleed Apr 17, 2020
@abcdefg30 abcdefg30 deleted the tdMoney branch April 17, 2020 08:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TD GDI missions 8 and 9 missing money crates
4 participants