[DONE][Suggestion] Terrain command as an array

[Finnik]

It is expected by the design and we would like to use it this way. Executions, as I understand, call corresponding method separately, so it rolls also separately. In turn, in my realization, for fillArea and addLine we roll once for all blocks added, and it is also what I wanted here.

[Meridian]

Progress report: I think I understood the necessary background for vertical terrain and will be able to merge and test this feature within next 1-3 days.

[Finnik]

Would it also use randomTerrains property? Or my additions are breaking something?

[wcho035]

Is this feature available with the current version of OXCE? Or is it still being tested?

[Meridian]

The changelog says quite clearly what is available in the current version of OXCE.

[wcho035]

I did read what is available in the current change log. However, seeing the screenshot from this thread, I just wish to confirm if this is being tested. A yes or no will do.

[Meridian]

If the possible answers are only yes and no... then I am forced to say no.

[wcho035]

Thank you.

[Meridian]

Would it also use randomTerrains property?

Yes.
And it was also renamed to just `randomTerrain`.

Or my additions are breaking something?

Sort of.
I wanted only one attribute (=the terrain vector) with 2 types of ruleset syntax. Not 2 different attributes.

Merged here: https://github.com/MeridianOXC/OpenXcom/commit/e0582e16acd70dd527ecaf5d64ccfbd3036a04ab

[wcho035]

Feedback.

I have tested this new feature today. I found the distribution of the different terrain map mix uneven.

Is it possible we can determine the weight of distribution with the terrain mix?

For example.

  - type: Randomseaterrain
    commands:
    - type: addCraft
      groups: 1
      randomTerrain: [SEAFORESTX, CORAL]
    - type: addUFO
      groups: 1
      #UFOName: STR_DREADNOUGHTLAND
      randomTerrain: [SEAFORESTX, CORAL]
    #- type: addUFO
    #  groups: 1
    #  UFOName: STR_FLEET_SUPPLY_CRUISERLAND
    #  randomTerrain: [SEAFORESTX, CORAL]
    - type: addBlock
      randomTerrain: [CORAL, SEAFORESTX]
      executions: 12
    - type: fillArea
      randomTerrain: [SEAFORESTX, CORAL]

Coral appeared around 8 of the maps with seaforestx for 4.

Perhaps something like.

      randomTerrain: [CORAL, SEAFORESTX]
      randomweight: [50,50]

The weight determine the chances of occurrence or the mixture of different terrain map generated?

[Solarius Scorch]

I've just realized I'd never properly thanked you for adding this... So many thanks