| *** ignas has quit IRC | 04:34 | |
| *** replaceafill has quit IRC | 04:35 | |
| *** khildin has joined #schooltool | 08:44 | |
| *** yvl has joined #schooltool | 09:39 | |
| *** menesis has joined #schooltool | 11:23 | |
| *** replaceafill has joined #schooltool | 15:58 | |
| th1a | hi replaceafill, menesis, yvl. | 16:31 |
|---|---|---|
| replaceafill | good morning/afternoon | 16:31 |
| menesis | hi | 16:32 |
| yvl | hi guys | 16:33 |
| th1a | So at the 1/3rd mark in the year, we've spent 60,500 of our 178,666 EUR budget. | 16:33 |
| th1a | That counts 5,000 EUR in travel expenses, which is the bulk of the year. | 16:34 |
| th1a | So basically, we're right on track. | 16:34 |
| yvl | that's great | 16:36 |
| th1a | As long as the Euro doesn't tank. | 16:36 |
| th1a | OK, replaceafill? | 16:36 |
| replaceafill | ok | 16:37 |
| replaceafill | so i tried fixing the school logo issue in remote pdfs | 16:37 |
| replaceafill | it was throwing a not found in a /logo url | 16:38 |
| replaceafill | i found the issue was caused by absolute(app, self.request) + '/logo' | 16:38 |
| replaceafill | a hardcoded path in the logo logic | 16:38 |
| replaceafill | which under regular circumstances produces the full url | 16:38 |
| replaceafill | http://localhost/logo | 16:38 |
| replaceafill | but under remote pdfs produces only /logo | 16:39 |
| replaceafill | so, as yvl suggested, i inserted the data of the logo instead of the url | 16:39 |
| replaceafill | yvl, could you please check: | 16:39 |
| replaceafill | http://bazaar.launchpad.net/~replaceafill/schooltool/celery_school_logo/revision/3586 | 16:39 |
| replaceafill | that seems to solve the problem | 16:39 |
| yvl | sure | 16:39 |
| yvl | looking | 16:39 |
| replaceafill | i used the data_uri adapter | 16:39 |
| replaceafill | on the logo image | 16:40 |
| replaceafill | i also found an issue with the ratio of the logo displayed in the pdf | 16:40 |
| yvl | yes | 16:40 |
| yvl | thanks | 16:40 |
| replaceafill | if you have an image of ~ (1200, 900) | 16:40 |
| yvl | fix looks good | 16:40 |
| replaceafill | ratio produces 1 | 16:40 |
| replaceafill | integer | 16:40 |
| replaceafill | so i inserted a float there | 16:41 |
| replaceafill | to keep the decimal part | 16:41 |
| replaceafill | when calculating the ratio | 16:41 |
| replaceafill | that also displays the logo correctly | 16:41 |
| replaceafill | no stretch, etc | 16:41 |
| replaceafill | yvl, thanks, will merge with trunk then | 16:41 |
| replaceafill | and the second issue i worked on was the <strong>...</strong> content | 16:42 |
| replaceafill | i "kind of" understand the issue | 16:42 |
| replaceafill | but i wanted to ask yvl here about pdf form | 16:42 |
| replaceafill | yvl, i see that for comment fields in pdf forms | 16:43 |
| replaceafill | a row is inserted in a table, right? | 16:43 |
| replaceafill | z3c-widget-table-row.pt | 16:43 |
| replaceafill | being the template for the row | 16:43 |
| replaceafill | then there's z3c-html-widget.pt to render the content of the field | 16:44 |
| yvl | trying to remember... | 16:44 |
| replaceafill | my question is: | 16:44 |
| replaceafill | does the form field to rml logic insert paragraphs inside the table cells? | 16:44 |
| replaceafill | because as far as i've tested, if you only have <td>...<strong>...</strong>...</td> | 16:45 |
| replaceafill | strong is not interpreted | 16:45 |
| replaceafill | but if you have: | 16:45 |
| replaceafill | <td><para>...</strong>...</strong>...</para></td> | 16:45 |
| replaceafill | notice the para element | 16:45 |
| yvl | yes | 16:46 |
| replaceafill | then strong is shown correctly | 16:46 |
| replaceafill | (as bold font) | 16:46 |
| replaceafill | i tracked the form to rml sequence looking for the <para> inclusion | 16:46 |
| replaceafill | but couldn't find it | 16:46 |
| replaceafill | that's why i said initially that i "kind of" understand the issue :) | 16:46 |
| replaceafill | i mean, i know how to fix it | 16:47 |
| replaceafill | i just don't know where pdf form does this part | 16:47 |
| replaceafill | as usually, i'm probably missing something obvious :D | 16:47 |
| yvl | ah | 16:47 |
| yvl | it is in one of the rml widgets | 16:50 |
| yvl | in flourish/skin/rml/z3c-* | 16:50 |
| yvl | it's a bit weird it did not work automatically | 16:51 |
| yvl | values of widgets are rendered with value/@@rml | 16:51 |
| yvl | that should put into multiple <para> | 16:52 |
| replaceafill | well, the base report card view doesn't use z3c form | 16:52 |
| yvl | can you point me to what it uses? | 16:52 |
| yvl | can't remember where it is quickly | 16:52 |
| replaceafill | hold on | 16:53 |
| yvl | ok | 16:53 |
| yvl | found it :) | 16:53 |
| replaceafill | buildHTMLParagraphs(unicode(score.value) | 16:53 |
| replaceafill | right? | 16:53 |
| replaceafill | which is the same logic as html2rml | 16:54 |
| replaceafill | it just doesn't do the <para>...</para> automatically | 16:55 |
| yvl | yes | 16:55 |
| replaceafill | i think we should kill report_utils.py, since it's only for buildHTMLParagraphs | 16:55 |
| replaceafill | and that's already in flourish report | 16:56 |
| yvl | yes | 16:56 |
| yvl | totally | 16:56 |
| replaceafill | will do | 16:56 |
| replaceafill | so, i can insert the <para>s on my own for now | 16:56 |
| replaceafill | and that'll keep me going | 16:56 |
| replaceafill | th1a, so now i'm working on adding the absences | tardy info | 16:57 |
| replaceafill | using a checkbox in the request dialog | 16:57 |
| th1a | OK. | 16:57 |
| replaceafill | i also want to get away from those custom forms for request dialogs | 16:57 |
| yvl | wait | 16:57 |
| replaceafill | since yvl already created some nice z3c.form alternatives | 16:57 |
| yvl | pleas don't insert paras on your own | 16:58 |
| yvl | please use @@rml where appropriate | 16:58 |
| replaceafill | yvl, sure i understand | 16:58 |
| yvl | or @@html2rml | 16:58 |
| yvl | for the life of me | 16:58 |
| yvl | I can't find where rml for report cards is | 16:59 |
| replaceafill | new or old? | 16:59 |
| replaceafill | i mean | 16:59 |
| replaceafill | styled or old? :D | 16:59 |
| yvl | the one that is not working for you :P | 16:59 |
| replaceafill | oh sorry | 16:59 |
| replaceafill | here: | 16:59 |
| replaceafill | http://bazaar.launchpad.net/~replaceafill/schooltool.gradebook/celery/revision/564 | 17:00 |
| replaceafill | it's not in trunk | 17:00 |
| replaceafill | i thought you were looking for the <para> in flourish report | 17:00 |
| replaceafill | my bad | 17:00 |
| yvl | ah, ok | 17:01 |
| replaceafill | <td><tal:block content="comment/heading" />: <tal:block content="structure comment/value" /></td> | 17:01 |
| replaceafill | this is what i need to modify | 17:01 |
| yvl | so basically report_utils in gradebook should be nuked if they're only used for reports ATM. | 17:01 |
| yvl | yes | 17:01 |
| replaceafill | yvl, right | 17:01 |
| yvl | also | 17:02 |
| yvl | it's a bit weird you use a single <td> | 17:02 |
| yvl | comments can be multi-line, right? | 17:02 |
| replaceafill | yes | 17:02 |
| replaceafill | after reading pdf forms | 17:02 |
| yvl | so it depends now on how you want them wrapped | 17:02 |
| replaceafill | i now know i have to use table cells only for the comment content | 17:03 |
| yvl | you can either have <td>heading:</td><td>comment/@@rml</td> | 17:03 |
| replaceafill | <td>activity heading</td><td>html2rml</td> | 17:03 |
| replaceafill | :) | 17:03 |
| yvl | or you should pass comment/text/@@rml | 17:03 |
| replaceafill | right | 17:03 |
| replaceafill | that's what i meant ;) | 17:03 |
| yvl | text being python: heading+': '+value | 17:03 |
| replaceafill | right | 17:04 |
| replaceafill | will do | 17:04 |
| replaceafill | thank you very much yvl | 17:04 |
| yvl | thank you! | 17:04 |
| replaceafill | i understand better now :) | 17:04 |
| * replaceafill done | 17:04 | |
| th1a | OK, cool. | 17:06 |
| replaceafill | (logo fix pushed to trunk) | 17:06 |
| yvl | \o/ | 17:06 |
| th1a | Good to have some communication. ;-) | 17:06 |
| th1a | Thanks replaceafill. | 17:06 |
| th1a | How are the French lessons coming? | 17:06 |
| replaceafill | really cool and weird at the same time :D | 17:06 |
| replaceafill | weird sounds ;) | 17:06 |
| yvl | oui! | 17:08 |
| replaceafill | :D | 17:08 |
| th1a | yvl, si vous plait? | 17:09 |
| replaceafill | :D | 17:09 |
| yvl | non | 17:09 |
| yvl | :) | 17:09 |
| yvl | oh wait | 17:10 |
| yvl | ok | 17:10 |
| yvl | pushed a fix for traceback with report filenames with unicode symbols | 17:10 |
| yvl | made pdf report dialogs use progressbar dialogs | 17:11 |
| yvl | so even if they don't have progress bars | 17:11 |
| yvl | they're refreshed automatically now | 17:11 |
| yvl | and look more consistent | 17:11 |
| yvl | reproduced the fail with missing catalogs | 17:12 |
| yvl | will push fix at some point | 17:12 |
| yvl | you have to use DB with ST plugin, then upgrade to celery without the plugin | 17:13 |
| yvl | so not a common scenario, I'd say | 17:13 |
| yvl | (and plugin disabling breaks ST anyway) | 17:13 |
| menesis | disabling of what plugin? | 17:14 |
| yvl | for example cando | 17:14 |
| yvl | use DB with cando, enter scores and such | 17:14 |
| yvl | disable cando, ST breaks | 17:14 |
| menesis | because of extra demographic field? | 17:15 |
| yvl | because of missing interfaces | 17:15 |
| yvl | like INode | 17:15 |
| yvl | and such | 17:15 |
| yvl | basically DB is filled with broken objects | 17:15 |
| menesis | :( | 17:15 |
| yvl | and we do not handle that gracefully now | 17:15 |
| replaceafill | same with intervention and a marker interface | 17:15 |
| yvl | ever, actually | 17:15 |
| yvl | yes | 17:16 |
| menesis | but something else than I encountered | 17:16 |
| menesis | gradebook and journal can be disabled | 17:16 |
| yvl | and everything else works well? | 17:16 |
| yvl | strange. | 17:16 |
| menesis | I think. | 17:16 |
| yvl | well, I will not argue there ;) | 17:17 |
| yvl | hm... | 17:17 |
| * yvl done | 17:17 | |
| th1a | I don't consider this an important problem at all at this point. | 17:19 |
| th1a | yvl on to big CanDo reports? | 17:19 |
| yvl | looked over everything, found some missed reports | 17:20 |
| yvl | Certificate of competency | 17:20 |
| yvl | Section Competencies | 17:20 |
| yvl | interventions PDF! | 17:21 |
| replaceafill | missed? | 17:21 |
| yvl | did not make properly remote yet :) | 17:21 |
| replaceafill | properly? | 17:21 |
| yvl | well, remote. | 17:21 |
| yvl | are they remote? | 17:21 |
| replaceafill | hhmm | 17:21 |
| replaceafill | yes | 17:21 |
| replaceafill | they're in my cando branch | 17:22 |
| replaceafill | iirc | 17:22 |
| replaceafill | let me see | 17:22 |
| replaceafill | we don't have a schooltool.cando/trunk yet, right? | 17:22 |
| menesis | that's in my report :) | 17:22 |
| replaceafill | :D | 17:22 |
| th1a | menesis? | 17:22 |
| menesis | I have created cando/virginia 0.7 (flourish) series/branches | 17:22 |
| yvl | https://code.launchpad.net/~schooltool-owners/schooltool.cando/trunk | 17:22 |
| menesis | but not merged replaceafill's changes yet | 17:23 |
| replaceafill | http://bazaar.launchpad.net/~replaceafill/schooltool.cando/celery/revision/217 | 17:23 |
| menesis | the big report was added to cando, but not removed from virginia | 17:24 |
| menesis | or so I thought | 17:24 |
| menesis | now I see all is good | 17:24 |
| yvl | ah, great - please check if those reports are still working | 17:24 |
| replaceafill | yvl, ah ok, will do | 17:24 |
| * yvl fiddled with base dialog classes a bit | 17:24 | |
| replaceafill | yvl, because of new changes? | 17:24 |
| replaceafill | ah ok | 17:24 |
| yvl | should be fine, but... you know. | 17:24 |
| replaceafill | sure | 17:24 |
| yvl | allright then | 17:25 |
| yvl | on to the aggregate VA report :) | 17:25 |
| * yvl done. | 17:26 | |
| menesis | replaceafill: so I will merge your cando/virginia celery branches to trunk | 17:26 |
| replaceafill | menesis, thanks! | 17:26 |
| menesis | yeah | 17:27 |
| replaceafill | menesis, could you please also check https://code.launchpad.net/~replaceafill/schooltool/release_fixes | 17:27 |
| replaceafill | rev 3557 in particular | 17:27 |
| replaceafill | i think that didn't land in any previous release yet | 17:27 |
| replaceafill | it's just a small fix to get rid of some unnecessary labels | 17:28 |
| menesis | um... no | 17:28 |
| menesis | it was not merged for release | 17:29 |
| menesis | will do | 17:29 |
| replaceafill | thanks menesis | 17:29 |
| th1a | Good catch replaceafill. | 17:29 |
| menesis | that's it about schooltool | 17:30 |
| menesis | have been syncing zope packages to debian | 17:30 |
| menesis | and taking over a project at pov... | 17:31 |
| * menesis done | 17:31 | |
| th1a | Have you guys seen this: http://amjith.blogspot.com/2012/05/python-profiling-part-1.html | 17:34 |
| th1a | This thing? http://www.vrplumber.com/programming/runsnakerun/ | 17:34 |
| replaceafill | i remember reading an article by Marius some time ago | 17:35 |
| replaceafill | about the same | 17:35 |
| replaceafill | (i think) | 17:35 |
| th1a | Yeah, it just came up in another context so I thought I'd mention it. | 17:35 |
| th1a | OK. The light at the end of the report tunnel is getting brighter. | 17:36 |
| yvl | http://mg.pov.lt/blog/you-gotta-love-profiling.html | 17:36 |
| replaceafill | that one :) | 17:37 |
| replaceafill | i loved profilehooks | 17:37 |
| th1a | I did try doing about 15 simultaneous xls exports on my sandbox. | 17:37 |
| replaceafill | :| | 17:37 |
| yvl | :D | 17:37 |
| replaceafill | how did it go? | 17:37 |
| th1a | Saturated my multi-core CPU nicely. | 17:37 |
| replaceafill | :D | 17:37 |
| th1a | But worked as advertised. | 17:37 |
| th1a | No crazy memory ballooning or anything. | 17:37 |
| yvl | that's good. | 17:38 |
| th1a | OK, thanks guys. | 17:39 |
| th1a | Keep up the good work. | 17:39 |
| th1a | Have a good week/end. | 17:39 |
| * th1a drops the bag of gravel. | 17:39 | |
| yvl | thanks guys | 17:39 |
| replaceafill | thanks everybody | 17:39 |
| yvl | good luck and have fun! ;) | 17:39 |
| replaceafill | au revoir | 17:40 |
| replaceafill | :D | 17:40 |
| yvl | :))) | 17:41 |
| *** replaceafill has quit IRC | 18:17 | |
| *** bigbrovar has joined #schooltool | 18:49 | |
| *** bigbrovar has quit IRC | 19:53 | |
| *** bigbrovar has joined #schooltool | 19:57 | |
| *** menesis has quit IRC | 19:58 | |
| *** menesis has joined #schooltool | 20:52 | |
| *** bigbrovar has quit IRC | 21:02 | |
| *** ignas has joined #schooltool | 21:26 | |
| *** menesis has left #schooltool | 21:36 | |
| *** menesis has joined #schooltool | 21:41 | |
| *** replaceafill has joined #schooltool | 21:49 | |
| * replaceafill finally understood what's wrong with fck values + rml | 22:29 | |
| replaceafill | it's a bug! | 22:29 |
| th1a | A bug! | 22:48 |
| replaceafill | :D | 22:48 |
| th1a | What kind of bug? | 22:48 |
| replaceafill | if your comment has only one paragraph (one line), we get <strong> tags | 22:49 |
| replaceafill | if the comment has several paragraphs (lines) everything works as expected | 22:49 |
| replaceafill | and i've been testing always with one line comments | 22:49 |
| th1a | Ah. | 23:04 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!