Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ping360 segment offset/sector direction #1001

Open
ES-Alexander opened this issue Aug 16, 2021 · 3 comments
Open

Ping360 segment offset/sector direction #1001

ES-Alexander opened this issue Aug 16, 2021 · 3 comments
Labels
enhance good-to-have Something to have in some use cases

Comments

@ES-Alexander
Copy link
Contributor

ES-Alexander commented Aug 16, 2021

Summary

Would be useful to have a segment offset option to set the primary direction of a sector scan.

Additional information

Would enable scanning just a particular side, or behind, instead of the current options which only allow full 360 scanning or in front.

Likely most intuitive to set the sector middle angle (rather than starting/end angle), because then you set the direction angle based on what you want to point at, and set the sector width (currently called 'Sector Angle (degrees)') based on how much you want to show.

@ES-Alexander ES-Alexander changed the title Segment offset Ping360 segment offset/sector direction Aug 17, 2021
@ES-Alexander ES-Alexander added enhance good-to-have Something to have in some use cases labels Aug 17, 2021
@ES-Alexander
Copy link
Contributor Author

Could also be used to support twisted installations of the sensor, where the front of the sensor is not aligned with the front of the vehicle.

Example brought up in this forum post

@ES-Alexander
Copy link
Contributor Author

Apparently

The code supports it, but we removed the UI feature since it appears to be not a common use case.

Would be good if we can add it back under an advanced settings checkbox or something, since it does seem to be important for the people who would actually want to use it. As a workaround in the interim, the removed UI code can just be copy-pasted back in for a custom build with the feature re-enabled.

@ES-Alexander
Copy link
Contributor Author

Could also be used to support twisted installations of the sensor, where the front of the sensor is not aligned with the front of the vehicle.

+1 from this forum post

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhance good-to-have Something to have in some use cases
Projects
None yet
Development

No branches or pull requests

1 participant