MOM5 icon indicating copy to clipboard operation
MOM5 copied to clipboard

unmask dxt, dxu, dyt, dyu, dxtn, dyte output

Open aekiss opened this issue 2 years ago • 2 comments

closes https://github.com/mom-ocean/MOM5/issues/375

aekiss avatar Feb 06 '23 03:02 aekiss

@aekiss I've had a think about this and I don't think I can endorse this as it stands. It's a change specifically introduced for the benefit of a particular piece of post processing software and you would have to prove that it's not going to break somebody else's software that relies on the current version. Its' a bad habit to get into. I suggest making it a conditional compile for ACCESS or ACCESS-OM only.

russfiedler avatar Feb 08 '23 03:02 russfiedler

OK that's fair enough, I wasn't thinking of the bigger picture. It's hard to imagine anybody needing land masking in these grid fields, but I take your point. I won't bother with conditional compilation - people can use this branch from my fork if they ever need this.

aekiss avatar Feb 09 '23 02:02 aekiss