sensor_msgs: range: added LASER radiation type #73
Conversation
|
LASER is not a radiation type. So it's not appropriate to add to this enumeration. The reason for this field is to know what types of objects will reflect vs not reflect. Ultrasonic vs infrared have very different behaviour when looking at glass for example. As mentioned, in that comment, lasers can have different types of radiation. We could consider adding VISIBLE and ULTRAVIOLET to complement the existing INFRARED definition. However I am not aware of a "light" based distance measuring device operating in the visible or ultraviolet frequencies. There are also definitely microwave based sensors which might be applicable. However we try not to speculatively update messages until there is a clearly known use case. From what sensor or sensors are you trying to represent the data using this message format? |
Laser - "A laser is a device that emits light through a process of optical amplification based on the stimulated emission of electromagnetic radiation." I accept that radiation definition, but then it should also add NEAR_INFRARED. In this case we also have two kinds of radiation: electromagnetic and mechanical.
The comment was referring to Lidar, which is a specific application for lasers.
The reference to the other types of wavelength/radiotion used by Lidars comes from http://en.wikipedia.org/wiki/Lidar. Type depends on the application/range.
|
|
For both of those just use INFRARED we're not trying to represent the exact wavelength or specific taxonomy. We can use the general definition of INFRARED to cover things between visible and microwave. https://en.wikipedia.org/wiki/Infrared |
|
Will accept then @tfoote. Thanks for your time! Offtopic: is there a possibility of inserting covariance in the message definition? |
Related to mavlink/mavros#292 (comment).
[Jade release]