| 08:26.39 | *** join/#infobot simonrvn_ (i=simon@unaffiliated/simonrvn) |
| 08:26.39 | *** mode/#infobot [+v simonrvn_] by ChanServ |
| 08:28.48 | *** join/#infobot simonrvn1 (n=simon@219.55-ppp.3menatwork.com) |
| 08:28.48 | *** mode/#infobot [+v simonrvn1] by ChanServ |
| 08:46.11 | *** join/#infobot ravenbird (i=ravenbir@unaffiliated/simonrvn/bot/ravenbird) |
| 14:38.57 | *** join/#infobot TimRiker (n=timr@70.1.67.186) |
| 14:38.57 | *** mode/#infobot [+o TimRiker] by ChanServ |
| 17:24.30 | troubled | simonrvn: http://picpaste.com/box.jpg |
| 17:24.35 | troubled | TimRiker: ^^^ |
| 17:24.57 | troubled | still need to put it to a pure white background with the reflection only and no plane visible, but its a start :) |
| 17:26.20 | simonrvn | heh |
| 17:26.38 | troubled | :) |
| 17:26.51 | troubled | could use a few bumpmaps on the circle part I think though |
| 17:27.58 | TimRiker | $99.99 at a store near you! |
| 17:28.04 | troubled | rofl |
| 17:28.30 | troubled | i forgot to put a "on sale" star and a price tag on it heh |
| 17:28.56 | TimRiker | :) |
| 17:29.08 | troubled | ill probably put the .blend and .svg's in the svn later though |
| 17:30.26 | troubled | oooh, i know: "As seen on IRC!" |
| 19:32.20 | *** join/#infobot MentalPower (n=MPower@WoWUIDev/Norganna/Administrator/MentalPower) |
| 19:33.08 | MentalPower | hey TimRiker, purl's seen command seems to be not working |
| 19:33.11 | MentalPower | ~seen MentalPower |
| 19:33.12 | infobot | i haven't seen 'mentalpower', MentalPower |
| 19:34.20 | troubled | ravenbird: seen MentalPower |
| 19:34.56 | ravenbird | troubled: i haven't seen 'mentalpower' |
| 19:35.21 | MentalPower | I'm guessing ravenbird is an infobot/bootbot too? |
| 19:35.28 | troubled | hmm, wonder if its being lc'd on acase sensitive field |
| 19:35.30 | simonrvn | yes |
| 19:35.31 | troubled | ya |
| 19:35.35 | MentalPower | does it share factoids with purl? |
| 19:35.40 | simonrvn | no |
| 19:35.41 | MentalPower | aka, is it the same brain? |
| 19:35.45 | troubled | ah no |
| 19:35.55 | MentalPower | so its a code issue? |
| 19:35.58 | troubled | ravenbird: seen simonrvn |
| 19:35.59 | simonrvn | yes |
| 19:35.59 | ravenbird | troubled: i haven't seen 'simonrvn' |
| 19:36.10 | troubled | simonrvn: are you runing +seen in here? |
| 19:36.16 | simonrvn | _default |
| 19:36.43 | troubled | i wonder if the tables are being updated or not |
| 19:36.55 | TimRiker | ~seen TimRiker |
| 19:36.56 | infobot | TimRiker: i haven't seen 'timriker' |
| 19:37.02 | TimRiker | hmm |
| 19:37.21 | troubled | TimRiker: seems its being lc'd, but I cant recall if the tables are case sensitive |
| 19:37.52 | TimRiker | should only be added lower case, so that should work. used to. |
| 19:38.22 | troubled | i did do some lc tweaks in a part of the code a while back. dont recall it being seen related though |
| 19:38.46 | troubled | but still, it should have found simonrvn anyways since he is lower case nick to begin with |
| 19:39.03 | TimRiker | yep. looks like the table entries are there... |
| 19:39.11 | TimRiker | <PROTECTED> |
| 19:39.16 | TimRiker | timriker|1205869015|#infobot|n=timr@pdpc/supporter/bronze/TimRiker|~seen TimRiker |
| 19:39.35 | troubled | so its being updated at least, so no loss of info |
| 19:39.42 | troubled | let me take a peek at the module |
| 19:40.49 | simonrvn | 19:35.58 !WARN! PERL: DBD::mysql::st execute failed: Unknown column 'messagecount' in 'field list' at /home/ravenbird/infobot/src/dbi.pl line 158. !WARN! sqlSelect failed. |
| 19:41.05 | simonrvn | and just previous to that: |
| 19:41.05 | TimRiker | ~seen random |
| 19:41.07 | infobot | TimRiker: i haven't seen 'random' |
| 19:41.07 | simonrvn | 19:35.58 cmdhooks: seen matched 'seen' 'simonrvn' |
| 19:41.07 | simonrvn | 19:35.58 Seen: Flushed 1 entries. |
| 19:41.09 | troubled | <PROTECTED> |
| 19:41.29 | troubled | did we have a schema change for the seen table or somethin? |
| 19:41.58 | troubled | <PROTECTED> |
| 19:42.11 | troubled | seems it was me that did the change. let me take a look at the old one |
| 19:43.08 | troubled | - my $select = 'nick,time,channel,host,message'; |
| 19:43.11 | troubled | + my $select = 'nick,time,channel,host,message,messagecount'; |
| 19:43.13 | troubled | - $i = $userstats{lc $seen[0]}{'Count'}; |
| 19:43.16 | troubled | + $i = $seen[6] || $userstats{lc $seen[0]}{'Count'}; |
| 19:43.37 | troubled | * Merge r1296 from Don's branch - Use message count instead of the per-invocation stuff |
| 19:44.00 | TimRiker | but no schema change to allow for it. |
| 19:44.28 | troubled | seems I pulled it from don's code for some reason. maybe stats related? I know I had to tweak some stuff for stats |
| 19:45.00 | troubled | ahh, pgsql schema has the change, not mysql or sqlite though |
| 19:45.09 | troubled | <PROTECTED> |
| 19:45.31 | simonrvn | fwaps troubled |
| 19:45.38 | TimRiker | infobot: reload |
| 19:45.38 | infobot | reloading... |
| 19:45.38 | infobot | reloaded: CommandStubs.pl |
| 19:45.40 | troubled | :) |
| 19:45.44 | TimRiker | infobot: seen troubled |
| 19:45.45 | infobot | troubled is currently on #botpark #debian #asterisk #infobot #debian-bots. Has said a total of 105 messages. Is idling for 5s, last said: ':)'. |
| 19:46.06 | troubled | simonrvn: didnt we have that fixed though? remember all that seenstat testing? |
| 19:46.07 | simonrvn | TimRiker: reverted, or added the column? |
| 19:46.32 | troubled | i would revert for now personally. take out the pgsql column for it i guess |
| 19:46.43 | TimRiker | nods |
| 19:46.58 | troubled | and wth is seenstats in pgsql? |
| 19:47.54 | CIA-26 | infobot: 03timriker * r1642 10/trunk/src/CommandStubs.pl: revert partial messagecount patch |
| 19:47.55 | troubled | <PROTECTED> |
| 19:47.57 | troubled | <PROTECTED> |
| 19:48.07 | simonrvn | *shrug* |
| 19:48.23 | troubled | there is something serious wrong with sf.net svn I think |
| 19:48.33 | TimRiker | yep, just in pgsql. glad we have so many different sql setup scripts. |
| 19:48.37 | simonrvn | @reload |
| 19:48.44 | simonrvn | ^reload |
| 19:48.45 | ravenbird | error: you do not have enough flags for that. (n required) |
| 19:48.52 | simonrvn | wtf |
| 19:48.57 | TimRiker | ~reload |
| 19:48.57 | infobot | reloading... |
| 19:48.57 | infobot | reloaded: |
| 19:49.00 | troubled | either someone is messing around with the repo, or sf has corruption issues. this is twice now a old patch got lost |
| 19:49.20 | simonrvn | probably #2 |
| 19:49.21 | TimRiker | what got lost? |
| 19:49.21 | troubled | Im starting to suspect that the old repo merge had corruption issues or something |
| 19:49.40 | TimRiker | the patch is in there for pgsql, but only for pgsql |
| 19:49.50 | troubled | The stats fix I did. I applied it allong time ago in bloot, then it disappeared in the infobot repo all of a sudden one day and I had to redo it |
| 19:50.19 | troubled | I sat down one day and made sure all the schemas were the same cept for index issues, not it seems to be gone again |
| 19:51.14 | troubled | * Updated BUGS list |
| 19:51.15 | TimRiker | bets on pebkac |
| 19:51.16 | troubled | * SQL table fix, channel size should 30 chars max (db table backup/drop/recreate/restore needed) |
| 19:51.20 | troubled | * New USAGE and FAQ file to help new users (location/filenames/format may change) |
| 19:51.23 | troubled | -r1587 |
| 19:51.25 | troubled | nod |
| 19:51.58 | troubled | major re-organisation of the different SQL files; cleanup and fixing of the postgres stuff -- troubled is da man |
| 19:52.19 | TimRiker | ~seen MentalPower |
| 19:52.21 | infobot | mentalpower is currently on #wowi-lounge (20m 1s) #infobot (20m 1s) #norganna (20m 1s) #wowace (20m 1s) #curseforge (20m 1s) #wowwiki (20m 1s). Has said a total of 22 messages. Is idling for 2m 8s, last said: '~seen ckknight '. |
| 19:52.24 | troubled | thats the log for -r1312. Thats missing all the reorg patchs I did. that was before I even had svn commit access |
| 19:52.45 | TimRiker | nods |
| 19:52.46 | troubled | so somewhere between -r1312 and HEAD we lost a few dosen patches |
| 19:52.55 | troubled | and this is twice now |
| 19:53.02 | TimRiker | communication issue I'm betting. |
| 19:53.02 | MentalPower | thanks TimRiker, troubled, simonrvn :) |
| 19:53.16 | TimRiker | MentalPower: np :) thanx for finding it! |
| 19:53.47 | MentalPower | nah, I'm just the messenger :) |
| 19:53.54 | MentalPower | see you later |
| 19:53.58 | MentalPower | poofs |
| 19:54.08 | *** part/#infobot MentalPower (n=MPower@WoWUIDev/Norganna/Administrator/MentalPower) |
| 19:55.16 | troubled | ~:)stats troubled |
| 19:55.40 | troubled | guess it needs stats to be enabled though |
| 19:56.01 | troubled | simonrvn: when ravenbird gets back, check to see if the stats patch is still in there |
| 19:56.32 | *** join/#infobot ravenbird (i=ravenbir@unaffiliated/simonrvn/bot/ravenbird) |
| 19:56.58 | simonrvn | ^seen simonrvn |
| 19:56.59 | ravenbird | simonrvn is currently on #debian #debian-bleh #botpark ##debian-offtopic #linux-quebec #debianppc #debian-bots #infobot #mlug #ldap. Has said a total of 1 messages. Is idling for 0s, last said: '^seen simonrvn'. |
| 19:58.44 | troubled | ^:)stats troubled |
| 19:58.53 | troubled | ^hehstats troubled |
| 19:59.04 | troubled | simonrvn: ^^^ broken again? |
| 20:00.14 | troubled | you and I tested that for hours, now the patch is gone? |
| 20:04.20 | troubled | 2007-10-21 is the day we tested, -r1590 it seems |
| 20:07.35 | simonrvn | must be |
| 20:07.44 | troubled | simonrvn has said :) 1 times (100.0 %), ranked 1/1 (percentile: 100.0 %) |
| 20:07.53 | troubled | that was ravenbirds reply from that day |
| 20:08.15 | troubled | ^:)stats simonrvn |
| 20:08.19 | ravenbird | simonrvn has not said :) yet. |
| 20:08.23 | simonrvn | :) |
| 20:08.31 | simonrvn | ^:)stats simonrvn |
| 20:08.31 | ravenbird | simonrvn has not said :) yet. |
| 20:08.36 | troubled | see, its broken again |
| 20:09.06 | troubled | seems to be every time a branch is used to do the patch and merged back, sf.net chokes and looses everything |
| 20:09.13 | simonrvn | loses |
| 20:09.18 | troubled | :) |
| 20:09.29 | simonrvn | sf is rather borked |
| 20:10.10 | simonrvn | gtg |
| 20:10.15 | troubled | TimRiker: we need to check into this. seems like we are loosing patches |
| 20:12.10 | troubled | hmm, the patch still seems to be applied from what I can tell |
| 20:12.46 | troubled | question is, what changed in ravenbird since then that broke the stats again |
| 20:13.44 | troubled | heh |
| 20:13.47 | troubled | infobot: hehstats troubled |
| 20:13.48 | infobot | troubled has said heh 1 times (100.0 %), ranked 1/1 (percentile: 100.0 %) |
| 20:14.11 | troubled | well that works fine. wth wrong with ravenbird then :/ |
| 20:14.40 | simonrvn | only diff is that ^^^ patch is reverted (the !seen stuff) |
| 20:15.02 | troubled | why isnt ravenbird replying to hehstats then? |
| 20:15.10 | troubled | it running +silent in here still? |
| 20:16.06 | simonrvn | only in #botpark |
| 20:16.59 | troubled | also, sqlite2's schema is out of sync with the rest with regard to the primary key change from nick,channel to nick only. I assume we are going with nick now instead? |
| 20:17.18 | troubled | simonrvn: anything in the log as to why its not working with hehstats then and infobot is? |
| 20:17.28 | troubled | ^hehstats |
| 20:17.29 | ravenbird | total count of heh on #infobot: 1. Top 1: RichiH -- 1 (100.0%) |
| 20:18.14 | troubled | simonrvn: can you paste the setting for "ircTextCounter" pls? |
| 20:18.30 | simonrvn | nothing unusual. sec. |
| 20:18.47 | simonrvn | ircTextCounters heh hah hehe haha :D :p :P :) ;) :( ;( lol ? hi hello morning |
| 20:19.09 | troubled | richih hasnt been in here in ages, and its somepletely missing all the testing we did. unless you reverted or erased your stats table after 2007-10-21 |
| 20:19.24 | troubled | is that _default or just some channels though? |
| 20:19.40 | simonrvn | _default |
| 20:19.52 | simonrvn | is it singular or plural? |
| 20:20.05 | troubled | sorry, plural |
| 20:20.14 | simonrvn | GTG |
| 20:20.17 | troubled | l8r |
| 20:30.35 | CIA-26 | infobot: 03djmcgrath * r1643 10/trunk/setup/ (mysql/seen.sql pgsql/seen.sql sqlite/seen.sql): |
| 20:30.35 | CIA-26 | infobot: * Synced schema for seen table in other engines to match sqlite2 schema (PKEY only nick now) |
| 20:30.36 | CIA-26 | infobot: * Removed extra pgsql fields in schema for stats |
| 20:35.19 | troubled | TimRiker: hmm nm then I guess. seems my patches are in there (all the size changes to sync the char fields). Guess I was thinking of something else |
| 20:36.08 | troubled | while im doing the schemas though, what is the stand on the not nulls and defaults? should we just allow nulls for easy lookups in the db and removed the defaults to allow the nulls? |
| 20:39.36 | TimRiker | for fields that "always" have values, NOT NULL is sensible. for sometimes fields like requested_by I think NULL should be allowed. |
| 20:40.26 | troubled | ya, i like nulls. easy to search for |
| 20:40.27 | TimRiker | for int fields, default value '0' has trivial overheard, so might as well use it. things like unix style dates. |
| 20:40.48 | troubled | why not just allow nulls everywhere so we can track down and replace stuff easier? |
| 20:40.54 | TimRiker | but I would not complain if you wanted to NUL those too. |
| 20:40.55 | troubled | including timestamps |
| 20:41.01 | TimRiker | fine. |
| 20:41.08 | troubled | sec, let me patch it |
| 20:43.27 | troubled | TimRiker: any chance you could null all timestamps for a particular record so we can observe how the bot handles them for "literal" and "factinfo" commands here? |
| 20:43.47 | troubled | ~patchtest is this is a test |
| 20:43.48 | infobot | troubled: okay |
| 20:43.53 | troubled | ~patchtest |
| 20:43.53 | infobot | rumour has it, patchtest is this is a test |
| 20:44.03 | troubled | that factoid should be safe to fubar :) |
| 20:50.42 | TimRiker | sure, you just wanna see the bot crash... |
| 20:52.59 | TimRiker | hmm. schema won't allow it. |
| 20:53.17 | TimRiker | so I guess that's a nope. |
| 20:53.26 | *** join/#infobot FactBot (n=factbot@pdpc/supporter/sustaining/troubled) |
| 20:53.26 | *** mode/#infobot [+v FactBot] by ChanServ |
| 20:53.29 | troubled | sec, ill bring a fresh sqlite db version of factbot online |
| 20:53.54 | troubled | $foobar is testing |
| 20:53.54 | FactBot | troubled: okay |
| 20:53.57 | troubled | $foobar |
| 20:53.57 | FactBot | extra, extra, read all about it, foobar is testing |
| 20:55.59 | troubled | guess it would help if I actually changed the schema first :) |
| 20:59.57 | troubled | TimRiker: btw, do we actually have a reason for having the sqlite2 schema? |
| 21:01.57 | troubled | $die |
| 21:02.34 | TimRiker | meaning what? infobot uses sqlite2 is that what you are asking? |
| 21:02.44 | *** join/#infobot FactBot (n=factbot@pdpc/supporter/sustaining/troubled) |
| 21:02.44 | *** mode/#infobot [+v FactBot] by ChanServ |
| 21:03.17 | troubled | do we need v2 instead of v3 though? |
| 21:03.31 | TimRiker | dunno. I'm using sqlite2 atm. |
| 21:03.45 | TimRiker | 3 would likely work, i've just not moved. |
| 21:04.01 | troubled | v3 is what i tested your dump in. seemed to work fine |
| 21:04.15 | troubled | $foobar is <reply> testing... |
| 21:04.16 | FactBot | troubled: okay |
| 21:04.19 | troubled | $foobar |
| 21:04.19 | FactBot | testing... |
| 21:04.26 | troubled | $factinfo foobar |
| 21:04.26 | FactBot | foobar -- created by troubled <n=troubled@pdpc/supporter/sustaining/troubled> 10s ago; it has been requested once, last by troubled, 6s ago. |
| 21:04.31 | troubled | $literal testing |
| 21:04.37 | troubled | $literal foobar |
| 21:04.37 | FactBot | "foobar" is "<reply> testing..." |
| 21:04.47 | troubled | heh |
| 21:04.50 | troubled | $hehstats |
| 21:04.50 | FactBot | total count of heh on #infobot: 1. Top 1: troubled -- 1 (100.0%) |
| 21:07.35 | troubled | $testing |
| 21:07.45 | troubled | $test |
| 21:07.52 | FactBot | rumour has it, test is this is a test |
| 21:08.06 | troubled | $factinfo test |
| 21:08.06 | FactBot | test -- it has been requested once, last by troubled, 20s ago. |
| 21:08.57 | troubled | $factinfo test |
| 21:08.57 | FactBot | test -- it has been requested once, last by troubled, 1m 12s ago. |
| 21:09.11 | troubled | $factinfo test |
| 21:09.11 | FactBot | troubled: no extra info on test |
| 21:09.36 | troubled | seems when requested_by is null, it doesnt report some stuff |
| 21:09.46 | troubled | $factinfo foobar |
| 21:09.46 | FactBot | foobar -- created by troubled <n=troubled@pdpc/supporter/sustaining/troubled> 5m 30s ago. |
| 21:09.58 | troubled | ahh ok, only if everything is null |
| 21:10.27 | troubled | $factinfo foobar |
| 21:10.27 | FactBot | foobar -- created by troubled <n=troubled@pdpc/supporter/sustaining/troubled> 6m 11s ago. |
| 21:10.43 | troubled | $factinfo foobar |
| 21:10.43 | FactBot | foobar -- created by troubled <n=troubled@pdpc/supporter/sustaining/troubled>. |
| 21:11.02 | troubled | seems to handle ok |
| 21:11.49 | troubled | basicallly it displays what isnt null |
| 21:26.17 | troubled | im going to set all the field sized to a max of 100 |
| 21:27.15 | troubled | (requested_by, created_by, modified_by and locked_by). I checked your db and the biggest you have is a 93 for modified by. should allow for people with legacy db's if they do a convert |
| 21:30.07 | TimRiker | k |
| 21:35.42 | troubled | sqlite> select max(length(factoid_key)) from factoids; |
| 21:35.42 | troubled | 78 |
| 21:35.53 | troubled | TimRiker: hmm, max in schema is 64 :/ that could be a problem for you |
| 21:38.08 | troubled | I can bump the shcema to 100 for you so you can convert the db without problems. Im checking in with don atm to see what he's got. 100 is fine for him for the *_by fields though |
| 21:50.11 | CIA-26 | infobot: 03djmcgrath * r1644 10/trunk/setup/ (4 files in 4 dirs): |
| 21:50.11 | CIA-26 | infobot: * Moved all VARCHAR fields to 100 size for easier legacy db upgrades (should handle max freenode sizes and then some) |
| 21:50.12 | CIA-26 | infobot: * Removed pgsql specific REVOKE from PUBLIC sql (users should make that decision) |
| 21:50.13 | CIA-26 | infobot: * Removed NOT NULL constraint from all but requested_count in factoids table |
| 21:50.14 | CIA-26 | infobot: * Most fields now default to NULL to allow for easier finding of problems, DB cleanup and importing of inconsisting |
| 21:50.15 | CIA-26 | infobot: DBs |
| 21:50.15 | infobot | i heard dbs is Disappearing Browser Syndrome. Commonly found in most Netscape products. |
| 21:51.21 | troubled | hehe |
| 22:05.28 | TimRiker | keys that long are silly. |
| 22:06.05 | TimRiker | so I'm find with at factoid_key length of 64 |
| 22:06.35 | TimRiker | the other entries are likely from the bot running in aggressive learn mode years ago. |
| 22:20.49 | troubled | well, don's max is 64, but you will have import problems with a new schems unless its 100 |
| 22:21.06 | troubled | for all i know the bot forces it to 64 anyways |
| 22:21.29 | troubled | its varchar anyways, not like it wastes space. but I can move it back to 64 i you like |
| 22:49.14 | troubled | TimRiker: im interested in this factpack idea, do we support factpacks still? or is that left over from the original source? |
| 22:58.02 | TimRiker | there was a working import scripts in scripts/ at one point. been ages since anyone has tested it. |
| 22:58.15 | TimRiker | s/scripts /script / |
| 23:00.08 | troubled | im thinking of using them, but loading them to memory on startup or something instead of into the main factoids table to allow for dynamic updates with svn |
| 23:00.57 | troubled | put them in the files or src tree under factpacks or something, then have a config setting for what factpacks to load/use |
| 23:13.27 | TimRiker | might be able to assume something with factoids that have no created_by field. |
| 23:13.35 | TimRiker | or modified_by either. |
| 23:13.39 | TimRiker | later... |
| 23:20.02 | troubled | $die |
| 23:24.44 | CIA-26 | infobot: 03djmcgrath * r1645 10/branches/don/proposed/ (33 files in 10 dirs): * Merged r1634:1644 from trunk |
