Quality of life improvements and new features

[freeman401]

No description provided.

[FranceBB]

I see what you're trying to do. I would even accept the new defaults being true for ChromaTargets, but I'm not really sure about HiConWave as I think it should at the very least be false.

ColorBars(pixel_type="YV24")

new=VideoTek2()
old=VideoTek()

StackVertical(new, old)

shows

as you can see, after going through

	if (HiConWave) {
		hcluma = luma.Crop(0,20,0,220).Levels(16, 2.0, 128, 0, 255, coring=false)
		luma = Overlay(luma, hcluma, 0, 20)
	}

the steps change completely (see the bottom being now one).

Setting HiConWave to false makes the bottom match the original version

ColorBars(pixel_type="YV24")

new=VideoTek2(HiConWave=false)
old=VideoTek()

StackVertical(new, old)

The whole point of a waveform monitor should be to represent what is in the picture exactly, so I'm not really sure about HiConWave. I can be fine with it, but only if it defaults to false.

Last but not least, I would prefer to have the Rec. info at the bottom and next to the chroma circle as optional like under a verbose info parameter. I like it, don't get me wrong, but the original idea behind this was to mimic what the Tektronix I have in the office was doing.

which doesn't show it by default

and it's also better not to display it when you're working with log like Slog3 Sgamut3 'cause it gives the "false" impression that it's BT709 etc.

[freeman401]

Yes, the default behavior probably shouldn't be changed, despite the introduction of new settings.

I only disagree on one thing. If the chroma targets are displayed, the user definitely should know at all times which ones they're working with. Lack of clear labeling is as bad as false impression. For example, in your case above ColorBars(pixel_type="YV24") actually supplies wrong matrix metadata and consequentially the wrong targets are displayed.