MoinMoin: MoinMoinChat/Logs/moin-dev/2010-03-10

   1 2010-03-10T00:13:18  <dreimark> eSyr: http://moinmo.in/ReimarBauer?action=AttachFile&do=get&target=planning.zip
   2 2010-03-10T00:13:24  <dreimark> very small
   3 2010-03-10T00:14:56  <eSyr> looks like empty zip file, isn't it?
   4 2010-03-10T00:15:10  <eSyr> ah, hm.
   5 2010-03-10T00:15:12  <eSyr> not empty.
   6 2010-03-10T00:15:15  <dreimark> no see http://moinmo.in/ReimarBauer?action=AttachFile&do=view&target=planning.zip
   7 2010-03-10T00:15:15  <eSyr> (sorry)
   8 2010-03-10T00:15:28  <eSyr> yes, i see ,it's moin package.
   9 2010-03-10T00:26:04  <eSyr> hm.
  10 2010-03-10T00:27:13  <eSyr> dreimark, table2rows just tries to convert numbers to Decimal?
  11 2010-03-10T00:30:38  <eSyr> dreimark, http://paste.pocoo.org/show/187710/ — apply against repo version
  12 2010-03-10T00:32:23  <xorAxAx> an mdash!
  13 2010-03-10T00:32:51  <eSyr> xorAxAx, of course.
  14 2010-03-10T00:32:59  <eSyr> i have Compose configured.
  15 2010-03-10T00:33:53  <xorAxAx> how do you compose an mdash?
  16 2010-03-10T00:34:07  <eSyr> Multi_Key - - -
  17 2010-03-10T00:34:15  <xorAxAx> ah, 3 times
  18 2010-03-10T00:34:21  <xorAxAx> i tried only twice
  19 2010-03-10T00:34:27  <eSyr> ndash — Multi_key - - .
  20 2010-03-10T00:34:30  <dreimark> eSyr: list: [[u'<a href="/Planning/Falcon">Falcon</a>', (u'<span class="anchor" id="line-1-5"></span>30000 ', u'30000'), (u'<span class="anchor" id="line-1-6"></span>F-16 ', u'F-16')],
  21 2010-03-10T00:34:38  <dreimark> thats table
  22 2010-03-10T00:34:47  <eSyr> ah, sorry, forgot to remove debug output
  23 2010-03-10T00:35:24  <eSyr> hm, no.
  24 2010-03-10T00:35:38  <eSyr> dreimark, yes, i've understood.
  25 2010-03-10T00:36:02  <eSyr> dreimark, try patch above.
  26 2010-03-10T00:36:30  <eSyr> i'm not sure that <a> tag remover is needed, but it's handy for first column in your example, for example.
  27 2010-03-10T00:39:54  <dreimark> that works now
  28 2010-03-10T00:39:56  <ThomasWaldmann> btw, there is #glashammer
  29 2010-03-10T00:40:12  <dreimark> in which timezone is ali ?
  30 2010-03-10T00:40:38  <xorAxAx> UTC
  31 2010-03-10T00:41:00  <xorAxAx> unlike his name would imply
  32 2010-03-10T00:43:47  <dreimark> eSyr: the js code sorttable.js is not in common.js. so i think we should not enable it in CollectLists
  33 2010-03-10T00:48:39  *** eSyr has quit IRC
  34 2010-03-10T00:50:55  *** eSyr has joined #moin-dev
  35 2010-03-10T00:52:41  *** evgSyr has joined #moin-dev
  36 2010-03-10T00:55:39  *** eSyr has quit IRC
  37 2010-03-10T00:55:43  *** evgSyr is now known as eSyr
  38 2010-03-10T01:17:43  <eSyr> ThomasWaldmann, have you anything against two new arrows in diff action (jump to first/last revision)?
  39 2010-03-10T01:20:16  <ThomasWaldmann> if it doesn't get too crowded there...
  40 2010-03-10T01:22:58  <eSyr> hm, i hope, not.
  41 2010-03-10T01:23:12  <eSyr> just one arrow to left pane and one to right.
  42 2010-03-10T01:27:30  <ThomasWaldmann> like |<--  and  -->| ?
  43 2010-03-10T01:27:57  <eSyr> yes.
  44 2010-03-10T01:28:17  <eSyr> ⇤ and ⇥.
  45 2010-03-10T01:33:29  <ThomasWaldmann> are they usually present in the fonts?
  46 2010-03-10T01:34:09  <eSyr> yes.
  47 2010-03-10T01:35:55  <eSyr> it's base plane, not exotic character, so it usually exists.
  48 2010-03-10T01:36:20  <ThomasWaldmann> ok
  49 2010-03-10T02:17:39  <CIA-47> Eugene Syromyatnikov <evgsyr@gmail.com> default * 5620:a02d87f66ce7 1.9/MoinMoin/ (5 files in 5 dirs): util/diff_html.py: Fixed whitespace in generated diff view table so white-space: pre can be used (and whitespace in diffs preserved).
  50 2010-03-10T02:17:41  <CIA-47> Eugene Syromyatnikov <evgsyr@gmail.com> default * 5621:173c6612e240 1.9/MoinMoin/ (5 files in 5 dirs): action/diff.py: Some minor visual improvements ? fixed table layout.
  51 2010-03-10T02:17:43  <CIA-47> Eugene Syromyatnikov <evgsyr@gmail.com> default * 5622:6df6ce00ee99 1.9/MoinMoin/action/diff.py: action/diff.py: Added links to first/last revision.
  52 2010-03-10T02:35:18  <CIA-47> Eugene Syromyatnikov <evgsyr@gmail.com> default * 5623:527bc2b8e4b1 1.9/MoinMoin/web/static/htdocs/ (17 files in 4 dirs): Some css whitespace cleanup.
  53 2010-03-10T02:37:06  <dreimark> gn
  54 2010-03-10T02:37:13  <eSyr> gn, dreimark
  55 2010-03-10T02:37:23  <eSyr> ThomasWaldmann, http://moinmo.in/4ct10n/info/MoinMoinBugs/AlignTopAlignLeft?action=diff&rev2=3&rev1=2 — proper whitespace now :)
  56 2010-03-10T02:38:53  <ThomasWaldmann> aah, much better :)
  57 2010-03-10T02:57:11  *** grzywacz has quit IRC
  58 2010-03-10T02:58:02  <eSyr> ThomasWaldmann, so, what's wrong with this patch?
  59 2010-03-10T02:58:50  <eSyr> we have problem when parsing arg with more than one name-value separator and quote after it.
  60 2010-03-10T02:59:31  <eSyr> because after second name-value separator we shouldn't set noquote to False.
  61 2010-03-10T03:00:24  <eSyr> (in other case, we will have more than two items in name-value tuple and got error at first attempt of unpacking it.
  62 2010-03-10T03:00:27  <ThomasWaldmann> that stuff is simply invalid imho
  63 2010-03-10T03:00:38  <eSyr> mmm, what do you mean?
  64 2010-03-10T03:01:09  <ThomasWaldmann> a="b" c="d" is invalid if the separator is not " "
  65 2010-03-10T03:01:17  <eSyr> parse_quoted_separated_ex is a slightly perverted fsm, but it's generally ok.
  66 2010-03-10T03:01:18  <ThomasWaldmann> but ","
  67 2010-03-10T03:01:29  <eSyr> no,  it isn't.
  68 2010-03-10T03:01:38  <eSyr> i mean, semantically may be.
  69 2010-03-10T03:01:45  <eSyr> but not syntatically.
  70 2010-03-10T03:01:51  <ThomasWaldmann> the " after b terminates the string, otherwise it would have to be ""
  71 2010-03-10T03:02:18  <eSyr> hm, at least in shell it isn't.
  72 2010-03-10T03:02:47  <eSyr> ok, we can raise exception there.
  73 2010-03-10T03:03:12  <ThomasWaldmann> in case of doubt, ask johill :)
  74 2010-03-10T03:03:38  <eSyr> % echo "'"a'b'
  75 2010-03-10T03:03:38  <eSyr> 'ab
  76 2010-03-10T03:03:59  <eSyr> ok, i'll ask him.
  77 2010-03-10T03:04:30  <ThomasWaldmann> btw, the bug report maybe should mention the missing ","
  78 2010-03-10T03:04:58  <ThomasWaldmann> it would be much less confusing then :)
  79 2010-03-10T03:05:34  <eSyr> well, i think now the main problem is that server fails with 500 :)
  80 2010-03-10T03:05:54  <eSyr> (when such erroneous string is given)
  81 2010-03-10T03:11:17  <ThomasWaldmann> should give something like syntax error
  82 2010-03-10T03:21:25  <CIA-47> Eugene Syromyatnikov <evgsyr@gmail.com> default * 5624:ed7044b59397 1.9/MoinMoin/parser/highlight.py: parser/highlight.py: More proper fix of MoinMoinBugs/LineNumberSpansForProcessInstructionsMissed (previous fix broke line number switching js).
  83 2010-03-10T04:20:51  *** ronny has joined #moin-dev
  84 2010-03-10T04:21:05  *** serek has joined #moin-dev
  85 2010-03-10T04:25:37  *** ronny_ has quit IRC
  86 2010-03-10T04:25:38  *** ser has quit IRC
  87 2010-03-10T08:24:00  *** kursor has joined #moin-dev
  88 2010-03-10T08:24:30  *** kursor has left #moin-dev
  89 2010-03-10T09:20:17  *** ronny has quit IRC
  90 2010-03-10T09:20:23  *** ronny has joined #moin-dev
  91 2010-03-10T09:23:00  <dreimark> moin
  92 2010-03-10T10:09:30  <ThomasWaldmann> moin
  93 2010-03-10T10:15:49  <eSyr> moin
  94 2010-03-10T11:01:43  <dreimark> can we know the locale someone used to write something into a wiki page?
  95 2010-03-10T11:03:20  <eSyr> from request object on page saving, i think.
  96 2010-03-10T11:18:29  <dreimark> eSyr: a new idea to get 3.12 sorted and 3,21
  97 2010-03-10T11:18:30  <dreimark> http://paste.pocoo.org/show/187835/
  98 2010-03-10T11:20:08  <eSyr> ah, i see, and you want to get locale from current request?
  99 2010-03-10T11:22:06  <eSyr> http://moinmo.in/MacroMarket/SystemPageLink?action=AttachFile&do=view&target=SystemPageLink-1.9.0-1.py
 100 2010-03-10T11:22:15  <dreimark> if there is no other convention for setting a decimal delimiter (i don't know currently) I can reset it to posix
 101 2010-03-10T11:22:38  <dreimark> because it makes no sense later in python to use , values
 102 2010-03-10T11:22:39  <eSyr> you can use page_lang or request.lang or request.cfg.language_default
 103 2010-03-10T11:23:43  <eSyr> yes, of course.
 104 2010-03-10T11:23:53  <ThomasWaldmann> dreimark: forget about locale or determining it from language
 105 2010-03-10T11:25:08  <dreimark> I think I understand the or det.. but why not using locale ?
 106 2010-03-10T11:25:15  <ThomasWaldmann> that's a pretty hopeless topic (even if you get the languag from the request and lookup the locale, who tells that the human follows that convention when entering data?)
 107 2010-03-10T11:25:27  <dreimark> i know, thought the same
 108 2010-03-10T11:25:59  * ThomasWaldmann for example pretty much always uses decimal point, except if the application forces me no to do that
 109 2010-03-10T11:26:04  <ThomasWaldmann> +t
 110 2010-03-10T11:26:06  <dreimark> but do people have other delimiter ?
 111 2010-03-10T11:26:17  <dreimark> than , or .?
 112 2010-03-10T11:26:44  <ThomasWaldmann> not afaik, but . and , have different uses as decimal and thousands separator
 113 2010-03-10T11:27:11  <eSyr> dreimark, in russian we have , as delimiter, but i have nothing against using . as input.
 114 2010-03-10T11:27:12  <ThomasWaldmann> if you start doing magic, things may get ugly
 115 2010-03-10T11:27:35  <eSyr> it's ok to use locale-dependent delimiter for input when data is stored as number, not as text, as in wiki.
 116 2010-03-10T11:27:54  <eSyr> so i should agree with Thomas.
 117 2010-03-10T11:28:18  <dreimark> ok, then we skip sorting of , values ok? (until someone has a very good idea?)
 118 2010-03-10T11:28:18  * ThomasWaldmann would only use it for output formatting, maybe not even that
 119 2010-03-10T11:31:06  * dreimark lunch bbl
 120 2010-03-10T11:32:13  <eSyr> dreimark, bon appétit
 121 2010-03-10T11:43:27  *** eSyr has quit IRC
 122 2010-03-10T14:07:01  <dreimark> http://paste.pocoo.org/show/187872/
 123 2010-03-10T14:07:41  <dreimark> the diff line 1158 of Attachfile explains why we want tuples.
 124 2010-03-10T14:14:11  <ThomasWaldmann> ok, so don't kill them :)
 125 2010-03-10T14:15:55  <dreimark> that looks much nicer now and it doesn not need special methods loke operator or decimal
 126 2010-03-10T16:15:48  <CIA-47> Reimar Bauer <rb.proj AT googlemail DOT com> default * 5625:78f1be3c8777 1.9/MoinMoin/ (action/AttachFile.py userform/admin.py widget/browser.py):
 127 2010-03-10T16:15:48  <CIA-47> MoinMoin.widget.browser: introduced feature for sorting tables (http://moinmo.in/FeatureRequests/SortableTables).
 128 2010-03-10T16:15:48  <CIA-47> DataBrowserWidget.setData: got two new optional keywords.
 129 2010-03-10T16:15:48  <CIA-47> (Defaults: sort_columns=None, reverse=False)
 130 2010-03-10T16:15:48  <CIA-47> MoinMoin.action.AttachFile.do_admin_browser: feature added to sort by first and second column (Page and Filename)
 131 2010-03-10T16:15:48  <CIA-47> MoinMoin.userform.admin.do_user_browser: feature added to sort by the first column (Username)
 132 2010-03-10T17:20:52  <dreimark> bbl
 133 2010-03-10T18:18:36  <ThomasWaldmann> dreimark: btw, you might need a list of booleans for reverse, not sure
 134 2010-03-10T22:22:32  <dreimark> re
 135 2010-03-10T22:22:42  * dreimark looks
 136 2010-03-10T22:33:55  *** grzywacz has joined #moin-dev
 137 2010-03-10T22:53:53  <dreimark> The reverse parameter is a Boolean value; if it's true, the list is sorted into reverse order.
 138 2010-03-10T22:59:33  <dreimark> we have currently two failing tests
 139 2010-03-10T23:08:45  <ThomasWaldmann> dreimark: someone may want to sort one column forward and another reverse
 140 2010-03-10T23:09:21  <dreimark> how should that work
 141 2010-03-10T23:09:54  <dreimark> you want to sort the table
 142 2010-03-10T23:09:58  <ThomasWaldmann> same way as otherwise
 143 2010-03-10T23:10:58  <dreimark> ah ok that means also a reverse comparison
 144 2010-03-10T23:11:27  <ThomasWaldmann> ?
 145 2010-03-10T23:12:13  <dreimark> i have problems to understand how n-sort could do that
 146 2010-03-10T23:12:34  <dreimark> if one is increasing order for e.f. first column
 147 2010-03-10T23:12:45  <dreimark> and for the second column decreasing
 148 2010-03-10T23:13:17  <ThomasWaldmann> it is no way different than other multi-column sorting
 149 2010-03-10T23:14:16  <ThomasWaldmann> Price Performance
 150 2010-03-10T23:14:26  <ThomasWaldmann> 100 123
 151 2010-03-10T23:14:32  <ThomasWaldmann> 100 111
 152 2010-03-10T23:14:36  <ThomasWaldmann> 100 99
 153 2010-03-10T23:14:44  <ThomasWaldmann> 200 456
 154 2010-03-10T23:14:48  <ThomasWaldmann> 200 321
 155 2010-03-10T23:14:52  <ThomasWaldmann> 200 111
 156 2010-03-10T23:16:51  <dreimark> hmm, ok.
 157 2010-03-10T23:18:22  <dreimark> i look at that tom. because i trace currently xapian.