Time |
Nick |
Message |
01:33 |
|
zerick joined #evergreen |
02:31 |
|
edoceo joined #evergreen |
03:24 |
|
RBecker joined #evergreen |
03:56 |
paxed |
jeff: re. yesterday's "Renew Patron" button discussion: from translation viewpoint, plain "Renew" is too damned vague (see bug 1178557) |
03:56 |
pinesol_green |
Launchpad bug 1178557 in Evergreen "Short translatable strings need context" (affected: 2, heat: 10) [Wishlist,Triaged] https://launchpad.net/bugs/1178557 |
07:08 |
|
timhome joined #evergreen |
07:36 |
|
rjackson-isl joined #evergreen |
07:53 |
|
jboyer-isl joined #evergreen |
08:07 |
|
adbowling-isl joined #evergreen |
08:29 |
|
Shae joined #evergreen |
08:33 |
|
akilsdonk joined #evergreen |
08:43 |
|
kmlussier joined #evergreen |
08:43 |
|
kbeswick joined #evergreen |
08:45 |
|
Dyrcona joined #evergreen |
08:54 |
|
mrpeters joined #evergreen |
08:57 |
|
ericar joined #evergreen |
09:28 |
|
akilsdonk_ joined #evergreen |
09:32 |
|
remingtron joined #evergreen |
09:32 |
bshum |
dbwells: Regarding bug 1244432, do you have any example steps on how to produce these breaking browse queries? |
09:32 |
pinesol_green |
Launchpad bug 1244432 in Evergreen "Browse search functions need to be 'stable'" (affected: 1, heat: 8) [Critical,New] https://launchpad.net/bugs/1244432 |
09:33 |
bshum |
dbwells: While we're not using a fully updated master in our production (we need to update to get the new title browse stuff), we do have the beginnings of browse in play and haven't seen any extremely long browse attempts yet. |
09:35 |
kmlussier |
bshum: I think it might be something that was just recently introduced because the browse queries had previously been working on Dyrcona's server. I believe it's just in the past few weeks that they became problematic. |
09:36 |
dbwells |
bshum: for our production data set, it seemed to be every query. I suspect it has to do with 0837 on large DBs. |
09:36 |
bshum |
Alternatively, is this something related to auth records being linked to browse? In which case, I couldn't properly test it anyways. |
09:36 |
bshum |
And yeah, 0837, gotcha..... |
09:37 |
dbwells |
bshum: Do you have 0837 in place? We haven't done any authority linking yet at all. |
09:37 |
dbwells |
also, test branch is moments away... |
09:38 |
bshum |
Aha, 0837 is not on production yet. |
09:38 |
bshum |
It's on my test server which is awaiting finishing touches |
09:38 |
bshum |
So I guess I'll test it there |
09:38 |
bshum |
dbwells++ # I'll let you know what I find out as well later today, if Dyrcona doesn't beat me to it. |
09:40 |
dbwells |
bshum: thanks! |
09:40 |
csharp |
browse is broken on our pines test server too: http://next.gapines.org |
09:41 |
csharp |
I was starting with the assumption that our cold cache and slow DB server is the cause |
09:41 |
kmlussier |
I don't know if Dyrcona changed something this morning, but the browse search on his server is happier today than it has been in a long time. |
09:41 |
csharp |
(that's running 2.5-rc1, btw) |
09:42 |
Dyrcona |
I applied the changes from lp 1244432 last night when I updated the code to fix a typo in one of my development branches. |
09:42 |
pinesol_green |
Launchpad bug 1244432 in Evergreen "Browse search functions need to be 'stable'" (affected: 1, heat: 8) [Critical,New] https://launchpad.net/bugs/1244432 |
09:45 |
paxed |
re. yesterday's "combined" hold that our current ils can do: patrons themselves cannot do those, only staff. |
09:45 |
paxed |
also, it's apparently pretty useful for eg. univ. lecture books and so on which may have 3-4 different books of which you need to read one, or something. |
09:47 |
csharp |
I can think of several use cases for that kind of hold, including outreach/homebound patrons |
09:48 |
kmlussier |
Dyrcona / dbwells: With those changes from bug 1244432 that Dyrcona applied, I can say there is a big improvemet in browse search. Retrieval of results is nice and speedy. |
09:48 |
pinesol_green |
Launchpad bug 1244432 in Evergreen "Browse search functions need to be 'stable'" (affected: 1, heat: 8) [Critical,New] https://launchpad.net/bugs/1244432 |
09:48 |
jeff |
both internally and in channel here, i think we've talked about the idea of a few different options for personal holds queues. |
09:49 |
jeff |
the "place a hold on these items, first item in cancels the other holds" option is one, and the "place a hold on these items, give me 2 at a time in this order until done" is another |
09:49 |
dbwells |
kmlussier: that's great :) |
09:49 |
jeff |
also, and this requires a significant amount of trust and faith in your metadata, series holds |
09:49 |
csharp |
in fact, that's one of the distinguishing features of our library for the blind's ILS - it lets people place a "pool" of holds and it selects whichever one is next available |
09:51 |
|
RoganH joined #evergreen |
09:51 |
csharp |
jeff: the latter example ("give me 2 at a time") would do what the GLASS libraries need, I think |
09:52 |
dbwells |
Dyrcona: Just FYI, I pushed a branch for bug 1244432. As noted on the bug, I marked the other two browse pivot functions as stable as well to be proactive, since they are the same overall shape (select wrappers). |
09:52 |
pinesol_green |
Launchpad bug 1244432 in Evergreen "Browse search functions need to be 'stable'" (affected: 1, heat: 8) [Critical,New] https://launchpad.net/bugs/1244432 |
09:53 |
Dyrcona |
dbwells: I'll take a look later. I'm trying to work on something else and should stop paying attention to IRC for a bit. |
09:53 |
dbwells |
Dyrcona: no problem, and thank you |
10:06 |
|
yboston joined #evergreen |
10:06 |
|
zerick joined #evergreen |
10:08 |
|
RoganH joined #evergreen |
10:13 |
paxed |
hm. it would be nice if it was possible to ignore the reingests in upgrade scripts, and then schedule one at the end. |
10:14 |
Dyrcona |
paxed: it is possible, just edit the upgrade scripts beforehand. |
10:14 |
Dyrcona |
That's what I do, anyway. |
10:14 |
dbs |
paxed: I think that's what we effectively did for the 2.3-2.4 upgrade, but if you're doing all of the /upgrade/ scripts, yeah... |
10:14 |
paxed |
right. i'm thinking more in the line of psql -v ignore_reinges=yes blahblah |
10:18 |
bshum |
Most recent reingest scripts are outside the COMMIT anyways. |
10:18 |
bshum |
Or at least, most of the ones I remember doing. |
10:19 |
bshum |
But I suppose it depends |
10:21 |
Dyrcona |
IIRC, the w.x to y.z upgrade scripts usually only do the ingest at the end. |
10:21 |
Dyrcona |
I edit the ingests out of the xxxx upgrade scripts and run a separate sql at the end. |
10:28 |
pinesol_green |
[evergreen|Jason Stephenson] Handle Missing copy status (4) in checkin_via_barcode2 in circ.util.js. - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=ef4d822> |
10:28 |
|
akilsdonk joined #evergreen |
10:37 |
|
test621613 joined #evergreen |
10:38 |
jeff |
paxed: are you suggesting that the individual upgrade scripts start having a "if this variable is set to FOO" check that skips the re-ingest? then a "stock reingest" that you'd run after you're done? |
10:38 |
paxed |
jeff: something like that, yes |
10:42 |
* bshum |
feels like it'd be easier to just print a message saying "please reingest after you're done with your upgrades" in that case. |
10:43 |
paxed |
or that, yes. |
10:43 |
jeff |
have incremental upgrade scripts never perform a reingest, and only \qecho an advisory that one is recommended? |
10:45 |
bshum |
jeff: To my recollection, the best upgrade scripts that require reingest have the message displayed along with being outside the main transaction, so you can cancel and do it later. |
10:45 |
bshum |
As far as how consistently that's been applied, I think that's something that varies... |
10:46 |
bshum |
And some reingests work differently on what's being reingested and how. |
10:46 |
paxed |
SELECT reingest_advisory(); |
10:48 |
bshum |
The latest one 0844 is an example of where the message is included and the reingest done outside the main transaction block. |
10:49 |
jeff |
bshum: yeah. there's the "do a full reingest" by ensuring that the flags are set correctly and doing a set id=id or set marc=marc on biblio.record_entry, but there's also metabib.reingest_metabib_field_entries and metabib.reingest_metabib_full_rec (and the former has some handy arguments for skipping facet/browse/search |
10:49 |
bshum |
0839 is another recent example of a variant of reingesting done outside the main commit body |
10:51 |
bshum |
I could see the wisdom of having some standard boiler plate message about reingesting needs though. |
10:51 |
bshum |
That said |
10:51 |
bshum |
The less reingesting required, the less painful upgrades get for those of us doing upgrades along master :D |
10:52 |
jeff |
bshum: what does a reingest look like for you in terms of WAL churn? |
10:53 |
|
sylvar joined #evergreen |
10:53 |
bshum |
jeff: I haven't really looked closely at that piece of it yet. |
10:54 |
sylvar |
As I continue to spell-check Evergreen, I've become fond of the typo 'hodlings'. I bet their first word is also always 'Hodor'. |
10:55 |
paxed |
*snrk* |
10:56 |
sylvar |
And by the way...was there supposed to be a file matching Open-ILS/src/sql/Pg/upgrade/0539\.*\.sql or was it just a 13th floor? |
10:57 |
bshum |
sylvar: Back then, upgrade numbers got stolen for other branches and master wasn't always updated with a stub |
10:58 |
paxed |
oh, that reminds me: there's one upgrade script that's missing the .sql extension: 0282.schema.user-visiable-holds |
10:58 |
bshum |
So it's entirely possible that 0539 only exists in some branch version of Evergreen. |
10:58 |
paxed |
(also: visiable?) |
10:58 |
sylvar |
bshum++ Thanks, I feel somewhat more sane. |
10:58 |
bshum |
In more recent times, I think we've tried to be more thorough in at least adding a file to master and noting which branch it ultimately ended up in. |
10:59 |
sylvar |
I noticed that too, paxed, and decided that renaming files was out of scope for my branch |
10:59 |
bshum |
If that happened |
10:59 |
jeff |
sylvar: 0458 is also not-present in master. |
11:00 |
jeff |
also, curse you for causing me to look |
11:00 |
sylvar |
jeff++ |
11:01 |
sylvar |
bless those who curse you, right? |
11:01 |
jeff |
heh |
11:02 |
sylvar |
jeff: there are 842 files in upgrade and the last one is 0844, so I suspect you can stop looking. :) |
11:02 |
jeff |
that's where i started. :-) |
11:02 |
jeff |
ls -1 | wc -l |
11:03 |
jeff |
followes by ls -1 | nl | less to make it rather easy to find. |
11:03 |
jeff |
followed. |
11:07 |
|
doug_ joined #evergreen |
11:09 |
doug_ |
How do you access the call number from a 3rd party client, given you already have the Marc Record? For example, an Android app. What is the API and what are its parameters? |
11:10 |
sylvar |
jeff: you could always submit a copy of 0561 as 0458 and 0539 :) |
11:12 |
|
nish joined #evergreen |
11:13 |
jeff |
doug_: do you have the marc record data, or a record id? |
11:13 |
doug_ |
We have the marc record id |
11:13 |
jeff |
doug_: are you interested in the bib call number contained in one of the marc fields, or do you want a list from evergreen of call numbers at specific libraries in the system? |
11:13 |
jeff |
doug_: do you want call numbers at all libraries, some libraries, or something else? |
11:14 |
doug_ |
All libraries |
11:14 |
jeff |
doug_: how concerned are you with size of data across the wire -- do you want a big response that includes a lot of the data you might want, or do you want a smaller, more focused response from the system? |
11:14 |
doug_ |
A smaller response |
11:15 |
doug_ |
If it is only possible to get the call number for a specific library, that is ok too |
11:15 |
|
mrpeters1 joined #evergreen |
11:18 |
dbwells |
doug_: not sure what you are up to, but there has been some renewed activity on the Android app from GSoC 2012. You can follow along here: http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/kenstir/android-minor-cleanup |
11:18 |
jeff |
doug_: a request like this will contain the full marcxml record as well as holdings. for example, curl http://evergreen.lib.in.us/opac/extras/supercat/retrieve/marcxml-full/record/15875572 |
11:18 |
jeff |
dbwells: good to hear! |
11:19 |
dbwells |
doug_: It's mostly built on older APIs, so there may be better ways to do some things, but it does work (mostly :) ) |
11:19 |
jeff |
doug_: for smaller, more focused responses, the api can be a little more complex. especially if you're part of the team still trying to work on an android client, i'd suggest looking at the newly-updated code at dbwells' link |
11:21 |
doug_ |
dbwells: Thanks for that link. I'll look more at that |
11:22 |
|
smyers_ joined #evergreen |
11:23 |
dbwells |
doug_: I supervised and reviewed most of the code there, but drizea and more recently kenstir have been doing the actual coding, so please do contact them if you are interested in collaborating. |
11:23 |
|
mrpeters joined #evergreen |
11:23 |
* Dyrcona |
shakes his head. "Humans." |
11:24 |
dbwells |
Dyrcona: Is that your thought, or a subject heading? ;) |
11:25 |
doug_ |
jeff: thanks, that does help a lot, and contains a lot of what I'm looking for. |
11:26 |
Dyrcona |
dbwells: Reaction. |
11:29 |
jeff |
"I just wish we could get course credit for this." |
11:31 |
Dyrcona |
dbwells / kmlussier: I'll apply the complete branch on lp 1244432 tonight for testing tomorrow. I want to reload my database to test a script for deleting tons of copies again tomorrow. |
11:31 |
pinesol_green |
Launchpad bug 1244432 in Evergreen "Browse search functions need to be 'stable'" (affected: 1, heat: 8) [Critical,New] https://launchpad.net/bugs/1244432 |
11:32 |
bshum |
jeff: Sounds like Marty to me. |
11:33 |
bshum |
:) |
11:33 |
jeff |
bshum: correct. |
11:33 |
bshum |
But I'm sure the line has been reused in other ways |
11:34 |
jeff |
paxed: regarding "Renew Patron" -- in our current translation workflow, is having a descriptive string ID enough? for example, "UPDATE_EXPIRE_DATE": "Update" |
11:39 |
doug_ |
jeff: Is there a URL that returns just the call number? That one you posted before had everything, which was great, but I'm unsure how to parse that to get just the call number |
11:40 |
jeff |
it's xml, so i'd recommend using xpath. |
11:43 |
doug_ |
jeff: ahh, okay. Thanks, will try that |
11:44 |
paxed |
jeff: all the translatable strings get pulled into the same msgid, and have a comment of where it is used. but if there's just "Renew", and in one place it means "Renew Patron" and somewhere else renewing something else ... |
11:45 |
paxed |
... the translation might be very very hard. (say, for example, the language may need a subject, there's no plain "renew" that would work) |
11:46 |
jeff |
sure, I get why "Renew" by itself is bad, I'm just wondering if there's context outside of the string that could come into play, so that the user-visible english string could still be "Renew" |
11:46 |
paxed |
the only good thing about Eg having ~40 different po-files is that there's not very often those collisions. |
11:47 |
paxed |
iirc, the context string isn't really supported. |
11:50 |
paxed |
jeff: but, yes, a descriptive string ID would be enough. |
11:53 |
jeff |
yeah, the "context" in question i was thinking of was the descriptive string id. :-) |
11:53 |
|
RoganH joined #evergreen |
11:53 |
jeff |
would there still be a collision, though? if the English string for foo.RENEW_ITEM and foo.RENEW_PATRON was "Renew", would it be imopssible to translate as two distinct words in another language? |
11:54 |
paxed |
if the strings were to appear in the same po-file, then yes, they'd be lumped together |
11:55 |
jeff |
okay. i think i understand now. thanks! |
11:55 |
|
michaels joined #evergreen |
11:56 |
paxed |
the descriptive string ID you mentioned is just used as a comment in the po-file, not a msgctxt |
11:59 |
|
rfrasur joined #evergreen |
12:09 |
rfrasur |
gmcharlt++ #opensrf release |
12:17 |
dconnor_ |
Is there, or is anyone working on, a fieldmapper system for taking a list of lists from a stored procedure and returning a list of objects (instead of calling the fieldmapper and hitting the DB recursively)? |
12:18 |
dconnor_ |
Or better yet, a function that takes a stored procedure and the relavent arguments and returns a list of mapped objects? |
12:19 |
Dyrcona |
dconnor_: Take a look at this branch: working/user/dyrcona/marque-export |
12:20 |
dconnor_ |
Dyrcona: I'll check it out. Thanks! |
12:20 |
csharp |
marque++ |
12:21 |
Dyrcona |
I am working on an improved version in working/user/dyrcona/lp1223903-MARC-export-Mk.2 as time permits. |
12:21 |
Dyrcona |
I like the version of from_bare_hash in the latter branch better. |
12:21 |
Dyrcona |
If you look at Marque.pm.in, it will show how you how to use fieldmapper with DBI/SQL queries. |
12:22 |
Dyrcona |
It is in the first branch. |
12:22 |
|
dbwells_ joined #evergreen |
12:23 |
Dyrcona |
I am using the first branch to export records for EBSCO, and it is much faster than the current marc_export. I'm just not satisfied with the design at the moment. |
12:25 |
dconnor_ |
Ya, I'm trying to cut down on recursive DB calls for performance. This looks promising! I appreciate it! |
12:26 |
|
mmorgan joined #evergreen |
12:26 |
Dyrcona |
dconnor_: Having more information about your specific problem might help. I often use fleshed retrievals in my custom scripts and when adding code to Evergreen. |
12:27 |
pastebot |
"Dyrcona" at 64.57.241.14 pasted "Fleshed search example" (15 lines) at http://paste.evergreen-ils.org/31 |
12:28 |
Dyrcona |
Just a sample from a script I'm working on at this moment. |
12:30 |
dconnor_ |
Dyrcona: I'm working on performance in the Holdings Maintenance system and have a stored procedure for grabbing everything in a single call and I end up with an array of arrays that needs to be mapped |
12:32 |
Dyrcona |
dconnor_: I think OpenILS/src/support_scripts/Marque.pm.in in the first branch does that with a map at one point. You will need the fieldmapper changes to add the from_bare_hash function. |
12:33 |
dconnor_ |
Dyrcona: Right on. Thanks so much for the help! |
12:36 |
Dyrcona |
yw |
12:39 |
Dyrcona |
my $title = $vol->record()->simple_record()->title(); # Lines like that make me wonder if I'm writing Perl or Java. :) |
12:41 |
Dyrcona |
Or C++ rather... |
12:41 |
Dyrcona |
@karma C |
12:41 |
pinesol_green |
Dyrcona: Karma for "C" has been increased 13 times and decreased 0 times for a total karma of 13. |
12:42 |
rfrasur |
clever |
12:43 |
rfrasur |
C gets automatic karma from its child (as it should be, I guess). |
12:47 |
Dyrcona |
C deserves more karma than does C++. |
12:48 |
* berick |
invents berick++ |
12:48 |
rfrasur |
y'all should invent yourselves as [you]++ |
12:49 |
rfrasur |
Dyrcona: I only have experience with C ++ and it was only a semester...so enough to say I learned something for a month or so...and then never use again. |
12:49 |
rfrasur |
(it was a fun elective though) |
12:50 |
|
Sato`kun joined #evergreen |
12:52 |
bshum |
dbwells: Tested browse with my test server and production data and you were right that it was pretty broken. |
12:52 |
bshum |
Applied those alter functions and things went smoothly after that. |
12:52 |
csharp |
csharp## |
12:53 |
bshum |
I'll do some more testing after lunch. |
12:53 |
rfrasur |
lol, csharp++ |
12:53 |
csharp |
@karma c++ |
12:53 |
pinesol_green |
csharp: c++ has neutral karma. |
12:53 |
csharp |
@karma c |
12:53 |
pinesol_green |
csharp: Karma for "c" has been increased 14 times and decreased 0 times for a total karma of 14. |
12:53 |
Dyrcona |
"I have always wished for my computer to be as easy to use as my telephone; my wish has come true because I can no longer figure out how to use my telephone." -- Bjarne Stroustrup, creator of c plus plus. |
12:57 |
ktomita |
bshum: What were those alter functions and where can I look at them? |
12:57 |
Dyrcona |
ktomita see lp 1244432 |
12:57 |
pinesol_green |
Launchpad bug 1244432 in Evergreen "Browse search functions need to be 'stable'" (affected: 1, heat: 8) [Critical,New] https://launchpad.net/bugs/1244432 |
12:58 |
ktomita |
Dyrcona: thanks |
12:59 |
dbwells |
bshum++ |
13:06 |
rfrasur |
There are few things that warm a director's heart more than hearing circulation staff do everything right with a patron account update. |
13:07 |
* rfrasur |
has a temporarily warm heart. |
13:07 |
|
test12345 joined #evergreen |
13:13 |
michaels |
Hi all, not sure if this is the best place to ask this but here goes. Is there a way to display large jackets using amazon added content? Currently our jackets only show a max height of 160px when clicked on even though the url indicates /ac/jacket/large. I've looked at the added content.pm file but thats about it. were on EG 4.3 if it matters. thanks |
13:17 |
* bshum |
wishes he was on EG 4.3..... |
13:17 |
kmlussier |
michaels: Heh, I'm guessing that's 2.3. :) |
13:17 |
rfrasur |
hehe |
13:18 |
rfrasur |
bshum++ #what exactly would that look like? |
13:18 |
kmlussier |
Some may argue that we should be on 4.x by now. |
13:18 |
Dyrcona |
5.0, but who's counting.... ;) |
13:19 |
bshum |
Technically we haven't supported Amazon jackets for a long time (pre 2.0 for me anyways) |
13:19 |
kmlussier |
Yup! Dyrcona was the someone I was thinking of. ;) |
13:20 |
bshum |
So I don't really know enough about it |
13:20 |
Dyrcona |
Was the Amazon code removed at some point, or was that something else? I seem to recall dbs working on something related to that. |
13:20 |
kmlussier |
rfrasur: I'm guessing 4.3 would have a speedy web-based client and hand out cupcakes when staff hit the mid-afternoon doldrums. |
13:21 |
bshum |
It was supposed to be removed and I thought it was. But it isn't. |
13:21 |
rfrasur |
Larger images would be nice (for Pinterest...I'm so ashamed) |
13:22 |
rfrasur |
kmlussier: that's what I was thinking too...maybe a nice latte...or something better for when working from home. |
13:22 |
bshum |
rfrasur: I've got some libs that use Pinterest to highlight their collections. |
13:23 |
jeff |
Dyrcona: I had a working branch that removed the OpenILS::WWW::AddedContent::Amazon, but couldn't figure out a politic way of wording the launchpad bug / pullrequest. |
13:23 |
michaels |
4.3 is awesome! google glass integration is limited but working :) |
13:23 |
rfrasur |
bshum: We use it for all our new stuff. It's linked into our website (and provides the update requirement when we can't get in an actually do real website updates). |
13:23 |
rfrasur |
michaels++ |
13:24 |
kmlussier |
michaels++ |
13:24 |
rfrasur |
o0(woah! I was thinking how much our landscapers needed to show up and clean up the yard this morning...and they're here...and I didn't call them) |
13:24 |
Dyrcona |
jeff: I have no problem being impolitic. |
13:24 |
jeff |
michaels: It is my understanding that Amazon has deprecated the API that is currently used in OpenILS::WWW::AddedContent::Amazon, and their documentation seems to have been updated to show the "new way", which is part of the "product advertising api" -- which may or may not be suitable for your use / general library use. |
13:24 |
rfrasur |
(telepathic facilities management) |
13:31 |
michaels |
jeff: should we be using another added content provider then? maybe were not using amazon now. how would i check? |
13:31 |
jeff |
michaels: check the added_content section of opensrf.xml |
13:34 |
michaels |
jeff: under added content it shows <module>OpenILS::WWW::AddedContent::Amazon</module> so I guess amazon is what we are using? |
13:40 |
jeff |
yep! |
13:40 |
bshum |
michaels: Just doing a google search like "amazon images 160px" it looks like it's Amazon that's passing back images of that size and no larger. |
13:40 |
bshum |
Evergreen probably is just doing what it can with what it gets. |
13:41 |
jeff |
i can confirm that there's no resizing being done on the evergreen side -- other than in some templates the html/css will set a client-side display height and/or width |
13:41 |
jeff |
(but even that is less common now, i think) |
13:41 |
csharp |
has anyone else seen a need for a single, unified holds pull list interface, rather than the four options that exist now? |
13:41 |
bshum |
csharp: It depends... |
13:41 |
bshum |
A long time ago |
13:41 |
* csharp |
is not sure of the history of each of those interfaces/options |
13:41 |
jeff |
csharp: yeah. it's one of our candidates for an early web staff ui. :-) |
13:41 |
bshum |
There was supposed to be a push towards using the simplified hold pull list |
13:41 |
bshum |
(the one made with dojo) |
13:42 |
csharp |
jeff: oh - awesome |
13:42 |
bshum |
And the xul ones are supposed to be deprecated for varying reasons |
13:42 |
jeff |
csharp: don't "awesome" me until something happens. :P |
13:42 |
csharp |
heh |
13:42 |
bshum |
Or rather, there have been bugs with the xul ones that nobody has had time to fix |
13:42 |
* rfrasur |
isn't sure what all that means. |
13:42 |
csharp |
@awesome jeff |
13:42 |
pinesol_green |
csharp: http://images.cryhavok.org/d/1291-1/Computer+Rage.gif |
13:42 |
csharp |
bshum: I see |
13:43 |
bshum |
That said, the xul alternate holds list shows more information about the title (like the subtitle portions) that make it clearer to know what copy you're pulling |
13:43 |
bshum |
I filed bugs on that for the simplified pull list, but that has yet to be modified either |
13:43 |
csharp |
yeah - was going to say - that looks like the best one |
13:44 |
bshum |
I think back in 2.3(?) when we added simplified pull list, it was generally agreed that was the future. |
13:44 |
jeff |
i'd like to survey/observe staff to see which ones they currently favor, and if it differs per library/department |
13:44 |
csharp |
that one looks like it has configurability, but it's sloooww for us |
13:44 |
bshum |
csharp: https://bugs.launchpad.net/evergreen/+bug/1048822 is the bug I reported. |
13:44 |
pinesol_green |
Launchpad bug 1048822 in Evergreen "Simplified Pull List - Need fuller title" (affected: 3, heat: 16) [Wishlist,Confirmed] |
13:45 |
|
RoganH joined #evergreen |
13:45 |
kmlussier |
The one potential issue I've heard about using just the simplified pull list is that you can perform actions on the holds from the other pull lists. You can't do so from the other pull lists. Though I would be curious to know how often it really happens in practice. |
13:45 |
csharp |
bshum: yeah - one of our libraries just pointed out the lack of subtitle on whichever one they're currently using |
13:45 |
csharp |
so I decided to investigate the pros/cons of the various options |
13:46 |
bshum |
Meh xul is dead :) |
13:47 |
kmlussier |
Oh, and the simplified pull list doesn't always display everthing that needs to be pulled. It prints them all, but there are often holds missing from the display. https://bugs.launchpad.net/evergreen/+bug/1099567 |
13:47 |
pinesol_green |
Launchpad bug 1099567 in Evergreen "simplified holds pull list interface does not always display all holds" (affected: 2, heat: 10) [Undecided,Triaged] |
13:48 |
csharp |
bshum: well, the xul one can leverage the column picker and the results can be exported into excel/calc and sorted however people need |
13:48 |
csharp |
so in some ways, it's the best from the "everything should be configurable" starting place ;-) |
13:49 |
csharp |
kmlussier: interesting |
13:50 |
bshum |
Yeah, eyeballing the dojo always sucks and people get really weirded out by that. |
13:50 |
* csharp |
is not a fan of the dojo interfaces :-/ |
13:50 |
bshum |
Deep down, I prefer the xul one too |
13:51 |
rfrasur |
hmm, pardon...but I know there are three active development projects right now. MassLNC's MVF, SCLends Inventory...and does anyone remember the other one? |
13:51 |
Dyrcona |
I'm working on billing fixes/changes for MassLNC. |
13:52 |
eeevil |
rfrasur: as for proposed dev, there are several more than that, actually... but you might be thinking of ESI's proposed staff client prototype |
13:52 |
rfrasur |
yes, proposed...eeevil, that's it. |
13:53 |
rfrasur |
sorry...firing on less cylinders than usual. |
13:53 |
kmlussier |
rfrasur: Those are the three that are looking for money. |
13:53 |
rfrasur |
That's what I was trying to remember. Many thanks. |
13:53 |
csharp |
@eightball will they find money? |
13:53 |
pinesol_green |
csharp: It is so. |
13:54 |
rfrasur |
Yay! |
14:03 |
|
leonard_ joined #evergreen |
14:04 |
|
ningalls joined #evergreen |
14:04 |
leonard_ |
hi all, could someone helpme please, I still can´t get the hold requests from the staff client, It is always empty |
14:05 |
csharp |
leonard_: is your hold targeter utility script running? |
14:05 |
leonard_ |
csharp: you mean that if I run the cron /openils/bin/hold_targeter.pl /openils/conf/opensrf_core.xml ? |
14:06 |
csharp |
yes |
14:07 |
|
ningalls joined #evergreen |
14:07 |
leonard_ |
yes I run it, and then I try Pull List for hold requests, but It doesn´t show anything |
14:10 |
csharp |
leonard_: have you checked your logs for messages about holds? |
14:11 |
jeff |
leonard_: holds will not be targeted if the library associated with your staff client session is not considered "open" today in Evergreen. |
14:12 |
jeff |
leonard_: are these new holds, migrated holds, existing holds that were placed in evergreen and not migrated? |
14:13 |
leonard_ |
jeff: I am testing how holds works, I am placing a hold in TPAC, inmediatly I run the hold targeter cron, and then I try Pull List for hold requests |
14:14 |
jeff |
if you look at the patron account, do the holds show up, and do they have a status and/or a value in "current copy"? |
14:14 |
leonard_ |
no, the status is in waiting for copy |
14:15 |
leonard_ |
the currente copy value is No Copy |
14:16 |
Dyrcona |
leonard_: You are 100% positive that you have a copy that could fill the hold given the copy's status, location, etc.? |
14:16 |
Dyrcona |
leonard_: You also have hold matrix matchpoints that allow the copy to fill the hold? |
14:16 |
jeff |
(and that the copy exists at the same library location that your workstation is associated with) |
14:17 |
jeff |
i.e., don't have a hold on a title whose only copies exist at BR1 and expect it to show on the pull list when you're logged in as being at BR2) |
14:18 |
leonard_ |
yes, actually we just have one branch (BR1) |
14:45 |
|
ericar_ joined #evergreen |
14:52 |
|
stevenyvr2 joined #evergreen |
15:00 |
|
mjingle joined #evergreen |
15:03 |
|
RoganH joined #evergreen |
15:09 |
bshum |
dbwells++ |
15:23 |
csharp |
okay testing the fix for bug 1244432 - before the fix, I saw some browse queries running for 1.5+ days |
15:23 |
pinesol_green |
Launchpad bug 1244432 in Evergreen "Browse search functions need to be 'stable'" (affected: 1, heat: 8) [Critical,New] https://launchpad.net/bugs/1244432 |
15:24 |
csharp |
now I see 11 seconds - still not great |
15:24 |
csharp |
but that may be may environment |
15:24 |
Dyrcona |
csharp: I killed some this morning running since the 22nd before that patch. |
15:24 |
csharp |
tons of data, not a super-powerful machine |
15:24 |
* Dyrcona |
recalls mentioning having queries running for over 10 days before in channel. |
15:26 |
* dbs |
guesses bug #1243023 should get targeted |
15:26 |
pinesol_green |
Launchpad bug 1243023 in Evergreen "Browse catalogue titles are doubly escaped?" (affected: 1, heat: 6) [Undecided,New] https://launchpad.net/bugs/1243023 |
15:32 |
csharp |
dbwells: just pushed a signoff branch for lp 1244432 |
15:32 |
pinesol_green |
Launchpad bug 1244432 in Evergreen "Browse search functions need to be 'stable'" (affected: 1, heat: 8) [Critical,New] https://launchpad.net/bugs/1244432 |
15:34 |
bshum |
I can push that through once I get home if dbwells doesn't beat me to it |
15:34 |
Dyrcona |
I can push it right now, but I was waiting to look at it again in the morning. |
15:35 |
Dyrcona |
If someone beats me to it, that's cool. |
15:38 |
rfrasur |
(we're naming the popcorn station at tonight's Halloween party "Children of the [pop]Corn" |
15:39 |
bshum |
Ha! |
15:39 |
rfrasur |
:D |
15:41 |
dbwells |
csharp: bshum: Dyrcona: thanks all for the review. I wont have a chance to push it today, so to whomever gets to it first, feel free :) |
15:41 |
csharp |
dbwells++ |
15:41 |
|
sseng joined #evergreen |
15:41 |
|
dconnor joined #evergreen |
15:42 |
|
ktomita joined #evergreen |
15:42 |
|
fparks_ joined #evergreen |
15:42 |
|
smyers__ joined #evergreen |
15:42 |
|
sseng joined #evergreen |
15:43 |
Dyrcona |
Grabbing 0845. |
15:45 |
|
dconnor joined #evergreen |
15:47 |
pinesol_green |
[evergreen|Dan Wells] Make browse pivot DB funcs STABLE - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=4b35a8c> |
15:47 |
pinesol_green |
[evergreen|Jason Stephenson] Stamping upgrade script for 0845: browse pivots should be stable. - <http://git.evergreen-ils.org/?p=Evergreen.git;a=commit;h=c9a4dee> |
15:59 |
|
sseng_ joined #evergreen |
15:59 |
|
ktomita joined #evergreen |
15:59 |
|
fparks joined #evergreen |
15:59 |
|
smyers_ joined #evergreen |
16:03 |
|
dconnor joined #evergreen |
16:16 |
jeff |
cute. if a dojo (1.3, at least) ValidationTextBox is both required and has an isValid function, the standard required behavior breaks after the first time the input gets focus. |
16:17 |
jeff |
(at least, that seems to be what i've narrowed this down to) |
16:27 |
|
hbrennan joined #evergreen |
16:33 |
|
mrpeters left #evergreen |
17:01 |
|
mtj_- joined #evergreen |
17:03 |
|
eby_ joined #evergreen |
17:09 |
|
mmorgan left #evergreen |
17:14 |
|
rri_ joined #evergreen |
17:14 |
|
berickm joined #evergreen |
17:47 |
|
Bmagic joined #evergreen |
18:59 |
|
stevenyvr2 left #evergreen |
20:10 |
|
serflog joined #evergreen |
20:10 |
|
Topic for #evergreen is now Welcome to the #evergreen library system channel! | We are publicly logged. | Large pastes at http://paste.evergreen-ils.org |
21:00 |
|
mjingle left #evergreen |