search

payu-org / payu

A workflow management tool for numerical models on the NCI computing systems
Apache License 2.0
18 stars 25 forks source link

Fix bug with years > 10000 for accessom2 #304

Closed aidanheerdegen closed 3 years ago

aidanheerdegen commented 3 years ago

Closes #303

Moved cice calendar manipulation inside logic which ensures this is only used for drivers which set the run calendar in config.yaml, i.e. access

Will only fix calendar overflow issue for accessom2 driver. Issue will persist for the access driver, but this is no longer widely used.

coveralls commented 3 years ago

Coverage Status

Coverage increased (+0.01%) to 42.199% when pulling 7f8fd9b7b0eeee69e7a0fc9ae72f194ed6f7a1da on issue-303 into a83d60b09f8ff93432cde302d36904edc4b79f16 on master.

aidanheerdegen commented 3 years ago

This is a pretty minor change, and will allow @hakaseh to continue his run, so will merge.