search

KumarRobotics / ouster_decoder

A low latency decoder for Ouster Lidars
BSD 3-Clause "New" or "Revised" License
17 stars 9 forks source link

[WIP] - Some improvements in README.md #2

Open fcladera opened 2 years ago

fcladera commented 2 years ago

A PR to discuss what to add to the README.md. Do not merge yet.

versatran01 commented 2 years ago

I have made some changes to the README in https://github.com/KumarRobotics/ouster_decoder/tree/unsafe Do you want to retarget that? What's the best way to do this?

fcladera commented 2 years ago

I can copy the changes of 9d3a48191a862c into this branch. Unfortunately, the commit also modifies src/decoder.cpp so I cannot cherry-pick it. Could you do rebase -i HEAD~2 in the unsafe branch to remove the changes in the README there?

Option b is to deal with the merge conflicts later.

versatran01 commented 2 years ago

If you know how to do it for option a, just go ahead and do it. Once the unsafe branch is sufficiently tested (probably today) I will merge it back to main.