*** tpb has joined #melange | 00:00 | |
*** tansell_laptop has quit IRC | 00:41 | |
*** dmitrig01|afk has quit IRC | 00:45 | |
*** dmitrig01|afk has joined #melange | 00:47 | |
*** ChanServ sets mode: +v dmitrig01|afk | 00:47 | |
*** arvind_khadri has joined #melange | 01:13 | |
*** dmitrig01|afk has quit IRC | 01:43 | |
*** dbentley-laptop has quit IRC | 01:54 | |
*** mithro has joined #melange | 02:05 | |
*** ChanServ sets mode: +v mithro | 02:05 | |
*** forever has joined #melange | 02:26 | |
*** arvind_khadri has quit IRC | 02:50 | |
*** solydzajs has joined #melange | 03:17 | |
*** ChanServ sets mode: +o solydzajs | 03:17 | |
*** solydzajs has quit IRC | 03:23 | |
*** solydzajs has joined #melange | 03:23 | |
*** ChanServ sets mode: +o solydzajs | 03:23 | |
*** SRabbelier has joined #melange | 03:29 | |
*** ChanServ sets mode: +o SRabbelier | 03:29 | |
*** mithro has quit IRC | 04:26 | |
*** Merio has joined #melange | 05:00 | |
*** ankitg has quit IRC | 05:36 | |
*** SRabbelier has quit IRC | 06:09 | |
*** SRabbelier1 has joined #melange | 06:10 | |
*** SRabbelier1 is now known as SRabbelier | 06:11 | |
*** ChanServ sets mode: +o SRabbelier | 06:11 | |
*** jbailey has quit IRC | 06:20 | |
*** jbailey has joined #melange | 06:20 | |
*** irc.freenode.net sets mode: +o jbailey | 06:20 | |
*** ankitg has joined #melange | 06:33 | |
*** florinciu has joined #melange | 06:58 | |
*** madrazr has joined #melange | 08:06 | |
*** mithro has joined #melange | 08:18 | |
*** ChanServ sets mode: +v mithro | 08:18 | |
madrazr | SRabbelier: Hi | 08:38 |
---|---|---|
SRabbelier | madrazr: heya! | 08:38 |
madrazr | SRabbelier: I have a question :P | 08:40 |
SRabbelier | madrazr: ask away | 08:40 |
madrazr | SRabbelier: Say I have a model and I create 2 instances of that model and store those instances in the data store | 08:40 |
SRabbelier | madrazr: sure | 08:40 |
madrazr | SRabbelier: one instance will have values for all the properties, where as the 2nd instance will have values for just 20% of the properties (as in if there are 10 properties, it holds values for just 2) | 08:41 |
SRabbelier | madrazr: assuming those values are not required, ok | 08:41 |
madrazr | will the 2nd instance consume less storage space in the data store? | 08:41 |
madrazr | SRabbelier: yeah | 08:41 |
SRabbelier | madrazr: hmmmm, yes, I do think it could consume less space, but in practice it'll probably be negligable, why? | 08:43 |
SRabbelier | madrazr: 0.47 of 602.00 GBytes | 08:43 |
SRabbelier | madrazr: we have quite a bit of storage space left :P | 08:43 |
madrazr | SRabbelier: ROFL! :D | 08:44 |
madrazr | SRabbelier: For GHOP tasks we are planning to store the entire history of a task | 08:45 |
SRabbelier | madrazr: aye | 08:45 |
madrazr | SRabbelier: I was thinking of storing them as ChangeSets | 08:45 |
madrazr | SRabbelier: ? | 08:45 |
SRabbelier | madrazr: mhhh, that's an option | 08:45 |
madrazr | SRabbelier: Lennie also suggested, storing the change as a single field and parsing this field to obtain the history | 08:48 |
SRabbelier | madrazr: as a JSON string? | 08:48 |
madrazr | SRabbelier: can you suggest something? | 08:48 |
madrazr | SRabbelier: yeah thats possible | 08:48 |
SRabbelier | madrazr: that would make gathering statistics harder and easier at the same time | 08:48 |
madrazr | SRabbelier: easier? | 08:49 |
SRabbelier | madrazr: e.g., you can't find out in one query how many students have completed a task, but you can gather _all_ statistics in one go very easily | 08:49 |
madrazr | SRabbelier: Oh yeah | 08:51 |
madrazr | SRabbelier: which of these 2 kinds are more often? | 08:51 |
SRabbelier | madrazr: mhhh, I think the latter | 08:52 |
madrazr | SRabbelier: Ok | 08:52 |
*** forever has quit IRC | 10:03 | |
*** solydzajs has quit IRC | 10:21 | |
*** dr__house has joined #melange | 10:52 | |
*** dr__house has quit IRC | 11:01 | |
*** dr__house has joined #melange | 11:02 | |
madrazr | SRabbelier: what should scope_path contain if I have to create a dummy organization entity? | 11:13 |
madrazr | SRabbelier: this question is in context of writing organization logic test | 11:13 |
SRabbelier | madrazr: the scope of some test program | 11:17 |
madrazr | SRabbelier: sorry, was on a call :( | 11:30 |
SRabbelier | madrazr: np | 11:31 |
madrazr | SRabbelier: so it can be some arbitrary string? or should I create a program too? | 11:31 |
SRabbelier | madrazr: no, it should be the key name of a program entity (sorry, not scope) | 11:31 |
SRabbelier | madrazr: might be good to re-use some code from seed_db to create what you need | 11:32 |
madrazr | SRabbelier: Ok | 11:32 |
*** dr__house has quit IRC | 11:39 | |
*** dr__house has joined #melange | 11:39 | |
*** florinciu has quit IRC | 12:05 | |
*** dr__house has quit IRC | 12:29 | |
*** dr__house has joined #melange | 12:30 | |
*** Merio has quit IRC | 12:52 | |
madrazr | SRabbelier: how do I run the tests I have added now? | 13:03 |
SRabbelier | madrazr: ./run.py | 13:11 |
madrazr | SRabbelier: did the same, but I get this error, "nose.config.ConfigError: Error reading config file 'setup.cfg': no such option 'cover-html-dir'" | 13:12 |
SRabbelier | madrazr: you need to install nose 0.11 | 13:12 |
madrazr | SRabbelier: ah Ok | 13:12 |
SRabbelier | madrazr: easy_install nose=0.11 | 13:12 |
madrazr | I have install 0.10 from repos | 13:12 |
madrazr | SRabbelier: will install it | 13:12 |
SRabbelier | madrazr: mind updating the wiki | 13:13 |
SRabbelier | ? | 13:13 |
madrazr | SRabbelier: I will do it | 13:13 |
SRabbelier | madrazr: sweet, thanks! | 13:13 |
madrazr | this should goto Testing Guidelines Wiki page? | 13:13 |
madrazr | SRabbelier: NP | 13:13 |
SRabbelier | madrazr: mhhhh, should go in some testing related document :D | 13:14 |
* SRabbelier is not sure where | 13:14 | |
madrazr | SRabbelier: Ok, thats the only document that talks about tests :P | 13:14 |
madrazr | so it should go there :D | 13:14 |
SRabbelier | hehe, maybe make a new one? | 13:14 |
SRabbelier | madrazr: GettingStartedOnTests? | 13:14 |
madrazr | SRabbelier: it already exists, you have created it | 13:15 |
madrazr | we need to update it | 13:15 |
madrazr | I will do it | 13:15 |
madrazr | is it Ok if I do it on that page? | 13:15 |
SRabbelier | madrazr: yes please :) | 13:16 |
madrazr | SRabbelier: Ok cool! | 13:16 |
madrazr | SRabbelier: this tests is making histories for me :D | 13:16 |
madrazr | Many first things, this is the first time I ever used easy_install :D | 13:17 |
SRabbelier | madrazr: cool! best way to learn new stuff is to use it :D | 13:18 |
madrazr | SRabbelier: yeah | 13:18 |
*** dr__house has quit IRC | 13:25 | |
*** dr__house has joined #melange | 13:26 | |
*** dhaun has joined #melange | 13:42 | |
Erant | *yawn* | 14:08 |
Erant | Reversing a wearleveling algo for large block is not fun. | 14:08 |
Erant | large block NAND too. | 14:08 |
*** dbentley-laptop has joined #melange | 14:25 | |
madrazr | SRabbelier: is it Ok to call seed_db.seed_org () and similar functions if necessary from the tests instead of writing the same code again? | 14:43 |
SRabbelier | madrazr: I think that's acceptable, jup | 14:53 |
madrazr | dr__house: Ok | 14:53 |
dr__house | madrazr: that was referred to SRabbelier I guess | 14:53 |
dr__house | :P | 14:53 |
madrazr | dr__house: I have really gone mad today :P | 14:54 |
dr__house | madrazr: no, thats not true. You were before :P | 14:54 |
madrazr | dr__house: no I was madrazr before :P | 14:54 |
dr__house | heh | 14:55 |
*** jbailey has left #melange | 15:37 | |
madrazr | SRabbelier: | 15:48 |
madrazr | http://paste.pocoo.org/show/117477/ | 15:48 |
tpb | Title: Paste #117477 | LodgeIt! (at paste.pocoo.org) | 15:48 |
madrazr | how do I solve this error? | 15:48 |
SRabbelier | madrazr: look at how run.py, it should set it | 16:02 |
SRabbelier | madrazr: if it doesn't, make it | 16:02 |
madrazr | SRabbelier: yeah I now just tried with os.environ['HTTP_HOST'] = 'localhost' | 16:03 |
madrazr | and that error is gone | 16:03 |
SRabbelier | madrazr: wooh | 16:03 |
madrazr | I have a new error now :P | 16:03 |
madrazr | SRabbelier: http://paste.pocoo.org/show/117482/ | 16:04 |
tpb | Title: Paste #117482 | LodgeIt! (at paste.pocoo.org) | 16:04 |
SRabbelier | madrazr: you forgot a .logic | 16:04 |
madrazr | SRabbelier: for? | 16:05 |
SRabbelier | madrazr: or.. you found a bug :P | 16:05 |
SRabbelier | madrazr: the error line | 16:05 |
SRabbelier | madrazr: key_name = model_logic.notification.logic.getKeyNameFromFields(fields) | 16:05 |
madrazr | SRabbelier: latter I am thinking :D | 16:05 |
SRabbelier | madrazr: should be model_logic.logic | 16:05 |
SRabbelier | madrazr: or at the top | 16:05 |
madrazr | SRabbelier: Ok | 16:06 |
madrazr | SRabbelier: import soc.logic.models as model_logic | 16:14 |
madrazr | The import statement looks like this | 16:14 |
madrazr | SRabbelier: so key_name = model_logic.notification.logic.getKeyNameFromFields(fields) should be correct | 16:14 |
madrazr | but still the test says module has no attribute notification | 16:14 |
*** dr__house has quit IRC | 16:24 | |
Erant | http://www.theregister.co.uk/2009/05/15/microsoft_banishes_memcpy/ | 16:24 |
tpb | <http://ln-s.net/3C4m> (at www.theregister.co.uk) | 16:24 |
Erant | Oh pulease. | 16:24 |
Erant | Microsoft has crappy programmers. | 16:24 |
*** madrazr1 has joined #melange | 16:33 | |
SRabbelier | madrazr, madrazr1: oh, maybe I misread | 16:33 |
*** madrazr has quit IRC | 16:34 | |
*** madrazr1 is now known as madrazr | 16:34 | |
madrazr | SRabbelier: Ok | 16:34 |
madrazr | but I am wondering why that error is occuring | 16:34 |
SRabbelier | madrazr: what's your full path? | 16:34 |
madrazr | SRabbelier: full path to? | 16:35 |
SRabbelier | madrazr: to you repo | 16:35 |
madrazr | SRabbelier: /media/netfs/melange/ | 16:35 |
SRabbelier | madrazr: and then /tests? | 16:36 |
madrazr | SRabbelier: yeah | 16:36 |
SRabbelier | madrazr: mhhh, weird then | 16:37 |
SRabbelier | madrazr: all the tests pass but that one? | 16:37 |
madrazr | SRabbelier: yeah | 16:37 |
madrazr | SRabbelier: and the weirdest part is, just replace model_logic.notification with any other logic say model_logic.user this error won't occur | 16:38 |
SRabbelier | madrazr: mhhhh, weird, it's working for me :( | 16:39 |
madrazr | SRabbelier: this is happening for the new test I have written | 16:39 |
madrazr | SRabbelier: I will mail the patch | 16:39 |
madrazr | apply and test | 16:39 |
madrazr | SRabbelier: whenever you are free | 16:39 |
SRabbelier | madrazr: ok, I can do that now :) | 16:40 |
madrazr | SRabbelier: Ok I will send it in 2 mins | 16:42 |
SRabbelier | madrazr: ok | 16:43 |
madrazr | SRabbelier: sent | 16:46 |
SRabbelier | madrazr: ok | 16:46 |
madrazr | SRabbelier: brb | 16:48 |
*** madrazr has left #melange | 16:48 | |
*** madrazr has joined #melange | 16:50 | |
madrazr | SRabbelier: back | 16:54 |
SRabbelier | madrazr: wb | 16:54 |
madrazr | SRabbelier: :) | 16:54 |
SRabbelier | madrazr: good news, I get the error too :P | 16:56 |
madrazr | SRabbelier: hehe :) | 16:56 |
madrazr | SRabbelier: that means I broke something? :P | 16:56 |
SRabbelier | madrazr: am checking | 16:59 |
madrazr | SRabbelier: Ok | 16:59 |
SRabbelier | madrazr: mhhh, maybe I know | 17:00 |
madrazr | SRabbelier: cool! whats that? | 17:00 |
SRabbelier | madrazr: fixed :) | 17:04 |
madrazr | SRabbelier: sooper | 17:04 |
madrazr | SRabbelier: did you commit it? | 17:04 |
SRabbelier | madrazr: doing so now | 17:04 |
madrazr | SRabbelier: Ok | 17:05 |
SRabbelier | madrazr: committed under git, then converted to hg and pushed! | 17:08 |
SRabbelier | hg-git ftw :) | 17:08 |
SRabbelier | mad props to durin42 and schacon :D | 17:08 |
* durin42 hears name, unlurks | 17:08 | |
durin42 | hee, thanks | 17:09 |
madrazr | SRabbelier: sooper | 17:10 |
SRabbelier | durin42: I hope to be able to help out more with hg-git, but everytime I pick something to fix schacon has already fixed it, amazing! :P | 17:11 |
madrazr | SRabbelier: I still did not understand what the problem was :( | 17:12 |
madrazr | SRabbelier: and what did your fix do? | 17:12 |
SRabbelier | madrazr: in python, if you do not import a module, it just does not exist in the parent module | 17:12 |
SRabbelier | madrazr: try this in an interactive python interpreter | 17:13 |
*** florinciu has joined #melange | 17:13 | |
SRabbelier | (in app) | 17:13 |
SRabbelier | import soc | 17:13 |
SRabbelier | dir(soc) | 17:13 |
SRabbelier | import soc.models | 17:13 |
SRabbelier | dir(soc) | 17:13 |
SRabbelier | and lo and behold ;) | 17:13 |
madrazr | SRabbelier: heh OK :) | 17:14 |
madrazr | got it now :D | 17:14 |
madrazr | it looks so trivial now ;-( | 17:14 |
madrazr | ;-) | 17:14 |
SRabbelier | madrazr: ^_^ | 17:15 |
SRabbelier | http://tinyurl.com/lol-birds | 17:15 |
tpb | Title: LOL burdz « Lolcats n Funny Pictures of Cats I Can Has Cheezburger? (at tinyurl.com) | 17:15 |
madrazr | SRabbelier: :) | 17:16 |
madrazr | SRabbelier: where should I get gaeftests module from? | 17:17 |
SRabbelier | madrazr: easy_install again :) | 17:17 |
madrazr | SRabbelier: heh Ok :) | 17:17 |
madrazr | SRabbelier: tests passed | 17:22 |
madrazr | SRabbelier: I have not changed anything in the patch sent | 17:22 |
SRabbelier | madrazr: yup :D | 17:22 |
SRabbelier | madrazr: that's what I saw too | 17:22 |
madrazr | is it possible to review the patch and give suggestion? | 17:22 |
madrazr | SRabbelier: heh Ok | 17:22 |
madrazr | SRabbelier: any suggestions for improvement? | 17:22 |
SRabbelier | madrazr: can you reuse the org_app properties from seed_db? | 17:36 |
madrazr | SRabbelier: as in call seed_db.seed_org_app() method? | 17:37 |
SRabbelier | madrazr: right, or modify seed_db to let you more easily do that, that'd be fine too | 17:37 |
madrazr | SRabbelier: Ok | 17:38 |
madrazr | that means just have request parameter take a default value as None | 17:38 |
SRabbelier | madrazr: mhhh, would be better to just make it not need the request parameter | 17:38 |
madrazr | SRabbelier: I did not get you | 17:39 |
SRabbelier | madrazr: what does it use request for? | 17:39 |
madrazr | SRabbelier: nothing | 17:39 |
madrazr | so totally remove request? | 17:39 |
SRabbelier | madrazr: mhhh, I think that would break other parts of seed_db | 17:40 |
madrazr | yeah | 17:40 |
madrazr | so making def seed_org_app(request=None, i): | 17:41 |
madrazr | works? | 17:41 |
SRabbelier | madrazr: yeah, I guess that'd work for this case :) | 17:42 |
madrazr | SRabbelier: Ok | 17:43 |
*** dhaun has quit IRC | 17:48 | |
madrazr | SRabbelier: if we re-use seed_db's seed_org_app after we get the properties back | 17:53 |
madrazr | we will have to manipulate the link_id | 17:54 |
SRabbelier | madrazr: how come? | 17:54 |
madrazr | SRabbelier: link_id of org and org_app needs to same no? | 17:54 |
SRabbelier | madrazr: right, but doesn't it use the 'i' value for that? | 17:54 |
madrazr | SRabbelier: it does, but the way seed_org and seed_org_app constructs link_id is different | 17:55 |
madrazr | seed_org_app has 'link_id': 'org_app_%d' % i, | 17:55 |
madrazr | seed_org has 'link_id': 'org_%d' % i, | 17:55 |
SRabbelier | madrazr: ah, right, that should probably be changed (org_app should be org) | 17:56 |
madrazr | SRabbelier: Ok | 17:56 |
madrazr | SRabbelier: then I think now we can reuse both seed_org and seed_org_app | 17:57 |
madrazr | shall I ? | 17:57 |
SRabbelier | madrazr: yup, go ahead | 17:57 |
madrazr | SRabbelier: ouch, seed_org_app uses request parameter :) | 18:04 |
madrazr | status = request.GET.get('status', 'pre-accepted') | 18:04 |
madrazr | SRabbelier: can we still re-use it? any tricks? | 18:05 |
SRabbelier | madrazr: yes, do the status extraction in the seed_db code, and pass that to seed_org_app | 18:06 |
madrazr | SRabbelier: so status is a 3rd parameter? | 18:08 |
madrazr | to seed_org_app | 18:08 |
SRabbelier | madrazr: instead of request | 18:09 |
madrazr | SRabbelier: I did not get this, please explain a bit | 18:09 |
SRabbelier | madrazr: sorry, was looking up The Law of Demeter on statckoverflow :P | 18:23 |
SRabbelier | madrazr: http://stackoverflow.com/questions/871090/what-is-the-name-of-this-programming-rule | 18:23 |
SRabbelier | :D | 18:23 |
tpb | <http://ln-s.net/3C7Q> (at stackoverflow.com) | 18:23 |
madrazr | SRabbelier: NP | 18:23 |
madrazr | I am reading it | 18:23 |
madrazr | SRabbelier: you posted that for me? :P | 18:24 |
SRabbelier | madrazr: yeah :P | 18:25 |
SRabbelier | madrazr: wanted to refer you to it, but couldn't remember the name! | 18:25 |
madrazr | SRabbelier: he he OK | 18:25 |
SRabbelier | madrazr: we're almost-but-not-quite violating it there | 18:25 |
madrazr | SRabbelier: stackoverflow is freaking fast :P | 18:25 |
SRabbelier | madrazr: but the idea is the same | 18:25 |
SRabbelier | madrazr: yes, it is, it's amazing | 18:25 |
madrazr | SRabbelier: Ok | 18:25 |
SRabbelier | madrazr: instead of doing foo(request) and then in foo have request.GET.get('status', 'pre-accepted') | 18:26 |
SRabbelier | madrazr: we should extract that value before calling foo, and then pass it as argument | 18:26 |
madrazr | SRabbelier: but that breaks other parts of seed_db no? | 18:26 |
SRabbelier | madrazr: well, that's the point, you need to fix those calles :P | 18:27 |
SRabbelier | madrazr: should only be one though | 18:27 |
madrazr | SRabbelier: Ok | 18:27 |
madrazr | SRabbelier: found out where to fix this | 18:30 |
SRabbelier | madrazr: yay | 18:30 |
madrazr | I will fix it now, test it and send it tomorrow | 18:30 |
madrazr | need to sleep now :) 4AM :D | 18:31 |
madrazr | SRabbelier: ^ | 18:31 |
SRabbelier | madrazr: lol, go to bed! | 18:31 |
madrazr | it is just one call as you said on line number 786 in seed_db props = action(request, i) :P | 18:31 |
madrazr | SRabbelier: yeah running | 18:31 |
madrazr | bye good night | 18:31 |
SRabbelier | madrazr: g'night :) | 18:32 |
*** madrazr has left #melange | 18:32 | |
*** SRabbelier has quit IRC | 20:10 | |
*** florinciu has quit IRC | 22:07 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!