Using Kinetics400 from references raises a deprecation warning

[datumbox]

🐛 Describe the bug

Using Kinetics400 from references leads to a deprecation warning. The fastest way to reproduce:

torchrun --nproc_per_node=8 train.py --data-path /datasets01/kinetics/070618/ --train-dir=val_avi-480p --val-dir=val_avi-480p --batch-size=64 --sync-bn --test-only --weights R2Plus1D_18_Weights.DEFAULT --cache-dataset

./vision/torchvision/io/video.py:160: UserWarning: The pts_unit 'pts' gives wrong results and will be removed in a follow-up version. Please use pts_unit 'sec'.
  warnings.warn(

It seems this is due to the following line:

vision/torchvision/datasets/video_utils.py

Line 327 in b7c59a0

video, audio, info = read_video(video_path, start_pts, end_pts)

Which calls the method with the default parameter for pts_unit="pts" which is deprecated:

vision/torchvision/io/video.py

Lines 155 to 163 in b7c59a0

	if pts_unit == "sec":
	start_offset = int(math.floor(start_offset * (1 / stream.time_base)))
	if end_offset != float("inf"):
	end_offset = int(math.ceil(end_offset * (1 / stream.time_base)))
	else:
	warnings.warn(
	"The pts_unit 'pts' gives wrong results and will be removed in a "
	+ "follow-up version. Please use pts_unit 'sec'."
	)

If the specific option is deprecated, TorchVision shouldn't be using the feature.

Versions

Latest main branch

cc @pmeier @YosuaMichael @bjuncek

[pmeier]

Plus, The Kinetics400 class is also deprecated:

vision/torchvision/datasets/kinetics.py

Lines 308 to 311 in 1ac6e8b

	warnings.warn(
	"The Kinetics400 class is deprecated since 0.12 and will be removed in 0.14."
	"Please use Kinetics(..., num_classes='400') instead."
	)

Not sure why this is not showing up for you though.

[datumbox]

@pmeier Good point. It does. I just missed this because the other warning is spammed 100s of times at the beginning:

UserWarning: The Kinetics400 class is deprecated since 0.12 and will be removed in 0.14.Please use Kinetics(..., num_classes='400') instead.

So we should does this mean that we need to update the reference scripts not to use Kinetics400 then?

[bjuncek]

So this entails of two things:

• fix dataset creation

• adapt videoclips object to avoid warning

Now, since we've had to revert the change because of which we introduced 2nd (see #5489), I'll simply remove the warning for now, and document this in #5720 for further action