IRC log of #schooltool for Tuesday, 2011-09-20

*** replaceafill has quit IRC00:07
*** replaceafill has joined #schooltool01:13
*** replaceafill has quit IRC03:10
*** jelkner has joined #schooltool05:01
*** aks has joined #schooltool06:11
*** aks has joined #schooltool06:11
*** aks has quit IRC07:58
*** aks has joined #schooltool08:02
*** aks has quit IRC10:50
*** aks has joined #schooltool10:53
*** menesis has joined #schooltool11:41
*** aks has quit IRC14:30
*** yvl has quit IRC14:59
*** yvl has joined #schooltool15:13
*** yvl has quit IRC15:46
*** yvl has joined #schooltool16:00
*** replaceafill has joined #schooltool16:22
* th1a has his coffee...16:33
th1ahi menesis, yvl, aelkner, replaceafill.16:33
replaceafillgood morning/afternoon16:33
yvlgood morning16:33
th1aOK, let's start with aelkner.16:34
aelknerok, i finished up the contact import with adding a errant data test file for them16:34
aelknerthen i took over the contact export bug and coded that16:35
aelkneri have to to tests for that next16:35
aelkneri have a question about importing/exporting contacts that are for a person16:35
aelkneris it ok to have the rule i set up which is as follows:16:36
aelknerin Contact Persons, if the ID is a person d, then ignore the first five cells16:37
aelknerbecause they are redundant with the same five cells in the person object16:37
aelkneri mean, what is the right thing to do wth those cells, override the person record?!16:38
th1aMaybe it should raise an error if they aren't blank and match a perons ID.16:38
aelknerand for the export part, should they be blank or match the person record?16:39
th1aI'd say.16:40
aelknerblank makes sense for the export, but should it just be an error if there is anything onimport16:41
aelknereven if it matches the person record16:41
aelknerjust to keep it consistent?16:41
th1aI would say yes.16:42
th1aAnything else, aelkner?16:43
aelknerok, error if anything filled in16:44
aelknerok, that's it for me16:45
th1aBy error I don't literally mean "throw a traceback" of course.16:46
aelknerof course, just an cell error like anyting else16:47
th1aYes.  OK.16:47
th1aOK, yvl.  I see you've got a proof of concept going.16:48
yvlit's rough and the test will likely fail on some machines16:48
yvlbut at least the web drivers seme to work16:48
th1aPerhaps replaceafill should try it today.16:49
yvlI'd be grateful16:49
yvlreplaceafill, do you have Iceweasel installed?16:49
replaceafillyvl yes16:49
replaceafilli also have chrome16:49
yvlplease try that, Chromium and htmlunit16:49
yvlperfect :)16:49
replaceafill6.0.472.63 (59945) Debian 6.0.216:49
replaceafillkind of old...16:50
yvlwell, try it out anyway16:51
yvland please report errors ;)16:51
replaceafilli should follow src/schooltool/app/browser/stests/app.txt as an example, right?16:52
yvlyou can run just selenium tests with16:52
yvlbin/test --tests-pattern=stests16:52
replaceafillah ok16:52
yvl"over". ;)16:52
th1aThanks yvl.16:53
yvloh, and the "test api" is basically the selenium webdriver / webelement which is quite cumbersome.16:53
th1aShould I be adding the schooltool-owners PPA to my Oneiric system repositories?16:53
menesisnot yet16:54
menesisand I think I will put into schooltool-owners/dev ppa first16:55
th1aOK.  How is it coming?16:56
th1aWhat's blocking you?16:58
menesisfixed some small problems in strings.16:59
menesismade an intervention release from trunk.16:59
th1aYou have blanket permission to commandeer yvl at any point to help get this done.16:59
menesisjqueryui in schooltool is different layout than in ubuntu package.16:59
menesisbut maybe I'll leave it as is16:59
th1aI would not sweat jquery.17:00
th1aWe can tidy that up for 12.0417:00
menesisI need timetable ftests fixed17:01
menesisnow not only ftests fail, but they abort on teardown of first layer17:02
menesisif anyone could look into that17:02
yvlyou have my permission to kill them17:02
yvlthe tests :)17:02
yvlor I can kill them17:02
th1aIs it worth having replaceafill look at it or is it only comprehensible to yvl?17:03
replaceafilli did look at them last time17:03
replaceafilldidnt know how to fix them17:03
replaceafillsince so many things have changed17:03
yvlthey're not broken AFAIR, they're just completely out-of-date and written for old timetabling implementation17:03
yvlas in - they're pointless, just a reminder of what cases we were testing17:04
th1aPerhaps yvl should do the killing.17:04
th1aCan you make it quick and painless, yvl?17:05
th1aA humane death?17:05
* yvl puts on his executioner mask17:05
th1aOK asap, yvl.17:06
th1aWhat else menesis?17:06
menesisfrom my point of view, timetables have been broken17:07
menesisno docs, no tests, no import17:07
yvlfor the purpose of old skin, tts *are* broken17:08
yvla little, but broken17:08
th1aWell, perhaps yvl will have to return to tts next week.17:08
yvlfixing things you mention was dropped in favor of developing flourish, so...17:09
yvlguys, is there a point in doing that?17:09
menesisso the old skin no longer works as before?17:09
yvlin flourish branch, the old skin works differently when it comes to tts17:09
yvlso tests go through the window17:09
menesisdifferently is ok17:10
th1aWe need to remember that the purpose of this release is to let people start working on translations more than anything else.17:10
th1aThis alpha, whatever release.17:11
th1aSo basically note what needs to be fixed in the next month and push it out.17:12
th1aI guess we have three weeks!17:13
th1aSo it is not time to be picky.17:13
menesisI have extracted strings for schooltool and needed to fix some. but that's the last thing I planned to do before releasing17:13
menesisth1a: maybe you want to update CHANGES.txt? :)17:14
menesisbecause now it says "New skin" and that's all17:14
menesismaybe will add "too many to list"17:15
th1aWe've got new skin, new timetables and then bugfixes, right?17:15
th1aThere should be specific bugfixes as usual, right?17:15
th1aOr are all they in the last trunk release.17:16
menesishaven't looked at everything in flourish17:16
menesisflourish fixes many things like "dead end" or "navigation difficult"17:16
menesismaybe there are bugs for them, maybe not17:16
th1aWell, assuming there are no generic bugfixes in flourish.17:16
th1aI don't care about the "bad ui" bugs fixed.17:17
th1a(don't care to enumerate them)17:17
th1aSo... to clarify.17:17
th1aWhat should go in CHANGES.txt is just whatever was in flourish + timetables?17:17
th1aNo additional non-flourish branch changes?17:18
menesisbugfixes and cleanup are in trunk17:21
menesisand released as 1.7.017:21
menesisand in oneiric already17:21
th1aI do need you to start sending, to me at least, very clear indications that new releases are being pushed so our users know.17:23
th1aI'm not sure if you think I can infer that from the general stream, but I really just need a ANNOUNCE NEW RELEASE NOW  -- NOTES HERE! email.17:24
* yvl killed tt tests btw17:27
th1aThanks yvl17:27
th1aAnything else menesis?17:29
th1aOK, thanks menesis.  Let's push this out the door.17:30
th1aWe've got three guys with nothing better to do than clean up whatever messes you find in the process.17:31
th1aWho is left... replaceafill?17:31
replaceafillquestion for yvl first17:32
replaceafillyvl where are the manager and bob browsers set in the pool?17:32
replaceafilli've been looking for them but couldnt find them17:32
replaceafilli've been looking at the diffs only though :)17:32
yvllook at schooltool/testing/ BrowserPool17:33
yvlthey're supposed to be painless auto-init browsers17:33
yvland auto-cleanup17:33
replaceafilli was CTRL+F for bob :D17:33
replaceafillk, thanks17:33
replaceafillth1a ok17:33
replaceafilli added the Contact Persons and Contact Relationships sheets17:34
replaceafill1,000 contacts17:34
replaceafillthey're all external17:34
th1ammmm green.17:34
* replaceafill didnt know we could set up bound contact data from the Contact Persons sheet...17:35
replaceafillalso, i updated the sheet to use 2010-2011 and 2011-201217:35
th1aWe need some in-sheet documentation there.17:35
th1aAlso, my current oneiric install is so fubar I can't actually look at it.17:36
replaceafillmaybe cell comments?17:36
replaceafillyou can see the new sample years here
replaceafilli used the 2010 and 2011 calendars from ;)17:36
th1aI have a SATA cable coming overnight so I should be able to have a nice clean installation tonight on a fresh drive.17:36
replaceafillset some holidays from it17:36
replaceafillfinally, set contact data for the 1k contacts created17:37
replaceafillsome students have 2 contacts17:38
replaceafillwith relationships17:38
replaceafillfor the small sample sheet, i was thinking maybe 50 person, contact would be enough?17:39
replaceafilland just a few sections17:39
replaceafillok, i'll work on the small sample sheet today and will test the selenium branch17:39
replaceafillthat's it from me17:39
th1aCool.  OK, thanks guys.17:40
th1aCatch you Thursday.17:40
* th1a drops the bag of gravel.17:40
yvlthanks guys17:40
yvlsee you soon :)17:41
replaceafillthank you everybody17:41
aelknercya guys thursday17:41
replaceafillyvl i had to use $ LANG=C bin/test --tests-pattern=stests to avoid locale issues19:26
replaceafillFile "/home/replaceafill/.sandboxes/schooltool-selenium/src/schooltool/app/browser/stests/app.txt", line 18, in app.txt19:27
replaceafillFailed example:19:27
replaceafill    print title.text19:27
replaceafillDifferences (ndiff with -expected +actual):19:27
replaceafill    - Calendar for Administrator, SchoolTool - Tuesday, February 1, 200519:27
replaceafill    ?                                          ^^   ^^^^^   ^^ ^^^^19:27
replaceafill    + Calendar for Administrator, SchoolTool - martes 1 de febrero de 200519:27
replaceafill    ?                                          ^^^^  +++ ^^^   ^ ^^^^19:27
replaceafillto avoid that ^19:27
*** menesis has quit IRC20:09
*** menesis has joined #schooltool21:11
*** menesis has quit IRC21:22
*** menesis has joined #schooltool22:02

Generated by 2.15.1 by Marius Gedminas - find it at!