Concern about new update regarding new wave machanic API

Hi!

With upcoming update very close and with "The next update will definitely introduce true ocean waves (including new boat behaviour of course)" i wonder if API is ready to handle the following scenario:

I use a plugin to make a boat to follow automatic route (Ferry Travel). This internally is handled by moving (teleport) the boat along some predefined route (points defined by x,z coordinate). For y axis (water level) is more ore less around 90 (depend on boat type - rowboat, sailboat, have different values).

Now questions:

- With new wave mechanic where water in a some point x,z will be dynamic (other than static 90) how i will determine correct height via API to know where to teleport a boat in a new point at surface of water?

So wee need some sort of API to can calculate real height of water (with wave corrections) in some point x,z.

Moreover this is not enough, because different boat size sink in water different depth (rowboat versus sailboat) so we need a real API function to obtain "correct" Z value so vehicle (rowboat,sailboat,etc.) can be teleported on valid,"naturally" point x,z at surface of water.

Now i move vehicle with some code like:

    // Some static config
    // Type of vehicle for allowed ferry travel
    String [] vehicleName= {"rowboat","sailboat"};
    // Water Level for vehicleType
    Float  [] vehicleWatterLevel= {89.9932f,89.49464f};

    // Return water level for a specific vehicle. If no info found return default water level (90)
    public float getWaterLevel(Vehicle v) {
        if (v==null) {
            return FerryConfig.WATER_LEVEL;
        }
        VehicleDefinition def=v.getDefinition();
        for (int i = 0; i < vehicleName.length; i++) {
            if (vehicleName[i].equals(def.name)) {
                return vehicleWatterLevel[i];
            }
        }
        return FerryConfig.WATER_LEVEL;
    }

// and actual move

// Get static water level by type of vehicle
float waterlevel=getWaterLevel(vehicle);   
Vector3f pos=vehicle.getPosition();    // current position
Vector3f target= new Vector3f(point.x,waterlevel,point.y);  // target position  <-- Here new API must called to get actual water level by type of vehicle
// make vehicle to look toward new target position but only if distance is greater than a min value (default 3)
float distance=Utils.MathUtils.distance(pos.x, pos.y, pos.z, target.x, target.y, target.z);
// if distance between source and target > MIN_DISTANCE calculate new rotation. else keep current rotation of vehicle
Quaternion rotation;
if (distance>FerryConfig.MIN_DISTANCE_FOR_ROTATE) {
            // Rotate vehicle to look at target from current position
            Vector3f direction = Utils.VectorUtils.getDirection(pos, target);
            rotation = new Quaternion().lookAt(direction);
        } else {
            // Keep rotation of vehicle
            rotation=vehicle.getRotation();
        }

  //Set rotation
  vehicle.setRotation(rotation);
  //Move vehicle at target
  vehicle.setPosition(target);

Current movement of rowboat is on flat water level. So first align vehicle to target and latter move. But with real wave ... i see this like more steps:

- First align direction of vehicle to desired direction

- Second move vehicle in new location

- Last step normalize rotation (keeping aligned direction) to naturally fit current wave (this i think need to be addressed via API too!)

red51 i hope you will take in account this API additions (if is not yet some method who can achieve this scope) for new release.

Thx in advance!

Quote from red51

Hey, that's a good point! Currently the waves are not taken into account, so setPosition() would still work, but it would cause the boat to temporarily hover above water or submerge. The API will provide a method to calculate the wave height at a specific location, but this will only help if you call vehicle.setPosition() very often (providing the final water position which also takes the waves into account)...

We could change the setPosition() method to take waves into account automatically, but I'm not sure if that would be a good idea... right now this method allows you to set an arbitrary position (you can even set a position on land or in the air). Maybe a new "setPositionOnWater()" method would be better in this case?

Or maybe we change setPosition() to teleport a vehicle instantly, and add a new "moveTo()" method (which optionally takes waves into account)? This could be a cleaner solution... IMO the current behaviour isn't very intuitive anyway (I'd expect a vehicle to move to a new position instantly when calling setPosition())... the "moveTo" method could then get a few parameters, e.g. to determine if the boat should automatically adapt to waves or if a player should still be able to take control over the boat. Maybe that would be the best solution?

I like vehicle.setPositionOnWater(x,z) if that take in account wave position.

But vehicle.moveOnWaterTo(x,z,adaptWave==true) with additional parameters don't sound bad. It can work for example to do last step to "normalize" position of boat with wave.