Collision Filters Still having some problems

bjsorrentino · May 28, 2013, 3:33pm

Yes, that’s correct (to set the body type). If you’re not getting any response from the player+wall collision, then it’s something else. I suggest you check and re-check your collision filter values, working through the chart again, if necessary.

http://forums.coronalabs.com/topic/2128-collision-filters-helper-chart/

Brent

mmk.verheijden · May 28, 2013, 3:49pm

Hi Brent,

changing randomStar.myName = “BCloud1” to randomStar.name = “BCloud1” fixed the problem of not detecting

mmk.verheijden · May 16, 2013, 11:25am

Hi this is complacently driving me bonkers. but is it possible to to use preCollision ?

local platform = display.newImage( "platform.png" ) platform.collType = "passthru" physics.addBody( platform, "static", { bounce=0.0, friction=0.3 } ) function character:preCollision( event ) local collideObject = event.other if ( collideObject.collType == "passthru" ) then event.contact.isEnabled = false --disable this specific collision! end end

here is a bigger snip of my code :

local starTable = {} -- Set up star table badc1CollisionFilter = { categoryBits = 1, maskBits = 16 } badc2CollisionFilter = { categoryBits = 2, maskBits = 16 } badc3CollisionFilter = { categoryBits = 4, maskBits = 16 } wallCollisionFilter = { categoryBits = 8, maskBits = 16 } playerCollisionFilter = { categoryBits = 16, maskBits = 15 } function initStar() &nbsp;&nbsp;&nbsp;&nbsp;local star1 = {} &nbsp;&nbsp;&nbsp;&nbsp;star1.imgpath = "Cloud1.png"; --Set Image Path for Star &nbsp;&nbsp;&nbsp;&nbsp;star1.movementSpeed = 10000; --Determines the movement speed of star &nbsp;&nbsp;&nbsp;&nbsp;table.insert(starTable, star1); --Insert Star into starTable &nbsp;&nbsp;&nbsp;&nbsp;local star2 = {} &nbsp;&nbsp;&nbsp;&nbsp;star2.imgpath = "Cloud2.png"; &nbsp;&nbsp;&nbsp;&nbsp;star2.movementSpeed = 12000; &nbsp;&nbsp;&nbsp;&nbsp;table.insert(starTable, star2);&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;local star3 = {} &nbsp;&nbsp;&nbsp;&nbsp;star3.imgpath = "Cloud3.png"; &nbsp;&nbsp;&nbsp;&nbsp;star3.movementSpeed = 14000; &nbsp;&nbsp;&nbsp;&nbsp;table.insert(starTable, star3); &nbsp;&nbsp;&nbsp;&nbsp;local star4 = {} &nbsp;&nbsp;&nbsp;&nbsp;star4.imgpath = "BCloud"..tostring(math.random(1, 3))..".png"; &nbsp;&nbsp;&nbsp;&nbsp;star4.movementSpeed = 16000; &nbsp;&nbsp;&nbsp;&nbsp;table.insert(starTable, star4);&nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;local star5 = {} &nbsp;&nbsp;&nbsp;&nbsp;star5.imgpath = "Cloud5.png"; &nbsp;&nbsp;&nbsp;&nbsp;star5.movementSpeed = 18000; &nbsp;&nbsp;&nbsp;&nbsp;table.insert(starTable, star5);&nbsp;&nbsp;&nbsp;&nbsp; end --END initStar()&nbsp;&nbsp;&nbsp;&nbsp; function getRandomStar() &nbsp;&nbsp;&nbsp;&nbsp;local temp = starTable[math.random(1, #starTable)] -- Get a random star from starTable &nbsp;&nbsp;&nbsp;&nbsp;local randomStar = display.newImage(temp.imgpath) -- Set image path for object &nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;if ( temp.imgpath == "BCloud1.png" ) then &nbsp;&nbsp;&nbsp;&nbsp;local objectb1 =&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;physics.addBody( randomStar, { density=.1, bounce=0.1, friction=.2, radius=45, filter=badc1CollisionFilter } ) &nbsp;&nbsp;&nbsp;&nbsp;randomStar.myName = "BCloud1" &nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;temp.imgpath = "BCloud"..tostring(math.random(1, 3))..".png"; &nbsp;&nbsp;&nbsp;&nbsp;end &nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;if ( temp.imgpath == "BCloud2.png" ) then &nbsp;&nbsp;&nbsp;&nbsp;physics.addBody( randomStar, { density=.1, bounce=0.1, friction=.2, radius=45, filter=badc2CollisionFilter } ) &nbsp;&nbsp;&nbsp;&nbsp;randomStar.myName = "BCloud2" &nbsp;&nbsp;&nbsp;&nbsp;temp.imgpath = "BCloud"..tostring(math.random(1, 3))..".png"; &nbsp;&nbsp;&nbsp;&nbsp;end &nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;if ( temp.imgpath == "BCloud3.png" ) then &nbsp;&nbsp;&nbsp;&nbsp;physics.addBody( randomStar, { density=.1, bounce=0.1, friction=.2, radius=45, filter=badc3CollisionFilter } ) &nbsp;&nbsp;&nbsp;&nbsp;randomStar.myName = "BCloud3" &nbsp;&nbsp;&nbsp;&nbsp;temp.imgpath = "BCloud"..tostring(math.random(1, 3))..".png"; &nbsp;&nbsp;&nbsp;&nbsp;end &nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;randomStar.myName = "star" -- Set the name of the object to star &nbsp;&nbsp;&nbsp;&nbsp;randomStar.movementSpeed = temp.movementSpeed; -- Set how fast the object will move &nbsp;&nbsp;&nbsp;&nbsp;randomStar.x = math.random(-30, \_W);&nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;randomStar.y = -35; &nbsp;&nbsp;&nbsp;&nbsp;randomStar.rotation = math.random(0,20) -- Rotate the object &nbsp;&nbsp;&nbsp;&nbsp;starMove = transition.to(randomStar, { &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;time=randomStar.movementSpeed,&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;y=500, &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;onComplete = function(self) self.parent:remove(self); self = nil;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;end &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;}) -- Move the star&nbsp;&nbsp;&nbsp;&nbsp; end--END getRandomStar() function newCollisionHandler( self, event ) &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;print("oh hello i have a Collision") &nbsp; &nbsp;if myName == "BCloud1" then &nbsp; &nbsp; &nbsp; &nbsp; print("Collided with star1") &nbsp; &nbsp;elseif event.other.myName == "BCloud2" then &nbsp; &nbsp; &nbsp; &nbsp; print("Collided with star2") &nbsp; &nbsp;elseif event.other.myName == "BCloud3" then &nbsp; &nbsp; &nbsp; &nbsp; print("Collided with star3") &nbsp; &nbsp;end end &nbsp; &nbsp; playerSpriteSheet = sprite.newSpriteSheet("player.png", 113, 55) &nbsp; &nbsp; playerSprites = sprite.newSpriteSet(playerSpriteSheet, 1, 4) &nbsp; &nbsp; sprite.add(playerSprites, "players", 1, 4, 1000, 0) &nbsp; &nbsp; player = sprite.newSprite(playerSprites) &nbsp; &nbsp; player.x = -80 &nbsp; &nbsp; player.y = 350 &nbsp; &nbsp; player:prepare("players") &nbsp; &nbsp; player:play() &nbsp; &nbsp; player.collided = false &nbsp; &nbsp; player:setReferencePoint(display.CenterReferencePoint); &nbsp; &nbsp; physics.addBody(player, "static", {density=.1, bounce=0.1, friction=.2, radius=10, filter=playerCollisionFilter } ) &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; player.gravityScale = 0 &nbsp; &nbsp; &nbsp; &nbsp; screenGroup:insert(player) &nbsp; &nbsp; playerIntro = transition.to(player,{time=2000, x=150, onComplete=playerReady}) function scene:enterScene(event)&nbsp;&nbsp;&nbsp;&nbsp;jet.collision = newCollisionHandler &nbsp;&nbsp;&nbsp;&nbsp;jet:addEventListener( "collision", jet ) end

bjsorrentino · May 16, 2013, 5:45pm

Hi Kevin,

You could use pre-collision, but I don’t think it’s necessary here, and I don’t suggest it because it’s a bit “greedy” and should generally be used with good cause. What you have here should be possible with normal collision filters, and I’m not sure what’s “not working” at this point. Could you possibly simplify this project down to one collision filter and two physics objects, so we can get the core in place before expanding it?

Regards,

Brent

mmk.verheijden · May 21, 2013, 1:45pm

Hi Brent,

so i made this small code to test the filters but I’m missing something here i think

local physics = require "physics" physics.start() myObject1CollisionFilter = { categoryBits = 1, maskBits = 3 } myObject2CollisionFilter = { categoryBits = 2, maskBits = 3 } movebox = { friction=0.4, bounce=0.8, filter=myObject1CollisionFilter } box = { friction=0.4, bounce=0.8, filter=myObject2CollisionFilter } local myObject1 = display.newRect( 0, 0, 100, 100 ) myObject1:setFillColor( 255 ) myObject1.myName = "movebox" physics.addBody( myObject1, "static", movebox ) local myObject2 = display.newRect( 0, 0, 100, 100 ) myObject2:setFillColor( 100 ) myObject2.x = 250 myObject2.y = 250 myObject2.myName = "box" physics.addBody( myObject2, "static", box ) function newCollisionHandler( self, event ) &nbsp;&nbsp;&nbsp;&nbsp;print("oh hello i have a Collision") &nbsp; &nbsp;if myName == "box" then &nbsp; &nbsp; print("test 1") &nbsp;&nbsp;&nbsp;&nbsp;end end function myObject1:touch( event ) &nbsp; &nbsp; if event.phase == "began" then &nbsp; &nbsp; &nbsp; &nbsp; self.markX = self.x&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; self.markY = self.y &nbsp; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;elseif event.phase == "moved" then &nbsp;&nbsp;&nbsp;&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; local x = (event.x - event.xStart) + self.markX &nbsp; &nbsp; &nbsp; &nbsp; local y = (event.y - event.yStart) + self.markY &nbsp; &nbsp; &nbsp; &nbsp;&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; self.x, self.y = x, y &nbsp; &nbsp;-- move object based on calculations above &nbsp; &nbsp; end &nbsp; &nbsp; return true end myObject1:addEventListener( "touch", myObject1 ) myObject1.collision = newCollisionHandler myObject1:addEventListener( "collision", myObject1 ) &nbsp;

Kevin-

bjsorrentino · May 21, 2013, 4:09pm

Hi Kevin,

I think this still goes back to where you say…

[lua]

if myName == “box” then

[/lua]

…because “myName” isn’t defined in that scope. You’ve set that as a property for both objects, so you need to specify the same in the collision handler. In this case, it will either be…

[lua]

self.myName

–OR–

event.other.myName

[/lua]

…depending on which object you want to detect is equal to “box”.

Brent

mmk.verheijden · May 21, 2013, 8:32pm

oky so if i understand this is correct ?

function newCollisionHandler( self, event ) &nbsp; &nbsp;print("oh hello i have a Collision") &nbsp; &nbsp;if randomStar.myName == "BCloud1" then &nbsp; &nbsp; &nbsp; &nbsp; print("Collided with star1") &nbsp; &nbsp;elseif event.other.randomStar.myName == "BCloud2" then &nbsp; &nbsp; &nbsp; &nbsp; print("Collided with star2") &nbsp; &nbsp;elseif event.other.randomStar.myName == "BCloud3" then &nbsp; &nbsp; &nbsp; &nbsp; print("Collided with star3") &nbsp; &nbsp;end end &nbsp;

bjsorrentino · May 21, 2013, 8:44pm

Not exactly… in a “local-style” collision, one object is represented by “self” and the other by “event.other”. Those are the direct references to the two objects involved in the collision, so you should deal with them at that level.

mmk.verheijden · May 28, 2013, 12:22pm

oky so i’m trying this in a other way now :

local function onCollision(event) &nbsp;&nbsp;&nbsp;&nbsp;if event.phase == "began" and gameIsActive == true then &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;local obj1 = event.object1;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;local obj2 = event.object2;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;if obj1.name == "player" and obj2.name == "BCloud1" then &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;print("hello 1") &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; elseif obj1.name == "player" and obj2.name == "BCloud2" then &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;print("hello 2") &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; elseif obj1.name == "player" and obj2.name == "BCloud3" then &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;print("hello 3") &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; elseif obj1.name == "player" and obj2.name == "wall" then &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;print("hello im a wall") &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;end &nbsp;&nbsp;&nbsp;&nbsp;end end

Is this correct?

kevin-

bjsorrentino · May 28, 2013, 3:15pm

Hi Kevin,

This looks correct at first glance, but you’ll need to test it. Of course, I assume you’ve applied the “name” property to everything involved in collisions.

Brent

mmk.verheijden · May 28, 2013, 3:21pm

Hi Brent,

i am using randomStar.myName = “BCloud1” the privies errors are gone, but still it is not printing the print(“hello im a wall”)

in the terminal, i will have a play about whit it but its getting there ps dos corona handel the publication to the app/play store as well or only export the native app?

Kevin-

bjsorrentino · May 28, 2013, 3:24pm

Hi Kevin,

Remember to check that the either the player or the wall is a “dynamic” physics body type. At least one body in a collision needs to be dynamic… you can’t get collisions from a “static” + “kinematic” event, for example.

And FYI, Corona doesn’t handle the app submission (to market). It just builds the compiled file which you’ll need to submit to Apple, Google, etc. separately.

Take care,

Brent

mmk.verheijden · May 28, 2013, 3:30pm

physics.addBody(player, “dynamic”, {density=.1, bounce=0.1, friction=.2, radius=10,} )

right ?

bjsorrentino · May 28, 2013, 3:33pm

Yes, that’s correct (to set the body type). If you’re not getting any response from the player+wall collision, then it’s something else. I suggest you check and re-check your collision filter values, working through the chart again, if necessary.

http://forums.coronalabs.com/topic/2128-collision-filters-helper-chart/

Brent

mmk.verheijden · May 28, 2013, 3:49pm

Hi Brent,

changing randomStar.myName = “BCloud1” to randomStar.name = “BCloud1” fixed the problem of not detecting