Changes between Initial Version and Version 10 of Ticket #10635


Ignore:
Timestamp:
09/09/11 00:31:37 (10 years ago)
Author:
leif
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #10635

    • Property Status changed from new to positive_review
    • Property Reviewers changed from to Mariah Lenox, William Stein, Simon Spicer
    • Property Keywords sd32 added
    • Property Authors changed from to Christopher Hall
  • Ticket #10635 – Description

    initial v10  
    1010I suggest for this ticket, at a minimum we add a quick "hasattr" check at the top of the current factor function, then leave the existing code.  Then new code can be written to use this.  In the near future somebody can move most of the imports and cases out the current factor function, so it becomes very short, and the code gets put in the right place.
    1111
    12 A major motivation for this is that people create their own new rings R in external code that depends on sage, and want to be able to factor polynomials over R.  They do *not* define a new class for polynomials over R.  It seems silly for them to have to patch the core sage library to get factorization functionality.
     12A major motivation for this is that people create their own new rings R in external code that depends on sage, and want to be able to factor polynomials over R.  They do *not* define a new class for polynomials over R.  It seems silly for them to have to patch the core sage library to get factorization functionality.
     13
     14----
     15
     16Apply only [attachment:trac_10635-new_version_with_tests.patch] to the Sage library.