Martin Losch

Results 223 comments of Martin Losch

@jm-c > We could decide to change this and define which control var to perturb by setting the control var/file name in data.grdchk (as one of the ctrl var/file name...

I can change this bullet point and add that we will not change this (and remove the "ticklable" bullet). What about the last one (`wunit.bib`)?

Hi @DanJonesOcean, more documentation is always good. Please go ahead and draft something for review!

This is the `FAST_BYTESWAP` part of the code. When compiling with `-fconvert=big-endian`, the byte swap code (written at a time, when some compilers could do the byteswapping on their own)...

@menemenlis I suggest that we remove this code, as it does not seem to work (anymore).

@menemenlis I'd be happy, if you can do this. I'll review (-:

@averdy 1. I went through `bling_bio_nitrogen.F` and adjusted store directives and reorganised the code a little to make it more straightforward for TAF. I have now a version that uses...

about comment 2: yes, you could do that, but that will give you recomputation warnings that need to be dealt with via store directives. The question is if you really...

> I run fwd, adm TAF and Tapenade testreport for exp. global_oce_biogeo_bling and all look good. I see minor changes to AD-monitor for the TAF one but I guess it's...

There is still issue #763, which this "spring cleaning" does not address. The authors/maintainers/users of `pkg/bling` should decide, if they want to follow the suggestions outlined in #763. I am...