Approved

You may replace the Bearing initialization line with

let a, b, c, d, e = Tuple5.of_enum (Enum.init 5 (fun id -> Bearing id))

to avoid warnings.

Haskell: The type of the initial solution should be Integer -> Int -> Integer.

If you have a billion-character string then calling ToLower() on it will create another billion-chararacter string. Most likely, it will be slower to create this second string than calling Char.ToLower() for each character twice.

Approved

Approved

Approved

Do not use global variables. The initial value of result will not be 0 when your solution is invoked several times. Move result inside your function and use nonlocal in your nested function.

Please do not change translations and invalidate existing solutions without any discussion. I will revert your changes.

The documentation does not mention that this method requires Batteries 3.7.0 (CW has 3.6.0) or OCaml 5.1.

I don't see Map.String.of_list in Batteries.

It is a common OCaml practice to raise an error instead of returning optional results. But optional results are also acceptable.

Approved

I suggest to format Z.t values as follows:

msg = Printf.sprintf "total_inc_dec %a" Z.sprint x in