#18357: Unicode Art
-------------------------------------+-------------------------------------
       Reporter:  vbraun             |        Owner:
           Type:  enhancement        |       Status:  needs_info
       Priority:  major              |    Milestone:  sage-6.8
      Component:  user interface     |   Resolution:
       Keywords:                     |    Merged in:
        Authors:  Volker Braun       |    Reviewers:  Vincent Delecroix
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
  u/vdelecroix/18357                 |  161c0daca2f4a4752725225dfb9238ffc9692ddf
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------

Comment (by vdelecroix):

 Replying to [comment:28 vbraun]:
 > Aah the evils of trailing whitespace. Of course we shouldn't make any
 improvements to Sage until we have conquered that one.

 These are very annoying. My `git diff` show them in red!

 > 1. Every object should inherit from `SageObject`. Doesn't really matter
 for stuff that is not exposed but why not?

 I thought `SageObject` was meant for mathematical objects. It helps for
 interfacing with third party libraries/softwares and has this `rename`
 feature. But I really do not see the point of using it here... Perhaps for
 the very elegant
 {{{
 def __hash__(self):
     return hash(self.__repr__())
 }}}

 Moreover avoiding it makes clear that this piece of code is not dependent
 on the rest of Sage. And that it can be recycled outside of it and/or
 possible externalized.

 > 5. Of course the bike shed can also be painted pink. Benefit? Feel free
 to change it if you must.

 I do not like "methods for testing purpose".

 > 6. So? Its a good case to show that the code doesn't just apply to
 parenthesis

 Right.

 > 7. Afaik yes, e.g. http://en.wikipedia.org/wiki/Bracket

 Hum. There are fine in the browser! Why unicode have special cases for
 `height=2` but no differenciation for `height` even versus `height` odd!?
 For `height=4` this is ugly
 {{{
 ⎧    ⎧
 ⎨    ⎪
 ⎪    ⎨
 ⎩    ⎩
 }}}

--
Ticket URL: <http://trac.sagemath.org/ticket/18357#comment:29>
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