==[REQUEST] == Seeding animation

**Imarok** · September 7, 2019

1 hour ago, Stan` said:

Add the code i'll make the variant

Index: binaries/data/mods/public/simulation/components/UnitAI.js
===================================================================
--- binaries/data/mods/public/simulation/components/UnitAI.js	(Revision 22862)
+++ binaries/data/mods/public/simulation/components/UnitAI.js	(Arbeitskopie)
@@ -2719,7 +2719,9 @@
 
 					this.FaceTowardsTarget(this.order.data.target);
 
-					this.SelectAnimation("build");
+					let cmpIdentity = Engine.QueryInterface(this.order.data.target, IID_Identity);
+
+					this.SelectAnimation(cmpIdentity && cmpIdentity.HasClass("Field") ? "seed" : "build");
 					this.StartTimer(1000, 1000);
 					return false;
 				},

But this doesn't consider the elephant worker issue. And not sure if the name of the animation fits our schema

**Stan`** · September 8, 2019

@Imarok Thanks.

It would seem that the current variants use 'Build_farm' instead of 'seed'. If you change that in the code and apply the following diff it should work.

Index: binaries/data/mods/public/art/variants/biped/build_farm.xml
===================================================================
--- binaries/data/mods/public/art/variants/biped/build_farm.xml	(revision 22869)
+++ binaries/data/mods/public/art/variants/biped/build_farm.xml	(working copy)
@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <variant name="Build_farm">
   <animations>
-    <animation event="0.5" file="biped/citizen/build.dae" name="Build_farm" speed="100"/>
+    <animation event="0.5" file="biped/gatherer/seeding.dae " name="Build_farm" speed="100"/>
   </animations>
   <props>
     <prop attachpoint="shield"/>
Index: binaries/data/mods/public/art/variants/biped/female_build_farm.xml
===================================================================
--- binaries/data/mods/public/art/variants/biped/female_build_farm.xml	(revision 22869)
+++ binaries/data/mods/public/art/variants/biped/female_build_farm.xml	(working copy)
@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <variant name="Build_farm">
   <animations>
-    <animation event="0.75" file="biped/citizen/build.dae" name="Build_farm" speed="70"/>
+    <animation event="0.5" file="biped/gatherer/seeding.dae " name="Build_farm" speed="100"/>
   </animations>
   <props>
     <prop attachpoint="weapon_L"/>

Lion.Kanzen · September 8, 2019

so... Its implemented?

**Stan`** · September 8, 2019

4 minutes ago, Lion.Kanzen said:

so... Its implemented?

No art is ready code is not yet

Lion.Kanzen · September 8, 2019

Just now, Stan` said:

No art is ready code is not yet

Just waIting...an announcement.

Hmmm question part of cycle of farming ....

Segar or mow mowing (?) with scythe.

An alternetive maybe

**Imarok** · September 8, 2019

29 minutes ago, Stan` said:

@Imarok Thanks.

It would seem that the current variants use 'Build_farm' instead of 'seed'. If you change that in the code and apply the following diff it should work.


Index: binaries/data/mods/public/art/variants/biped/build_farm.xml
===================================================================
--- binaries/data/mods/public/art/variants/biped/build_farm.xml	(revision 22869)
+++ binaries/data/mods/public/art/variants/biped/build_farm.xml	(working copy)
@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <variant name="Build_farm">
   <animations>
-    <animation event="0.5" file="biped/citizen/build.dae" name="Build_farm" speed="100"/>
+    <animation event="0.5" file="biped/gatherer/seeding.dae " name="Build_farm" speed="100"/>
   </animations>
   <props>
     <prop attachpoint="shield"/>
Index: binaries/data/mods/public/art/variants/biped/female_build_farm.xml
===================================================================
--- binaries/data/mods/public/art/variants/biped/female_build_farm.xml	(revision 22869)
+++ binaries/data/mods/public/art/variants/biped/female_build_farm.xml	(working copy)
@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <variant name="Build_farm">
   <animations>
-    <animation event="0.75" file="biped/citizen/build.dae" name="Build_farm" speed="70"/>
+    <animation event="0.5" file="biped/gatherer/seeding.dae " name="Build_farm" speed="100"/>
   </animations>
   <props>
     <prop attachpoint="weapon_L"/>

That works, but it doesn't resolve the elephant issue.
I guess they need a separate variant file.

Edit: And I guess they aren't supposed to seed with a mallet in their hands, aren't they?

**Stan`** · September 8, 2019

8 minutes ago, Imarok said:

That works, but it doesn't resolve the elephant issue.
I guess they need a separate variant file.

Edit: And I guess they aren't supposed to seed with a mallet in their hands, aren't they?

Elephants dont use those variants they are for biped only

Elephants will use another one.

**Imarok** · October 5, 2019

On 9/8/2019 at 12:48 PM, Stan` said:

Elephants will use another one.

So that's still needed, right?

And have you seen my edit? ;)

On 9/8/2019 at 12:38 PM, Imarok said:

Edit: And I guess they aren't supposed to seed with a mallet in their hands, aren't they?

**Stan`** · October 5, 2019

3 minutes ago, Imarok said:

So that's still needed, right?

And have you seen my edit?

Well for now they can keep what they used to do previously Units seeding would be a nice addition.

Maybe @Alexandermb wants to animate elephant seeding, but I guess he deserves a break after the reexport

**Imarok** · October 5, 2019

3 minutes ago, Stan` said:

Well for now they can keep what they used to do previously Units seeding would be a nice addition.

Couldn't you just equip them with some kind of bowl?

**Stan`** · October 5, 2019

@Imarok Like this ?

**Imarok** · October 5, 2019

10 minutes ago, Stan` said:

@Imarok Like this ?

No, my comment was regarding the humans using a mallet for seeding instead of a bowl. xD

**Stan`** · October 5, 2019

Oh sorry it's late ^^ @Imarok that should do the trick

file.diff

**Imarok** · October 7, 2019

That looks better.

Are you sure the female variant needs all the weapon attachpoints:

+    <prop attachpoint="shield"/>
+    <prop attachpoint="shield_arm"/>
+    <prop attachpoint="helmet"/>
+    <prop attachpoint="weapon_R"/>
+    <prop attachpoint="weapon_bow"/>
+    <prop attachpoint="ammo"/>

**Stan`** · October 7, 2019

2 minutes ago, Imarok said:

That looks better.

Are you sure the female variant needs all the weapon attachpoints:


+    <prop attachpoint="shield"/>
+    <prop attachpoint="shield_arm"/>
+    <prop attachpoint="helmet"/>
+    <prop attachpoint="weapon_R"/>
+    <prop attachpoint="weapon_bow"/>
+    <prop attachpoint="ammo"/>

Well it doesn't hurt to clean them in case they would be full

**Imarok** · October 7, 2019

OK. Done: D2362.

A bit Offtopic: Is it intended units loose their helmet/cap and cape while carrying or gathering?

**Alexandermb** · October 7, 2019

10 minutes ago, Imarok said:

A bit Offtopic: Is it intended units loose their helmet/cap and cape while carrying or gathering?

Yep, thats how its have been all the time thought not the most immersive feature imho.

**Stan`** · October 7, 2019

24 minutes ago, Alexandermb said:

Yep, thats how its have been all the time thought not the most immersive feature imho.

Some players have been giving negative feedback about it, maybe we should remove this. @wowgetoffyourcellphone opinions on this ?

**Alexandermb** · October 7, 2019

7Id rather go for the shield on back and keep helmet even raised for corinthian if possible.

Looks better than deleting the whole equipment.

But im pretty sure @wowgetoffyourcellphone would say the problem isnt the the variants but the citizen soldier feature.

but for the most close solution to keep both citizen soldier and equipment would be only delete weapon while gathering. There are some sheats so they store the sword for the swordsman.

And other advanced feature would be just like Dawn of Man. There is a call to arms button when you are attacked by raiders. It makes al citizen run no the nearest house to pick up weapons and shields if they had any and go no the raiders town direction.

sphyrth · October 7, 2019

9 hours ago, Alexandermb said:

But im pretty sure @wowgetoffyourcellphone would say the problem isnt the the variants but the citizen soldier feature.

Yep. It's the Realism argument against the Citizen Soldier itself. Many people have pointed it out that it's too unrealisitic for people people who are carrying Wood to be able to instantly whiff out their battle-gear and vice versa.

As much as I prefer Gameplay Over Realism at this point, they're correct. Even the Marian Romans (who are the prime example of Gather-Build Soldiers) need time to ready up their formation and brace for battle.

**Stan`** · October 7, 2019

I guess they should transform at the civ center, like a localized upgrade feature...

19 minutes ago, Alexandermb said:

7Id rather go for the shield on back and keep helmet even raised for corinthian if possible.

Looks better than deleting the whole equipment.

Me too

**Sundiata** · October 8, 2019

17 hours ago, Stan` said:

I guess they should transform at the civ center, like a localized upgrade feature...

Barracks can double as an armory. Not even really an abstraction. It would also make people place barracks at more strategic locations.

av93 · October 8, 2019

Not only the immersive thing, but also one of the core problems IMHO of 0 a.d. Defend and collect resources it's generally better than get the workers to fight.

But back to somehow to topic about elephants and farms: what about a real worker that guides the elephant, like the merchant? In this case, the elephant could be unloading grain sacks, and the worker seeding them

**Stan`** · October 8, 2019

3 minutes ago, av93 said:

But back to somehow to topic about elephants and farms: what about a real worker that guides the elephant, like the merchant? In this case, the elephant could be unloading grain sacks, and the worker seeding them

Could work.

balduin · October 14, 2019

On 10/8/2019 at 12:07 PM, av93 said:

what about a real worker that guides the elephant, like the merchant?

Yes, a mahout on the back of the elephant would me a good idea.

==[REQUEST] == Seeding animation

Recommended Posts

Link to comment

Share on other sites

Top Posters In This Topic

Popular Days

Top Posters In This Topic

Popular Days

Popular Posts

Silier

Stan`

Sundiata

Posted Images

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Join the conversation