Learning Metrics operators support
All threads resolved!
All threads resolved!
Context
To Support Adam Optimizer, this MR adds operator Sqrt's forward implementation. It also fixes ReduceMean and ReduceSum in case no reduction is needed (encountered in BCE and MSE unit tests)
Modified files
-
SqrtImpl.hpp,SqrtImpl.cpp,SqrtImpl_CUDA_kernels.hpp,SqrtImpl_CUDA_kernels.cuandTest_SqrtImpl.cpp, add Sqrt forward and backward operation; -
ReduceMeanImpl.cpp, fix ReduceMean forward; -
ReduceSumImpl.cpp, fix ReduceSum forward;
TODO
-
Add Sqrt -
Fix ReduceMean -
Fix ReduceSum
Edited by Houssem ROUIS
Merge request reports
Activity
added 5 commits
-
6eac0630...0b246d6a - 3 commits from branch
eclipse/aidge:dev - 8ccffa3a - Merge branch 'dev' of gitlab.eclipse.org:eclipse/aidge/aidge_backend_cuda into...
- 231304c9 - update PowImpl header
-
6eac0630...0b246d6a - 3 commits from branch
added 17 commits
Toggle commit listadded 7 commits
Toggle commit listassigned to @hrouis
added Fix 🔥🔥 TopicOperator labels
added 10 commits
-
e9e6dc4c...0f4c668b - 3 commits from branch
eclipse/aidge:dev - 9da44e3b - add Sqrt operator
- ea20231d - add Sqrt Tests
- f67c6982 - fix ReduceMean when no reduction is needed
- 953870d1 - fix ReduceSum when no reduction is needed
- 8e5ae6eb - update PowImpl header
- 88d1a83d - use AIDGE_Assert
- f80d33b3 - add Sqrt backward Impl
Toggle commit list-
e9e6dc4c...0f4c668b - 3 commits from branch
added 2 commits
added 2 commits
- Resolved by Houssem ROUIS
- Resolved by Houssem ROUIS
added 5 commits
-
85fc4d90...325360b5 - 3 commits from branch
eclipse/aidge:dev - b59092cb - Merge branch 'dev' of gitlab.eclipse.org:eclipse/aidge/aidge_backend_cuda into...
- b05fa723 - remove Sqrt backward random tests
-
85fc4d90...325360b5 - 3 commits from branch
- Resolved by Houssem ROUIS
requested review from @cmoineau
mentioned in commit 9d8dcd08
Please register or sign in to reply