-
Notifications
You must be signed in to change notification settings - Fork 605
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adjust the filenames of decoder test files and update licenses #2844
Conversation
Make them follow the same naming as readers. Add missing license Signed-off-by: Krzysztof Lecki <klecki@nvidia.com>
Signed-off-by: Krzysztof Lecki <klecki@nvidia.com>
!build |
CI MESSAGE: [2240374]: BUILD STARTED |
CI MESSAGE: [2240374]: BUILD PASSED |
@@ -1,3 +1,17 @@ | |||
# Copyright (c) 2020, NVIDIA CORPORATION. All rights reserved. |
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.
Technicaly, you do edit this file, so this should be 2020-2021
, right? ;)
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.
Technically yes, but I just took the scope of the edits before adding this.
@@ -38,7 +52,7 @@ def check_roi_random_crop(ndim=2, max_batch_size=16, | |||
in_shape = dali.fn.shapes(shape_like_in, dtype=types.INT32) | |||
|
|||
crop_shape = [(crop_min_extent + crop_max_extent) // 2] * ndim if random.choice([True, False]) \ | |||
else fn.random.uniform(range=(crop_min_extent, crop_max_extent + 1), | |||
else fn.random.uniform(range=(crop_min_extent, crop_max_extent + 1), |
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.
please trim the whitespaces after else
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.
Oh, there is double space. I don't want to rerun CI and this will touch the code.
Make them follow the same naming as readers.
Add missing license in several files.
Signed-off-by: Krzysztof Lecki klecki@nvidia.com#### Why we need this PR?
What happened in this PR?
馃憖 spot no license, add it
Python tests.
None, just please remember to add them
CI
Nope
JIRA TASK: [NA]