methusalah / OpenRTS

Real-Time Strategy game 3D engine coded in pure java
MIT License
1.29k stars 151 forks source link

Guide for dataset contribution #74

Closed roroah closed 9 years ago

roroah commented 9 years ago

I'll add some reference on what i think is missing for the assets contribution page.

https://github.com/methusalah/OpenRTS/wiki/Guide-for-dataset-contribution#unit

https://github.com/methusalah/OpenRTS/wiki/Guide-for-dataset-contribution#actor https://github.com/methusalah/OpenRTS/wiki/Guide-for-dataset-contribution#mover

As example of another way to present information (be kind i don't fully understand what each value is) :

Unit

<Unit id="tank">
    <UIName value="simple combat armored vehicle"/>
    <Race value="human"/>
    <Radius value="0.8"/>
    <Speed value="4"/>
    <Mass value="1"/>
    <MaxHealth value="120"/>
    <MoverLink value="GroundProne"/>
    <WeaponList WeaponLink="TankCanon" TurretLink="TankTurret"/>
    <WeaponList WeaponLink="TankMachinegun" TurretLink="TankMachinegun"/>
    <WeaponList WeaponLink="TankFrontLaser"/>
    <ActorLink value="Tank"/>
</Unit>

Required value :

Optional value :

About the WeaponList the current one is kinda tricky, i think that it should be renamed WeaponLink and work like all other value, this give us :

Then the turret value would be stored inside the Weapon himself as :

Weapon

<Weapon id="TankCanon".>
    <UIName value="ubber powerful canon"/> (I don't think this is needed)
    <EffectLink value="StdDamage"/>
    <Range value="5"/>
    <ScanRange value="7"/>
    <Period value="1.2"/>
    <ActorLink value="TankCanonMainActor"/>
    <SourceBone value="BMuzzle01"/>
    <DirectionBone value="BMuzzleVec01"/>
    <TurretLink value="TankTurret"/>
 </Weapon>

Required value :

Optional value :

methusalah commented 9 years ago

"GroundProne" is the ID of the mover to use. I will change it to a more generic term in the wiki.

About the presentation, indeed, I need to add info and make myself clearer :) I'll try it.

In the current nomenclature, $Link is a parameter containing the ID of another definition, $List is a parameter that may be used many times. It may be not wise to store the turret inside the unit def, because :

Note that UInames shouldn't be optionnal, as it is shown inside the GUI on the mouseover for exemple.

roroah commented 9 years ago

$Link is a parameter containing the ID of another definition, $List is a parameter that may be used many times.

Edit : /o/ i'm a colaborator

methusalah commented 9 years ago

I adopted you suggestion because it seemed wise to me, thanks ! I will add a short description for the element to avoid any confusion about the "turret" term.

I also added you to collaborators, so you can contribute to the wiki as you wish.

If these changes fit this issue, we may close here :)

roroah commented 9 years ago

You welcome. Yes we've done there, if you don't mind I'll add an issue for each missing part of the wiki.