[ playerstage-Patches-3014384 ] camera1394: corrected documentation, use libdc1394s debayer

"SourceForge.net" <[email protected]>
Newsgroups gmane.science.robotics.playerstage
Message-ID <[email protected]>
Patches item #3014384, was opened at 2010-06-10 16:22
Message generated for change (Tracker Item Submitted) made by bst00
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=433166&aid=3014384&group_id=42445

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Player
Group: Release-3.0
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Sebastian Scherer (bst00)
Assigned to: Brian Gerkey (gerkey)
Summary: camera1394: corrected documentation, use libdc1394s debayer

Initial Comment:
This patch changes two things:
- The documentation was wrong about the configuration parameters brightness, exposure, and shutter: It said you should set it to "auto" for automatic mode. This will not work as the above parameters are stored in IntPropertes, i.e. "auto" will not be recognized as a valid value and will be set to 0 instead of PROPERTY_AUTO (defined as -1) as it should. This patch changes the documentation to reflect that the parameters mentioned should be set to -1 for automatic mode.

- camera1394 currently uses its own very simple debayering methods, even though libdc1394 provides a plethora of debayering methods itself: some with much better results. I changed camera1394 to support all of libdc1394 debayering methods. 

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=433166&aid=3014384&group_id=42445

------------------------------------------------------------------------------
This SF.net email is sponsored by Sprint
What will you do first with EVO, the first 4G phone?
Visit sprint.com/first -- http://p.sf.net/sfu/sprint-com-first
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.