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
Change DALIOpType to dali::OpType enum class #597
Conversation
Build 662132 |
eb3cbd0
to
40662fc
Compare
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.
Minor comments. Looks mostly ok.
dali/test/dali_test_matching.h
Outdated
|
||
|
||
DALIOpType m_nOpType = DALI_GPU; | ||
OpType m_nOpType = OpType::GPU; |
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 get rid of m_n
prefix. First of all n
should be for number of
which is not the case here.
@@ -30,7 +30,7 @@ | |||
|
|||
namespace dali { | |||
|
|||
typedef int64 NodeID; | |||
using NodeID = int64_t; |
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.
I thought you were renaming it in another PR, weren't you? In that case, you may leave the typedef here and spare yourself one conflict to resolve while rebasing...
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.
Everything is already rebased.
Build 663208 |
Remove cpu_op(), gpu_op(), etc from OpGraph to simplify access to nodes. Signed-off-by: Krzysztof Lecki <klecki@nvidia.com>
Build 663348 |
Remove cpu_op(), gpu_op(), etc from OpGraph to simplify access to nodes. Signed-off-by: Krzysztof Lecki <klecki@nvidia.com>
Remove cpu_op(), gpu_op(), etc from OpGraph
to simplify access to nodes.
Signed-off-by: Krzysztof Lecki klecki@nvidia.com