Defined ncid_mask for met data files with multiple sites. Fixes #306.

[rml599gh]

CABLE

Thank you for submitting a pull request to the CABLE Project.

Description

The model crashed when trying to run multiple sites in the met data file. This was because ncid_mask was not defined.

Fixes #306

Type of change

Please delete options that are not relevant.

• [X ] Bug fix

Checklist

• The new content is accessible and located in the appropriate section.
• I have checked that links are valid and point to the intended content.
• I have checked my code/text and corrected any misspellings

Please add a reviewer when ready for review.

---

📚 Documentation preview 📚: https://cable--308.org.readthedocs.build/en/308/

[JhanSrbinovsky]

Have you requested a particular reviewer Rachel? It on the top right of the PR page. I suggest Claire and she can advise of any further testing NRI requires.

[ccarouge]

Normally, we want some test results as part of the pull request. This isn't a case that is covered by benchcab yet (something to add). But considering this is a bug fix with a clear crash/no crash case that's a good enough test for now.

I have added one small change but I'm approving anyway. You can apply the suggestion and then you can merge the PR. The policy is for the author to merge as that gives you a last chance to spot an error.

@rml599gh I'd be curious to get your configuration to add it to benchcab. Then we could later check that we get the same results running the fluxnet sites individually or bundled in the met file.

[access-hive-bot]

This pull request has been mentioned on ACCESS Hive Community Forum. There might be relevant details there:

https://forum.access-hive.org.au/t/cable-site-runs-with-access-forcing/2108/7