1 2012-05-11T00:08:13 *** DasIch_
2 2012-05-11T00:11:04 *** bilal_
3 2012-05-11T00:11:15 *** DasIch
4 2012-05-11T00:11:15 *** bilal
5 2012-05-11T00:11:15 *** bilal_
6 2012-05-11T00:11:15 *** DasIch_
7 2012-05-11T00:26:49 *** RogerHaase
8 2012-05-11T02:43:13 *** spy
9 2012-05-11T03:11:08 *** xjjk
10 2012-05-11T03:18:05 *** xjjk
11 2012-05-11T03:41:16 *** sudo_dirk
12 2012-05-11T03:42:53 *** sudo_dirk1
13 2012-05-11T05:49:09 <xiaq> dreimark: plz see EP
14 2012-05-11T05:49:12 <xiaq> ThomasWaldmann: ^
15 2012-05-11T07:08:24 *** xiaq
16 2012-05-11T07:08:57 *** xiaq
17 2012-05-11T08:00:25 <dreimark> re
18 2012-05-11T08:17:17 <dreimark> xiaq: i answered in the chat there
19 2012-05-11T09:10:36 *** MattMaker
20 2012-05-11T09:38:50 <xiaq> dreimark: saw it, thanks :) now waiting for ThomasWaldmann's reply so that i can tidy EP up
21 2012-05-11T09:51:03 <dreimark> xiaq: :)
22 2012-05-11T10:01:26 <xiaq> dreimark: does moin2 have macro?
23 2012-05-11T10:01:43 <xiaq> (answer my own question) yes
24 2012-05-11T10:03:00 <xiaq> dreimark: moin1 (ab)used macros for many kinds of things - RecentChanges, Comment, etc.
25 2012-05-11T10:03:15 <xiaq> dreimark: moin2 seems to prefer using individual views for that
26 2012-05-11T10:05:53 <xiaq> dreimark: i wonder if i update my summary meta key patch, should i upload again and create another issue on cr?
27 2012-05-11T10:06:12 <xiaq> dreimark: also, regarding your comment on cr:
28 2012-05-11T10:06:51 <xiaq> my point is that, if we stick to CAPITALIZED_CONSTANTS, we won't want to write form['meta_text']
29 2012-05-11T10:07:08 <ThomasWaldmann> moin
30 2012-05-11T10:07:09 <xiaq> instead we should assign a constant to 'meta_text' and use that instead...
31 2012-05-11T10:07:18 <xiaq> which i found too cumbersome
32 2012-05-11T10:07:25 <xiaq> ThomasWaldmann: moin
33 2012-05-11T10:08:15 <ThomasWaldmann> xiaq: if you update code you have submitted to CR, use the same id for it
34 2012-05-11T10:08:22 <ThomasWaldmann> same issue
35 2012-05-11T10:08:59 <xiaq> ah, `upload.py -i xxx` isn't it
36 2012-05-11T10:13:23 <xiaq> ThomasWaldmann: plz see line 14, 64, 122, 180 on EP
37 2012-05-11T10:14:27 <xiaq> ThomasWaldmann: also what's your opinion on CAPITALIZED_CONSTANTS? :)
38 2012-05-11T10:15:01 <xiaq> (ps. i know that constants in python is usually all-cap,
39 2012-05-11T10:15:29 <xiaq> but i doubt the usefulness of making a new constant for each meta key used in the code
40 2012-05-11T10:18:15 <ThomasWaldmann> btw, in general for everybody: if you changed a EP and you want people to look at it, post URL and line numbers
41 2012-05-11T10:18:41 <ThomasWaldmann> (same for CR, wiki, ...)
42 2012-05-11T10:18:47 <xiaq> http://etherpad.osuosl.org/moin-issue-tracker line 16,64,122,180
43 2012-05-11T10:18:58 <ThomasWaldmann> thanks
44 2012-05-11T10:33:13 <dreimark> xiaq: sure more constants, needs a refacroring / review in one go
45 2012-05-11T10:35:21 <dreimark> I am not sure if not using constants simplifies it, currently typos on different places makes life much harder
46 2012-05-11T10:39:56 <xiaq> ThomasWaldmann: moin
47 2012-05-11T10:40:10 * ThomasWaldmann was suggesting to dynamically handles some metadata values based on the values used before for them
48 2012-05-11T10:41:20 <xiaq> ThomasWaldmann: most likely we will want a list of currently available values for classification properties (say "version")
49 2012-05-11T10:41:23 <ThomasWaldmann> so, e.g. if somebody used "open" and "closed" for status before, it would offer: "open", "closed" and [add new value]
50 2012-05-11T10:42:06 <xiaq> ThomasWaldmann: and the value set needs to be cached somewhere
51 2012-05-11T10:42:32 <ThomasWaldmann> caching is something that can be added when needed, we do not need to discuss about that
52 2012-05-11T10:44:13 <xiaq> (afk for ~4min, again...)
53 2012-05-11T10:44:16 <ThomasWaldmann> so what are the pros/cons for that in contrary to having to maintain some wiki pages / config entries with the available value sets?
54 2012-05-11T10:47:39 <ThomasWaldmann> pro: you can just start using it, no need to fill in available values somewhere
55 2012-05-11T10:48:14 <ThomasWaldmann> pro: no extra item/config needed for defining the value sets
56 2012-05-11T10:50:22 <ThomasWaldmann> pro: very flexible, just add values you need. soft security: one can change a unwanted value to some wanted one to make it vanish from the set again
57 2012-05-11T10:52:12 <ThomasWaldmann> con: if one uses it for a long time (e.g. from version 0.1 to 10.0), it might show values that are not really "supported" any more
58 2012-05-11T10:52:31 <xiaq> hm
59 2012-05-11T10:52:47 <xiaq> so agree on dynamic value set for the moment -
60 2012-05-11T10:52:56 <xiaq> what about the set of properties?
61 2012-05-11T10:53:04 <xiaq> version, component, status, etc.
62 2012-05-11T10:53:23 <ThomasWaldmann> i was just thinking about handling that in the same way, but that is a bit more difficult
63 2012-05-11T10:53:57 <xiaq> yeah, and if you let the ticket system handle that automatically
64 2012-05-11T10:54:20 <xiaq> how do you control the order they appear?
65 2012-05-11T10:54:45 <ThomasWaldmann> i think we need some meta-infos per property
66 2012-05-11T10:55:04 <ThomasWaldmann> like type (int, unicode, ...), sort order, ...
67 2012-05-11T10:55:22 <xiaq> yes
68 2012-05-11T10:55:44 <xiaq> i'd prefer to do that explicitly
69 2012-05-11T10:55:55 <xiaq> the idea of allowing the user to add to them at any time
70 2012-05-11T10:56:01 <xiaq> can be implemented on the ui layer
71 2012-05-11T10:56:17 <xiaq> so that they don't need to touch the schema directly
72 2012-05-11T10:56:48 <ThomasWaldmann> well, I think for now we could start with dynamic value sets and just keep the idea of dynamic properties for later
73 2012-05-11T10:58:46 <xiaq> why not start with static value sets and implement dynamic ones later :)
74 2012-05-11T10:58:57 <ThomasWaldmann> (additionally to the already mentioned meta-infos for properties, we could also have suggestions for values [that are not used yet])
75 2012-05-11T10:58:58 <xiaq> as said above the dynamic part may be just some ui work
76 2012-05-11T11:00:07 <ThomasWaldmann> sure, just keep these ideas on the pad
77 2012-05-11T11:00:51 <xiaq> ok, though i need some time to arrange and formalize that
78 2012-05-11T11:01:49 <xiaq> i'll be focus on whoosh stuff during the next 6 days
79 2012-05-11T11:18:31 *** greg_f
80 2012-05-11T11:34:16 <ThomasWaldmann> xiaq: hmm, didn't you update your codereview issue? i only see 1 changeset there.
81 2012-05-11T11:36:36 <xiaq> ThomasWaldmann: not yet
82 2012-05-11T11:38:01 <xiaq> updated
83 2012-05-11T11:46:04 <ThomasWaldmann> reviewed
84 2012-05-11T12:09:22 *** sudo_dirk
85 2012-05-11T12:47:35 *** spy
86 2012-05-11T13:34:45 *** spy
87 2012-05-11T13:49:21 *** DasIch
88 2012-05-11T14:04:54 <ThomasWaldmann> breton: http://etherpad.osuosl.org/moin2-bms looking there now
89 2012-05-11T14:19:21 *** RogerHaase
90 2012-05-11T14:24:58 *** spy
91 2012-05-11T14:25:05 <RogerHaase> ThomasWaldmann: please review http://codereview.appspot.com/6091043/
92 2012-05-11T14:26:18 <RogerHaase> ThomasWaldmann: fix for traceback when macroname is invalid: http://codereview.appspot.com/6208046/
93 2012-05-11T14:40:53 <breton> ThomasWaldmann: thanks, working on it now.
94 2012-05-11T14:42:51 *** dave_largo
95 2012-05-11T14:47:02 <ThomasWaldmann> RogerHaase: you got a uncatched PluginMissingError when using a unknown macro?
96 2012-05-11T14:47:19 <RogerHaase> yes
97 2012-05-11T14:49:47 <ThomasWaldmann> RogerHaase: ok, reviewed that macro exception patch, looks fine
98 2012-05-11T14:50:24 <ThomasWaldmann> hopefully that plugin loader stuff will get replaced this GSOC by a better one, IIRC macros are the last users of that code
99 2012-05-11T14:56:27 <ThomasWaldmann> RogerHaase: submit must be outside dl?
100 2012-05-11T14:57:21 * ThomasWaldmann wonders...
101 2012-05-11T14:58:52 <RogerHaase> This is what I see on login validation: line 174 column 40 - Error: Element input not allowed as child of element dl in this context. (Suppressing further errors from this subtree.)
102 2012-05-11T14:59:15 <ThomasWaldmann> but aren't the other fields there inputs also?
103 2012-05-11T14:59:34 <ThomasWaldmann> ah, there is a dt/dd missing maybe :)
104 2012-05-11T15:00:29 * ThomasWaldmann submitted review for http://codereview.appspot.com/6091043/
105 2012-05-11T15:10:45 *** kanha_
106 2012-05-11T15:11:45 *** kanha
107 2012-05-11T15:15:00 *** DasIch
108 2012-05-11T15:42:44 <ThomasWaldmann> DasIch: http://etherpad.osuosl.org/moin2-PasteBin 91
109 2012-05-11T15:46:52 <DasIch> ThomasWaldmann: I'm tending towards making a report system that allows specifying a reason e.g. Spam|Illegal|Compromising Information
110 2012-05-11T15:47:10 <ThomasWaldmann> good idea, keep it on the EP :)
111 2012-05-11T15:52:44 *** dave_largo
112 2012-05-11T15:58:27 *** MattMaker
113 2012-05-11T16:08:12 *** dave_largo
114 2012-05-11T17:00:01 <RogerHaase> ThomasWaldmann: added comments to http://codereview.appspot.com/6091043/
115 2012-05-11T17:17:09 <ThomasWaldmann> RogerHaase: ok. did you look how the form looks like before/after your change?
116 2012-05-11T17:18:44 <RogerHaase> yes, looks the same.
117 2012-05-11T17:19:03 <ThomasWaldmann> ok, then it looks like finished :)
118 2012-05-11T17:19:06 <ThomasWaldmann> \o/
119 2012-05-11T17:19:30 <RogerHaase> or maybe .........\o/
120 2012-05-11T17:19:48 <ThomasWaldmann> yeah, that was a loooong one
121 2012-05-11T17:39:49 *** DasIch_
122 2012-05-11T17:41:12 *** DasIch
123 2012-05-11T17:41:12 *** DasIch_
124 2012-05-11T17:44:28 <RogerHaase> ThomasWaldmann: pushed to https://bitbucket.org/RogerHaase/moin-2.0
125 2012-05-11T17:47:40 <ThomasWaldmann> [EuroPython] Special grant for students and unemployed < got this by mail
126 2012-05-11T17:48:12 <ThomasWaldmann> anyone interested? grant includes conf ticket and hotel bed.
127 2012-05-11T17:49:20 <ThomasWaldmann> ah, plus there is a diversity grant from google for femaile computer scientists
128 2012-05-11T18:57:32 *** ThomasWaldmann
129 2012-05-11T18:57:32 *** ThomasWaldmann
130 2012-05-11T19:01:02 *** bilal
131 2012-05-11T19:01:51 *** bilal
132 2012-05-11T19:02:33 *** MattMaker
133 2012-05-11T19:04:26 *** bilal
134 2012-05-11T19:32:31 *** greg_f
135 2012-05-11T19:38:38 *** MattMaker
136 2012-05-11T21:27:48 <dreimark> spy: please work on those XXX in the pad
137 2012-05-11T21:31:36 <dreimark> 10 days left to get a good plan
138 2012-05-11T21:43:18 <dreimark> DasIch: please resolve XXX on your EP
139 2012-05-11T21:45:35 <dreimark> bbl
140 2012-05-11T22:06:13 <ThomasWaldmann> xiaq: see http://etherpad.osuosl.org/moin-issue-tracker line 154
141 2012-05-11T22:06:50 <ThomasWaldmann> (i got that idea while trying to write a feature request for whoosh and finding that it can be done with existing features)
142 2012-05-11T22:15:05 <ThomasWaldmann> hmm, would only work for float, not x.y.z version numbers
143 2012-05-11T22:15:13 * ThomasWaldmann writes a whoosh fr
144 2012-05-11T22:35:43 <RogerHaase> ThomasWaldmann: fix for #127 footnote placement ready for review: http://codereview.appspot.com/6195077/
145 2012-05-11T22:39:35 *** RogerHaase
146 2012-05-11T22:39:59 *** RogerHaase
147 2012-05-11T22:44:41 *** MattMaker
148 2012-05-11T22:56:50 *** MattMaker
149 2012-05-11T23:07:56 *** dave_largo
150 2012-05-11T23:31:38 *** RogerHaase
151