Git Product home page Git Product logo

Comments (10)

Schmoozerd avatar Schmoozerd commented on June 16, 2024

possible indeed that you caught an error there :)
Just go thorugh code and check if the z-pos is updated at some point

from mangos.

fr1nge avatar fr1nge commented on June 16, 2024

afaik the vmap height is calculated on the following line:
creature.UpdateAllowedPositionZ(destX, destY, destZ);

check the definiton of that function tho, i might be mistaken.

from mangos.

Lillecarl avatar Lillecarl commented on June 16, 2024

fr1nge that function checks for dynamic height. But i dont see why the creature wouldnt just walk to destinationz instantly anyways.. We could test and set z to 0 just to see if it does matter or not ^^.

from mangos.

VladimirMangos avatar VladimirMangos commented on June 16, 2024

you can't use z==0 always. Function update Z from current to floor Z so if you under ground it can't up you to ground level.

from mangos.

Lillecarl avatar Lillecarl commented on June 16, 2024

@VladimirMangos if you are in a 45 degree slope then, isnt it possible that it sets xy under the slope? then z will be under. Or are there some failsafe marginal?

from mangos.

teyrnon avatar teyrnon commented on June 16, 2024

it is very important to know current z position to determine proper position in UpdateAllowedPositionZ

from mangos.

VladimirMangos avatar VladimirMangos commented on June 16, 2024

@Lillecarl You are right in worry about this. UpdateZ select for Z search not exactly Z coordinate (most cases character stanging Z, but + 2.0 (~ character model length). This is not nice way and possible false cases with "a 45 degree slope" like cases or not so near point from character. From other side Under* check LOS also so reject Z coordinates from other side of wall or other side of floor. Main problem: we can't follow by current floor from character to expected point for select always correct Z :((

from mangos.

Lillecarl avatar Lillecarl commented on June 16, 2024

That is why we need pathfinding? ;)

from mangos.

VladimirMangos avatar VladimirMangos commented on June 16, 2024

Ok, maybe old way outdated now and possible batter sollution. Nice if so. But worry about performance part: UpdateZ is often called method. Maybe need rewrite code that use it instead then...

from mangos.

Lillecarl avatar Lillecarl commented on June 16, 2024

Ah, well i think im out of this discussion now ^^, Knowledge lacking ;)

EDIT: Maby add a parameter to the function to enable or disable "good z update" or smth ^^,

from mangos.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.