Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@Eldies
Copy link

@Eldies Eldies commented Aug 5, 2025

Summary

  1. If StreamDatasetStorage has only one subset, it's StreamSubset should not calculate length itself and should instead use StreamDatasetStorage length
  2. If StreamDatasetStorage has no transformations, it can use its source length

How to test

Checklist

  • I have added unit tests to cover my changes.​
  • I have added integration tests to cover my changes.​
  • I have added the description of my changes into CHANGELOG.​
  • I have updated the documentation accordingly

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) 2025 CVAT.ai Corporation
#
# SPDX-License-Identifier: MIT

@sonarqubecloud
Copy link

sonarqubecloud bot commented Aug 5, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
E Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@zhiltsov-max zhiltsov-max merged commit 26d5f35 into develop Aug 11, 2025
5 of 6 checks passed
@zhiltsov-max zhiltsov-max deleted the dl/optimizations-for-streaming branch August 11, 2025 18:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants