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

postprocess: hailo: Handle multiple scaler crops on the ISP outputs #703

Merged
merged 3 commits into from
Aug 29, 2024

Conversation

naushir
Copy link
Collaborator

@naushir naushir commented Aug 6, 2024

Add a HailoPostProcessingStage::ConvertInferenceCoordinates() helper that converts from the inference (low res) image co-ordinate space to the main image space, accounting for different scaler crops.

Fixup the Yolo inference and pose estimation stages to use this new helper.

@naushir naushir marked this pull request as ready for review August 29, 2024 14:27
The change for rpi::ScalerCrops changes the behaviour of how crops are
advertised - we now need a non-RAW output stream to be configured to
return the available crop control.

Signed-off-by: Naushir Patuck <naush@raspberrypi.com>
Add a HailoPostProcessingStage::ConvertInferenceCoordinates() helper
that converts from the inference (low res) image co-ordinate space
to the main image space, accounting for different scaler crops.

Fixup the Yolo inference and pose estimation stages to use this new
helper.

Signed-off-by: Naushir Patuck <naush@raspberrypi.com>
Add a new command line argument "--lores-par" that when set, preserves
the 1:1 aspect ratio of the low res stream. This is only possible on the
PiSP platform. The default behaviour is to have this switch disabled,
allowing for identical behaviour betwen VC4 and PiSP platforms. This
requires the use of the rpi::ScalerCrops vendor control.

This switch can also be triggered via the "rpicam-apps.lores.par" key
in the postprocessing JSON file.

Signed-off-by: Naushir Patuck <naush@raspberrypi.com>
@naushir naushir merged commit efdace5 into main Aug 29, 2024
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant