Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

lrs_volume does not work with long numbers #13117

Closed
dkrenn opened this issue Jun 14, 2012 · 4 comments
Closed

lrs_volume does not work with long numbers #13117

dkrenn opened this issue Jun 14, 2012 · 4 comments

Comments

@dkrenn
Copy link
Contributor

dkrenn commented Jun 14, 2012

The following (minimal) example does not work:

sage: prec = 24
sage: Polyhedron(vertices=[[10^prec,0], [0,10^prec], [-10^prec,0], [0,-10^prec]]).lrs_volume(verbose=True)
V-representation
begin
 4 3 rational
 1 1000000000000000000000000 0
 1 0 1000000000000000000000000
 1 -1000000000000000000000000 0
 1 0 -1000000000000000000000000
end
volume

*lrs:lrslib v.4.2b, 2006.10.31(32bit,lrsmp.h)
*Copyright (C) 1995,2006, David Avis   avis@cs.mcgill.ca 
*Input taken from file /home/daniel/.sage//temp/nops/2917//tmp_0
*volume
H-representation
begin
***** 3 rational
 1000000000000000000000000  1 -1 
 1000000000000000000000000  1  1 
Overflow at digits=100
Initialize lrs_mp_init with  n > 100L

Traceback (most recent call last)
...
ValueError: lrs did not return a volume

It seems, that the lines cannot be longer than 100 characters and that that is somewhere hard-coded. It would be good if that can be changed somewhere.

CC: @dimpase @mkoeppe @fchapoton @videlec @jdemeyer

Component: packages: optional

Keywords: lrs volume long numbers

Reviewer: Travis Scrimshaw

Issue created by migration from https://trac.sagemath.org/ticket/13117

@dkrenn dkrenn added this to the sage-5.11 milestone Jun 14, 2012
@jdemeyer jdemeyer modified the milestones: sage-5.11, sage-5.12 Aug 13, 2013
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.1, sage-6.2 Jan 30, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.2, sage-6.3 May 6, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.3, sage-6.4 Aug 10, 2014
@mkoeppe
Copy link
Member

mkoeppe commented Jun 28, 2016

comment:5

This seems to be fixed with the version of lrs installed in current Sage.

sage: prec = 24
sage: P = Polyhedron(vertices=[[10^prec,0], [0,10^prec], [-10^prec,0], [0,-10^prec]])
sage: P.volume(engine='lrs')
2e+48

@mkoeppe mkoeppe removed this from the sage-6.4 milestone Jun 28, 2016
@tscrim
Copy link
Collaborator

tscrim commented Jun 29, 2016

Reviewer: Travis Scrimshaw

@tscrim
Copy link
Collaborator

tscrim commented Jun 29, 2016

comment:6

I agree.

@embray
Copy link
Contributor

embray commented Aug 30, 2016

comment:7

Determined to be invalid/duplicate/wontfix (closing as "wontfix" as a catch-all resolution).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants