Bugfix/kle 2 d

[lohitv96]

Corrected the quasi correlation function computation in KLE 2D sampling code

Description

The original computation of the quasi correlation function was performed by taking diagonals. This is incorrect and a for loop is used.

How Has This Been Tested?

This change introduces more comprehensive tests to test the KLE 2D method. A new test is introduced which compares ensemble correlation value agains the theoretical correlation function.

Types of changes

What types of changes does your code introduce? Put an x in all the boxes that apply:

• [x] Bug fix (non-breaking change which fixes an issue)
• [ ] New feature (non-breaking change which adds functionality)
• [ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

Go over all the following points, and put an x in all the boxes that apply. If you're unsure about any of these, don't hesitate to ask. We're here to help!

• [x] My code follows the code style of this project.
• [ ] My change requires a change to the documentation.
• [ ] I have updated the documentation accordingly.
• [x] I have read the CONTRIBUTING document.
• [x] I have added tests to cover my changes.
• [x] All new and existing tests passed.

[sonarcloud[bot]]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

100.0% Coverage
0.0% Duplication