PointCloudLibrary/pcl

OniGrabber wrongly assumes image data is RGB

Open

#683 aberto em 19 de mai. de 2014

Ver no GitHub
 (4 comments) (0 reactions) (0 assignees)C++ (4.506 forks)batch import
good first issuekind: todomodule: ioneeds: author reply

Métricas do repositório

Stars
 (9.023 stars)
Métricas de merge de PR
 (Mesclagem média 8d 7h) (3 fundiu PRs em 30d)

Description

In openni_device_oni.cpp:

boost::shared_ptr<openni_wrapper::Image> 
openni_wrapper::DeviceONI::getCurrentImage(boost::shared_ptr<xn::ImageMetaData> image_meta_data) const throw ()
{
  return (boost::shared_ptr<openni_wrapper::Image> (new openni_wrapper::ImageRGB24 (image_meta_data)));
}

This leads to weirdly corrupted images when .oni files that have a different encoding are played back in pcl_openni_viewer or when using the -oni switch in kinfu_app.

The openni_wrapper::DeviceONI::DeviceONI constructor should check the XnPixelFormat returned by image_generator_.GetPixelFormat() and pick the correct Image type, or at least throw an error if that's not RGB instead of wrongly interpreting the data.

Guia do colaborador

Direção de pesquisa
Inspecione o método `openni wrapper::DeviceONI::getCurrentImage` e garanta que o formato de pixel seja verificado antes de criar o objeto de imagem.
Pilha de tecnologia
None
Domain
computer graphics
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60