Make the sea ground height of Elephantine independent from the mapsize.
Needs ReviewPublic
Actions

Authored by phosit on Sun, Sep 17, 12:24 PM.

Details

Reviewers

Freagarach
real_tabasco_sauce

Trac Tickets

#6848

Summary

I increased the number, so that the sea/river is still passable on tiny maps.

Test Plan

Places docks using different map sizes

Diff Detail

Repository

rP 0 A.D. Public Repository

Lint

Lint Skipped

Unit

Unit Tests Skipped

Build Status

Buildable 22390
Build 54780: Vulcan Build	Jenkins
Build 54779: Vulcan Build (macOS)	Jenkins
Build 54778: Vulcan Build (Windows)	Jenkins

Event Timeline

phosit created this revision.Sun, Sep 17, 12:24 PM

Build failure - The Moirai have given mortals hearts that can endure.

Link to build: https://jenkins.wildfiregames.com/job/macos-differential/7297/display/redirect

Harbormaster completed remote builds in B22346: Diff 22269.Sun, Sep 17, 12:56 PM

phosit requested review of this revision.Sun, Sep 17, 12:57 PM

phosit retitled this revision from Make the height of the sea ground independent from the mapsize. to Make the sea ground height of Elephantine independent from the mapsize..Sun, Sep 17, 1:00 PM

sera added a subscriber: sera.Mon, Sep 18, 9:07 AM

sera added inline comments.

binaries/data/mods/public/maps/random/elephantine.js
133–134	Doesn't it seems more natural to make shore fixed height?

According to elexis all heights should be scaled by the map size.
For me this makes sense for hills and mountains but definitly not for waves or shores.
I'm uncertain if the sea ground height should be scaled by the map size. Conceptualy it should be treated like hills but in practice a deep see ground hurts performance and isn't noticed by players.

I'd say a map should try to preserve the feel across the different sizes. That isn't the same as blatantly scaling, in fact constructing counter examples isn't all that hard (eg, think of vertical range of ranged units). Also you already violate this supposed rule with your fix. I'd say a map author should go with the most sensible choice case by case.

I know that i violate this rule

Make the shore also independent.

Build failure - The Moirai have given mortals hearts that can endure.

Link to build: https://jenkins.wildfiregames.com/job/macos-differential/7330/display/redirect

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/8419/display/redirect

Harbormaster completed remote builds in B22390: Diff 22316.Sun, Sep 24, 4:52 PM