CABLE-LSM / benchcab

Tool for evaluation of CABLE land surface model
https://benchcab.readthedocs.io/en/latest/
Apache License 2.0
2 stars 4 forks source link

Replace dict of dicts with list of dicts #66

Closed SeanBryan51 closed 1 year ago

SeanBryan51 commented 1 year ago

The realisations and science_configurations keys should be lists instead of key value pairs in the config file.

Add the ability for the user to specify $N > 0$ realisations in the config file.

Fixes #57

Testing

github-actions[bot] commented 1 year ago

PR Preview Action v1.2.0 :---: Preview removed because the pull request was closed. 2023-05-03 07:51 UTC

SeanBryan51 commented 1 year ago

I've resolved merge conflicts with the master branch and have re-ran the following commands:

SeanBryan51 commented 1 year ago

Good catch, I have not run a full test with one or greater than two branches in the config, will do that now...

SeanBryan51 commented 1 year ago

I have now ran benchcab run successfully using 1, 2 or 3 branches in the config file. This should be ok to merge.

ccarouge commented 1 year ago

Great, go and merge.