-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
Update Camera FoV as well when using simSetCameraFoV API #3278
Conversation
This works for us. |
814f210
to
9bb2b0c
Compare
9bb2b0c
to
2264a17
Compare
2264a17
to
2d94be7
Compare
52f144b
to
bdb77d0
Compare
Shouldn't be #3573 be added now to the fixes list? |
@jonyMarino I had added it earlier but later on removed it since #3573 deals with the FOV set in settings.json rather than the API. It's a bit of a separate issue that isn't affected by this PR |
Previously only Render texture FoVs were being updated
Thanks for making this fix @rajat2004! Tested on my side and it appears to work just fine |
Is this only fixed for SimMode="ComputerVision"? |
This PR only affects the API call |
Fixes #2977
About
Currently, only Render texture FoVs are being updated. This results in an incorrect FoV in the Camera info. Note that this is a behavioural change, previously when changing the FoV, the main viewport did not change, only the rendered images were affected. This change will cause the viewport to also change if the camera is the main one, such as the front center camera
How Has This Been Tested?
Using the script added in this PR, and the output -
Output
Earlier, the second CameraInfo FoV would be 90.0
Screenshots (if appropriate):
Old FoV viewport -
After FoV update -
Recorded images -
airsim_cv_mode.zip