Add Livox Avia in scanners_tls.xml

[jqnm]

No description provided.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 67.15%. Comparing base (025ede7) to head (154d2a3).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #696   +/-   ##
=======================================
  Coverage   67.15%   67.15%           
=======================================
  Files         485      485           
  Lines       15563    15563           
  Branches     1765     1765           
=======================================
  Hits        10451    10451           
  Misses       5112     5112           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[han16nah]

Awesome, thanks @jqnm

Some first comments/additions:

• Can you add "maxNOR" = "3" to the Livox Avia?
• We could think about adding "maxRange_m" to the Livox Avia and DJI Zenmuse L2 scanners (cf. "Detection Range" in the datasheets)
• I think the beam divergence of the Zenmuse L2 is wrong. If we take the bigger divergence of the elliptical footprint (given in https://enterprise.dji.com/de/zenmuse-l2/specs at the FWHM), it would be 0.001 rad, correct? Can you check and if so, add this?
• We should also update the "rangeMin_m" of the L2 to 3 m, according to the datasheet.
• Once the Livox Avia is validated, we can also add it to the scanners_als.xml, here I'd suggest to name it DJI Zenmuse L1

[jqnm]

Awesome, thanks @jqnm

Some first comments/additions:

• Can you add "maxNOR" = "3" to the Livox Avia?
• We could think about adding "maxRange_m" to the Livox Avia and DJI Zenmuse L2 scanners (cf. "Detection Range" in the datasheets)
• I think the beam divergence of the Zenmuse L2 is wrong. If we take the bigger divergence of the elliptical footprint (given in https://enterprise.dji.com/de/zenmuse-l2/specs at the FWHM), it would be 0.001 rad, correct? Can you check and if so, add this?
• We should also update the "rangeMin_m" of the L2 to 3 m, according to the datasheet.
• Once the Livox Avia is validated, we can also add it to the scanners_als.xml, here I'd suggest to name it DJI Zenmuse L1

Everything is done, except the last comment