| # | Time | Nick | Message |
|---|
| # | 00:10:39 | KingNightWolf has quit IRC |
| # | 00:28:14 | KingNightWolf has joined #evergreen |
| # | 01:00:34 | KingNightWolf has quit IRC |
| # | 01:36:56 | pmplett has quit IRC |
| # | 02:00:21 | mayanks43 has joined #evergreen |
| # | 03:50:08 | AbhijitKane has joined #evergreen |
| # | 04:00:59 | drake01 has joined #evergreen |
| # | 04:05:22 | drake01 has quit IRC |
| # | 04:06:02 | drake01 has joined #evergreen |
| # | 04:07:29 | drake01 is now known as drake- |
| # | 04:07:34 | drake- is now known as drake_ |
| # | 04:09:26 | drake_ has quit IRC |
| # | 04:21:49 | drake01 has joined #evergreen |
| # | 04:24:29 | drake01 has quit IRC |
| # | 04:26:22 | AbhijitKane has quit IRC |
| # | 04:27:27 | drake01 has joined #evergreen |
| # | 04:28:18 | drake01 has joined #evergreen |
| # | 04:29:06 | drake01 has joined #evergreen |
| # | 04:34:08 | mayanks43 has quit IRC |
| # | 04:37:57 | drake01 has joined #evergreen |
| # | 04:38:28 | drake01 has joined #evergreen |
| # | 04:40:51 | drake01 has quit IRC |
| # | 04:49:25 | Jbergy has joined #evergreen |
| # | 04:52:10 | drake01 has joined #evergreen |
| # | 04:53:19 | drake01__ has joined #evergreen |
| # | 04:54:01 | drake01 has quit IRC |
| # | 04:54:14 | AbhijitKane has joined #evergreen |
| # | 04:54:36 | drake01 has joined #evergreen |
| # | 04:56:21 | drake01__ has joined #evergreen |
| # | 04:56:22 | drake01 has quit IRC |
| # | 04:56:26 | drake01__ has quit IRC |
| # | 04:57:17 | drake01 has joined #evergreen |
| # | 04:59:05 | drake01 has joined #evergreen |
| # | 04:59:57 | drake01 has joined #evergreen |
| # | 05:01:30 | drake01 is now known as drake_ |
| # | 05:04:04 | drake_ is now known as drake01_ |
| # | 05:06:06 | drake01 has joined #evergreen |
| # | 05:07:06 | drake01 has quit IRC |
| # | 05:07:06 | drake01_ has quit IRC |
| # | 05:07:48 | drake01 has joined #evergreen |
| # | 05:08:12 | drake01 is now known as drake01_ |
| # | 05:12:05 | drake01 has joined #evergreen |
| # | 05:13:01 | AbhijitKane has quit IRC |
| # | 05:14:18 | drake01__ has joined #evergreen |
| # | 05:16:10 | drake01_ has quit IRC |
| # | 05:16:10 | drake01 has quit IRC |
| # | 05:16:10 | drake01__ has quit IRC |
| # | 05:16:29 | drake01 has joined #evergreen |
| # | 05:18:35 | drake01__ has joined #evergreen |
| # | 05:29:34 | drake01__ has quit IRC |
| # | 05:29:34 | drake01 has quit IRC |
| # | 05:29:57 | drake01 has joined #evergreen |
| # | 05:42:09 | drake01 has quit IRC |
| # | 05:42:32 | drake01 has joined #evergreen |
| # | 05:46:03 | drake01 has quit IRC |
| # | 05:46:16 | drake01 has joined #evergreen |
| # | 06:02:44 | drake01 has quit IRC |
| # | 06:02:50 | drake01__ has joined #evergreen |
| # | 06:02:58 | drake01__ is now known as drake01 |
| # | 06:12:55 | Jbergy has quit IRC |
| # | 06:21:41 | mayanks43 has joined #evergreen |
| # | 06:27:32 | drake01 has quit IRC |
| # | 06:27:55 | drake01 has joined #evergreen |
| # | 06:28:33 | drake01 has joined #evergreen |
| # | 06:36:48 | drake01 has quit IRC |
| # | 07:02:33 | mtisi has quit IRC |
| # | 07:03:11 | rickd_ has joined #evergreen |
| # | 07:21:49 | bshum has quit IRC |
| # | 07:27:42 | AbhijitKane has joined #evergreen |
| # | 07:38:58 | csharp | @later tell dbs The Ubuntu 10.04 buildslave is all go, but the 8.04 buildslave failed when trying to install pyxmpp. I'm assuming this is a missing dependency... I'll try my best to troubleshoot, but may not get to it immediately ;-) |
| # | 07:38:58 | pinesol` | csharp: The operation succeeded. |
| # | 07:56:43 | collum has joined #evergreen |
| # | 08:01:07 | AbhijitKane has quit IRC |
| # | 08:14:39 | AbhijitKane has joined #evergreen |
| # | 08:21:40 | isl-rjacksom has joined #evergreen |
| # | 08:26:19 | mayanks43 has quit IRC |
| # | 08:36:54 | KingNightWolf has joined #evergreen |
| # | 08:40:14 | mtisi has joined #evergreen |
| # | 08:42:39 | StephenGWills has joined #evergreen |
| # | 08:43:15 | jennam has quit IRC |
| # | 08:56:04 | dbs has joined #evergreen |
| # | 08:56:04 | dbs has joined #evergreen |
| # | 08:59:25 | sfortin has joined #evergreen |
| # | 09:04:39 | AbhijitKane has quit IRC |
| # | 09:05:06 | AbhijitKane has joined #evergreen |
| # | 09:06:15 | kmlussier has joined #evergreen |
| # | 09:07:44 | AbhijitKane has quit IRC |
| # | 09:08:35 | jasonb_ has joined #evergreen |
| # | 09:08:55 | jasonb_ | mrpeters: You called? |
| # | 09:10:16 | bshum has joined #evergreen |
| # | 09:13:54 | mrpeters-isl | jason - yeah |
| # | 09:15:19 | mrpeters-isl | guys - any thoughts on this error http://paste.lisp.org/display/120769 "message":"stat_cat is undefined", upon check in? the item(s) in question don't have any copy stat cat's associated... |
| # | 09:16:05 | mrpeters-isl | appears, at this time, to only impact check in - item statuses, etc. are able to be displayed ok |
| # | 09:16:51 | dbs | my thought is: "we need test cases for common operations in both in-db and scripted circ mode to catch this stuff before it goes live" |
| # | 09:17:04 | jasonb_ | Yes. |
| # | 09:18:15 | mrpeters-isl | dbs: don't disagree there - i'm, happy to donate our circ scripts |
| # | 09:18:18 | jasonb_ | Does xulrunner support external scripting? (I don't know a whole lot about it's capabilities) Just having a test srfsh script would catch some things, but possibly not everything. |
| # | 09:19:05 | mrpeters-isl | also soundsl ike our grace day isn't working :( |
| # | 09:19:16 | mrpeters-isl | so i think our legacy scripts may be really borked |
| # | 09:19:46 | dbs | jasonb_: yes, there are frameworks for exercising xulrunner - mozilla has automated tests via mozmill for example |
| # | 09:21:36 | csharp works on test cases for 1.6.1.7 |
| # | 09:22:00 | bshum | legacy circ scripts, boo... :( |
| # | 09:22:21 | csharp | we're going live on 1.6 with legacy scripts, for continuity's sake |
| # | 09:22:34 | csharp hopes that works |
| # | 09:23:06 | mrpeters-isl | we didn't have any problems there schapr |
| # | 09:23:10 | mrpeters-isl | *csharp |
| # | 09:23:18 | tsbere | mrpeters-isl: grace day? As in grace period? As in the thing we recently found was borked at checkin? |
| # | 09:23:21 | csharp | mrpeters-isl: good! |
| # | 09:24:00 | mrpeters-isl | tsbere: item checked in yesterday, 25 cent fine was charged - library expects no fine since we give 1 grace day |
| # | 09:24:08 | mrpeters-isl | if it's not back at the end of the second dya, then we charge for both days |
| # | 09:24:52 | tsbere | mrpeters-isl: What version you running currently? |
| # | 09:25:58 | mrpeters-isl | ESI just upgraded us to 2.0.4 |
| # | 09:26:01 | mrpeters-isl | just came online this morning |
| # | 09:28:03 | tsbere | That should have the fine generator for daily grace period fix. |
| # | 09:28:43 | mrpeters-isl | leed: any thoughts on that? |
| # | 09:28:57 | tsbere | Though I don't know if it works for backdating |
| # | 09:34:09 | mrpeters-isl | ok - we'll get ESI involved |
| # | 09:35:13 | dbwells | mrpeters-isl: if you were on a version before 2.0.4 when you checked in the item yesterday, then yes, it was broken. The fix is only in the aborted 2.0.3 and 2.0.4. |
| # | 09:37:00 | lisppaste | dbwells pasted "QueryParser,pm, honor active flag" at http://paste.lisp.org/display/120771 |
| # | 09:38:51 | mrpeters-isl | we weren't |
| # | 09:39:00 | mrpeters-isl | this item was checked in via offline mode just recently |
| # | 09:39:59 | dbwells | eeevil: just pasted a small bug fix for QueryParser.pm to get it to honor the 'active' flag in search.relevance_adjustment. I think it is straightforward, but wanted to give you chance for feedback on it before I commit. |
| # | 09:42:05 | dbwells | mrpeters-isl: alright, thanks. We just came up on 2.0.4 on Saturday, so I'll be hearing about this soon if the fix didn't work as intended. |
| # | 09:42:47 | mrpeters-isl | 10-4 |
| # | 09:42:52 | jenny has joined #evergreen |
| # | 09:49:44 | jasonb_ is now known as jasonb_AFK |
| # | 09:52:39 | gmcharlt | mrpeters-isl: by the way, the code that is throwing the error you're seeing is dealing with user stat cats, not copy stat cats |
| # | 09:56:50 | mrpeters-isl | gmcharlt: interesting i was thinking a check-in wouldn't touch user stat cat |
| # | 09:58:51 | dbs | gmcharlt: I'd be interested in knowing how you sorted that out; JSAN always feels like it's in the way. |
| # | 09:59:40 | dbs | mrpeters-isl: perhaps for printing hold slips |
| # | 09:59:41 | gmcharlt | mrpeters-isl: the code in question is gathering patron stat cats for printing on the hold shelf slip |
| # | 10:00:28 | mrpeters-isl | ah right |
| # | 10:00:31 | gmcharlt | dbs: I keyed on the checkin_callback string; the line numbers in the trace actually refer to circ/util.js |
| # | 10:00:53 | dbs | gmcharlt: yeah, that was how I arrived there too |
| # | 10:01:27 | adbowling-isl has joined #evergreen |
| # | 10:02:21 | mrpeters-isl | maybe the %hold_for_msg% macro is the case? |
| # | 10:02:29 | mrpeters-isl | i don't see any stat cat info in the reciept |
| # | 10:03:54 | tsbere | mrpeters-isl: It collects it even if it never uses it |
| # | 10:04:08 | tsbere | Because technically it does a replace on %var% regardless |
| # | 10:04:16 | tsbere | Even if %var% never appears |
| # | 10:07:05 | mrpeters-isl | 10-4 |
| # | 10:09:47 | jennam has joined #evergreen |
| # | 10:09:55 | jennam has quit IRC |
| # | 10:09:55 | jennam has joined #evergreen |
| # | 10:17:47 | mrpeters-isl | shouldn't ln -s /openils/var/web/opac/skin/default /openils/var/web/opac/skin/craftsman be sufficent to send all requests to the craftsman skin back to the default? |
| # | 10:18:05 | gmcharlt | dbs: so it just looks like it's a matter of figuring out how to fix the incorrect filenames in the stack trace |
| # | 10:22:11 | eeevil | mrpeters-isl: it should be, but client-side caching may cause issues |
| # | 10:24:32 | jenny has quit IRC |
| # | 10:24:59 | jasonb_AFK has quit IRC |
| # | 10:26:33 | eeevil | dbwells: looks good to me |
| # | 10:29:29 | mrpeters-isl | eeevil: yeah, seems to be a 403 denial...not worth screwnig with right now |
| # | 10:29:46 | mrpeters-isl | maybe apache isn't following symlinks and i don't want to bother with it - we warned people it'd no longer be available |
| # | 10:29:47 | eeevil | mrpeters-isl: ahh.. follow symlinks on? |
| # | 10:30:04 | jenny has joined #evergreen |
| # | 10:30:09 | mrpeters-isl | ^^ not sure - don't want to restart apache on 5 servers with all of the other stuff we have brewing |
| # | 10:34:16 | mrpeters-isl | <Directory /openils/var/web> Options FollowSymLinks should do it, right? |
| # | 10:34:25 | eeevil | mrpeters-isl: understood ... fwiw, I am having to forcibly clear my cache to get everything happy in some cases |
| # | 10:34:35 | mrpeters-isl | ok |
| # | 10:34:47 | eeevil | hrm... not certain that you won't need more specific Dir ... |
| # | 10:35:30 | mrpeters-isl | hmm |
| # | 10:36:25 | atheos | mrpeters-isl, just drop an index file into /openils/var/web/opac/skin/craftsman that says "update your links dummy!" |
| # | 10:36:37 | mrpeters-isl | haha |
| # | 10:36:48 | atheos | or something more professional |
| # | 10:37:07 | mrpeters-isl | by the time i get to it, everyone will have figured it out haha |
| # | 10:37:35 | mrpeters-isl | could probably make index.xml redirect too |
| # | 10:38:17 | jennam has quit IRC |
| # | 10:43:16 | eeevil | mrpeters-isl: might be easiest to rewrite the url |
| # | 10:43:49 | eeevil | something like (.+)/craftsman/(.+) $1/default/$2 |
| # | 10:44:10 | jennam has joined #evergreen |
| # | 10:44:24 | jennam has joined #evergreen |
| # | 10:45:01 | dbwells | Since migrating to 2.0 on Saturday, we have discovered a pretty significant bottleneck in the 2.0 search plan. This might be better list fodder, but I thought I would throw it out here first for initial reactions. Basically, the penalty for running naco_normalize() over the .value column can be very severe for lengthy records. |
| # | 10:45:08 | adbowling-isl has quit IRC |
| # | 10:45:41 | dbwells | In the default config, naco_normalize() runs over .value for the 'word_order' bump. |
| # | 10:46:51 | eeevil | dbwells: might want to check that search results are qualitatively ok without the rel bumps |
| # | 10:47:01 | dbwells | In an actual case in our catalog, a keyword search for 'united states' took 19 seconds with naco_normalize() removed, and 3:17 with the normalize in place. |
| # | 10:47:31 | dbs | dbwells: yay for digging through search performance! |
| # | 10:48:43 | mrpeters-isl | "Also, she has to scroll to the side to get to 'more actions'. Is this new version formatted to wide screen monitors?" -- really? yeah, we just decided to make everyone buy new screens... |
| # | 10:48:55 | mrpeters-isl is sorry, needed to vent |
| # | 10:49:24 | dbwells | I haven't been able to do any real qualitative comparisons, but at this point we don't really have a choice but to leave the 'word_order' bump off. |
| # | 10:49:24 | eeevil | dbwells: 2.1/trunk includes a change to use rank_cd, which removes the need for rel adjustments for the most part, which might be enough to correct any regression in ordering caused by removing the rel adjustments |
| # | 10:49:31 | phasefx | more testing greatly appreciated before production use :) |
| # | 10:50:07 | dbwells | eeevil: thank you for the information. I was going to ask if any testing had been done with rank vs rank_cd. |
| # | 10:50:08 | mrpeters-isl | phasefx: you know how that goes... |
| # | 10:50:23 | mrpeters-isl | we've had a test server up for people since beta 1 - no one used it, apparently |
| # | 10:50:26 | phasefx | mrpeters-isl: sadly, I do |
| # | 10:50:39 | bshum sympathizes with mrpeters-isl |
| # | 10:50:54 | phasefx | but the community is getting bigger, we'll catch more of these sooner as time goes on |
| # | 10:51:04 | mrpeters-isl | agreed! |
| # | 10:51:18 | mrpeters-isl | if it's valid (doesn't appear so here, though) i'll file a bug |
| # | 10:51:19 | eeevil | dbwells: rank_cd used to be much slower than rank. it's less so now, and faster than rank+rel_adj. also, 2.1/trunk supports QP modifiers to set the algo(s) used |
| # | 10:51:32 | mrpeters-isl | i don't have all that big of a screen and i'm not seeing the same thing - maybe it's a font setting pushing things around |
| # | 10:52:06 | eeevil | mrpeters-isl: set the res to 640x480 and use 22pt monospace fonts ... ;) |
| # | 10:53:36 | bshum | Screen size does make a difference in certain administrative interfaces. |
| # | 10:53:42 | dbs | phasefx: and/or given sample sets of data and automated tests, we can see when response time for the same search with the same data goes from 20 seconds to 3 minutes |
| # | 10:53:49 | bshum | And in Item Status for the alternate view |
| # | 10:54:12 | dbs | bshum: for sure - and for columns that are scoped to be as wide as the longest entry in a drop-down list, etc |
| # | 10:54:30 | bshum | dbs: Oh sure, bring *that* one back up... |
| # | 10:54:58 | bshum | On the plus side, it gave me the leverage I needed to explain why I thought certain copy locations were named too lengthy. |
| # | 10:55:44 | eeevil | as a side note, there are probably optimizations that can be applied to many of the plperl SPs, particularly in the area of regexp ... they've grown large and complicated, so time spent there would be worth-while, IMO, if someone wanted to add eyes to them |
| # | 10:56:11 | mrpeters-isl | yikes |
| # | 10:56:14 | mrpeters-isl | 19621 |
| # | 10:56:14 | mrpeters-isl | This is what I DID: |
| # | 10:56:14 | mrpeters-isl | We lost the cursor during check out. |
| # | 10:56:25 | phasefx | dbs: I'm all for that. Maybe folks can collectively fund some fulltime QA if we make enough of a case for it |
| # | 10:56:41 | mrpeters-isl | what does that even mean? |
| # | 10:56:44 | dbs | phasefx: GSoC will fund test case development as well |
| # | 10:56:50 | tsbere | Side note for people with pre-2.1 and post-2.1 clients in the future: A good indicator of "they are still running the old staff client" is "They still have an 'About this client' button". Have run into that today. |
| # | 10:57:35 | dbs | dbwells: how did you determine that naco_normalize() was a bottleneck - just reading the code and trying variations with and without? or were you able to break down the search and have it report how long each section was taking? |
| # | 10:57:37 | phasefx crosses fingers for GSoC |
| # | 10:59:07 | mrpeters-isl | ahaha oh good god that "nuclear siren" just scared the crap out of me |
| # | 11:00:52 | phasefx | we used to have some cowbell |
| # | 11:01:02 | phasefx | and a scream |
| # | 11:01:09 | mrpeters-isl | i usually don't have headphoens in, it spooked me |
| # | 11:02:46 | bshum | mrpeters-isl: Hmm, what about in holdings maintenance, I'm seeing that sometimes if the staff client isn't maximized, that interface can render strangely cutting off the right side of the screen. There's a scrollbar along the bottom, of course |
| # | 11:03:03 | dbwells | dbs: yeah, nothing high-tech, once I had the slow query isolated, I just tried a few hunches until I boiled it down. |
| # | 11:03:22 | mrpeters-isl | bshum: just ahd someone tell me they couldnt find that button - i wonder if that's it |
| # | 11:03:27 | mrpeters-isl | maybe they're not scrolling |
| # | 11:03:33 | dbs | dbwells: okay, cool. I always hope that there's a magic time-saving technique to adopt :) |
| # | 11:03:37 | bshum | Yeah, it wasn't immediately obvious to my eyes |
| # | 11:03:39 | dbs | dbwells++ |
| # | 11:04:27 | mrpeters-isl | i dont know though....i'm making my window small and actions for this record always stays right on there |
| # | 11:04:37 | bshum | For this record, sure |
| # | 11:04:45 | bshum | For the sub display though |
| # | 11:05:03 | bshum | There's two "Actions for this..." in that interface |
| # | 11:06:15 | bshum | Yeah, the "Actions for the selected rows" button, that's part of the holdings maintenance area |
| # | 11:06:18 | bshum | And that |
| # | 11:06:28 | bshum | seems to be a separate piece from the record portion |
| # | 11:08:04 | bshum | That, and the columns of holdings maintenance seem to prefer a set width, and that's why it doesn't always show up, unless you scroll right to find it. |
| # | 11:09:05 | bshum | I'll ask some others here to test that behavior. |
| # | 11:09:21 | bshum | To see what it looks like on a Windows machine, could just be my own laptop :) |
| # | 11:09:49 | csharp | where would the courier codes live in the /openils tree - does anyone know? |
| # | 11:10:29 | bshum | csharp: Isn't that a library setting now? Or wait, scripted... |
| # | 11:10:57 | csharp | bshum: well, I'm in 1.4, but maybe they are in the DB |
| # | 11:11:00 | csharp looks |
| # | 11:12:54 | csharp | arg - no luck |
| # | 11:13:14 | bshum | Ah, no |
| # | 11:13:23 | bshum | They weren't added as a library setting till like... I want to say 1.6.1 |
| # | 11:13:32 | bshum | Cause we didn't see that in our first 1.6.0 |
| # | 11:13:34 | b_bonner has joined #evergreen |
| # | 11:13:55 | csharp | bshum: they were added for PINES in late 2009 I think |
| # | 11:14:00 | csharp | maybe early 2010 |
| # | 11:14:09 | csharp | so that would be post-1.6.0 |
| # | 11:14:25 | bshum | mrpeters-isl: Tested that smaller screen stuff with a non-maximized staff client on a windows machine and observed similar behavior with holdings maintenance. |
| # | 11:14:54 | dbs | phasefx: those tests for object.ilsevent look a bit redundant, as (foo != null) and (foo != 0) also mean (foo !== '') |
| # | 11:15:04 | bshum | mrpeters-isl: As a fun side note, apparently if you shrink your staff client view smaller than the width of the button bar, it adds another scroll bar to the bottom of the whole shebang to scroll farther to the right to see your buttons. |
| # | 11:15:31 | bshum | mrpeters-isl: Oh right, you guys don't actively use buttons right? |
| # | 11:15:32 | dbs | phasefx: maybe you want "if (object.ilsevent === '')" ? |
| # | 11:16:03 | dbs | (and !== for your other operators) |
| # | 11:16:21 | bshum | csharp: Hmm, then I'm not too sure on that one. |
| # | 11:16:28 | phasefx | dbs: we may want to not have such events in the first place. senator is poking at that angle |
| # | 11:17:37 | dbs | sure, that's an option. I'm trying to avoid the type-coercing operator confusion angle :) |
| # | 11:18:29 | phasefx | yeah. I still remember when things changed from being 0 to '0' |
| # | 11:18:39 | mrpeters-isl | dont think so bshum |
| # | 11:18:40 | phasefx | by whatever JSON conversions where happening |
| # | 11:19:17 | phasefx | dbs: lots of tests littered about that could be changed.. stuff trying Number() this, etc. |
| # | 11:19:56 | bshum | csharp: http://ur1.ca/3mmqn |
| # | 11:20:28 | bshum | csharp: Am I thinking of the wrong thing? |
| # | 11:22:04 | csharp | bshum: that's it - I found it |
| # | 11:22:07 | csharp | bshum: thanks |
| # | 11:22:38 | dbs | heh, I seen senator pointed out the same thing as me :0 |
| # | 11:22:49 | AbhijitKane has joined #evergreen |
| # | 11:24:38 | senator | over here in the silo i share with phasefx. heh my bad. |
| # | 11:25:11 | senator | so i'm working under the theory that nothing should be returning OpenILS::Event objects that look like that anyway (with ilsevent set to '') |
| # | 11:25:16 | senator | silly question: |
| # | 11:25:53 | senator | how do i duplicate the situation that led to this and give a patron a "circ and renew" block? |
| # | 11:26:11 | senator | does that just result from situations like max fines, or is it something else (set manually for a patron)? |
| # | 11:28:27 | AbhijitKane has quit IRC |
| # | 11:28:38 | sergey_ has joined #evergreen |
| # | 11:31:17 | senator | ah found it out (patron -> messages -> apply standing penalty/message) |
| # | 11:33:29 | dbwells | I didn't track the changes too closely as they happened, but I am wondering what we gained by storing the metabib.*_field_entry.value in a non-normalized form. That used to be normalized in 1.6, right? |
| # | 11:41:19 | dbs | In theory, the index_vector should be doing the heavy lifting on normalization for us |
| # | 11:42:46 | jcpl-jasonb has joined #evergreen |
| # | 11:45:26 | Meliss has joined #evergreen |
| # | 11:50:11 | tsbere | So, apparently cloned rows (at least by dojo) aren't getting the required attribute. :( |
| # | 11:50:27 | tsbere | Specifically in patron registration context |
| # | 11:53:13 | tsbere | Looks to be a simple "add three lines to register.js" fix, though. |
| # | 12:01:53 | tsbere | phasefx/eeevil: My "clonesuggested" branch (on git://git.mvlcstaff.org/tsbere/ILS repo) has a 3 line fix for stat cats and surveys (surveys part untested) to make them actually show up when in "suggested" mode on the patron registration screen. Care to take a look? |
| # | 12:06:04 | AbhijitKane has joined #evergreen |
| # | 12:08:51 | gmcharlt | dbs: do any easy bugfixes or tiny enhancements come to mind that involve Dojo? |
| # | 12:09:23 | gmcharlt | I'm realizing that next time we do GSoC, we should also set up a EG-janitor's page or something for microprojects for applicants to work on during the applicaiton period |
| # | 12:09:49 | dbs | gmcharlt: rewrite the "Add a new bookbag" dialog from the IE-soul-crushing native JS dialog to a Dojo dialog |
| # | 12:10:09 | gmcharlt | dbs++ |
| # | 12:10:14 | dbs | gmcharlt: hell, we should have that EG-janitor's page (or just bugs tagged as such?) all the time |
| # | 12:10:30 | moodaepo | Do we need to run reingest after upgrading from 2.0.3 to 2.0.4? We did need to do this when upgrading to 2.0.3 since there were index def changes. |
| # | 12:10:32 | tsbere | gmcharlt: I do believe there is a "make autofieldwidget cache the format.date setting for when used multiple times on the same page". Preferably in a generic way that supports other such settings? |
| # | 12:10:45 | mrpeters-isl | any documentation on the new "floating" feature in the copy editor? |
| # | 12:10:53 | mrpeters-isl | or just a quickie explanation of what it is/does |
| # | 12:13:09 | bshum | People have explained that floating items don't necessarily return "home" but stay at wherever they're checked in. |
| # | 12:13:24 | bshum | I haven't really gotten the full story on how that works with consortia though |
| # | 12:13:40 | mrpeters-isl | ok, so if something was transited out, instead of going home after returned it just stays at the library that returned it? |
| # | 12:13:53 | mrpeters-isl | yikes - i need to set up a way to keep that OFF and don't let anyone turn it on by accident |
| # | 12:13:57 | mrpeters-isl | is there a global setting |
| # | 12:13:57 | bshum | I think that's how it works in theory. Never practiced it. |
| # | 12:15:26 | mrpeters-isl is really afraid of someone turning that on, then complaining that the item never came home... |
| # | 12:15:45 | moodaepo | bshum: I presume you have upgraded your 2.0.3 to 2.0.4 do you know if reingest is really required? I'd like to take that out of the upgrade notes if we don't need to do that step (quite time consuming). |
| # | 12:15:56 | bshum | moodaepo: Unfortunately you presume incorrectly :S |
| # | 12:16:32 | moodaepo | Hah hey you have more important things to take care of so no worries! |
| # | 12:16:33 | bshum | We're still hanging out at 2.0.1 db (more or less with some minor replaced functions) |
| # | 12:17:08 | bshum | Upgrading and maintaining our web/xul changes is turning into a crazy nightmare. |
| # | 12:17:20 | bshum | We're looking to learn version control approaches for that. |
| # | 12:18:00 | tsbere | bshum: I recommend git in that case. Really easy to move changes across branches. |
| # | 12:18:01 | mrpeters-isl | git |
| # | 12:18:08 | mrpeters-isl | ^^ |
| # | 12:18:17 | bshum | mrpeters-isl: Floating is a field on the copy itself, I think. You can always hide that from the item editor screen. |
| # | 12:18:31 | bshum | mrpeters-isl: That would prevent someone from creating an item with floating = true |
| # | 12:18:33 | mrpeters-isl | always end up hand patching, but at least we have record of what to change |
| # | 12:18:41 | mrpeters-isl | that'd be server side xul then? |
| # | 12:18:46 | bshum | Yes. |
| # | 12:18:55 | bshum | We didn't remove that field from ours, but we might yet. |
| # | 12:19:00 | mrpeters-isl | hmm will wait for a potentially better solution |
| # | 12:19:08 | mrpeters-isl | but that might be our best option |
| # | 12:19:14 | bshum | It's the most immediate. |
| # | 12:19:19 | mrpeters-isl | a little weary about doing surgery on something i don't know a lot about |
| # | 12:19:22 | bshum | I don't see any library settings or global to control that behavior. |
| # | 12:19:25 | dbs | We really should adopt Fossil instead of git. Why distribute only code, when you could also distribute your wiki and bug tracking system all at once? |
| # | 12:19:27 | mrpeters-isl | yeah i need to get rid of it before someone finds it and decides to play |
| # | 12:20:21 | dbs | mrpeters-isl: or run a SQL script that periodically sets floating back to false for all items where floating is true |
| # | 12:20:30 | mrpeters-isl | that was my initial thought |
| # | 12:20:43 | mrpeters-isl | but i like bshum's idea of removing it...otu of sight out of mind |
| # | 12:21:04 | bshum | That was always our opinion on taking care of fields we didn't want our librarians worrying about. |
| # | 12:22:18 | mrpeters-isl | bshum: still snowed in? got some time to work with me on this since we may both need it? |
| # | 12:22:46 | mrpeters-isl | started in copy_editor.xul, but nothing about "float" that i can see |
| # | 12:23:07 | phasefx | mrpeters-isl: use the dom inspector, find out the 'id' given to the element you want to hide (and try using the dom inspector to add/remove styles until you know exactly what you want). Then use custom.css to hide the widget based on the id |
| # | 12:24:55 | phasefx | sadly, the id's in that interface are based on localized strings, so this method isn't robust |
| # | 12:25:17 | phasefx | groupbox_Floating? is probably what you want to hide |
| # | 12:25:35 | mrpeters-isl | should we not go the css route then? |
| # | 12:25:47 | mrpeters-isl | i dont think we're using any translations here...ive not seen anyone doing so |
| # | 12:26:05 | phasefx | it'll work until you have someone switch to a french locale :) |
| # | 12:26:10 | mrpeters-isl | hehe |
| # | 12:26:16 | jcpl-jasonb | If you wanted to be particularly tricky, you could just change both the True and False selections to only ever return false. |
| # | 12:26:38 | mrpeters-isl | haha now jason, you know that would just make people ask why they can't make it true :) |
| # | 12:26:49 | phasefx | mrpeters-isl: long term, we should rework that interface to use better id's (based on the IDL) |
| # | 12:26:55 | jcpl-jasonb | Handwaving and permissions, of c ourse. |
| # | 12:27:01 | mrpeters-isl | heh |
| # | 12:27:08 | gmcharlt | but on the whole, I recommend using custom.css when possible; the fewer changes you have to make to the underlying xul templates, the easier it is to upgrade |
| # | 12:27:34 | mrpeters-isl | example of this anywhere? |
| # | 12:28:24 | mrpeters-isl | assuming we want to touch /openils/common/var/web/xul/rel_2_0_4/server/skin/cat_custom.css ? |
| # | 12:28:29 | phasefx | use server/skin/cat_custom.css, and inside there.. ah, I'm not up on my CSS selectors. something for id, maybe #groupbox_Floating? { display: none } |
| # | 12:28:30 | atheos | well, something changes with our gateway API moving to 2.0.4 , anyone have a guess as to why this won't return anything? https://208.119.1.2/osrf-gateway-v1?method=open-ils.actor.user.transactions.history.have_balance&service=open-ils.actor¶m="removed"¶m="4" |
| # | 12:28:36 | atheos | changes/changed |
| # | 12:30:42 | phasefx | mrpeters-isl: and it's still just a kludge; if folks were deadset on mayhem, they could hack their clients, hack exported/imported item attribute templates, etc. We'd probably want some server-side setting to control that feature long-term, if there isn't already one (I don't know) |
| # | 12:31:32 | bshum | mrpeters-isl: phasefx's approach is much more elegant than ours, we hacked the file directly. |
| # | 12:31:49 | phasefx | but, for the most part, I say trust your staff not to go to such lengths to wreak havoc :-) once you've hidden it in the UI |
| # | 12:31:49 | mrpeters-isl | ok - ill try and play around with the css |
| # | 12:33:31 | mrpeters-isl | no dice on the #groupbox_Floating? { display: none } in cat_custom.css -- unless it requires an apache reload or something to see the custom css |
| # | 12:34:17 | phasefx | look up CSS selectors, find out what the actual CSS should be for selecting on id |
| # | 12:34:57 | phasefx | it may be that question marks aren't valid for use in id's with CSS |
| # | 12:35:31 | mrpeters-isl | The following ID selector matches the H1 element whose ID attribute has the value "chapter1": h1#chapter1 { text-align: center } |
| # | 12:35:41 | mrpeters-isl | seems like yours would have worked then |
| # | 12:41:33 | tsbere | Looks like ? isn't valid in a css name? |
| # | 12:42:11 | mrpeters-isl | crap |
| # | 12:42:45 | mrpeters-isl | so that groupbox should probably renamed to groupbox_Floating |
| # | 12:45:51 | senator | [id^=groupbox_Floating] { display: none; } |
| # | 12:45:58 | senator | you'd probably get away with that |
| # | 12:46:17 | senator | it means, "id starts with groupbox_Floating" |
| # | 12:46:38 | senator | it could theoretically match additional dom elements you don't want to hide, but unlikely |
| # | 12:46:41 | phasefx | mrpeters-isl: you could rename it in lang.dtd.. still better than editing xul files |
| # | 12:46:53 | phasefx likes senator's idea better |
| # | 12:47:30 | tsbere | Potential issue: If the parser isn't assigning the ID at all due to an invalid character? |
| # | 12:48:04 | mrpeters-isl | senator: will try |
| # | 12:48:24 | dbs | yeesh, i18n ids - there's a completely unintended consequence |
| # | 12:49:11 | mrpeters-isl | awesome that worked, thanks senator |
| # | 12:49:25 | bshum | senator++ |
| # | 12:49:26 | senator | no prob |
| # | 12:49:57 | finnapz2 has joined #evergreen |
| # | 12:50:21 | bshum | Does the custom_cat.css file remain over upgrades? Or is that another thing we have to keep track of? It's just there to be "custom" |
| # | 12:51:22 | bshum | Nvm, that's a silly question, I think I reasoned it out in my own mind |
| # | 12:52:16 | phasefx | tsbere: I don't think the id gets used for anything other as a hook for CSS |
| # | 12:52:49 | phasefx | wasn't lang.dtd, but cat.properties, most like, so cat_custom.properties if that CSS hadn't worked |
| # | 12:59:02 | bshum | Are there any adverse reactions that occur if we change hours of operation for a library to change them from being closed to open during certain parts of the year? |
| # | 12:59:25 | bshum | Alternatively, we assume that we could also enter closed day entries for that, but that seemed like alot of dates to enter. |
| # | 12:59:29 | gmcharlt | dbs: I'll be using 'bitesize' as a tag for trivial bugs that useful for newbies to look at |
| # | 13:00:08 | jcpl-jasonb has quit IRC |
| # | 13:00:15 | dbs | berick: am I right in thinking the csedit.py is the only place where we currently instantiate IDL definitions as Python objects? Trying to do some gateway requests and thought there would be something built into idl.py to generate to/from JSON list for a single IDL def |
| # | 13:00:35 | dbs | s/Python objects/Python class definitions/ |
| # | 13:01:22 | jcpl-jasonb has joined #evergreen |
| # | 13:01:36 | dbs | gmcharlt: excellent |
| # | 13:06:50 | brian_f has joined #evergreen |
| # | 13:08:11 | mrpeters-isl | any thoughts on how to "unclog" a hung offline processing? library reports its been stuck for a few hours...i can confirm |
| # | 13:12:02 | jcpl-jasonb has quit IRC |
| # | 13:13:57 | gmcharlt | mrpeters-isl: check to see if there's a offline.pl process still running |
| # | 13:15:33 | mrpeters-isl | nope :( |
| # | 13:16:10 | mrpeters-isl | checked each brick and drone - none |
| # | 13:16:43 | mrpeters-isl | files in the pending directory for that org unit haven't been modified since about 10AM either |
| # | 13:17:10 | AbhijitKane has quit IRC |
| # | 13:18:31 | dbs | berick: I guess, given the array of IDLField instances generated for a given IDLClass in idl.py, one could fairly easy create a method to convert properties to array positions using the .position property |
| # | 13:22:07 | dbs | gmcharlt: http://ur1.ca/3mnsq may all be bitesize candidates; also note that 499499 is a dupe of your recently opened bug, but perhaps we can let the gsoc candidates determine that :) |
| # | 13:23:08 | gmcharlt | dbs: agreed, they're all good candidates, though we should probably apologize for tossing a lot of IE-related bugs their way ;) |
| # | 13:23:34 | dbs | "Isn't that fixed by Google Chrome Frame?" |
| # | 13:24:17 | gmcharlt | dbs: as far as 499499 is concerned, oddly, I didn't run into a failure as described in the bug |
| # | 13:24:26 | berick | dbs: i'm not following what your're asking, exactly. to_json will turn IDL-based objects into the correct array-based json object. |
| # | 13:24:35 | shopkins has joined #evergreen |
| # | 13:27:06 | mrpeters-isl | phasefx: another similar question - in the credit card payment xul - is tehre a way to make "process external payment" the only selectable option/ |
| # | 13:27:18 | dbs | berick: maybe I've been looking too hard and need to relax. I was expecting something along the lines of JavaScript's 'foo = new acn(); acn.label('1234'); gateway_request('service', 'method', arg1, arg2, acn);' |
| # | 13:27:19 | mrpeters-isl | we do NOT want someone entering a credit card, thinking they're making payments |
| # | 13:28:35 | dbs | sounds like acn = oils.utils.idl.IDLParser.get_class('acn') / acn.label = '1234' / request('service', 'method', arg1, arg2, acn.to_json()) is the rough equivalent? |
| # | 13:30:17 | berick | dbs: such a thing exists. sec.. |
| # | 13:30:26 | dbs | also, acn may be returned from a previous gateway request rather than created wholecloth; presumably there's a way to parse the response to autovivify the acn instance rather than poking through the class hint manually |
| # | 13:30:33 | dbs | berick: super |
| # | 13:31:31 | jennam has quit IRC |
| # | 13:32:09 | AbhijitKane has joined #evergreen |
| # | 13:33:34 | jcpl-jasonb has joined #evergreen |
| # | 13:38:23 | jcpl-jasonb has quit IRC |
| # | 13:38:29 | csharp | so... what effect, if any, would commenting out the "main.menu.acq" entry in /build/chrome/content/main/menu_frame_overlay.xul have on staff client performance? |
| # | 13:39:37 | csharp | Acquisitions is a touchy subject in PINES and we think suppressing the "Preview" menu (1.6.1.7/8) will be politically desirable ;-) |
| # | 13:39:54 | dbs | ah, hmm - osrf.net_obj appears to be critical |
| # | 13:40:13 | lisppaste | berick pasted "python dbs" at http://paste.lisp.org/display/120780 |
| # | 13:40:15 | dbs was expecting all of the OO stuff to be over in Evergreen land |
| # | 13:40:45 | gmcharlt | csharp: would be no effect on staff client performance |
| # | 13:40:49 | dbs | heh, nice timing berick - thanks! |
| # | 13:41:45 | csharp | gmcharlt: thanks gmcharlt - that's good news! |
| # | 13:42:02 | berick | dbs: I had probs w/ that last echo command, but i'm not sure why |
| # | 13:42:03 | csharp | of course that means we have to package our own SC, but we can live with that |
| # | 13:42:19 | dbs | gmcharlt: cutting out the rendering time for that menu might save microseconds over the course of a year! |
| # | 13:42:26 | csharp | dbs: heh |
| # | 13:42:44 | berick | it should in theory work, and i'm pretty sure constrictor is testing reading encoded json objs from the network, but I could be wrong |
| # | 13:43:00 | gmcharlt | dbs: indeed. I'll await the check for $0.02 for all of the staff time saved! |
| # | 13:43:40 | gmcharlt | that is, unless csharp and I have to thumb wrestle for it |
| # | 13:43:48 | dbs | berick: yeah, i was using constrictor as a rough guide for putting together a sample script that has a nice balance between abstraction and it_just_works |
| # | 13:43:50 | csharp | gmcharlt: you're on! |
| # | 13:45:02 | dbs | berick: trying to help artunit and gfawcett get away from the hardcoded array indexes in the likes of http://svn.open-ils.org/trac/ILS-Contrib/changeset/1272 |
| # | 13:46:06 | dbs | damn, totally glazed over the osrf.net_obj.register_hint() call inside idl.py - that would have helped, dbs |
| # | 13:46:09 | berick | yeah, that's long-term pain for sure |
| # | 13:46:50 | sergey_ has quit IRC |
| # | 13:49:22 | phasefx | AbhijitKane: how are you attempting to check things out with subversion? |
| # | 13:49:47 | dbs | gmcharlt: fwiw, that dialog bug might have been an IE6-only issue. bad dbs for not specifying what version he was testing with in the bug report |
| # | 13:50:13 | AbhijitKane | im using TortoiseSVN, I'm getting a "server failed to respond in time" error |
| # | 13:50:20 | gmcharlt | dbs: or some variation on IE's "security" settings |
| # | 13:50:44 | dbs | gmcharlt: yep, that's quite possible too |
| # | 13:51:54 | phasefx | AbhijitKane: haven't used TortoiseSVN in a long time; is it trying to check out everything? You may want to restrict it to trunk |
| # | 13:52:27 | mrpeters-isl | AbhijitKane: i get that often, yo just have to keep "checking out" until it finishes |
| # | 13:53:07 | dbs | alternatively, install a Linux distro and run from the command line |
| # | 13:54:13 | phasefx stopped using tortoiseSVN when building the client in cygwin started depending on autotools. May be possible, but I'm pain-adverse :) |
| # | 13:55:03 | mrpeters-isl | so using the dom inspector in trying to get the credit card payment default to "process externally" and i can see that the dropdown is "menulist id = where process" -- but i can't seem to find where to remove the "process through evergreen" option |
| # | 13:55:15 | AbhijitKane | <phasefx>: I'm just trying to checkout trunk |
| # | 13:55:37 | mrpeters-isl | this is bill_cc_info.xul - btw |
| # | 13:57:24 | mrpeters-isl | can this be accomplished with the same id hiding? |
| # | 13:57:35 | phasefx | mrpeters-isl: it seems like that those values might be supplied by the database? |
| # | 13:58:23 | phasefx | there's no id on the menuitems (keep digging into the menulist, and you'll see a menupopup with menuitems), but you _could_ select on the value attributes |
| # | 13:58:50 | mrpeters-isl | ah yeah i see the value 1 and 2 |
| # | 13:59:09 | phasefx | AbhijitKane: let us know if mrpeters-isl's advice on just repeating the process doesn't work. Alternately, you may need to switch tools, or ask the tortoiseSVN community what's up |
| # | 14:02:36 | phasefx | mrpeters-isl: so it's not db-supplied, but hardcoded in the xul. should still be selectable by CSS, but more care will be needed. value is a common attribute, and 2 a common value, so you don't want to affect other parts of the client. You may want to specify id="where_process" and then subselect on descendants with value=2 |
| # | 14:03:12 | phasefx needs to get away from IRC for a while *waves* |
| # | 14:03:25 | mrpeters-isl | yeah, this is a bit out of my expertise |
| # | 14:03:25 | mrpeters-isl | crap |
| # | 14:03:50 | mrpeters-isl | we've got to get rid of this credit card stuff...cant have people entering CC payments, but we need people to be able to record external payments like they could in 1.6 |
| # | 14:04:22 | phasefx | could edit the xul as a stop-gap, come up with something more upgrade friendly later |
| # | 14:04:50 | mrpeters-isl | not sure i'm seeing where to take it out there, even |
| # | 14:06:18 | mrpeters-isl | just removing the menu item enough? |
| # | 14:06:25 | mrpeters-isl | <menuitem label="&staff.patron.bill_cc_info.process_int.label;" value="1"/> |
| # | 14:09:26 | lisppaste | berick pasted "working python for dbs" at http://paste.lisp.org/display/120783 |
| # | 14:09:57 | berick | dbs: my original test was using the legacy gateway.. so json parsing failed (as it should) |
| # | 14:10:04 | gmcharlt | berick: there should be a newline in the middle of the last line, I assume? |
| # | 14:10:23 | berick | gmcharlt: yes indeed |
| # | 14:10:25 | berick | thanks |
| # | 14:10:25 | dbs | thanks berick |
| # | 14:12:16 | AbhijitKane has quit IRC |
| # | 14:17:35 | suho has joined #evergreen |
| # | 14:33:00 | suho | hi dbs |
| # | 14:33:14 | dbs | hi suho |
| # | 14:33:38 | suho | after bit of trouble I was able to download the image :) |
| # | 14:33:49 | parsr has joined #evergreen |
| # | 14:34:57 | suho | I'm following this guide line |
| # | 14:34:59 | suho | http://evergreen-ils.org/~denials/README-Evergreen_trunk_2011_03_19_Squeeze |
| # | 14:35:35 | dbs | Okay - did you see my message on the mailing list about the image filename? |
| # | 14:36:07 | suho | yes that was very helpful |
| # | 14:36:30 | suho | I was able to load the image and start that |
| # | 14:37:15 | suho | now i have also done with the "Starting Evergreen" section in the read me |
| # | 14:38:08 | suho | I was also able to successfully start the evergreen desktop client |
| # | 14:39:06 | suho | but when I enter "localhost" to the server host name I'm getting a .. |
| # | 14:40:18 | suho | 443 : uses an invalid security certificate .. error |
| # | 14:41:21 | gmcharlt | suho: try using the add SSL exception button in the staff client |
| # | 14:41:56 | suho | :) |
| # | 14:42:40 | suho | how did I miss that ! |
| # | 14:42:50 | suho | gmcharlt, thanks |
| # | 14:43:09 | gmcharlt | no problem |
| # | 14:43:15 | KingNightWolf has quit IRC |
| # | 14:51:16 | dbs | suho: you have fresh eyes, those are valuable to us |
| # | 14:51:29 | dbs | I'll try to remember to update the README accordingly |
| # | 14:52:06 | audiodevelop has joined #evergreen |
| # | 14:55:52 | nehiljain__ has joined #evergreen |
| # | 14:57:09 | nehiljain__ has quit IRC |
| # | 14:57:16 | jenny has quit IRC |
| # | 15:05:24 | suho | dbs, great |
| # | 15:05:50 | suho | also in the read me it states , |
| # | 15:05:52 | suho | xulrunner-1.9 xul/staff_client/build/application.ini |
| # | 15:06:07 | adbowling-isl has joined #evergreen |
| # | 15:06:21 | suho | but the xulrunner in the image is xulrunner-1.9.1 |
| # | 15:06:58 | dbs | suho: ah, right on - will fix that too! I'm so used to typing "xul<TAB>" that I don't even see the numbers anymore |
| # | 15:07:14 | phasefx | and for the dev vm, probably want 1.9.2, if it's available |
| # | 15:08:04 | phasefx | of course, the client Makefile itself can grab that |
| # | 15:10:03 | audiodevelop has left #evergreen |
| # | 15:10:40 | dbs | phasefx: I'm just using what's there |
| # | 15:11:33 | phasefx | as long as it's not 1.8, can't really complain |
| # | 15:16:12 | LBA has quit IRC |
| # | 15:17:41 | LBA has joined #evergreen |
| # | 15:18:06 | tsbere | phasefx: I dunno. I think we need to say "as long as it is 1.9 series" as 2.0 isn't 1.8, but is likely to not work at all. |
| # | 15:18:24 | phasefx | tsbere: true that |
| # | 15:23:29 | pmplett has joined #evergreen |
| # | 15:25:17 | adbowling-isl has quit IRC |
| # | 15:25:39 | shopkins has quit IRC |
| # | 15:25:39 | brian_f has quit IRC |
| # | 15:26:23 | isl-rjacksom has quit IRC |
| # | 15:28:54 | pmplett has quit IRC |
| # | 15:29:15 | pmplett has joined #evergreen |
| # | 15:34:31 | dbs | D'oh. I hate it when I'm testing something and beating my head against a wall due to an error I can't fathom, only to find out that it's a nonsensical error and I should have tried testing on a stable version first |
| # | 15:42:34 | rickd_ has quit IRC |
| # | 15:42:56 | rickd_ has joined #evergreen |
| # | 15:45:41 | bshum | Does anyone know offhand what the selfcheck interface URL would be in 2.0? |
| # | 15:45:55 | bshum | There's a new one I remember, but I can't recall the specific path |
| # | 15:47:48 | sfortin has quit IRC |
| # | 15:55:04 | dbs | should be defined in the apache examples |
| # | 15:57:34 | mrpeters-isl | hostname/eg/circ/selfcheck/main |
| # | 15:58:46 | bshum | dbs, mrpeters-isl: Ah, okay, thanks guys! |
| # | 15:59:33 | dbs | berick: fwiw, http://svn.open-ils.org/trac/ILS-Contrib/changeset/1274 seems like a reasonable sample - the one thing I couldn't figure out was how to access fleshed fields (near bottom of script). but I've skipped lunch so far today and need some nourishment |
| # | 15:59:51 | Meliss has quit IRC |
| # | 16:04:18 | AaronZ-PLS | In 1.6.0.8 reports, what sets the order of the report output folders? I have seen EG sort them by most recently edited at the top, most recently edited at the bottom and alphebetically, but cannot figure out what makes it tick |
| # | 16:06:14 | dbs | AaronZ-PLS: without looking, it might not be sorted at all - which would leave the order to the whims of the database |
| # | 16:06:33 | AaronZ-PLS | Lovely |
| # | 16:06:54 | collum has quit IRC |
| # | 16:06:55 | dbs | csharp++ # your work on ubuntu 10.04 buildslave has paid dividends sir, thanks for that |
| # | 16:07:53 | dbwells | Hello again folks. Just ran into a problem which is a bit over my head and could use some ideas. We have a utility script which runs nightly and does some cstore queries. |
| # | 16:08:10 | dbwells | Unfortunately, one of the queries is failing, but I can't figure out why. |
| # | 16:08:56 | dbwells | I am trying to select a particular set of around 30,000 ISSNs from metabib.full_rec. |
| # | 16:09:41 | dbwells | If I put a limit of 20,000 on the query, it returns in around 20 seconds. If I raise the limit to 25,000 it never returns (well, at least tested up to a 10 minute timeout). |
| # | 16:10:29 | AaronZ-PLS | dbs: They go from being sorted in aplhabetical/id order to latest updated last when I share them |
| # | 16:10:34 | dbwells | This same query worked fine on EG 1.6.1.7 and OSRF 1.2. |
| # | 16:10:50 | dbwells | I should say this same code. |
| # | 16:11:43 | dbs | AaronZ-PLS: Yeah, there's no order by attribute in open-ils.reporter.folder.visible.retrieve method - would make a good bitesize bug |
| # | 16:13:21 | dbs | dbwells: can you capture the statement from the postgresql logs and EXPLAIN it with and without the LIMIT? |
| # | 16:13:37 | dbs | just to see if it's the query planner picking a bad plan |
| # | 16:14:12 | berick | dbs: re:fleshing, you'll need to use the field name.. barcode_copy.copy_location().name() |
| # | 16:14:34 | berick | or is it location().. /me checks idl |
| # | 16:14:43 | dbs | oh, man, I'm an idiot |
| # | 16:15:13 | berick | yeah, .location() |
| # | 16:15:19 | dbs | given that I was shooting myself in the foot in _exactly_ the same way with plain old barcode_copy.acn() instead of barcode_copy.call_number() before |
| # | 16:15:26 | dbs hangs head in shame |
| # | 16:16:17 | dbwells | dbs: If I run the same query with no limit at all directly in the DB, no problem, finishes in less than a second :( |
| # | 16:16:43 | jeff | dbwells: the same query that cstore is generating? |
| # | 16:17:11 | berick encourages dbs to keep poking the python |
| # | 16:17:51 | dbwells | jeff: I'll double check, but AFAIK, yes |
| # | 16:18:03 | lisppaste | dbs pasted "bad entries in IDL?" at http://paste.lisp.org/display/120788 |
| # | 16:18:46 | berick | dbs: s/bad/known less-than-full IDL linkages/ |
| # | 16:18:52 | berick | not hurting anything per se |
| # | 16:19:02 | berick | the python code will just let you know about it |
| # | 16:19:10 | berick | would be good to repair in time |
| # | 16:19:51 | berick | it used to croak, but I had to put a stop to that ;) |
| # | 16:21:36 | dbs | okay, thanks |
| # | 16:25:47 | dbwells | dbs: jeff: verified that I am using the exact query as generated by cstore. As I said before, it only takes about a second to run, it isn't anything fancy. |
| # | 16:26:05 | dbwells | Very puzzled by this :( |
| # | 16:27:15 | dbs | response size maybe? |
| # | 16:33:07 | dbwells | Just tried it on a different test server. Same configuration, just less RAM. Now my results are: limit 5000 -> function returns in about 3 seconds, limit 6000 -> does not return. |
| # | 16:39:19 | parsr has quit IRC |
| # | 16:39:51 | lisppaste | dbwells pasted "Simple test program, does not finish when too many results returned" at http://paste.lisp.org/display/120790 |
| # | 16:41:44 | brian_f has joined #evergreen |
| # | 16:46:27 | dbwells | If anyone out there is willing to try the test program I pasted, you can also change the where line to "'where' => { 'tag' => '245', '+bre' => {'deleted' => 0}, '+acp' => {'opac_visible' => 1} }" and get it to die with even few results. On my machine, with 3GB of ram, it dies between 2,000 and 3,000. Seems to be some memory related hard limit to the size. |
| # | 16:47:01 | mayanks43 has joined #evergreen |
| # | 16:47:05 | dbwells | dbs: when you say 'response size', is there some specific value I can change or check? |
| # | 16:47:52 | gmcharlt | the XMPP message size limit, methinks |
| # | 16:50:30 | kmlussier has quit IRC |
| # | 16:53:51 | dbwells | well, finally found some relevant error messages, I think. I'll try to search around for them and see what I find. |
| # | 17:10:50 | dbwells | Ok, it turns out that I was hitting the 'max_stanza_size' for ejabberd. Two things still concern me. First, I had it set at the recommended '2000000', and this used to work under the previous OpenSRF/Evergreen. Second, it silently fails and hangs the program until it times out. |
| # | 17:11:36 | dbwells | Well, silent other than logging the error. The Perl script doesn't seem to know what happened. |
| # | 17:12:35 | dbs | Nice detective work! That is a bit concerning. You're sure you didn't miss a zero in there? |
| # | 17:12:47 | dbs | What version of OpenSRF, btw? |
| # | 17:16:11 | dbs | It's particularly bizarre given that you invoked the streaming method, in theory that should avoid any size issues. But in 1.6 there was a change that allowed multiple requests per XMPP message, maybe that had an unanticipated side effect? |
| # | 17:17:06 | bshum has quit IRC |
| # | 17:17:49 | dbs really heads for food, finally |
| # | 17:17:52 | dbwells | I am using '.atomic', so that makes it non-streaming, right? |
| # | 17:18:14 | jeff | right |
| # | 17:18:16 | dbs | oh yeah |
| # | 17:18:31 | dbs | don't do that :) |
| # | 17:18:33 | dbwells | Changing that seems like a reasonable solution. Just wish I knew why it stopped working. |
| # | 17:19:16 | jeff | wait, if you were hitting a max stanza limit, why were your results different on different machines? |
| # | 17:19:27 | jeff | did you find something that explains that also? |
| # | 17:19:46 | dbwells | jeff: yeah, that was my mistake, I did have missing '0' on the test machine :) |
| # | 17:19:54 | jeff | got it. |
| # | 17:35:22 | b_bonner has left #evergreen |
| # | 17:37:41 | finnapz2 has quit IRC |
| # | 17:40:57 | mayanks431 has joined #evergreen |
| # | 17:43:14 | mayanks43 has quit IRC |
| # | 17:45:40 | dbwells | For posterity, it was "working" on OSRF1.2/EG1.6.1.7, but maybe 'atomic' transactions were different enough in that environment. |
| # | 17:45:52 | dbwells | Also, streaming of course works fine. |
| # | 17:53:28 | phasefx | bigger objects? |
| # | 17:53:49 | phasefx | making for bigger aggregate |
| # | 18:07:44 | dbwells | no, the script and all data are the same. The learner in me would love to find out why, but it will probably go into the cold case file at this point. |
| # | 18:08:24 | dbwells | thanks, everyone, for the input |
| # | 18:10:41 | tsbere | Could it be an encoding issue? UTF can take more space than ascii. |
| # | 18:12:48 | gdunbar has quit IRC |
| # | 18:13:09 | suho has quit IRC |
| # | 18:26:12 | suho has joined #evergreen |
| # | 19:06:00 | mayanks431 has quit IRC |
| # | 19:15:58 | mayanks43 has joined #evergreen |
| # | 19:16:26 | mrpeters has joined #evergreen |
| # | 19:16:31 | mrpeters | eeevil: ping? |
| # | 19:34:53 | suho has quit IRC |
| # | 19:37:24 | LBA1 has joined #evergreen |
| # | 19:37:53 | LBA has quit IRC |
| # | 19:42:58 | rickd_ has quit IRC |
| # | 20:43:33 | brian_f has quit IRC |
| # | 20:49:29 | pmplett has quit IRC |
| # | 21:05:12 | StephenGWills has left #evergreen |
| # | 21:52:59 | mrpeters-isl_ has joined #evergreen |
| # | 21:53:11 | mrpeters-isl_ | hey all, any ESI folks lurking? leed? eevil? |
| # | 21:53:36 | pmplett has joined #evergreen |
| # | 22:23:48 | mayanks43 has left #evergreen |
| # | 22:34:51 | brian_f has joined #evergreen |
| # | 22:42:17 | phasefx | mrpeters-isl_: I lurk |
| # | 22:43:09 | phasefx | dbs++ # trunk vm README.html. I assume asciidoc? For the network connection bit, may want to make sure they shutdown Evergreen if started before issuing reboot |
| # | 22:43:30 | dbs | phasefx: asciidoc, of course :) |
| # | 22:43:44 | dbs | I need to put this stuff into a repository somewhere |
| # | 22:45:26 | Aki has joined #evergreen |
| # | 23:00:23 | mayanks43 has joined #evergreen |
| # | 23:12:02 | Aki has quit IRC |
| # | 23:30:22 | dbs has quit IRC |
| # | 23:47:20 | Jbergy has joined #evergreen |
April 10-13