Whats wrong with this raycast?

[Brian Smithers]

function fxDTSBrick::calculateNeighbors(%this,%group)
{
   %thisPos = %this.getPosition();
   for(%i=0;%i<%group.getCount();%i++)
   {
      %brick = %group.getObject(%i);
      %pos =  %brick.getPosition();
      %dist = vectorDist(%thisPos,%pos);
      if(%dist <= $Paths::MaxDist && %brick !$= %this)
      {
-snip-
            %ray = containerRayCast(%thisPos,%pos,$TypeMasks::FxBrickAlwaysObjectType,%this);
-debug snip-
            %obj = firstWord(%ray);
               if(isObject(%obj) && %brick.getPosition() $= %obj.getPosition())
            {
               talk("ray works");
               %this.neighbor[%this.neighbors++] = %brick;
            }
      }
-snip-
   }
   %this.neighbors = %this.neighbors++;
}

just focus on the part with the raycast.
it seems to never want to work.
The debugs always show it just returning 0 everytime.

[Ad Bot]

[otto-san]

is it not $TypeMask::?
you have $TypeMasks::. i may just be crazy or something though

i am

[Treynolds416]

Give us a more concise version of your code. I honestly don't have the patience to go reading through your code to find out what your ambiguous variables mean. We're the ones that are going to help you, you should make it easier.

Anyways, your typemasks are a little redundant, fxBrickAlwaysObjectType encompasses all cases where fxBrickObjectType applies

[Brian Smithers]

Give us a more concise version of your code. I honestly don't have the patience to go reading through your code to find out what your ambiguous variables mean. We're the ones that are going to help you, you should make it easier.

Anyways, your typemasks are a little redundant, fxBrickAlwaysObjectType encompasses all cases where fxBrickObjectType applies

i removed a part
but the rest is really straight forward.

[otto-san]

%self is undefined but that's obviously not your problem

[Brian Smithers]

%self is undefined but that's obviously not your problem

didn't change anything.
oh just reread yeah
but it did fix 1 thing.

[Greek2me]

Have you looked at this if statement?

Code: [Select]

if(%dist <= $Paths::MaxDist && %brick !$= %self)

[Brian Smithers]

Have you looked at this if statement?

I fixed that.

updated OP

[Greek2me]

I fixed that.

updated OP

Well yeah, but you should actually echo all those values and see if they come out right.

[Brian Smithers]

Well yeah, but you should actually echo all those values and see if they come out right.

I do
and they do.
I snipped the debug line.

[Trinick!]

$TypeMasks::FxBrickObjectType | $TypeMasks::FxBrickAlwaysObjectType

It's because of this. Use $TypeMasks::FxBrickAlwaysObjectType, there is no reason to have both.

[Brian Smithers]

It's because of this. Use $TypeMasks::FxBrickAlwaysObjectType, there is no reason to have both.

I changed and it didn't fix

[Port]

Try using this:

%ray = containerRayCast( %this.position, %brick.position, $TypeMasks::FxBrickAlwaysObjectType, %this );

if ( isObject( %col = firstWord( %ray ) ) && %col == %brick )
{
    echo( %this SPC "hit" SPC %brick );
}

If that doesn't work then the issue is with another part of your code.