-
Notifications
You must be signed in to change notification settings - Fork 630
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
Dependency update 02/2025 #5801
Conversation
Signed-off-by: Szymon Karpiński <[email protected]>
Signed-off-by: Szymon Karpiński <[email protected]>
This tag points to the same version as `boost-1.86.0` Signed-off-by: Szymon Karpiński <[email protected]>
Signed-off-by: Szymon Karpiński <[email protected]>
@@ -7,11 +7,11 @@ This part of the repository contains extra dependencies required to build DALI, | |||
+==================================================================+===========================================================================================================================+===================================================================================================================+ | |||
| `Google Benchmark <https://github.com/google/benchmark>`_ | `1.9.1 <https://github.com/google/benchmark/releases/tag/v1.9.1>`_ | `Apache License 2.0 <https://github.com/google/benchmark/blob/master/LICENSE>`_ | | |||
+------------------------------------------------------------------+---------------------------------------------------------------------------------------------------------------------------+-------------------------------------------------------------------------------------------------------------------+ | |||
| `Boost Preprocessor <https://github.com/boostorg/preprocessor>`_ | `1.86.0 <https://github.com/boostorg/preprocessor/releases/tag/boost-1.86.0>`_ | `Boost Software License 1.0 <https://github.com/boostorg/boost/blob/master/LICENSE_1_0.txt>`_ | | |||
| `Boost Preprocessor <https://github.com/boostorg/preprocessor>`_ | `1.87.0 <https://github.com/boostorg/preprocessor/releases/tag/boost-1.87.0>`_ | `Boost Software License 1.0 <https://github.com/boostorg/boost/blob/master/LICENSE_1_0.txt>`_ | |
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.
Tag 1.87.0
points to the same commit as 1.86.0
(see https://github.com/boostorg/preprocessor/tags), so this is just a cosmetic change.
Signed-off-by: Szymon Karpiński <[email protected]>
CI MESSAGE: [23435016]: BUILD STARTED |
CI MESSAGE: [23441741]: BUILD STARTED |
CI MESSAGE: [23435016]: BUILD PASSED |
CI MESSAGE: [23441741]: BUILD FAILED |
CI MESSAGE: [23441741]: BUILD PASSED |
@@ -87,7 +87,7 @@ For more details please check the | |||
# the latest astunparse (1.6.3) doesn't work with any other six than | |||
# 1.16 on python 3.12 due to import six.moves | |||
'six >= 1.16, <= 1.16', | |||
'dm-tree <= 0.1.8', | |||
'dm-tree <= 0.1.9', |
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.
nit, we have those now in https://github.com/NVIDIA/DALI/blame/main/conda/dali_python_bindings/recipe/meta.yaml
too. Although, they seem to bound the version the other way round. :D
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.
That's right. Not sure why it's like that, but with >=0.1.8
they have already picked 0.1.9
even without my update
Category:
Other (e.g. Documentation, Tests, Configuration)
Description:
See also: NVIDIA/DALI_deps#127
Additional information:
Affected modules and functionalities:
Key points relevant for the review:
Tests:
Checklist
Documentation
DALI team only
Requirements
REQ IDs: N/A
JIRA TASK: N/A
DALI-4168