MoinMoin: MoinMoinChat/Logs/moin-dev/2008-12-18

2008-12-18T00:07:07  <nwp> actually it appears to be a bug in python
2008-12-18T00:07:10  <nwp> 2.5
2008-12-18T00:07:55  <nwp> docs for long say that "If the argument is a string, it must contain a possibly signed number of arbitrary size, possibly embedded in whitespace."
2008-12-18T00:08:35  <nwp> but when I run long(78.50801) and long('78.50801') it doesn't do the same thing
2008-12-18T00:09:35  <nwp> using float() with those two args gives the same thing either way
2008-12-18T00:17:28  <nwp> seems it's beta1 of 1.8... so I'm trying the released version
2008-12-18T00:20:11  <nwp> doesn't appear to be any different around that part of the code
2008-12-18T00:20:33  <nwp> so not going to build packages to try it unless someone tells me it will work
2008-12-18T00:28:10  <ThomasWaldmann> re
2008-12-18T00:29:15  <ThomasWaldmann> nwp: that likely means your edit-log is corrupted, you should see the filename in the traceback
2008-12-18T00:30:37  <ThomasWaldmann> you could open that file with an editor (vim or so) and carefully look and fix what's wrong
2008-12-18T00:31:09  <nwp> ThomasWaldmann: it's trying to convert "78.50801" to a long
2008-12-18T00:31:26  <nwp> python docs for long() indicate that this is fine, but it throws an exception if you pass it that as a string
2008-12-18T00:31:32  <nwp> which presumably it is doing
2008-12-18T00:32:05  <nwp> so result.ed_time_usecs needs to be a float/double to be converted to a long like that
2008-12-18T00:32:11  <ThomasWaldmann> show me the edit log +/- some lines
2008-12-18T00:32:16  <nwp> or it needs to be an int already
2008-12-18T00:32:17  <nwp> will look
2008-12-18T00:32:19  <ThomasWaldmann> > pastebin
2008-12-18T00:33:33  <nwp> where is pastebin round here?
2008-12-18T00:33:46  <ThomasWaldmann> http://paste.pocoo.org
2008-12-18T00:33:58  <nwp> ooh. it is buggered, you're right
2008-12-18T00:34:03  <nwp> I see what's happened I think
2008-12-18T00:34:54  <nwp> http://paste.pocoo.org/show/95833/
2008-12-18T00:35:30  <nwp> something's converted something that didn't need converting
2008-12-18T00:36:13  <nwp> actually no...
2008-12-18T00:36:25  <nwp> it's like that in the backup before I tried converitng anything
2008-12-18T00:36:35  <nwp> and it was working fine before (with 1.6)
2008-12-18T00:38:41  <nwp> http://paste.pocoo.org/show/95834/  without the linebreaks
2008-12-18T00:38:50  <ThomasWaldmann> is line 1 complete?
2008-12-18T00:39:33  <nwp> better look at the one without the linebreaks :-)
2008-12-18T00:39:48  <nwp> all lines look well-formed
2008-12-18T00:39:56  <nwp> except for tghe double-decimal-point
2008-12-18T00:40:29  <nwp> but I don't know what that is used for, so it could be correct
2008-12-18T00:40:30  <ThomasWaldmann> looks valid
2008-12-18T00:40:58  <nwp> in ipython, long("78.50801") throws exception while long(78.50801) doesn't
2008-12-18T00:41:36  <nwp> so if result.ed_time_usecs is a string, it will throw exception
2008-12-18T00:41:52  <ThomasWaldmann> the only thing converted to long is the leftmost item
2008-12-18T00:42:14  <ThomasWaldmann> so i guess you are looking at the wrong lines
2008-12-18T00:42:23  <nwp> "supposed to be converted to long..." ;-)
2008-12-18T00:42:41  <nwp> no, this only happens since I made an edit to the page after running with 1.8
2008-12-18T00:42:48  <nwp> so I don't see how that can be so
2008-12-18T00:43:08  <nwp> and the exception explicitly mentions 78.50801
2008-12-18T00:43:46  <nwp> which only appears in lines that appear very similar to those
2008-12-18T00:44:24  <ThomasWaldmann> i bet it appears at the left somewhere
2008-12-18T00:44:41  <nwp> nope
2008-12-18T00:44:55  <nwp> and other pages which I haven't yet tried to edit with 1.8 still view fine
2008-12-18T00:45:00  <ThomasWaldmann> your are sure you look at the file where it failed?
2008-12-18T00:45:35  <nwp> it's the post request to do a new edit that fails I think
2008-12-18T00:45:41  <nwp> not 100% sure though
2008-12-18T00:45:56  <nwp> yep
2008-12-18T00:45:58  <ThomasWaldmann> don't you see the filename in the traceback?
2008-12-18T00:46:07  <nwp> firefox helpfully reposted the thing when it autocompleted the url for me
2008-12-18T00:47:03  <nwp> hang on, now I've managed to view the frontpage by retyping the URL I am not getting the exception
2008-12-18T00:47:08  <nwp> will have to try editing again
2008-12-18T00:47:14  <nwp> I know what it is
2008-12-18T00:47:42  <nwp> it must be the missing field in the line added by the conversion causing the thing to fail when it's creating the editlog for the new edit
2008-12-18T00:47:56  <nwp> or must it
2008-12-18T00:47:59  <nwp> hang on
2008-12-18T00:48:30  <ThomasWaldmann> there is no missing field
2008-12-18T00:48:42  <ThomasWaldmann> it is just empty
2008-12-18T00:49:10  <nwp> yep
2008-12-18T00:49:16  <nwp> but if I remove that line, it works
2008-12-18T00:50:10  <nwp> recentchanges blows up with same exception btw
2008-12-18T00:50:48  <ThomasWaldmann> i still don't see the reason for failure
2008-12-18T00:52:29  <nwp> what is that number with two decimal points in it? If I cheat and add one in manually to the edit log for the conversion line, it all works
2008-12-18T00:52:49  <nwp> so for some reason it is definitely misbehaving when that field is empty
2008-12-18T00:53:53  <nwp> hang on
2008-12-18T00:53:54  <nwp> again
2008-12-18T00:53:58  <ThomasWaldmann> uid
2008-12-18T00:54:33  <nwp> ah
2008-12-18T00:54:51  <nwp> it may not have been that exact edit that caused it to work
2008-12-18T00:54:58  <nwp> just trying to reproduce now
2008-12-18T00:58:11  <nwp> can't make it break on that page now
2008-12-18T00:58:15  <nwp> will try another one
2008-12-18T00:58:37  <nwp> nope, not breaking
2008-12-18T00:58:39  <nwp> weird
2008-12-18T00:58:59  <nwp> recentchanges now no longer barfs either
2008-12-18T00:59:26  <nwp> so editing FrontPage's editlog seems to have somehow made it all happy
2008-12-18T00:59:29  <nwp> but no idea how
2008-12-18T00:59:35  <nwp> future date or something perhaps?
2008-12-18T01:01:22  <nwp> if I get a chance I will re-run the conversion this afternoon and see whether it barfs again
2008-12-18T01:01:39  <nwp> have to have lunch now though
2008-12-18T01:04:01  * ThomasWaldmann will sleep soon
2008-12-18T01:11:54  <nwp> will let you know anything I find tomorrow then
2008-12-18T02:12:50  <nwp> ThomasWaldmann: so... it seems that this is a little unpredictable - what I was finding before was probably affected by the apache child that served it up :(
2008-12-18T02:13:02  <nwp> so I need to restart apache every time I test it
2008-12-18T04:32:47  <starshine> test in cgi mode (which reloads moin all the time) then switch to another mode when you like the results.
2008-12-18T04:33:59  <starshine> it also gives more chances for ways to track the errors.
2008-12-18T08:58:55  <ThomasWaldmann> moin
2008-12-18T10:56:34  <LotekThirteen> moinmoin
2008-12-18T10:57:06  <LotekThirteen> may ask, if somebody saw my bug report: http://moinmo.in/MoinMoinBugs/TrivialChangesDoesNotUpdateTheXapianIndex
2008-12-18T10:57:31  <LotekThirteen> it's a bit annoying that after trivial saves the wikipages get removed from the index
2008-12-18T13:47:52  <CIA-27> Reimar Bauer <rb.proj AT googlemail DOT com> default * 320:a9f7a24a567d 1.7-extensions/data/plugin/parser/text_x_arnica.py: text_x_arnica: bug fix of the only_item list problem of multiple aliases for one image by skipping duplications.
2008-12-18T13:48:16  <dreimark> moin
2008-12-18T14:03:38  <dreimark> wb LotekThirteen
2008-12-18T14:45:04  <CIA-27> Reimar Bauer <rb.proj AT googlemail DOT com> default * 321:c958b4dbfa4a 1.7-extensions/data/plugin/parser/text_x_arnica.py: text_x_arnica: verify the image for the item list ( *[[image|alias]]) by wikiutil.isPicture
2008-12-18T14:45:12  <LotekThirteen> hi dreimakr
2008-12-18T14:46:24  <dreimark> LotekThirteen: Hi have a bit more time between christmas and new year to look at the xapian issue (btw. tab completes the nick name)
2008-12-18T14:47:02  <dreimark> +, I
2008-12-18T14:47:35  <LotekThirteen> dreimark: oh! dreimark dreimark - works :-)
2008-12-18T14:48:21  <LotekThirteen> thx for takeing care of this xapian issue...
2008-12-18T14:49:20  <dreimark> currently I do have the usual end of year rush hour at work (assuming also anyone else)
2008-12-18T18:08:43  <dreimark> bbl
2008-12-18T20:13:41  <ThomasWaldmann> re
2008-12-18T23:45:28  <dreimark> re
2008-12-18T23:45:47  <ThomasWaldmann> moin dreimark
2008-12-18T23:47:27  <dreimark> moin ThomasWaldmann