Use in-place operations for cross-correlations

[dafeda]

No description provided.

[tommyod]

Some comments in Norwegian:

Alt ser riktig ut, men litt plukk som du kan vurdere å gjøre noe med:

• Metoden dokumenterer ikke at X blir mutert, jeg ville skrevet det i caps lock omtrent, fordi det er relativt uvanlig. Aller helst ville jeg hatt et "overwrite" flagg, som default er False. (da er ting litt lettere å teste også)
• Metoden både muterer et input argument, og returnerer referanse til argumentet. Men funksjonene "_corr_to_cov_inplace" muterer et input argument, og returnerer None. Her tenker jeg at vi bør gjøre det samme for å unngå overraskelser. Jeg lener mot å returnere svaret uansett. Å returnere None og mutere et input argument føles veldig lite Python-aktig ut.

Det er jo i experimental, så om du vil merge med en gang så er det OK. Men jeg ville selv ha lagt på et "overwrite" flagg for å være eksplisitt, og alltid returnert referanse til svaret og aldri bare None.

[dafeda]

Thanks @tommyod !
I have implemented your suggestions and also added a change that calculates stds_X and stds_Y only once.

[tommyod]

Looks good to me! Good idea to compute stds once too!

[dafeda]

Ran Drogon on Stavanger node and it ran in about 2h 40min. This is about an hour slower than last time I ran drogon on the Bergen node. Will need to do more testing, but at least it did not run out of memory so merging.