Skip to content

Conversation

pmavridis
Copy link
Contributor

@pmavridis pmavridis commented Oct 27, 2020

Purpose of this PR

There is an issue with linking the physical camera FoV with the focal length, discussed in this internal slack thread:
https://unity.slack.com/archives/GHD5LADU7/p1603794331473300
(see also the video from Pierre regarding the issue)

Testing status

  • when changing the FoV the focal length of the physical camera does not change any more, unless the "link fov to phyiscal camera" option is selected.

@pmavridis pmavridis added the HDRP label Oct 27, 2020
@pmavridis pmavridis requested review from a team, RSlysz and sebastienlagarde October 27, 2020 19:21
@pmavridis pmavridis changed the title Hdrp/physical camera fix2 [Hdrp] Fix issue with linking FoV to physical camera Oct 27, 2020
The version number for this package has increased due to a version update of a related graphics package.

### Fixed
- Fixed issue when linking the field of view with the focal length in physical camera
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

move it to 10.2

@sebastienlagarde sebastienlagarde removed the request for review from a team October 29, 2020 20:28
@sebastienlagarde
Copy link
Contributor

Merging for 10.2 package

@sebastienlagarde sebastienlagarde marked this pull request as ready for review October 29, 2020 20:29
@sebastienlagarde sebastienlagarde merged commit aaa9574 into master Oct 29, 2020
@sebastienlagarde sebastienlagarde deleted the HDRP/physical_camera_fix2 branch October 29, 2020 20:29
sebastienlagarde added a commit that referenced this pull request Oct 29, 2020
* Fix linking of the field of view with the focal length in physical camera

* Update changelog

* Update changelog 2

* Revert small change to make behavior similar to URP

* Update changelog - move to 10.2

Co-authored-by: sebastienlagarde <sebastien@unity3d.com>
sebastienlagarde added a commit that referenced this pull request Oct 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants