| *** menesis has quit IRC | 00:28 | |
| *** replaceafill has quit IRC | 01:57 | |
| *** th1a has quit IRC | 02:50 | |
| *** menesis has joined #schooltool | 13:04 | |
| *** khildin has joined #schooltool | 13:38 | |
| *** th1a has joined #schooltool | 13:39 | |
| *** menesis has quit IRC | 14:01 | |
| *** replaceafill has joined #schooltool | 16:27 | |
| th1a | Ah, where's yvl? | 16:31 |
|---|---|---|
| th1a | hi replaceafill. | 16:32 |
| th1a | Or menesis... | 16:32 |
| replaceafill | hey th1a | 16:32 |
| replaceafill | holiday in .lt? | 16:32 |
| th1a | No... | 16:35 |
| th1a | replaceafill: So what was the status of reports in the packages you tested yesterday? | 16:35 |
| th1a | Did they work? | 16:35 |
| replaceafill | i didn't test the reports using glenda's spreadsheets | 16:36 |
| replaceafill | i tested them before that | 16:36 |
| replaceafill | and they worked | 16:36 |
| replaceafill | i can set up the instance again and check | 16:37 |
| th1a | Might as well. | 16:37 |
| th1a | I'm going to have to document what to do with the new report error messages. | 16:38 |
| replaceafill | they're kind of "obscure?" | 16:38 |
| replaceafill | i mean, for average users | 16:38 |
| th1a | Well, like in Glenda's email. | 16:40 |
| th1a | I assume with this system you can't guarantee that the error will get back to SchoolTool. | 16:40 |
| replaceafill | well, you can see the traceback under Server | 16:41 |
| th1a | Ah. | 16:41 |
| replaceafill | Server -> Server information -> Scheduled Tasks | 16:41 |
| * replaceafill starts importing glenda's spreadsheets | 16:42 | |
| th1a | OK, well in that case, I guess I should tell Glenda to look there. | 16:42 |
| replaceafill | right, the traceback *there* is useful for us | 16:43 |
| th1a | We should probably put it back where it usually goes. | 16:44 |
| th1a | Even though it is kind of scary. | 16:44 |
| *** yvl has joined #schooltool | 16:47 | |
| yvl | good morning guys | 16:47 |
| th1a | Ah, yvl. | 16:47 |
| th1a | Hi. | 16:47 |
| replaceafill | good afternoon yvl | 16:47 |
| th1a | Glenda is now into report bugs, so you're needed. | 16:48 |
| yvl | ok | 16:48 |
| th1a | I guess she should dredge up tracebacks for us from the Scheduled Tasks page? | 16:48 |
| * yvl is on EDGE GSM btw, so network is somewhat sluggish | 16:49 | |
| yvl | that would be best | 16:49 |
| yvl | also, logs | 16:49 |
| yvl | celery_report.log would be most valuable | 16:49 |
| th1a | When she goes into the Scheduled Tasks page, she should click on the links for the failed events? | 16:50 |
| th1a | And then there'll be a traceback? | 16:50 |
| yvl | yes | 16:51 |
| th1a | OK. | 16:51 |
| yvl | they should have red TRACEBACK status | 16:51 |
| th1a | I'm also having a problem where on my Saucy install absolutely nothing happens when I click on a report link (no dialog, no nothing). | 16:51 |
| th1a | OK. | 16:51 |
| yvl | yes | 16:51 |
| yvl | so those would be most detailed | 16:52 |
| yvl | but some info can be dug out from celery_report.log | 16:52 |
| yvl | that one is weird | 16:53 |
| yvl | I guess menesis would know more about it | 16:54 |
| th1a | I guess we should see if replaceafill can reproduce it. | 16:54 |
| th1a | I suspect I have some kind of jquery problem. | 16:54 |
| replaceafill | i can set up a Saucy vm and try | 16:55 |
| replaceafill | it doesnt happen in Raring for me | 16:55 |
| th1a | Sorry... I meant Glenda's bug. | 16:55 |
| th1a | Switching context. | 16:56 |
| replaceafill | :D | 16:56 |
| th1a | We can make menesis check out the saucy issue. | 16:56 |
| replaceafill | ah yes, i'm importing skills now | 16:56 |
| th1a | yvl: Do you know where menesis is? | 16:56 |
| yvl | calling him... | 16:57 |
| yvl | should be right with us | 16:57 |
| *** menesis has joined #schooltool | 16:58 | |
| yvl | returning to Glenda's bug | 16:58 |
| yvl | ticked id does not say much | 16:58 |
| yvl | maybe we should add downloadable tracebacks | 16:58 |
| th1a | Yes. | 16:59 |
| yvl | users get tracebacks anyway when stuff breaks | 16:59 |
| yvl | ok, will do then | 16:59 |
| th1a | Glenda is being tolerant at this point as long as we keep things going forward. | 17:00 |
| th1a | I had to make replaceafill skip French yesterday though. | 17:00 |
| th1a | Why don't you go ahead with your report while we wait for replaceafill's test instance to come together. | 17:01 |
| th1a | yvl. | 17:01 |
| yvl | ok | 17:01 |
| yvl | really little to report | 17:01 |
| yvl | decided to go to work from our summer house | 17:02 |
| yvl | so half a day lost traveling | 17:02 |
| yvl | I fixed some bugs, will flag which ones and upload them | 17:02 |
| yvl | once I get my internet more stable | 17:02 |
| th1a | Ah, you're in the country now. | 17:03 |
| yvl | yes :) | 17:03 |
| yvl | still internets, should have 3G, but not stable at the moment | 17:03 |
| th1a | OK. Getting Glenda's reports going will be priority #1. | 17:03 |
| yvl | of course | 17:05 |
| yvl | if you do find how to reproduce them, please share | 17:05 |
| yvl | I do expect there will be either weird simple security flaws | 17:05 |
| yvl | or some case of mishandled empty report somewhere | 17:05 |
| th1a | replaceafill: You're doing this from your PPA instance, right? | 17:06 |
| replaceafill | th1a, ehh, no, sandbox | 17:06 |
| replaceafill | although, i can move the database | 17:06 |
| th1a | I should probably fire up virtualbox... | 17:07 |
| replaceafill | ok, everything set up... trying the report | 17:09 |
| replaceafill | ouch, login form in request dialog! | 17:11 |
| replaceafill | http://imagebin.org/index.php?mode=image&id=269003 | 17:11 |
| replaceafill | trying as clerk | 17:11 |
| th1a | Yeah, that's always an ugly one. | 17:12 |
| yvl | sigh, another loose end that's not a low hanging fruit | 17:12 |
| replaceafill | ok, system failure as a clerk | 17:13 |
| replaceafill | checking the logs | 17:13 |
| replaceafill | hhmm | 17:14 |
| yvl | excellent | 17:14 |
| replaceafill | can't set attribute | 17:14 |
| replaceafill | student_gradebook.__parent__ = gradebook | 17:14 |
| yvl | full logs please | 17:14 |
| yvl | student gradebook's __parent__ was changed to a property recently | 17:14 |
| yvl | AFAIK all places that set it explicitily were deleted | 17:15 |
| replaceafill | i'll try just removing that line, ok? | 17:15 |
| yvl | from where? | 17:15 |
| replaceafill | since it's a property now | 17:15 |
| replaceafill | from the cando task | 17:15 |
| replaceafill | CanDoStudentGradebookReportTask has the line | 17:16 |
| yvl | looking at code.. | 17:17 |
| yvl | yes please | 17:17 |
| yvl | from both places in that file | 17:17 |
| replaceafill | done | 17:18 |
| th1a | Did you get the same kind of error message as Glenda? | 17:19 |
| replaceafill | yes | 17:20 |
| replaceafill | it's a generic error message though | 17:20 |
| replaceafill | yvl, if i remove both .__parent__ lines: | 17:21 |
| replaceafill | 'Section' object has no attribute 'values' | 17:21 |
| replaceafill | for worksheet in worksheets.values() | 17:21 |
| replaceafill | let me paste the full traceback | 17:21 |
| replaceafill | http://pastebin.com/KyrbSs1V | 17:22 |
| yvl | well the self.context.__parent__.__parent__.__parent__ is wrong :D | 17:24 |
| replaceafill | on it | 17:25 |
| th1a | When I relied on self.context.__parent__.__parent__.__parent__ in my old Zope 2 application, I thought it was pretty hacky. | 17:26 |
| yvl | it was :) | 17:26 |
| th1a | Yet surprisingly reliable. | 17:26 |
| th1a | Until you tried to change something... | 17:26 |
| yvl | and when anything in the chain changes... difficult to track down | 17:27 |
| yvl | right | 17:27 |
| yvl | it should be either self.context.__parent__ | 17:27 |
| yvl | or self.context.__parent__.__parent__ :) | 17:27 |
| replaceafill | wow, you guys are good! | 17:27 |
| replaceafill | report successfully generated ;) | 17:28 |
| replaceafill | pushing the fix | 17:28 |
| yvl | :D | 17:28 |
| yvl | remote debugging FTW | 17:28 |
| yvl | teamwork! :D | 17:29 |
| replaceafill | hhmm | 17:29 |
| replaceafill | one problem though :( | 17:30 |
| replaceafill | that's in a baseclass | 17:30 |
| replaceafill | and the __parent__ change affects the Score student view | 17:30 |
| replaceafill | TypeError: ('Could not adapt', <Skill u'Demonstrate positive work ethic.'>, <InterfaceClass schooltool.cando.interfaces.ISkillsGradebook>) | 17:30 |
| replaceafill | 17:30 | |
| yvl | ok | 17:31 |
| yvl | I guess just change to a property | 17:31 |
| yvl | @prop | 17:31 |
| yvl | def worksheets(self): | 17:31 |
| yvl | return self.context.___.___.___ | 17:31 |
| yvl | and change it in children | 17:31 |
| replaceafill | kk | 17:31 |
| yvl | for worksheet in self.worksheets is more readable anyway | 17:32 |
| replaceafill | fixing | 17:32 |
| replaceafill | success | 17:35 |
| replaceafill | :) | 17:35 |
| replaceafill | this OOP thing seems powerful :P | 17:35 |
| replaceafill | checking the other reports as clerk before pushing... | 17:36 |
| th1a | Oh, OK, yvl. | 17:37 |
| th1a | It isn't the top priority probably, but I did put in a bug for an auto-deprecate form. | 17:37 |
| yvl | it does look useful | 17:38 |
| yvl | very useful | 17:38 |
| th1a | Only in a system as insane as VA CTE is it necessary. | 17:38 |
| yvl | true | 17:39 |
| th1a | Anyhow, does that make sense to you? | 17:39 |
| yvl | well | 17:39 |
| yvl | yes | 17:39 |
| yvl | I would implement one thing differently though | 17:39 |
| yvl | add checkboxes in the results table | 17:39 |
| yvl | called deprecated | 17:40 |
| yvl | and a button at the end of the table called Save | 17:40 |
| yvl | or Apply | 17:40 |
| yvl | or something | 17:40 |
| th1a | OK | 17:40 |
| yvl | and a checkbox to select/deselect all of course | 17:40 |
| yvl | so in case of batched results | 17:40 |
| yvl | users would only change displayed ones | 17:41 |
| yvl | but when they click show all | 17:41 |
| yvl | they can change everything | 17:41 |
| yvl | I hope this makes sense | 17:41 |
| th1a | Yes. | 17:41 |
| yvl | kk | 17:41 |
| th1a | OK, might as well excuse yvl then. | 17:42 |
| yvl | thanks :) | 17:43 |
| yvl | I'll stick around just in case | 17:43 |
| th1a | Have a good week/end guys. Enjoy the country... productively. ;-) | 17:43 |
| * th1a drops the bag of gravel. | 17:43 | |
| yvl | thanks! | 17:43 |
| replaceafill | th1a, i still need to fix the forbidden in the request dialog | 17:53 |
| replaceafill | but i have to go now | 17:54 |
| th1a | Sure. There is no rush, she won't get the fixes until tomorrow. | 17:54 |
| replaceafill | ah right | 17:54 |
| replaceafill | ok, will continue when i come back then | 17:54 |
| replaceafill | see you guys later | 17:54 |
| *** replaceafill has quit IRC | 17:59 | |
| *** yvl has quit IRC | 18:25 | |
| *** khildin has quit IRC | 20:11 | |
| *** menesis has quit IRC | 20:59 | |
| *** replaceafill has joined #schooltool | 21:24 | |
| *** khildin has joined #schooltool | 22:15 | |
| *** khildin has quit IRC | 22:16 | |
| th1a | replaceafill: Would you mind zipping up and sending Glenda's spreadsheets to me? | 23:28 |
| *** replaceafill has quit IRC | 23:42 | |
| *** replaceafill has joined #schooltool | 23:43 | |
| replaceafill | th1a, sure | 23:43 |
| th1a | Thanks. | 23:44 |
| replaceafill | th1a, sent | 23:53 |
Generated by irclog2html.py 2.15.1 by Marius Gedminas - find it at mg.pov.lt!