Basic Transport Units

[silnarm]

Thanks Hailstone  

Firstly r699 is very unstable on my system (win7). It throws "runtime" errors and refuses to load most tilesets and factions :/ However I know this is just a WIP build so I'm not too worried.

This is the exception handler getting in the way, for now set MiscCatchExceptions=false in glestadv.ini, I'll strip it out completely for 0.3.

One more bug I noticed, the load skill is being executed after every move, even if it isn't within load range.

[Omega]

This is the exception handler getting in the way, for now set MiscCatchExceptions=false in glestadv.ini, I'll strip it out completely for 0.3.

Really? I haven't gotten around to testing this yet, but in the past, I tried without the exception catching, and rather than throwing an error and shutting down, it just flat out crashed and gave a generic system error... What is the exception handler anyway? I just assumed it caught bugs and shut the program down rather than let the program crash?

[wyvern]

I have a question, will units inside a transport/building be able to shoot, for example, smg gunners shooting from inside an apc or archers ahooting from a tower

[hailstone]

Thanks for the feedback. Also thanks for committing the xml file Silnarm.

Maximum load is implemented now. Should the unit still move to the carrier if it's full?

I've prevented carriers from loading themselves, loading nothing and loading a unit multiple times.

Units not loading might be because of a command queue limit although I couldn't find it, only 4 units appear to load in one try.

I have a question, will units inside a transport/building be able to shoot, for example, smg gunners shooting from inside an apc or archers ahooting from a tower

Not at the moment but it will be discussed once I've finished the basics.

[wyvern]

Ok thanks

[hailstone]

I've made load the default command for right click on target for a carrier and combined multiple unit loads into a single command. It's now possible to load more than 4 units at a time.

I think we are going to need yet another command type, 'be_loaded'.

As is, units that are to be loaded have no knowledge of that fact, and just get a move command to the carrier unit, if the carrier moves away, they could never be loaded, while staying on the command queue for the carrier, causing it to cease to function correctly.

If another command is issued that isn't queued such as movement, the current commands are canceled as far as I know. I don't think anything worse happens than not being loaded.

[silnarm]

I think we are going to need yet another command type, 'be_loaded'.

As is, units that are to be loaded have no knowledge of that fact, and just get a move command to the carrier unit, if the carrier moves away, they could never be loaded, while staying on the command queue for the carrier, causing it to cease to function correctly.

If another command is issued that isn't queued such as movement, the current commands are canceled as far as I know. I don't think anything worse happens than not being loaded.

True enough... but not being loaded when the user wanted them loaded is poor form. I'm happy for you to ignore it for now though, I'll just sneak it in sometime when you're looking the other way 

ps: This wouldn't be an 'explicit' Xml defined command, it would be added automagically to movable units (and later only those that can actually be loaded into transports).

[Zoythrus]

you know, the transport should be able to enhance it's passengers through an emanation. for example: a tower that gives its inhabitants with a +2 range bonus.

[John.d.h]

you know, the transport should be able to enhance it's passengers through an emanation. for example: a tower that gives its inhabitants with a +2 range bonus.

And vice versa.  I think it would be a good addition to the "master only" and "pet only" effects.  In this case, I suppose "carrier only" and "passenger only".

[Omega]

you know, the transport should be able to enhance it's passengers through an emanation. for example: a tower that gives its inhabitants with a +2 range bonus.

And vice versa.  I think it would be a good addition to the "master only" and "pet only" effects.  In this case, I suppose "carrier only" and "passenger only".

Agreed. Minor things like slowing down a unit when they carry, or increasing the range of a unit on a tower would be great.

Relatedly, a way to change the models when you have a unit carried could be very handy, as in, change the model for any skill. Perhaps best done with an optional tag in the skill itself, for example:
    <skill>
      <type value="move" />
      <name value="move_skill" />
      <ep-cost value="0" />
      <speed value="190" />
      <anim-speed value="130" />
      <animation path="models/normal_move.g3d" />
      <loaded-animation path="models/carrying_move.g3d" />
      <sound enabled="false" />
    </skill>