pickle lacks float('inf')

[anthonydoggett]

BPO	445484
Nosy	@gvanrossum, @tim-one, @brettcannon, @birkenfeld

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = None
closed_at = <Date 2005-07-17.13:11:28.000>
created_at = <Date 2001-07-28.15:21:02.000>
labels = ['type-feature']
title = "pickle lacks float('inf')"
updated_at = <Date 2005-07-17.13:11:28.000>
user = 'https://bugs.python.org/anthonydoggett'

bugs.python.org fields:

activity = <Date 2005-07-17.13:11:28.000>
actor = 'georg.brandl'
assignee = 'none'
closed = True
closed_date = None
closer = None
components = ['None']
creation = <Date 2001-07-28.15:21:02.000>
creator = 'anthonydoggett'
dependencies = []
files = []
hgrepos = []
issue_num = 445484
keywords = []
message_count = 7.0
messages = ['53198', '53199', '53200', '53201', '53202', '53203', '53204']
nosy_count = 5.0
nosy_names = ['gvanrossum', 'tim.peters', 'brett.cannon', 'georg.brandl', 'anthonydoggett']
pr_nums = []
priority = 'low'
resolution = 'fixed'
stage = None
status = 'closed'
superseder = None
type = 'enhancement'
url = 'https://bugs.python.org/issue445484'
versions = []

[anthonydoggett]

Support for float('inf') still appears to be missing in

Python 2.1.1 (#1, Jul 28 2001, 14:15:01)
[GCC egcs-2.91.66 19990314/Linux (egcs-1.1.2 release)]
on linux2

>>> cPickle.dumps(float('inf'), 1)
Traceback (most recent call last):
  File "<stdin>", line 1, in ?
SystemError: frexp() result out of range
>>> pickle.dumps(float('inf'), 1)
Traceback (most recent call last):
  File "<stdin>", line 1, in ?
  File "/var/ajd111/python/lib/python2.1/pickle.py",
line 943, in dumps
    Pickler(file, bin).dump(object)
  File "/var/ajd111/python/lib/python2.1/pickle.py",
line 109, in dump
    self.save(object)
  File "/var/ajd111/python/lib/python2.1/pickle.py",
line 211, in save
    f(self, object)
  File "/var/ajd111/python/lib/python2.1/pickle.py",
line 273, in save_float
    self.write(BINFLOAT + pack('>d', object))
SystemError: frexp() result out of range

Both structmodule.c and cPickle.c require changes.
Surely something like

  if (x == HUGE_VAL) {    /* Inf */
      e = 1024;
      f = 0.0;
  }
  else {
      f = frexp(x, &e);
      ...

and

  if (e == 1024)
      x = HUGE_VAL;		/* Inf */
  else {

is all that is required for all IEEE754 machines?

(structmodule.c requires similar changes for Float
also)

[tim-one]

Logged In: YES
user_id=31435

Note that Python has no intentional support for Infs and
NaNs anywhere; even that float('inf') doesn't blow up when
you do it is a platform accident (it blows up ("invalid
literal") on my box).

Given that, in the absence of a comprehensive plan for
supporting this stuff across the board, and worker bees to
implement it, I downgraded the priority. A good plan would
refactor the code so that dealing with the platform double
and float formats occurs in only one place.

[gvanrossum]

Logged In: YES
user_id=6380

Is there a point in keeping this bug report open
indefinitely? A "won't fix" would make just as much sense.
Maybe add Inf support to PEP-42.

[tim-one]

Logged In: YES
user_id=31435

Changed to Feature Request, and added to new "Non-
accidental 754 support" section of PEP-42.

[brettcannon]

Logged In: YES
user_id=357491

PEP-754 should make sure to deal with this.

[tim-one]

Logged In: YES
user_id=31435

This appears to be outside the scope of PEP-754 (as
defined by the PEP's author -- he's just aiming at symbolic
constants and inquiry functions).

[birkenfeld]

Logged In: YES
user_id=1188172

This is in CVS for 2.5.