Jump to content

==[REQUEST] == Seeding animation


Silier
 Share

Recommended Posts

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 ;)

Link to comment
Share on other sites

@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"/>

 

  • Like 1
Link to comment
Share on other sites

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?

  • Haha 1
Link to comment
Share on other sites

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.

Link to comment
Share on other sites

  • 4 weeks later...
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?

 

Link to comment
Share on other sites

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 :D

 

 

 

 

Link to comment
Share on other sites

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"/>

 

Link to comment
Share on other sites

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 :)

  • Like 1
Link to comment
Share on other sites

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.

  • Like 1
Link to comment
Share on other sites

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.

  • Like 1
Link to comment
Share on other sites

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 :)

Link to comment
Share on other sites

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

  • Like 2
Link to comment
Share on other sites

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.

  • Like 4
Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

 Share

×
×
  • Create New...