strands-project / strands_morse

A MORSE-based simulation for the STRANDS project
4 stars 28 forks source link

Add option to ScitosA5 to generate openni stack topics #68

Closed nilsbore closed 10 years ago

nilsbore commented 10 years ago

This adds a few things to be able to get all the topics of the normal openni camera in the simulator. The main drawback is that simulating the cameras in morse at this size and frequency is costly. However, I've found it to work out ok. The original settings are used if you do not set the flag when creating the robot. To run:

If @kunzel could have a look at these changes before they are merge that would be great.

hawesie commented 10 years ago

@kunzel when you give it the ok I'll merge.

nilsbore commented 10 years ago

It would be great if someone could try this out, it works good for me and it improves how the cameras are modeled, with accurate focal lengths etc.

hawesie commented 10 years ago

Waiting on @kunzel

nilsbore commented 10 years ago

I just realized I forgot one thing. Since this uses the openni_wrapper stack to generate the topics, that has to be installed on the system. We should consider moving this from scitos_robot to another package since it is not really robot specific, and in this case will have other uses than just for a physical camera. I'll open an issue for that.

cdondrup commented 10 years ago

See: https://github.com/strands-project/scitos_robot/issues/33

RaresAmbrus commented 10 years ago

I think there is an open issue for that. I'll get on on Friday or next week Monday and move that package somewhere else.

nilsbore commented 10 years ago

Ok, thanks.

kunzel commented 10 years ago

Sorry, I completely forgot about this. I'll check it tonight or tomorrow. Thanks @nilsbore

kunzel commented 10 years ago

Nice work @nilsbore! Thanks a lot!