| *** menesis has quit IRC | 00:21 | |
| *** yvl has joined #schooltool | 09:50 | |
| *** menesis has joined #schooltool | 09:58 | |
| *** menesis has quit IRC | 12:41 | |
| *** yvl has quit IRC | 13:15 | |
| *** yvl has joined #schooltool | 13:15 | |
| *** menesis has joined #schooltool | 13:54 | |
| *** ignas__ has joined #schooltool | 14:12 | |
| *** pgulley has joined #schooltool | 15:12 | |
| *** pgulley has quit IRC | 15:14 | |
| *** th1a has joined #schooltool | 16:08 | |
| *** pgulley has joined #schooltool | 16:10 | |
| *** pgulley has quit IRC | 16:12 | |
| *** replaceafill has joined #schooltool | 16:23 | |
| th1a | hi replaceafill, aelkner, menesis, yvl. | 16:30 |
|---|---|---|
| aelkner | morning | 16:30 |
| menesis | hi | 16:30 |
| replaceafill | good morning/afternoon | 16:31 |
| *** pgulley has joined #schooltool | 16:31 | |
| yvl | good morning | 16:32 |
| th1a | hi yvl. Would you like to start us off? What's the timeline for long running processes at this point? | 16:33 |
| yvl | I'm not sure | 16:33 |
| th1a | ... | 16:33 |
| yvl | it's not bogged, but I don't know when it will be done | 16:33 |
| yvl | sorry | 16:33 |
| yvl | I was bogged last week though | 16:34 |
| yvl | and took the Thu off | 16:34 |
| yvl | btw, this Fri is also a bank holiday here | 16:34 |
| th1a | This Wednesday is here. | 16:35 |
| yvl | good to know, thanks | 16:35 |
| aelkner | th1a, are you taking off wed? | 16:35 |
| aelkner | funny that welsh said he was meeting with glenda then | 16:36 |
| th1a | Oh, we may have a meeting. Let's see what is going on. | 16:36 |
| yvl | so it feels like there's "a lot" to do, but not "*that* much" | 16:36 |
| th1a | Where are you now? | 16:36 |
| yvl | I think I got the Zope machinery starting up correctly in spawned processes when using process pool | 16:37 |
| th1a | Actually, what's the 50,000 foot view of how this is going to work? | 16:38 |
| yvl | mkay... | 16:38 |
| yvl | so ST is now split into services controlled by supervisor | 16:39 |
| yvl | we have schooltoo, zeo and celery | 16:39 |
| yvl | zeo provides us many connections to our db | 16:39 |
| yvl | oh, and we have redis DB for temporary task storage | 16:39 |
| yvl | schooltool (or some other app, if we want to) pushes the task to celery | 16:40 |
| yvl | that task gets routed around, queued and processed at some point | 16:41 |
| yvl | schooltool meanwhile stores the id of the task | 16:41 |
| yvl | and once in a while, if user is looking via web view, it, say, polls celery for the status of the task | 16:42 |
| th1a | What triggers the poll? | 16:42 |
| yvl | timer :) | 16:42 |
| yvl | or anything else | 16:43 |
| yvl | there are techniques | 16:43 |
| yvl | most likely - javascript timer | 16:43 |
| yvl | there are of course open questions on what for we actually use the long running processes | 16:44 |
| th1a | We can send tasks to celery from a cron job. | 16:44 |
| th1a | Right? | 16:44 |
| yvl | right | 16:44 |
| yvl | or we can use celery beat for that | 16:44 |
| yvl | it has "cron jobs" built in | 16:44 |
| th1a | OK. | 16:44 |
| yvl | if we allow scheduling an import to happen at night, but scheduled from the web | 16:45 |
| yvl | we'll need a place to show what happened and so on | 16:45 |
| th1a | So if we want to print a big report, what happens? | 16:45 |
| yvl | if we do "offline" large pdf generation, we'll need UI to show when PDF is generated | 16:45 |
| yvl | user may, and will, close the browsers | 16:45 |
| yvl | browser | 16:46 |
| th1a | Yes, but just walk me through here. | 16:46 |
| th1a | A task is sent to celery. | 16:46 |
| yvl | I imagine - but we'll need to develop that - | 16:46 |
| yvl | ok | 16:46 |
| yvl | so, a task is sent to celery, it generates a file, saves to ZODB Blob | 16:46 |
| yvl | and knows that it's finished | 16:46 |
| yvl | the user needs UI to find the download link for that PDF | 16:47 |
| th1a | Yes, but celery... | 16:47 |
| yvl | and the download link should expire at some point maybe? | 16:47 |
| yvl | yes? | 16:47 |
| th1a | Celery adds it to its task list... | 16:47 |
| yvl | when time comes, or immediately, it starts executing the task | 16:47 |
| *** ignas__ is now known as ignas | 16:47 | |
| th1a | And then probably (but not necessarily) starts on the task. | 16:48 |
| th1a | RIght. | 16:48 |
| th1a | OK, so it spawns a new process? | 16:48 |
| th1a | What kind of process? | 16:48 |
| yvl | it depend on a configuration, but let's say yes | 16:48 |
| yvl | keep in mind, that here you are asking me | 16:48 |
| yvl | how precisely I am going to configure a service | 16:48 |
| yvl | (and that's usually a sysadmins job, and they might and will change that) | 16:49 |
| yvl | (at will) | 16:49 |
| yvl | but yes, it has a pool of available processes, by default the same amount as CPUs on the server | 16:49 |
| yvl | and those workers handle the tasks one by one | 16:50 |
| yvl | and handling the task... | 16:50 |
| yvl | well, it connects to ZEO, reads what it wants from our DB | 16:50 |
| yvl | makes a PDF | 16:50 |
| yvl | same code as normal SchoolTool | 16:51 |
| yvl | if you wonder about memory consumption... it should use the same as normal schooltool threads | 16:52 |
| yvl | we have 17 of those I think | 16:52 |
| yvl | but then again, the memory should get freed after, say, the imports | 16:53 |
| th1a | OK. | 16:53 |
| th1a | I'm not particularly worried about it. I just should know what is going on. | 16:54 |
| th1a | So we'll probably want, say, a little printer icon that might appear in the gray bar with a number next to it when reports are ready. | 16:54 |
| yvl | right | 16:55 |
| yvl | but I think it's best to think about that when we actually have the code for long running processes working | 16:56 |
| yvl | easier to prototype | 16:56 |
| th1a | Yes, of course. | 16:56 |
| th1a | Do you have any vacation plans, yvl? | 16:58 |
| yvl | yes | 16:58 |
| yvl | the week of Aug 8th | 16:58 |
| yvl | or 6th | 16:59 |
| yvl | I don't remember precisely now :) | 16:59 |
| th1a | OK. | 16:59 |
| yvl | point being - not this month | 16:59 |
| th1a | As long as we're not imminently going to crash into it. | 16:59 |
| yvl | but I plan to take a few days off this month | 16:59 |
| yvl | sure | 17:00 |
| th1a | btw, I'm going to be at the parents/in-laws for a good part of this month, but mostly working. | 17:00 |
| yvl | ah, ok | 17:01 |
| th1a | Anything else, yvl? | 17:02 |
| yvl | no, that is it | 17:02 |
| yvl | interesting to work on this though ;) | 17:02 |
| th1a | As long as it isn't so interesting that you work on in forever. | 17:03 |
| th1a | ;-) | 17:03 |
| th1a | menesis: Anything to report? Philippines going ok? | 17:04 |
| yvl | :D | 17:04 |
| menesis | th1a: yes | 17:04 |
| menesis | yes about the Philippines | 17:04 |
| menesis | fixed several bugs with table sorting, broken objects appearing in catalogs, etc. | 17:05 |
| menesis | now have more features to work on | 17:06 |
| th1a | OK, so those are things we'd like to have in core? | 17:06 |
| th1a | Or, I mean, did you do those changes for us or them? | 17:06 |
| menesis | no, those were bugs in that plugin | 17:06 |
| th1a | OK. | 17:06 |
| menesis | some small things can go in core | 17:07 |
| menesis | or already are there | 17:08 |
| menesis | e.g. after removing a person its demographics were left in the db | 17:08 |
| menesis | now fixed | 17:08 |
| th1a | Don't mix in code they paid for until we have a signed agreement from them. | 17:08 |
| th1a | (which is kind of waiting for a clear list of things we want) | 17:08 |
| menesis | sure | 17:09 |
| th1a | OK. | 17:09 |
| menesis | I have been able to make everything we needed in the plugin | 17:09 |
| menesis | without changing core | 17:09 |
| menesis | but some things are workarounds for bugs in core | 17:09 |
| menesis | so I'm fixing them in core as well | 17:10 |
| th1a | OK. That's fine. | 17:10 |
| th1a | Anything else, menesis? | 17:12 |
| menesis | don't remember anything important | 17:12 |
| th1a | Thanks menesis. | 17:13 |
| th1a | aelkner? | 17:13 |
| aelkner | ok, so i got the three sections sheets we discussed to import and export | 17:13 |
| aelkner | i'm currently working on tests for them, including round-tripping | 17:14 |
| aelkner | i figured out how to take the result of a xls export download and upload it to the importer | 17:15 |
| th1a | Ah, good. | 17:15 |
| aelkner | and it proved that it wouldn't crash | 17:15 |
| th1a | Can we do tests for unexpected data types? | 17:15 |
| aelkner | ah, yes, thanks for reminding me, i already coded checking for them | 17:16 |
| aelkner | i'll make sure to include them in the tests | 17:16 |
| aelkner | one thing i didn't do yet, is the sheet that we discussed that wouldn't export, LinkedSectionImport | 17:16 |
| aelkner | do we definitely still want that sheet? | 17:17 |
| th1a | That's the one with start and end? | 17:18 |
| aelkner | i can still do it if you want, it just can't be exported | 17:18 |
| aelkner | yes | 17:18 |
| aelkner | and thus, can't be round-tripped | 17:18 |
| th1a | Yes, I think we should have it. | 17:18 |
| aelkner | ok, i have some useful code to the side, so that won't take long | 17:18 |
| aelkner | i think i want to merge what i already have to trunk so replaceafill can get | 17:19 |
| aelkner | the resource import bug fix into his demo instance | 17:19 |
| aelkner | the one that i did last week to put the descriptions in | 17:20 |
| aelkner | i just think it would look better in the demo to glenda to have the building descriptions | 17:20 |
| aelkner | i'll coordinate with replaceafill on getting that into his demo instnace | 17:20 |
| aelkner | anyway | 17:21 |
| aelkner | i also added code to schooltool.virginia to auto-create the demos on AppInit/Startup | 17:21 |
| aelkner | so replaceafill can pull that change to his demo instance, too | 17:21 |
| aelkner | one thing that happened as a result is that i can't get schooltool.export tests to pass | 17:22 |
| aelkner | if the buildout.cfg has schooltool.virginia in it | 17:22 |
| aelkner | because in adding the demos fields, it breaks the default schooltool setup of the demos | 17:23 |
| aelkner | and thus, the schooltool.export tests fail | 17:23 |
| aelkner | this is not a problem as long as one tests schooltool without schooltool.virginia in the buildout | 17:23 |
| aelkner | just thought i'd mention that | 17:24 |
| aelkner | anyway, i have a request of yvl, replaceafill | 17:24 |
| aelkner | can we change the buildout.cfg that is in trunk to have the selenium setup and other things | 17:24 |
| aelkner | that i currently have different in my own buildout.cfg? | 17:25 |
| aelkner | it's a pain to have to always bzr revert buildout.cfg in order to commit changes | 17:25 |
| replaceafill | i've been doing it for a long time | 17:25 |
| yvl | you can do "bzr commit src" you know | 17:26 |
| yvl | and bzr diff src | 17:26 |
| replaceafill | i don't think it's safe to set selenium in trunk's buildout from the buildbot POV? | 17:26 |
| aelkner | yeah, but i'd prefer not to have to do that | 17:26 |
| aelkner | ah, i see | 17:27 |
| yvl | maybe we can remove the buildout.cfg and add buildout.cfg.tmpl | 17:27 |
| aelkner | ok, never mind, i'll deal with it | 17:27 |
| yvl | and when running make just copy it | 17:27 |
| aelkner | oh, that would be nice | 17:27 |
| yvl | and set bzr to ignore buildout.cfg | 17:27 |
| replaceafill | i always make a buildout.cfg.bak file | 17:27 |
| replaceafill | revert before commiting | 17:27 |
| replaceafill | restore after committing | 17:27 |
| yvl | and set make realclean to remove buildout.cfg | 17:28 |
| replaceafill | better make clean :) | 17:28 |
| replaceafill | i wouldn't want to lose my test db ;) | 17:28 |
| aelkner | my point was i was hoping to not have to do separate steps, that it would be automatic | 17:29 |
| yvl | see http://www.cs.utah.edu/dept/old/texinfo/make/make.html | 17:29 |
| aelkner | but that seems like a pipe dream, so i guess i will do wthout it | 17:29 |
| yvl | I think realclean is better | 17:29 |
| yvl | aelkner, yes, that's what we're discussing | 17:29 |
| yvl | automatic | 17:29 |
| aelkner | oh, ok, sorry, didn't understand what you were saying | 17:30 |
| aelkner | but yes, i wouldn't want the db to be wiped out as part of the auto process | 17:30 |
| aelkner | i tend to keep a db around wh all the packages loaded in it | 17:30 |
| yvl | it's more that... buildout.cfg would never be refreshed | 17:30 |
| yvl | unless you delete it | 17:30 |
| yvl | so make clean it is | 17:31 |
| aelkner | and bzr ignored, right? | 17:31 |
| replaceafill | :) | 17:31 |
| yvl | menesis, opinions? | 17:31 |
| yvl | yes, aelkner | 17:31 |
| yvl | (I have to think about it is an opinion also ;) ) | 17:31 |
| menesis | yes | 17:31 |
| menesis | reading.. | 17:32 |
| yvl | thanks | 17:32 |
| menesis | I have done something about buildout.cfg | 17:32 |
| menesis | split it into two files: run.cfg and develop.cfg | 17:33 |
| menesis | on 'make' if buildout.cfg does not exist, run.cfg is copied to buildout.cfg | 17:33 |
| menesis | buildout.cfg is ignored so whatever you change in it, does not show up in bzr diff | 17:33 |
| aelkner | that sounds great, can we get that into trunk? | 17:34 |
| menesis | to enable all the developer tools, run 'make develop' | 17:34 |
| menesis | that will replace the line in buildout.cfg from 'extends = run.cfg' to 'extends = develop.cfg' | 17:35 |
| menesis | that will make omelette, coverage, selenium, ipdb, zodbbrowser, etc. available | 17:36 |
| menesis | that are not needed just to run schooltool | 17:36 |
| yvl | thank you very much | 17:36 |
| menesis | I had this working, but not committed anywhere | 17:37 |
| menesis | I can bring it back from the shelf and commit soon | 17:37 |
| yvl | yes please | 17:37 |
| aelkner | thanks menesis | 17:37 |
| replaceafill | +1 menesis :) | 17:38 |
| th1a | OK, anything else aelkner? | 17:39 |
| aelkner | no, that's it for me, other than preparing for the glenda demo | 17:40 |
| aelkner | do we have a meeting planned with welsh? | 17:40 |
| th1a | When is the glenda demo? | 17:41 |
| aelkner | when welsh meets with her, this wed i think, she will hopefully be very impressed | 17:41 |
| aelkner | and that would mean more money down the line if we're lucky | 17:42 |
| th1a | Well, it doesn't actually mean more or less money. | 17:42 |
| th1a | But yes, it would be good if she's happy. | 17:43 |
| aelkner | anyway, that's it for me | 17:43 |
| th1a | Can we shoot for having the import/export and tests done Wednesday? | 17:44 |
| aelkner | yes, that sounds right | 17:44 |
| th1a | OK. We need to put this behind us. | 17:44 |
| aelkner | so are we meeting wed? | 17:45 |
| th1a | I'd say we need to. | 17:45 |
| aelkner | ok, i'll be available | 17:45 |
| th1a | Thanks aelkner. | 17:45 |
| th1a | replaceafill? | 17:45 |
| replaceafill | ok | 17:45 |
| replaceafill | can you log in as manager at: | 17:46 |
| replaceafill | http://69.164.203.135:6660/schoolyears/2012-2013/assign-courses-skills.html | 17:46 |
| replaceafill | so, the attributes that work right now | 17:47 |
| replaceafill | Alternate ID for courses | 17:47 |
| replaceafill | Course for layer | 17:47 |
| replaceafill | and Description for node attributes | 17:47 |
| replaceafill | after clicking search with those values you should see the skillsets | 17:47 |
| replaceafill | to match | 17:47 |
| th1a | We should have a cancel button. | 17:48 |
| replaceafill | ah! true :) | 17:48 |
| replaceafill | i'll add one | 17:48 |
| th1a | And we can add some hints to the form after the meeting. | 17:48 |
| replaceafill | sure | 17:48 |
| replaceafill | i have a couple of questions for yvl | 17:48 |
| yvl | shoot | 17:48 |
| replaceafill | but maybe he's out of gas? | 17:48 |
| yvl | no harm in asking ;) | 17:49 |
| replaceafill | :) | 17:49 |
| replaceafill | ok | 17:49 |
| replaceafill | is there any way to have one single skill in a skillset related to a course? | 17:49 |
| replaceafill | i guess not, right? | 17:49 |
| replaceafill | since you can only use skillsets in those relationships | 17:49 |
| yvl | wow, I really don't remember anymore | 17:50 |
| replaceafill | :) | 17:50 |
| yvl | would need to look at the code | 17:50 |
| yvl | but one skill in skillset does not sound right | 17:50 |
| replaceafill | dwelsh has come up with the "core" and "additional" concepts | 17:51 |
| replaceafill | and sometimes you just want to select a few, but not all skills in a skill set | 17:51 |
| aelkner | can't we give him a worksheet to put those skills in? | 17:51 |
| th1a | You can add one skill from a skillset to a project, right? | 17:51 |
| replaceafill | th1a, we will (after i fix that view) | 17:52 |
| yvl | didn't we discuss that this winter? | 17:52 |
| replaceafill | but i don't think that's what dwelsh wants | 17:52 |
| replaceafill | he wants to see the "additional" ones in the skills gradebook | 17:52 |
| yvl | will it kill him to see the additional ones in projects gradebook? | 17:53 |
| * yvl is not objecting, just asking | 17:53 | |
| replaceafill | i can show him that i think | 17:53 |
| yvl | because that was kind of the concept | 17:53 |
| replaceafill | i know | 17:53 |
| yvl | the skills is something that is, well, almost enforced by somebody | 17:53 |
| yvl | the projects is something that is, well, additional :) | 17:54 |
| replaceafill | :D | 17:54 |
| yvl | and teacher has more control there | 17:54 |
| th1a | I would have preferred if the relationship was between individual skills and courses all along. | 17:55 |
| yvl | ? | 17:55 |
| * yvl definitely has been out of the game for a while | 17:55 | |
| th1a | I never had any particular interest in the concept of skillsets. | 17:56 |
| th1a | It is ok though. | 17:56 |
| th1a | So... is this something replaceafill should just add to course/skill relationships or should we punt on it for now? | 17:57 |
| replaceafill | th1a, i think i should fix the "add skill" view in projects and show that to dwelsh | 17:58 |
| replaceafill | and explain him this ^ | 17:58 |
| aelkner | not to throw a wrench into the works, but i can guess he'll be disappointed | 17:58 |
| th1a | Can we connect projects to courses (that is, add that feature?) | 17:58 |
| aelkner | i think he wants the teachers to get these things automatically | 17:58 |
| aelkner | without having to think about projects | 17:58 |
| aelkner | his goal is to set everything up as an administrator | 17:59 |
| aelkner | and just have the teachers show up and see everything in their gradebooks | 17:59 |
| aelkner | not for them to have to create projects or anything | 17:59 |
| yvl | technically you can duplicate a skillset, add an additional skill to it, and assign it to the course | 17:59 |
| th1a | I'm not necessarily against it, but perhaps we should start writing reports first. | 18:00 |
| replaceafill | i could try the "duplicate the skillset" story | 18:01 |
| *** pgulley has quit IRC | 18:02 | |
| th1a | hm? | 18:02 |
| replaceafill | yvl's last message | 18:03 |
| replaceafill | "technically you can duplicate a skillset, add an additional skill to it, and assign it to the course" | 18:03 |
| th1a | Oh. | 18:04 |
| th1a | Why can't you just associate a skill with a course? | 18:04 |
| th1a | It doesn't seem like it would be very hard to add that. | 18:05 |
| yvl | kind of intentionally | 18:05 |
| yvl | think about the gradebook | 18:05 |
| replaceafill | imho that would break the whole model | 18:05 |
| yvl | it has to display skills in *some* worksheets | 18:05 |
| yvl | as in somewhere | 18:05 |
| yvl | so that somewhere is a "skillset" | 18:05 |
| yvl | or a worksheet in normal gradebook | 18:05 |
| aelkner | right | 18:05 |
| aelkner | it could help to get welsh to be more specific about how he would want it to work | 18:06 |
| aelkner | like, what the ui for adding these additional comps would be | 18:06 |
| aelkner | perhaps in so doing, he might change his mind and decide that projects is the way to go? | 18:06 |
| yvl | and I think it would help us if we knew actual live examples where Welsh wants that | 18:07 |
| th1a | OK, so why not just have an "additional" skillset defined for each course? | 18:07 |
| yvl | I think he should know those by heart | 18:07 |
| aelkner | th1a, that may be it | 18:07 |
| yvl | bery doable th1a | 18:08 |
| yvl | very | 18:08 |
| yvl | :) | 18:08 |
| aelkner | bery, bery indeed :) | 18:08 |
| th1a | OK. Try that route. | 18:08 |
| replaceafill | ok, where does that "additional" skillset live? | 18:08 |
| * yvl shrugs. In the skillset container, "in the wild" if that makes sense | 18:09 | |
| yvl | out of document model probably | 18:09 |
| replaceafill | courses annotations? | 18:09 |
| th1a | Sure -- definitely not in the document model. | 18:09 |
| yvl | nah, global skillset container is fine | 18:09 |
| yvl | unless it clutters the searches or something | 18:10 |
| replaceafill | that's what i was thinking | 18:10 |
| aelkner | yeah, that's the only question, how to search, assign | 18:10 |
| replaceafill | every additional skillset would show up there | 18:10 |
| replaceafill | in /skills | 18:10 |
| yvl | is this a bad thing, if the skillset is named somehow properly? Like math - A - additional or something? | 18:11 |
| yvl | if it's a bad thing, you can probably put those in annotations | 18:11 |
| replaceafill | right | 18:11 |
| yvl | should additional skills always be there? | 18:12 |
| yvl | or created on demand? | 18:12 |
| yvl | is this a common case, or it's one of those best handled by importer? | 18:12 |
| yvl | as in - do we need explicit UI maybe? | 18:12 |
| replaceafill | yvl, i think all dwelsh wants is the "old cando" way | 18:13 |
| yvl | yes | 18:13 |
| yvl | I'm pretty sure about that | 18:13 |
| replaceafill | the ui for adding additional skills was the same as adding "regular" skills | 18:13 |
| yvl | but if those are handled by imports by admins... ;) | 18:13 |
| yvl | so I guess the final call is to be made by th1a, as usual | 18:14 |
| yvl | after some heated discussions with David :) | 18:14 |
| th1a | Oh... | 18:14 |
| th1a | I guess we'll need another importer. | 18:14 |
| yvl | it's the question of "how common of the usecase is this" | 18:14 |
| yvl | maybe it can even be handled by the current one? | 18:15 |
| th1a | It could be fairly common. | 18:15 |
| replaceafill | we don't have importers for assigning skillsets to courses...? | 18:15 |
| replaceafill | right? | 18:15 |
| th1a | In terms of UI, don't we just need a "Add Additional Skills" action in courses? | 18:16 |
| th1a | We do have a skillset/course importer, iirc. | 18:16 |
| replaceafill | :| | 18:16 |
| aelkner | replaceafill, we do have an iporter for assigning skillsets to courses | 18:17 |
| aelkner | CourseSkills | 18:17 |
| replaceafill | what sheet is that? | 18:17 |
| aelkner | look in cando's importer for CourseSkillsImporter | 18:18 |
| replaceafill | do we have a CourseSkills sample xls? | 18:19 |
| replaceafill | i mean for dwelsh? | 18:19 |
| aelkner | i have on, but i'm not sure what it matches | 18:20 |
| replaceafill | could you send me that one please? | 18:21 |
| aelkner | scratch that, it matches the live data, not the autoism demo | 18:21 |
| aelkner | i can send that to you, but it only has two courses in it | 18:21 |
| aelkner | really, what we should do is change the get_verso.py to generate it | 18:21 |
| replaceafill | that's ok, i just want a sample one | 18:21 |
| aelkner | i'll send it to you | 18:22 |
| replaceafill | thanks aelkner | 18:22 |
| replaceafill | ok, i'll work on the additional skillset | 18:22 |
| replaceafill | th1a, that's it from me | 18:22 |
| aelkner | email sent | 18:22 |
| replaceafill | thanks | 18:23 |
| replaceafill | btw | 18:23 |
| replaceafill | we should put these sample xls in a single place :) | 18:23 |
| replaceafill | maybe bzr is not the best | 18:23 |
| th1a | All right. Thanks guys. | 18:23 |
| replaceafill | but the vps | 18:23 |
| * replaceafill done | 18:23 | |
| th1a | OK, so we'll meet Wednesday. | 18:24 |
| aelkner | replaceafill, yes, you and i need to coordinate that | 18:24 |
| th1a | Oh, send me your invoices pls. | 18:24 |
| th1a | Have a good one. | 18:24 |
| * th1a drops the bag of gravel. | 18:24 | |
| aelkner | thanks guys, cya wed | 18:24 |
| yvl | good luck guys! | 18:24 |
| replaceafill | imho CourseSkills should be CourseNodes :) | 18:25 |
| replaceafill | the match should be with nodes, not skillsets | 18:25 |
| th1a | Hm? | 18:25 |
| replaceafill | i mean, the same way my view works | 18:26 |
| replaceafill | but maybe it's not the same use case | 18:26 |
| aelkner | yeah, imports tend to need to be precise | 18:26 |
| aelkner | and flexible, too | 18:26 |
| aelkner | we are, in the data model, matching courses to skillsets | 18:26 |
| replaceafill | yeah, but nodes have skillsets too | 18:27 |
| * yvl is running a commandline script to asynchronously set school name :) | 18:27 | |
| aelkner | so the importer, and at some point, and exporter, needs to reflects that | 18:27 |
| yvl | proof-of-concept | 18:27 |
| aelkner | yvl, sounds cool | 18:27 |
| replaceafill | aelkner, i think ST courses should ask nodes about skillsets | 18:28 |
| replaceafill | that's all | 18:28 |
| aelkner | yeah, but you didn't say anything when yvl built the data model | 18:28 |
| aelkner | that would have been the time to raise that point | 18:29 |
| aelkner | and i'm not sure it would be best anyway | 18:29 |
| replaceafill | aelkner, come on, THAT'S IN THE DATA MODEL | 18:29 |
| th1a | It is a data model issue. | 18:29 |
| aelkner | assigning nodes to courses?! | 18:29 |
| replaceafill | that's the way the "assign-courses-skills.html" view works | 18:29 |
| replaceafill | nodes have skillsets | 18:29 |
| replaceafill | course nodes have the proper skillsets | 18:30 |
| replaceafill | the courseskills importer should ask the course nodes | 18:30 |
| aelkner | ah, you're not talking about a model change, just a process change | 18:32 |
| aelkner | yes, the importer could have a column for nodes rather than skillsets | 18:33 |
| replaceafill | ;) | 18:33 |
| aelkner | and the processor could find the nodes, get their skillsets and assign them | 18:33 |
| replaceafill | ;) | 18:33 |
| aelkner | but, then what do we do about additional skills? | 18:33 |
| aelkner | they need skillsets, too, not attached to the document nodes | 18:34 |
| aelkner | perhaps we could have two columns in the importer | 18:34 |
| replaceafill | i think so | 18:34 |
| replaceafill | but you see my point anyway | 18:34 |
| aelkner | the nodes, and additonal skillsets | 18:34 |
| replaceafill | not need to change that now | 18:34 |
| replaceafill | aelkner, something like that, yes | 18:34 |
| aelkner | i do see your point, so we could make a change of that sort at some point | 18:35 |
| aelkner | sorry, i jumped to the conclusion that you were suggesting a change to the model | 18:35 |
| replaceafill | no man, you hurt my feelings :''''( | 18:35 |
| replaceafill | (joking) :D | 18:35 |
| aelkner | :) | 18:35 |
| *** menesis has quit IRC | 19:28 | |
| *** menesis has joined #schooltool | 19:29 | |
| *** pgulley has joined #schooltool | 19:34 | |
| *** menesis has quit IRC | 19:46 | |
| *** paulproteus has quit IRC | 20:07 | |
| *** paulproteus has joined #schooltool | 20:10 | |
| *** ignas has quit IRC | 20:39 | |
| *** menesis has joined #schooltool | 20:44 | |
| *** pgulley has quit IRC | 21:01 | |
| *** menesis has quit IRC | 22:14 | |
| *** menesis has joined #schooltool | 22:23 | |
| *** menesis has quit IRC | 23:29 | |
| *** replaceafill has quit IRC | 23:30 | |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!