leapmotion / VRIntro

Leap Motion's intro to VR
56 stars 19 forks source link

std::invalid_argument exception when updating distortion texture #9

Open jhurlbut opened 9 years ago

jhurlbut commented 9 years ago

I get a std::invalid_argument exception whenever the VRIntro tries to update the distortion texture here: m_distortion.UpdateTexture(GLTexture2PixelDataReference(GL_RG, GL_FLOAT, data, m_distortion.Params().Width()_m_distortion.Params().Height()_8));

It's hard to debug without the source code for GLTexture2PixelDataReference being available but the error appears to be thrown in VRIntro.exe!GLTexture2::VerifyPixelDataOrThrow(const GLTexture2PixelData & pixel_data) Line 126 C++

wmisha commented 9 years ago

Hi Jhurlbut,

Thank you for bringing this to my attention. This is actually a regression I introduced recently, as the code expected a newer version of the libraries.

I updated them just now, please re-download the "DemoLibraries" zip file from https://leapmotion.box.com/s/do2tm39l6d3klsfzt5qa and rebuild the app. You will also need Update 3 of MSVC 2013 to do this; I've updated the instructions to reflect this.

I hope this resolves your problem. I feel your frustration at the inability to step through parts of the code. We also plan to eventually release the source code to GLTexture2PixelDataReference (along with the rest of our internal GL libraries), but for now they aren't in a ready enough state yet.

Thanks for your understanding.

Jimmy - Software Engineer Leap Motion

ruiying commented 9 years ago

the new dependencies crash my compiler (VS2013):optimizing compiler has stopped working...at

components\include\primitives\PrimitiveBase.h(87) model_view.Multiply(SquareMatrixAdaptToDim<4>(global_properties.AffineTransform().AsFullMatrix(), EigenTypes::MATH_TYPE(1)));

On 7 November 2014 07:51, jhe- notifications@github.com wrote:

Hi Jhurlbut,

Thank you for bringing this to my attention. This is actually a regression I introduced recently, as the code expected a newer version of the libraries.

I updated them just now, please re-download the "DemoLibraries" zip file from https://leapmotion.box.com/s/do2tm39l6d3klsfzt5qa and rebuild the app. You will also need Update 3 of MSVC 2013 to do this; I've updated the instructions to reflect this.

I hope this resolves your problem. I feel your frustration at the inability to step through parts of the code. We also plan to eventually release the source code to GLTexture2PixelDataReference (along with the rest of our internal GL libraries), but for now they aren't in a ready enough state yet.

Thanks for your understanding.

Jimmy - Software Engineer Leap Motion

— Reply to this email directly or view it on GitHub https://github.com/leapmotion-examples/VRIntro/issues/9#issuecomment-62109720 .

wmisha commented 9 years ago

Hi Ruiying,

I was getting similar errors before I upgraded to MSVC update 3. if you haven't already, can you try that and see if it goes away?

Thanks, Jimmy