Page MenuHomeWildfire Games

[gameplay] move some trainable animals to higher phases
AcceptedPublic

Authored by Angen on Jan 12 2021, 8:59 PM.

Details

Reviewers
borg-
Group Reviewers
Balancing
Summary

This is just an idea.
Currently player has all animals available in phase 1.
So he will likely train one type of the animal right away.
This is trying to add some progress with moving animals with higher food production to higher phases.

Civ code will be removed, it is there just because structure tree is currently buggy and does not work with gaia civ codes properly.

Test Plan

discuss

Event Timeline

Angen created this revision.Jan 12 2021, 8:59 PM
Owners added a subscriber: Restricted Owners Package.Jan 12 2021, 8:59 PM
Angen published this revision for review.Jan 12 2021, 9:00 PM
Angen edited the summary of this revision. (Show Details)

Build is green

builderr-debug-macos.txt
../../../source/graphics/ShaderProgram.cpp:86:15: warning: 'Reload' overrides a member function but is not marked 'override' [-Winconsistent-missing-override]
        virtual void Reload()
                     ^
../../../source/graphics/ShaderProgram.h:124:15: note: overridden virtual function is here
        virtual void Reload() = 0;
                     ^
../../../source/graphics/ShaderProgram.cpp:121:15: warning: 'Bind' overrides a member function but is not marked 'override' [-Winconsistent-missing-override]
        virtual void Bind()
                     ^
../../../source/graphics/ShaderProgram.h:135:15: note: overridden virtual function is here
        virtual void Bind() = 0;
                     ^
../../../source/graphics/ShaderProgram.cpp:131:15: warning: 'Unbind' overrides a member function but is not marked 'override' [-Winconsistent-missing-override]
        virtual void Unbind()
                     ^
../../../source/graphics/ShaderProgram.h:140:

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

Nescio added a subscriber: Nescio.Jan 12 2021, 9:15 PM

Animals belong on the countryside, so I don't think adding a city requirement makes much sense.
If it's to be done, you can do it simply by inserting a line into template_unit_fauna_herd_domestic_cattle.xml.

borg- added a subscriber: borg-.Jan 12 2021, 9:32 PM

I am in favor. It doesn't make historical sense or realism but for gameplay it is fair and it also avoids a possible snowball on maps with a lot of hunting (elephants).

borg- accepted this revision.Wed, Feb 10, 2:16 PM
This revision is now accepted and ready to land.Wed, Feb 10, 2:16 PM