#18286: Region plot fails when there is equation condition
-------------------------+-------------------------------------------------
       Reporter:  yzh    |        Owner:
           Type:         |       Status:  needs_review
  defect                 |    Milestone:  sage-6.7
       Priority:  major  |   Resolution:
      Component:         |    Merged in:
  graphics               |    Reviewers:
       Keywords:         |  Work issues:
  region_plot            |       Commit:
        Authors:  Yuan   |  7e336a1101995eea35abf373639db5d3a397399c
  Zhou                   |     Stopgaps:
Report Upstream:  N/A    |
         Branch:         |
  u/yzh/region_plot      |
   Dependencies:         |
  #16907                 |
-------------------------+-------------------------------------------------

Comment (by ppurka):

 @vdelecroix: It is not quite a duplicate, but yzh has put both the patches
 in the same branch and has the same branch in both the tickets. It is fine
 though, we can close both the tickets at the same time, and close that one
 as a duplicate of this for merging purposes, or remove the branch from the
 other ticket.

 @yzh: I finally had the time to look into it. The patch looks all good to
 me except for the warn part. Can you change that to the following
 {{{
 from warnings import warn
 warn("There are at least 2 equations;......")
 }}}
 This will give the warning just once to the user ''per sage session'',
 instead of each time a similar command is run. After that it is up to the
 user to take note of this warning.

--
Ticket URL: <http://trac.sagemath.org/ticket/18286#comment:5>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

-- 
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.

Reply via email to