Hospitaller - IMPLEMENTED

User avatar
makazuwr32
Posts: 7830
Joined: Tue Oct 17, 2017 9:29 am
Location: Moscow, Russia

Re: Hospitaller - ACCEPTED

Post by makazuwr32 »

It is unit by itself but in topic there is suggested the unit and its upgrade and research for it must be tied to knight templar.
makazuwr32 wrote: Mon Sep 16, 2019 7:54 amWhen you ask to change something argument why...
Put some numbers, compare to what other races have and so on...
© by Makazuwr32™.
AoF Dev Co-Leader
Image
User avatar
COOLguy
Posts: 4005
Joined: Sat Jul 12, 2014 2:58 am
Location: Nenuial, Arnor

Re: Hospitaller - ACCEPTED

Post by COOLguy »

Ok that makes sense. Thanks
Thanks!
Josh
User avatar
StormSaint373
Posts: 1801
Joined: Mon Nov 27, 2017 5:35 pm
Location: USA, East Ohio

Re: Hospitaller - ACCEPTED

Post by StormSaint373 »

This unit is currently by itself, but when using upgrades I think that the Defenders of the Faith, (Unit recruited from the church) should be in a set, like the Romans...

Would make things easier, in a way. But the Choice is entirely up to you.
Beware the calm before the Tempest. . .
User avatar
Gral.Sturnn
Posts: 1877
Joined: Tue Feb 13, 2018 11:29 pm

Re: Hospitaller - ACCEPTED

Post by Gral.Sturnn »

What sprite shall be used?
~Gral.Sturnn
User avatar
makazuwr32
Posts: 7830
Joined: Tue Oct 17, 2017 9:29 am
Location: Moscow, Russia

Re: Hospitaller - ACCEPTED

Post by makazuwr32 »

Both because one is normal one and second is upgrade.
makazuwr32 wrote: Mon Sep 16, 2019 7:54 amWhen you ask to change something argument why...
Put some numbers, compare to what other races have and so on...
© by Makazuwr32™.
AoF Dev Co-Leader
Image
User avatar
Gral.Sturnn
Posts: 1877
Joined: Tue Feb 13, 2018 11:29 pm

Re: Hospitaller - ACCEPTED

Post by Gral.Sturnn »

Looks good
~Gral.Sturnn
User avatar
godOfKings
Posts: 3302
Joined: Fri Sep 22, 2017 4:50 pm

Re: Hospitaller - ACCEPTED

Post by godOfKings »

How about the unit gets a separate upgrade tech instead of putting with knight templar?


Tech: To Knight Hospitaller
Cost: 4 turns (much lower upgrade cost for single unit upgrade)

Builder: church/cathedral
Attachments
tech_hospitaller_upgrade.png
tech_hospitaller_upgrade.png (272 Bytes) Viewed 3731 times
There is no place for false kings here, only those who proves themselves to b the true kings of legend, or serves under me

For I watch over this world looking for those worthy to become kings, and on the way get rid of the fakes and rule over the fools
User avatar
StormSaint373
Posts: 1801
Joined: Mon Nov 27, 2017 5:35 pm
Location: USA, East Ohio

Re: Hospitaller - ACCEPTED

Post by StormSaint373 »

That looks pretty accurate to the actual symbol of the order, OK you've got it!
Beware the calm before the Tempest. . .
User avatar
StormSaint373
Posts: 1801
Joined: Mon Nov 27, 2017 5:35 pm
Location: USA, East Ohio

Re: Hospitaller - ACCEPTED

Post by StormSaint373 »

@Stratego (dev)

What else is needed for this?
Attachments
tech_upgrade_hospitaller_.json
(774 Bytes) Downloaded 70 times
unit_knight_hospitaller.json
(3.49 KiB) Downloaded 66 times
unit_hospitaller.json
(3.48 KiB) Downloaded 63 times
tech_hospitaller_upgrade.png
tech_hospitaller_upgrade.png (272 Bytes) Viewed 3235 times
AoS Hospitaller, Knight.png
AoS Hospitaller, Knight.png (885 Bytes) Viewed 3235 times
AoS Hospitaller.png
AoS Hospitaller.png (830 Bytes) Viewed 3235 times
Beware the calm before the Tempest. . .
Stratego (dev)
Site Admin
Posts: 15734
Joined: Fri Apr 25, 2014 9:28 pm

Re: Hospitaller - ACCEPTED

Post by Stratego (dev) »

is it a standalone unit "hospitaller"
and there is a tech "tech_upgrade_hospitaller" that can upgrade it to "hospitaller knight"

right?

(jsons have some wrong setting tell me if i need to list them here or if i only need to fix them and put in :) )
User avatar
StormSaint373
Posts: 1801
Joined: Mon Nov 27, 2017 5:35 pm
Location: USA, East Ohio

Re: Hospitaller - ACCEPTED

Post by StormSaint373 »

Correct.
Beware the calm before the Tempest. . .
User avatar
StormSaint373
Posts: 1801
Joined: Mon Nov 27, 2017 5:35 pm
Location: USA, East Ohio

Re: Hospitaller - ACCEPTED

Post by StormSaint373 »

Unfortunately, @Stratego (dev) , I only know how to edit numbers...

What else needs to be done?
Beware the calm before the Tempest. . .
Stratego (dev)
Site Admin
Posts: 15734
Joined: Fri Apr 25, 2014 9:28 pm

Re: Hospitaller - ACCEPTED

Post by Stratego (dev) »

i will fix the json and you will see if you compare it to yours - ok?
User avatar
StormSaint373
Posts: 1801
Joined: Mon Nov 27, 2017 5:35 pm
Location: USA, East Ohio

Re: Hospitaller - ACCEPTED

Post by StormSaint373 »

Ok
Beware the calm before the Tempest. . .
Stratego (dev)
Site Admin
Posts: 15734
Joined: Fri Apr 25, 2014 9:28 pm

Re: Hospitaller - ACCEPTED

Post by Stratego (dev) »

i still had no time as i am doing online multiplayer logic coding, if anyone have time please fix the jsons
(as i see they are now mostly archer and crossbow jsons barely modified)

best if someone can try with modding before sending to me - thanks!
User avatar
StormSaint373
Posts: 1801
Joined: Mon Nov 27, 2017 5:35 pm
Location: USA, East Ohio

Re: Hospitaller - ACCEPTED

Post by StormSaint373 »

Performed minor alterations to match Templar
Attachments
AoS Hospitaller, Knight.png
AoS Hospitaller, Knight.png (936 Bytes) Viewed 3167 times
AoS Hospitaller.png
AoS Hospitaller.png (861 Bytes) Viewed 3167 times
Beware the calm before the Tempest. . .
User avatar
StormSaint373
Posts: 1801
Joined: Mon Nov 27, 2017 5:35 pm
Location: USA, East Ohio

Re: Hospitaller - ACCEPTED

Post by StormSaint373 »

@Endru1241 , perhaps you can aid in this Json...

I still haven't grasped jsons appropriately yet.
Beware the calm before the Tempest. . .
User avatar
Endru1241
Posts: 2717
Joined: Fri Sep 11, 2015 8:43 am
Location: Poland

Re: Hospitaller - ACCEPTED

Post by Endru1241 »

I corrected it. Now gonna test if works properly.
He had too much heal - changed it to 4 ( and at range 1) and 8 for upgraded one.
Please change names of graphics to unit_hospitaller.png and unit_hospitaller_upg.png


Edit - actually posted wrong file, also names of graphic assets
Last edited by Endru1241 on Thu Sep 05, 2019 2:31 pm, edited 4 times in total.
Age of Strategy design leader
Stratego (dev)
Site Admin
Posts: 15734
Joined: Fri Apr 25, 2014 9:28 pm

Re: Hospitaller - ACCEPTED

Post by Stratego (dev) »

AWESOME you are!
remark: the "32_" should not be in image names - only very old assets have that (i ran into problems when i changed unit image to be 64px and i had to rename everywhere, so please dont put image size information in image names - thanks!)
User avatar
StormSaint373
Posts: 1801
Joined: Mon Nov 27, 2017 5:35 pm
Location: USA, East Ohio

Re: Hospitaller - ACCEPTED

Post by StormSaint373 »

Shouldn't the heal Rate be the same as Templars?
Beware the calm before the Tempest. . .
User avatar
Endru1241
Posts: 2717
Joined: Fri Sep 11, 2015 8:43 am
Location: Poland

Re: Hospitaller - ACCEPTED

Post by Endru1241 »

Not sure - I wanted to test first if lower heal will be enough, but I felt 5 on base, and especially 10 on upgrade is too much (he would take no damage after heal from single basic skirmisher, so lowered a little. And upgraded hospitaller with 8 heal is set to mitigate master skirmisher damage instead of slowly healing).
Age of Strategy design leader
Stratego (dev)
Site Admin
Posts: 15734
Joined: Fri Apr 25, 2014 9:28 pm

Re: Hospitaller - ACCEPTED

Post by Stratego (dev) »

awesome! can usend me jsons to implement?
User avatar
Endru1241
Posts: 2717
Joined: Fri Sep 11, 2015 8:43 am
Location: Poland

Re: Hospitaller - ACCEPTED

Post by Endru1241 »

Jsons + correctly named pngs.
Attachments
tech_hospitaller_upgrade.png
tech_hospitaller_upgrade.png (272 Bytes) Viewed 3140 times
unit_hospitaller_upg.png
unit_hospitaller_upg.png (936 Bytes) Viewed 3140 times
unit_hospitaller.png
unit_hospitaller.png (861 Bytes) Viewed 3140 times
tech_upgrade_hospitaller.json
(796 Bytes) Downloaded 61 times
unit_hospitaller_upg.json
(3.49 KiB) Downloaded 54 times
unit_hospitaller.json
(3.43 KiB) Downloaded 62 times
Age of Strategy design leader
Stratego (dev)
Site Admin
Posts: 15734
Joined: Fri Apr 25, 2014 9:28 pm

Re: Hospitaller - ACCEPTED

Post by Stratego (dev) »

but as i understood you had no chance the test, i upload new dev version in 5 minutes so u can test prior.
User avatar
Endru1241
Posts: 2717
Joined: Fri Sep 11, 2015 8:43 am
Location: Poland

Re: Hospitaller - ACCEPTED

Post by Endru1241 »

Stratego (dev) wrote: Thu Sep 05, 2019 8:00 pm but as i understood you had no chance the test, i upload new dev version in 5 minutes so u can test prior.
Now tested - it works - he shoots, he kills, he heals and he upgrades properly - only in unit details icons are still missing (and texts, but that I understand why).
Age of Strategy design leader
Stratego (dev)
Site Admin
Posts: 15734
Joined: Fri Apr 25, 2014 9:28 pm

Re: Hospitaller - ACCEPTED

Post by Stratego (dev) »

awesome you are!
User avatar
Endru1241
Posts: 2717
Joined: Fri Sep 11, 2015 8:43 am
Location: Poland

Re: Hospitaller - ACCEPTED

Post by Endru1241 »

Actually, after using him few times I noticed that he could be a little more resilient and more importantly it's a little too weak to matter to other units -I would change that heal +2(6) /+1(9).
Also shadow look weird.
@StormSaint373 Are you willing to correct the shadow?
Attachments
unit_hospitaller.json
(3.46 KiB) Downloaded 57 times
unit_hospitaller_upg.json
(3.53 KiB) Downloaded 57 times
Age of Strategy design leader
User avatar
Puss_in_Boots
Posts: 3206
Joined: Fri Apr 21, 2017 1:23 am
Location: New Jersey

Re: Hospitaller - ACCEPTED

Post by Puss_in_Boots »

My edits.
Attachments
unit_hospitaller_upg.png
unit_hospitaller_upg.png (915 Bytes) Viewed 3109 times
unit_hospitaller.png
unit_hospitaller.png (890 Bytes) Viewed 3109 times
OLÉ
User avatar
StormSaint373
Posts: 1801
Joined: Mon Nov 27, 2017 5:35 pm
Location: USA, East Ohio

Re: Hospitaller - ACCEPTED

Post by StormSaint373 »

Ah, I see... Thx, @Puss_in_Boots .
Beware the calm before the Tempest. . .
Stratego (dev)
Site Admin
Posts: 15734
Joined: Fri Apr 25, 2014 9:28 pm

Re: Hospitaller - ACCEPTED

Post by Stratego (dev) »

Thanks!

all is in!

mega thanks!
Post Reply

Return to “Unit - ranged (like archers)”