Hi everybody,

I fixed most of the mentioned issues, see below for details.

> ***
> x86_64-Linux
> ***
>
> sage -t  devel/sage-main/sage/graphs/graph.py
> **********************************************************************
> File "graph.py", line 3465:
>     sage: enum(graphs.CubeGraph(3))
> Expected:
>     6100215452666565930L
> Got:
>     6100215452666565930

Fixed in appended patch by coercing to Integer before returning.

> sage -t  devel/sage-main/sage/schemes/elliptic_curves/ell_rational_field.py
> **********************************************************************
> File "ell_rational_field.py", line 1515:
>     sage: f.period_lattice()
> Expected:
> (1.269209304279553421688794616754547305219492241830608667967136921230408338
>613,
> 0.6346046521397767108443973083772736526097461209153043339835684606152041693
>064 +
> 1.4588166169384952293308896129036752571592434289526651614696187624505378966
>09*I) Got:
> (1.269209304279553421688794616754547305219492241830608667967136921230408338
>6127777226903623059215126073116452962783212874372817003284768439764927140105
>7075,
> 0.6346046521397767108443973083772736526097461209153043339835684606152041693
>0638886134518115296075630365582264813916064371864085016423842198824635700528
>5375 +
> 1.4588166169384952293308896129036752571592434289526651614696187624505378966
>0902872639765673368315820172095257526042401249237362183079269125313009041993
>832*I)
> ********************************************************************** File
> "ell_rational_field.py", line 1520:
>     sage: f.period_lattice()
> Expected:
> (1.088521592904229173504308311539594823105140504301377799086597419750048367
>573,
> 1.7676106702337894758813231444978152337342893789841398371463638100967392018
>10*I) Got: 
> (1.088521592904229173504308311539594823105140504301377799086597419750048367
>5728119672461559129451260417579305643351232486754302413473483910493476008994
>7025,
> 1.7676106702337894758813231444978152337342893789841398371463638100967392018
>0953691706599273805495417094215579677634900614786897226142483706622542207437
>740*I)

Fixed by adding #64-bit and #32-bit comments to doctests.

> In addition, the following tests reported:
>
> Exception pexpect.ExceptionPexpect: ExceptionPexpect() in <bound
> method spawn.__del__ of <pexpect.spawn instance at 0x2aa86767a0>>
> ignored
>
>      devel/sage-main/sage/functions/constants.py
>      devel/sage-main/sage/functions/functions.py
>      devel/sage-main/sage/functions/piecewise.py
>      devel/sage-main/sage/gsl/integration.pyx
>      devel/sage-main/sage/matrix/matrix1.pyx
>      devel/sage-main/sage/rings/contfrac.py

I see those too. The problem is that Python ignores exceptions in __del__ 
calls but the pexpect destructor depends heavily on exceptions. I didn't fix 
those yet because I didn't want to dig into the pexpect stuff, yet.

> Also (reported previously)
>
> sage -t  devel/sage-main/sage/geometry/lattice_polytope.py  [Errno 39]
> Directory not empty: '/home/kate/.sage//tmp/5852/'

I cannot reproduce that one on my machine.

> ***
> ia64-Linux
> ***
> After modifying
>     sage-2.4.1.1/spkg/standard/gap-4.4.9/spkg-install
> so that
>   if [ $? =0 -o `uname -p` = "ia64" ]; then
> becomes
>   if [ $? = 0 -o `uname -p` = "ia64" ]; then
> (In other words, add a space between the equal sign and the 0.
> Necessary so that GAP works on ia64.)

I uploaded a fixed GAP spkg to /home/malb/pkgs/gap-4.4.9.p1.spkg.

Kate, thanks for pointing those bugs out and sorry that nobody acted earlier.
Martin

-- 
name: Martin Albrecht
_pgp: http://pgp.mit.edu:11371/pks/lookup?op=get&search=0x8EF0DC99
_www: http://www.informatik.uni-bremen.de/~malb
_jab: [EMAIL PROTECTED]


--~--~---------~--~----~------------~-------~--~----~
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/sage-support
URLs: http://sage.math.washington.edu/sage/ and http://sage.scipy.org/sage/
-~----------~----~----~----~------~----~------~--~---

# HG changeset patch
# User 'Martin Albrecht <[EMAIL PROTECTED]>'
# Date 1176929165 -7200
# Node ID 9e4a5cd7d8212168b9d9a87b2eea826bf317bd9e
# Parent  a62ae3f310f2a583235aaf13c382f52fa5b276d8
fixing bugs/doctest issues reported by Kate Minola
 * ell_rational_field.py added special doctests for 64-bit systems
 * graph.py coercing to ZZ before returning in enum()

diff -r a62ae3f310f2 -r 9e4a5cd7d821 sage/graphs/graph.py
--- a/sage/graphs/graph.py	Wed Apr 18 15:18:44 2007 +0200
+++ b/sage/graphs/graph.py	Wed Apr 18 22:46:05 2007 +0200
@@ -215,6 +215,9 @@ from sage.structure.sage_object import S
 from sage.structure.sage_object import SageObject
 from sage.plot.plot import Graphics, GraphicPrimitive_NetworkXGraph
 import sage.graphs.graph_fast as graph_fast
+from sage.rings.integer import Integer
+from sage.rings.integer_ring import ZZ
+
 
 class GenericGraph(SageObject):
     """
@@ -1140,7 +1143,6 @@ class Graph(GenericGraph):
         if format == 'graph6':
             if not isinstance(data, str):
                 raise ValueError, 'If input format is graph6, then data must be a string'
-            from sage.rings.integer import Integer
             n = data.find('\n')
             if n == -1:
                 n = len(data)
@@ -3382,7 +3384,7 @@ class DiGraph(GenericGraph):
                             map[vert] = aa
                             break
                 return True, map
-            else:
+            else: 
                 return False, None
         else:
             from sage.graphs.graph_isom import search_tree
@@ -3457,25 +3459,25 @@ def enum(graph):
     EXAMPLES:
         sage: from sage.graphs.graph import enum
         sage: enum(graphs.DodecahedralGraph())
-        646827340296833569479885332381965103655612500627043016896502674924517797573929148319427466126170568392555309533861838850L
+        646827340296833569479885332381965103655612500627043016896502674924517797573929148319427466126170568392555309533861838850
         sage: enum(graphs.MoebiusKantorGraph())
-        29627597595494233374689380190219099810725571659745484382284031717525232288040L
+        29627597595494233374689380190219099810725571659745484382284031717525232288040
         sage: enum(graphs.FlowerSnark())
-        645682215283153372602620320081348424178216159521280462146968720908564261127120716040952785862033320307812724373694972050L
+        645682215283153372602620320081348424178216159521280462146968720908564261127120716040952785862033320307812724373694972050
         sage: enum(graphs.CubeGraph(3))
-        6100215452666565930L
+        6100215452666565930
         sage: enum(graphs.CubeGraph(4))
-        31323620658472264895128471376615338141839885567113523525061169966087480352810L
+        31323620658472264895128471376615338141839885567113523525061169966087480352810
         sage: enum(graphs.CubeGraph(5))
-        56178607138625465573345383656463935701397275938329921399526324254684498525419117323217491887221387354861371989089284563861938014744765036177184164647909535771592043875566488828479926184925998575521710064024379281086266290501476331004707336065735087197243607743454550839234461575558930808225081956823877550090L
+        56178607138625465573345383656463935701397275938329921399526324254684498525419117323217491887221387354861371989089284563861938014744765036177184164647909535771592043875566488828479926184925998575521710064024379281086266290501476331004707336065735087197243607743454550839234461575558930808225081956823877550090
         sage: enum(graphs.CubeGraph(6))
-        17009933328531023098235951265708015080189260525466600242007791872273951170067729430659625711869482140011822425402311004663919203785115296476561677814427201708237805402966561863692388687547518491537427897858240566495945005294876576523289206747123399572439707189803821880345487300688962557172856432472391025950779306221469432919735886988596366979797317084123956762362685536557279604675024249987913439836592296340787741671304722135394212035449285260308821361913500205796919484488876249630521666898413890977354122918711285458724686283296097840711521153201188450783978019001984591992381570913097193343212274205747843852376395748070926193308573472616983062165141386183945049871456376379041631456999916186868438148001405477879591035696239287238767746380404501285533026300096772164676955425088646172718295360584249310479706751274583871684827338312536787740914529353458829503642591918761588296961192261166874864565050490306157300749101788751129640698534818737753110920871293122429238702542726347017441416450649382146313791818349648006634724962025571237208317435310419071153813687071275479812184286929976456778629116002591936357623320676067640749567446551071011889378108453641887998273235139859889734259803684619153716302058849155208478850L
+        17009933328531023098235951265708015080189260525466600242007791872273951170067729430659625711869482140011822425402311004663919203785115296476561677814427201708237805402966561863692388687547518491537427897858240566495945005294876576523289206747123399572439707189803821880345487300688962557172856432472391025950779306221469432919735886988596366979797317084123956762362685536557279604675024249987913439836592296340787741671304722135394212035449285260308821361913500205796919484488876249630521666898413890977354122918711285458724686283296097840711521153201188450783978019001984591992381570913097193343212274205747843852376395748070926193308573472616983062165141386183945049871456376379041631456999916186868438148001405477879591035696239287238767746380404501285533026300096772164676955425088646172718295360584249310479706751274583871684827338312536787740914529353458829503642591918761588296961192261166874864565050490306157300749101788751129640698534818737753110920871293122429238702542726347017441416450649382146313791818349648006634724962025571237208317435310419071153813687071275479812184286929976456778629116002591936357623320676067640749567446551071011889378108453641887998273235139859889734259803684619153716302058849155208478850
     """
     M = graph.am()
     enumeration = 0
     n = graph.order()
     for i, j in M.nonzero_positions():
         enumeration += 1 << ((n-(i+1))*n + n-(j+1))
-    return enumeration
-
-
+    return ZZ(enumeration)
+
+
diff -r a62ae3f310f2 -r 9e4a5cd7d821 sage/interfaces/expect.py
--- a/sage/interfaces/expect.py	Wed Apr 18 15:18:44 2007 +0200
+++ b/sage/interfaces/expect.py	Wed Apr 18 22:46:05 2007 +0200
@@ -808,7 +808,8 @@ class ExpectElement(RingElement):
                 P = self.parent()
                 if not (P is None):
                     P.clear(self._name)
-        except RuntimeError, msg:    # needed to avoid infinite loops in some rare cases
+                
+        except (RuntimeError, pexpect.ExceptionPexpect), msg:    # needed to avoid infinite loops in some rare cases
             #print msg
             pass
 
diff -r a62ae3f310f2 -r 9e4a5cd7d821 sage/schemes/elliptic_curves/ell_rational_field.py
--- a/sage/schemes/elliptic_curves/ell_rational_field.py	Wed Apr 18 15:18:44 2007 +0200
+++ b/sage/schemes/elliptic_curves/ell_rational_field.py	Wed Apr 18 22:46:05 2007 +0200
@@ -1513,12 +1513,15 @@ class EllipticCurve_rational_field(Ellip
         EXAMPLES:
             sage: f = EllipticCurve('11a')
             sage: f.period_lattice()
-            (1.269209304279553421688794616754547305219492241830608667967136921230408338613, 0.6346046521397767108443973083772736526097461209153043339835684606152041693064 + 1.458816616938495229330889612903675257159243428952665161469618762450537896609*I)
+            (1.269209304279553421688794616754547305219492241830608667967136921230408338613, 0.6346046521397767108443973083772736526097461209153043339835684606152041693064 + 1.458816616938495229330889612903675257159243428952665161469618762450537896609*I) # 32-bit
+            (1.26920930427955342168879461675454730521949224183060866796713692123040833861277772269036230592151260731164529627832128743728170032847684397649271401057075, 0.634604652139776710844397308377273652609746120915304333983568460615204169306388861345181152960756303655822648139160643718640850164238421988246357005285375 + 1.45881661693849522933088961290367525715924342895266516146961876245053789660902872639765673368315820172095257526042401249237362183079269125313009041993832*I) # 64-bit
+
             sage: f.period_lattice_is_rectangular()
             False
             sage: f = EllipticCurve('37b')
             sage: f.period_lattice()
-            (1.088521592904229173504308311539594823105140504301377799086597419750048367573, 1.767610670233789475881323144497815233734289378984139837146363810096739201810*I)
+            (1.088521592904229173504308311539594823105140504301377799086597419750048367573, 1.767610670233789475881323144497815233734289378984139837146363810096739201810*I) # 32-bit
+            (1.08852159290422917350430831153959482310514050430137779908659741975004836757281196724615591294512604175793056433512324867543024134734839104934760089947025, 1.76761067023378947588132314449781523373428937898413983714636381009673920180953691706599273805495417094215579677634900614786897226142483706622542207437740*I) # 64-bit
             sage: f.period_lattice_is_rectangular()
             True        
 

Reply via email to