| *** highvoltage has quit IRC | 02:46 | |
| *** highvoltage has joined #schooltool | 02:46 | |
| *** ignas has quit IRC | 03:08 | |
| *** aks has joined #schooltool | 06:15 | |
| *** aks has joined #schooltool | 06:15 | |
| *** aks has quit IRC | 08:55 | |
| *** aks has joined #schooltool | 09:07 | |
| *** yvl has joined #schooltool | 10:17 | |
| *** highvoltage has quit IRC | 10:36 | |
| *** highvoltage has joined #schooltool | 10:38 | |
| *** menesis has joined #schooltool | 11:46 | |
| *** ignas has joined #schooltool | 12:23 | |
| *** aks has quit IRC | 14:17 | |
| *** menesis has quit IRC | 14:32 | |
| *** menesis has joined #schooltool | 15:15 | |
| *** th1a has joined #schooltool | 16:31 | |
| *** th1a has quit IRC | 16:32 | |
| *** th1a has joined #schooltool | 16:33 | |
| th1a | hi aelkner_, yvl, menesis. | 16:33 |
|---|---|---|
| aelkner_ | morning | 16:33 |
| yvl | good morning guys! | 16:34 |
| th1a | yvl: How are we doing? | 16:35 |
| yvl | lost momentum :| | 16:35 |
| yvl | or rather focus | 16:35 |
| yvl | I was actually wondering | 16:36 |
| yvl | where are the latest cando branchers of Alan and Douglas? | 16:37 |
| th1a | lol | 16:37 |
| * yvl thought of adding score history to cando gradebookish views, but was unable to find anything that works | 16:37 | |
| th1a | aelkner_? | 16:38 |
| yvl | as in - where one can create skills, make them available somehow, and score them in a gradebook | 16:38 |
| yvl | probably that's a question for replaceafill | 16:38 |
| aelkner_ | yvl, you had provided replaceafill with a hack to allow him to code gradebook | 16:39 |
| th1a | I guess I wasn't being paranoid when I suggested we sort this out today. | 16:39 |
| yvl | no, I did not, aelkner_ | 16:39 |
| yvl | ah, I know what you mean | 16:40 |
| yvl | but that was not a hack | 16:40 |
| aelkner_ | well, i shouldn't be telling you what you and he did, only you and he know that i guess | 16:40 |
| aelkner_ | it had to do with the fact swe had no competencies yet | 16:40 |
| yvl | but we did had competencies | 16:40 |
| yvl | we did not have "document model" or how you call it | 16:40 |
| aelkner_ | right, that | 16:41 |
| th1a | Documents are not at all required to enter competencies and use the gradebook. | 16:41 |
| aelkner_ | th1a, how is that, i don't understand | 16:41 |
| * th1a sighs. | 16:41 | |
| yvl | :))) | 16:41 |
| aelkner_ | i can sigh, too, but it doesn't help to do so | 16:42 |
| yvl | true :) | 16:42 |
| yvl | well, in short | 16:42 |
| yvl | like we talked many many many times before | 16:42 |
| yvl | you don't see document model in the gradebook | 16:42 |
| th1a | We have this data model which works, except it doesn't make sense to the average person as a way to enter and view the skills tree as a whole. | 16:43 |
| yvl | hence you don't need it to have a gradebook | 16:43 |
| yvl | true | 16:43 |
| yvl | so if you want to create a Project (a thing we discussed quite a lot this winter) | 16:43 |
| yvl | you don't need a tree-like structure | 16:43 |
| th1a | So for the 80% (or more) of users which really have simple tree-like documents they're trying to type in, we have the document views. | 16:43 |
| yvl | you just need a class to teach | 16:43 |
| th1a | You don't need anything other than skillsets. | 16:44 |
| yvl | there you go, 2 explanations :) | 16:44 |
| yvl | in any case, it would help to know where the latest working gradebook is | 16:45 |
| yvl | so, you know, I can add UI *there* :) | 16:45 |
| aelkner_ | look, i do my part to merge my changes to trunk, so you don't have to ask me where my code is | 16:46 |
| yvl | cool | 16:46 |
| th1a | It would also help to know where replaceafill is... | 16:46 |
| aelkner_ | as far as the gradebook working without assigning skillsets, i don't know how replaceafiull did that | 16:47 |
| th1a | Probably having connection problems. | 16:47 |
| aelkner_ | i haven't seen the code either | 16:47 |
| yvl | probably, th1a | 16:47 |
| yvl | well, I did merge *something* from replaceafill | 16:47 |
| yvl | and I merged something else, because aelkner_'s changes needed something in gradebook that was not in gradebook trunk | 16:47 |
| yvl | (or maybe that was replaceafill's changes that needed that extra thing) | 16:48 |
| aelkner_ | right, he needed it | 16:48 |
| yvl | so that's one thing to keep in mind for the demo | 16:48 |
| aelkner_ | that's what i was referring to as your hack, something you added to gradebook that he used in his cando | 16:48 |
| yvl | update not only cando, but also others | 16:48 |
| th1a | OK, well, can we make sure replaceafill knows where what yvl is working on is, so we can probably get this definitively straightened out this afternoon? | 16:49 |
| yvl | I want to add cando score history views to cando gradebooks | 16:50 |
| yvl | for that I need, well, cando gradebooks | 16:51 |
| yvl | (this was for the record :) | 16:51 |
| th1a | Yes, ofc. | 16:51 |
| aelkner_ | obviously we are missing replaeafill here | 16:51 |
| yvl | true | 16:51 |
| yvl | btw, offtopic th1a - http://elinux.org/BeagleBoardUbuntu#BeagleBone | 16:52 |
| yvl | no need to rewrite in node.js :) | 16:52 |
| th1a | Look, if everyone could send emails to everyone else with the links to their relevant branches, that would be awesome. | 16:52 |
| th1a | Yes, yvl. | 16:52 |
| th1a | We may just need to wait for Raspberry Pi to be widely available. | 16:52 |
| aelkner_ | if only we had a web site, say a launchpad or something, where one cold go to find code | 16:53 |
| yvl | th1a, mine is in place, except for the no-yet-implemented part I'm blocking on | 16:53 |
| yvl | aelkner_'s code is in | 16:53 |
| yvl | we need replaceafill :) | 16:53 |
| th1a | Did you look here: https://code.launchpad.net/~replaceafill | 16:54 |
| yvl | aelkner_, if you can find me a working gradebook, that would be swell | 16:54 |
| aelkner_ | i haven't touched the gradebook for months | 16:54 |
| th1a | yvl: You don't have to worry about it at this point. | 16:54 |
| th1a | The ball is in replaceafill's court. | 16:54 |
| yvl | ok | 16:54 |
| aelkner_ | and replaceafill has code that he has been keeping to himself for months | 16:54 |
| * yvl too, maybe, but that is a secret! | 16:55 | |
| th1a | It is probably here: https://code.launchpad.net/~replaceafill/schooltool/cando | 16:55 |
| yvl | I checked that out, it seems like an unfinished merge | 16:55 |
| aelkner_ | yes, that's his branch, funny how launchpad let's you find that :) | 16:56 |
| aelkner_ | note he hasn't commited to it for a month | 16:56 |
| th1a | Well, anyhow, we'll get it sorted out. | 16:56 |
| th1a | He's mostly not been working on CanDo lately. | 16:56 |
| yvl | oooh :D | 16:56 |
| * yvl thought it was 04.28 today :D | 16:57 | |
| th1a | Anyhow, anything else yvl? | 16:57 |
| yvl | so I assumed Douglas pushed something Fri :D | 16:57 |
| yvl | no, th1a | 16:57 |
| yvl | that's that | 16:57 |
| th1a | OK, no use banging your head over this, we'll get it sorted out. | 16:57 |
| th1a | aelkner_? | 16:58 |
| aelkner_ | i added the layer table to the document add view | 16:58 |
| aelkner_ | http://69.164.203.135:36660/documents | 16:58 |
| aelkner_ | it uses javascript to allow the user to add to the table, if they hit the Add layer button, it adds a row | 16:59 |
| aelkner_ | it doesn't allow them to add a row if the top row is blank | 16:59 |
| aelkner_ | it doesn't care if they have blank rows when they submit, it just ignores them | 16:59 |
| th1a | OK. | 17:00 |
| th1a | That'll do for now. | 17:00 |
| th1a | What is the Course List for? | 17:00 |
| aelkner_ | what Course list? | 17:01 |
| aelkner_ | the add document view doesn't have a course list | 17:02 |
| aelkner_ | so what are you asking about? | 17:02 |
| th1a | The Skills Document view does. | 17:02 |
| aelkner_ | link please | 17:02 |
| th1a | http://69.164.203.135:36660/documents/new-standards | 17:02 |
| aelkner_ | so, you see that your top layer is Course | 17:03 |
| aelkner_ | so there's an Add link titled Course | 17:03 |
| th1a | Oh, right. | 17:03 |
| th1a | I tricked myself. ;-) | 17:03 |
| aelkner_ | and a Course list, i.e., the child nodes | 17:03 |
| th1a | np. | 17:03 |
| th1a | OK. | 17:04 |
| th1a | I'll have to remember that one when someone asks me. ;-) | 17:04 |
| aelkner_ | the other think i worked on was groking the getverso.py module yvl pointed us to | 17:05 |
| aelkner_ | thing | 17:05 |
| aelkner_ | i see how it goes to the cte site to download json data for the cte skills tree | 17:05 |
| aelkner_ | and it populates a crazy pickle db with these tricky python dictionary/clever __getattr__ lazy load objects | 17:06 |
| yvl | that is crazy | 17:06 |
| th1a | crazy, tricky and clever? | 17:07 |
| aelkner_ | the tripple threat :) | 17:07 |
| yvl | but if you edit only getverso.py to make an xls or whatever | 17:07 |
| yvl | you don't need to think about how it's done under the hood | 17:07 |
| yvl | I wanted it to be able to resume downloading, and this was a simple hack | 17:08 |
| yvl | (since downloading everything is slooooooooooooooooooooow) | 17:08 |
| yvl | it took many an hour | 17:08 |
| yvl | but I wrote an example how to look at them | 17:08 |
| yvl | and I hoped if you stepped over that with pdb it would be really clear | 17:09 |
| yvl | (the example starts at if __name__ == '__main__') | 17:09 |
| *** Douglas has joined #schooltool | 17:09 | |
| aelkner_ | i looked at that and did pdb around | 17:09 |
| Douglas | good morning! | 17:09 |
| yvl | hi man! | 17:09 |
| th1a | Who is this Douglas? | 17:10 |
| aelkner_ | morning Douglas! | 17:10 |
| * Douglas is at the mall :( | 17:10 | |
| th1a | Getting some wifi action? | 17:10 |
| Douglas | th1a: did you get my phone message? | 17:10 |
| Douglas | yes :) | 17:10 |
| Douglas | sorry for interrupting, i'll go read the logs | 17:10 |
| th1a | Douglas: Oh, no. I didn't check. | 17:11 |
| th1a | That's not a very reliable way to get in touch with me I'm afraid. | 17:11 |
| Douglas | th1a: ah, no prob | 17:11 |
| th1a | OK, so... back to the aelkner_ and yvl show... | 17:13 |
| th1a | Isn't there just something aelkner_ can iterate through? | 17:13 |
| yvl | it is | 17:14 |
| yvl | let's let aelkner_ continue | 17:14 |
| aelkner_ | right | 17:14 |
| aelkner_ | so what we need is a script that runs outside of the schooltool server that reads from the cte site | 17:15 |
| aelkner_ | and creates an xls file, something that will take hours as yvl just said | 17:15 |
| yvl | no | 17:15 |
| yvl | getverso.py has a database and it loads it | 17:15 |
| yvl | you need to iterate the loaded database and create an xls file | 17:16 |
| yvl | i.e., edit getverso.py, and change it to output and xls | 17:16 |
| aelkner_ | i know what i would need to do at this point, i was just wondering why not have created the xls directly | 17:17 |
| yvl | did not have the format back then | 17:17 |
| yvl | and wanted to preload everything because | 17:18 |
| yvl | - getting everything is fast | 17:18 |
| yvl | - xls format might change for some reason | 17:18 |
| yvl | sorry | 17:18 |
| aelkner_ | sure, ok | 17:18 |
| yvl | getting everything is slow :D | 17:18 |
| th1a | So... | 17:19 |
| th1a | Can you handle this now aelkner_? More questions? | 17:19 |
| aelkner_ | ok, so the idea is to create an xls file that contains nodes and layers to import | 17:21 |
| th1a | Yes. | 17:21 |
| aelkner_ | we already have an importer that handles Nodes and Layers sheets, so that's in place | 17:21 |
| th1a | Yes... | 17:21 |
| aelkner_ | it's raw, and it doesn't know about documents | 17:21 |
| aelkner_ | so we need a Documents sheet i guess | 17:22 |
| th1a | That is not the point of this operation aelkner_. | 17:22 |
| th1a | We do eventually. | 17:22 |
| aelkner_ | and it would have a table, one row per document | 17:22 |
| th1a | But we want to import all the skills. | 17:22 |
| th1a | For the demo. | 17:22 |
| th1a | And then we DO need to quickly write an importer that connects skillsets to courses. | 17:23 |
| th1a | Today, if possible. | 17:23 |
| aelkner_ | the title of the sheet for course skillsets should be 'Course SkillSets'? | 17:24 |
| aelkner_ | and it would have a table with year, course id, skillsets as columns | 17:24 |
| th1a | That sounds pretty good. | 17:25 |
| th1a | (it isn't very complicated) | 17:25 |
| aelkner_ | actually, yvl, looking at course.py, i see that a course has only one skillset, right? | 17:27 |
| th1a | I don't think it should. | 17:27 |
| yvl | no | 17:27 |
| aelkner_ | actually, a course doesn't have link to skillset | 17:28 |
| yvl | there is a thing called CourseSkillSet | 17:28 |
| yvl | it is put in CourseSkills I think | 17:28 |
| yvl | not it, they | 17:28 |
| yvl | they (CourseSkillSet) are put in CourseSkills I think | 17:28 |
| aelkner_ | CourseSkillSet is derived from GenericWorksheet | 17:28 |
| yvl | yes | 17:29 |
| aelkner_ | so it's just a gradebook worksheet, not a cando skillset | 17:29 |
| yvl | yes | 17:29 |
| yvl | but it allows access to the actual skillset | 17:30 |
| aelkner_ | __name__ | 17:30 |
| yvl | this is because, at least in the CTE case | 17:30 |
| aelkner_ | the __name__ of the course skillset worksheet thingy matches that of a skillset in the ISkillSetsContainer | 17:31 |
| yvl | they had different "required" flags when deploying to different courses | 17:31 |
| yvl | yes, aelkner_ | 17:31 |
| yvl | and when you iterate the CourseSkillSet | 17:32 |
| yvl | you get CourseSkill (s) | 17:32 |
| aelkner_ | so, the importer would create one CourseSkillSet object for each skillset in the third column | 17:32 |
| aelkner_ | and attach it to the course | 17:32 |
| yvl | umm... maybe? :) | 17:33 |
| yvl | yes | 17:33 |
| yvl | I think | 17:33 |
| aelkner_ | CourseSkills is the list of worksheets | 17:33 |
| aelkner_ | each __name__ of which is the name of a global skillset | 17:33 |
| yvl | right | 17:33 |
| yvl | so, yes | 17:33 |
| aelkner_ | ok, the first thing i'll do is write the Course Skills importer today, make sure that works with the gradebook | 17:35 |
| aelkner_ | that's assuming i can get the gradebook code that uses the course skills | 17:35 |
| aelkner_ | or is that in trunk already? | 17:35 |
| th1a | Douglas: There is some confusion about the location of your code. | 17:36 |
| *** Douglas_ has joined #schooltool | 17:36 | |
| th1a | Douglas_: There is some confusion about the location of your code. | 17:37 |
| Douglas_ | ah yes | 17:37 |
| Douglas_ | can i do my report now? | 17:37 |
| aelkner_ | i'm done | 17:37 |
| th1a | Thanks aelkner_. | 17:37 |
| Douglas_ | yvl: sorry for blocking you today :( | 17:38 |
| yvl | that's ok Douglas_ | 17:38 |
| Douglas_ | i lost my connection on friday and i was expecting to getting it back on the weekend to merge my last changes | 17:38 |
| *** Douglas has quit IRC | 17:38 | |
| Douglas_ | and to request the merges | 17:38 |
| Douglas_ | ping? | 17:39 |
| th1a | hi | 17:39 |
| Douglas_ | ah | 17:39 |
| Douglas_ | ok, so i hope my ISP changes the damn modem today, if they don't i'll go to Marcela's office to push my branches | 17:40 |
| Douglas_ | i can't ssh from here | 17:40 |
| Douglas_ | yvl: question | 17:40 |
| Douglas_ | i saw you already merged "some" stuff from my branch | 17:40 |
| Douglas_ | for the skills gradebook i uncommitted the last revision (a trunk merge iirc) | 17:41 |
| Douglas_ | is it easier for you if i merge to trunks? | 17:41 |
| yvl | it's fine any way | 17:41 |
| yvl | just let me know :) | 17:41 |
| Douglas_ | ah ok | 17:41 |
| Douglas_ | sure, yvl will do | 17:42 |
| Douglas_ | apologies again | 17:42 |
| Douglas_ | th1a: i'm done | 17:42 |
| th1a | OK, so Douglas_ is merging to trunk and then we should be set? | 17:42 |
| * Douglas_ thinks so | 17:43 | |
| th1a | And Douglas is starting to work on IEP's? | 17:43 |
| Douglas_ | th1a: i did | 17:43 |
| Douglas_ | i haven't finished the ui yet though | 17:43 |
| Douglas_ | i'll work on that today | 17:44 |
| th1a | OK. | 17:44 |
| aelkner_ | th1a, Douglas_can we meet later today, after Douglas_has merged perhaps? | 17:44 |
| th1a | This evening? | 17:45 |
| aelkner_ | i'm free all day today and tomorrow | 17:45 |
| Douglas_ | aelkner_: i'll get online in the afternoon for sure | 17:46 |
| aelkner_ | this evening, or late afternoon, after Douglas_ has merged, i'd like to check in to discuss status | 17:46 |
| aelkner_ | ah, at home you mean | 17:46 |
| Douglas_ | either at home or Marcela's office | 17:46 |
| aelkner_ | can we pick a time to meet, any is good for me | 17:47 |
| th1a | Are you two hours behind us Douglas_? | 17:47 |
| Douglas_ | yes | 17:47 |
| Douglas_ | 6 pm your time? | 17:47 |
| Douglas_ | is that good? | 17:47 |
| th1a | No... | 17:47 |
| th1a | What about 4:00 our time? | 17:48 |
| Douglas_ | hhm ok | 17:48 |
| aelkner_ | that's 2:00 in Marcell's office, right? | 17:48 |
| Douglas_ | yes | 17:48 |
| aelkner_ | is that too soon? | 17:48 |
| aelkner_ | how about 8:00 this evening our time | 17:48 |
| th1a | This merge shouldn't take long. | 17:48 |
| Douglas_ | nope | 17:49 |
| Douglas_ | aelkner_: i'm ok with that time | 17:49 |
| Douglas_ | if th1a is | 17:49 |
| th1a | No, let's do 4:00 | 17:49 |
| Douglas_ | ok | 17:49 |
| aelkner_ | i could need as much as an hour after i know that the merge has happened to grok the diff and | 17:49 |
| aelkner_ | then i may have final questions about how everything will integrate together | 17:50 |
| aelkner_ | oh, i know | 17:50 |
| Douglas_ | aelkner_: we can meet tomorrow too if you want | 17:50 |
| Douglas_ | i mean you and i | 17:50 |
| aelkner_ | Douglas_, what's the dependencies on schooltool.gradebook changes | 17:50 |
| aelkner_ | and are those changes in schooltool.gradebook trunk now? | 17:50 |
| Douglas_ | my flourish branch | 17:50 |
| Douglas_ | aelkner_: no not yet | 17:50 |
| Douglas_ | but they work | 17:50 |
| Douglas_ | together | 17:51 |
| aelkner_ | but that's part of what you will be doing this afternoon, right, merging into schooltool.gradebook trunk | 17:51 |
| Douglas_ | yes | 17:51 |
| aelkner_ | got it, ok, so then i'll want to see the two diffs and then we can meet | 17:52 |
| Douglas_ | ok | 17:52 |
| Douglas_ | i'll be here at 4 pm your time | 17:52 |
| aelkner_ | when will i be able to start looking at the diffs | 17:52 |
| aelkner_ | cause there's no point to meeting until after that | 17:52 |
| Douglas_ | ok | 17:53 |
| aelkner_ | we could meet at 4pm our time if you have the merge by 2pm our time? | 17:53 |
| Douglas_ | that's lunch time at Marcela's office aelkner_ , but if it's possible i'll merge before and send an email about it | 17:54 |
| Douglas_ | i mean before 2pm your time | 17:54 |
| aelkner_ | ok, sounds good | 17:54 |
| Douglas_ | sorry about all this mess guys | 17:55 |
| aelkner_ | i'll let you go Douglas_ | 17:55 |
| aelkner_ | np | 17:55 |
| th1a | OK, thanks guys. | 17:55 |
| aelkner_ | happy merging, see you at 4pm | 17:55 |
| yvl | good luck guys! | 17:55 |
| aelkner_ | th1a, i have a question for you | 17:55 |
| Douglas_ | thanks everybody | 17:55 |
| th1a | So we're meeting a half hour early Wednesday and then meeting with Welsh. | 17:55 |
| aelkner_ | yvl, you don't have any updates to getverso.py, rght? | 17:55 |
| yvl | nope | 17:56 |
| aelkner_ | ok, cool | 17:56 |
| aelkner_ | th1a, can i call you, or you call me? | 17:57 |
| th1a | You can call me. | 17:57 |
| aelkner_ | after the bag of gravel, ofc | 17:57 |
| th1a | oh... | 17:58 |
| * th1a drops the bag of gravel. | 17:58 | |
| Douglas_ | see you guys | 17:58 |
| *** Douglas_ has quit IRC | 17:58 | |
| *** aelkner_ has quit IRC | 18:22 | |
| *** aelkner has joined #schooltool | 19:35 | |
| *** th1a has quit IRC | 19:43 | |
| *** menesis has quit IRC | 19:51 | |
| *** aelkner has quit IRC | 19:52 | |
| *** aelkner has joined #schooltool | 19:55 | |
| *** replaceafill has joined #schooltool | 20:00 | |
| *** ignas has quit IRC | 20:22 | |
| replaceafill | aelkner, you around? | 20:39 |
| aelkner | yes | 20:39 |
| replaceafill | aelkner, we don't have an xls for assigning skills to courses yet, right? | 20:39 |
| replaceafill | xls importer* | 20:39 |
| aelkner | that's right, i'll be adding that today | 20:40 |
| replaceafill | ah ok | 20:40 |
| replaceafill | i just was wondering if i missed it | 20:40 |
| replaceafill | thanks! | 20:40 |
| aelkner | btw, the importers are broken now | 20:40 |
| replaceafill | oh | 20:40 |
| replaceafill | in trunk? | 20:40 |
| aelkner | i'm fixing them now to work of the app context | 20:40 |
| aelkner | yes, in trunk | 20:40 |
| replaceafill | ok | 20:41 |
| aelkner | i'm fixing them right now to work off pf app context | 20:41 |
| replaceafill | cool, i'll use the UI then | 20:41 |
| aelkner | and the link for Import Skill Data will go right below Import XLS in the School tab view | 20:41 |
| aelkner | replaceafill, ayt? | 21:33 |
| replaceafill | aelkner, yes | 21:33 |
| aelkner | ok, i just pushed some more changes to cando trunk | 21:34 |
| aelkner | the importer and exporter now work off of app context | 21:34 |
| aelkner | the links are fond in the school tab view | 21:34 |
| * replaceafill goes to see | 21:34 | |
| replaceafill | ah cool, i can use the importer now | 21:35 |
| aelkner | except for the course skills which i'm doing next | 21:36 |
| aelkner | i don't know if you saw the chat before, but i | 21:36 |
| aelkner | i'll be making a Course Skills sheet that has a three-column table | 21:36 |
| replaceafill | yes, you explained that while i was online | 21:36 |
| aelkner | Year ID, Course ID, SkillSet IDs | 21:36 |
| aelkner | ok, good | 21:37 |
| replaceafill | can you help me with something | 21:37 |
| replaceafill | i don't understand | 21:37 |
| aelkner | sure, shoot | 21:37 |
| replaceafill | see cando.course.SectionSkills | 21:37 |
| replaceafill | that's the class to get the "worksheets" for the skills gradebook | 21:37 |
| replaceafill | similar to the old Activities interface | 21:38 |
| aelkner | yes | 21:38 |
| replaceafill | the thing is, all_worksheets spits ICourseSkillSet objects | 21:39 |
| replaceafill | which is similar to the gradebook worksheet (it stores the skills) | 21:39 |
| aelkner | right | 21:39 |
| replaceafill | but i cannot navigate from a ICourseSkillSet back to the section :( | 21:40 |
| replaceafill | which the gradebook view depends on | 21:40 |
| replaceafill | i can only navigate to the course | 21:40 |
| replaceafill | for example in the gradebook we have: | 21:41 |
| replaceafill | self.section = self.context.__parent__.__parent__ | 21:41 |
| replaceafill | meaning worksheet/activities/section | 21:41 |
| aelkner | yes, i know | 21:42 |
| aelkner | i think what needs to change is: | 21:42 |
| aelkner | all_worksheets needs to locate the sheets as it serves them up | 21:42 |
| replaceafill | using a location proxy, right? | 21:43 |
| aelkner | like the gradebook does for the worksheet, to give you an example to use, yes | 21:43 |
| aelkner | then the parent thing should work | 21:43 |
| aelkner | btw, self.section = self.context.__parent__.__parent__ | 21:44 |
| aelkner | that should be handled with an adapter ISection(worksheet) | 21:44 |
| replaceafill | +1 | 21:44 |
| aelkner | perhaps we never got around to refactoring it | 21:44 |
| aelkner | let's talk about that after the demo | 21:44 |
| replaceafill | sure, i think we have enough tests now to do that kind of thing | 21:45 |
| aelkner | we can have a code status discussion and maybe do a little refactoring | 21:45 |
| aelkner | yes, thank god for the tests! | 21:45 |
| replaceafill | ok, i'll review my location proxy, something is not right in my configuration | 21:45 |
| replaceafill | __parent__ still sends me to the course | 21:45 |
| replaceafill | i'll pdb it | 21:46 |
| replaceafill | thanks aelkner | 21:46 |
| aelkner | sure, i hope we are allowed to reproxy | 21:46 |
| aelkner | or relocate | 21:46 |
| aelkner | good luck pdbing :) | 21:46 |
| replaceafill | thanks | 21:46 |
| replaceafill | see you later | 21:46 |
| replaceafill | i'm home now btw ;) | 21:46 |
| aelkner | excellent! | 21:47 |
| aelkner | i'll look for your notification of when you did the merges | 21:47 |
| replaceafill | sure, i'll just fix this and will merge | 21:47 |
| aelkner | cool | 21:47 |
| aelkner | replaceafill, btw, another thing i thought of, a risk i think | 21:49 |
| aelkner | if two courses have a skillset in common, then the all_worksheets would serve the same sheet up twice | 21:50 |
| aelkner | what's more, since the sheet has __name__ that matches the __name__ of the skillset in the skillset container | 21:50 |
| aelkner | having two section worksheets that come from two different courses but are for the same skillset | 21:51 |
| aelkner | would lead to an ambiguous definition of what course_worksheet['foo'] would be | 21:51 |
| aelkner | if there are two sheets that have the same __name__, 'foo' | 21:52 |
| aelkner | sorry, course_worksheet['foo'], for that i meant to say section_skillsets['foo'] | 21:52 |
| aelkner | but do you see my point? | 21:53 |
| aelkner | replaceafill? | 21:55 |
| aelkner | i'm thinking that all_worksheets need to check if sheet.__name__ in self.keys() | 21:56 |
| aelkner | or something like that | 21:57 |
| replaceafill | yes | 21:57 |
| replaceafill | i added it | 21:57 |
| replaceafill | def keys(self): |from schooltool.cando.interfaces import ICourseSkills, ICourseSkillSet | 21:57 |
| replaceafill | return [i.__name__ for i in self.values()] | 21:57 |
| replaceafill | oops | 21:57 |
| replaceafill | sorry | 21:57 |
| replaceafill | def keys(self): | 21:57 |
| replaceafill | return [i.__name__ for i in self.values()] | 21:58 |
| replaceafill | aelkner, but that would happen if the section has many courses, right? | 21:59 |
| aelkner | only then, but we need to handle it | 22:01 |
| replaceafill | yes | 22:01 |
| aelkner | that can wait for after the demo as well | 22:01 |
| replaceafill | good point though | 22:01 |
| *** th1a has joined #schooltool | 22:10 | |
| *** menesis has joined #schooltool | 22:13 | |
| th1a | aelkner? replaceafill? | 23:01 |
| replaceafill | i'm here | 23:01 |
| replaceafill | i'm putting the mode buttons in the gradebooks | 23:01 |
| replaceafill | about to finish to show you | 23:01 |
| th1a | OK. | 23:02 |
| aelkner | here | 23:02 |
| replaceafill | i think we have an error in the assign.html view for skills | 23:14 |
| aelkner | yeah, i've been struggling with it myself as i check my import/export | 23:16 |
| replaceafill | in the button, right? | 23:17 |
| replaceafill | the add all displayed button | 23:17 |
| th1a | assign.html? | 23:17 |
| replaceafill | yes | 23:17 |
| th1a | Where is that? | 23:17 |
| aelkner | go to a course | 23:17 |
| replaceafill | go to a course -> skills | 23:17 |
| replaceafill | :) | 23:17 |
| aelkner | hit the Skills link | 23:17 |
| aelkner | then Assign Skillset | 23:17 |
| th1a | How long has that been there? Before today? | 23:18 |
| aelkner | oh, yes, yvl did all of that stuff | 23:18 |
| aelkner | discussions he had with replaceafill to get the gradebook changes to demo | 23:18 |
| th1a | Oh yeah. | 23:18 |
| th1a | Oh, so we're in much better shape than I thought! | 23:18 |
| th1a | I'd forgotten that worked at all. | 23:19 |
| th1a | (that's why we needed to review everything now...) | 23:19 |
| aelkner | it was a hack, but it's moments away from being a feature :) | 23:19 |
| replaceafill | :D | 23:19 |
| replaceafill | ok | 23:19 |
| replaceafill | th1a, aelkner http://69.164.203.135:6662 | 23:19 |
| replaceafill | teacher001 | 23:20 |
| th1a | So we can do the demo without the course/skillset importer. | 23:20 |
| replaceafill | click the CanDo tab | 23:20 |
| replaceafill | which takes you by default to the projects gradebook | 23:20 |
| replaceafill | (that's one question) | 23:20 |
| replaceafill | there's also a 'XXX Mode XXX' at the top of the sidebar | 23:20 |
| aelkner | th1a, you can answer replaceafill questions about the view, but i just wanted to say, course skillsets import/export any minute | 23:22 |
| th1a | The radio buttons don't actually work yet? | 23:22 |
| replaceafill | it does | 23:22 |
| replaceafill | reload your file resources | 23:22 |
| replaceafill | or... | 23:22 |
| * replaceafill goes try chrome :) | 23:22 | |
| th1a | Yeah, the radio button itself isn't working. | 23:23 |
| th1a | Is everything merged to trunk now or whatever? | 23:23 |
| replaceafill | i just need to get rid of the XXX part | 23:23 |
| replaceafill | is Mode ok? | 23:24 |
| th1a | Maybe just call it "View" | 23:24 |
| replaceafill | oopsy | 23:24 |
| replaceafill | onfocus="..." :D | 23:25 |
| replaceafill | ok, changing to "View" | 23:25 |
| replaceafill | th1a, the radio buttons work now | 23:26 |
| replaceafill | and i changed to "View" | 23:27 |
| replaceafill | i think i messed up my skills importing | 23:27 |
| * replaceafill goes to see | 23:27 | |
| th1a | I think that might be best as a plain old radio button. Hard to be confused by it. | 23:27 |
| th1a | I want to work some on that label box tomorrow if possible. | 23:28 |
| th1a | I still don't really want it there. | 23:28 |
| replaceafill | sure | 23:28 |
| replaceafill | th1a, can you explain the "plain old radio button" part a little more please? | 23:29 |
| th1a | I mean, as opposed to trying to come up with something fancier. | 23:29 |
| th1a | (I don't even know what) | 23:30 |
| replaceafill | ah, you mean you like it the current way? | 23:30 |
| th1a | Yes. | 23:30 |
| replaceafill | i thought you wanted some change | 23:30 |
| replaceafill | ok | 23:30 |
| replaceafill | aelkner, do you mind if i clean this a little bit before merging to trunk? | 23:30 |
| aelkner | replaceafill, just pushed CourseSkills import/export to trunk | 23:30 |
| th1a | At this point I think the assumption tends to be something fancier than a plain old HTML widget is the assumption. | 23:31 |
| aelkner | replaceafill, please do | 23:31 |
| replaceafill | i can trunk after merging too | 23:31 |
| replaceafill | aelkner, cool, thanks | 23:31 |
| * replaceafill really wants to add a couple of tests for the mode options | 23:31 | |
| replaceafill | at least that the buttons take you to the right worksheets | 23:31 |
| th1a | That would be prudent. | 23:33 |
| th1a | OK, we good? Can I go light the grill? | 23:33 |
| replaceafill | :D | 23:33 |
| replaceafill | aelkner, thanks for the help with the proxy | 23:33 |
| * th1a goes to light the grill | 23:33 | |
| replaceafill | i had misconfigured the registration :) | 23:33 |
| th1a | Thanks guys. | 23:33 |
| aelkner | replaceafill, the proxy works ok? | 23:33 |
| replaceafill | now i'll get rid of one gradebook class | 23:33 |
| replaceafill | yes | 23:33 |
| aelkner | th1a, can we meet later? | 23:34 |
| th1a | Umm... not really. Why? | 23:34 |
| aelkner | ok, 2:00 tomorrow then | 23:34 |
| th1a | OK. See you then. | 23:34 |
| aelkner | happy grilling :) | 23:34 |
| replaceafill | aelkner, http://bazaar.launchpad.net/~replaceafill/schooltool/cando/revision/16/src/schooltool/cando/course.py | 23:35 |
| replaceafill | line 167 | 23:35 |
| aelkner | nice | 23:36 |
| aelkner | when you merge, do me a favor | 23:36 |
| replaceafill | yes? | 23:36 |
| aelkner | oh, never mind, i'm just paranoid | 23:37 |
| replaceafill | :| | 23:37 |
| replaceafill | ok | 23:38 |
| aelkner | ok, i'm gong to say it | 23:38 |
| * replaceafill goes to get lunch, will continue cleaning after that | 23:38 | |
| replaceafill | yes? | 23:38 |
| aelkner | please just commit the change and don't overwrite the turnk | 23:38 |
| aelkner | i know that it might have only been yvl and menesis who have done that, but it really annoys me | 23:39 |
| aelkner | so sorry if i'm stating the obvious | 23:39 |
| replaceafill | i don't understand you | 23:39 |
| replaceafill | don't overwrite the trunk? | 23:39 |
| aelkner | yeah, just add a revision to it | 23:40 |
| replaceafill | opposed to...? | 23:40 |
| replaceafill | ah | 23:40 |
| replaceafill | i think i know what you mean | 23:40 |
| aelkner | as opposed to merging trunk to your branch and forcing yours into trunk | 23:40 |
| replaceafill | when the history gets hidden | 23:40 |
| aelkner | right, you remember now | 23:40 |
| aelkner | yes | 23:40 |
| replaceafill | got it | 23:40 |
| replaceafill | sure | 23:40 |
| aelkner | thanks | 23:40 |
| replaceafill | i hate that too :D | 23:40 |
| aelkner | :) | 23:40 |
| replaceafill | i think i haven't done it in the past (sorry if i have) | 23:41 |
| replaceafill | i won't | 23:41 |
| aelkner | np | 23:41 |
| aelkner | btw, what's the eta on your trunk commit, this evening? | 23:41 |
| replaceafill | for sure | 23:43 |
| aelkner | ok, i'll be around, so ping me | 23:43 |
| replaceafill | other than getting rid of hacks i think i have all the functionality in | 23:43 |
| replaceafill | kk, will do | 23:43 |
| aelkner | we can look over the newly fully integrated parts together | 23:43 |
| replaceafill | sure | 23:44 |
| replaceafill | i'll go get lunch now | 23:44 |
| replaceafill | see you later aelkner | 23:44 |
| aelkner | cya later | 23:44 |
Generated by irclog2html.py 2.15.1 by Marius Gedminas - find it at mg.pov.lt!