| *** menesis has quit IRC | 00:30 | |
| *** mattva01 has joined #schooltool | 01:14 | |
| *** replaceafill has quit IRC | 01:18 | |
| *** alga has quit IRC | 02:24 | |
| *** alga has joined #schooltool | 02:25 | |
| *** th1a has joined #schooltool | 03:28 | |
| *** alga has quit IRC | 04:09 | |
| *** th1a_ has joined #schooltool | 05:32 | |
| th1a | aelkner: You around? | 05:32 |
|---|---|---|
| *** aks has joined #schooltool | 06:14 | |
| *** aks has joined #schooltool | 06:14 | |
| *** th1a has quit IRC | 07:31 | |
| *** menesis has joined #schooltool | 09:52 | |
| *** menesis has quit IRC | 11:20 | |
| *** menesis has joined #schooltool | 14:13 | |
| *** alga has joined #schooltool | 15:08 | |
| *** th1a has joined #schooltool | 15:11 | |
| *** aks has quit IRC | 15:11 | |
| *** ignas has joined #schooltool | 15:40 | |
| *** replaceafill has joined #schooltool | 15:50 | |
| *** replaceafill has quit IRC | 16:10 | |
| *** replaceafill has joined #schooltool | 16:22 | |
| th1a | Hi menesis, replaceafill, aelkner. | 16:31 |
| th1a | Is yvl off? | 16:31 |
| replaceafill | good morning/afternoon | 16:31 |
| th1a | Perhaps he told me that on Monday... | 16:31 |
| aelkner | morning | 16:31 |
| menesis | hi | 16:32 |
| th1a | aelkner: Thanks for sending the clip from the README last night. | 16:32 |
| th1a | Apparently we've got actual teachers in Thailand using the gradebook through SiD's company. | 16:33 |
| aelkner | that's good to hear | 16:34 |
| aelkner | i hope the logic from the README file is clear enough | 16:34 |
| th1a | Ah, I see from an email that yvl will not be here. | 16:34 |
| th1a | OK. | 16:35 |
| th1a | So working on updating the book I noticed yesterday that the default page selection in the tabs view didn't make the last release. Correct? | 16:36 |
| th1a | (I'm just trying to make sure I know what is going on...) | 16:36 |
| th1a | ... | 16:38 |
| th1a | Do you know what I'm talking about menesis? | 16:39 |
| menesis | tabs view was in.. | 16:40 |
| menesis | let me check | 16:41 |
| th1a | OK. Yes the tabs view is there. | 16:41 |
| th1a | There is just another set of changes. | 16:41 |
| th1a | aelkner: Would you like to start in the meantime? | 16:41 |
| aelkner | ok, well, i got the section linkage part of the import to work, tested | 16:42 |
| aelkner | i coded the timetable part as well, but haven't tested yet | 16:42 |
| aelkner | i started to, but i fond out that the oldskin ui crashes even in the case of the old import | 16:43 |
| aelkner | by that i mean, going to the Schedule link from the section view, it crashes | 16:43 |
| menesis | th1a: no, calendar tab and setting default tab came after the release | 16:43 |
| aelkner | but so far, i can only say that is the case of running from within a tests, using browser.serve() | 16:44 |
| aelkner | i need to try this out with live running app, oldskin | 16:44 |
| th1a | OK. That will need to get in before summer, menesis, so either the final precise release or the first precise update. | 16:44 |
| aelkner | perhaps the problem has to do with the test setup | 16:44 |
| th1a | We would know if the actual app didn't work, correct? | 16:45 |
| menesis | th1a: it is merged for precise release | 16:45 |
| th1a | menesis: OK, thanks! | 16:45 |
| aelkner | menesis, do you know how i can tweek my buildout to run the oldskin? | 16:45 |
| menesis | aelkner: edit instance/schooltool.conf | 16:46 |
| menesis | no | 16:46 |
| menesis | edit instance/skin.zcml | 16:46 |
| menesis | and replace | 16:46 |
| menesis | <include package="schooltool.skin.flourish.instance" /> | 16:46 |
| menesis | with | 16:46 |
| menesis | <include package="schooltool.standard" /> | 16:46 |
| menesis | but I have no idea what is available in the old skin. what was there before flourish, should still work, but what of the many new features is available, don't know. | 16:48 |
| aelkner | i got it to run under oldskin, thanks | 16:49 |
| aelkner | apparently, section view, Schedule action doesn't crash in live running instance | 16:50 |
| aelkner | only testing it crashes | 16:50 |
| aelkner | at least i can test manually in the meantime | 16:51 |
| aelkner | i'll see what i can do figuring out the test environment | 16:52 |
| th1a | Can you see if replaceafill has the same problem? | 16:52 |
| aelkner | but at the least, i'll get it to work testing manually and get the export to work | 16:52 |
| th1a | Do you guys have any thoughts about importing photos? | 16:53 |
| aelkner | replaceafill would have to run the test himself inserting a browser.serve() after submitting the xls import | 16:53 |
| aelkner | but he doesn't have to get into that if he doesn't want to | 16:57 |
| * th1a notes that this is the sort of thing which should be fairly easy with a distributed version control system... | 16:57 | |
| th1a | Photos? Bueller? | 16:57 |
| replaceafill | th1a, i'm not sure if the filenames idea would work | 16:58 |
| th1a | Fetch them via http or just put them in the filesystem? | 16:58 |
| th1a | I guess it assumes you have access to the server. | 16:58 |
| replaceafill | http we can do, filesystem i don't know | 16:58 |
| th1a | Right... http might make more sense. | 16:59 |
| th1a | Unless we'd do something more complicated like letting you upload a zip file that contained all the photos. | 16:59 |
| th1a | Which is probably not worth the bother. | 16:59 |
| th1a | http should be easy for us and doable for motivated schools. | 17:00 |
| menesis | don't think adding photos to excel sheet is any easier than upload through the web | 17:00 |
| th1a | Schools that know they want to upload images in bulk should be able to put them in a directory on a web server. | 17:00 |
| menesis | unless you want the exported xls to have photos | 17:00 |
| th1a | You kind of have to have a baseline of technical capacity to even know you want to do it. | 17:01 |
| aelkner | menesis, good point about the export | 17:01 |
| aelkner | we always need to think about export whenever we do importing | 17:01 |
| menesis | to import a zip file, the photos should be named username001.jpg or something | 17:01 |
| th1a | We have to think about it but we don't HAVE to do it. | 17:01 |
| menesis | the same about a directory, or directory on the web.. | 17:02 |
| th1a | It is an issue though. | 17:02 |
| th1a | we do store the images in the zodb, right? | 17:02 |
| menesis | but if they have a 100 photos named DSC0045.JPG without names, they have a problem... | 17:03 |
| replaceafill | th1a, using blobs yes | 17:03 |
| th1a | Hm. | 17:03 |
| th1a | I guess we do have to have some kind of story for this. | 17:03 |
| th1a | Can you put photos in spreadsheet cells? | 17:04 |
| menesis | we store them in zodb... but if blob-dir is set in configuration file, they end up in a blobs directory on the disk | 17:04 |
| th1a | Uh... as files? | 17:04 |
| th1a | Regular image files? | 17:04 |
| replaceafill | i dont think so | 17:05 |
| replaceafill | menesis? :) | 17:05 |
| menesis | sadly, no | 17:05 |
| menesis | I have one image uploaded | 17:05 |
| menesis | and on the disk it is named... | 17:05 |
| th1a | I don't think there is any straightforward way to literally put an image in a cell. | 17:06 |
| menesis | instance/var/blobs/0x00/0x00/0x00/0x00/0x00/0x01/0xda/0xff/0x0394bc19d9f91dcc.blob | 17:06 |
| menesis | :( | 17:06 |
| menesis | so, the photos are efficiently stored on the disk. but not in a form usable outside zodb | 17:07 |
| replaceafill | th1a, to me the easiest way to do it is the big zip file | 17:07 |
| th1a | I think images should be a completely separate import/export step. | 17:07 |
| replaceafill | and maybe providing the user some way to map the photo with the user | 17:07 |
| th1a | And the convention should be image name maps to id. | 17:08 |
| th1a | Export gives you a zip file with all the images named after person id. | 17:08 |
| th1a | You can import the same thing, just matching by id. | 17:08 |
| th1a | I don't think adding a second step there is a big deal, since the alternatives are also complicated. | 17:09 |
| th1a | Objections? | 17:10 |
| replaceafill | not from me | 17:10 |
| aelkner | as long as somebody who understands what to do does it, i can't object :) | 17:11 |
| th1a | I'm thinking this is a separate task. | 17:11 |
| replaceafill | i this for the october release? | 17:11 |
| replaceafill | or april? | 17:11 |
| replaceafill | or the middle? :) | 17:12 |
| th1a | Nothing is for April at this point. | 17:12 |
| replaceafill | ah ok | 17:12 |
| th1a | It isn't a big priority, October. | 17:12 |
| replaceafill | i'd like to work on it | 17:12 |
| th1a | I'll give you the bug. | 17:12 |
| th1a | aelkner: So you should be on course to wrap your task up this week? | 17:13 |
| aelkner | i believe so, yes | 17:13 |
| aelkner | barring any unanticipated problems getting the automated tests to not crash | 17:14 |
| aelkner | perhaps yvl can help me with that issue early next week | 17:14 |
| aelkner | but in the meantime, as i said, i can at least do manual tests | 17:14 |
| aelkner | and it's only the schedule tests that crash | 17:15 |
| th1a | It would be nice if we could quickly determine whether the test crashing is strictly an artifact of your sandbox configuration. | 17:15 |
| aelkner | i'd need someone to reproduce the problem | 17:16 |
| th1a | Can't you push your current code to a branch? | 17:16 |
| th1a | Just a personal junk branch? | 17:16 |
| aelkner | don't need to, the problem is reproducable with the trunk code | 17:17 |
| aelkner | the old section import has timetable importing | 17:17 |
| th1a | OK, can we have replaceafill reproduce it then? | 17:18 |
| aelkner | replaceafill, can you? | 17:18 |
| replaceafill | ok | 17:18 |
| th1a | It seems to me this sort of problem has come up before and ended up being something weird in your sandbox. | 17:18 |
| replaceafill | aelkner, what are the branches involved and what do i have to do exactly to reproduce? | 17:19 |
| replaceafill | trunk? | 17:19 |
| aelkner | replaceafill, you can use trunk | 17:19 |
| replaceafill | i have a trunk instance | 17:19 |
| replaceafill | what do i have to do to reproduce? | 17:19 |
| aelkner | just insert a browser serve in sample_data.txt after the first successful Submit | 17:20 |
| aelkner | you need to run the following command: | 17:20 |
| aelkner | bin/test -vs schooltool.export --at-level 2 -t sample_data.txt | 17:20 |
| aelkner | if you put the browser.serve() after the Submit, you will have the oldskin running | 17:21 |
| replaceafill | aelkner, are you talking about line 32? | 17:21 |
| replaceafill | using line numbers is easier to follow :) | 17:21 |
| aelkner | go to any section and hit the Schedule link | 17:21 |
| aelkner | no, see the errors there | 17:22 |
| aelkner | you have to go to the next one | 17:22 |
| aelkner | line 42 | 17:22 |
| replaceafill | ok, so i insert the serve() call in line 43 | 17:22 |
| replaceafill | doing it... | 17:22 |
| replaceafill | and now running the test command | 17:23 |
| replaceafill | ok, i'm in http://localhost:8000/import.html now | 17:23 |
| aelkner | go to the first section for the first term | 17:24 |
| replaceafill | 2006 Fall? | 17:24 |
| aelkner | when you hit Schedule from there, you should get: LocationError: (None, 'label') | 17:24 |
| replaceafill | http://localhost:8000/schoolyears/2006-2008/2006-fall/sections/art_a_2006-fall_teacher010_000? | 17:25 |
| aelkner | sure | 17:25 |
| replaceafill | aelkner, yes, i get the LocationError | 17:25 |
| replaceafill | it's because the new content providers | 17:25 |
| aelkner | yet if you import the file in question manually in a running instance, it doesn't crash | 17:25 |
| replaceafill | they're only for flourish | 17:25 |
| replaceafill | why are we using the old skin for this? | 17:26 |
| * th1a was going to ask that... | 17:26 | |
| aelkner | when we did the flourish skin last summer, we didn't update the at-level=2 test skin | 17:26 |
| aelkner | if we did, then all of the automated tests would need to be rewitten | 17:27 |
| aelkner | which is perhaps why we didn't do it | 17:27 |
| replaceafill | imho, new tests should use the selenium test runner + flourish | 17:28 |
| aelkner | th1a, rewriting all the tests for flourish layer is something that we could do | 17:28 |
| th1a | Perhaps in this case we should re-write the test suite. | 17:28 |
| replaceafill | i'm talking "new tests" | 17:29 |
| th1a | That is FOR THIS CASE. | 17:29 |
| th1a | Not everything. | 17:29 |
| th1a | Port the import tests. | 17:29 |
| aelkner | the sample_data.txt file covers the import of the large test file | 17:29 |
| th1a | ? | 17:29 |
| aelkner | success tests, so to speak | 17:30 |
| aelkner | then we have other files for specific error tests | 17:30 |
| replaceafill | iirc, sample_data.txt was mostly a load test | 17:30 |
| aelkner | and now we'll have some more new files for the new feature tests that i'm adding | 17:30 |
| replaceafill | that's why it was put in level 2 | 17:30 |
| aelkner | it's not just that file that is put at level 2 | 17:31 |
| th1a | My pov is: | 17:31 |
| aelkner | it's the whole testing of schoooltool.export | 17:31 |
| th1a | 1) we're never going to stop and spend 6 weeks re-writing all tests at once; | 17:31 |
| aelkner | the importer, that is | 17:31 |
| th1a | 2) if there is a problem with old tests and it is simpler to just write fresh new tests, do that. | 17:31 |
| aelkner | what if it is not simler? | 17:32 |
| aelkner | simpler | 17:32 |
| aelkner | i can't say at the moment how i would test some files with oldskin and others with new | 17:32 |
| aelkner | perhaps it would be a similar project to when i broke up the schooltool.app selenium tests | 17:33 |
| aelkner | i'd have to look into that | 17:33 |
| th1a | What do you think replaceafill? | 17:33 |
| th1a | I REALLY don't want to sink up to our waists in this. | 17:33 |
| replaceafill | i'd write new selenium tests | 17:34 |
| replaceafill | we need to keep extending that capability imo | 17:34 |
| replaceafill | not using the old test browser | 17:34 |
| aelkner | i'll look into adding an stest layer to schooltool.export | 17:35 |
| th1a | Yes, I'd say move forward. | 17:35 |
| aelkner | ok | 17:35 |
| replaceafill | and you'd be sure you have flourish available | 17:35 |
| th1a | Just get rid of crufty old tests. | 17:35 |
| replaceafill | not fighting the old skin views | 17:35 |
| th1a | Right. | 17:35 |
| replaceafill | th1a, not get rid of them | 17:35 |
| replaceafill | th1a, add more | 17:35 |
| aelkner | yeah, we don't need to get rid of anyting | 17:35 |
| th1a | OK. | 17:35 |
| aelkner | anyway, i'll report my findings on monday | 17:36 |
| aelkner | that's it for me | 17:36 |
| replaceafill | and from what i've found in the new gradebook tests, you end up with simpler smaller tests | 17:36 |
| replaceafill | several of them | 17:36 |
| th1a | OK. Let's keep this moving aelkner. | 17:36 |
| th1a | Thanks. | 17:36 |
| th1a | replaceafill? | 17:36 |
| replaceafill | ok | 17:36 |
| replaceafill | http://69.164.203.135:6662/schoolyears/2011-2012/20112012/sections/13/projects/Demonstrating%20Workplace%20Readiness%20Skills%3A%20Personal%20Qualities%20and%20People%20Skills/gradebook | 17:37 |
| replaceafill | teacher007 | 17:37 |
| replaceafill | i fixed the selectors in the cando gradebook | 17:37 |
| replaceafill | and also, i set up better data for a demo | 17:37 |
| replaceafill | you could log in as teacher000 for accounting skills | 17:37 |
| replaceafill | teacher001 for drawing skills | 17:38 |
| replaceafill | and so on | 17:38 |
| replaceafill | i think that help dwelsh to see the whole picture better | 17:38 |
| replaceafill | check: http://69.164.203.135:6662/schoolyears/2011-2012/20112012/sections/13/projects/Preparing%20for%20a%20Career%20in%20Auto%20Body%20Repair/gradebook | 17:38 |
| replaceafill | and see the column labels | 17:38 |
| replaceafill | some of them represent optional skills | 17:39 |
| replaceafill | (the black ones) | 17:39 |
| replaceafill | there may be a better way to represent that, but i just followed the old cando skin :) | 17:39 |
| th1a | Which teacher is that last one? | 17:39 |
| replaceafill | teacher007 too | 17:39 |
| replaceafill | i added a resource library for cando | 17:40 |
| th1a | Perhaps making required skills bold. | 17:40 |
| replaceafill | so it can have it's own css, resources, js, etc | 17:40 |
| replaceafill | th1a, i tried it, but i didnt like it :D | 17:40 |
| replaceafill | i'll show it to you | 17:40 |
| replaceafill | also, i was thinking of hiding the total columns always in cando? | 17:40 |
| replaceafill | they don't use those in the "current" version | 17:41 |
| th1a | Sure. | 17:41 |
| replaceafill | they won't need the absences/tardies columns either | 17:41 |
| replaceafill | and hiding these will take me closer to use the generic worksheet yvl has designed for this | 17:41 |
| replaceafill | and now, i'm no this one: http://69.164.203.135/jquery-jquery-ui-4ade134/demos/#tooltip|custom-style | 17:42 |
| replaceafill | hover the text input | 17:42 |
| replaceafill | i'm going to use that for the hover box | 17:42 |
| replaceafill | i've been also reading about the position function of jquery-ui, to add the left/right arrows to make the tertiary nav horizontally scrollable | 17:43 |
| replaceafill | and we could also use it for the menus | 17:43 |
| replaceafill | it has collision detection! | 17:43 |
| replaceafill | http://69.164.203.135/jquery-jquery-ui-4ade134/demos/#position|default | 17:43 |
| th1a | OK. | 17:44 |
| replaceafill | th1a, are you going to send the link to virginia after we have these last features in? | 17:45 |
| replaceafill | hover box and scrollable third nav | 17:45 |
| th1a | Yes, but you might be better able to direct them to what you've done. | 17:45 |
| replaceafill | i think i can have them done today | 17:45 |
| th1a | Write that up and send it to me. | 17:46 |
| replaceafill | ah ok | 17:46 |
| replaceafill | i'll try to fill the others teachers two | 17:46 |
| replaceafill | there are like 3 courses missing skills only | 17:46 |
| replaceafill | i can finish those ones too | 17:46 |
| replaceafill | ok, that's it from me | 17:47 |
| th1a | You don't have to go nuts on the examples. | 17:47 |
| replaceafill | ok | 17:47 |
| replaceafill | yeah, what we have now is enough :) | 17:48 |
| th1a | We don't even want them to look too closely at these because then they'll feel like they have to tell us all the things which are obviously not finished. | 17:48 |
| replaceafill | (data-wise) | 17:48 |
| replaceafill | right | 17:48 |
| replaceafill | we could send screenshots? | 17:48 |
| replaceafill | or put that warning before the demo link :) | 17:48 |
| th1a | The point here is just to show that it exists and doesn't fall over. | 17:49 |
| replaceafill | "work in progress" | 17:49 |
| replaceafill | ah, kk | 17:49 |
| th1a | So actually clicking on it is good. | 17:49 |
| th1a | I'll explain it. | 17:49 |
| replaceafill | cool thanks | 17:49 |
| th1a | There is an ongoing debate about what the schedule for this is. | 17:49 |
| th1a | dwelsh keeps telling VA CTE that it won't be done until August. | 17:49 |
| th1a | Because he's being conservative about it, | 17:50 |
| replaceafill | and he said the third week of april to see the first stuff | 17:50 |
| replaceafill | :) | 17:50 |
| th1a | but I keep pointing out that we say we're going to deliver it a month after we think we really will, then everyone else will add a month to *that* and think it won't be ready. | 17:50 |
| replaceafill | :)) | 17:51 |
| th1a | So it is head games, basically. | 17:51 |
| th1a | But a sys admin somewhere did pretty quickly say that it was going to be a problem if it wasn't ready by August (which of course it would be). | 17:51 |
| th1a | All the core functionality should be ready for testing by June 1 I think. | 17:52 |
| replaceafill | me too | 17:52 |
| th1a | We're a little behind but yvl is going to take some of the tasks slated for aelkner. | 17:52 |
| th1a | I really had yvl working on non-CanDo stuff almost from here on out. | 17:53 |
| th1a | So we can juggle that as necessary. | 17:53 |
| th1a | OK. Thanks replaceafill. | 17:53 |
| th1a | Oh -- don't forget the spanish screenshot, replaceafill. | 17:54 |
| th1a | menesis? | 17:54 |
| replaceafill | oops | 17:54 |
| replaceafill | will create them right now | 17:54 |
| menesis | nothing to say | 17:54 |
| menesis | was talking/designing/drawing | 17:55 |
| th1a | OK. Good enough. | 17:56 |
| th1a | Thanks guys. | 17:57 |
| th1a | Have a great week/weekend and Happy Easter! | 17:57 |
| aelkner | happy easter everyone | 17:57 |
| replaceafill | happy holidays guys | 17:57 |
| *** menesis has quit IRC | 18:50 | |
| replaceafill | th1a, zyt_ | 19:18 |
| th1a | hi replaceafill. | 19:20 |
| replaceafill | th1a, is this ok: http://img209.imageshack.us/img209/4861/pantallazodel2012040410.png | 19:20 |
| th1a | Perfecto! | 19:22 |
| replaceafill | :D | 19:22 |
| *** ignas has quit IRC | 19:45 | |
| *** ignas has joined #schooltool | 21:37 | |
| th1a | aelkner: ayt? | 22:21 |
| *** th1a has quit IRC | 23:58 | |
Generated by irclog2html.py 2.15.1 by Marius Gedminas - find it at mg.pov.lt!
