Java icon indicating copy to clipboard operation
Java copied to clipboard

Implementation of cross-correlation, CrossCorrelation test class, auto-correlation and AutoCorrelation test class

Open AthinaSw opened this issue 2 years ago • 2 comments

  • [x] I have read CONTRIBUTING.md.
  • [x] This pull request is all my own work -- I have not plagiarized it.
  • [x] All filenames are in PascalCase.
  • [x] All functions and variable names follow Java naming conventions.
  • [x] All new algorithms have a URL in their comments that points to Wikipedia or other similar explanations.
  • [x] All new code is formatted with clang-format -i --style=file path/to/your/file.java

AthinaSw avatar Dec 19 '23 14:12 AthinaSw

This solves the issues #4982 and #4983

AthinaSw avatar Dec 19 '23 14:12 AthinaSw

Please fix the PR check image

siriak avatar Dec 26 '23 14:12 siriak

In the details of the "Clang format linter / build" , it says this : --- ./src/test/java/com/thealgorithms/maths/AutoCorrelationTest.java (original) +++ ./src/test/java/com/thealgorithms/maths/AutoCorrelationTest.java (reformatted) @@ -34,4 +34,4 @@ } return result; } -}+} I don't really understand what I sould do here

AthinaSw avatar Jan 02 '24 20:01 AthinaSw

In the details of the "Clang format linter / build" , it says this : --- ./src/test/java/com/thealgorithms/maths/AutoCorrelationTest.java (original) +++ ./src/test/java/com/thealgorithms/maths/AutoCorrelationTest.java (reformatted) @@ -34,4 +34,4 @@ } return result; } -}+} I don't really understand what I sould do here

Ok I fixed it

AthinaSw avatar Jan 02 '24 21:01 AthinaSw

Please fix the PR check image

Done!

AthinaSw avatar Jan 03 '24 16:01 AthinaSw

Looks good, thanks!

You're welcome!

AthinaSw avatar Jan 03 '24 22:01 AthinaSw