Changes between Initial Version and Version 5 of Ticket #13207


Ignore:
Timestamp:
07/17/12 08:33:07 (9 years ago)
Author:
jdemeyer
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #13207

    • Property Status changed from new to closed
    • Property Component changed from PLEASE CHANGE to interfaces
    • Property Milestone changed from sage-5.2 to sage-duplicate/invalid/wontfix
    • Property Owner changed from tbd to was
    • Property Reviewers changed from to Marshall Hampton
    • Property Resolution changed from to duplicate
  • Ticket #13207 – Description

    initial v5  
    33I made a few changes to the package in a RFE (request for enhancement) to the gfan 0.5 fedora package at https://bugzilla.redhat.com/show_bug.cgi?id=837857. Most noticeable one should be that I added a gfan-permissive.patch to make it work if called as `gfan` instead of erroring out and telling to call it `gfan_bases`. But it could be changed to be called as 'gfan_bases' in `sage/interfaces/gfan.py`.
    44
    5 The major issue I noticed is that, the first previous doctest failure due to it wanting to be called `gfan_bases` now gives a different result in one of the tests:
    6 
    7 {{{
    8 $ sage -t  -force_lib "devel/doc/en/tutorial/tour_advanced.rst"
    9 sage -t -force_lib "devel/doc/en/tutorial/tour_advanced.rst"
    10 **********************************************************************
    11 File "/usr/share/sagemath/devel/doc/en/tutorial/tour_advanced.rst", line 66:
    12     sage: F.reduced_groebner_bases ()
    13 Expected:
    14     [[-c^2 + b*d, -b*c + a*d, -b^2 + a*c],
    15      [c^2 - b*d, -b*c + a*d, -b^2 + a*c],
    16      [c^2 - b*d, b*c - a*d, -b^2 + a*c, -b^3 + a^2*d],
    17      [c^2 - b*d, b*c - a*d, b^3 - a^2*d, -b^2 + a*c],
    18      [c^2 - b*d, b*c - a*d, b^2 - a*c],
    19      [-c^2 + b*d, b^2 - a*c, -b*c + a*d],
    20      [-c^2 + b*d, b*c - a*d, b^2 - a*c, -c^3 + a*d^2],
    21      [c^3 - a*d^2, -c^2 + b*d, b*c - a*d, b^2 - a*c]]
    22 Got:
    23     [[-c^2 + b*d, -b*c + a*d, -b^2 + a*c], [-c^2 + b*d, b^2 - a*c, -b*c + a*d], [-c^2 + b*d, b*c - a*d, b^2 - a*c, -c^3 + a*d^2], [c^3 - a*d^2, -c^2 + b*d, b*c - a*d, b^2 - a*c], [c^2 - b*d, -b*c + a*d, -b^2 + a*c], [c^2 - b*d, b*c - a*d, -b^2 + a*c, -b^3 + a^2*d], [c^2 - b*d, b*c - a*d, b^3 - a^2*d, -b^2 + a*c], [c^2 - b*d, b*c - a*d, b^2 - a*c]]
    24 **********************************************************************
    25 1 items had failures:
    26    1 of   8 in __main__.example_1
    27 ***Test Failed*** 1 failures.
    28 For whitespace errors, see the file /home/pcpa/.sage//tmp/tour_advanced_19053.py
    29          [5.8 s]
    30  
    31 ----------------------------------------------------------------------
    32 The following tests failed:
    33 
    34 
    35         sage -t -force_lib "devel/doc/en/tutorial/tour_advanced.rst"
    36 Total time for all tests: 5.8 seconds
    37 
    38 }}}
    39 For easier noticing the different result, the diff is:
    40 
    41 {{{
    42 $ diff -u ~/{old,new}
    43 --- /home/pcpa/old      2012-07-05 12:27:50.823418301 -0400
    44 +++ /home/pcpa/new      2012-07-05 12:27:56.591418588 -0400
    45 @@ -1,8 +1,8 @@
    46      [[-c^2 + b*d, -b*c + a*d, -b^2 + a*c],
    47 +     [-c^2 + b*d, b^2 - a*c, -b*c + a*d],
    48 +     [-c^2 + b*d, b*c - a*d, b^2 - a*c, -c^3 + a*d^2],
    49 +     [c^3 - a*d^2, -c^2 + b*d, b*c - a*d, b^2 - a*c],
    50       [c^2 - b*d, -b*c + a*d, -b^2 + a*c],
    51       [c^2 - b*d, b*c - a*d, -b^2 + a*c, -b^3 + a^2*d],
    52       [c^2 - b*d, b*c - a*d, b^3 - a^2*d, -b^2 + a*c],
    53 -     [c^2 - b*d, b*c - a*d, b^2 - a*c],
    54 -     [-c^2 + b*d, b^2 - a*c, -b*c + a*d],
    55 -     [-c^2 + b*d, b*c - a*d, b^2 - a*c, -c^3 + a*d^2],
    56 -     [c^3 - a*d^2, -c^2 + b*d, b*c - a*d, b^2 - a*c]]
    57 +     [c^2 - b*d, b*c - a*d, b^2 - a*c]]
    58 }}}
     5See #11395 instead.