*** 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 2.15.1 by Marius Gedminas - find it at mg.pov.lt!