IRC log of #schooltool for Thursday, 2011-04-07

aelknerfsufitch, what time do you knock off today?00:28
fsufitcharound 7 probably00:28
fsufitchmaking good progress, but im taking it in little steps to make sure i don't break anything00:29
aelknercould you send me your diff?00:29
fsufitchi'm in the middle of writing ISchoolYearContainer and its adapter, i'll send it to you once i do that00:30
fsufitcher00:30
fsufitchICourseInfoSchoolYearContainer00:30
aelknercool00:31
*** alga has quit IRC00:38
fsufitchaelkner: ping01:13
fsufitchthe diff is here http://pastebin.com/0UpsrXK801:13
aelknerlooking...01:13
aelknerdid you ever get the free laptop shaky box when visiting that site?01:14
aelknerit took me to something that looks like a phishing site01:15
fsufitchwhat?01:15
fsufitchi have adblock01:16
fsufitchi dont see that stuff01:16
aelknerdid you bzr add adapters.py?01:17
aelknerbecause i don't see it in the diff01:17
fsufitchyes01:17
fsufitchhm.01:17
aelkneryou can justbzr diff > foo and email me the conents of foo01:18
aelknerand adapters.py would be included in that01:18
fsufitchhttp://pastebin.com/4bcrs2ym01:19
fsufitchthat's what i did01:19
fsufitchanyway there's the adapters.py01:19
fsufitchoh i didnt bzr add it properly01:19
fsufitchoops01:19
aelknerroot = getCourseInfoRoot(ISchoolToolApplication(None))01:20
aelkneryou could still use the adapter even thought the method is right there01:21
fsufitchtrue01:22
fsufitchhang on i broke tests, i just realized01:22
fsufitchi needed to clear the Data.fs first01:22
aelkneryou don't need to use DAta.fs01:23
aelknerremember you're doing bin/test -u01:23
fsufitchnono i mean i hadn't run bin/test at first :)01:23
fsufitchi just checked it in the browser01:23
fsufitchi ran bin/test as you were readin gthe code01:23
aelknerok, so you know what01:24
aelkneryou cold make it even cleaner, smaller methods, if you create ICourseInfoSchoolYearContaner(root)01:24
aelknerlet it do the logic in the first half01:25
fsufitchhmm ok01:25
fsufitchi only have 30 minutes left though01:25
aelknerok, do you have passing tests?01:26
fsufitchno, the tests are testing that the subscriber thing works01:26
fsufitchwhich of course doesnt as there is no more subscriber01:26
aelkneryeah, get rid of those tests then01:27
fsufitchugh, none of the unit tests use the adapter structure01:28
aelkneryou should be changing README.txt to work with the changes to the implementation01:28
fsufitchthey need to be rewritten01:28
aelknerthat's right01:28
aelknercommits will almost always have changes to tests01:28
aelknerunless you just re-implement something to work exactly the same with existing tests01:29
aelknerbut here you are changing the data model, adapters available, etc.01:30
aelknerso yes, tests will have to change01:30
fsufitchright01:31
fsufitchim just changing the tests now01:31
*** replaceafill has quit IRC01:37
fsufitchaelkner: tests pass01:38
aelknergreat, could you sne dme the diff via email rather than paste?01:39
fsufitchok01:39
fsufitchsent01:40
aelknerok, before you commit, we have time for small canges01:42
aelknerschoolyear = app['schooltool.schoolyear']['2009-2010']01:42
aelkneryou can use ISchoolYearContainer(app), in fact we should always avoid using app['something']01:43
aelknerexcept app['person'] until we have IPersonContainer(app)01:43
aelkneralso, you could move from schooltool.courseinfo.adapters import getCourseInfoRoot01:44
aelknerand provideAdapter(getCourseInfoSchoolYear)01:44
aelknerto test_courseinfo.py in setup()01:45
fsufitchyeah, i have yet to do that01:45
aelknerit's looking nice, though01:45
aelknerand i agree that you can add more adapters next week01:45
fsufitchadapterss!!01:46
aelknerliking them?01:48
fsufitchheh yeah01:48
fsufitchbut for the ISchoolYearContainer(app), how do i define/register the adapter?01:48
aelknergood question01:48
aelknerlook at intervetnion tests, i thnk01:49
fsufitchok01:49
fsufitchoh... it just works? huh.01:50
aelkneryou found it?01:51
aelknertests/tests.py01:51
fsufitchoh there it is01:51
aelknerprovideAdapter(getSchoolYearContainer)01:51
aelknerbut yes, that adapter just works01:51
aelknerand it uses the descriptors, too, so it's that easy to register/provide01:52
fsufitchsweet01:52
aelknerso setup() is the place to register any adpater like that that you find you need01:52
aelknerjust provide them as you need them01:53
fsufitchyup01:55
aelknerso, are you down to zero provideAdapter calls in README.txt?01:55
fsufitchyup, now i am01:56
fsufitchand tests still pass01:56
fsufitchalso no more app[]01:56
aelknerexcellent01:56
aelkneryou can commit and push now01:57
fsufitchcommitted and pushed01:57
fsufitchwoo!01:57
aelknersweet01:57
aelknerwe can pick up where we left off next week01:57
aelknergood job01:58
fsufitch:D01:59
aelknerfsufitch, so that's five hours02:01
aelknerare you doing another five hours before next wed?02:01
fsufitchill try, but no guarantees, i only have wednesdays as the schooltool-allocated time02:03
fsufitchanyway, i have to go02:04
fsufitchsee you around!02:04
aelknercya next week02:05
*** fsufitch has left #schooltool02:06
*** jelkner has quit IRC02:25
*** replaceafill has joined #schooltool04:17
*** replaceafill has joined #schooltool04:17
*** fsufitch has joined #schooltool05:51
*** replaceafill has quit IRC06:35
*** replaceafill has joined #schooltool07:04
*** aks has joined #schooltool07:21
*** th1a has quit IRC07:47
*** aks has quit IRC08:17
*** alga has joined #schooltool08:36
*** alga has quit IRC08:43
replaceafillyvl u around?09:33
yvlhey replaceafill09:33
replaceafilli have a question09:33
replaceafilli just realized that i'm making cambodia depend a lot on custom demographics09:33
replaceafillbut custom demographics are global for the user09:33
replaceafilli mean, they have no info about the schoolyear09:34
replaceafillright?09:34
replaceafillfor instance i wont be able to say "yvl's salary scale in 2011 was A.I"09:35
yvlyou're right09:35
replaceafilldo you think i should better create custom objects for these related to the schoolyear?09:35
replaceafilllike im doing with shifts (based on levels) etc09:36
yvlit is your choice in the end,09:36
replaceafill:/09:36
yvlbut if you need per-schoolyear data, then custom objects it is09:36
yvlsorry, but we don't have decent API for that yet09:37
yvlso it's gonna be a lot of copy-paste09:37
replaceafillright09:37
replaceafillwell, i'll talk to th1a about it09:38
yvlsure09:38
replaceafillthanks yvl09:39
yvlthis is actually a data model issue09:39
yvlso you need to speak with the customers about this09:39
yvland I promise I'll improve API in a month or two09:39
yvlsomething along the lines of multi-object annotations09:40
replaceafillright, that's another thought of mine, "we can fix it later" :(09:40
yvl:/09:40
replaceafill:D09:40
yvlthat's life :/09:41
yvlanyway, at least teh code will be consistent09:41
yvlcreate a per-year container on app startup, separate items per-int-id.... gah.09:42
replaceafillright09:42
replaceafilllike courses, levels, groups, etc etc etc :P09:42
yvl... :)09:47
*** alga has joined #schooltool10:02
*** yvl has quit IRC10:02
*** yvl has joined #schooltool10:06
*** replaceafill has quit IRC10:38
*** yvl has quit IRC10:47
*** menesis has joined #schooltool11:37
*** yvl has joined #schooltool12:01
*** menesis has quit IRC12:46
*** alga has quit IRC13:45
*** yvl has quit IRC13:46
*** alga has joined #schooltool14:07
*** menesis has joined #schooltool14:34
*** yvl has joined #schooltool15:07
*** yvl has quit IRC15:53
*** menesis has quit IRC15:53
*** Lumiere has quit IRC15:58
*** menesis has joined #schooltool16:02
*** Lumiere has joined #schooltool16:03
*** th1a has joined #schooltool17:17
*** replaceafill has joined #schooltool17:45
*** ignas has joined #schooltool19:07
*** alga has quit IRC19:11
*** menesis has quit IRC19:36
*** ignas has quit IRC20:43
*** menesis has joined #schooltool20:49
*** alga has joined #schooltool22:09
*** replaceafill has quit IRC23:23
*** alga has quit IRC23:27
*** alga has joined #schooltool23:38

Generated by irclog2html.py 2.15.1 by Marius Gedminas - find it at mg.pov.lt!