Mailing list for all users of the OCaml language and system.
 help / color / mirror / Atom feed
From: Pierre Weis <Pierre.Weis@inria.fr>
To: filliatr@csl.sri.com
Cc: caml-list@inria.fr
Subject: Re: wish: hash function for module Num
Date: Wed, 1 Mar 2000 18:04:34 +0100 (MET)	[thread overview]
Message-ID: <200003011704.SAA04197@pauillac.inria.fr> (raw)
In-Reply-To: <14523.7164.321282.875777@cylinder.csl.sri.com> from "Jean-Christophe Filliatre" at Feb 28, 2000 05:08:12 PM

> [ francais ci-dessous ]
> 
> It would be  nice if the module Num would provide  a hash function for
> type num, with the expected property:
> 
>      eq_num n m  =>  hash_num n = hash_num m
> 
> The generic  hash function cannot be  used, since equal  nums may have
> different  representations, on  which the  generic hash  function will
> return   different   value  (even   with   normalization  after   each
> computation):
> 
> ======================================================================
> # open Num;;
> # let _ = Arith_status.set_normalize_ratio  true;;
> - : unit = ()
> 
> # div_num (Int 1) (Int 1);;
> - : Num.num = Ratio <abstr>
> 
> # Hashtbl.hash (div_num (Int 1) (Int 1));;
> - : int = 504151169
> # Hashtbl.hash (Int 1);;                  
> - : int = 1
> ======================================================================
> 
> It is  quite a mean  trick, and makes  the writing of a  hash function
> rather tedious, since you  have to normalize the representation first.

You just have to write one line of code:

let hash_num n = Hashtbl.hash (normalize_num n);;

However, I agree that the best way to achieve this when nums are
embedded into more complex data structures would be to attach a hash
function to nums (that would just be the preceding hash_num function).

> More  generally, I  would expect  the Num  module to  use  a canonical
> representation (as soon as we set_normalize_ratio).

Yes. We also expected that. I corrected the code of some ``optimized''
functions in the module Num (addition, multiplication and division),
reported the correct versions of Caml Light 0.75.

> == francais =================================================================
> 
> La fonction generique de hachage  d'ocaml ne peut etre utilisee sur le
> type  Num.num,  parce  qu'un  meme  entier  peut  etre  represente  de
> plusieurs manieres differentes, donnant des resultats differents (voir
> exemple ci-dessus).  Une fonction  de hash fournie  par le  module Num
> serait la  bienvenue.

Oui. La fonction en question consiste en 1 seule ligne de code:

let hash_num n = Hashtbl.hash (normalize_num n);;

Cependant, il vaudrait mieux utiliser le nouveau mécanisme qui permet
d'attacher une telle fonction de hachage spécialisée sur les valeurs
d'un type. Ainsi le hachage serait corrrect même si le nombre est à
l'intérieur d'une structure de données plus complexe.

> Plus generalement, il serait  souhaitable que le
> module Num utilise une representation canonique pour chaque rationnel.

Pour des raisons évidentes d'efficacité il ne serait pas souhaitable
que le module Num utilise une représentation canonique pour chaque
rationnel. Cependant, la bibliothèque est bien entendu censée le faire
lorsque l'utilisateur le demande explicitement. C'est donc un bug de
la bibliothèque nums dans Objective Caml: peut-être une optimisation
hâtive lors du portage ? Ou peut-être une correction de la
bibliothèque non reportée dans Objective Caml ?
Quoiqu'il en soit, j'ai reporté les versions de add_num mult_num et
div_num que j'avais debogguées dans Caml Light.

Pierre Weis

INRIA, Projet Cristal, Pierre.Weis@inria.fr, http://cristal.inria.fr/~weis/





      reply	other threads:[~2000-03-01 17:08 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2000-02-29  1:08 Jean-Christophe Filliatre
2000-03-01 17:04 ` Pierre Weis [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200003011704.SAA04197@pauillac.inria.fr \
    --to=pierre.weis@inria.fr \
    --cc=caml-list@inria.fr \
    --cc=filliatr@csl.sri.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox