CharacteristicMappingMethod / cmm-turbulence

CMM Turbulence code
GNU General Public License v3.0
1 stars 0 forks source link

Merge new variable structure and simple Python testing #52

Closed Arcadia197 closed 1 year ago

Arcadia197 commented 1 year ago

New variable structure.

Python testing

Additional comments can be found in the individual commits. I hope that this clarifies the general structure and helps with streamlining the simulation variables

Arcadia197 commented 1 year ago

I will do a test on Anticythere or Idris and once this succeeds I will merge it

Arcadia197 commented 1 year ago

First test looked okay. The differences need to be adjusted, as they can range up to 10^-10. One variable bug was not yet applied for reference data. Particle P2 for remapping showed higher difference. I should check that

Arcadia197 commented 1 year ago

I noticed issues with saving maps due to OOM error for the MapStack on Idris production run. This needs to be investigated

Arcadia197 commented 1 year ago

I noticed issues with saving maps due to OOM error for the MapStack on Idris production run. This needs to be investigated

Those OOM errors were not related to the changes but rather slurm workload management not allocating enough memory.

Arcadia197 commented 1 year ago

I consider this enough tested to be merged.

Arcadia197 commented 1 year ago

Oh, I accidentely closed without merging