-
Notifications
You must be signed in to change notification settings - Fork 618
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use different stream base for different videos. #1592
Use different stream base for different videos. #1592
Conversation
Can you add some test for it as well? |
Description: Used for testing NVIDIA/DALI#1592 Signed-off-by: Abhishek Sansanwal <asansanwal@nvidia.com>
Signed-off-by: Abhishek Sansanwal <asansanwal@nvidia.com>
8e716e4
to
865f81c
Compare
CI MESSAGE: [1044086]: BUILD STARTED |
CI MESSAGE: [1044086]: BUILD FAILED |
865f81c
to
d6c7b98
Compare
CI MESSAGE: [1044125]: BUILD STARTED |
CI MESSAGE: [1044125]: BUILD FAILED |
d6c7b98
to
dc0372c
Compare
CI MESSAGE: [1044338]: BUILD STARTED |
CI MESSAGE: [1044338]: BUILD PASSED |
Description: Used for testing NVIDIA/DALI#1592 Signed-off-by: Abhishek Sansanwal <asansanwal@nvidia.com>
Description: Used for testing NVIDIA/DALI#1592 Signed-off-by: Abhishek Sansanwal <asansanwal@nvidia.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Just need to update DALI_EXTRA_VERSION
Signed-off-by: Abhishek Sansanwal <asansanwal@nvidia.com>
dc0372c
to
a29058e
Compare
!build |
CI MESSAGE: [1051621]: BUILD STARTED |
CI MESSAGE: [1051621]: BUILD PASSED |
Signed-off-by: Abhishek Sansanwal asansanwal@nvidia.com
Why we need this PR?
We were using the time_base from the first video for all the videos.
If a dataset contained videos with different frame rates, we would fail to decode as mentioned in The pixels generated from DALI are all 0 #1583
TODO: add testcases.
What happened in this PR?
Fill relevant points, put NA otherwise. Replace anything inside []
[ Explain solution of the problem, a new feature added here. ]
[ Describe here what was changed, added, removed. ]
[ Describe here what is the most important part that reviewers should focus on. ]
[ Describe here if and how this PR is tested. ]
[ Describe here if documentation and examples were updated. ]
JIRA TASK: [Use DALI-XXXX or NA]