-
Notifications
You must be signed in to change notification settings - Fork 113
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
Unit tests for Weighted Arithmetic Mean and Distribution #248
Conversation
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.
Thanks for implementing these tests!
e394bfc
to
473ce87
Compare
- Added unit tests for WAM. Signed-off-by: nathannaveen <42319948+nathannaveen@users.noreply.github.com>
- Added unit tests for distribution Signed-off-by: nathannaveen <42319948+nathannaveen@users.noreply.github.com>
- formatted WAM - formatted wam_test - Edited test for wam_test Addressed Code Review Comments - Included additional tests - Refactored structs within tests - Re-formatted code Signed-off-by: nathannaveen <42319948+nathannaveen@users.noreply.github.com>
Signed-off-by: nathannaveen <42319948+nathannaveen@users.noreply.github.com>
Signed-off-by: nathannaveen <42319948+nathannaveen@users.noreply.github.com>
Signed-off-by: nathannaveen <42319948+nathannaveen@users.noreply.github.com>
c3ecae3
to
534e3e2
Compare
Signed-off-by: nathannaveen <42319948+nathannaveen@users.noreply.github.com>
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.
Looking good, just one minor change!
Signed-off-by: nathannaveen <42319948+nathannaveen@users.noreply.github.com>
Signed-off-by: nathannaveen <42319948+nathannaveen@users.noreply.github.com>
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.
Thanks!
Yay! Thanks for all your reviews! Merged! 🥳 |
* Unit tests for Weighted Arthmetic Mean - Added unit tests for WAM. Signed-off-by: nathannaveen <42319948+nathannaveen@users.noreply.github.com> * Unit tests for distribution - Added unit tests for distribution Signed-off-by: nathannaveen <42319948+nathannaveen@users.noreply.github.com> * Fixed WAM issues - formatted WAM - formatted wam_test - Edited test for wam_test Addressed Code Review Comments - Included additional tests - Refactored structs within tests - Re-formatted code Signed-off-by: nathannaveen <42319948+nathannaveen@users.noreply.github.com> * Refactored Code Signed-off-by: nathannaveen <42319948+nathannaveen@users.noreply.github.com> * Updated based on code review comments Signed-off-by: nathannaveen <42319948+nathannaveen@users.noreply.github.com> * Removed the ignored files Signed-off-by: nathannaveen <42319948+nathannaveen@users.noreply.github.com> * Fixed the validation after unmarshalling Signed-off-by: nathannaveen <42319948+nathannaveen@users.noreply.github.com> * Fixed code review comment Signed-off-by: nathannaveen <42319948+nathannaveen@users.noreply.github.com> * Fixed based on comment Signed-off-by: nathannaveen <42319948+nathannaveen@users.noreply.github.com> Signed-off-by: nathannaveen <42319948+nathannaveen@users.noreply.github.com> Co-authored-by: Caleb Brown <calebbrown@google.com>
Signed-off-by: nathannaveen 42319948+nathannaveen@users.noreply.github.com