[Bf-vfx] Tomato camera sensor changes

[Ejner Fergo]

Hi François,

>> I'm sorry I don't understand what you mean? "All the rest parameters"
>> in the camera settings is sensor dimensions (width/height). What is so
>> confusing about that? It's rather simple really, so what needs deeper
>> thought?
>
> its easy to understand when you know what you are talking about, but for
> most user, filmback or sensor size... well first they don't even know that
> param exists and if so, you can do a poll I pretty sure most of them won't
> know what it does affect.
> But I agree with you, it is not a reason to hide it entirely. Sergey is just
> looking for a nice way to put it so it doesn't get to confusing for people
> who don't use it much. Talk to many 3DS Max or other 3D packages user
> (beside maya) about filmgate and you'll see what I mean ;)

Troy answered this best. If Blender should accommodate most users, we
could remove 80% of all settings. But this is a non issue, since most
users will not be affected by having both sensor dimensions available
(they use the default Blender preset - all is as before). Just because
the settings are there doesn't mean users have to learn/use it! And
even if many 3DS Max and others don't know what a filmgate is, these
apps still support it. Having both X and Y dimensions is not
confusing, it's the correct way.

>> The vertical FOV, as you know, is defined by the sensor height, so we
>> need to input this. Preferably the actual dimension as before, or with
>> a device aspect.
>
> yeah but the heigh can be guessed by the width and the pixel ratio, that's
> what he meant by having params working together.
> No matter if you change one or another it will always affect the others.
> width, height, image ratio, pixel ratio

Image resolution and render resolution are two different things. To
get the correct camera VFOV we need either the sensor height or device
aspect (sorry to repeat myself). That means an extra setting is needed
anyway.

>> Why have the vertical FOV as a read-only property? Are importers not
>> as important (no pun..) as exporters? The Channel file format uses the
>> vertical FOV, so this should be able to be written to. The FBX format
>> also uses the vertical FOV both during export and import, so having
>> this as a read-only property is not so helpful.
>
> I agree with that :)

Cool :)

All in all I just don't understand why we can't just use the sensor
patch as it was. The "confusing" part is a non-issue IMHO.

Best regards,

Ejner