num icon indicating copy to clipboard operation
num copied to clipboard

Undocumented exception(s) in the Num module

Open vicuna opened this issue 9 years ago • 4 comments

Original bug ID: 7065 Reporter: SpiceGuid Status: acknowledged (set by @gasche on 2015-11-28T20:19:38Z) Resolution: open Priority: low Severity: feature Target version: later Category: documentation

Bug description

Some people assume they can compute square root or cubic root by providing a 1/2 or 1/3 ratio argument to the Num.power_num function.

Unfortunately they can't :

# #load "nums.cma";;
# open Num;;
# int_of_num (Int 27 **/ (Int 1 // Int 3));;
Exception: Invalid_argument "power_num".

The problem is not they can't, the problem is they assume they can and are encouraged by the silent documentation to do so.

I suggest that all Num functions should be totally documented, including exceptions and conditions that can raise them.

vicuna avatar Nov 28 '15 20:11 vicuna

Comment author: @gasche

Patches to propose documentation would be warmly welcome :-)

vicuna avatar Nov 28 '15 20:11 vicuna

I guess this should be moved to ocaml/num @xavierleroy

nojb avatar Mar 16 '19 18:03 nojb

You're right. I derive great pleasure from moving old issues around!

xavierleroy avatar Mar 16 '19 18:03 xavierleroy

As mentioned earlier, pull requests that propose improvements to the documentation would be warmly welcome.

xavierleroy avatar Oct 25 '23 16:10 xavierleroy