[FEAT] Implemented covariance calculation Closes #48#66
Open
jiyo4476 wants to merge 10 commits intoNumPower:mainfrom
Open
[FEAT] Implemented covariance calculation Closes #48#66jiyo4476 wants to merge 10 commits intoNumPower:mainfrom
jiyo4476 wants to merge 10 commits intoNumPower:mainfrom
Conversation
+ Covariance Test cases + NDArray::cov function in numpower to expose to PHP + Covariance implementation without Hstack since not in branch
jiyo4476
changed the title
jiyo4476
changed the title
Combined transpose and matrix multiplication in Covariance function.
Fixed naming of NDArray param for covariance function
- Cleaned up unused variables for covariance exposure function in numpower.c + added argument for inputting matrix with the rows defined as cols
+ Refactored the NDArray_cov function in statistics.c to utilize the slice method for data extraction. + Replaced manual data copying with the slice method to improve code readability and maintainability. + Verified that the function maintains its original functionality and correctness after the refactor.
- Removed the unused a_data array - Removed the unused col_shape array - Ensured that the function maintains its original functionality and correctness. - Verified that all tests pass successfully, confirming the correctness of the changes.
+ Changed name of norm_a to centered_a + Changed name of norm_vectors to centered_vectors - Ensured that the function maintains its original functionality and correctness. - Verified that all tests pass successfully, confirming the correctness of the changes.
+ Replaced efree calls with NDArray_FREE for NDArrays in the covariance function to ensure proper memory management. - Ensured that the function maintains its original functionality and correctness. - Verified that all tests pass successfully, confirming the correctness of the changes.
jiyo4476
force-pushed
the
FEAT-48-Implement-covariance-calculation
branch
from
f696e4c to
af8c992
Compare
…ered variable + Refactored the covariance function in statistics.c to combine the mean calculation and subtraction into a single centered variable. + Simplified code by removing the intermediate mean and subtracted variables - Ensured that the function maintains its original functionality and correctness. - Verified that all tests pass successfully, confirming the correctness of the changes.
… time + Refactored the covariance function in statistics.c to move the allocation of the centered_vectors pointer array closer to its usage. + Reduced the time centered_vectors exists in the heap, improving memory management - Ensured that the function maintains its original functionality and correctness. - Verified that all tests pass successfully, confirming the correctness of the changes.
Contributor
|
so, will this PR be merged? |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Submission Checklist:
Due to the inherent complexity of this library, we created this checklist to remind everyone of the essential steps to have an MR approved depending on the type of change that is made. You can delete this.
export USE_ZEND_ALLOC=1 && make testexport USE_ZEND_ALLOC=0 && make testChange to methods and operations
NDARRAY_VCHECKoption enabled and no VRAM memory leaks were displayed?export NDARRAY_VCHECK=1 && make test