#9623: interacts for high school level education
----------------------------------------------------------------------------------+
   Reporter:  schilly                                                           
  |       Owner:  itolkov, jason
       Type:  enhancement                                                       
  |      Status:  needs_review  
   Priority:  major                                                             
  |   Milestone:  sage-4.6.2    
  Component:  interact                                                          
  |    Keywords:                
     Author:  Lauri Ruotsalainen, Harald Schilly, Robert Marik, Marshall 
Hampton  |    Upstream:  N/A           
   Reviewer:  Jason Grout, Karl-Dieter Crisman, John Thurber                    
  |      Merged:                
Work_issues:                                                                    
  |  
----------------------------------------------------------------------------------+
Changes (by kcrisman):

  * reviewer:  Jason Grout => Jason Grout, Karl-Dieter Crisman, John
               Thurber


Comment:

 I am really glad I looked at these.  In general, they are really nice, and
 ready for prime time - and SO much easier to use than sifting through the
 wiki!

 But oh my goodness... SO many minor things that still had to be taken care
 of.  I apologize for not creating a new diff - I had already started
 making the changes quite a bit when I realized I should have just made a
 reviewer patch.  I tried to make the commit message maximally informative,
 though.  There is already too much stuff on here to try to avoid the patch
 bomb; in retrospect, we should have added these a little more
 incrementally so as to avoid this situation.

 So, '''very''' unfortunately, I must still ask for review on this.  There
 are too many changes.  '''BUT''' I feel like many of them are ones which
 would have impacted the professionalism pretty significantly.   Read on.

 Things I fixed:

  * We needed to import `fast_callable` for the Julia set one, for whatever
 reason.

  * I removed the hemisphere, tried to improve the wording of the coin as
 well as I could - it's a nice example, in fact.

  * Many formatting things, $, ticks in doc, confusing wording, bad
 parenthesis in label of difference quotient, etc.

  * Fixed bad thing in function tool where deprecation warning appeared
 because of `f(x*a)`-type stuff in the code.

  * The two formulas on the Simpson one were on top of each other in Safari
 (not FF).  Since the author used the equals sign incorrectly in any case,
 I added some words to clarify it was an approximation, and that fixed it.
 I think Safari may not like math without words between math (see below for
 another possible example).  Same on trapezoid.

  *  In fact, the entire "Area" line in the trigonometry one was gone on
 Safari, probably because of `\text`, which I removed.  Also note the
 triangle was called `A`, not `ABC` as it should have been.

 PS - I love the fact you can sage -b while the notebook is still running.
 This would have been impossible otherwise.

 Things I did not fix:

  * Really annoying thing where using these seems to reset my worksheet
 every so often - I mean it just stops and returns me to the top.  This
 could be because I had the worksheet open in FF and Safari at the same
 time.  (?)

  * Annoying thing that I could not access the documentation for an
 interact in the notebook once I had already viewed that particular
 interact.   This might be a general interact bug.

  * I could not easily figure out how to get this in the reference manual;
 I think I have to create a new directory and a .rst file that will somehow
 autogenerate, but I don't know how to do this (though I'm sure it's easy).
 The original patch of Harald's did not work for me.

  * Polar prime spiral consistently complains about failing to evaluate the
 function at various points.  I had no idea how to fix this, since there
 are so many functions I couldn't easily identify which one wasn't
 evaluating, though perhaps it's one of the parametric plots (makes most
 sense).

  * Trig properties angle and circle and text ALL fail to format properly
 in Safari (not FF).

  * The unit circle one should somehow have formatted `pi`, but I still
 don't know how to do that.

  * Julia set thing doesn't zoom when you zoom, unless you zoom a lot.
 Instead it just cuts off the picture for the zoom area.  Maybe it needs a
 simultaneous plot point uppage?  Incidentally, this one could REALLY use
 an 'update' button.

  * Julia and Mandelbrot ones look kind of sad compared to easy-to-download
 apps available nowadays, even at max plot points.  I suppose this is the
 limit of what we can do with an interact of this kind.

  * Some of these definitely need a teacher to interpret for someone who is
 brand new to things.  That's not a bug, but worth pointing out, especially
 if there is a way to get these in the reference manual.  For instance, the
 trig on a triangle one is a little weird, because you're putting points on
 a unit circle labeled by degree.  Okay, but then there are still tick
 marks labeled on the axes!  So we're sending a mixed - and weird - message
 with that.  Not a game-stopper, but still somewhat rough.

  * Quad. Eq. one stacks a little too close in the formatting even in the
 opening gambit.

  * Some of the graphers need more flexible ranges.  I'm not sure how to do
 this conceptually, though, since we have to hardcode something in with the
 interacts, and too big a range will make it too hard to zoom.  I guess my
 point is these are not universal.

  * Trapezoid and simpson didn't always immediately respond to the 'from
 keyboard' change.  Maybe you have to click that first, before doing any
 keyboard changes, or it doesn't like it?

  * To me, trig isn't calculus, but I'm not arguing that it can show up
 there, so I left that one in both calculus and geometry.

 So tickets that should be opened after someone (''please''! unless I
 missed something) gives this positive review:

  * Fixing the problem with buttons in multiple rows Jason mentioned.

  * Putting the hemisphere plot back in, if we want to.

  * Polar prime spiral plot point issue.

  * Figure out how to get this in the manual

  * Figuring out how to truly doctest them.  For instance, there could at
 least be a search for deprecation warnings, or even random input typed...
 but at least more than just checking that an html block is formed.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/9623#comment:30>
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 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-trac?hl=en.

Reply via email to