PCL can't read some newer .ONI files

[sdmiller]

I noticed this a few months ago, but was recently reminded by a colleague:

In openni_device.cpp, we seem to be expecting that any Depth Generator have embedded in it a ShadowValue and NoSampleValue. It seems that newer versions of openni no longer require this to be saved -- so plenty of ONI files return an error when read by PCL.

There's a simple quick fix for this: stop throwing an error, and instead default to something reasonable for these cases (e.g. 0 represents no sample -- is there a similar default for shadow?) Should we do this, or is there a deeper reason for this error?

[gedikli]

Hi,
if we can figure out the version of the ONI file, i would recommend to keep
throwing an exception for the older versions and use default values for
newer ones.
Otherwise, returning an error code and continueing with default values
would work as an workaround too.

-Suat

2014/1/22 Stephen Miller notifications@github.com

I noticed this a few months ago, but was recently reminded by a colleague:

In openni_device.cpp, we seem to be expecting that any Depth Generator
have embedded in it a ShadowValue and NoSampleValue. It seems that newer
versions of openni no longer require this to be saved -- so plenty of ONI
files return an error when read by PCL.

There's a simple quick fix for this: stop throwing an error, and instead
default to something reasonable for these cases (e.g. 0 represents no
sample -- is there a similar default for shadow?) Should we do this, or is
there a deeper reason for this error?

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/461
.

[stale]

Marking this as stale due to 30 days of inactivity. It will be closed in 7 days if no further activity occurs.

[kunaltyagi]

Closing due to lack of interest