| *** ignas has joined #schooltool | 00:38 | |
| *** jelkner has joined #schooltool | 01:06 | |
| jelkner | replaceafill, u here? | 01:07 |
|---|---|---|
| jelkner | th1a, u here? | 01:07 |
| *** mattva01 has joined #schooltool | 01:07 | |
| *** mattva01 has joined #schooltool | 01:08 | |
| *** jelkner has quit IRC | 01:22 | |
| *** th1a has quit IRC | 01:23 | |
| *** wjohnsto has left #schooltool | 01:58 | |
| *** replaceafill has quit IRC | 02:00 | |
| *** mattva01 has quit IRC | 02:08 | |
| *** ignas has quit IRC | 02:14 | |
| *** th1a has joined #schooltool | 04:52 | |
| *** aks has joined #schooltool | 05:29 | |
| *** th1a has quit IRC | 06:01 | |
| *** aks has quit IRC | 06:04 | |
| *** aks has joined #schooltool | 06:05 | |
| *** aks has quit IRC | 06:55 | |
| *** aks has joined #schooltool | 07:08 | |
| *** aks has quit IRC | 07:08 | |
| *** aks has joined #schooltool | 07:08 | |
| *** yvl has quit IRC | 08:48 | |
| *** yvl has joined #schooltool | 08:50 | |
| *** aks has quit IRC | 09:46 | |
| *** aks has joined #schooltool | 10:16 | |
| *** aks has joined #schooltool | 10:16 | |
| *** Lumiere has joined #schooltool | 10:33 | |
| *** aks has quit IRC | 11:21 | |
| *** aks has joined #schooltool | 11:22 | |
| *** aks has joined #schooltool | 11:22 | |
| *** aks has quit IRC | 11:42 | |
| *** aks has joined #schooltool | 11:42 | |
| *** aks has joined #schooltool | 11:42 | |
| *** yvl has quit IRC | 12:53 | |
| *** ignas has joined #schooltool | 13:51 | |
| *** menesis has joined #schooltool | 13:56 | |
| *** aks has quit IRC | 14:17 | |
| *** yvl has joined #schooltool | 15:19 | |
| *** th1a has joined #schooltool | 16:04 | |
| *** replaceafill has joined #schooltool | 16:23 | |
| th1a | hi menesis, yvl, aelkner_, replaceafill. | 16:30 |
| aelkner_ | morning | 16:31 |
| yvl | good morning | 16:31 |
| replaceafill | good morning/afternoon | 16:31 |
| th1a | OK, so we're going to do some kind of demo/walkthrough with Welsh next Wednesday. | 16:32 |
| th1a | Starting a half hour from now (then). | 16:32 |
| th1a | Can we meet a half hour early next Wed? | 16:33 |
| yvl | fine by me :) | 16:33 |
| aelkner_ | no objections here | 16:33 |
| th1a | OK. | 16:33 |
| th1a | Also, next Mon is a holiday here, but I'm ok with meeting. aelkner_? | 16:34 |
| aelkner_ | i was wondering whether you were planning on taking a three day weekend | 16:34 |
| aelkner_ | i was just going to do whatever you decided | 16:35 |
| th1a | We'll try to keep it short Monday, but it is looking like we'll need to do the big merge at the beginning of the week. | 16:36 |
| th1a | Get everything working in one branch/trunk. | 16:37 |
| th1a | Not that I expect any major problems with that. | 16:37 |
| aelkner_ | true, well, let's say that i'll be available monday morning then | 16:37 |
| aelkner_ | can we have our demo server running only trunk branches so that we can all control what it has | 16:38 |
| th1a | Hm? | 16:38 |
| yvl | aelkner_ means only branches owned by schooltool-owners | 16:39 |
| yvl | then again, if I run a branch in my home dir, there's not much aelkner_ can do about it anyway ;) | 16:40 |
| yvl | (unless replaceafill granted him mega admin rights :) ) | 16:40 |
| th1a | aelkner_: Demo server instances are pretty much anarchy. | 16:40 |
| replaceafill | yvl, i think we all have sudo on the vps, right? | 16:40 |
| th1a | Just worry about your own. | 16:40 |
| * yvl hasn't checked | 16:40 | |
| th1a | Should yvl and/or menesis do this merge Monday Lithuania time? | 16:41 |
| * th1a is not actually expecting this to be a difficult process. | 16:41 | |
| yvl | sure, I can do that | 16:42 |
| th1a | OK. | 16:42 |
| th1a | Make a note of that yvl. | 16:42 |
| yvl | but I guess replaceafill has merged aelkner_'s work anyway | 16:42 |
| yvl | so I doubt there's much to merge ;) | 16:42 |
| yvl | as in - nothing to merge :) | 16:43 |
| th1a | Yes, just making sure here. | 16:43 |
| th1a | aelkner_ is starting to make me paranoid. | 16:43 |
| replaceafill | we all have sudo on the vps btw | 16:43 |
| yvl | cool | 16:43 |
| yvl | in any case, if anyone wants me to merge something on Monday, please notify me of your branches | 16:44 |
| aelkner_ | no wait, there's no need for anyone to notify about merging | 16:44 |
| aelkner_ | we all ust do it now, and as th1a said, it should just work | 16:44 |
| aelkner_ | i've been merging to trunk routinely for a while now | 16:45 |
| th1a | This may be a non-issue. | 16:46 |
| yvl | likely | 16:46 |
| th1a | Lets discuss it for 15 more minutes just in case. | 16:46 |
| th1a | OK... menesis, anything to report? | 16:47 |
| th1a | How's ignas? | 16:47 |
| menesis | Ignas is quickly coding the needed features | 16:47 |
| menesis | :) | 16:47 |
| ignas | not quickly really :) schooltool makes for some fun brain twisters though :) | 16:47 |
| menesis | I fixed buildbot failure yesterday (versions of new dependencies) | 16:47 |
| menesis | and that's it | 16:48 |
| th1a | Thanks ignas & menesis. | 16:48 |
| th1a | Good to have you hacking on ST again, ignas. | 16:48 |
| th1a | yvl? | 16:50 |
| yvl | umm | 16:50 |
| yvl | well, I added some options to Selenium test running | 16:51 |
| yvl | menesis should be able to run it on buildbot now I guess | 16:51 |
| yvl | selenium tests, that is | 16:51 |
| yvl | developers can also run tests on virtual display | 16:52 |
| yvl | they can set latest schooltool.devtools as develop, or wait for menesis to release that | 16:52 |
| yvl | a fun feature is that you can specify screenshots dir now | 16:52 |
| menesis | I have pressed the 'make release' button on buildbot | 16:52 |
| yvl | so if headless tests fail, near to the failure, relevant screenshots are listed | 16:53 |
| yvl | (you can even click on them in your xterm window :) ) | 16:53 |
| yvl | thank you menesis ! | 16:53 |
| menesis | so you will get latest devtools on bin/buildout | 16:53 |
| th1a | Nice. | 16:53 |
| yvl | you will also need latest flourish trunk though | 16:54 |
| yvl | I think the testing part is not compatible with earlier schooltool.devtools version | 16:54 |
| yvl | (s) | 16:55 |
| yvl | :) | 16:55 |
| yvl | anyway | 16:55 |
| yvl | I also twiddled with gradebook a bit, will commit later on | 16:55 |
| yvl | I didn't add score history to cando though | 16:55 |
| yvl | yet | 16:55 |
| yvl | :) | 16:56 |
| * yvl done | 16:56 | |
| th1a | I'd like to see CanDo history on Monday. | 16:56 |
| th1a | Thanks yvl. | 16:56 |
| yvl | sure | 16:56 |
| th1a | aelkner_? | 16:57 |
| aelkner_ | i added the hierarchy table to the document view | 16:57 |
| aelkner_ | i also added an edit view that works like the layer's parents view, i.e., with available, current tables | 16:58 |
| aelkner_ | as manager, visit http://69.164.203.135:36660/documents/virginia-cte | 16:58 |
| aelkner_ | layers have to be created by the user in the layers part of the app | 16:59 |
| aelkner_ | you may want it to be easier for the user, so we should discuss how you would like that | 17:00 |
| aelkner_ | i've been mulling over in my head how to do that, but haven't decided yet | 17:00 |
| yvl | if I may interject | 17:00 |
| aelkner_ | sure, shoot | 17:00 |
| yvl | just to remind that I have downloaded the whole CTE, umm, document | 17:00 |
| yvl | so at some point I could try to make an import | 17:01 |
| yvl | or I could give you the code + data pickle (auto-loading) | 17:01 |
| th1a | What form is that in now yvl? | 17:02 |
| yvl | just thought to remind you, because it looks like everything's shaping up | 17:02 |
| yvl | pure python pickles | 17:02 |
| aelkner_ | i'm not sure what you are referring to, downloading whole CTE, how did you do that? | 17:02 |
| yvl | memory dump, to put in another way | 17:02 |
| yvl | a script, aelkner_ | 17:02 |
| yvl | they have JSON API | 17:03 |
| th1a | Which pickles are you talking about? | 17:03 |
| yvl | these http://docs.python.org/library/pickle.html | 17:03 |
| th1a | Yes, yvl. | 17:04 |
| yvl | the script basically built pythonic objects in memory | 17:04 |
| th1a | OK. | 17:04 |
| yvl | and then dumped the memory to hdd | 17:04 |
| th1a | So we should be able to turn that into a spreadsheet if we wanted to. | 17:04 |
| yvl | that's the point :) | 17:05 |
| th1a | Gotcha. | 17:05 |
| aelkner_ | ah, this is a standalone python script that you wrote, right? | 17:05 |
| yvl | correct | 17:05 |
| yvl | lemme share that with you guys... just don't brake a leg in that code :) | 17:05 |
| ignas | *break :) | 17:05 |
| yvl | you don't say. | 17:06 |
| aelkner_ | brake is the thing | 17:06 |
| yvl | yes, sorry :D | 17:06 |
| yvl | brainglitch | 17:06 |
| aelkner_ | i didn't even notice it until ignas pointed it out :) | 17:07 |
| yvl | or probably a typo, because I was sure I typed break | 17:07 |
| yvl | ok, sorry for interruption | 17:07 |
| aelkner_ | th1a, let's meet after the meeting to discuss the hierarchy ui, ok? | 17:09 |
| aelkner_ | i don't need to take up meeting time for that | 17:09 |
| th1a | But basically we should try to get that working before Wednesday. | 17:09 |
| th1a | That is get the import working. | 17:09 |
| aelkner_ | oh, ok, back to that | 17:09 |
| aelkner_ | yvl, did you say you have the import script working? | 17:09 |
| yvl | no | 17:10 |
| yvl | I have the verso db | 17:10 |
| yvl | ought to write it as XLS or something | 17:10 |
| aelkner_ | and a script that you wrote that accesses it, right? | 17:10 |
| yvl | yes | 17:11 |
| aelkner_ | and that script is still a work in progress, or is it complete? | 17:12 |
| aelkner_ | and what is the next task, to integrate the script into schooltool import xls code? | 17:13 |
| yvl | in a sense it's complete | 17:14 |
| th1a | What I'd like to see is a full test of the existing import code, so the next step would be exporting the data currently in a python data structure into a spreadsheet. | 17:14 |
| yvl | yes :) | 17:14 |
| yvl | and that is a TODO :) | 17:14 |
| th1a | aelkner_ should be able to do that, unless you came up with a seriously obtuse data structure. | 17:14 |
| yvl | I hope so | 17:16 |
| th1a | OK. | 17:16 |
| aelkner_ | anyway, my report is done | 17:17 |
| th1a | Anyhow, if you could push that script yvl, we can take a look at it. | 17:17 |
| yvl | sure | 17:17 |
| th1a | Thanks aelkner_. We'll continue immediately after the meeting. | 17:17 |
| aelkner_ | ok | 17:17 |
| th1a | replaceafill? | 17:18 |
| replaceafill | ok | 17:19 |
| replaceafill | http://69.164.203.135:6661/schoolyears/2011-2012/2012-spring/sections/math_a_2012-spring_teacher001_000/ | 17:19 |
| replaceafill | manager or teacher001 | 17:19 |
| replaceafill | Reports -> Roster | 17:19 |
| replaceafill | we still need a report description for the request dialog | 17:19 |
| replaceafill | yvl, i have a rml/reportlab question | 17:19 |
| replaceafill | do you know of a way to make a <td> to wrap its content | 17:20 |
| *** jelkner has joined #schooltool | 17:20 | |
| replaceafill | so far i've used <td><para>...</para></td> | 17:20 |
| replaceafill | because i haven't found a way to do it | 17:20 |
| replaceafill | and paragraphs give you that | 17:20 |
| th1a | Can we have a check mark instead of the name of the section? | 17:21 |
| th1a | Since we really don't know that it will be meaningful. | 17:21 |
| yvl | hmm, sorry, don't recall how | 17:21 |
| replaceafill | th1a, ah ok | 17:21 |
| yvl | I think using para is the way to go | 17:21 |
| replaceafill | yvl, ah ok | 17:21 |
| replaceafill | :) | 17:21 |
| th1a | Also, put a period at the end of the list of instructors. | 17:21 |
| replaceafill | ok | 17:22 |
| th1a | Are you at a good point to pause on the report styles, replaceafill? | 17:23 |
| replaceafill | th1a, hhmm yes, i think you said you don't want me to work on the existing ones, correct? | 17:23 |
| replaceafill | they need fixing | 17:23 |
| replaceafill | or reviewing | 17:23 |
| th1a | Yes, but we need to get you back on CanDo. | 17:24 |
| th1a | Now at least we can do new ones correctly. | 17:24 |
| replaceafill | ok | 17:24 |
| th1a | That was kind of the main priority. | 17:24 |
| th1a | No point in doing new CanDo reports wrong. | 17:24 |
| replaceafill | ok | 17:25 |
| th1a | You have two big CanDo tasks -- awesome skills browser and IEPs. | 17:25 |
| replaceafill | can i start with iep? | 17:25 |
| replaceafill | i think that's easier | 17:25 |
| th1a | awesome skills browser is going to require some thought. | 17:25 |
| replaceafill | yes | 17:25 |
| replaceafill | btw, yvl i do want you to review/merge my gradebook changes please | 17:27 |
| th1a | Well, I don't understand the IEP functionality at all, so you're kind of on your own (with dwelsh) on that one. | 17:27 |
| replaceafill | th1a, iirc, yvl was ok with using the old implementation | 17:28 |
| replaceafill | and i understand that one well | 17:28 |
| yvl | sure replaceafill, please ask a merge on LP if that's ok | 17:28 |
| aelkner_ | replaceafill, isn't that a situation where there is another tree of skills? | 17:28 |
| replaceafill | yvl, will do | 17:28 |
| replaceafill | aelkner_, in the current implementation, you just mark the iep skills for the student | 17:28 |
| yvl | th1a, IEP is basically replacing all (or many) skills in the worksheet for an individual student | 17:29 |
| replaceafill | and you handle calculations differently based on that | 17:29 |
| aelkner_ | it's just a flag added to the comp object? | 17:29 |
| aelkner_ | annotation? | 17:29 |
| replaceafill | the comp (old term) data is stored in the person's annotations | 17:29 |
| th1a | I do understand it generally. ;-) | 17:29 |
| aelkner_ | the reason i ask, i'm guessing it impacts the data model, no? | 17:30 |
| replaceafill | hhmm i don't think so, why do you think that? | 17:30 |
| th1a | I try to stay out of it though because "IEP" is kind of a big topic, and if I'd start putting my IEP ideas in, I'd almost certainly screw up what dwelsh wants. | 17:30 |
| aelkner_ | the user needs a way to mark certain comps as IEP, right? | 17:31 |
| replaceafill | aelkner_, yes | 17:31 |
| aelkner_ | ok, so where is that mark reflected in the data? | 17:31 |
| aelkner_ | in the new cando | 17:31 |
| aelkner_ | where do you think it should be stored? | 17:31 |
| replaceafill | person's annotations again? | 17:32 |
| replaceafill | iirc, you store enough info to track the skill | 17:33 |
| replaceafill | when you're calculating grades for that person, you check which ieps he has | 17:33 |
| aelkner_ | yes, i can't see any reason to do that differently in the new cando, unless someone else does | 17:34 |
| aelkner_ | oh, wait | 17:34 |
| aelkner_ | when you say info to track the skill, what are talking about? | 17:34 |
| aelkner_ | is it the external_id, i wonder | 17:35 |
| replaceafill | iirc, year, term, section, comp | 17:35 |
| aelkner_ | we can discuss this later | 17:35 |
| replaceafill | sure | 17:35 |
| replaceafill | yvl, can you expand your "replacing" idea above | 17:35 |
| yvl | ? | 17:36 |
| replaceafill | "th1a, IEP is basically replacing all (or many) skills in the worksheet for an individual student" | 17:36 |
| yvl | ah | 17:36 |
| yvl | it's not an idea | 17:36 |
| yvl | that's just the way I understand it | 17:36 |
| replaceafill | ah | 17:36 |
| th1a | replaceafill: I only don't understand it enough to be helpful to someone who already does. | 17:36 |
| replaceafill | th1a, i think i can have something working quickly, and we could decide from there? | 17:37 |
| aelkner_ | this sounds like a schooltool.virginia feature, doesn't it? | 17:37 |
| th1a | No. | 17:37 |
| th1a | Get going on it replaceafill. | 17:38 |
| aelkner_ | oh, you see IEP as a standard cando thing, ok | 17:38 |
| replaceafill | th1a, kk | 17:38 |
| * replaceafill done | 17:38 | |
| replaceafill | ah btw | 17:38 |
| replaceafill | i have my meeting today at 1:30 :) | 17:38 |
| replaceafill | at the ministry | 17:38 |
| th1a | Oh RIGHT! | 17:38 |
| th1a | Good luck! | 17:39 |
| replaceafill | i'll let you know how it goes :) | 17:39 |
| replaceafill | thanks! | 17:39 |
| aelkner_ | ah, yes, jelkner mentioned it, good luck! | 17:39 |
| th1a | Have a good week and weekend gentlemen! | 17:39 |
| * th1a drops the bag of gravel. | 17:39 | |
| replaceafill | thanks everybody | 17:39 |
| th1a | aelkner_: Need a 2 minute break. | 17:40 |
| aelkner_ | sure, replaceafill, do we want to continue? | 17:40 |
| aelkner_ | i mean, you have your meeting coming, perhaps tomorrow? | 17:40 |
| aelkner_ | it's up to you | 17:41 |
| replaceafill | we can continue | 17:41 |
| replaceafill | in the current implementation, there's an adapter to mark students | 17:41 |
| replaceafill | if they have iep | 17:41 |
| aelkner_ | so that's one annotation, right? | 17:41 |
| replaceafill | yes | 17:42 |
| * th1a is on the phone with dwelsh now. | 17:42 | |
| aelkner_ | that's just a yes/no flag | 17:42 |
| replaceafill | everything is person's annotations in current iep | 17:42 |
| aelkner_ | so there's a second annotation key for the connection between person and particular comp, right? | 17:42 |
| replaceafill | well, it's the same adapter, but yes | 17:42 |
| aelkner_ | which file is that in? | 17:43 |
| replaceafill | cando/iep/iep.py | 17:43 |
| aelkner_ | ok, so i'm wondering if we want to do it with relationships this time around? | 17:46 |
| aelkner_ | the new data model is just full of them, and it is the advantage that object database gives us | 17:47 |
| aelkner_ | namely, deleting an object doesn't break other objects that refer to it | 17:47 |
| aelkner_ | isn't that a vulnerability of old cando? | 17:47 |
| replaceafill | we could | 17:47 |
| replaceafill | instead of a plain list | 17:47 |
| aelkner_ | so if you create the relationship between ISkill and IBasicPerson, that will in turn create the annotation for you | 17:49 |
| aelkner_ | also, the whole schoolyear part won't be needed anymore | 17:49 |
| aelkner_ | because we are keeping the skills outside of years in the new cando | 17:49 |
| replaceafill | hhmm that i have to check | 17:50 |
| aelkner_ | ok, let me know what you find | 17:52 |
| aelkner_ | replaceafill, is there anything else you'd like to ask/say, or are we done for now? | 17:54 |
| replaceafill | no aelkner_ , we're done | 17:54 |
| replaceafill | i'll try to get there quickly to get your feedback again | 17:54 |
| aelkner_ | ok, cool, looking forward to seeing your findings | 17:55 |
| replaceafill | thanks aelkner_ | 17:55 |
| aelkner_ | th1a, are 2 minutes up? | 17:57 |
| th1a | I'm off the phone with Welsh. | 17:58 |
| th1a | Regarding the levels in documents -- I think the levels should be designated and created as part of the add document form. | 17:59 |
| aelkner_ | you see, have the new levels versus the existing levels to sort out | 18:01 |
| aelkner_ | i mean, if a school wants to revise the document for the coming year | 18:01 |
| aelkner_ | they will create a new document, but they won't need to create new levels necessarily | 18:01 |
| aelkner_ | and even if the document revision needs a new level, they would only need to create that | 18:02 |
| aelkner_ | and then have it be linked parent to child to the existing levels | 18:02 |
| th1a | Let's not worry about that case at this point. | 18:03 |
| aelkner_ | so there needs to be a way to combine creating new levels with using existing | 18:03 |
| * th1a 's perspective is getting into "lets get this done and move on" mode. | 18:03 | |
| aelkner_ | well, what you are suggesting is to get it to work for just one document | 18:04 |
| th1a | Yes. | 18:04 |
| aelkner_ | and if a second document is created, then a whole new set of levels are | 18:04 |
| th1a | I understand aelkner_. We might fix this later. | 18:04 |
| aelkner_ | that isn't out of the question for me, i'm ust checking | 18:04 |
| aelkner_ | so the add form can be the place where the hierarchy is defined simply as a list of level titles | 18:05 |
| th1a | Yes. | 18:05 |
| aelkner_ | really a text are that llows newlines would work for that | 18:05 |
| aelkner_ | allows | 18:05 |
| aelkner_ | the validation would be each non-blank line is a level | 18:06 |
| aelkner_ | would it need to be any more involved than that? | 18:06 |
| th1a | Oh... | 18:06 |
| aelkner_ | text area is what i meant to type | 18:07 |
| th1a | Well, you need to have at least two. | 18:07 |
| aelkner_ | you know, as i mentioned on monday, the ui can handle 0, 1 or more than 1 level | 18:07 |
| th1a | That strikes me as less reliable than letting you add levels like in, say, the score system view. | 18:08 |
| aelkner_ | how do you mean reliable, i'm not following | 18:09 |
| aelkner_ | are you saying that it needs to work like the score system add view | 18:10 |
| aelkner_ | because each level may have more than just the title attribute | 18:10 |
| aelkner_ | so it needs to be a table like that view with add button? | 18:10 |
| aelkner_ | to add a new row? | 18:10 |
| th1a | You're just more likely to get garbage in a textbox. | 18:10 |
| th1a | I just prefer a form where there is one row per level. | 18:11 |
| aelkner_ | so am i right about comparison to score system add view | 18:11 |
| aelkner_ | add button next to last row, being empty row? | 18:11 |
| th1a | Yes, but the only problem with that is that the the lowest two should exist by default. | 18:12 |
| th1a | So it kind of screws up the top down workflow. | 18:12 |
| aelkner_ | what should the default two be, and should there be a delete button as well? | 18:13 |
| aelkner_ | ok, here's the thing | 18:13 |
| aelkner_ | i understand that you want to support renaming skillsets and skills | 18:14 |
| aelkner_ | virginia gives our first example of using that feature | 18:14 |
| aelkner_ | but it's not a requirement for the user, is it? | 18:14 |
| aelkner_ | i mean skillsets and skills are usable concepts for a default user, aren't they? | 18:14 |
| th1a | There have to be at minimum the bottom two levels -- skillsets and skills. | 18:14 |
| th1a | I guess there needs to be a delete button. | 18:15 |
| aelkner_ | we could have the form load as follows: | 18:15 |
| aelkner_ | blank row with add button | 18:15 |
| aelkner_ | Skillset row with delete button | 18:15 |
| aelkner_ | Skill row with delete button | 18:15 |
| aelkner_ | the user can sort it out from there | 18:16 |
| aelkner_ | btw, Add and Remove? | 18:16 |
| aelkner_ | remove is better user english than delete, more techy, right? | 18:17 |
| th1a | Sure. | 18:17 |
| aelkner_ | so, the user could remove both existing rows if they want to right? | 18:18 |
| aelkner_ | and we wold allow the document to be added with no levels? | 18:18 |
| aelkner_ | as i said, the ui already supports it | 18:18 |
| aelkner_ | the default names of the add lnks and table legends is Skillset and Skill | 18:18 |
| th1a | No, there have to be those two. | 18:19 |
| aelkner_ | i had to code the views to handle no hierarchy anyway, otherwise they would be dangerous | 18:19 |
| aelkner_ | ok, so regardless of the fact that we can handle no hierachy, we won't allow it in the add form | 18:19 |
| aelkner_ | they have to define at least two levels | 18:20 |
| th1a | Yes. | 18:20 |
| aelkner_ | and they can just just use the defaults the the add form loads with | 18:20 |
| aelkner_ | ok, i'm good for the add form change | 18:20 |
| aelkner_ | do we want to keep the table edit pencil and view that i put there? | 18:21 |
| aelkner_ | i mean in the document index.html view, after it is added | 18:21 |
| aelkner_ | did you follow the pencil yet? | 18:22 |
| th1a | I don't see anything wrong with it at this point. | 18:23 |
| aelkner_ | ok, i can make the add form have only Remove buttons next to rows above bottom two | 18:24 |
| aelkner_ | and it cold use javascript for adding/removing rows rather than hitting the server each time | 18:24 |
| aelkner_ | as it does in the score system add view | 18:24 |
| th1a | That would be fine. | 18:25 |
| aelkner_ | ok, i know what i need to do now, thanks | 18:25 |
| th1a | Thanks aelkner_. | 18:27 |
| th1a | Also... | 18:27 |
| th1a | We're going to need to stick a pin in this and work on making sure the imports are rock solid for Wednesday. | 18:27 |
| th1a | So... be looking for a good point to pause. | 18:27 |
| aelkner_ | ok, i understand how you're setting this up in that we can come back to address other issues later | 18:28 |
| aelkner_ | like using existing levels or having nodes that have children at different levels, etc. | 18:28 |
| *** ignas has quit IRC | 18:29 | |
| aelkner_ | but regarding imports, right now all we have is the raw skills/nodes/levels importer | 18:30 |
| aelkner_ | no document importer | 18:30 |
| th1a | That is not a concern at the moment. | 18:30 |
| th1a | Actually, I'm most worried about assigning skillsets to sections/courses. | 18:30 |
| aelkner_ | ah, that, we don't have that yes, true | 18:31 |
| aelkner_ | yet | 18:31 |
| th1a | So we should probably at least write an importer for that before Wed. | 18:32 |
| aelkner_ | i'm noticing that we don't have a relationship set up for that | 18:32 |
| aelkner_ | it is currently implemented with old-style annotations | 18:33 |
| th1a | Otherwise we need the awesome skills browser to do it ttw. | 18:33 |
| aelkner_ | i'm wondering why we have these two different inconsisten apporaches in the data model | 18:33 |
| aelkner_ | i mean, the document, layers, nodes and skilllsets are all linked via relationship | 18:34 |
| aelkner_ | and that good, talking advantage of the zope feature, no need to worry about delete triggers | 18:34 |
| aelkner_ | currently, i'm seeing that course.py uses old-style annotations which are not so safe | 18:34 |
| aelkner_ | i guess this is a bad time to think about that, leading up to wednesday | 18:35 |
| aelkner_ | we could get the import to work with current data model | 18:35 |
| aelkner_ | then after the demo, we could fix the model, then the import | 18:35 |
| th1a | Yes. | 18:36 |
| aelkner_ | we should take care of the fix before installing at a client site if we can possibly do that | 18:37 |
| aelkner_ | save us some hassle | 18:37 |
| aelkner_ | so as for the course skillset importer, it could be a simple sheet, with year id as title | 18:38 |
| aelkner_ | and wth one row per course, course id and skillset id list as two columns | 18:38 |
| th1a | I'd just make year a row. | 18:38 |
| aelkner_ | one big sheet | 18:39 |
| th1a | Yes. | 18:39 |
| th1a | In CanDo now are these associated per course or section? | 18:39 |
| aelkner_ | three columns year id, course id, skillsets list | 18:39 |
| aelkner_ | by course | 18:39 |
| th1a | skillset id's? | 18:39 |
| aelkner_ | sorry, both | 18:39 |
| th1a | OK, so basically plan on having that done one way or another before next wed. | 18:39 |
| aelkner_ | so, two sheets | 18:40 |
| aelkner_ | CourseSkills and SectionSkills | 18:40 |
| aelkner_ | CourseSkills has thre columns, year, course, skillset list | 18:40 |
| aelkner_ | secction one has year, term, section id, skillset list | 18:40 |
| th1a | replaceafill: Do both course and section skillsets work in the current gradebook? | 18:41 |
| replaceafill | th1a, skills are extracted from the course | 18:42 |
| replaceafill | i mean, from the section's courses | 18:42 |
| th1a | Not the section at this point, so for demo purposes we should only worry about course? | 18:43 |
| replaceafill | i think so | 18:43 |
| replaceafill | yes | 18:43 |
| aelkner_ | replaceafill, so you're not using ISectionSkills(section) | 18:43 |
| replaceafill | aelkner_, i am | 18:44 |
| replaceafill | but that adapter extracts the skillsets from the courses | 18:44 |
| replaceafill | right? | 18:44 |
| aelkner_ | no, it uses the section skills key, annother annotation | 18:44 |
| aelkner_ | i'm guessing you didn't write course.py, am i right? | 18:44 |
| replaceafill | nope | 18:45 |
| replaceafill | yvl did | 18:45 |
| *** alga has left #schooltool | 18:45 | |
| aelkner_ | i'm seeing XXX at bottom that says maybe relationship :) | 18:45 |
| replaceafill | aelkner_, see the all_worksheets property | 18:45 |
| replaceafill | @property | 18:46 |
| replaceafill | def all_worksheets(self): | 18:46 |
| replaceafill | sheets = [] | 18:46 |
| replaceafill | courses = self.courses | 18:46 |
| replaceafill | for course in courses: | 18:46 |
| replaceafill | skills = ICourseSkills(course) | 18:46 |
| replaceafill | sheets.extend(skills.values()) | 18:46 |
| replaceafill | return sheets | 18:46 |
| replaceafill | sorry for the flooding... | 18:46 |
| replaceafill | that's how you get the worksheets in the gradebook | 18:46 |
| replaceafill | that's what i meant when i said the skills are extracted from the courses | 18:47 |
| aelkner_ | yeah, i'm looking at the same code | 18:47 |
| aelkner_ | annotations_current_worksheet_key is defined but never used | 18:48 |
| aelkner_ | ah, but that's not relavant, sorry | 18:48 |
| replaceafill | :) | 18:48 |
| aelkner_ | i see your point about how section skills are just dynamically built from course skills | 18:49 |
| aelkner_ | so, th1a, to answer your earlier question, we only need a course skill importer | 18:49 |
| th1a | kk | 18:49 |
| aelkner_ | replaceafill, this is weird, course skillsets are really sub-classes of GenericWorksheet | 18:51 |
| aelkner_ | and the __name__ of the course skillset worksheet thing is used to access global skillsets container | 18:53 |
| aelkner_ | so a course can have only one skillset | 18:53 |
| * th1a has to step out for a bit. | 18:53 | |
| aelkner_ | replaceafill, does that sound right? | 18:53 |
| th1a | Catch you guys later. | 18:54 |
| replaceafill | sorry, i was on the phone | 18:54 |
| aelkner_ | replaceafill, we don't need to sort this out now, you are heading to an important meeting | 18:54 |
| replaceafill | aelkner_, ah right, i think yvl designed it that way, so the gradebook code works on them | 18:55 |
| replaceafill | (skillsets) | 18:55 |
| aelkner_ | it's a risky implementation, but we can fix it after the demo | 18:56 |
| aelkner_ | anyway, i'll let you go for now, good luck today! | 18:57 |
| replaceafill | thanks aelkner_ ! | 18:58 |
| replaceafill | i'm done with my slides :) | 18:58 |
| replaceafill | long time of not using impress :P | 18:58 |
| aelkner_ | never used it, is that the slide show creator of choice? | 18:58 |
| replaceafill | it's the one in libreoffice | 18:58 |
| aelkner_ | that would be a yes :) | 18:59 |
| yvl | hey guys | 19:06 |
| yvl | I uploaded cte-db.tar.gz @ 69.164.203.135 | 19:06 |
| yvl | it's in my home dir | 19:06 |
| yvl | you should have the permissions, so please fetch that | 19:07 |
| yvl | aelkner_, can you please confirm that I did not mess up access rights there? | 19:07 |
| yvl | and that you can get it? | 19:07 |
| replaceafill | i see it and can copy it | 19:10 |
| yvl | excellent | 19:10 |
| yvl | good luck to you today | 19:10 |
| replaceafill | thanks yvl | 19:11 |
| replaceafill | i'll need an explanation on Doodad though | 19:11 |
| replaceafill | ;) | 19:11 |
| yvl | :D | 19:11 |
| replaceafill | (joking) | 19:11 |
| yvl | ;) | 19:12 |
| * yvl signs off | 19:12 | |
| yvl | happy coding guys :) | 19:12 |
| *** yvl has quit IRC | 19:12 | |
| *** th1a has quit IRC | 19:58 | |
| *** th1a has joined #schooltool | 20:09 | |
| *** jelkner has quit IRC | 20:16 | |
| *** menesis has quit IRC | 20:31 | |
| eein | google moog = now work done | 21:02 |
| eein | no work | 21:03 |
| th1a | Mmmmm... moog. | 21:03 |
| *** replaceafill has quit IRC | 22:07 | |
| *** menesis has joined #schooltool | 22:14 | |
Generated by irclog2html.py 2.15.1 by Marius Gedminas - find it at mg.pov.lt!