Page MenuHomeWildfire Games

River Archipelago random map script
ClosedPublic

Authored by elexis on May 9 2017, 4:14 PM.

Details

Summary

A unique map composing of strips of land.
Needs lots of optimizing but @elexis wanted it.

Original design by Hannibal_Barca:

Updated design by elexis, 50% chance for each:


Test Plan

Test map for uniqueness, playability, resource collisions and the like.
Paint extra terrain under Civic Centers to ensure they won't be underwater.
Base resources same problem.

Event Timeline

Hannibal_Barca created this revision.May 9 2017, 4:14 PM
Vulcan added a subscriber: Vulcan.May 9 2017, 8:56 PM

Build is green

Updating workspaces.
Build (release)...
Build (debug)...
Running release tests...
Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!
Running debug tests...
Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!

http://jw:8080/job/phabricator/1106/ for more details.

elexis commandeered this revision.May 17 2017, 4:22 AM
elexis edited reviewers, added: Hannibal_Barca; removed: elexis.
elexis edited edge metadata.

River island archipelago is the correct geomorphological type I guess.
Wrong continent, but the only fitting location I could find: https://en.wikipedia.org/wiki/Anavilhanas_National_Park

elexis updated this revision to Diff 1992.May 17 2017, 4:25 AM

Fix various brokenness and most style issues.
Ensure terrain on player territory.
Currently no trees in the middle strips.

elexis planned changes to this revision.May 17 2017, 4:25 AM

Build is green

Updating workspaces.
Build (release)...
Build (debug)...
Running release tests...
Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!
Running debug tests...
Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!

http://jw:8080/job/phabricator/1234/ for more details.

Build is green

Updating workspaces.
Build (release)...
Build (debug)...
Running release tests...
Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!
Running debug tests...
Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!

http://jw:8080/job/phabricator/1235/ for more details.

elexis updated this revision to Diff 2105.May 22 2017, 2:45 AM

Refactor strip code, especially mirror the strip size and position.
Smaller strips, thus more space for ship navigation but much smaller likelihood of having enemies connected.
Fix ugly grass patches underwater. Remove unpaint calls.
Distinguish clPlayer from the new clPlayerTerritory class, especially to add forests close to the territory border.
createBumps.
Much more fish.
Make mountains actually appear.
Fix city radius painting.
Fix few more style mistakes and resource collisions.
Tested with tiny mapsize.

elexis updated this revision to Diff 2106.May 22 2017, 2:48 AM

Rename to River Archipelago.
Also had added gaia.

Extended map description welcome @Hannibal_Barca

Executing section Default...
Executing section Source...
Executing section JS...
|    | [NORMAL] ESLintBear (no-multi-spaces):
|    | Multiple spaces found before '{'.
|----|    | /mnt/data/jenkins-phabricator/workspace/phabricator_lint/binaries/data/mods/public/maps/random/strips.js
|    |++++| /mnt/data/jenkins-phabricator/workspace/phabricator_lint/binaries/data/mods/public/maps/random/strips.js
| 143| 143| 		new LayeredPainter([tRoadWild, tRoad], [1]),
| 144| 144| 		null);
| 145| 145| 
| 146|    |-	placeCivDefaultEntities(fx, fz, id,  { 'iberWall': 'towers' });
|    | 146|+	placeCivDefaultEntities(fx, fz, id, { 'iberWall': 'towers' });
| 147| 147| 
| 148| 148| 	placeDefaultChicken(fx, fz, clBaseResource, undefined, oPeacock);
| 149| 149| 

binaries/data/mods/public/maps/random/strips.js
| 204| »   let·stripIndex·=·i·==·0·?·0·:·stripWidths.length·-·1;
|    | [NORMAL] JSHintBear:
|    | Use '===' to compare with '0'.
Executing section Python...
Executing section Perl...

http://jw:8080/job/phabricator_lint/8/ for more details.

elexis retitled this revision from New map - Strips to River Archipelago random map script.May 22 2017, 4:16 AM
elexis edited the summary of this revision. (Show Details)
Executing section Default...
Executing section Source...
Executing section JS...
|    | [NORMAL] ESLintBear (no-multi-spaces):
|    | Multiple spaces found before '{'.
|----|    | /mnt/data/jenkins-phabricator/workspace/phabricator_lint/binaries/data/mods/public/maps/random/river_archipelago.js
|    |++++| /mnt/data/jenkins-phabricator/workspace/phabricator_lint/binaries/data/mods/public/maps/random/river_archipelago.js
| 143| 143| 		new LayeredPainter([tRoadWild, tRoad], [1]),
| 144| 144| 		null);
| 145| 145| 
| 146|    |-	placeCivDefaultEntities(fx, fz, id,  { 'iberWall': 'towers' });
|    | 146|+	placeCivDefaultEntities(fx, fz, id, { 'iberWall': 'towers' });
| 147| 147| 
| 148| 148| 	placeDefaultChicken(fx, fz, clBaseResource, undefined, oPeacock);
| 149| 149| 

binaries/data/mods/public/maps/random/river_archipelago.js
| 204| »   let·stripIndex·=·i·==·0·?·0·:·stripWidths.length·-·1;
|    | [NORMAL] JSHintBear:
|    | Use '===' to compare with '0'.
Executing section Python...
Executing section Perl...

http://jw:8080/job/phabricator_lint/9/ for more details.

In D467#21587, @elexis wrote:

Rename to River Archipelago.
Also had added gaia.

Extended map description welcome @Hannibal_Barca

I have thought also to something like kerala backwaters, which fits the biome.

Build is green

Updating workspaces.
Build (release)...
Build (debug)...
Running release tests...
Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!
Running debug tests...
Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!

http://jw:8080/job/phabricator/1300/ for more details.

Build is green

Updating workspaces.
Build (release)...
Build (debug)...
Running release tests...
Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!
Running debug tests...
Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!

http://jw:8080/job/phabricator/1301/ for more details.

Hannibal_Barca accepted this revision.May 22 2017, 6:26 PM

Thank you for fixing map up, works and looks very nice.

This revision is now accepted and ready to land.May 22 2017, 6:26 PM
Phormio commandeered this revision.May 23 2017, 1:53 PM
Phormio added a reviewer: elexis.
Phormio added a subscriber: Phormio.
  • improving map description
Phormio updated this revision to Diff 2149.May 23 2017, 1:54 PM
  • improved map description
Owners added a subscriber: Restricted Owners Package.May 23 2017, 1:54 PM
Phormio added inline comments.May 23 2017, 1:58 PM
binaries/data/mods/public/maps/random/river_archipelago.json
5 ↗(On Diff #2149)

This is the modified part, I left the rest as it was.

elexis commandeered this revision.May 23 2017, 2:07 PM
elexis edited reviewers, added: Phormio; removed: elexis.
Phormio added inline comments.May 23 2017, 2:34 PM
binaries/data/mods/public/maps/random/river_archipelago.json
5 ↗(On Diff #2149)

Let's change that to '[...] plenty of game, but beware [...]'

Phormio requested changes to this revision.May 23 2017, 3:09 PM

Replace "plenty of game and fish" with "plenty of game", since fish is included in game (https://en.wikipedia.org/wiki/Game_fish), then I'm happy with the map.

This revision now requires changes to proceed.May 23 2017, 3:09 PM

Build is green

Updating workspaces.
Build (release)...
Build (debug)...
Running release tests...
Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!
Running debug tests...
Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!

http://jw:8080/job/phabricator/1325/ for more details.

Executing section Default...
Executing section Source...
Executing section JS...
|    | [NORMAL] ESLintBear (no-multi-spaces):
|    | Multiple spaces found before '{'.
|----|    | /mnt/data/jenkins-phabricator/workspace/phabricator_lint/binaries/data/mods/public/maps/random/river_archipelago.js
|    |++++| /mnt/data/jenkins-phabricator/workspace/phabricator_lint/binaries/data/mods/public/maps/random/river_archipelago.js
| 143| 143| 		new LayeredPainter([tRoadWild, tRoad], [1]),
| 144| 144| 		null);
| 145| 145| 
| 146|    |-	placeCivDefaultEntities(fx, fz, id,  { 'iberWall': 'towers' });
|    | 146|+	placeCivDefaultEntities(fx, fz, id, { 'iberWall': 'towers' });
| 147| 147| 
| 148| 148| 	placeDefaultChicken(fx, fz, clBaseResource, undefined, oPeacock);
| 149| 149| 

binaries/data/mods/public/maps/random/river_archipelago.js
| 204| »   let·stripIndex·=·i·==·0·?·0·:·stripWidths.length·-·1;
|    | [NORMAL] JSHintBear:
|    | Use '===' to compare with '0'.
Executing section Python...
Executing section Perl...

http://jw:8080/job/phabricator_lint/18/ for more details.

This revision was automatically updated to reflect the committed changes.

Nice, thank you very much!

elexis edited the summary of this revision. (Show Details)May 24 2017, 1:44 PM