-
Notifications
You must be signed in to change notification settings - Fork 607
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
Reduce DALI conda package installation time #3995
Conversation
Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
CI MESSAGE: [5136988]: BUILD STARTED |
CI MESSAGE: [5138198]: BUILD STARTED |
- libprotobuf-static >=3.18.0 | ||
- libjpeg-turbo >=2.1.1 | ||
- future | ||
- protobuf |
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.
Hmmm... actually, while touching this one - shouldn't we require <4 ? There were some serious issues with protobuf 4.
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.
We link it statically and we are fine with any version. The problem is with TF compiled with one version and used with other.
CI MESSAGE: [5136988]: BUILD FAILED |
CI MESSAGE: [5136988]: BUILD PASSED |
CI MESSAGE: [5138198]: BUILD PASSED |
- reduces DALI conda package installation time by loosing the package dependency constrains Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
constrains
Signed-off-by: Janusz Lisiecki jlisiecki@nvidia.com
Category:
Other (e.g. Documentation, Tests, Configuration)
Description:
constrains
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