Jump to content

  • Log In with Google      Sign In   
  • Create Account

updating D3DLIGHT9


Old topic!
Guest, the last post of this topic is over 60 days old and at this point you may not reply in this topic. If you wish to continue this conversation start a new topic.

  • You cannot reply to this topic
9 replies to this topic

#1 Prads   Members   -  Reputation: 139

Like
0Likes
Like

Posted 30 January 2011 - 04:58 AM

While trying to update the directional light using SetLight, I get D3DERR_INVALIDCALL error.

Here's what I did:

D3DLIGHT9 updatedLight;

//Update the Light Value
ZeroMemory(&updatedLight, sizeof(updatedLight);
 updatedLight.Ambient = D3DXCOLOR(0.4f, 0.4f, 0.4f, 1.0f);
updatedLight.Diffuse = D3DXCOLOR(0.9f, 0.9f, 0.9f, 1.0f);
updatedLight.Direction = D3DXVECTOR3(0.707f, -0.707f, 0.707f);
updatedLight.Specular = D3DXCOLOR(0, 0, 0, 1.0f);
updatedLight.Type = D3DLIGHT_DIRECTIONAL;

d3dDevice->SetLight(0, &updatedLight);

Can anyone please tell me proper way to do it? Thanks!
My first 3D game: Click Here

Sponsor:

#2 Buckeye   Crossbones+   -  Reputation: 5045

Like
0Likes
Like

Posted 30 January 2011 - 09:46 AM

First, the code you posted shouldn't compile as you're missing a parentheses before the semi-colon in the ZeroMemory line.

Second, in which line does the error occur? Where do you check for errors?

Please don't PM me with questions. Post them in the forums for everyone's benefit, and I can embarrass myself publicly.


#3 Prads   Members   -  Reputation: 139

Like
0Likes
Like

Posted 31 January 2011 - 03:34 AM

Sorry, I was typing fast. That was just an example, not the real code I am using but should show what I am doing.

Error is that the return value of d3dDevice->SetLight is D3DERR_INVALIDCALL. I looked the function up in msdn ( http://msdn.microsof...6(v=vs.85).aspx ) and it says, "If a set of lighting properties exists at this index, it is overwritten by the new properties specified in pLight." And that is exactly what I am trying to do. I had defined a light property in index 0, now I want to update it with new property.
Can you tell me what I am doing wrong? Thanks!
My first 3D game: Click Here

#4 programci_84   Members   -  Reputation: 335

Like
1Likes
Like

Posted 31 January 2011 - 08:09 AM

Does the light that you want to update (at index 0) work properly?

I recommend you to go step by step:
First, call GetLight() and get the light at index 0. Then, only update diffuse color. If no error occurs, then update ambient color. And if no error occurs, update other property etc. etc. etc.

Also, doc says "You can call IDirect3DDevice9::SetLight with new information as needed to update the light's illumination properties.". There's a point here: "light's illumination properties". What do they exactly mean with illumination property?
Are position and direction vectors illumination property?
Or type?

I dunno, but I think that this is where the problem comes.

hth.
-R
There's no "hard", and "the impossible" takes just a little time.

#5 Buckeye   Crossbones+   -  Reputation: 5045

Like
0Likes
Like

Posted 31 January 2011 - 01:02 PM

That was just an example, not the real code

The "example" code (with the correction I noted above) compiles and runs fine.

As an example answer, I'd guess there's an error in your real code. If you want more help, post your "real" code.

If you don't get an error the first time you set the light, post the "real" code for that, too.

Please don't PM me with questions. Post them in the forums for everyone's benefit, and I can embarrass myself publicly.


#6 unbird   Crossbones+   -  Reputation: 5084

Like
0Likes
Like

Posted 31 January 2011 - 02:16 PM

And turn the DX debug runtime on to get more out of that invalid call. Hopefully.

#7 Prads   Members   -  Reputation: 139

Like
0Likes
Like

Posted 31 January 2011 - 10:14 PM

Does the light that you want to update (at index 0) work properly?

I recommend you to go step by step:
First, call GetLight() and get the light at index 0. Then, only update diffuse color. If no error occurs, then update ambient color. And if no error occurs, update other property etc. etc. etc.

Also, doc says "You can call IDirect3DDevice9::SetLight with new information as needed to update the light's illumination properties.". There's a point here: "light's illumination properties". What do they exactly mean with illumination property?
Are position and direction vectors illumination property?
Or type?

I dunno, but I think that this is where the problem comes.

hth.
-R


Oh man, you're genius. I should call GetLight and then only change the properties, and at last call SetLight. Thank you very much!
My first 3D game: Click Here

#8 Evil Steve   Members   -  Reputation: 1983

Like
0Likes
Like

Posted 01 February 2011 - 04:16 PM

And turn the DX debug runtime on to get more out of that invalid call. Hopefully.

This.

Steve Macpherson
Systems Programmer

Rockstar North


#9 Prads   Members   -  Reputation: 139

Like
0Likes
Like

Posted 02 February 2011 - 05:28 AM


And turn the DX debug runtime on to get more out of that invalid call. Hopefully.

This.


It was in the debug mode the whole time, only error it showed me was:


Direct3D9: (ERROR) :SetLight failed.

D3D9 Helper: IDirect3DDevice9::SetLight failed: D3DERR_INVALIDCALL

My first 3D game: Click Here

#10 unbird   Crossbones+   -  Reputation: 5084

Like
0Likes
Like

Posted 02 February 2011 - 06:07 AM

So much for hope :/

From a quick glance I can't spot a violation of the parameters, but sometimes even documentation is wrong or incomplete. So programci_84's approach (combined with a "binary search") from a working light is probably your best shot. Just be aware that reading back states from a device is not always possible (pure device). Looking at the SDK tutorials might also help (e.g. they actually do sometimes set a range for directional lights, though the docs state it's ignored).




Old topic!
Guest, the last post of this topic is over 60 days old and at this point you may not reply in this topic. If you wish to continue this conversation start a new topic.



PARTNERS