Skip to content
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

ls_method = 1 bugs #4

Open
barronh opened this issue Oct 15, 2020 · 0 comments
Open

ls_method = 1 bugs #4

barronh opened this issue Oct 15, 2020 · 0 comments

Comments

@barronh
Copy link

barronh commented Oct 15, 2020

ls_method = 1 does not appear to be operational. There are two major bugs that occur when using ls_method = 1 with correl or covar and gauss

  1. arrays used within loop are deallocated after the first bin. This causes a segmentation fault.

deallocate (field_2d)

deallocate (cov)

deallocate (cor)

deallocate (mean)

deallocate (var)

  1. cov is passed directly to get_covariance_masked rather than passing as cov(b, :). It is not clear what the effect would be

Also, ls_method = 1 appears to only work with single level variables. Is this as intended?

I am pursuing ls_method = 1 instead of 2 because I am getting surprising results and wanted to explore other options.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant