Closed WPringle closed 2 years ago
@SorooshMani-NOAA No reason for not merging in, only Zach didn't have time to review and fix any conflicts. Yes, the karhunenloeve_polynomialchaos scripts replace the old kl_surrogate ones, and would be nice to have that in main as they were used in the paper.
@WPringle, the changes in this branch seems to be mostly in
examples
, and a few lines of modifications in the main code. I don't think it's going to cause any conflicts in merging. Is there any reason for not merging them tomain
other than failed lint tests? If so, we can spend some time on fixing the lint errors and then merge to main. So thekarhunenloeve_polynomialchaos
scripts fully replace the functionality ofkl_surrogate
scripts?