2007-08-17T00:00:22 <dreimark> 2768:803c75dc2aa1
2007-08-17T00:02:37 <dreimark> neagulm: some more MoinMoin/support/art.py", line 152, in step \n currentData = array(input) \n NameError: global name 'array' is not defined
2007-08-17T00:03:09 <neagulm> dreimark, yes, you need to have numpy instaled
2007-08-17T00:03:37 <dreimark> neagulm: is there a docu about what is needed?
2007-08-17T00:04:16 <neagulm> dreimark, yes, see the today commit-ed CHANGES.recommender file
2007-08-17T00:04:31 <neagulm> dreimark, I'm currently updating the wiki page
2007-08-17T00:04:46 <dreimark> good
2007-08-17T00:04:58 <dreimark> pulling
2007-08-17T00:07:19 <dreimark> and reading
2007-08-17T00:09:36 <dreimark> neagulm: last bug seems to be gone with installing of numpy now a new one
2007-08-17T00:11:17 <neagulm> dreimark, i have updated MarianNeagul/RecommendationSystem with the latest instructions.
2007-08-17T00:12:03 <dreimark> neagulm: I got: shape mismatch: objects cannot be broadcast to a single shape
2007-08-17T00:12:09 <neagulm> dreimark, you might need to delete "art.db features.db tokens.db recommendation_*.db"
2007-08-17T00:13:12 <neagulm> dreimark, delete the files and follow the instructions. If logged in as a a valid user delete wiki/data/recommenderdb too
2007-08-17T00:14:03 <neagulm> dreimark, and don't forget to stop MM. I will update in 5 minutes the wiki page again
2007-08-17T00:15:04 <dreimark> doing
2007-08-17T00:29:36 <neagulm> dreimark, I have updated the instructions
2007-08-17T00:34:21 <dreimark> neagulm: For testing run "cluster" to update the cache (more verbose please :) )
2007-08-17T00:35:32 <dreimark> ThomasWaldmann: should an edit lock any item change lock too?
2007-08-17T00:36:04 <dreimark> currently if I get the warning for editing I can delete an attachment without getting a warning
2007-08-17T00:37:48 <dreimark> neagulm: recommender cluster fails !
2007-08-17T00:37:57 <dreimark> AttributeError: PluginScript instance has no attribute 'recommender'
2007-08-17T00:39:23 <CIA-14> moin: Marian Neagul marian@info.uvt.ro main * 2778:722a8525182e 1.7-classify-mneagul/MoinMoin/script/recommender/cluster.py: Fix cluster script
2007-08-17T00:41:34 <xorAxAx> grzywacz: DW poland - http://youtube.com/watch?v=oOGNwLbzooI :)
2007-08-17T00:44:33 <dreimark> neagulm: :)
2007-08-17T00:44:56 <neagulm> dreimark, did you manage to get it work ?
2007-08-17T00:44:59 <neagulm> dreimark, :D
2007-08-17T00:46:21 <grzywacz> xorAxAx: wtf? :P
2007-08-17T00:48:08 <xorAxAx> grzywacz: its a parody about the bad foreign tv stations of the DW (which is some state founded international thing that brings german television into other countries)
2007-08-17T00:48:23 <dreimark> neagulm: no error yet, if I try for testing "--underlay yes" on which page should I get similiar pages
2007-08-17T00:48:26 <grzywacz> xorAxAx: oh ;-)
2007-08-17T00:48:50 <xorAxAx> grzywacz: also a poland parody of course :)
2007-08-17T00:48:58 <xorAxAx> interms of living standards :)
2007-08-17T00:49:08 <grzywacz> Really? ;)
2007-08-17T00:49:17 <dreimark> neagulm: you have typed Simmilar thats a typo it should be similar or ?
2007-08-17T00:49:44 <neagulm> dreimark, fixing now
2007-08-17T00:50:18 <neagulm> dreimark, for FrontPage I get "PaginaPrincipală FrontPage НачалнаСтраница"
2007-08-17T00:50:45 <neagulm> dreimark, PaginaPrincipală means FrontPage in romanian
2007-08-17T00:51:23 <neagulm> dreimark, and i suppose НачалнаСтраница means the same in whatever language it is (I suppose Russian)
2007-08-17T00:51:51 <dreimark> neagulm: then something is wrong or missing I dont have any line
2007-08-17T00:51:58 <neagulm> dreimark, use the WikiClusters macro to see the clusters
2007-08-17T00:52:34 <neagulm> dreimark, are you logged in as a valid user? If yes then you need to logout.
2007-08-17T00:52:46 <neagulm> dreimark, have you trained the classifier ?
2007-08-17T00:53:01 <neagulm> dreimark, train with some pages using the "Star Page" action
2007-08-17T00:53:40 <neagulm> dreimark, and then use the "moin.py ... recommender cluster" (you don't need to restart MM)
2007-08-17T00:54:12 <dreimark> neagulm: not trained yet do I need to do it if I use the "--underlay yes" option too?
2007-08-17T00:54:50 <neagulm> dreimark, yes. With the --underlay yes you tell the script to extract features from the entire wiki. It does no training
2007-08-17T00:55:03 <dreimark> neagulm: ahh ok
2007-08-17T00:55:31 <dreimark> neagulm: why do I have to logout?
2007-08-17T00:55:33 <neagulm> dreimark, training is done using "moin.py ... recommender train" step or by using the "Star Page" action
2007-08-17T00:56:17 <neagulm> dreimark, I didn't finish yet the user part. If you are logged in it will use the user art.db and not the global one
2007-08-17T00:56:34 <neagulm> dreimark, I will fix today this
2007-08-17T00:56:53 <dreimark> neagulm: ok :)
2007-08-17T00:57:09 <dreimark> when will you be onliner today?
2007-08-17T00:57:23 <dreimark> neagulm:
2007-08-17T00:57:40 <neagulm> dreimark, I recommend you to train with at least 10 pages
2007-08-17T00:59:04 <neagulm> dreimark, From 8:00 to 10:00 I need to give a presentation about Formal Verification. After that I will be online
2007-08-17T01:00:39 <dreimark> neagulm: fine
2007-08-17T01:01:16 <dreimark> starshine: ping
2007-08-17T01:03:05 <CIA-14> moin: Marian Neagul marian@info.uvt.ro main * 2779:66690ecd8ead 1.7-classify-mneagul/MoinMoin/theme/__init__.py: Fix Typo
2007-08-17T01:03:41 <neagulm> dreimark, I'm going offline
2007-08-17T01:03:56 <neagulm> dreimark, I'l be back in the morning
2007-08-17T01:04:07 <dreimark> neagulm: good night
2007-08-17T01:04:24 <neagulm> dreimark, good night
2007-08-17T02:18:46 <dreimark> gn
2007-08-17T02:19:26 <grzywacz> xorAxAx: I'll write bot-wiki api specs later on, ok?
2007-08-17T02:20:04 <grzywacz> 'cos at the moment it's all in my head/code, but to keep it maintainable there should be some written documentation ;)
2007-08-17T02:31:19 <grzywacz> hm, something is totally broken with attachment upload on my wiki..
2007-08-17T02:41:59 <grzywacz> http://spoko.math.uni.lodz.pl/~grzywacz/forms/form_attach.png
2007-08-17T02:56:34 <grzywacz> http://spoko.math.uni.lodz.pl/~grzywacz/forms/attach_text.GIF
2007-08-17T02:56:36 <grzywacz> ;]
2007-08-17T03:53:08 <CIA-14> moin: Karol Nowak <grzywacz@sul.uni.lodz.pl> * 2687:da94375a625c 1.7-jabber-knowak/ (3 files in 2 dirs): Towards full data forms support. Not finished, probably broken. Don't merge.
2007-08-17T04:01:01 <CIA-14> moin: Karol Nowak <grzywacz@sul.uni.lodz.pl> * 2688:ef7573c78b43 1.7-jabber-knowak/docs/CHANGES.jabber: Reformat CHANGES.jabber to make it mergable with main CHANGES file
2007-08-17T04:30:56 <CIA-14> moin: Karol Nowak <grzywacz@sul.uni.lodz.pl> * 2689:248a1ff78c24 1.7-jabber-knowak/jabberbot/_tests/test_xmlrpcbot.py: Add two silly tests for the xmlrpc bot.
2007-08-17T04:30:57 <CIA-14> moin: Karol Nowak <grzywacz@sul.uni.lodz.pl> * 2690:64eae2cfb6dc 1.7-jabber-knowak/jabberbot/xmlrpcbot.py: Improve comment.
2007-08-17T04:30:58 <CIA-14> moin: Karol Nowak <grzywacz@sul.uni.lodz.pl> * 2691:42eaeb7584d0 1.7-jabber-knowak/docs/CHANGES.jabber: Merge with my branch ?
2007-08-17T04:31:21 <grzywacz> That was weird...
2007-08-17T08:44:34 <ThomasWaldmann> moin
2007-08-17T08:58:05 <grzywacz> moin
2007-08-17T10:41:00 <CIA-14> moin: Ville-Pekka Vainio <vpivaini AT cs DOT helsinki DOT fi> * 2893:b8e8fe6c713d 1.7-maninfo-vpv/MoinMoin/script/import/manimport.py: some better comments, remove dangerous empty list as a default argument
2007-08-17T10:41:08 <CIA-14> moin: Ville-Pekka Vainio <vpivaini AT cs DOT helsinki DOT fi> * 2904:0f864e1fc4bc 1.7-maninfo-vpv/ (7 files in 4 dirs): merge from main
2007-08-17T10:43:35 <neagulm> hello
2007-08-17T11:58:48 <grzywacz> hi neagulm
2007-08-17T11:58:53 <grzywacz> So hot again today...
2007-08-17T11:59:21 <neagulm> grzywacz, by you too ? :)
2007-08-17T12:01:23 <grzywacz> neagulm: yes. And I have a construction site just outside the window, so it's noisy and smells funny from time to time :P
2007-08-17T12:03:05 <grzywacz> (a good reason to start coding in the evening)
2007-08-17T12:11:46 <neagulm> grzywacz, similar situation to me: http://www.flickr.com/photos/7997141@N04/1146691357/?rotated=1&cb=1187345387658
2007-08-17T12:12:31 <neagulm> grzywacz, the workers did not start yet trowing tiles from the building across the street
2007-08-17T12:13:11 <neagulm> grzywacz, but when they do they will make a very nice sound :)
2007-08-17T12:22:49 <xorAxAx> neagulm: fixable by head phones :)
2007-08-17T12:23:55 <grzywacz> neagulm: http://spoko.math.uni.lodz.pl/~grzywacz/constr.jpg
2007-08-17T12:23:55 <grzywacz> :D
2007-08-17T12:24:31 <neagulm> grzywacz, :D nice
2007-08-17T12:26:44 <xorAxAx> grzywacz wins :)
2007-08-17T12:27:29 <grzywacz> :P
2007-08-17T12:29:22 <grzywacz> So in the meantime I'm gonna do some housekeeping and cook myself a dinner. :P
2007-08-17T12:30:10 <neagulm> xorAxAx, grzywacz yes
2007-08-17T12:30:30 <xorAxAx> bon appétit, grzywacz
2007-08-17T12:34:18 <ThomasWaldmann> http://www.selenic.com/mercurial/wiki/index.cgi/UpcomingReleaseNotes looks interesting
2007-08-17T12:41:30 <CIA-14> moin: Ville-Pekka Vainio <vpivaini AT cs DOT helsinki DOT fi> * 2905:0ef66d72d348 1.7-maninfo-vpv/MoinMoin/script/import/manimport.py:
2007-08-17T12:41:30 <CIA-14> moin: Refactor parts of handle_repository() to handle_repo_infos() and
2007-08-17T12:41:30 <CIA-14> moin: handle_repo_mans(), this way handle_repository() will mainly be a caller of
2007-08-17T12:41:30 <CIA-14> moin: other smaller functions. Also take urlgrabber related imports to module level,
2007-08-17T12:41:32 <CIA-14> moin: it's pure python, so it should be installable easily on non-RH systems too.
2007-08-17T12:47:27 <grzywacz> xorAxAx: thanks, but it will take some time. 8)
2007-08-17T12:48:33 <grzywacz> neagulm: http://www.flickr.com/photos/7997141@N04/691203243/in/photostream/ ?:D
2007-08-17T12:50:55 <neagulm> grzywacz, that picture was taken when students were leaving the campus...
2007-08-17T12:51:49 <grzywacz> ok;-)
2007-08-17T12:55:22 <CIA-14> moin: Marian Neagul marian@info.uvt.ro main * 2780:b4a3297facbd 1.7-classify-mneagul/MoinMoin/ (classifier/recommendation_system.py theme/__init__.py): Use a TableOfContents like layout for displaying recommendations
2007-08-17T12:58:40 <CIA-14> moin: Marian Neagul marian@info.uvt.ro main * 2781:3048eedccfe9 1.7-classify-mneagul/MoinMoin/classifier/recommendation_system.py: Omit current page in displayed recommendations
2007-08-17T13:08:31 <CIA-14> moin: Ville-Pekka Vainio <vpivaini AT cs DOT helsinki DOT fi> * 2906:fea3a5692652 1.7-maninfo-vpv/MoinMoin/script/import/manimport.py: remove some instance attributes
2007-08-17T13:29:22 <grzywacz> itadakimasu! ;]
2007-08-17T13:40:48 <grzywacz> http://spoko.math.uni.lodz.pl/~grzywacz/food/ :-P
2007-08-17T13:42:38 * johill is hungry now
2007-08-17T13:43:20 <grzywacz> :D
2007-08-17T13:54:10 <dreimark> moin
2007-08-17T13:57:15 <grzywacz> hey dreimark
2007-08-17T13:57:31 <dreimark> hi grzywacz
2007-08-17T14:00:09 <neagulm> hello dreimark
2007-08-17T14:00:44 <CIA-14> moin: Marian Neagul marian@info.uvt.ro main * 2867:727d4ba71ce2 1.7-classify-mneagul/ (31 files in 17 dirs): Merge with main
2007-08-17T14:00:45 <dreimark> hi neagulm sorry I am ab it late - jetlag
2007-08-17T14:02:02 <neagulm> dreimark, it look nice using the ToC style for recommendations, it was a very good idea
2007-08-17T14:05:37 <dreimark> neagulm: you should not show recomendations on system pages e.g. the login page
2007-08-17T14:06:11 <dreimark> neagulm: http://localhost:8081/FrontPage?action=login
2007-08-17T14:08:14 <dreimark> neagulm:I dont understand "For testing run the "cluster" script to update the cache"
2007-08-17T14:08:44 <dreimark> when exactly do I use this command?
2007-08-17T14:10:29 <neagulm> dreimark, You use the command the update the classifications...
2007-08-17T14:10:46 <neagulm> dreimark, it should be used after some training to update the recommendations
2007-08-17T14:11:51 <dreimark> neagulm: then better write "Run the "cluster" script to update the cache"
2007-08-17T14:12:48 <dreimark> because some lines before "For testing you could pass the option "--underlay yes" to the script." It could be easy wrong interpreted
2007-08-17T14:13:22 <dreimark> and write the explanaition you gave to me there too
2007-08-17T14:15:37 <dreimark> neagulm: SystemInfo should not show recommendations too
2007-08-17T14:15:51 <dreimark> you excluded allready RecentChanges
2007-08-17T14:16:42 <neagulm> dreimark, i will add a configuration option to skip displaying recommendations on system pages. This will fix the problem
2007-08-17T14:17:14 <dreimark> neagulm: fine, an other question is there a config flag to set the max number listed in similiar pages too?
2007-08-17T14:17:40 <neagulm> dreimark, I will add this too now.
2007-08-17T14:17:44 <dreimark> :)
2007-08-17T14:18:04 <dreimark> bb 5mins
2007-08-17T14:29:40 <dreimark> bb
2007-08-17T14:29:55 <dreimark> grzywacz:I guess you get my late note on MM
2007-08-17T14:42:49 <grzywacz> dreimark: what? :)
2007-08-17T14:44:05 <dreimark> grzywacz: Jabber Support - anonymous edits not prefered
2007-08-17T14:44:51 <ThomasWaldmann> neagulm: you src needs better docstrings
2007-08-17T14:45:20 <dreimark> moin ThomasWaldmann
2007-08-17T14:45:35 <ThomasWaldmann> e.g. the docstring for the classifier package shouldnt be just 'MoinMoin - classifier module'
2007-08-17T14:45:41 <ThomasWaldmann> wb dreimark :)
2007-08-17T14:46:12 <dreimark> thanks
2007-08-17T14:46:18 <ThomasWaldmann> But need 1,2 sentences about what it does.
2007-08-17T14:48:16 <ThomasWaldmann> neagulm: reverend_bayes tells something about being a divmod thing, but divmod is not in the (c)?
2007-08-17T14:49:48 <ThomasWaldmann> neagulm: did you write any tests for your stuff? i dont see a _tests subdir under classifier.
2007-08-17T14:51:23 <ThomasWaldmann> ah, there is at least one under MoinMoin/_tests - please use hg mv to move it to the right place (in a separate changeset)
2007-08-17T14:53:51 <CIA-14> moin: Marian Neagul marian@info.uvt.ro main * 2868:44aee84d8b58 1.7-classify-mneagul/MoinMoin/ (2 files in 2 dirs): Move test_classifier.py to MoinMoin/classifier/
2007-08-17T14:55:11 <ThomasWaldmann> neagulm: you have quite some places with try ... except: - this is bad except for very special applications, you should try to catch more specific Exception(s).
2007-08-17T14:55:53 <neagulm> ThomasWaldmann, (C) DivMod people is in MM/support/Reverend.py
2007-08-17T14:57:02 <ThomasWaldmann> then you maybe wanted to say in your stuff "interface to divmod's ...."
2007-08-17T14:57:36 <ThomasWaldmann> word order and the "to" is important here :)
2007-08-17T14:59:11 <ThomasWaldmann> __init__.py -> _tokenize_attachments_fast - what does it yield when the open fails?
2007-08-17T15:00:14 <ThomasWaldmann> (same q. applies for some other places with similar constructs)
2007-08-17T15:01:16 <ThomasWaldmann> the comment in recommendation_system is also saying nothing you didnt already see from the filename
2007-08-17T15:02:40 <ThomasWaldmann> "rez" as variable name?
2007-08-17T15:03:56 <ThomasWaldmann> btw, maybe slowly read through all your docstrings, looking for improvements in content and english syntax/grammar
2007-08-17T15:05:02 <ThomasWaldmann> if x == {} -> if not x (can be done for every empty thing because it evaluates to False)
2007-08-17T15:08:01 <ThomasWaldmann> neagulm: please optimize the inner construct of remove_page. start with "if token not in seen:"
2007-08-17T15:09:43 <ThomasWaldmann> neagulm: check your usage of for key in x.keys(): - if the datatype of x is a std python type, you don't need the .keys().
2007-08-17T15:10:26 <ThomasWaldmann> (unfortunately this is not true for some libraries that need .keys() for their objects)
2007-08-17T15:11:03 <ThomasWaldmann> return df*(df - 1)
2007-08-17T15:11:13 <ThomasWaldmann> pep8 (spacing)
2007-08-17T15:15:17 <ThomasWaldmann> neagulm: _learn can only return True. Sense?
2007-08-17T15:15:42 <ThomasWaldmann> isspam: pep8 violation
2007-08-17T15:16:51 <ThomasWaldmann> neagulm: please also check if the TODOs in the src are still valid
2007-08-17T15:17:41 <ThomasWaldmann> if pageClass == self.spam_page:
2007-08-17T15:17:41 <ThomasWaldmann> return True
2007-08-17T15:17:41 <ThomasWaldmann> else:
2007-08-17T15:17:41 <ThomasWaldmann> return False
2007-08-17T15:18:15 <ThomasWaldmann> -> return pageClass == self.spam_page
2007-08-17T15:18:49 <CIA-14> moin: Marian Neagul marian@info.uvt.ro main * 2869:855246deccf2 1.7-classify-mneagul/MoinMoin/ (4 files in 2 dirs): Fix coding style
2007-08-17T15:18:52 <CIA-14> moin: Marian Neagul marian@info.uvt.ro main * 2870:d91fad8e80a8 1.7-classify-mneagul/MoinMoin/classifier/recommendation_system.py: Optimize the inner construct of remove_page
2007-08-17T15:18:55 <CIA-14> moin: Marian Neagul marian@info.uvt.ro main * 2871:cf1365ea3bf6 1.7-classify-mneagul/MoinMoin/classifier/recommendation_system.py: Remove x.keys() usage
2007-08-17T15:18:55 <CIA-14> moin: Marian Neagul marian@info.uvt.ro main * 2872:dc47ec0add15 1.7-classify-mneagul/MoinMoin/classifier/reverend_bayes.py: Fix redundant return True
2007-08-17T15:18:57 <CIA-14> moin: Marian Neagul marian@info.uvt.ro main * 2873:bcd1b6c8e223 1.7-classify-mneagul/MoinMoin/classifier/spam_bayes.py: Fix redundant if
2007-08-17T15:19:51 <ThomasWaldmann> SBTokenizer._tokenize_text pep8 viol
2007-08-17T15:21:27 <ThomasWaldmann> RemoteSBClassifier docstring is badly formatted
2007-08-17T15:21:38 <ThomasWaldmann> (_learn)
2007-08-17T15:22:10 <ThomasWaldmann> _learn contains bad if/else/True/False usage
2007-08-17T15:22:32 <ThomasWaldmann> same for isspam there
2007-08-17T15:23:29 <ThomasWaldmann> and you never need to write return (x, y) - you can just write return x, y (the tuple operator is the comma, not the parantheses)
2007-08-17T15:23:45 <ThomasWaldmann> ok, I'm through :)
2007-08-17T15:28:53 <ThomasWaldmann> if pageClass == self.spam_page:
2007-08-17T15:28:53 <ThomasWaldmann> return True
2007-08-17T15:28:53 <ThomasWaldmann> else:
2007-08-17T15:28:53 <ThomasWaldmann> return False
2007-08-17T15:28:57 <ThomasWaldmann> oops
2007-08-17T15:31:39 <CIA-14> moin: Marian Neagul marian@info.uvt.ro main * 2874:d9b6230d7a23 1.7-classify-mneagul/MoinMoin/classifier/__init__.py: Fix spacing
2007-08-17T15:31:40 <CIA-14> moin: Marian Neagul marian@info.uvt.ro main * 2875:a122a21f7b34 1.7-classify-mneagul/MoinMoin/classifier/ (4 files): More coding style fixes
2007-08-17T15:36:09 <ThomasWaldmann> neagulm: you have 1 test failing that doesnt fail in main. please check if you correctly merged main.
2007-08-17T15:42:00 <ThomasWaldmann> test_classification should also test the isspam = True case
2007-08-17T15:42:36 <ThomasWaldmann> in TestReverend
2007-08-17T15:42:52 <CIA-14> moin: Marian Neagul marian@info.uvt.ro main * 2876:79cdf9184e13 1.7-classify-mneagul/MoinMoin/action/ (edit.py info.py): Fix whitespaces
2007-08-17T15:43:29 <ThomasWaldmann> py.test.skip("SpamBayes not instaled, skiping test")
2007-08-17T15:43:38 <ThomasWaldmann> Speling eror :)
2007-08-17T15:44:57 <ThomasWaldmann> TestSpamBayes should also test the "is spam" case
2007-08-17T15:47:02 <ThomasWaldmann> assert (TCR > 1.0)
2007-08-17T15:47:15 <ThomasWaldmann> no need for parantheses
2007-08-17T15:48:14 <ThomasWaldmann> s/kiping/kipping/g
2007-08-17T15:55:03 <ThomasWaldmann> s/parantheses/parentheses/g
2007-08-17T15:55:10 <ThomasWaldmann> :)
2007-08-17T15:57:56 <ThomasWaldmann> pity that the pep8 checker doesnt find some of those things (maybe it were some of those checks I had to disable because of too many false positives)
2007-08-17T16:01:11 <ThomasWaldmann> vpv: your stuff has still \ at EOL in cases where it is not needed due to parentheses
2007-08-17T16:04:46 <vpv> so it seems, I can't commit right now but I have fixed the ones I found
2007-08-17T16:06:34 <ThomasWaldmann> you have also some cases with = followed by multiple blanks (and not needed due to alignment)
2007-08-17T16:07:20 <ThomasWaldmann> (this is also the E222 pep8 error, but I had to disabled it because some of our src aligns stuff)
2007-08-17T16:08:24 <ThomasWaldmann> vpv: btw, you can copy MoinMoin/_tests/pep8.py to your directory and run python pep8.py manimport.py to see some more
2007-08-17T16:09:36 <vpv> alright :) I've run test_sourcecode and pylint so far and tried to fix what they report
2007-08-17T16:10:52 <ThomasWaldmann> (you don't need to fix everything it reports, some stuff is is strange)
2007-08-17T16:11:03 <ThomasWaldmann> gtg, bbl
2007-08-17T16:11:25 <vpv> yup, the pylint output has to be read with care ;)
2007-08-17T16:12:53 <dreimark> neagulm: what are the stop lists?
2007-08-17T16:15:33 <neagulm> dreimark, lists with common words that should not be used as inputs to the classifier
2007-08-17T16:16:44 <dreimark> neagulm: I guess you describe that too
2007-08-17T16:17:49 <neagulm> dreimark, "Common words are defined in so called "stop word" lists. Currently we provide "stop word lists" for English, German, Russian, Spanish, Portuguese, Norwegian, Finnish and Dutch."
2007-08-17T16:18:48 <dreimark> neagulm: recommendation_system if you have imported os you don't need import os.path too
2007-08-17T16:19:38 <dreimark> neagulm: add this description on the wiki page too
2007-08-17T16:19:51 <neagulm> dreimark, it's added
2007-08-17T16:20:53 <dreimark> neagulm: you are right, I didn't get it at once
2007-08-17T16:25:36 <CIA-14> moin: Ville-Pekka Vainio <vpivaini AT cs DOT helsinki DOT fi> * 2907:9b74e532f81e 1.7-maninfo-vpv/MoinMoin/script/import/manimport.py: cleanup some ifs in handle_repository()
2007-08-17T16:25:37 <CIA-14> moin: Ville-Pekka Vainio <vpivaini AT cs DOT helsinki DOT fi> * 2908:43bd65d26f9c 1.7-maninfo-vpv/MoinMoin/script/import/manimport.py: Add save_original(), a function useful for patch generation. Some PEP8 fixes too
2007-08-17T16:34:56 <CIA-14> moin: Heinrich Wendel <h_wendel@cojobo.net> * 2825:0063ddfa0998 1.7-storage-hwendel/ (3 files in 3 dirs): improve documentation
2007-08-17T16:39:44 <CIA-14> moin: Marian Neagul marian@info.uvt.ro main * 2877:8d7dfe3d3a3e 1.7-classify-mneagul/MoinMoin/macro/_tests/test_Action.py: Restore test_Action.py
2007-08-17T16:39:46 <CIA-14> moin: Marian Neagul marian@info.uvt.ro main * 2878:b1cbdba34827 1.7-classify-mneagul/MoinMoin/classifier/recommendation_system.py: Remove os.path import
2007-08-17T16:39:47 <CIA-14> moin: Marian Neagul marian@info.uvt.ro main * 2879:94f4894a5be9 1.7-classify-mneagul/MoinMoin/classifier/_tests/test_classifier.py: Fix coding style
2007-08-17T16:41:15 <dreimark> neagulm: review my changes on MarianNeagul/RecommendationSystem
2007-08-17T16:46:26 <neagulm> dreimark, ok
2007-08-17T16:52:57 <CIA-14> moin: Marian Neagul marian@info.uvt.ro main * 2880:d78f0aa8594f 1.7-classify-mneagul/MoinMoin/classifier/recommendation_system.py: Restore ClassificationCache.get_all to original state
2007-08-17T16:53:12 <neagulm> dreimark, I need to leave for 1h
2007-08-17T16:53:31 <dreimark> neagulm: ok
2007-08-17T17:12:08 <CIA-14> moin: Heinrich Wendel <h_wendel@cojobo.net> * 2826:258caafbb300 1.7-storage-hwendel/ (8 files in 6 dirs): improve code in common.py
2007-08-17T17:33:04 <xorAxAx> lanius: + * If changing an underlay page it will no longer be copied to the standard pages directory, but edited right in the underlay.
2007-08-17T17:33:07 <xorAxAx> lanius: thats a bug ...
2007-08-17T17:57:40 <dreimark> bb ~ 1h
2007-08-17T18:19:42 <DrFrasierCrane> hmmmmm
2007-08-17T18:20:57 <DrFrasierCrane> is it a good idea to start MoinMoin in a thread in Newton using standalone server? after commenting out registerSignalHandlers function (because as I learned Python doesn't support singal handlers in thread other than main) it seems to work OK
2007-08-17T18:21:38 <xorAxAx> DrFrasierCrane: should work fine
2007-08-17T18:21:55 <xorAxAx> just exiting the app might not be so easy
2007-08-17T18:22:52 <lanius> xorAxAx: no, i talked with you about it and you said that is ok
2007-08-17T18:24:16 <DrFrasierCrane> xorAxAx, yup, I realize that.. anyway, is ThreadingServer or ThreadPoolServer better choice for Newton needs (it's going to be a single-user desktop wiki - so rather little load)
2007-08-17T18:24:36 <DrFrasierCrane> I'm going rather for low resource usage than performance
2007-08-17T18:28:17 <xorAxAx> lanius: then you misunderstood me :)
2007-08-17T18:28:25 <xorAxAx> lanius: or you are right
2007-08-17T18:28:40 <xorAxAx> lanius: nah, i couldnt have said that it should be like this
2007-08-17T18:28:49 <xorAxAx> because this breaks essential stuff
2007-08-17T18:29:01 <xorAxAx> lanius: if you edit FrontPage, it must go to the data dir
2007-08-17T18:29:26 <xorAxAx> lanius: please try to find the particular channel logs
2007-08-17T19:26:23 <grzywacz> DrFrasierCrane: IMHO Pool will be a waste.
2007-08-17T19:29:47 <DrFrasierCrane> probably
2007-08-17T19:30:06 <DrFrasierCrane> I'll mainly use xml-rpc to communicate with MoinMoin
2007-08-17T20:26:16 <dreimark> bb
2007-08-17T20:32:35 <ThomasWaldmann> re
2007-08-17T20:59:20 <dreimark> ThomasWaldmann: http://en.wikipedia.org/wiki/Special:Search?go=Go&search=attachment:moinmoin.png|attachment:another.png linking of images is not yet added
2007-08-17T20:59:28 <dreimark> ThomasWaldmann: [ [attachment:moinmoin.png|attachment:another.png] ] linking of images is not yet added
2007-08-17T21:02:16 <ThomasWaldmann> dreimark: you mean on test17?
2007-08-17T21:02:25 <dreimark> yes
2007-08-17T21:06:57 <DrFrasierCrane> ThomasWaldmann, I'm just doing some hacking related to MoinMoin in Newton and I consider writing my own RequestHandler so that I wouldn't have to actually run Moin in the background... just pass requests internally
2007-08-17T21:07:04 <ThomasWaldmann> dreimark: do you mean [ [attachment:moinmoin.png|{{attachment:another.png}}] ]?
2007-08-17T21:07:47 <DrFrasierCrane> is that a good idea?
2007-08-17T21:08:19 <ThomasWaldmann> maybe you can use RequestCLI somehow
2007-08-17T21:09:04 <dreimark> ThomasWaldmann: exactly
2007-08-17T21:09:15 <ThomasWaldmann> dreimark: [[ means "linkto", {{ means "embed"
2007-08-17T21:09:50 <ThomasWaldmann> (or include or transcluse or inline or however you want to call that)
2007-08-17T21:10:11 <dreimark> ThomasWaldmann: great, so it will finish the discussion Image/ImageLink too
2007-08-17T21:10:54 <ThomasWaldmann> the open question for any advanced use is how to give more parameters
2007-08-17T21:11:12 <ThomasWaldmann> currently there is only target|text
2007-08-17T21:11:43 <ThomasWaldmann> if we restrict text to not have | in it, it could be target|text|more_args
2007-08-17T21:12:18 <ThomasWaldmann> and more_args then parsed by argument parser
2007-08-17T21:13:00 <ThomasWaldmann> someone using mediawiki? how do they do it?
2007-08-17T21:17:02 <dreimark> [ [attachment:icon.png|{{attachment:fullsize.png}}] ] renders not correct image + <<<>>>
2007-08-17T21:18:33 <ThomasWaldmann> you want to show the fullsize and link to the icon??
2007-08-17T21:25:38 <DrFrasierCrane> I think I'll try to write my own request handler :-) just to know MoinMoin internals better :D
2007-08-17T21:31:52 <dreimark> ThomasWaldmann: [ [ImageLink(icon.png,attachment:fullsize.png)] ] you see the icon on click you get fullsize loaded
2007-08-17T21:32:51 <ThomasWaldmann> left is target, right is desc
2007-08-17T21:32:52 <dreimark> http://de.wikipedia.org/wiki/Hilfe:Bilder Syntax for mediawiki
2007-08-17T21:32:58 <ThomasWaldmann> but there is a bug
2007-08-17T21:34:12 <dreimark> mediawiki uses | too
2007-08-17T21:35:46 <ThomasWaldmann> yeah, but differently: target|arg1|arg2|...|text
2007-08-17T21:40:08 <dreimark> why can't we use text as last parameter ?
2007-08-17T21:40:46 <ThomasWaldmann> this stuff was inspired by creole
2007-08-17T21:41:02 <ThomasWaldmann> i dont think they specify more than target and text
2007-08-17T21:46:23 <dreimark> they dont http://wikicreole.org/wiki/Images. but when there are only two parameters currently then its no difference to say that in an argument list the last is text
2007-08-17T21:47:53 <grzywacz> Another silly pyxmpp bug...
2007-08-17T21:48:48 <ThomasWaldmann> dreimark: yes, but a creole compatible other parser would parse it incorrectly
2007-08-17T21:49:12 <ThomasWaldmann> (maybe, depending on the implementation)
2007-08-17T21:49:19 <xorAxAx> grzywacz: revert already fixed? :)
2007-08-17T21:50:12 <grzywacz> xorAxAx: no
2007-08-17T21:50:48 <dreimark> neagulms hour is long
2007-08-17T21:51:56 <grzywacz> xorAxAx: IIRWIIR ;)
2007-08-17T21:53:11 <xorAxAx> alexander@tuxpaddy:~$ wtf IIRWIIR
2007-08-17T21:53:11 <xorAxAx> Gee... I don't know what IIRWIIR means...
2007-08-17T21:55:05 <dreimark> xorAxAx: You promised a root server for 5¤/month some weeks ago. Can you give some more info please
2007-08-17T21:55:33 <grzywacz> xorAxAx: wesbot always says that when someone asks about a release. ;)
2007-08-17T21:57:13 <DrFrasierCrane> hmm.. it is ready when it is ready? :D
2007-08-17T21:57:29 <DrFrasierCrane> dunno
2007-08-17T21:58:20 <dreimark> :)
2007-08-17T22:04:44 <xorAxAx> dreimark: not really
2007-08-17T22:05:02 <xorAxAx> dreimark: i have a vserver for 3 EUR / month, though
2007-08-17T22:05:12 <xorAxAx> grzywacz: ah, i remember
2007-08-17T22:05:32 <xorAxAx> grzywacz: well, this is more important than a wesnoth release to lower the negative impact on other branches
2007-08-17T22:05:43 <xorAxAx> dreimark: www.netdirekt.de
2007-08-17T22:06:33 <dreimark> xorAxAx: great
2007-08-17T22:07:13 <grzywacz> xorAxAx: it has no negative impact per se, does it?
2007-08-17T22:07:19 <grzywacz> I'm going to correct it, no worries.
2007-08-17T22:08:59 <xorAxAx> well, if it was fixed this evening, lanius would have to merge it before 20th
2007-08-17T22:09:10 <xorAxAx> --> less mess afterwards
2007-08-17T22:17:46 <dreimark> bbl
2007-08-17T22:32:00 <grzywacz> xorAxAx: http://pyxmpp.jajcus.net/trac/ticket/18
2007-08-17T22:36:10 <xorAxAx> :-)
2007-08-17T22:36:28 <xorAxAx> well, strings are iterable and usually resulting api quirks are easy to spot )
2007-08-17T22:45:17 <CIA-14> moin: Marian Neagul marian@info.uvt.ro main * 2881:67dabacb1ffe 1.7-classify-mneagul/MoinMoin/classifier/recommendation_system.py: Restore keys() usage for shelve instances
2007-08-17T22:45:19 <CIA-14> moin: Marian Neagul marian@info.uvt.ro main * 2882:0b858937fb5e 1.7-classify-mneagul/MoinMoin/classifier/recommendation_system.py: Sanity Check. Check if the input length expected by the ART layes is the same as the one provided by FeatureDB.extract_features
2007-08-17T22:45:25 <xorAxAx> neagulm: you still have code in the edit action ... i said a few months ago that it must be moved into the event handlers (This message has been postponed.)
2007-08-17T22:45:28 <xorAxAx> neagulm: + html = u'<div>Similar pages: ' -- missing i18n/gettext! (This message has been postponed.)
2007-08-17T22:47:13 <neagulm> xorAxAx, I have code in the edit action because I needed to store info in the extra field of editlog
2007-08-17T22:47:30 <xorAxAx> ugh
2007-08-17T22:48:15 <neagulm> xorAxAx, If I remove it then the recent pages spam info will be gone
2007-08-17T22:48:35 <neagulm> xorAxAx, if that is wanted I will remove it in a few minutes :)
2007-08-17T22:49:13 <xorAxAx> well, its not so nice to solve it like this
2007-08-17T22:49:22 <xorAxAx> neagulm: what does it show in RC?
2007-08-17T22:49:28 <neagulm> xorAxAx, I didn't find another way (without using a separate log) to keep this data
2007-08-17T22:50:33 <neagulm> xorAxAx, the page classification (SPAM/HAM)
2007-08-17T22:51:15 <xorAxAx> well
2007-08-17T22:51:32 <xorAxAx> i suggest that you define a return value class that advises the storage to extend the extra field
2007-08-17T22:51:47 <xorAxAx> and then you return an instance of that class in the event handler
2007-08-17T22:52:12 <xorAxAx> thats cleaner than patching the page editor
2007-08-17T23:01:25 <neagulm> xorAxAx, but in PageEditor.saveText, where the PagePreSaveEvent is sent, i don't see any way to change the page attributes
2007-08-17T23:01:41 <neagulm> xorAxAx, am I wrong?
2007-08-17T23:16:43 <xorAxAx> neagulm: is it too late?
2007-08-17T23:16:54 <xorAxAx> well, you would need to introduce a kwarg
2007-08-17T23:16:57 <xorAxAx> that you pass down
2007-08-17T23:17:01 <xorAxAx> one level
2007-08-17T23:22:15 <neagulm> from where?
2007-08-17T23:23:08 <neagulm> xorAxAx, as I see it I need to modify PageEditor to allow modifying saveText kwargs from event handlers , right ?
2007-08-17T23:23:16 <xorAxAx> oh, you dont even need to pass it
2007-08-17T23:23:19 <xorAxAx> where is the problem?
2007-08-17T23:23:25 <xorAxAx> neagulm: no!
2007-08-17T23:23:43 <xorAxAx> the event handlers return results
2007-08-17T23:23:54 <xorAxAx> and one of the results is an instance of the class i just described
2007-08-17T23:24:02 <xorAxAx> it would advise the pageditor to amend the extra field
2007-08-17T23:24:04 <xorAxAx> where is the problem?
2007-08-17T23:25:09 <neagulm> xorAxAx, PageEditor does only "msg = msg + result.reason" with each result
2007-08-17T23:25:24 <xorAxAx> no, it doesnt
2007-08-17T23:25:35 <neagulm> and this only if isinstance(result, Abort)
2007-08-17T23:25:37 <xorAxAx> only if they are abort instances
2007-08-17T23:25:39 <xorAxAx> yes
2007-08-17T23:25:48 <xorAxAx> of course you need to add some code there
2007-08-17T23:25:53 <neagulm> xorAxAx, :D
2007-08-17T23:26:01 <xorAxAx> some generic code that triggers on instances of that class
2007-08-17T23:26:09 <xorAxAx> should be obvious, no?
2007-08-17T23:26:09 <neagulm> xorAxAx, that was what I was trying to say
2007-08-17T23:26:20 <xorAxAx> well
2007-08-17T23:26:22 <neagulm> xorAxAx, yes, it was from the beginning
2007-08-17T23:26:31 <xorAxAx> you dont modify it from pagehandlers
2007-08-17T23:26:47 <xorAxAx> the idea is that it remains functional
2007-08-17T23:26:59 <xorAxAx> and composable
2007-08-17T23:27:15 <xorAxAx> i.e. the modification is done in the pageeditor
2007-08-17T23:27:28 <xorAxAx> but the contents for this modification is in the results
2007-08-17T23:28:44 <xorAxAx> in the end, it will be much cleaner than the monkey patching you did :)
2007-08-17T23:29:41 <neagulm> xorAxAx, yes, I'm a code monkey (http://blog.segfault.ro/2007/08/code-monkey.html)
2007-08-17T23:30:19 <neagulm> xorAxAx, :)
2007-08-17T23:31:07 <xorAxAx> :D
2007-08-17T23:31:40 <xorAxAx> is that a wow-machinima?
2007-08-17T23:32:44 <neagulm> yes
2007-08-17T23:33:50 <xorAxAx> wow, i didnt know that romanian is a romance language
2007-08-17T23:35:09 <grzywacz> ooh, code monkey. good old vid ;]
2007-08-17T23:35:23 <CIA-14> moin: Karol Nowak <grzywacz@sul.uni.lodz.pl> * 2692:dde1f1ee1d80 1.7-jabber-knowak/MoinMoin/events/notification.py: Always add page name to results dict.
2007-08-17T23:35:25 <CIA-14> moin: Karol Nowak <grzywacz@sul.uni.lodz.pl> * 2693:6500951853e2 1.7-jabber-knowak/jabberbot/ (commands.py xmppbot.py): Make "get page info" work with data forms.
2007-08-17T23:36:49 <grzywacz> xorAxAx: the more I write the less I like my code :X
2007-08-17T23:37:10 <grzywacz> I'll run for features till 20th, then will start thinking about some kind of refactoring... -_-;
2007-08-17T23:37:30 <xorAxAx> sounds good
2007-08-17T23:40:16 <neagulm> xorAxAx, romanian is derived from "Vulgar Latin" and is similar to italian
2007-08-17T23:40:26 <grzywacz> I'll probably tend to be away for a few days starting 21st, 'cos I desperately need to finish my flat hunt and plan how to move some stuff
2007-08-17T23:41:05 <xorAxAx> neagulm: yeah, noticed when skimming your blog :)
2007-08-17T23:43:39 <neagulm> it's pity that some make confusions between roma language and romanian
2007-08-17T23:43:47 <neagulm> xorAxAx, ok. I'm going back to work
2007-08-17T23:44:33 <xorAxAx> :)
2007-08-17T23:48:13 <grzywacz> xorAxAx: I also talked with Jacek Konieczny today, and he's in favour of giving me access to pyxmpp repo when I happen to have time to do some development there.
2007-08-17T23:48:23 <grzywacz> So most likely I'm filling these bug reports for myelf. ;>
2007-08-17T23:48:26 <grzywacz> myself even
2007-08-17T23:48:38 <xorAxAx> fair enough
2007-08-17T23:48:49 <xorAxAx> its useful to have open todo lists for such stuff
2007-08-17T23:49:01 <grzywacz> yes
2007-08-17T23:54:57 <xorAxAx> - return df * (df - 1)
2007-08-17T23:54:59 <xorAxAx> + return df*(df - 1)
2007-08-17T23:55:09 <xorAxAx> neagulm: sometimes your commits look a bit insane :)
2007-08-17T23:55:15 <grzywacz> :D
2007-08-17T23:55:56 <xorAxAx> maybe you want to add both spaces again :-)
2007-08-17T23:56:26 <neagulm> xorAxAx, :(( I reverted to a wrong revision and forgot to add the spaces :(
2007-08-17T23:56:32 <xorAxAx> :-)
2007-08-17T23:56:55 <xorAxAx> the niceties of code review MoinMoin: MoinMoinChat/Logs/moin-dev/2007-08-17 (last edited 2007-10-29 19:19:34 by localhost)
