Fix 2827

[zhiltsov-max]

Motivation and context

Fixes #2827

• Added skipping of frames outside of the task boundaries in TaskData.group_by_frames

How has this been tested?

Unit tests

Checklist

• I submit my changes into the develop branch
• I have added description of my changes into CHANGELOG file
• I have updated the documentation accordingly
• I have added tests to cover my changes
• I have linked related issues (read github docs)
• I have increased versions of npm packages if it is necessary (cvat-canvas,
  cvat-core, cvat-data and cvat-ui)

License

• I submit my code changes under the same MIT License that covers the project.
  Feel free to contact the maintainers if that's a concern.
• I have updated the license header for each file (see an example below)

# Copyright (C) 2021 Intel Corporation
#
# SPDX-License-Identifier: MIT

[coveralls]

Coverage increased (+0.1%) to 71.735% when pulling 0f6f8a6 on zm/fix-2827 into cc2b8ad on develop.

[bsekachev]

If we put the condition before iteration code (a bit changed):

if shape["frame"] >= end_frame:
    break

We probably do not need additional check in cvat/apps/dataset_manager/bindings.py. Right?

[zhiltsov-max]

I'm not sure if interpolation is the only source of this problem, so I'd better keep the check in bindings.py anyway. I wouldn't remove this check in the interpolation function above, because the code using this function definitely expects at least 1 frame in a track, so putting it before loop would break this convention.