1 2011-08-04T00:02:03 *** sinha
2 2011-08-04T01:13:59 *** MattMaker
3 2011-08-04T02:35:21 *** MattMaker
4 2011-08-04T03:04:53 *** MattMaker
5 2011-08-04T04:17:07 *** MattMaker
6 2011-08-04T05:02:42 *** Marchael
7 2011-08-04T05:42:11 *** Marchael
8 2011-08-04T06:05:08 *** Marchael
9 2011-08-04T07:24:35 *** Marchael
10 2011-08-04T08:19:23 <dreimark> TheSheep: have a look at http://moinmo.in/AkashSinha/Gsoc2011Diary/2011-08-03
11 2011-08-04T08:19:58 <dreimark> do you have some ideas to improve this ?
12 2011-08-04T08:21:14 <TheSheep> dreimark: no
13 2011-08-04T08:21:23 <TheSheep> dreimark: what is the problem?
14 2011-08-04T08:22:06 <dreimark> is it too much navigation on that page ?
15 2011-08-04T08:22:35 <TheSheep> dreimark: sure, but that's the norm in MoinMoin anyways :)
16 2011-08-04T08:22:40 <dreimark> hehe
17 2011-08-04T08:24:53 <dreimark> TheSheep: currently i want to use for mass download a "saveas moin mimetype", so a user has to accept only one mimetype to get all different items
18 2011-08-04T08:25:04 *** Marchael
19 2011-08-04T08:25:16 <dreimark> do you know that this makes some problems?
20 2011-08-04T08:29:43 <TheSheep> no idea
21 2011-08-04T08:31:34 <dreimark> i guess we find out once we have that add
22 2011-08-04T08:31:44 <dreimark> ed,
23 2011-08-04T08:44:33 *** Marchael
24 2011-08-04T08:46:07 *** Marchael
25 2011-08-04T09:05:01 *** Marchael
26 2011-08-04T09:09:33 *** Marchael
27 2011-08-04T11:02:14 *** MattMaker
28 2011-08-04T11:05:34 *** m4k3r
29 2011-08-04T11:16:09 *** greg_f
30 2011-08-04T11:18:24 <ThomasWaldmann> moin
31 2011-08-04T11:19:52 *** m4k3r_
32 2011-08-04T11:22:31 *** m4k3r
33 2011-08-04T11:32:30 *** m4k3r_
34 2011-08-04T11:39:15 *** m4k3r
35 2011-08-04T11:40:37 <ThomasWaldmann> http://pitapoison.de:9001/p/moinmoin
36 2011-08-04T11:40:44 <ThomasWaldmann> etherpad lite testing
37 2011-08-04T11:51:33 *** m4k3r
38 2011-08-04T11:51:46 *** m4k3r
39 2011-08-04T12:33:24 <CIA-114> Thomas Waldmann <tw AT waldmann-edv DOT de> default * 311:902378b07e99 2.0/ (5 files in 5 dirs): (log message trimmed)
40 2011-08-04T12:33:24 <CIA-114> simplify interwiki resolution, remove special case for 'Self' / own interwikiname
41 2011-08-04T12:33:24 <CIA-114> Note: it is required now that the interwiki_map contains entries for our
42 2011-08-04T12:33:24 <CIA-114> own wikiname and also for 'Self' (same url):
43 2011-08-04T12:33:24 <CIA-114> interwiki_map = {
44 2011-08-04T12:33:24 <CIA-114> 'Self': 'http://ourserver/ourpath/',
45 2011-08-04T12:33:25 <CIA-114> 'OurInterWikiName': 'http://ourserver/ourpath/',
46 2011-08-04T15:28:08 * ThomasWaldmann braindumps about interwiki
47 2011-08-04T15:44:10 *** m4k3r
48 2011-08-04T16:22:42 *** pkumar
49 2011-08-04T19:10:30 <pkumar> ThomasWaldmann : hi, regarding import_export_xml stuff - there are just two tests which use that;test_wikiutil check for 'is_syspage' for the item in flaskg.storage . I'm not sure if we need to load items in backend only for tests?
50 2011-08-04T19:11:08 <pkumar> in there
51 2011-08-04T19:18:31 *** Marchael
52 2011-08-04T19:19:01 <Marchael> moin
53 2011-08-04T19:29:26 *** raignarok
54 2011-08-04T19:47:07 <ThomasWaldmann> pkumar: well, some tests "count" stuff
55 2011-08-04T19:47:14 <ThomasWaldmann> a) that's bad
56 2011-08-04T19:47:43 <ThomasWaldmann> b) they might need that count from import_export_xml
57 2011-08-04T19:48:04 <ThomasWaldmann> just get rid of that kind of stuff and replace it by something more sane
58 2011-08-04T19:48:17 <ThomasWaldmann> that counter is blocking quite some other refactorings
59 2011-08-04T19:49:58 <brunomartin> ThomasWaldmann: Hi!
60 2011-08-04T19:51:24 *** greg_f
61 2011-08-04T19:54:00 <ThomasWaldmann> hi brunomartin
62 2011-08-04T19:54:11 <brunomartin> ThomasWaldmann: We are going to "release" a new version of our prototype here... I was thinking that would be great to incorporate all last changes to test then! ;-)
63 2011-08-04T19:55:22 <brunomartin> ThomasWaldmann: We have about 500MB of data on moin 2 and about 300 users testing it!
64 2011-08-04T19:57:26 <Marchael> ThomasWaldmann: please review http://codereview.appspot.com/4837052
65 2011-08-04T19:57:29 <ThomasWaldmann> cool, hopefully it doesn't blow up too often
66 2011-08-04T19:58:11 <Marchael> oops
67 2011-08-04T19:58:40 <brunomartin> ThomasWaldmann: hehehe until now, its fine! The Marchael index is working fine too! :-)
68 2011-08-04T19:58:42 <Marchael> ThomasWaldmann: patchset 5
69 2011-08-04T19:59:06 <Marchael> brunomartin: I thought that is whoosh index :)
70 2011-08-04T19:59:17 <pkumar> ThomasWaldmann : do we need to keep those items in backends, if it is test specific ,we can have it for those particular tests. else we can write some alternate tests for the failing ones which serves the purpose but without using those items
71 2011-08-04T19:59:20 <Marchael> and no, it working in some cases
72 2011-08-04T19:59:55 <Marchael> I still fixing my issues, so my stuff is under deep bugfixing
73 2011-08-04T20:00:24 <brunomartin> Marchael: for internal searches, it works fine!
74 2011-08-04T20:00:30 <Marchael> heh
75 2011-08-04T20:00:51 <Marchael> It may working in some cases
76 2011-08-04T20:01:04 <Marchael> but you can look on my issue tracker :)
77 2011-08-04T20:01:22 <brunomartin> ThomasWaldmann: I realised that the latest commits are spread over some repos... so I need to know witch repos should I merge... or if you prefer to merge than to know witch changeset I am testing... what do you think?
78 2011-08-04T20:01:51 <brunomartin> Marchael: I see your issue tracker every day! ;-)
79 2011-08-04T20:01:58 <Marchael> :D
80 2011-08-04T20:02:10 <brunomartin> Marchael: and yours commits too... :-P
81 2011-08-04T20:05:01 *** sinha
82 2011-08-04T20:24:12 <ThomasWaldmann> pkumar: i already noted some weeks ago that the item loading at app startup is expensive. you need a concept to load stuff when it is needed.
83 2011-08-04T20:26:27 <ThomasWaldmann> brunomartin: it soon will have less repos (after soc is over, we'll try to merge whatever is mergable)
84 2011-08-04T20:26:48 <pkumar> ThomasWaldmann : ok, will do it, thanx :)
85 2011-08-04T20:31:33 <brunomartin> ThomasWaldmann: ok... for now, I have to make this "release" for next wiki, and I will merge with yours and Marchael's repos...
86 2011-08-04T20:32:08 <pkumar> xorAxAx : hi, how do I deal with tests related to users i.e flaskg.user and its attributes. I'm not sure but seems like a possibility of different users in certain cases?
87 2011-08-04T20:34:28 <pkumar> s/related to/depended on
88 2011-08-04T20:36:52 <Marchael> ThomasWaldmann: how can I know username which using search?
89 2011-08-04T20:38:00 <xorAxAx> pkumar: well, as i said before, if a test modifies the user, it should clean up the user again
90 2011-08-04T20:38:06 <xorAxAx> pkumar: can you give examples?
91 2011-08-04T20:38:54 <pkumar> xorAxAx : like in security/textcha
92 2011-08-04T20:39:08 <pkumar> xorAxAx : hmm, may be I can create a particular user for a particular test. lemme look into it
93 2011-08-04T20:40:07 <ThomasWaldmann> Marchael: what's the point of that _search change?
94 2011-08-04T20:40:44 <Marchael> you mean removing searchform from params?
95 2011-08-04T20:41:18 <ThomasWaldmann> the whole change in that fn
96 2011-08-04T20:41:28 <Marchael> in last review you told me what I should remove that small search form, and don't make changes to layout.html
97 2011-08-04T20:41:45 <pkumar> xorAxAx : the test there uses the flaskg.user.name. I was referring to such kind of cases
98 2011-08-04T20:41:50 <brunomartin> ThomasWaldmann: One more thing, I really need to fix this: https://bitbucket.org/thomaswaldmann/moin-2.0/issue/42/problem-rendering-image-on-page
99 2011-08-04T20:41:52 <brunomartin> is there anyone working on that area? I think I can fix that if you could give me some hints...
100 2011-08-04T20:42:04 <Marchael> because I can't ovverride it, I just remove it
101 2011-08-04T20:42:13 <Marchael> I mean don't use
102 2011-08-04T20:42:39 <xorAxAx> pkumar: why is that a problem?
103 2011-08-04T20:43:26 <Marchael> I take some parameters from SearchForm and pass them to search_results.html
104 2011-08-04T20:43:55 <Marchael> So I use only thoose params what need for long-search form
105 2011-08-04T20:45:54 <pkumar> xorAxAx : If not specified can't there be different user.name in different cases depending on request as in setup_user() in app
106 2011-08-04T20:46:14 <xorAxAx> i dont understand
107 2011-08-04T20:50:48 <pkumar> xorAxAx : is there a possibility that there can be different user.name value at that place in different cases
108 2011-08-04T20:51:44 <ThomasWaldmann> Marchael: removing it is as easy as not giving search_form to the render call
109 2011-08-04T20:52:04 <pkumar> xorAxAx : actually I was not talking into account that I can change that value for that test to, huh my bad
110 2011-08-04T20:52:17 <pkumar> too*
111 2011-08-04T20:53:08 <Marchael> ThomasWaldmann: hm, yes. I don't give search_form to template
112 2011-08-04T20:53:11 <ThomasWaldmann> Marchael: like in "you could use medium_search_form"
113 2011-08-04T20:53:51 <ThomasWaldmann> (just another name, maybe even resuing same form class)
114 2011-08-04T20:53:55 <Marchael> ah'
115 2011-08-04T20:53:58 <xorAxAx> pkumar: well, if the env setup is the same, why should there be a different username? only if tests change it, but they should cleanup
116 2011-08-04T20:54:04 <Marchael> ok
117 2011-08-04T20:56:06 <pkumar> xorAxAx : ok, thanks. now my battery is dying. heavy rain, no electricity.
118 2011-08-04T20:56:09 *** m4k3r
119 2011-08-04T20:56:18 <xorAxAx> ah, ok
120 2011-08-04T20:56:31 <xorAxAx> and i am gone jogging now :)
121 2011-08-04T20:56:51 <ThomasWaldmann> brunomartin: yes, have seen the object bug report. but busy on other stuff currently.
122 2011-08-04T20:57:20 <pkumar> k :)
123 2011-08-04T20:58:09 *** pkumar
124 2011-08-04T21:01:09 * ThomasWaldmann gets rid of the resolve_interwiki crap
125 2011-08-04T21:04:12 <Marchael> ThomasWaldmann: http://codereview.appspot.com/4837052/ patchset 6
126 2011-08-04T21:04:16 <Marchael> rather simple
127 2011-08-04T21:09:02 <ThomasWaldmann> looks better, does it work
128 2011-08-04T21:09:24 <Marchael> mm, yes
129 2011-08-04T21:09:34 <Marchael> I'll make 2 clean changesets
130 2011-08-04T21:14:58 *** m4k3r
131 2011-08-04T21:16:20 <brunomartin> ThomasWaldmann: it's ok... I don't wanna disturb you... I will try to fix that object issue...
132 2011-08-04T21:16:55 <sinha> dreimark: i guess we need to work on this select all thing and need time to figure out this, so i will complete this download item thing and it wont be problem to play with selection thing later, agree ?
133 2011-08-04T21:17:17 *** m4k3r
134 2011-08-04T21:20:23 <CIA-114> Michael Mayorov <marchael@kb.csu.ru> * 65c2c01aa266 r387 moin-2.0/MoinMoin/ (search/revision_converter.py storage/backends/indexing.py): Fixing issue with double content indexing, cosmetic changes in debug output
135 2011-08-04T21:20:23 <CIA-114> Michael Mayorov <marchael@kb.csu.ru> * a9b78e0231c6 r388 moin-2.0/MoinMoin/ (templates/search_results.html apps/frontend/views.py): Remove small search field at top right on search result template
136 2011-08-04T21:22:00 <Marchael> ThomasWaldmann: I guess https://bitbucket.org/marchael/moin-2.0/issue/6/unpretty-search-results-rendering is resolved?
137 2011-08-04T21:23:20 <dreimark> sinha: later is ok
138 2011-08-04T21:23:22 <Marchael> brunomartin: when you test my code, do you have this https://bitbucket.org/marchael/moin-2.0/issue/7/duplicates-in-search-results ?
139 2011-08-04T21:23:32 <dreimark> it is just an idea and it needs investigation
140 2011-08-04T21:24:04 <ThomasWaldmann> Marchael: i didn't try your code yet
141 2011-08-04T21:24:24 <ThomasWaldmann> my current (mobile) internet is a bit slow, even chatting is a pain
142 2011-08-04T21:24:48 <Marchael> ok
143 2011-08-04T21:25:37 <Marchael> how can I know username which using search?
144 2011-08-04T21:29:01 <dreimark> sinha: it is not important - just see it as a feature request.
145 2011-08-04T21:29:49 *** m4k3r
146 2011-08-04T21:29:56 <brunomartin> Marchael: I didn't inspect the index, but there is no duplicated results when performing internal searches, like by some metadata...
147 2011-08-04T21:30:27 <dreimark> Marchael: flaskg.user.name ?
148 2011-08-04T21:30:36 <Marchael> brunomartin: like for Home, rst, creole, etc?
149 2011-08-04T21:31:01 <Marchael> dreimark: and if user doesn't login?
150 2011-08-04T21:31:40 *** m4k3r
151 2011-08-04T21:32:45 <dreimark> then you don't have a username
152 2011-08-04T21:33:18 <Marchael> ok
153 2011-08-04T21:34:12 <ThomasWaldmann> Marchael: for what do you neede need username?
154 2011-08-04T21:34:21 <dreimark> is your question related to acls? if so the rule for All: covers non logged in users
155 2011-08-04T21:34:25 <ThomasWaldmann> gna, umts :|
156 2011-08-04T21:34:49 <dreimark> looks like gprs
157 2011-08-04T21:35:02 <Marchael> I want to check "Is that user have rights to read that object?"
158 2011-08-04T21:35:31 <ThomasWaldmann> flaskg.user is the current user, see methods of user object
159 2011-08-04T21:36:06 <Marchael> ok
160 2011-08-04T21:36:14 <dreimark> Marchael: just search for that
161 2011-08-04T21:36:25 <dreimark> there are tests and lots of examples where it is used
162 2011-08-04T21:37:25 <brunomartin> Marchael: what you mean with Home, rst, creole?!
163 2011-08-04T21:38:09 <Marchael> It's the preloaded docs
164 2011-08-04T21:39:53 <brunomartin> Marchael: and?! I'm not understanding... :-|
165 2011-08-04T21:40:05 <sinha> dreimark: okay, i will carry on now
166 2011-08-04T21:41:16 <Marchael> brunomartin: for what search query you have duplications?
167 2011-08-04T21:42:14 <brunomartin> Marchael: but I said I didn't have any! :-D
168 2011-08-04T21:42:26 <Marchael> ???
169 2011-08-04T21:43:59 <brunomartin> brunomartin 04-08-2011 16:26:34
170 2011-08-04T21:44:01 <brunomartin> Marchael: I didn't inspect the index, but there is no duplicated results when performing internal searches, like by some metadata...
171 2011-08-04T21:46:04 <Marchael> hm, /me not quite understand.
172 2011-08-04T21:46:04 <Marchael> Have you duplicating? If yes, for what document names it's happening.
173 2011-08-04T21:46:41 <Marchael> We have issue with uuid for preloaded items like "Home", "creole", "rst"
174 2011-08-04T21:47:19 <Marchael> if you find duplictaion for one of that docs, just ignore that.
175 2011-08-04T21:47:34 *** m4k3r_
176 2011-08-04T21:49:05 *** m4k3r
177 2011-08-04T21:51:35 <dreimark> Marchael: if he starts wit load_xml = None, he won't see those
178 2011-08-04T21:52:28 <Marchael> ThomasWaldmann: what you think about that formatting style http://paste.pocoo.org/show/452755/ ?
179 2011-08-04T21:53:48 *** m4k3r_
180 2011-08-04T21:55:51 *** m4k3r
181 2011-08-04T22:04:12 *** m4k3r_
182 2011-08-04T22:05:13 *** m4k3r
183 2011-08-04T22:05:23 <Marchael> brunomartin: ?
184 2011-08-04T22:08:27 <brunomartin> Marchael: I don't have duplication.. I have only searched items with latest_revision items, by some custom metadata (I have added some index on my whoosh schema) and it works just fine. And fast! :-)
185 2011-08-04T22:08:42 <Marchael> oh
186 2011-08-04T22:09:14 <Marchael> ok, then I concentrate on acl issues
187 2011-08-04T22:10:08 <brunomartin> Marchael: but I'm not using the initial data... I have almost 500MB of data here... I can do some tests tomorrow if you want... ;-)
188 2011-08-04T22:10:26 <Marchael> of course!
189 2011-08-04T22:10:49 <Marchael> any information will be helpful
190 2011-08-04T22:15:06 *** m4k3r_
191 2011-08-04T22:15:50 *** m4k3r
192 2011-08-04T22:18:55 <brunomartin> Marchael: Ok! I will load my data again after a merge with your repo, and try the different ways to build the index (build, update, build on moin init, etc) and let you know what happens... ;-) I will do this next week, cause I'm work on another project now, and I have to fix that object issue first...
193 2011-08-04T22:19:23 <brunomartin> Marchael: *working
194 2011-08-04T22:19:46 <Marchael> ok, thx!
195 2011-08-04T22:20:56 *** m4k3r
196 2011-08-04T22:26:41 *** m4k3r
197 2011-08-04T22:29:55 <Marchael> gn/bbl
198 2011-08-04T22:43:34 *** m4k3r
199 2011-08-04T22:44:24 <ThomasWaldmann> Marchael: name first please, see isssue report
200 2011-08-04T22:44:24 *** m4k3r
201 2011-08-04T22:57:18 *** m4k3r_
202 2011-08-04T22:59:34 *** m4k3r
203 2011-08-04T23:14:36 *** grzywacz
204 2011-08-04T23:14:37 *** grzywacz
205 2011-08-04T23:24:16 *** m4k3r_
206 2011-08-04T23:25:52 *** grzywacz
207 2011-08-04T23:27:31 *** m4k3r
208 2011-08-04T23:38:24 *** sinha
209 2011-08-04T23:47:53 *** sinha
210 2011-08-04T23:48:40 <dreimark> gn
211 2011-08-04T23:56:04 *** m4k3r
212 2011-08-04T23:58:09 *** m4k3r
213