[gameplay] Move ptolemaic slinger to phase one.
ClosedPublic
Actions

Authored by Nescio on Jan 16 2021, 2:43 AM.

Details

Reviewers

borg-
wraitii

Group Reviewers

Balancing

Commits

rP24649: Swap Ptolemaic Slinger & Javelineer

Summary

This patch is a request by @Stockfish0ad.
D3269/rP24635 reduced mercenary total cost, but increased their metal cost and prevented them from gathering resources. Because the Ptolemies have a mercenary javelineer in the village phase, they're at a disadvantage. Therefore their slinger becomes their village-phase ranged citizen infantry instead.

Test Plan

Check for mistakes.

Diff Detail

Repository

rP 0 A.D. Public Repository

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

Event Timeline

borg- created this revision.Jan 16 2021, 2:43 AM

Herald added a reviewer: Balancing. · View Herald TranscriptJan 16 2021, 2:43 AM

Build has FAILED

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/4547/display/redirect
See console output for more information: https://jenkins.wildfiregames.com/job/docker-differential/4547/display/redirectconsole

Build has FAILED

Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/3976/display/redirect
See console output for more information: https://jenkins.wildfiregames.com/job/vs2015-differential/3976/display/redirectconsole

borg- requested review of this revision.Jan 16 2021, 2:45 AM

borg- updated this revision to Diff 15368.

borg- removed rP 0 A.D. Public Repository as the repository for this revision.

Build has FAILED

Link to build: https://jenkins.wildfiregames.com/job/macos-differential/2888/display/redirect
See console output for more information: https://jenkins.wildfiregames.com/job/macos-differential/2888/display/redirectconsole

Harbormaster completed remote builds in B15096: Diff 15367.Jan 16 2021, 2:45 AM

This _could_ be the wrong patch ;)

Can someone please update this patch? I'm far from home for two days.

At the request of @borg-.

do the right thing

Owners added a subscriber: Restricted Owners Package.Jan 16 2021, 12:53 PM

This is an indirect buff, because slingers cost Stone & that makes it slightly easier to get ahead in the village phase. Seeing that feldfeld already felt Ptolemies were the strongest civ (I think), we might need to consider doing something else.

In D3378#150102, @wraitii wrote:

This is an indirect buff, because slingers cost Stone & that makes it slightly easier to get ahead in the village phase. Seeing that feldfeld already felt Ptolemies were the strongest civ (I think), we might need to consider doing something else.

Not really. in a23 you can train javelin mercenaries which is more advantageous than slinger, so it is not a buff.

Keep in mind you still need food.

Build is green

builderr-debug-macos.txt
ld: warning: text-based stub file /System/Library/Frameworks//CoreAudio.framework/CoreAudio.tbd and library file /System/Library/Frameworks//CoreAudio.framework/CoreAudio are out of sync. Falling back to library file for linking.
ld: warning: text-based stub file /System/Library/Frameworks//AudioToolbox.framework/AudioToolbox.tbd and library file /System/Library/Frameworks//AudioToolbox.framework/AudioToolbox are out of sync. Falling back to library file for linking.
ld: warning: text-based stub file /System/Library/Frameworks//ForceFeedback.framework/ForceFeedback.tbd and library file /System/Library/Frameworks//ForceFeedback.framework/ForceFeedback are out of sync. Falling back to library file for linking.
ld: warning: text-based stub file /System/Library/Frameworks//CoreVideo.framework/CoreVideo.tbd and library file /System/Library/Frameworks//CoreVideo.framework/CoreVideo are out of sync. Falling back to library file for linking.
ld: warning: text-based stu

See https://jenkins.wildfiregames.com/job/macos-differential/2893/display/redirect for more details.

Harbormaster completed remote builds in B15107: Diff 15379.Jan 16 2021, 1:38 PM

Anyway this patch seems necessary. Without patch, pto will start with -2 gathers.

Ptolemies were unique in having a forum Merc, this seems necessary indeed.

This revision is now accepted and ready to land.Jan 16 2021, 4:52 PM

Closed by commit rP24649: Swap Ptolemaic Slinger & Javelineer (authored by wraitii). · Explain WhyJan 16 2021, 4:57 PM

This revision was automatically updated to reflect the committed changes.

wraitii added a commit: rP24649: Swap Ptolemaic Slinger & Javelineer.