![https://scontent-a.xx.fbcdn.ne[…]4_101205169_o.jpg](https://scontent-a.xx.fbcdn.net/hphotos-ash3/t31/q74/s720x720/1956821_371391689669494_101205169_o.jpg){kind=link}
![http://wac.450f.edgecastcdn.ne[…]er-1303842437.jpg](http://wac.450f.edgecastcdn.net/80450F/comicsalliance.com/files/2011/04/sadcover-1303842437.jpg){kind=link}
← Previous day | Today | Next day → | Search | Index
All times shown according to UTC.
| Time | Nick | Message |
|---|---|---|
| 00:39 | BobB joined #koha | |
| 00:43 | talljoy joined #koha | |
| 00:54 | talljoy joined #koha | |
| 00:54 | BobB joined #koha | |
| 01:06 | bag | @seen rangi |
| 01:06 | huginn | bag: rangi was last seen in #koha 6 hours, 15 minutes, and 26 seconds ago: <rangi> morning |
| 01:06 | rangi | sup |
| 01:20 | shalom joined #koha | |
| 01:21 | shalom left #koha | |
| 01:42 | BobB joined #koha | |
| 01:54 | jenkins_koha | Starting build #382 for job master_maria (previous build: SUCCESS) |
| 01:58 | Starting build #1658 for job Koha_master (previous build: SUCCESS) | |
| 02:00 | huginn | New commit(s) kohagit: Bug 11705 - change versioning of build-git-snapshot <http://git.koha-community.org/[…]9b103892eb7dc6721> |
| 02:23 | eythian | damnit gmcharlt, I was just about to submit patches for that configuration thing, and then saw you'd already done it :) |
| 02:24 | gmcharlt | eythian: just trying to focus your attention on debconf support ;) |
| 02:32 | * mtompset | laughs. |
| 02:49 | jenkins_koha | Project master_maria build #382: SUCCESS in 55 min: http://jenkins.koha-community.[…]master_maria/382/ |
| 02:49 | Robin Sheat: Bug 11705 - change versioning of build-git-snapshot | |
| 02:49 | huginn | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11705 normal, P4, ---, robin, Pushed to Master , New Debian policy enforcement is breaking the build script |
| 03:11 | mtompset | woah! Nice flood of new stuff. |
| 03:45 | wajasu | mtompset: i talk a bit earlier 17;15 -17:46 about my current strategy for getting the facet counts correct for OpacHiddenItems. anything you can think of that i might have missed. i'm still working inthe patch. |
| 03:47 | mtompset | Just scrolled back... |
| 03:48 | Okay. there are THREE hiding mechanisms in Koha currently. | |
| 03:48 | OpacSuppression. | |
| 03:48 | wahanui | i think opacsuppression is different. |
| 03:48 | mtompset | It's at the Biblio level. |
| 03:48 | OpacHiddenItems. | |
| 03:48 | wahanui | opachiddenitems is a YAML rule set for deciding whether to hide an item or not. |
| 03:48 | mtompset | And is at the item level. |
| 03:49 | And then... let's see if I remember where that third one is. | |
| 03:50 | Actually... four now that I think some... | |
| 03:51 | There is a visibility tag which affects the framework... | |
| 03:51 | so it is individual fields. | |
| 03:53 | My gut is saying something $n. | |
| 03:55 | and I believe it is only in the biblioitems marcxml. | |
| 03:56 | If you suppress a biblio with OpacSuppression, you clearly want all the items for it hidden too. :) | |
| 03:56 | -- sorry, having to reindex. | |
| 03:58 | jenkins_koha | Project Koha_master build #1658: SUCCESS in 2 hr 2 min: http://jenkins.koha-community.[…]Koha_master/1658/ |
| 03:58 | Robin Sheat: Bug 11705 - change versioning of build-git-snapshot | |
| 03:58 | mtompset | If you hide items with OpacHiddenItems, you may or may not want the Biblio hidden, but there is no way to tell, since your intent is items, not biblios. |
| 03:58 | huginn | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11705 normal, P4, ---, robin, Pushed to Master , New Debian policy enforcement is breaking the build script |
| 03:59 | mtompset | And if your frameworks hides various fields in different ways, the only tells you about that particular field/subfield pairing. |
| 04:00 | If you hide everything at the framework level, are you trying to hide items and biblios too? | |
| 04:01 | Ahh... 942$n. | |
| 04:02 | I guess that is the OpacSuppression? | |
| 04:03 | Okay... three way still then. :) | |
| 04:06 | I'm wondering if using the OpacSuppression system preference would be an incorrect overloading of functionality. | |
| 04:07 | That is, if all the items are hidden, the OpacSuppression system preference would indicate to hide the biblio. Of course, it may be cleaner to create a separate system preference. | |
| 04:09 | jenkins_koha | Starting build #383 for job master_maria (previous build: SUCCESS) |
| 04:12 | huginn | New commit(s) kohagit: Bug 108661: (follow-up) enshrine letting CardnumberLength specify just a maximum <http://git.koha-community.org/[…]a7cc3461c3f8c3d48> / Bug 10861: (follow-up) use minlength/maxlength rather than pattern for client-side... <http://git.koha-community.org/[…]0e60f9a1da53476d9> / Bug 10861: DBrev 3.15.00.023 <http://git.k |
| 04:12 | jenkins_koha | Starting build #1659 for job Koha_master (previous build: SUCCESS) |
| 04:14 | mtompset | oh, and wajasu bugs 10589 10895 10900 |
| 04:16 | So, you can override OpacHiddenItems based on patron category. :) | |
| 04:18 | And speaking of hiding at a framework level... | |
| 04:19 | 11592 | |
| 04:24 | Oak joined #koha | |
| 04:25 | Oak | kia ora #koha |
| 04:25 | hello mtompset! | |
| 04:25 | mtompset | Greetings, Oak. :) |
| 04:25 | Oak | :) |
| 04:27 | mtompset | wajasu: Does that rambling all make some sense? |
| 04:46 | Have a great day, #koha. Bye, Oak wajasu. | |
| 04:47 | wajasu | mtpompset: makes sense. |
| 04:48 | thx | |
| 05:04 | jenkins_koha | Project master_maria build #383: SUCCESS in 55 min: http://jenkins.koha-community.[…]master_maria/383/ |
| 05:04 | * Jonathan Druart: Bug 10861: Add a check on cardnumber length | |
| 05:04 | * Marcel de Rooy: Bug 10861: (follow-up) for typo in patrons.pref | |
| 05:04 | * Jonathan Druart: Bug 10861: (follow-up) various refactoring | |
| 05:04 | * Kyle M Hall: Bug 10861: (follow-up) Clarify preference description | |
| 05:04 | * Galen Charlton: Bug 10861: DBrev 3.15.00.023 | |
| 05:04 | huginn | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=10861 enhancement, P5 - low, ---, jonathan.druart, Pushed to Master , Add a check on cardnumber length |
| 05:04 | jenkins_koha | * Galen Charlton: Bug 10861: (follow-up) use minlength/maxlength rather than pattern for client-side validation |
| 05:04 | * Galen Charlton: Bug 108661: (follow-up) enshrine letting CardnumberLength specify just a maximum | |
| 05:04 | huginn | 04Bug http://bugs.koha-community.org[…]bug.cgi?id=108661 was not found. |
| 06:13 | Oak joined #koha | |
| 06:27 | cait joined #koha | |
| 06:40 | jenkins_koha | Project Koha_master build #1659: SUCCESS in 2 hr 29 min: http://jenkins.koha-community.[…]Koha_master/1659/ |
| 06:40 | * Jonathan Druart: Bug 10861: Add a check on cardnumber length | |
| 06:40 | * Marcel de Rooy: Bug 10861: (follow-up) for typo in patrons.pref | |
| 06:40 | * Jonathan Druart: Bug 10861: (follow-up) various refactoring | |
| 06:40 | * Kyle M Hall: Bug 10861: (follow-up) Clarify preference description | |
| 06:40 | * Galen Charlton: Bug 10861: DBrev 3.15.00.023 | |
| 06:40 | * Galen Charlton: Bug 10861: (follow-up) use minlength/maxlength rather than pattern for client-side validation | |
| 06:40 | huginn | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=10861 enhancement, P5 - low, ---, jonathan.druart, Pushed to Master , Add a check on cardnumber length |
| 06:40 | jenkins_koha | * Galen Charlton: Bug 108661: (follow-up) enshrine letting CardnumberLength specify just a maximum |
| 06:40 | huginn | 04Bug http://bugs.koha-community.org[…]bug.cgi?id=108661 was not found. |
| 06:40 | sophie_m joined #koha | |
| 06:50 | cait | good morning #koha :) |
| 07:38 | reiveune joined #koha | |
| 07:38 | reiveune | hello |
| 07:56 | lds joined #koha | |
| 07:58 | Joubu joined #koha | |
| 07:58 | Joubu | hi |
| 07:59 | JesseM joined #koha | |
| 07:59 | JesseM | Good Morning |
| 08:04 | cait joined #koha | |
| 08:05 | bag | good morning |
| 08:05 | magnuse joined #koha | |
| 08:06 | * magnuse | waves in norwegian |
| 08:08 | pianohacker | how does one wave in norwegian exactly? very properly and only when it is your turn? |
| 08:08 | (also hello) | |
| 08:08 | ashimema joined #koha | |
| 08:18 | alex_a joined #koha | |
| 08:19 | gerundio joined #koha | |
| 08:27 | JesseM | @later tell talljoy that Foxriver Server is not yet set up. I will take care of SIP and OCLC connexion once its up. |
| 08:27 | huginn | JesseM: The operation succeeded. |
| 08:30 | magnuse | pianohacker: you missed it? too late now! :-) |
| 08:30 | developer's meeting in 6,5 hours, unless i am mistaken! http://wiki.koha-community.org[…]ng,_March_12_2014 | |
| 08:30 | cait | oh |
| 08:30 | thx for reminding us | |
| 08:31 | magnuse | please double check the time, the links on the wiki were wrong... |
| 08:32 | cait | JesseM: bug 10936 |
| 08:32 | huginn | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=10936 enhancement, P5 - low, ---, oleonard, NEW , Standardnumber doubled up in Advanced search - more options |
| 08:32 | cait | JesseM: maybe bug 11762 - moving the code from the detail pge to result lists as well |
| 08:32 | huginn | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11762 normal, P5 - low, ---, gmcharlt, NEW , Sequence of MARC21 245 subfields different on XSLT result list and detail page |
| 08:36 | JesseM | thanks cait I will take a look at these |
| 08:42 | magnuse | numbers? we've got numbers! http://wiki.koha-community.org[…]lle_hackfest_2014 |
| 08:45 | paul_p joined #koha | |
| 08:50 | cait | magnuse++ |
| 08:50 | thx for numbers! | |
| 08:52 | nlegrand_ | Hey #koha :) |
| 08:52 | ho | |
| 08:55 | sophie_m joined #koha | |
| 08:59 | pianohacker | magnuse++ |
| 08:59 | * magnuse | bows |
| 09:03 | tries to resurrect bug 7844 | |
| 09:03 | huginn | 04Bug http://bugs.koha-community.org[…]w_bug.cgi?id=7844 enhancement, P5 - low, ---, dpavlin, In Discussion , plack scripts for developers |
| 09:09 | magnuse_ joined #koha | |
| 09:10 | reiveune1 joined #koha | |
| 09:12 | sophie_m joined #koha | |
| 09:13 | cait joined #koha | |
| 09:22 | Joubu joined #koha | |
| 09:34 | gerundio joined #koha | |
| 09:51 | magnuse | anyone else having trouble with "git bz attach"? i get lots of html in my terminal... |
| 09:56 | gah, my fault | |
| 09:56 | dpavlin++ | |
| 09:58 | bug 7844 is signed off now - i hope it can make it into master quickly :-) | |
| 09:58 | huginn | 04Bug http://bugs.koha-community.org[…]w_bug.cgi?id=7844 enhancement, P5 - low, ---, dpavlin, Signed Off , plack scripts for developers |
| 10:04 | cait | magnuse: hm today so far it works quite well for me |
| 10:04 | magnuse: might be you are not logged in correctly to bugzilla or trying to switch to astatus that does not work without switching to some other status first | |
| 10:09 | pianohacker | does anyone know how to remotely debug perl? I'm having a frustrating error that I can't duplicate from the command line... |
| 10:10 | bag | pianohacker: get binoculars and roll the chair back :) is that remote enough? |
| 10:11 | pianohacker | /me slaps bag around with a large http://search.cpan.org/dist/En[…]gger.pod#From_gdb |
| 10:11 | curse you space | |
| 10:12 | bag | I think we are within an hour of cheese! |
| 10:16 | sophie_m1 joined #koha | |
| 10:29 | paul_p joined #koha | |
| 10:31 | magnuse | cait: yeah, it was the latter |
| 10:32 | cait | magnuse: been there... |
| 10:36 | Oak | magnuse++ |
| 10:36 | cait++ | |
| 10:42 | gaetan_B joined #koha | |
| 10:44 | sophie_m joined #koha | |
| 10:45 | magnuse_ joined #koha | |
| 10:46 | reiveune joined #koha | |
| 11:01 | gerundio joined #koha | |
| 11:03 | sophie_m joined #koha | |
| 11:05 | cait joined #koha | |
| 11:10 | magnuse joined #koha | |
| 11:11 | reiveune joined #koha | |
| 11:11 | sophie_m joined #koha | |
| 11:12 | JesseM_ joined #koha | |
| 11:14 | cait | Please all remember to assign a bug to yourself :) |
| 11:14 | drojf joined #koha | |
| 11:18 | * cait | sends some cookies to the god of networks |
| 11:19 | magnuse | cait: assign a bug to ourselves? |
| 11:19 | cait | like fill in the assignee filed |
| 11:19 | field | |
| 11:19 | because some bugsin the queues are assigned to the bug list, instead of to the patch writer :) | |
| 11:21 | fridolin joined #koha | |
| 11:26 | magnuse | ah, thanks cait |
| 11:28 | jwagner joined #koha | |
| 11:28 | bag | magnuse: |
| 11:28 | JesseM joined #koha | |
| 11:56 | drojf joined #koha | |
| 11:58 | meliss joined #koha | |
| 12:00 | fridolin | Giannis Kourmoulis made first patch : BZ11927 : add greek sorting chr file |
| 12:00 | whouuu | |
| 12:08 | cait joined #koha | |
| 12:11 | Joubu joined #koha | |
| 12:15 | barton joined #koha | |
| 12:17 | marcelr joined #koha | |
| 12:18 | marcelr | hi #koha, hi marseille |
| 12:18 | Dyrcona joined #koha | |
| 12:18 | cait | hi marcelr :) |
| 12:18 | marcelr | :) |
| 12:19 | cait | having lunch right now - today is cheese and salad |
| 12:19 | so it might get a bit louder in a bit again :) | |
| 12:19 | marcelr | good |
| 12:28 | oleonard joined #koha | |
| 12:29 | oleonard | Hi #koha |
| 12:32 | * cait | waves at oleonard with a piece of cheese in her hand |
| 12:33 | * oleonard | grumbles at his no cheese |
| 12:33 | * cait | hides it away |
| 12:33 | cait | we need teleporters... |
| 12:40 | tcohen joined #koha | |
| 12:40 | magnuse | giannis++ |
| 12:40 | tcohen | morning |
| 12:42 | nengard joined #koha | |
| 12:50 | cait | morning tcohen :) |
| 12:51 | oleonard: trying to get my queue down | |
| 12:51 | but it's hard! | |
| 12:51 | well 'the qa queue' :) | |
| 12:53 | oleonard | Something has changed: Now after a db update Koha doesn't ask me to log in as a regular user anymore. |
| 12:54 | I'm simply left logged in as the database user. | |
| 12:56 | pianohacker | oleonard: Try going to the opac not logged out and clicking "log in" |
| 12:57 | *not logged in | |
| 12:57 | oleonard | Why? |
| 12:58 | pianohacker | oleonard: Currently horribly breaks, wondering if related |
| 12:59 | oleonard | Not seeing a problem pianohacker |
| 12:59 | pianohacker | aw bugger |
| 12:59 | jcamins | Works fine for me. |
| 12:59 | pianohacker | try refreshing the page? |
| 12:59 | oleonard | Bootstrap theme? What am I looking for? |
| 13:00 | pianohacker | oleonard: the second time I load the page it gives me 'Can't use an undefined value as an ARRAY reference at /usr/lib/perl5/DBI.pm line 2064' |
| 13:00 | oleonard | Load which page? |
| 13:00 | marcelr | oleonard: i did the upgrade to current master just now and had to login again |
| 13:01 | pianohacker | due to get_template... returning rather than showing the login screen and getalerts being called with an undefined borrowernnumber |
| 13:01 | oleonard: if you go to opac-user.pl then refresh the page, is when it happens for me | |
| 13:02 | and I don't think the theme has a chance to do anything yet | |
| 13:04 | oh bugger it's my own code leaving a broken session behind, nvm... | |
| 13:08 | fridolin1 joined #koha | |
| 13:11 | pianohacker | ... it's a heisenbug |
| 13:11 | oleonard: any idea why your problem is happening? | |
| 13:11 | oleonard | After watching Breaking Bad "heisenbug" sounds like a bug that cooks meth. |
| 13:12 | bag | oh oleonard we had froglegs last night |
| 13:13 | oleonard | bag: Did it make you feel like Doc Hopper in "The Muppet Movie?" |
| 13:14 | rhcl joined #koha | |
| 13:14 | bag | actually I thought of you and was like - no way I can go home without having froglegs - owen would be disappointed ;) |
| 13:14 | oleonard | :D |
| 13:15 | jcamins | We've been considering buying frogs' legs. |
| 13:15 | They started stocking them at our grocery store. | |
| 13:15 | They also sell frogs, but I'm not really sure how one butchers a live frog. | |
| 13:16 | rhcl | @wunder 64507 |
| 13:16 | oleonard | jcamins: You get him hooked on cigarettes and way 20-30 years. |
| 13:16 | huginn | rhcl: The current temperature in Wyatt Park, St Joseph, Missouri is -0.5°C (8:16 AM CDT on March 12, 2014). Conditions: Clear. Humidity: 78%. Dew Point: -4.0°C. Windchill: 0.0°C. Pressure: 30.19 in 1022 hPa (Rising). |
| 13:16 | oleonard | wait, that is. |
| 13:16 | jcamins | oleonard: I feel like the meat might end up a bit tough. |
| 13:17 | oleonard | If I log out, set back my Koha version, and trigger the update that way I stay logged in as the db user. |
| 13:18 | If I log in, set back the Koha version, refresh the page, go through update, I stay logged in as myself. | |
| 13:18 | bag | you fill the frog up with buckshot - like the mark twain story - so the frog can't jump away from you |
| 13:19 | "The Celebrated Jumping Frog of Calaveras County" becomes ""The Celebrated Jumping Frog of Camins Apt NY" | |
| 13:27 | jcamins | lol |
| 13:30 | Joubu joined #koha | |
| 13:38 | francharb | good morning #koha |
| 13:39 | oleonard | Hi francharb |
| 13:39 | Dyrcona | jcamins: You never dissected frogs in biology class, I take it? |
| 13:39 | jcamins | Dyrcona: nope. |
| 13:39 | Dyrcona: and what we did dissect arrived pre-killed. | |
| 13:41 | francharb | jcamins, wait, they sell live frogs? |
| 13:41 | jcamins | francharb: yes. |
| 13:41 | Dyrcona | francharb: It's easier than catching your own. |
| 13:42 | jcamins | They're pretty big. |
| 13:42 | francharb | do they sell all kind of animals to be butchered? |
| 13:42 | laurence joined #koha | |
| 13:42 | laurence left #koha | |
| 13:42 | jcamins | francharb: nah, mostly just frogs and seafood. |
| 13:43 | cait | frogs... and seafood... |
| 13:43 | francharb | weird |
| 13:43 | jcamins | francharb: presumably if you don't live in a big city you can get other animals. |
| 13:44 | In Morocco you commonly purchase your birds and goats live, too. | |
| 13:44 | francharb | I guess but I'm such a city kind that it never cross my mind |
| 13:45 | jcamins | Here in New York, I haven't seen anyone with a live goat. |
| 13:45 | francharb | ahahahah |
| 13:46 | fridolin joined #koha | |
| 13:47 | francharb | lets focus on frogs. Do they give you a bag of live frogs? |
| 13:47 | Can you pick them | |
| 13:47 | ? | |
| 13:48 | jcamins | You choose them one at a time. |
| 13:49 | cait | so now francharb is going to tell us on which criteria best to pick live frogs? |
| 13:49 | francharb | dont tell me where it is or I'll go next time I'll be in New york and buy them all |
| 13:49 | jcamins | You like frogs a lot? |
| 13:49 | francharb | I dont want them to be cooked |
| 13:49 | magnuse | dpavlin++ for signing off bug 10003 |
| 13:49 | francharb | ;) |
| 13:49 | huginn | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=10003 enhancement, P5 - low, ---, tomascohen, Signed Off , koha-* scripts (packages) should provide tab-completion in bash |
| 13:50 | magnuse | francharb: you prefer them raw? ;-) |
| 13:50 | jcamins | Ooh, frog sushi! |
| 13:51 | * oleonard | donates to francharb's frog sanctuary |
| 13:51 | francharb | \0/ |
| 13:52 | ColinC joined #koha | |
| 13:52 | francharb | Because, in the end, frogs or snails, it only tastes butter, garlic and persley |
| 13:53 | jcamins | francharb: this is the kind: http://en.wikipedia.org/wiki/Chinese_edible_frog |
| 13:53 | francharb | so I dont get the point to eat them |
| 13:53 | ahah | |
| 13:54 | pianohacker | frog legs are surprisingly good |
| 13:54 | like really tender and tasty funny-shaped chicken | |
| 13:55 | francharb | pianohacker, anithing with butter, garlic and parsley is good |
| 13:55 | ;) | |
| 13:55 | pianohacker | true! |
| 13:55 | Dyrcona | pianohacker: is right. They are better than snails. Plain snails don't taste like anything. |
| 13:55 | tcohen | dpavlin++ |
| 13:55 | francharb | who eats plain snails anyway? |
| 13:56 | oleonard | Crows? |
| 13:56 | francharb | oleonard, true! |
| 13:56 | Dyrcona | Me, if I've run out of earth worms. |
| 13:58 | * dpavlin | hides in the corner -- I had good intention to sign off few LDAP patches, but I had just to leave comments on them instead |
| 13:58 | francharb | ok! thanks for the laugh. Now, lets go back to salt mine! |
| 13:58 | :) | |
| 13:58 | edveal joined #koha | |
| 13:59 | rocio joined #koha | |
| 13:59 | maximep joined #koha | |
| 14:01 | * Dyrcona | goes back to figuring out what happened to open-ils.storage.biblio.multiclass,search_fts in Evergreen. |
| 14:03 | pianohacker | http://devopsreactions.tumblr.[…]endors-benchmarks |
| 14:03 | gmcharlt | @quote random |
| 14:03 | huginn | gmcharlt: Quote #61: "*wizzyrea doesn't like hoity toity technical talk...<wizzyrea> wait, that's a lie" (added by jdavidb at 05:19 PM, February 24, 2010) |
| 14:04 | paul_p joined #koha | |
| 14:04 | jcamins | pianohacker: that is remarkably convincing running in place. |
| 14:04 | pianohacker | haha right? |
| 14:05 | I also love how kohadevreactions went on a massive jennifer lawrence kick for a bit | |
| 14:06 | oleonard | Whatever the percentage of Jennifer Lawrence gifs on kohadevreactions it can't match that of the internet as a whole. |
| 14:07 | fridolin joined #koha | |
| 14:07 | jcamins | I just watched the Hunger Games film a few weeks or months ago, and I have no recollection of Katniss looking like her. |
| 14:08 | pianohacker | that's because the internet defies silly math by being roughly 137% jennifer lawrence |
| 14:08 | especially reddit | |
| 14:08 | oleonard | On the other hand, the percentage of Deep Space 9 gifs on kohadevreactions seems a little out of proportion |
| 14:08 | jcamins | I just remember that the Hunger Games types seemed all wrong to me. |
| 14:10 | oleonard | jcamins: ? |
| 14:10 | jcamins | oleonard: I just looked up Jennifer Lawrence on Wikipedia, and learned that she was the star in Hunger Games. |
| 14:11 | I don't remember the person in the Hunger Games looking like her, but I thought that everyone looked different from how they were described in the book. | |
| 14:11 | gmcharlt | https://www.youtube.com/watch?v=r3CuL7N82-k |
| 14:13 | jcamins | lol |
| 14:14 | * oleonard | gets in line for the Katniss vs. Predator movie |
| 14:15 | pianohacker | gmcharlt++ |
| 14:16 | for pushing like a madman | |
| 14:16 | I would be tweeting in response but I have no twitter and the reaction from most of my fb would be "u wot m8" | |
| 14:17 | gmcharlt | pianohacker: oh, but that's part of the fun of mixing in work stuff on Facebook! |
| 14:19 | marcelr | gmcharlt++ #trying to clear the PQA queue |
| 14:20 | pianohacker | well, here's the reverse (my parent's new kitten): https://scontent-a.xx.fbcdn.ne[…]4_101205169_o.jpg |
| 14:20 | gmcharlt | color-words++ |
| 14:20 | * gmcharlt | melts into a puddle |
| 14:20 | jcamins | pianohacker: it should wear a bowtie. |
| 14:20 | pianohacker | she would look adorable in a bowtie |
| 14:21 | before, from what my parents have told me, squirming out of it and ripping it to shreds | |
| 14:21 | jcamins | pianohacker: http://www.thatcutesite.com/ju[…]th-a-bow-tie.html |
| 14:22 | pianohacker | I almost just squeed in the middle of hackfest |
| 14:22 | gmcharlt | jcamins: I believe the gauntlet has just been thrown |
| 14:23 | jcamins | gmcharlt: heh. That's about the cutest picture ever, though, so I'm not sure what could top it. |
| 14:25 | mtompset_away joined #koha | |
| 14:26 | mtompset | Greetings, #koha. |
| 14:27 | I was noticing that .019 and .020 for the master use the older version number comparison method. Shouldn't we be using CheckVersion() on a forward basis? | |
| 14:28 | gmcharlt | mtompset: indeed; I'll correct that the next time I push an updatedatabase patch |
| 14:29 | mtompset | Great. :) |
| 14:29 | * pianohacker | is confused |
| 14:29 | pianohacker | has something changed? |
| 14:35 | jcamins | pianohacker: http://wiki.koha-community.org[…]/Database_updates |
| 14:37 | magnuse | huh, the packages don't install on debian 7.4? |
| 14:37 | JesseM_ joined #koha | |
| 14:37 | thd | Do arrays return undefined for trailing elements containing empty strings? |
| 14:37 | JesseM_ | cait bug 8760 |
| 14:37 | huginn | 04Bug http://bugs.koha-community.org[…]w_bug.cgi?id=8760 normal, P5 - low, ---, koha-bugs, NEW , Phone number Field tags are reversed in patron record |
| 14:40 | * pianohacker | had to look a bit to see what had changed |
| 14:40 | nengard left #koha | |
| 14:40 | pianohacker | gmcharlt: Are you going to push http://bugs.koha-community.org[…]w_bug.cgi?id=7167 ? |
| 14:40 | huginn | 04Bug 7167: new feature, P1 - high, ---, jonathan.druart, Passed QA , updatedatabase improvements |
| 14:41 | * pianohacker | remembers the bad old days when we tried to assign numbers ourself... |
| 14:43 | gmcharlt | pianohacker: no |
| 14:45 | thd | gmcharlt: Do you have experience of mysteriously appearing undefined values from an array? |
| 14:46 | pianohacker | thd: Is something setting an artificially high index in the array? I believe perl will autofill undef's |
| 14:47 | pastebot | "gmcharlt" at 127.0.0.1 pasted "for thd" (4 lines) at http://paste.koha-community.org/155 |
| 14:48 | mtompset | gmcharlt: Do you think 7167 will ever get pushed? Or do you forsee it becoming unnecessary? |
| 14:49 | bag | ok pianohacker you ready? |
| 14:49 | I am | |
| 14:49 | pianohacker | bag: I'm spongebob |
| 14:49 | thd | gmcharlt: Is your example a negative or modular index? |
| 14:50 | mtompset | pianohacker: Are you wearing square pants? :P |
| 14:50 | thd | pianohacker: How high would an "artificially high" index be? |
| 14:51 | gmcharlt | mtompset: in its current form, it adds too much complexity and risk for breaking upgrades; the current approach, although not without its flaws, has allowed rather stable upgrades for the past four or five years |
| 14:52 | * oleonard | wonders why his VM is running so slowly... |
| 14:53 | thd | gmcharlt: The example you show is for non-sequential array element creation. Why would I have undefined values in a sequentially created array? |
| 14:55 | mtompset | oleonard: Too many processes hogging memory, slowing down other processes? |
| 14:55 | gmcharlt | are you /completely/ sure that all the values you are inserted are defined, and that you are not missing any steps in the sequence? |
| 14:58 | thd | gmcharlt, pianohacker: I have some empty values such as my $element = ''; push (@array, $element); For some empty values later my $elem = $array[$i]; has some empty strings and some mysteriously undefined. |
| 14:58 | huginn | New commit(s) kohagit: Bug 11832: fix JavaScript error on staff client place hold screen if record has no... <http://git.koha-community.org/[…]bbc574699404c7c69> / Bug 11821: disable catalog edit menu items and provide explanations when actions... <http://git.koha-community.org/[…]0a8950e1417695176> / Bug 11833 - fix filtering on saved repo |
| 15:00 | thd | gmcharlt, pianohacker: I am wondering if there is some feature of arrays similar to split which leaves trailing empty strings undefined by default? |
| 15:01 | gmcharlt | #start_meeting Developer meeting, 12 March 2014 (part1) |
| 15:01 | #link http://wiki.koha-community.org[…]ng,_March_12_2014 Agenda | |
| 15:01 | #topic Introductions | |
| 15:01 | wahanui | #info wahanui, a bot that has become sentient |
| 15:01 | gmcharlt | #info gmcharlt = Galen Charlton, 3.16 RM |
| 15:02 | ColinC | #info Colin Campbell, PTFS Europe |
| 15:02 | oleonard | #info Owen Leonard, Athens County Public Libraries |
| 15:02 | marcelr | #info Marcel de Rooy, Netherlands |
| 15:02 | jcamins | #info Jared Camins-Esakov, C & P Bibliography Services |
| 15:02 | thd | #info Thomas Dukleth, Agogme, New York City |
| 15:03 | dpavlin | #info Dobrica Pavlinušić, FFZG Zagreb, Croatia |
| 15:03 | cait | #info Katrin Fischer, BSZ Germany |
| 15:04 | ashimema joined #koha | |
| 15:04 | ashimema | #info Martin Renvoize, PTFS Europe |
| 15:04 | bag | #info Brendan Gallagher ByWater |
| 15:04 | pianohacker | #info Jesse Weaver ByWater |
| 15:04 | cait | dpavlin++ for reminding us about the meeting :) |
| 15:05 | paul_p | #info paul Poulain, BibLibre, France |
| 15:05 | JesseM_ | #info Jesse Maseto, ByWater |
| 15:06 | gmcharlt | great |
| 15:06 | #topic Hackfest update | |
| 15:06 | folks at the hackfest, please use #info to supply updates about all the good stuff that is going on | |
| 15:08 | ok, I'm not at the hackfest, but I'll get this started :) | |
| 15:08 | #info numbers from the hackfest - http://wiki.koha-community.org[…]lle_hackfest_2014 | |
| 15:08 | magnuse | #info Magnus Enger, Oslo Public Library |
| 15:08 | cait | #info Lots of sign-offs has QA fighting to keep up :) |
| 15:08 | paul_p | info = our wifi sometimes get crazy. In this case, mose of us will be disconnected from IRC. |
| 15:09 | rhcl | #info Greg Lawson, Rolling Hills Consolidated Library |
| 15:09 | magnuse | #info bug status numbers for this week are available from http://wiki.koha-community.org[…]lle_hackfest_2014 |
| 15:09 | thd | paul_p: Hard wires are better. |
| 15:09 | paul_p | #info clrh & alex_a will submit soon a patch to collect datas about Koha setups&users |
| 15:09 | jenkins_koha | Starting build #384 for job master_maria (previous build: SUCCESS) |
| 15:09 | paul_p | #info I'll send a mail to koha-devel to request a DNS entry to host the stats server |
| 15:09 | cait | #info there was a nice demo about the new textual cataloguing interface ByWater's pianohacker is working on |
| 15:10 | bag | #info professional catalogers thingy has been submitted - good comments and discussion on using it for more than just Marc21 - bug 11559 |
| 15:10 | huginn | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11559 enhancement, P5 - low, ---, jweaver, Failed QA , Professional cataloger's interface |
| 15:10 | bag | thanks cait |
| 15:10 | cait | #info We discussed SIP implementations in various libraries with various vendors and how it works currently and how it could be improved |
| 15:11 | gmcharlt | hmm... how to pick an icon for a thingy ;) |
| 15:12 | oleonard | http://thenounproject.com/term/monster/24989/ |
| 15:12 | bag | #info CHEESE!!! and froglegs too |
| 15:12 | gmcharlt | oleonard++ |
| 15:12 | cait | #info Joubu gave a good introduction on profiling and debugging - showing us how to turn on Plack and use NYTprof |
| 15:13 | jenkins_koha | Starting build #1660 for job Koha_master (previous build: SUCCESS) |
| 15:13 | Joubu joined #koha | |
| 15:13 | magnuse | #info we also discussed RFID, based on work that is being done at Oslo Public Library, see bug 11858 |
| 15:13 | huginn | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11858 enhancement, P5 - low, ---, koha-bugs, NEW , RFID for circulation |
| 15:13 | pianohacker | oleonard: not http://wac.450f.edgecastcdn.ne[…]er-1303842437.jpg |
| 15:13 | ? | |
| 15:13 | dpavlin | #info plack bug 7844 got signed off (magnuse++) so we might get something into koha soon |
| 15:13 | huginn | 04Bug http://bugs.koha-community.org[…]w_bug.cgi?id=7844 enhancement, P5 - low, ---, dpavlin, Signed Off , plack scripts for developers |
| 15:14 | cait | #info dpavlin also looked at a bunch of LDAP related bugs and patches |
| 15:14 | dpavlin++ | |
| 15:14 | pianohacker | dpavlin++ |
| 15:14 | ashimema | I'm looking at linking up the makefile patch for plack/nginx with bug 11858 later today too.. |
| 15:14 | magnuse | yeah, the sooner 7844 is qa'ed and pushed and used by developers, the better, if you ask me ;-) |
| 15:15 | ashimema++ | |
| 15:15 | bag | also some great discussion on getting a testing database/suite set up for plackify the intranet |
| 15:15 | magnuse | bug 11858 |
| 15:15 | cait | #info There was also work on sample SQL reports to install with Koha |
| 15:15 | magnuse | in short: lots of work! |
| 15:15 | cait | #info And looking into a new default XSLT for UNIMARC |
| 15:15 | ashimema | yeah..bag++ for suggesting and hopefully acting upon getting a big customers live use recorded for a day so we can use it for testing |
| 15:15 | gaetan_B joined #koha | |
| 15:16 | bag | yes that must happen :) it will be my goal |
| 15:16 | ashimema | :) woop, woop |
| 15:16 | bag | and cheese |
| 15:16 | ashimema | mmmm, cheese |
| 15:16 | magnuse | ooh, cheesy |
| 15:16 | pianohacker | all wallace and gromit up in here |
| 15:17 | cait | #info ... there also was a lot of cheese... |
| 15:17 | :) | |
| 15:17 | gmcharlt | I see the cheese-fest is a success! ;) |
| 15:17 | bag | :D |
| 15:17 | ashimema | cheese fuelled qa'ing.. |
| 15:18 | good plan/bad plan? | |
| 15:18 | bag | great plan |
| 15:18 | magnuse | #url https://twitter.com/Jmaseto/st[…]285581312/photo/1 |
| 15:18 | pianohacker | I thought the cheese had failed qa the first time I read that sentence |
| 15:18 | paul_p | it seems it didn't prevented ppl from working after "cheesing"... |
| 15:18 | pianohacker | was about to be quite sad |
| 15:18 | gmcharlt | heh |
| 15:18 | ok, moving on | |
| 15:19 | #topic Performance - Plack | |
| 15:19 | ashimema | plack is good for performance ;) |
| 15:19 | thought i'de break the silence | |
| 15:19 | gmcharlt | #info Plack bug 7844 is in progress |
| 15:20 | magnuse | i added that - our meeting about it seems to have cleared up the issues i tried/failed to forumlate on the agenda |
| 15:20 | huginn | 04Bug http://bugs.koha-community.org[…]w_bug.cgi?id=7844 enhancement, P5 - low, ---, dpavlin, Signed Off , plack scripts for developers |
| 15:20 | gmcharlt | #info http://git.catalyst.co.nz/gw?p[…]ack.git;a=summary contains some stuff for Debian-package management of Plack for the OPAC |
| 15:21 | #info Catalyst also has some patches for dealing with restarting Zebra connections | |
| 15:21 | bag | cool |
| 15:21 | gmcharlt | the upshot, from my POV, is that it seems more likely that we can achieve a full set of Plack configurations to ship with 3.16 |
| 15:21 | pianohacker | Are there plans to gradually make parts of Koha Plack/PSGI-native once we get Plack/CGI working? |
| 15:21 | cait | I tihk we had a link to that in the minutes from the second meeting last time |
| 15:21 | gmcharlt | at least a an experimental basis |
| 15:23 | pianohacker: desires to do so, at any rate | |
| 15:23 | magnuse | a koha-plackify script for the packages would be awesome - and 7844 more or less provides that, i think |
| 15:23 | bag | I am pursuing getting a recording of a butload of transactions across all intranet - so that we can have that with a debug screen available to, rewind, press play etc. (with plack turned on) - so we could quickly catch errors |
| 15:23 | magnuse | bag++ |
| 15:23 | ashimema | there are begginings of a koha-plackify script somewhere.. |
| 15:23 | gmcharlt | ok |
| 15:24 | ashimema | catalyst did it I think.. |
| 15:24 | bag | ashimema: I think libsysguy started that |
| 15:24 | Elliott | |
| 15:24 | ashimema | yeah.. that's what gmcharlt linked to |
| 15:24 | magnuse | yup |
| 15:24 | bag | right |
| 15:24 | magnuse | not sure how the catalyst repo relates to the work from dpavlin? |
| 15:24 | ashimema | yeah.. it's elliots makefile work I want to make dependant upon dpavlins bug |
| 15:25 | magnuse | seems like we could do with some more coordination here ;-) |
| 15:25 | ...and cheese | |
| 15:25 | gmcharlt | magnuse: it means that there's going to have to be a reconciliation |
| 15:25 | ashimema | currently they're rather out fo sync and confusing from a user point of view which one should apply and in what order |
| 15:25 | gmcharlt | since I know that the Catalyst stuff, for instance, has been in use in production (for the OPAC) |
| 15:25 | ashimema | I'm happy to take on trying to get plack collaboration.. |
| 15:25 | magnuse | ashimema++ |
| 15:25 | ashimema | we have one customer using elliots stuff in production. |
| 15:26 | gmcharlt | #info RM announces Plack configs for both OPAC and intranet as a release goal for 3.16 |
| 15:26 | ashimema | but yeah.. generally it needs a good tidy up |
| 15:26 | paul_p | pianohacker about making parts of Koha Plack/PSGI-native = we never spoke of this before AFAIK. it looks a good idea, and a "natural" one |
| 15:26 | gmcharlt | #info Plack support will almost certainly be experimental for production use, but will be encouraged for development environments to shake out remaining bugs |
| 15:27 | cait | maybe gater some stuff on the wiki and have some kind of action plan? |
| 15:27 | ashimema++ | |
| 15:27 | dpavlin | my work is mostly useful for developers since it has a bunch of debugging stuff (turned off by default) in it |
| 15:28 | gmcharlt | #topic Performance and QA |
| 15:28 | #info Question: Should performance be an issue during the QA process? | |
| 15:28 | ashimema | indeed.. dpavlin.. your stuff add's all the debugging goodness to the plackup scripts |
| 15:28 | elliots is more suitable for production | |
| 15:28 | bag | gmcharlt: yes |
| 15:28 | ashimema | and the catalyst stuff is all about koha-plackify as such. |
| 15:28 | that's my current understanding. | |
| 15:28 | gmcharlt | my view of it is that performance does need to be considered, but with a little care |
| 15:29 | bag | "but with a little care" can you expand on that |
| 15:29 | gmcharlt | for example, a patch that adds a minor enhancment but makes displaying a bib record in the OPAC ten times slower likely should fail QA |
| 15:29 | bag | ah you are |
| 15:29 | pianohacker | I agree with that |
| 15:29 | cait | gmcharlt: I think we can't say 'no' here, but i am not sure how to include it in my review |
| 15:30 | ashimema | as I said in our meet the yesterday.. it's hard to test performance during qa.. allot of performance slow down is 'creep' as aposed to big performance hits. |
| 15:30 | gmcharlt | but a patch that adds a new module that itself is a little slow, doesn't necessarily need to be blocked if it doesn't drag overall speed down |
| 15:30 | bag | hmm maybe we could profile before patch and then profile afterwards cait ? |
| 15:30 | gmcharlt | i.e., there, it would be OK to allow performance enhancements to come later |
| 15:30 | ashimema | bag++ before/after profiling would be nice. |
| 15:30 | though means all qa team learning this stuff properly. | |
| 15:30 | gmcharlt | bag: need to be careful there, though -- a mandatory profling step would slow down QA |
| 15:30 | pianohacker | I don't know about profiling beyond 'dang stuff is slow now', QA is already a rather complex process |
| 15:31 | ashimema | maybe add some prof stuff to the qa scripts.. if poss? |
| 15:31 | bag | ah good thought ashimema |
| 15:31 | cait | I have to admit, while i see the advantages, it is a bit of a scary thing to me right now |
| 15:31 | marcelr | i think we should not add performance as a qa task |
| 15:31 | pianohacker | I can see a proper automated profile in the qa scripts as a highly complicated task |
| 15:31 | ColinC | very difficult to generate rules for performance testing so many factors in play |
| 15:32 | gmcharlt | cait: marcelr: how about this |
| 15:32 | paul_p | is everybody aware of the scripts in misc/load_testing/ ? |
| 15:32 | gmcharlt | 1. performance testing is a desirable but not mandatory part of QA, particularly for large patches |
| 15:32 | 2. HOWEVER, performance issues, if observed, /are/ considered a valid reason to fail a patch series | |
| 15:33 | ashimema | sounds fair |
| 15:33 | gmcharlt | there are also some design points that can be considered |
| 15:33 | cait | agreed |
| 15:33 | i tihnk we had that recently with some changes o the letter processing | |
| 15:33 | marcelr | the QAer could ask for feedback on performance from dev or signoffer |
| 15:33 | gmcharlt | for example, if a new page displays an arbitrary number fo recrords that can be large... if it doesn't implement some sort of paging or lazy loading, it is potentially a trouble-spot |
| 15:33 | pianohacker | I agree with gmcharlt |
| 15:34 | magnuse | sounds like a good start to me |
| 15:34 | ashimema | ++ |
| 15:34 | gmcharlt | give me a moment to draft something |
| 15:34 | cait | :) |
| 15:34 | thd | Some performance tradeoffs for useful but performance draining features could be regulated by a system preference. |
| 15:34 | bag | paul_p: yes but aren't there only a few tests in there? |
| 15:34 | cait | it often is the case i think |
| 15:34 | pianohacker | thd: I believe there is precedent for that |
| 15:34 | cait | but then checking that code is not executed when the feature is turned off |
| 15:34 | could be a good guideline as well | |
| 15:35 | paul_p | bag = they test "common" cases, like searching a patron, checkout/checkin, ... Only a few cases, you're right |
| 15:35 | but it can be used as a frameork to add more | |
| 15:35 | bag | but it's an addtional thing that I forgot about that I can add to my QA steps |
| 15:35 | paul_p | and they can detect performance caveats in the most common places. |
| 15:36 | janPasi joined #koha | |
| 15:36 | paul_p | having a performance problem in admin/marc_subfield_structure.pl is much less critical than in circulation/returns.pl ! |
| 15:36 | pastebot | "gmcharlt" at 127.0.0.1 pasted "Proposed QA policy regarding performance" (3 lines) at http://paste.koha-community.org/156 |
| 15:37 | gmcharlt | ok, comments on that wording? |
| 15:37 | bag | @wunder marseille france |
| 15:37 | huginn | bag: The current temperature in Marseille, France is 17.0°C (4:30 PM CET on March 12, 2014). Conditions: Clear. Humidity: 42%. Dew Point: 4.0°C. Pressure: 30.33 in 1027 hPa (Steady). |
| 15:37 | gmcharlt | Holy wrapping, Batman! |
| 15:38 | cait | :) |
| 15:38 | pastebot | "gmcharlt" at 127.0.0.1 pasted "Proposed QA policy regarding performance" (9 lines) at http://paste.koha-community.org/157 |
| 15:38 | gmcharlt | let's try this ^^ |
| 15:38 | cait | reads fine to me, just some grammar/typo issue in the last bits |
| 15:38 | ashimema | sounds fiar to me gmcharlt |
| 15:38 | marcelr | requested to watch out? or something less mandatory? |
| 15:38 | pianohacker | +1 |
| 15:38 | cait | +1 |
| 15:38 | paul_p | +1 for paste 157 |
| 15:38 | magnuse | +1 |
| 15:38 | ColinC | +1 |
| 15:39 | bag | +1 |
| 15:39 | cait | where to put it on the wiki? coding guidelines? I think I never got around to start a qa page *hides* |
| 15:39 | pastebot | "gmcharlt" at 127.0.0.1 pasted "Proposed QA policy regarding performance" (10 lines) at http://paste.koha-community.org/158 |
| 15:40 | gmcharlt | marcelr: does ^^ read more clearly to you? |
| 15:40 | marcelr | improvement :) |
| 15:40 | we need common sense here | |
| 15:40 | ColinC | yes coding guidelines .... makes people aware who are coding it |
| 15:40 | gmcharlt | cait: my proposing I think is best for a QA page |
| 15:40 | * gmcharlt | is drafting a statement for the coding guidlines |
| 15:40 | gmcharlt | one sec |
| 15:40 | ashimema | I'll be happy to add qa guidelines wiki page (in terms of editing wiki).. guided by cait for the content. |
| 15:40 | * cait | tries to volunteer ashimema to create one ;) |
| 15:41 | ashimema | sometime at hackfest.. whilst we're sitting next ot each other ;) |
| 15:41 | * cait | might need a better hiding spot tomorrow ;) |
| 15:42 | * ashimema | see a copy paste from gmcharlt's statement coming soon ;) |
| 15:42 | mtompset | #info Mark Tompsett |
| 15:42 | +1 for paste 157 | |
| 15:42 | pianohacker | hey there slowpoke |
| 15:43 | gmcharlt | could I have a round for +1/-1 for paste 158? |
| 15:43 | marcelr | +1 for 158 |
| 15:43 | gmcharlt | (both to set the final version, and to give me time to finish by coding guidelines draft ;) |
| 15:44 | ashimema | +1 for 158 |
| 15:44 | :) | |
| 15:44 | * cait | is reading |
| 15:44 | cait | +1 |
| 15:44 | magnuse | +1 for 158 |
| 15:44 | mtompset | +1 for 158 # the QA-rs who don't have profiling all set up will be relieved. :) |
| 15:45 | paul_p | +1 for 158 |
| 15:45 | pianohacker | +1 for 158 |
| 15:46 | paul_p | what about setup jenkins to run benchmark tests before an after each build, and if a difference of, for example, more than +5% is detected, hilight that ? |
| 15:46 | pianohacker | paul_p: is that difficult to do? |
| 15:46 | paul_p | pianohacker I strictly don't know... |
| 15:46 | s/strictly/really/ | |
| 15:46 | marcelr | :) |
| 15:47 | * pianohacker | agrees in principle though |
| 15:47 | Joubu | It depends on the load of the server. I think sometime several runs are done at the same time. |
| 15:47 | cait | maybe we cold log this as an idea? |
| 15:47 | ashimema | paul_p++ I like that idea.. and would suggest it mysefl but don't fully understand the implications of setting that up |
| 15:47 | pianohacker | Joubu: good point |
| 15:47 | mtompset | if the runs are serialized, it would be possible. |
| 15:47 | gmcharlt | paul_p: I think some automated performance benchmarking is a good idea, though some care will have to be taken |
| 15:48 | e.g., running it on test servers that aren't running other things | |
| 15:48 | pastebot | "gmcharlt" at 127.0.0.1 pasted "Coding guidelines regarding performance" (22 lines) at http://paste.koha-community.org/160 |
| 15:48 | ashimema | maybe we needs a performance profiling server.., or to set resrouces aside for performance profileing only. |
| 15:48 | magnuse | https://wiki.jenkins-ci.org/di[…]erformance+Plugin ? |
| 15:48 | paul_p | gmcharlt agreed, of course. We must be sure that conditions are the same (same DB, same resources available,...) |
| 15:49 | gmcharlt | #agreed http://paste.koha-community.org/158 is provisionally approved as a QA guideline for performance t |
| 15:49 | bag | sorry kind of odd having a meeting sitting in a room with 3/4 of the people in the meeting and it's so quiet - I could always imagine magnus rasing his hand to speak or something - sorry to say it doesn't happen |
| 15:49 | paul_p | lol. magnuse rised his (right) hand ;-) |
| 15:49 | gmcharlt | thoughts on the proposed coding guidelines wording? http://paste.koha-community.org/160 |
| 15:49 | mtompset | ashimema++ # like the idea of the profiling server. |
| 15:49 | bag | magnuse++ |
| 15:50 | cait | bit worried that the meetingbot is not working... can we check? |
| 15:50 | paul_p | oups, it was the left one :D |
| 15:50 | pianohacker | oh god everything is messed up now |
| 15:50 | marcelr | 160 reads good |
| 15:50 | pianohacker | we have to start the meeting from scratch |
| 15:50 | #info Jesse Weaver ByWater | |
| 15:50 | * gmcharlt | will generate the meeting log and minutes manually |
| 15:50 | pianohacker | phew :) |
| 15:50 | bag | don't forget the cheese |
| 15:51 | mtompset | gmcharlt++ # for manual awesomeness. ;) |
| 15:51 | ashimema | it's all about the cheese gromit. |
| 15:51 | magnuse | mmm, cheese... |
| 15:51 | ashimema | gmcharlt++ |
| 15:51 | paul_p | gmcharlt = I promise there was no alcohol with the cheese. If things becomes mad now, it's not my fault ;-) |
| 15:51 | * pianohacker | didn't intend to throw that large of a wrench into the works |
| 15:51 | pianohacker | just a small one that makes a bit of a rattle |
| 15:51 | paul_p | ( +1 for 160 ) |
| 15:51 | magnuse | +1 for 160 |
| 15:51 | gmcharlt | paul_p: alcholic cheese that could affect me from 5,000 miles away would be... formidable |
| 15:51 | ColinC | +1 for 160 |
| 15:52 | paul_p | or very very strong ;-) |
| 15:52 | bag | +1 |
| 15:52 | * cait | is reading :) |
| 15:52 | oleonard | gmcharlt: I like the double English/French meaning of that word, formidable :) |
| 15:53 | cait | +1 |
| 15:53 | gmcharlt | #agreed http://paste.koha-community.org/160 is provisionally accepted as an amendment to the coding guidelines regarding performance |
| 15:54 | #topic Action items from previous meeting | |
| 15:54 | ashimema | I'm going to run the idea of donating a small server for performance profiling past 'the big boss man' here.. |
| 15:55 | gmcharlt | #info gmcharlt was laid low by his sinuses over the weekend, still workingo n DBIC examples and clearing the PQA queue |
| 15:55 | ashimema++ | |
| 15:55 | pianohacker | If this is how slow gmcharlt is when he's sick... |
| 15:55 | gmcharlt | good work at the hackfest, folks! |
| 15:55 | mtompset | +1 for 160. |
| 15:55 | cait | gmcharlt++ # glad you are better now |
| 15:55 | marcelr | gmcharlt++ |
| 15:55 | * pianohacker | readies weaponized kindergarteners |
| 15:56 | gmcharlt | are there any other last minute announcements? |
| 15:56 | magnuse | sinuses-- |
| 15:56 | * pianohacker | wants to grab the ear of some developers to talk cronjobs, but that can happen post-meeting |
| 15:56 | gmcharlt | ashimema: dpavlin: whoever: could I ask that one of you write up a summary of the Plack work done at hackfest to send to koha-devel? |
| 15:56 | mtompset | gmcharlt++ # rather productive still! |
| 15:56 | cait | pianohacker: you could write up a summary for the second dev meeting tonihgt... :) |
| 15:57 | pianohacker | cait: as penance? |
| 15:57 | dpavlin | gmcharlt: sure, but we didn't really did much work (yet). patch in question is two years old |
| 15:58 | gmcharlt | dpavlin: that's OK, I mostly just want the email thread to exist as a coordination point |
| 15:58 | dpavlin | gmcharlt: ack |
| 15:58 | cait | dpavlin++ |
| 15:58 | janPasi joined #koha | |
| 15:58 | ashimema | happy to try and co-ordinate that gmcharlt |
| 15:58 | gmcharlt | great |
| 15:58 | ashimema | I'll grab magnus, dpavlin later (no running away boys) |
| 15:58 | magnuse | i think what would be good wuld be for ashimema to thnk about how the different pieces of the puzzle can fit together/be reconciled |
| 15:59 | * magnuse | runs away |
| 15:59 | * pianohacker | just notices typo in above harassment of gmcharlt |
| 15:59 | ashimema | exactly ;) |
| 15:59 | * pianohacker | is bad at typing for a keyboard geek |
| 15:59 | gmcharlt | I typ reel gud too |
| 15:59 | bag | pianohacker: was that typo kindergartners? |
| 15:59 | gmcharlt | OK, thanks, everybody |
| 15:59 | nengard joined #koha | |
| 16:00 | gmcharlt | extra cheese to those of you who stay up to the second part! |
| 16:00 | #endmeeting | |
| 16:00 | cait | :) |
| 16:00 | we might still be out partying ... who knows? | |
| 16:00 | bag | I may be there - I can't sleep on this much cheese :) |
| 16:00 | magnuse | three cheese for gmcharlt! |
| 16:00 | fridolin left #koha | |
| 16:00 | gmcharlt | lol |
| 16:01 | cait | more cheese! |
| 16:03 | mtompset | Please sir, can I have some more? ;) |
| 16:04 | pianohacker | gmcharlt: your opinion in particular would be helpful on cronjobs |
| 16:04 | gmcharlt | pianohacker: what are the questions? |
| 16:05 | pianohacker | I'm trying to figure out where to store the metadata/info on cronjobs and their options |
| 16:05 | jenkins_koha | Project master_maria build #384: SUCCESS in 56 min: http://jenkins.koha-community.[…]master_maria/384/ |
| 16:05 | * Galen Charlton: Bug 10012: remove last traces of NoZebra | |
| 16:05 | * Galen Charlton: Bug 10012: DBRev 3.15.00.024 | |
| 16:05 | * Jonathan Druart: Bug 11550: Add UT for C4::Acquisition::GetOrder | |
| 16:05 | huginn | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=10012 minor, P5 - low, ---, gmc, Pushed to Master , Remove all traces of NoZebraIndexes |
| 16:05 | jenkins_koha | * Fridolin Somers: Bug 11648 - Use word item instead of copy - intranet |
| 16:05 | * Fridolin Somers: Bug 11648 - Use word item instead of copy - opac bootstrap | |
| 16:05 | * Fridolin Somers: Bug 11648 - Use word item instead of copy - opac prog | |
| 16:05 | * Owen Leonard: Bug 11686: make "please note your reason here..." translatable | |
| 16:05 | * Galen Charlton: Bug 11820: remove dead code in basket group management | |
| 16:05 | huginn | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11550 minor, P5 - low, ---, jonathan.druart, Pushed to Master , Add a couple of unit tests for C4::Acq::GetOrder |
| 16:05 | jenkins_koha | * Isaac Brodsky: Bug 1047: Punctuation in site names no longer breaks Edit/Delete z39.50 server administration |
| 16:05 | * Owen Leonard: Bug 11833 - fix filtering on saved report subgroup | |
| 16:05 | huginn | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11648 minor, P5 - low, ---, oleonard, Pushed to Master , Use word item instead of copy |
| 16:05 | jenkins_koha | * Owen Leonard: Bug 11821: disable catalog edit menu items and provide explanations when actions cannot be done |
| 16:05 | * Owen Leonard: Bug 11832: fix JavaScript error on staff client place hold screen if record has no items | |
| 16:05 | huginn | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11686 normal, P5 - low, ---, oleonard, Pushed to Master , "please note your reason here..." does not get translated |
| 16:05 | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11820 minor, P5 - low, ---, gmc, Pushed to Master , dead code in basketgroup.pl : "add" operation is never invoked without a booksellerid | |
| 16:05 | 04Bug http://bugs.koha-community.org[…]w_bug.cgi?id=1047 normal, P2, ---, isaacbrodsky, Pushed to Master , Punctuation in site names breaks Edit/Delete z39.50 server administration | |
| 16:05 | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11833 normal, P5 - low, ---, oleonard, Pushed to Master , Filtering on saved report subgroup doesn't work | |
| 16:05 | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11821 normal, P5 - low, ---, oleonard, Pushed to Master , Staff client catalog: delete menu items greyed out without explanation | |
| 16:05 | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11832 minor, P5 - low, ---, oleonard, Pushed to Master , JavaScript error on staff client place hold screen if record has no items | |
| 16:05 | pianohacker | doing it entirely in JS is easy to implement and can easily be translated |
| 16:05 | bag | pianohacker: how about the user could choose to store them in the database or in a text file? |
| 16:05 | gmcharlt | pianohacker: what sorts of metadata, specifically? |
| 16:05 | bag | sorry not text file :P |
| 16:05 | but file | |
| 16:06 | pianohacker | gmcharlt: Available options, meanings of those options, what values they can be set to, etc. |
| 16:06 | bag | now that was a typo pianohacker |
| 16:06 | pianohacker | plus information on the cronjob itself |
| 16:06 | gmcharlt | well, there is always the boring man name_of_cronjob |
| 16:06 | and name_of_cronjob --help | |
| 16:07 | I think I'm missing some context | |
| 16:07 | pianohacker | I'd also like to refactor the cronjobs so they can optionally output nice, machine-readable information about progress/errors |
| 16:07 | gmcharlt: I think so, apologies | |
| 16:07 | gmcharlt: I thought I had talked to you about this, but my memory has tripped me up :) I'm working on unifying the cronjobs | |
| 16:08 | gmcharlt | yeah, we had had a prior discussion |
| 16:08 | magnuse | one cronjob to rule them all!? |
| 16:08 | * thd | has to sleep more. |
| 16:08 | pianohacker | i.e., with the exceptions of process_message_queue and rebuild_zebra, only one script has to be added to the crontab |
| 16:08 | good night thd | |
| 16:08 | bag | typo magnuse it's ring ;) |
| 16:08 | pianohacker | and the cronjobs can be configured from the web interface |
| 16:08 | bag | typo bag it's cheese :P |
| 16:09 | pianohacker | the second part is the tricky bit |
| 16:09 | because I don't want to just say "and put the command line options here" | |
| 16:09 | I'd like to expose a UI that, for instance, hooks up the LOST authval for the -c option of longoverdue | |
| 16:09 | limits relevant parameters to numerical values, etc. | |
| 16:10 | and for that I need to either refactor the cronjobs to take their config differently or store metadata on their options | |
| 16:10 | * pianohacker | finishes braindumping and goes to get coffee |
| 16:11 | gmcharlt | pianohacker: I'll have a paste for you in a few minutes |
| 16:11 | magnuse | one cheese to rule them all? nah, i prefer there to be lots and lots and lots of different ones! |
| 16:11 | pianohacker | gmcharlt: thanks |
| 16:11 | mtompset | Oooo... cool, pianohacker. Will read log file after getting back from my chauffeur gig. |
| 16:15 | Joubu joined #koha | |
| 16:17 | magnuse | i can haz fresh numbers? sure! http://wiki.koha-community.org[…]dnesday_afternoon |
| 16:18 | pianohacker | magnuse++ |
| 16:19 | brb | |
| 16:19 | pianohacker joined #koha | |
| 16:22 | pastebot | "gmcharlt" at 127.0.0.1 pasted "for pianohacker -- initial thoughts on a database schema for a unified job runner" (46 lines) at http://paste.koha-community.org/161 |
| 16:22 | pianohacker | gmcharlt: translation? |
| 16:22 | wahanui | it has been said that translation is translate.koha-community.org |
| 16:23 | gmcharlt | pianohacker: what are you asking? |
| 16:23 | pianohacker | I like the flexibility of it, but descriptions of cronjobs/options/option values have to be stored somewhere |
| 16:24 | gmcharlt | pianohacker: perhaps a separate i18n table that contains a key, a locale, and a translation |
| 16:24 | there are other efforts underway to get somethign like that started | |
| 16:25 | pianohacker | okay... I like that particular idea but I thought we were trying to get translatable strings out of the DB |
| 16:25 | Joubu left #koha | |
| 16:25 | Joubu joined #koha | |
| 16:25 | gmcharlt | pianohacker: well, sticking them in the DB is fine... as long as one provides a translation mechanism |
| 16:26 | i.e., a i18n table implies the following | |
| 16:26 | - adjustments to the translation scripts to generate a PO file from it | |
| 16:26 | - install and upgrade hooks to stick translation into it during updates | |
| 16:27 | cait | hm if this is for existing cronjobs, maybe we could do it a bit simpler and move the descriptions into the template? |
| 16:27 | pianohacker | again, I like the idea but that's a crapload of work |
| 16:27 | gmcharlt | what's problematic, of course, is just sticking plain English into the database |
| 16:27 | cait | or is the user supposed to add to it? |
| 16:27 | gmcharlt | pianohacker: indeed, it's a lot of work |
| 16:27 | as a less worky alternative, maybe adapting what you did with the sysprefs editor would suffice | |
| 16:27 | gaetan_B joined #koha | |
| 16:28 | cait | aah pref file |
| 16:28 | pianohacker | gmcharlt: was the main reason for sticking the cronjobs in the DB to allow user-defined scheduled tasks? |
| 16:28 | cait | yaml? |
| 16:28 | gmcharlt | pianohacker: the main reason for that proposal is keeping as much configuration as possible into the DB |
| 16:29 | thereby making it accessible to normal users | |
| 16:30 | * oleonard | pictures barton_ingesting in a vat of nutrients, absorbing food through his skin. |
| 16:30 | gmcharlt | if there's no configuraiton interface there, then frankly I think we'd be better off sticking with the current approach and just tidying up the perldoc and perhaps imposing more consistentcy |
| 16:32 | pianohacker | gmcharlt: how about a mixed approach |
| 16:32 | storing the actual configuration/structure of the tasks in the DB | |
| 16:33 | but having data related to display/description of them in yaml or even just a JS struct | |
| 16:33 | so user-defined cronjobs can be added, and descriptions stored in the DB | |
| 16:33 | but slightly-nicer, translatable descriptions are stored elsewhere | |
| 16:35 | (the latter being for stock cronjobs like fines/longoverdue/advance_notice/etc.) | |
| 16:36 | gmcharlt | what does a user-defined cronjob mean, in this case |
| 16:36 | a new task? or a new job? | |
| 16:36 | pianohacker | gmcharlt: A new task, to use the schema's terminology |
| 16:37 | gmcharlt: If users can't define new tasks, why have descriptions stored in the DB? | |
| 16:37 | * magnuse | needs to take a break |
| 16:38 | magnuse | so long and thanks for all the cheese! |
| 16:38 | gmcharlt | pianohacker: as a convience for the script, but yes, if we don't let them create new tasks (and I think we shouldn't), then yeah, task descriptions and parameter descirpitons could be moved outside |
| 16:38 | a la the sysprefs editor | |
| 16:40 | pianohacker: a revised paste coming soon | |
| 16:43 | pianohacker | gmcharlt: thanks. I hope I'm not coming off too rude, just trying to nail down something really good so it's a true improvement for devs and users, and do so before I start coding |
| 16:44 | We're not talking about MARC or the livecd, so we're not at each others' throats at least | |
| 16:44 | druthb | “Through three cheese trees three free fleas flew. While these fleas flew, freezy breeze blew. Freezy breeze made these three trees freeze. Freezy trees made these trees' cheese freeze. That's what made these three free fleas sneeze.” |
| 16:44 | * pianohacker | blinks |
| 16:44 | pianohacker | hi druthb |
| 16:44 | gmcharlt | pianohacker: I don't see an argument going on here |
| 16:44 | pastebot | "gmcharlt" at 127.0.0.1 pasted "for pianohacker -- rev 2" (55 lines) at http://paste.koha-community.org/162 |
| 16:44 | pianohacker | gmcharlt: just a very straightforward discussion? :) |
| 16:44 | druthb | hi, pianohacker. :) |
| 16:45 | gmcharlt | pianohacker: hmm |
| 16:45 | I think it might be better if job_task_options => job_schedule_task_options | |
| 16:46 | pianohacker | gmcharlt: I agree with that, and your comment on reinventing cron |
| 16:46 | wajasu | pianohacker: wouldn't a DB driven set of cronjobs be something to be done under the scheduler rewrite? |
| 16:46 | pianohacker | in fact, my plan was to limit this to only-nightly jobs, at least at first |
| 16:46 | gmcharlt | right |
| 16:47 | pianohacker | wajasu: scheduler rewrite? |
| 16:47 | gmcharlt | so cron.daily would be something like run-koha-jobs --periodcity daily |
| 16:47 | cron.hourly run-koha-jobs --periodicity houlry | |
| 16:47 | etc. | |
| 16:47 | wajasu | bug 1993 :) |
| 16:47 | huginn | 04Bug http://bugs.koha-community.org[…]w_bug.cgi?id=1993 major, P3, ---, sophie.meynieux, In Discussion , Task Scheduler Needs Re-write |
| 16:47 | pianohacker | gmcharlt: I like that quite a lot. One question, what is the sequence column in task_parameters for? |
| 16:48 | gmcharlt | hmm... |
| 16:48 | not really necessary | |
| 16:48 | pianohacker | ah, kk. |
| 16:48 | wajasu: I'd say that covers related but separate functionality | |
| 16:49 | wajasu | ok |
| 16:49 | pianohacker | wajasu: I'm glad that's finally getting resurrected, actually, because it takes away one thing I thought I'd have to implement! |
| 16:49 | not often a bug in the 1000s gets saved | |
| 16:52 | gmcharlt | wajasu: pianohacker: in my paste, the job table is pretty much implicitly a /recurring/ job |
| 16:52 | pianohacker | yup |
| 16:52 | gmcharlt | but that's not to say that one of the tasks could be "run all scheuled reports" |
| 16:52 | *couldn't be | |
| 16:52 | or to get out of the double-negative trap | |
| 16:52 | one of the tasks could be "run scheduled reports" | |
| 16:53 | pianohacker | makes sense :) Would still warrant a separate config screen and structure |
| 16:53 | (for specifying which reports to run and when) | |
| 16:54 | gmcharlt: Thanks very much for your input, I like the schema you've come up with | |
| 16:55 | gmcharlt | you're welcome |
| 16:56 | * mtompset | perks up. "Did someone say MARC? 942$n! ;)" |
| 16:56 | * pianohacker | hits mtompset with the MARC stick |
| 16:56 | * wajasu | supresses |
| 16:57 | pianohacker | We spend a lot of time at ByWater dealing with frustration and confusion from our customers related to being middlemen between them and the fines/longoverdue/notices cronjobs |
| 16:57 | mtompset | pianohacker: Actually, you probably had to Mark Breaker the MRC stick to transform it into a MRK stick. ;) |
| 16:58 | pianohacker | I think a structured ui for nightly cronjobs could help reduce that greatly |
| 16:58 | Okay, mtompset has filled a row on the Geeky Joke bingo card, we can all go home now | |
| 16:59 | mtompset | What? No props for wajasu getting the OPAC Suppression reference? ;) |
| 17:00 | pianohacker | oh god I missed that |
| 17:00 | * mtompset | cackles evilly, "You will be traumatized by hiding records too!" |
| 17:00 | pianohacker | friggin' opacsuppression and all of its bugs and confusing behavior |
| 17:02 | gmcharlt | /etc/init.d/apache stop # hey look, a perfectly suppressed OPAC! ;) |
| 17:02 | wajasu | i'm writing a patch for some of that now. collaborating with mtompset. of course he showed me lots of other bugs last night i read over. |
| 17:02 | * mtompset | laughs. |
| 17:02 | mtompset | YES! I knew there was a way to do it, gmcharlt. |
| 17:03 | gmcharlt | of course, Plack+Nginx would defeat that dastardly scheme |
| 17:03 | pianohacker | gmcharlt: 502 bad gateway :) |
| 17:04 | bag | hey gmcharlt if we are switching from grs1 to dev (and yes this is on a dev machine) is it fine to just change koha-conf - or best to just redo the make install |
| 17:04 | grs1 to dom* | |
| 17:04 | mtompset | Wasn't someone working on a koha-conf.xml comparing tool? |
| 17:04 | gmcharlt | bag: a make update_zebra_conf after rerunnign Makefile.PL should do it |
| 17:04 | pianohacker | gmcharlt: I'm thinking of making a Koha::Job class eventually that will allow for reporting progress/error information in a structured way, but that'll be later :) |
| 17:04 | oh god, yes | |
| 17:04 | update_zebra_conf+++++++++++ | |
| 17:04 | so so nice | |
| 17:05 | gmcharlt | pianohacker: speak up, I can't quite hear you ;) |
| 17:05 | mtompset | where is update_zebra_conf? |
| 17:05 | * pianohacker | ? |
| 17:05 | pianohacker | exciteable and verbose? |
| 17:05 | I think you have the wrong person | |
| 17:06 | bag | bag? |
| 17:06 | wahanui | I LIKE BASEBALL |
| 17:06 | mtompset | wajasu: Oh, the hiding stuff also reminds me of a conversation I had with papa. |
| 17:07 | His idea was to supercede OpacHiddenItems and hence my sysprefs by moving the hiding capability into circulation rules. | |
| 17:09 | wajasu | can someone fine savy look at the bug 11573 patch, it basically makes us derive a consistent description for "account" and "pay fines" tabs from the other fields. |
| 17:09 | huginn | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11573 new feature, P5 - low, ---, koha-bugs, Needs Signoff , change description of charges in account fines |
| 17:09 | gmcharlt | mtompset: circ rules really sounds like the wrong place to put catalog visiblity settings |
| 17:11 | druthb | druthb? |
| 17:11 | wahanui | Well, she finally snapped, like we all knew she would. |
| 17:11 | * druthb | snaps |
| 17:12 | druthb | I used to think I'd seen some peculiar things in the Koha codebase. Now I've seen it all. |
| 17:12 | mtompset | gmcharlt: Well, perhaps the screen, not necessarily the table. |
| 17:13 | jenkins_koha | Project Koha_master build #1660: SUCCESS in 2 hr 2 min: http://jenkins.koha-community.[…]Koha_master/1660/ |
| 17:13 | * Galen Charlton: Bug 10012: remove last traces of NoZebra | |
| 17:13 | * Galen Charlton: Bug 10012: DBRev 3.15.00.024 | |
| 17:13 | * Jonathan Druart: Bug 11550: Add UT for C4::Acquisition::GetOrder | |
| 17:13 | * Fridolin Somers: Bug 11648 - Use word item instead of copy - intranet | |
| 17:13 | huginn | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=10012 minor, P5 - low, ---, gmc, Pushed to Master , Remove all traces of NoZebraIndexes |
| 17:13 | jenkins_koha | * Fridolin Somers: Bug 11648 - Use word item instead of copy - opac bootstrap |
| 17:13 | * Fridolin Somers: Bug 11648 - Use word item instead of copy - opac prog | |
| 17:13 | * Owen Leonard: Bug 11686: make "please note your reason here..." translatable | |
| 17:13 | * Galen Charlton: Bug 11820: remove dead code in basket group management | |
| 17:13 | huginn | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11550 minor, P5 - low, ---, jonathan.druart, Pushed to Master , Add a couple of unit tests for C4::Acq::GetOrder |
| 17:13 | jenkins_koha | * Isaac Brodsky: Bug 1047: Punctuation in site names no longer breaks Edit/Delete z39.50 server administration |
| 17:13 | * Owen Leonard: Bug 11833 - fix filtering on saved report subgroup | |
| 17:13 | huginn | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11648 minor, P5 - low, ---, oleonard, Pushed to Master , Use word item instead of copy |
| 17:13 | jenkins_koha | * Owen Leonard: Bug 11821: disable catalog edit menu items and provide explanations when actions cannot be done |
| 17:13 | * Owen Leonard: Bug 11832: fix JavaScript error on staff client place hold screen if record has no items | |
| 17:13 | huginn | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11686 normal, P5 - low, ---, oleonard, Pushed to Master , "please note your reason here..." does not get translated |
| 17:13 | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11820 minor, P5 - low, ---, gmc, Pushed to Master , dead code in basketgroup.pl : "add" operation is never invoked without a booksellerid | |
| 17:13 | 04Bug http://bugs.koha-community.org[…]w_bug.cgi?id=1047 normal, P2, ---, isaacbrodsky, Pushed to Master , Punctuation in site names breaks Edit/Delete z39.50 server administration | |
| 17:13 | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11833 normal, P5 - low, ---, oleonard, Pushed to Master , Filtering on saved report subgroup doesn't work | |
| 17:13 | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11821 normal, P5 - low, ---, oleonard, Pushed to Master , Staff client catalog: delete menu items greyed out without explanation | |
| 17:13 | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11832 minor, P5 - low, ---, oleonard, Pushed to Master , JavaScript error on staff client place hold screen if record has no items | |
| 17:15 | mtompset | nengard: Did you get a chance to look at the bug I asked you about yesterday? |
| 17:16 | nengard | yes, remember i couldn't apply the patch |
| 17:16 | didn't hear back after that to try again | |
| 17:16 | mtompset | Oh my... I'm going senile! |
| 17:16 | nengard | hehe |
| 17:16 | no worries we all have a lot to remember | |
| 17:17 | at first i was going to say 'no i forgot ...' | |
| 17:17 | but then I remembered | |
| 17:18 | mtompset | Right, Joubu and mathieu saby were going to figure out where and why and how. :) |
| 17:21 | tcohen | hi mtompset nengard |
| 17:22 | nengard | hiya |
| 17:23 | talljoy | back |
| 17:25 | pianohacker | druthb: you talking about the bug 11573 patch? |
| 17:25 | huginn | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11573 new feature, P5 - low, ---, koha-bugs, Needs Signoff , change description of charges in account fines |
| 17:27 | paul_p joined #koha | |
| 17:28 | pianohacker | ashimema: https://github.com/joonty/vdebug |
| 17:28 | ashimema: http://docs.activestate.com/ko[…].4/debugperl.html | |
| 17:29 | druthb | no, pianohacker, something I found in the cPanel language-translation files. |
| 17:30 | pianohacker | druthb: dare I ask for a pastebin? |
| 17:30 | druthb | # grep 'cow=' /usr/local/cpanel/base/frontend/x3/lang/english-utf8 |
| 17:30 | cow=I had ~[_1#number of cows~] cows in the morning and ~[_2#number of night cows~] at night. | |
| 17:30 | pianohacker | oy vey that translation syntax |
| 17:30 | written by a perler I'm guessing? | |
| 17:31 | druthb | probably. We have some pretty squirrely translation tools, even weirder than Koha's |
| 17:31 | pianohacker | and you have an up on webmin, they have no bovine administration procedures ;) |
| 17:31 | druthb | (they work; we're 100% on 19 languages or more…but they're strange. |
| 17:31 | paul_p | hello nengard ! |
| 17:32 | nengard | :) |
| 17:32 | there you are | |
| 17:32 | pianohacker | translation is probably the ultimate example of "if it's stupid but it works it ain't stupid" |
| 17:33 | as martin and my suggestion to make everyone just speak english and have names within ASCII doesn't seem to be going through | |
| 17:37 | jcamins | Microfilm-- |
| 17:37 | while (1) { microfilm--; } | |
| 17:38 | pianohacker | line 2: incompatible implicit declaration of builtin variable `Microfilm' |
| 17:39 | but in all seriousness... microfilm? | |
| 17:39 | tcohen | has anyone experienced fatal error on searching if the order is other than relevance? |
| 17:39 | paul_p | thanks nengard ! |
| 17:40 | nengard | No prob! Some of it is old, but hopefully it helps with whatever you're trying to do |
| 17:44 | jcamins | pianohacker: it's awful. |
| 17:44 | pianohacker | why do you have to deal with it? |
| 17:45 | jcamins | pianohacker: the book I want is available at four libraries in the US. |
| 17:45 | Only one on the East Coast. | |
| 17:45 | Thankfully it's one in New York. | |
| 17:46 | Unfortunately, it's the microfilm here. | |
| 17:46 | pianohacker | ouch... |
| 17:46 | sorry to hear that | |
| 17:46 | * jcamins | too. |
| 17:46 | jcamins | And I get to manually advance the roll, too. |
| 17:47 | The reel is too short. | |
| 17:47 | No problem! It's only a 180-page book! | |
| 17:48 | pianohacker | gmcharlt: do you have a particular preference between a JSON blob and separate rows? The former is a bit quirky but seems simpler to implement/update |
| 17:48 | jcamins: what in the seven hells are you reading? | |
| 17:48 | jcamins | And it's out of paper. I hate you, microfilm. |
| 17:48 | A 1962 bibliography of books about film. | |
| 17:49 | gmcharlt | pianohacker: I mildly prefer a separate table, but I suggest that you make an inventory of the task parameters in use first before committing |
| 17:50 | pianohacker | fair enough |
| 17:56 | reiveune | bye |
| 17:56 | reiveune left #koha | |
| 17:57 | gmcharlt | bag: khall_away: I'd the name of a specific staff person for the commit for bug 11531, please |
| 17:57 | huginn | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11531 normal, P5 - low, ---, kyle, Passed QA , Cannot add new libraries to the transport cost if previously used. |
| 17:57 | gmcharlt | though I assume the patch was written by khall |
| 17:59 | cait | gmcharlt: probably safe to assume in tha tcase |
| 18:00 | bag | ah gmcharlt kyle has run to the hospital with his wife |
| 18:00 | cait | !! |
| 18:00 | bag | so it maybe baby time soon |
| 18:00 | gmcharlt | great! |
| 18:00 | cait | hackfest baby! |
| 18:00 | tcohen | !! |
| 18:00 | bag | could be - best of luck kyle |
| 18:03 | huginn | New commit(s) kohagit: Bug 11531: make it possible to edit transport cost for a new libraries <http://git.koha-community.org/[…]3a04577bfd7e462bc> / Bug 11861: Add ISSN column in serials claims table <http://git.koha-community.org/[…]a521281854b0ed055> / Bug 11718 - Use new DataTables include in reports templates <http://git.koha-community. |
| 18:07 | pianohacker | gmcharlt: in discussion with other developers, I've realized that having the definitions of each job and its options in the database raises some tricky problems with updates |
| 18:07 | if it's a JSON blob, you have to respecify the entire blob each time you update it, and if it's a broken-out table, updates get a little tricky | |
| 18:07 | gmcharlt | pianohacker: that's one of the reasons for not permitting users to create tasks directly |
| 18:08 | cait | oleonard: should i be able to access course reserves without having permissions to it? |
| 18:08 | gmcharlt | as far as broken-out tables, arranging FKs for cascading updates can help |
| 18:08 | cait | my user only has the staff access permission and nothing else right now |
| 18:08 | gmcharlt | but I could certainly see a case where a new mandatory parameter gets established for a task |
| 18:08 | but that would be where our general policy of not breaking existing functionality would come into play | |
| 18:09 | jenkins_koha | Starting build #385 for job master_maria (previous build: SUCCESS) |
| 18:09 | cait | i think i figured it out |
| 18:09 | gmcharlt | however, I don't think it's necessarily /logically/ different from what would happen if one radically changed an existing cronjob |
| 18:09 | cait | can't change anything - so it's only displaying like in the opac |
| 18:13 | jenkins_koha | Starting build #1661 for job Koha_master (previous build: SUCCESS) |
| 18:16 | pianohacker | gmcharlt: yes, but if you need to update a particular option or change its value, you'll need to put a big, duplicated blob'o'json in updatedatabase.pl or have a rather verbose sql statement |
| 18:16 | *add a new option or change its properties | |
| 18:17 | gmcharlt | well, that argues against the JSON |
| 18:17 | pianohacker | yes, and at least somewhat against the separate table |
| 18:18 | gmcharlt | er, how is "INSERT INTO task_parameters (task_parameter_code, task_code...)" is complicated? |
| 18:19 | pianohacker | well, to account for the many possible variations of parameters |
| 18:19 | hmm, no, those could probably be rolled into one column | |
| 18:19 | sorry, I'm pulling a dcook and thinking out loud | |
| 18:20 | pastebot | "gmcharlt" at 127.0.0.1 pasted "for pianohacker - rev 3 - add default_value and mandatory to task_parameters; get rid of JSON" (47 lines) at http://paste.koha-community.org/163 |
| 18:22 | pianohacker | gmcharlt: that seems reasonable enough, you think we're safe from that table getting too complicated? Also, why active specified per task and not per job? |
| 18:22 | And shouldn't we be careful with terminology to avoid confusing with the task scheduler? | |
| 18:23 | job_type rather than task, say? | |
| 18:23 | jcamins | pianohacker: aren't you replacing the task scheduler? |
| 18:23 | gmcharlt | pianohacker: to accoutn for the possiblity of wanting to disable a give task in the stock jobs across teh board -- however, I'd accept an argument taht task.active is superflous |
| 18:23 | huginn | New commit(s) kohagit: Bug 10415 - Add course reserves to staff client home page <http://git.koha-community.org/[…]2a6ad31f4604961a2> / Bug 11845 - set overlay and import status translatable in addorderiso2709.tt <http://git.koha-community.org/[…]b90fc73ed652c84bd> |
| 18:24 | gmcharlt | pianohacker: 2 - what jcamins said |
| 18:24 | I'm not opposed to a better name, but job_type isn't it | |
| 18:24 | as it doesn't express the concept that a job consists of one or more whazzits | |
| 18:26 | pianohacker | jcamins: Uh, not intentionally, that was more for scheduled reports and such rather than cronjobs? |
| 18:26 | (right?) | |
| 18:27 | oleonard | cait: Is it working correctly? |
| 18:27 | jcamins | pianohacker: if it were possible to schedule cron jobs, it would be possible to schedule a cron job that ran a report. |
| 18:28 | And presumably your proposal is intended to work. | |
| 18:28 | cait | oleonard: i think so |
| 18:28 | i paassed it and it'S PUSHED | |
| 18:28 | so you can't come in now and say i mssed up there :) | |
| 18:28 | ans sorry for the caps - tired typing fingers | |
| 18:28 | we are going to find some dinner now | |
| 18:28 | already 19:30 around here... | |
| 18:28 | * oleonard | wishes he could come! |
| 18:29 | gmcharlt | pianohacker: jcamins: right, hence [09:52:41] <gmcharlt> one of the tasks could be "run scheduled reports" |
| 18:29 | * tcohen | thinks cait is at least in the hackfest :-( |
| 18:29 | cait left #koha | |
| 18:29 | pianohacker | gmcharlt: Yes, but ze reports to be run should be scheduled seperately |
| 18:29 | * oleonard | hear's pianohacker's new French accent coming out |
| 18:29 | pianohacker | they take different kind of parameters |
| 18:29 | gmcharlt | right - nothign says that the "run scheduled reports" task couldn't chck a separate reports schedule queue |
| 18:30 | the point of such a task is to avoid writting a separate cronjob or daemon just to run reports | |
| 18:30 | not that doing so would be the worst thing in the world | |
| 18:30 | pianohacker | right right |
| 18:30 | gmcharlt | and a "run scheduled reports" task would general want to be part of a job with hourly or even minutely periodicity |
| 18:31 | pianohacker | gmcharlt: makes sense |
| 18:31 | I think I have enough for a reasonable first prototypes, I can find xargs sed -i the name of tables later | |
| 18:37 | Sean-CCLSD joined #koha | |
| 18:37 | Sean-CCLSD | Hi all |
| 18:41 | mtompset | Greetings, tcohen. :) |
| 18:42 | tcohen | hi mtompset |
| 18:52 | mtompset | @wunder l7e 5y5 |
| 18:53 | huginn | mtompset: The current temperature in Schomberg, Ontario is -8.0°C (2:52 PM EDT on March 12, 2014). Conditions: Light Snow. Humidity: 85%. Dew Point: -10.0°C. Windchill: -8.0°C. Pressure: 29.56 in 1001 hPa (Steady). |
| 18:53 | mtompset | That explains the blowing snow falling outside my window. YUCK! |
| 18:57 | oleonard | @wunder 45701 |
| 18:57 | huginn | oleonard: The current temperature in Windy Ridge, Athens, Ohio is 2.5°C (2:57 PM EDT on March 12, 2014). Conditions: Overcast. Humidity: 96%. Dew Point: 2.0°C. Windchill: -3.0°C. Pressure: 29.39 in 995 hPa (Falling). Wind Advisory in effect until midnight EDT tonight... |
| 19:00 | rhcl joined #koha | |
| 19:03 | jenkins_koha | Project master_maria build #385: SUCCESS in 54 min: http://jenkins.koha-community.[…]master_maria/385/ |
| 19:03 | * Galen Charlton: Bug 11459: use drop-down to set OrderPdfFormat syspref | |
| 19:03 | * Sophie Meynieux: Bug 11760: In notices, output road type description rather than code | |
| 19:03 | * Jonathan Druart: Bug 11493: Revert "Bug 6869 Batch item modification fails whit accented characters" | |
| 19:03 | huginn | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11459 enhancement, P5 - low, ---, mathsabypro, Pushed to Master , [Signed-Off] OrderPdfFormat pref description could be more verbose |
| 19:03 | jenkins_koha | * Owen Leonard: Bug 11718 - Use new DataTables include in reports templates |
| 19:03 | * Julian Maurice: Bug 11861: Add ISSN column in serials claims table | |
| 19:03 | * Kyle M Hall: Bug 11531: make it possible to edit transport cost for a new libraries | |
| 19:03 | huginn | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11760 normal, P5 - low, ---, sophie.meynieux, Pushed to Master , In notices, code of road type is used instead of descritpion |
| 19:03 | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11493 critical, P5 - low, ---, jonathan.druart, Pushed to Master , Batch Item Modification not handling unicode characters | |
| 19:03 | 04Bug http://bugs.koha-community.org[…]w_bug.cgi?id=6869 normal, P3, ---, fcapovilla, CLOSED FIXED, Batch item modification fails if accented characters are used in the modification form. | |
| 19:03 | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11718 normal, P5 - low, ---, oleonard, Pushed to Master , Use new DataTables include in reports templates | |
| 19:03 | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11861 enhancement, P5 - low, ---, julian.maurice, Pushed to Master , Add ISSN column in serials claims table | |
| 19:03 | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11531 normal, P5 - low, ---, kyle, Pushed to Master , Cannot add new libraries to the transport cost if previously used. | |
| 19:09 | jenkins_koha | Starting build #386 for job master_maria (previous build: SUCCESS) |
| 19:09 | rangi | we have another meeting in a couple of hours eh? |
| 19:11 | gmcharlt | yep |
| 19:12 | rangi | cool |
| 19:13 | gmcharlt | rangi: http://meetings.koha-community[…]014_part1.log.txt |
| 19:14 | rangi | ta |
| 19:16 | aquaman joined #koha | |
| 19:20 | gmcharlt | ah, even better |
| 19:20 | http://meetings.koha-community[…]h_2014_part1.html | |
| 19:20 | Topic for #koha is now Next general meeting is 9 April 2014 at 15:00 and 21:00 UTC. Welcome to the IRC home of Koha http://koha-community.org. Please use http://paste.koha-community.org for pastes. | |
| 19:21 | rangi | ah yep, thats easier to read |
| 19:21 | gmcharlt | had to figure out how to generate them |
| 19:21 | as I hadn't started the meeting properly for meetboot | |
| 19:21 | rangi | ahhh |
| 19:25 | sophie_m joined #koha | |
| 19:27 | sophie_m left #koha | |
| 19:29 | francharb joined #koha | |
| 19:32 | maximep left #koha | |
| 19:34 | gerundio joined #koha | |
| 19:34 | mtompset | gmcharlt: http://meetings.koha-community[…]og.log.html#l-204 -- Internal Server Error? |
| 19:35 | (trying to click on any links) | |
| 19:36 | pastebot | "wajasu" at 127.0.0.1 pasted "hiddenitems patch work in progress 1" (34 lines) at http://paste.koha-community.org/164 |
| 19:37 | wajasu | probably need perl scope related help ^^ |
| 19:37 | gmcharlt | mtompset: I've fixed the link |
| 19:41 | mtompset | wajasu: looking.... |
| 19:45 | wajasu: What led you to this function? | |
| 19:45 | wajasu | rebuild_zebra uses it. |
| 19:47 | mtompset | wajasu: Why would you want to hide anything in the MARC XML? |
| 19:47 | You need everything indexed. | |
| 19:48 | Unless you are ADDing to the MARCXML to note which are hidden and which are not. | |
| 19:48 | wajasu | i'm adding to the exported MARCXML just for the export index |
| 19:49 | a 999h=1 for hidden. | |
| 19:49 | meliss joined #koha | |
| 19:50 | wajasu | then the build facets code just does a 'next' to skip if it sees that in the opac. then the facet counters will be ok. |
| 19:50 | mtompset | So, while 999 be repeated for each 952? |
| 19:50 | ^while^will^ | |
| 19:51 | wajasu | no. its a flag to hide the biblio. i really use the mapping and will have biblio.visible or such later. |
| 19:51 | i only set it if all items are hidden | |
| 19:52 | so the biblios don't show up with all items hidden. | |
| 19:53 | since getRecords builds facets and its too messy to try to move facet building after searchResults. | |
| 19:54 | mtompset | wajasu: So you are differentiating between 942$n and 999$h in that both hide, but the reasoning is different? |
| 19:54 | wajasu | yes. |
| 19:55 | mtompset | and 999$h will be related to how to hide the biblio based on lost and/or hidden items? |
| 19:56 | wajasu | hidden items. thats why i chose 'h' for now. but will put it in mappings. |
| 19:56 | i took care of item_lost in that patch we were testing the other night. | |
| 19:57 | mtompset | did you put up your tweak then? |
| 19:57 | wajasu | since itemlost wants to show biblios when all items are lost. |
| 19:57 | i am working on the tweak now. | |
| 19:57 | mtompset | Did you see gmcharlt's follow up comment? |
| 19:58 | wajasu | let me look |
| 20:00 | i will add a syspref to 'show biblio when all items hidden' for those libraries that wish that. | |
| 20:02 | jenkins_koha | Project master_maria build #386: SUCCESS in 53 min: http://jenkins.koha-community.[…]master_maria/386/ |
| 20:02 | * Fridolin Somers: Bug 11845 - set overlay and import status translatable in addorderiso2709.tt | |
| 20:02 | * Owen Leonard: Bug 10415 - Add course reserves to staff client home page | |
| 20:02 | huginn | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11845 minor, P5 - low, ---, koha-bugs, Pushed to Master , set overlay and import status translatable in addorderiso2709.tt |
| 20:02 | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=10415 normal, P5 - low, ---, oleonard, Pushed to Master , Add course reserves to staff client home page | |
| 20:14 | mtompset | lost items, wajasu. |
| 20:14 | the problem is the overlap between hidden and lost. | |
| 20:15 | wajasu | i got rid of that overlap in the patch from the other night i believe |
| 20:15 | jenkins_koha | Project Koha_master build #1661: SUCCESS in 2 hr 4 min: http://jenkins.koha-community.[…]Koha_master/1661/ |
| 20:15 | * Galen Charlton: Bug 11459: use drop-down to set OrderPdfFormat syspref | |
| 20:15 | * Sophie Meynieux: Bug 11760: In notices, output road type description rather than code | |
| 20:15 | * Jonathan Druart: Bug 11493: Revert "Bug 6869 Batch item modification fails whit accented characters" | |
| 20:15 | * Owen Leonard: Bug 11718 - Use new DataTables include in reports templates | |
| 20:15 | * Julian Maurice: Bug 11861: Add ISSN column in serials claims table | |
| 20:15 | huginn | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11459 enhancement, P5 - low, ---, mathsabypro, Pushed to Master , [Signed-Off] OrderPdfFormat pref description could be more verbose |
| 20:15 | jenkins_koha | * Kyle M Hall: Bug 11531: make it possible to edit transport cost for a new libraries |
| 20:15 | huginn | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11760 normal, P5 - low, ---, sophie.meynieux, Pushed to Master , In notices, code of road type is used instead of descritpion |
| 20:15 | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11493 critical, P5 - low, ---, jonathan.druart, Pushed to Master , Batch Item Modification not handling unicode characters | |
| 20:15 | 04Bug http://bugs.koha-community.org[…]w_bug.cgi?id=6869 normal, P3, ---, fcapovilla, CLOSED FIXED, Batch item modification fails if accented characters are used in the modification form. | |
| 20:15 | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11718 normal, P5 - low, ---, oleonard, Pushed to Master , Use new DataTables include in reports templates | |
| 20:16 | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11861 enhancement, P5 - low, ---, julian.maurice, Pushed to Master , Add ISSN column in serials claims table | |
| 20:16 | mtompset | Yes, but when talking about hiding or showing the biblio, you need to flavour the answer. |
| 20:16 | huginn | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11531 normal, P5 - low, ---, kyle, Pushed to Master , Cannot add new libraries to the transport cost if previously used. |
| 20:17 | mtompset | all hidden none lost, all hidden some lost, all hidden all lost, some hidden none lost, some hidden some lost all items, some hidden some lost not all items, some hidden none lost, none hidden none lost, none hidden some lost, none hidden all lost |
| 20:17 | wajasu | that clears things up tremendously :) |
| 20:18 | mtompset | Oops... was there a duplicate in that list... let's check... |
| 20:19 | all hidden none lost, all hidden some lost, all hidden all lost, some hidden none lost, some hidden some lost all items, some hidden some lost not all items, some hidden all lost, none hidden none lost, none hidden some lost, none hidden all lost | |
| 20:19 | messed up on a none vs all. :) | |
| 20:19 | jenkins_koha | Starting build #1662 for job Koha_master (previous build: SUCCESS) |
| 20:21 | mtompset | I would expect: hide (biblio), hide, hide, show, maybe, show, depends on hidelost, show, show, depends on hidelost. |
| 20:22 | the maybe being that strange all items being covered by lost items and hidden items together. | |
| 20:23 | themaybe is what the system preference needs to help define. :) | |
| 20:25 | wajasu | your all hidden some lost case: is it okay to hide the biblio in OPAC? what if the user logged in searches. would we need to show that in search? or would they get to the details another way? |
| 20:25 | rocio left #koha | |
| 20:25 | wajasu | the user logged in being the one who lost the item |
| 20:26 | mtompset | wajasu: Perhaps it isn't hidden to them based on the PatronEveryBranch or PatronSingleBranch system preferences? |
| 20:27 | wajasu | i'll confirm the other routes are there |
| 21:01 | nengard left #koha | |
| 21:01 | mtompset | when was round 2 supposed to be? |
| 21:02 | rangi | now ish |
| 21:07 | i dont really have anything to add tho | |
| 21:13 | thd | rangi: Would you have any special tricks for creating a hash of hashes in a loop while using use strict? |
| 21:14 | francharb | bye #koha |
| 21:17 | magnuse joined #koha | |
| 21:19 | mtompset | Greetings, magnuse. :) |
| 21:20 | magnuse | hiya mtompset |
| 21:20 | eythian | hi |
| 21:20 | mtompset | Greetings, eythian. :) |
| 21:21 | magnuse | is the meeting over already? or did i mess up the time? |
| 21:21 | mtompset | So, who is chairing meeting 2? It hasn't really started. |
| 21:23 | * magnuse | thought gmcharlt did |
| 21:23 | * gmcharlt | checks watch |
| 21:23 | mtompset | 23 minutes late? |
| 21:24 | gmcharlt | shoot |
| 21:24 | sorry | |
| 21:24 | wajasu | that was a good 1st meeting |
| 21:24 | mtompset | No problem. you were probably debugging and pushing things. :) |
| 21:25 | gmcharlt | #startmeeting Dev Meeting, 12 March 2014, part 2 |
| 21:25 | huginn | Meeting started Wed Mar 12 21:25:09 2014 UTC. The chair is gmcharlt. Information about MeetBot at http://wiki.debian.org/MeetBot. |
| 21:25 | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 21:25 | Topic for #koha is now (Meeting topic: Dev Meeting, 12 March 2014, part 2) | |
| 21:25 | huginn | The meeting name has been set to 'dev_meeting__12_march_2014__part_2' |
| 21:25 | rangi | #info Chris Cormack |
| 21:25 | magnuse | yeah, the first meeting was the best ever - with lots of cheese in it ;-) |
| 21:25 | mtompset | #info Mark Tompsett |
| 21:25 | magnuse | #info Magnus Enger, Oslo Public Library |
| 21:25 | mtompset | Don't mention the bonus cheese for those who attend both. ;) |
| 21:26 | thd | #info Thomas Dukleth, Agogme, New York City |
| 21:26 | gmcharlt | #link http://wiki.koha-community.org[…]ng,_March_12_2014 Agenda |
| 21:26 | #info gmcharlt = Galen Charlton, 3.16 RM | |
| 21:27 | #link http://meetings.koha-community[…]h_2014_part1.html Minutes of first part of meeting | |
| 21:28 | eythian | #info Robin Sheat, Catalyst IT |
| 21:29 | gmcharlt | #topic Performance - Plack |
| 21:29 | Topic for #koha is now Performance - Plack (Meeting topic: Dev Meeting, 12 March 2014, part 2) | |
| 21:29 | gmcharlt | at the first part there was some discussion about work on Plack at the hackfest |
| 21:29 | jcamins | #info Jared Camins-Esakov, C & P Bibliography Services |
| 21:30 | gmcharlt | and I believe dpavlin and ashimema will be trying to organize a consolidation of the various threads of Plack support |
| 21:30 | some stuff I know folks are aware of and have patches for are the Zebra reconnection issues | |
| 21:31 | I, hopefully notfoolheartedly, announced in-core Plack configs for both intranet and OPAC as a goal for the 3.16 release | |
| 21:31 | wizzyrea | #info Liz Rea, Catalyst IT |
| 21:31 | gmcharlt | comments, particularly from Catalyst? |
| 21:31 | eythian | We have one site in production with plack, minor issues have shown up, but nothing world-ending. |
| 21:32 | rangi | yeah issues have gone now we have switched to just making a z3950 connection per search and not trying to reuse it |
| 21:32 | gmcharlt | OPAC-only or with staff as well |
| 21:32 | rangi | (which is what happens under cgi anyway) |
| 21:32 | OPAC only | |
| 21:32 | eythian | Though, there is a bigger issue when it comes to having many sites on one server using it: that won't work very well atm, as there's no ability to pool resources between them. |
| 21:32 | Solutions to this are possible, I just haven't had a chance to look. | |
| 21:32 | wizzyrea | even two seems dodgy. |
| 21:32 | gmcharlt | OK, well that keeps in the realm of experimental |
| 21:33 | * magnuse | hands eythian a bunch of rund tuits |
| 21:33 | gmcharlt | my goal is mostly to have Plack be reasonably functional enough for dev installations that more devs will actually use it that way |
| 21:33 | eythian | yeah, there's no reason that won't work pretty well on the OPAC side. |
| 21:33 | * magnuse | thinks that is an excellent plan |
| 21:33 | eythian | We're hoping to look into the staff client side some time soon. |
| 21:34 | magnuse | eythian: wht resurces would yu want to pool? |
| 21:34 | wajasu | i will attempt to signoff 7844 tonight if someone else doesn't beat me to it. |
| 21:34 | eythian | magnuse: memory |
| 21:34 | mtompset | out of curiousity... if it does get into 3.16 experimentally... does that mean the next release it could go production, or will there be more time than that? |
| 21:34 | eythian | magnuse: the current plack system pre-allocates everything, and doesn't "bubble". |
| 21:35 | So, if you have 5 hosts on it, you use 5 times the RAM as when you have one, even when idle | |
| 21:35 | so if one site gets busy, there's a whole lot of wasted memory you can't use. | |
| 21:35 | magnuse | mtompset: no set schedule - it will be ready for production when we think there are no more evil bugs lurking |
| 21:35 | eythian: ah, thanks | |
| 21:36 | eythian | (because each site has its own plack instance in order to do UID separation properly.) |
| 21:36 | * mtompset | mutters something about version numbers. ;) |
| 21:36 | magnuse | eythian: is that a problem with plack or with koha? |
| 21:36 | thd | eythian: is the it to which you are referring Plack or Zebra? |
| 21:36 | eythian | magnuse: mostly neither |
| 21:37 | thd: *So, if you have 5 hosts on one server | |
| 21:37 | you meant that one? | |
| 21:38 | jcamins | Just as a comment for anyone who isn't familiar with that part of the code, if C4::Context's context switching were to be resurrected and finetuned, Plack could probably run multiple instances from one process, but you'd lose the user separation, which would be bad. |
| 21:38 | eythian | magnuse: if I were to fix it, I'd fix it in plack: allowing it to say "keep 3 around, but go up to 10 if needed and then back down to 3 when thigns are quiet." |
| 21:38 | this may be possible using a different thing than starman. | |
| 21:38 | magnuse | eythian: sounds like a good thing to have, yes |
| 21:38 | eythian | (that is how apache works, fwiw) |
| 21:39 | thd | eythian: Yes, multiple hosts per server with an ambiguous 'it'. |
| 21:43 | magnuse | eythian: sounds like a problem that needs to be solved before we see widespread adoption of plack among vendors... |
| 21:43 | eythian | yup |
| 21:43 | magnuse | (in productin) |
| 21:43 | eythian | the instance we have running it has its own server for unrelated reasons. |
| 21:43 | * magnuse | looks quizzically at the "o" key on his keyboard |
| 21:44 | eythian | and once I did some benchmarking and tuning, making sure it can't run out of ram due to leaks, it's been handling load marvelously. |
| 21:44 | gmcharlt | OK, great |
| 21:44 | moving on | |
| 21:44 | magnuse | but that should not stop us from testing things with plack, using the stuff from bug 7844 |
| 21:44 | huginn | 04Bug http://bugs.koha-community.org[…]w_bug.cgi?id=7844 enhancement, P5 - low, ---, dpavlin, Needs Signoff , plack scripts for developers |
| 21:44 | gmcharlt | #topic Performance and QA |
| 21:44 | Topic for #koha is now Performance and QA (Meeting topic: Dev Meeting, 12 March 2014, part 2) | |
| 21:45 | gmcharlt | #info Question: Should performance be an issue during the QA process? |
| 21:45 | two proposals were made at the first part for policies | |
| 21:45 | first, for QA policy - http://paste.koha-community.org/158 | |
| 21:45 | second, for an addition to the coding guidelines, http://paste.koha-community.org/160 | |
| 21:47 | feedback? objections? +1s? | |
| 21:47 | eythian | That seems reasonable. We do have some bad database latency issues as it is, it'd be good to not have any more. |
| 21:47 | cait joined #koha | |
| 21:48 | * magnuse | waves to cait |
| 21:48 | cait | :) |
| 21:48 | the others are still out | |
| 21:48 | rangi | +1 |
| 21:48 | mtompset | I voted last time, don't want to double vote. :) |
| 21:48 | cait | we had couscous and tajine? tonight |
| 21:49 | oh meeting time? | |
| 21:49 | wahanui | meeting time is always going to favour one section of the globe |
| 21:49 | thd | I would like to suggest an amendment. |
| 21:49 | wizzyrea | +1 - we should definitely pay close attention to performance in circ. |
| 21:49 | cait | #info Katrin Fischer, BSZ Germany |
| 21:50 | magnuse | qam in da house! |
| 21:51 | gmcharlt | thd: yes? |
| 21:51 | thd | Perhaps the policy should acknowledge the possibility that some possible useful features would intrinsically degrade performance and thus may be required to have a system preference to enable or disable them. |
| 21:52 | eythian | I'd also like to include database accesses in the coding guidelines, that's where a _large_ proportion of time is spent at the moment. |
| 21:54 | gmcharlt | thd: -1 to that amendment -- the text as written is loose enough to cover that possiblity, but I don't think it wise to open the door to either "turn this system preference on to make your Koha catalog slow" or "let's syspref away a bad initial implementation of a feature" |
| 21:56 | mtompset | I know in the first meeting there were talks of profiling and perhaps even having some process flag things that generate a 5% slow down. |
| 21:56 | gmcharlt | eythian: I don't think database access per se needs to be optimized for -- sure, a lot of the time a good way to avoid latency is to avoid reading from the database when you can read from a faster cache, but that strikes me as an implementation detail, not a matter for a broad guideline |
| 21:56 | eythian | gmcharlt: I'd put in the same bucket as bandwidth, storage, and memory. |
| 21:56 | gmcharlt | mtompset: yep, there was general agreement that some sort of automated performance testing would be a nice thing to set up |
| 21:58 | eythian | gmcharlt: the issue being that currently an opac-search causes 1,718 database hits, which starts to be counted in actual seconds. |
| 21:58 | (for example) | |
| 21:58 | mtompset | 1718?! |
| 21:59 | eythian | http://debian.koha-community.o[…]obin/opac-search/ <-- mtompset |
| 21:59 | wajasu | N+1 select problem |
| 21:59 | gmcharlt | eythian: we don't get billed for database accesses, just time, as it were |
| 22:00 | eythian | well, it's specifically a performance issue. |
| 22:00 | and the heading is "Performance" :) | |
| 22:00 | gmcharlt | I am not disagreeing that a useful strategy would be to reduce the number of queries made, through various means, but I do not see it as a primary thing that we're trying to reduce or conseve |
| 22:00 | eythian | fair enough. |
| 22:00 | though it is the biggest single performance killer :) | |
| 22:03 | gmcharlt | #agreed http://paste.koha-community.org/158 is accepted as a QA guideline |
| 22:03 | #agreed http://paste.koha-community.org/160 is accepted as a new coding gudeline | |
| 22:04 | any new items folks want to discuss? | |
| 22:05 | eythian | elastic search integration is making progress |
| 22:05 | it works, but it can't yet do anything particularly advanced. | |
| 22:05 | magnuse | yay for progress |
| 22:06 | wajasu | jcamins said i might start testing that stuff. |
| 22:06 | eythian | testing is probably a bit of a strong word at the moment, but you're welcome to have a poke at it. |
| 22:08 | wajasu | that is a java/lucene based index engine. so would that be something we would begin to require java jvm, or would it be optional |
| 22:08 | eythian | you shouldn't run ES on the same server you're running koha. |
| 22:08 | a production ES installation requires a minimum of 3 servers in a cluster for data integrity. | |
| 22:09 | So, you would have your ES servers and point to them from Koha. | |
| 22:09 | (I don't expect ES to be a requirement any time in the near future anyway though.) | |
| 22:09 | magnuse | and zebra should remain as an option as long as there is no z39.50/SRU based on ES, methinks. |
| 22:09 | eythian | yup, that too |
| 22:10 | gmcharlt | eythian++ # thanks for the update |
| 22:10 | anything else? | |
| 22:10 | wahanui | anything else is a guess |
| 22:10 | gmcharlt | wajasu: forget anything else |
| 22:10 | wahanui | gmcharlt: I forgot anything else |
| 22:10 | magnuse | in other news, look out for bug 11926 |
| 22:10 | huginn | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11926 enhancement, P5 - low, ---, wizzyrea, NEW , Render community koha statistic usages |
| 22:11 | eythian | ohh |
| 22:11 | magnuse | we might try to have it signed off and qa'ed before the end of the week... |
| 22:11 | eythian | I want to have realtime stats pushed to logstash one day |
| 22:11 | ah, that's a different thing, ignore me | |
| 22:12 | wajasu | no. I'm just all in that search code and understand many issues now. |
| 22:12 | mtompset | wasn't there discussion of piwik or something on the devel list a while back? |
| 22:13 | * mtompset | goes hunting for the email. |
| 22:14 | BobB joined #koha | |
| 22:14 | magnuse | yeah, paul_p talked about having one central piwik install that any koh site could use, but i *think* it was abandoned, because it would be too much data |
| 22:14 | rangi | plus its already been done |
| 22:15 | http://piwik.koha-community.org/ | |
| 22:15 | magnuse | there is now a basic dancer app to collect data: https://github.com/clrh/hea-app |
| 22:15 | rangi | koha-community.org has been using it for years |
| 22:15 | magnuse | rangi: ah, kewl |
| 22:15 | rangi | and a couple of other koha |
| 22:16 | eythian | can someone who really knows what it is put a description on bug 11926 and the wiki page? It doesn't actually say much about what it is going to do/be. |
| 22:16 | mtompset | Isn't just a matter of putting some js in a system preference to use it? |
| 22:16 | wajasu | should the schema get regenerated? |
| 22:16 | huginn | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11926 enhancement, P5 - low, ---, wizzyrea, NEW , Render community koha statistic usages |
| 22:16 | jenkins_koha | Project Koha_master build #1662: SUCCESS in 1 hr 59 min: http://jenkins.koha-community.[…]Koha_master/1662/ |
| 22:16 | * Fridolin Somers: Bug 11845 - set overlay and import status translatable in addorderiso2709.tt | |
| 22:16 | * Owen Leonard: Bug 10415 - Add course reserves to staff client home page | |
| 22:16 | huginn | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=11845 minor, P5 - low, ---, koha-bugs, Pushed to Master , set overlay and import status translatable in addorderiso2709.tt |
| 22:16 | 04Bug http://bugs.koha-community.org[…]_bug.cgi?id=10415 normal, P5 - low, ---, oleonard, Pushed to Master , Add course reserves to staff client home page | |
| 22:17 | magnuse | eythian: the idea is to enable koha to report stats to the dancer app, like number of records/items, settings for some interesting sysprefs etc |
| 22:18 | * magnuse | has not been involved in the work, but heard a prsentation from the people who have this afternoon |
| 22:21 | * gmcharlt | agrees with eythian that more detail would be nice |
| 22:21 | magnuse | the wiki page does sum it up ,i think |
| 22:21 | eythian | it doesn't if you have no context. |
| 22:21 | wizzyrea | ^^ |
| 22:21 | I'm pretty baffled too, tbh. | |
| 22:22 | magnuse | so, there will be a syspref: do not share data/share data/share data anonymously |
| 22:22 | wizzyrea | something like "a central repository for anonymous worldwide koha usage" would be nice, if that's what it is. |
| 22:23 | magnuse | if sharing is enabled, every month koha will push data to the dancer app |
| 22:23 | that's what it is | |
| 22:23 | to be able to know how many libraries use koha, and their sizes | |
| 22:24 | it was also a goal to report on the setting of some sysprefs, to be able to tell how koha is used, if there are features that no-one uses etc | |
| 22:24 | also the versions that are in use | |
| 22:24 | rangi | but it has to be off by default |
| 22:25 | magnuse | yup, of course |
| 22:25 | * mtompset | agrees with rangi. "Off by default." |
| 22:25 | rangi | so why would someone go turn it on? |
| 22:25 | or even know too? | |
| 22:25 | magnuse | and an option to shar anonymously |
| 22:25 | mtompset | Perhaps something in the about? |
| 22:25 | magnuse | for the same reason they register on the wiki or in web-lib-cats? |
| 22:25 | rangi | this feels like something that should be a plugin |
| 22:25 | wizzyrea | ^^ |
| 22:26 | eythian | it seems that it would be better emailing, properly configured servers shouldn't be allowing port 80 access to the outside world anyway. |
| 22:26 | rangi | and that |
| 22:26 | magnuse | um, a plugin woul hardly increase use? |
| 22:26 | cool, add your thoughts to the bug or the wiki page folks! | |
| 22:26 | :-) | |
| 22:27 | rangi | naw i dont think i will |
| 22:27 | magnuse | rangi: remember bug 6293? ;-) |
| 22:27 | huginn | 04Bug http://bugs.koha-community.org[…]w_bug.cgi?id=6293 enhancement, P5 - low, ---, paul.poulain, RESOLVED DUPLICATE, Add a button to the intranet for registering with the Koha community |
| 22:28 | rangi | yep totally different thing |
| 22:28 | wizzyrea | it doesn't continuously collect data |
| 22:28 | rangi | a button that takes you off to a form to fill in somewhere |
| 22:28 | wizzyrea | and it's just a button, "report in" |
| 22:29 | rangi | plus it was a dumb idea then too |
| 22:29 | i have plenty of those | |
| 22:30 | magnuse | lol |
| 22:30 | ok, i'll shut up now | |
| 22:31 | wizzyrea | :) |
| 22:31 | gmcharlt | any other topics (briefly?) |
| 22:32 | ok, thanks everybody | |
| 22:32 | #endmeeting | |
| 22:32 | Topic for #koha is now Next general meeting is 9 April 2014 at 15:00 and 21:00 UTC. Welcome to the IRC home of Koha http://koha-community.org. Please use http://paste.koha-community.org for pastes. | |
| 22:32 | huginn | Meeting ended Wed Mar 12 22:32:43 2014 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) |
| 22:32 | Minutes: http://meetings.koha-community[…]-03-12-21.25.html | |
| 22:32 | Minutes (text): http://meetings.koha-community[…]4-03-12-21.25.txt | |
| 22:32 | Log: http://meetings.koha-community[…]12-21.25.log.html | |
| 22:34 | wajasu | rangi: is it worth regenerating the schema,koha-community.org or wait until a later release, since its from last august. |
| 22:34 | gmcharlt | wajasu: definitely worth it to do it for 3.14 |
| 22:34 | wajasu | ok. thats what i thought |
| 22:37 | eythian | wajasu: if you're interested in having a look at the ES stuff: http://git.catalyst.net.nz/gw?[…]ds/elastic_search |
| 22:37 | wajasu | ok. |
| 22:40 | i am thinking i can eventually do testing with ES running on my HOST OS, and the koha debian VM talking to it. | |
| 22:41 | eythian | yeah |
| 22:42 | I set it up on one of our servers (it was using a bit much ram on my desktop.) | |
| 22:42 | thd | eythian: I never understood which "it" was which from earlier. |
| 22:43 | eythian | thd: you'll have to re-quote the sentence in question. |
| 22:44 | chrisvella94 joined #koha | |
| 22:46 | irma | g'day chrisvella94 |
| 22:46 | thd | eythian: Was it Zebra or Plack which you were distinguishing for difficulties with multiple hosts on a single server? |
| 22:46 | eythian | thd: plack |
| 22:46 | chrisvella94_ | Hey everyone |
| 22:46 | wahanui | hmmm... everyone is doin' the hot new dance the Cracked Out Kitty Tail Shiver |
| 22:46 | irma | g'day chrisvella94_ |
| 22:46 | wizzyrea | lol. |
| 22:47 | pablito joined #koha | |
| 22:47 | pablito | hi all |
| 22:48 | can someone give me a quick intro to OAI in Koha? | |
| 22:50 | gmcharlt | ... not in 45 seconds we can't |
| 22:51 | magnuse_ joined #koha | |
| 22:55 | wizzyrea | ... |
| 22:56 | dcook | huh |
| 22:56 | * dcook | will never understand |
| 23:01 | magnuse_ | have fun #koha! |
| 23:01 | cait | good night |
| 23:01 | wahanui | If you feel like someone is looking through your window, it's OK, it's just me. |
| 23:01 | cait left #koha | |
| 23:05 | wajasu | i know GetMarcBiblio also got items embedded, so I'l like to get_fields for the itemtag, but don't see any such thing in MARC:Record. i guess I'll just query the DB. |
| 23:07 | eythian | It's not in the MARC in biblioitems, it's recreated when needed from the items table. |
| 23:07 | wajasu | oh |
| 23:21 | tcohen joined #koha | |
| 23:22 | tcohen | eythian: i've just tested your last patch for 10942, marked SO |
| 23:23 | * tcohen | feels marking it Passed-QA would be a bit "endogamic" |
| 23:25 | eythian | tcohen: the debconf one? it worked? |
| 23:25 | colour me surprised. | |
| 23:25 | tcohen | two typos only |
| 23:26 | perlish syntax on bash :-D | |
| 23:27 | eythian | heh |
| 23:28 | maybe let gmcharlt or Martin have a look and see if they can QA. | |
| 23:28 | Oh, there are some minor issues with the messages about translations too, they should go to stderr, not stdout as stdout is for communication with debconf and there's a risk of them getting confused. | |
| 23:29 | s/them/debconf/ | |
| 23:30 | * eythian | -> lunch now though |
| 23:49 | papa joined #koha |
← Previous day | Today | Next day → | Search | Index