| *** tpb has joined #melange | 00:00 | |
| *** dukeleto has quit IRC | 01:42 | |
| *** jamtoday_ has left #melange | 02:04 | |
| *** jamtoday has joined #melange | 02:05 | |
| *** madrazr has joined #melange | 02:40 | |
| *** ChanServ sets mode: +v madrazr | 02:40 | |
| *** dukeleto has joined #melange | 03:53 | |
| *** mithro has quit IRC | 04:24 | |
| *** solydzajs has joined #melange | 04:26 | |
| *** ChanServ sets mode: +o solydzajs | 04:26 | |
| *** tansell_laptop has quit IRC | 04:26 | |
| *** MatthewWilkes has quit IRC | 04:54 | |
| *** Lennie has joined #melange | 05:08 | |
| *** ChanServ sets mode: +o Lennie | 05:08 | |
| Lennie | hi jamtoday ajaksu | 05:08 |
|---|---|---|
| jamtoday | Lennie: pong | 05:09 |
| Lennie | hi | 05:09 |
| jamtoday | is this a good time for some real-time discussion? | 05:10 |
| jamtoday | discussion + coding | 05:10 |
| Lennie | i think so | 05:10 |
| *** Lennie sets mode: +v jamtoday | 05:10 | |
| Lennie | I just read the survey_helper patch from you | 05:11 |
| Lennie | and I don't feel happy about passing along the user | 05:11 |
| Lennie | just a method that adds the grade field to an existing form is enogh | 05:12 |
| jamtoday | I see, good idea | 05:12 |
| Lennie | the overriding subclass will just call that method to get its form | 05:12 |
| jamtoday | yes, makes sense | 05:12 |
| Lennie | and the rest can stay the same | 05:12 |
| Lennie | too bad daniel's patch is giving errors :( | 05:12 |
| jamtoday | Can a subclass perform any commands after the base class? | 05:12 |
| Lennie | yes | 05:12 |
| Lennie | or before | 05:13 |
| Lennie | or during | 05:13 |
| Lennie | I'm thinking about making a new method for getting the appriopriate form in the view | 05:13 |
| Lennie | which the subclass will override and just add the grade field to it | 05:13 |
| jamtoday | I see, so editGet and take and _public would all call one view | 05:14 |
| jamtoday | to be fair, they have some important differences | 05:14 |
| Lennie | no not one view :) | 05:14 |
| Lennie | one method in view classs | 05:14 |
| Lennie | that gives them their form | 05:14 |
| Lennie | or something liek it | 05:14 |
| jamtoday | yes, and then that in turn calls the helper | 05:14 |
| Lennie | indeed | 05:14 |
| Lennie | one view is nearly impossible | 05:14 |
| Lennie | that would be too ugly :D | 05:15 |
| jamtoday | I didn't know if I had the right idea about passing the project key as a GET param | 05:15 |
| jamtoday | did you notice that? | 05:15 |
| Lennie | ? | 05:15 |
| Lennie | I havent seen it | 05:15 |
| jamtoday | So....this is evaluation specific. | 05:15 |
| Lennie | I know I didi it :P | 05:15 |
| jamtoday | Imagine you have multiple mentor projects to choose from, and you've already taken the survey once for one project | 05:15 |
| jamtoday | I need to be able to get the saved_project as a param, and I get that by adding it to the URL when you choose your project from the dropdown field | 05:16 |
| Lennie | hmm | 05:16 |
| jamtoday | it's a pattern I've used in other web apps, but wasn't sure if it was in the GSOC style | 05:16 |
| Lennie | I'm not showing dropdown :) | 05:16 |
| Lennie | I'm just showing a list with information | 05:16 |
| jamtoday | yeah its broken now, one of the things I'm working on | 05:16 |
| jamtoday | a list of information...? | 05:16 |
| Lennie | yeah a list of all possible projects for which you can take this survey | 05:16 |
| jamtoday | So the survey is shown once per project | 05:17 |
| jamtoday | and when you submit you submit for all of them | 05:17 |
| jamtoday | that's another way to do it, I thought it would be strange to do it that way | 05:17 |
| Lennie | it's the easiest way for me atm | 05:17 |
| ajaksu | hi guys, good time for me :) | 05:17 |
| Lennie | your patch is borked ajaksu :P | 05:17 |
| *** MatthewWilkes has joined #melange | 05:18 | |
| jamtoday | Lennie: what error is being thrown? | 05:18 |
| ajaksu | Lennie: looking into it :) | 05:18 |
| Lennie | something related to his patch | 05:18 |
| Lennie | its on the mail | 05:18 |
| Lennie | which you can't read :( | 05:19 |
| Lennie | damn your connection :P | 05:19 |
| jamtoday | I discovered a terrible Tmobile exploit, but more on that later | 05:19 |
| jamtoday | my andy passed the 30 day free deal, so I am doing all kind of odd things to try to access gmail | 05:19 |
| Lennie | :P | 05:19 |
| jamtoday | ajaksu: just blame it on app engine :) | 05:21 |
| Lennie | ajaksu: while youare looking into it | 05:21 |
| Lennie | can you move the COMMENT_PREFIX to models/survey | 05:21 |
| Lennie | and just do from soc.models.survey import COMMENT_PREFIX ? | 05:22 |
| Lennie | since it is not really view specific but more data-storage related | 05:22 |
| jamtoday | ajaksu: the required fields is working great | 05:23 |
| Lennie | his form validation patch is what fails here | 05:23 |
| Lennie | so haven't tested required fields yet :) | 05:23 |
| Lennie | I gotta fix some bug I overlooked brb :P | 05:23 |
| ajaksu | Lennie: getSurveyResponseFromPost:747 should be "schema = SurveyContentSchema(survey.survey_content.schema)"... | 05:24 |
| Lennie | ajaksu: Send me an edited patch :D | 05:24 |
| ajaksu | Lennie: sure, just giving real time feedback :) | 05:25 |
| Lennie | ok :) | 05:25 |
| Lennie | also | 05:25 |
| Lennie | I renamed record to post_dict | 05:25 |
| Lennie | since it is so confusing with survey_record :) | 05:25 |
| ajaksu | so line 170 should be "if comment_name in post_dict or comment_name in self.survey_record:" | 05:27 |
| Lennie | hmm | 05:28 |
| ajaksu | re-diffing :) | 05:28 |
| Lennie | yeah | 05:28 |
| Lennie | but | 05:28 |
| Lennie | that would still break | 05:28 |
| Lennie | since survey_record is not always defined | 05:28 |
| Lennie | that was the error I was getting | 05:29 |
| Lennie | when creating a survey | 05:29 |
| ajaksu | right, self.survey_record needs hasattr instead | 05:29 |
| ajaksu | if it's None, it hasn'tattr | 05:29 |
| *** MatthewWilkes has quit IRC | 05:29 | |
| *** MatthewWilkes has joined #melange | 05:30 | |
| ajaksu | ( I guess I only tested this with the second patch applied, as the line becomes 'if schema.getHasComment(field):' ) | 05:30 |
| Lennie | hehe | 05:30 |
| Lennie | patch dependencies hurray :P | 05:31 |
| ajaksu | Lennie: can I literally edit the patch or should I re-diff? | 05:32 |
| Lennie | I think rediff is saver? | 05:32 |
| Lennie | since I asked for some renames and moving the PREFIX | 05:32 |
| ajaksu | moving the PREFIX? where did you ask that? | 05:34 |
| Lennie | in IRC :P | 05:37 |
| Lennie | my fail ^_^ | 05:37 |
| Lennie | When I reviewed it I moved it to the survey module | 05:37 |
| Lennie | because it is about datastrage | 05:37 |
| Lennie | and not just view-specific | 05:37 |
| ajaksu | Lennie: done fixing and renaming, do you want me to include the moving or can you use yours? | 05:46 |
| Lennie | just include it :) | 05:46 |
| Lennie | shouldn't be too hard afaik | 05:47 |
| Lennie | put it in the model and do from soc.models.survey import COMMENT_PREFIX | 05:47 |
| ajaksu | and rediff the other patch :) | 05:47 |
| Lennie | hehe ^^ | 05:47 |
| *** florinciu has joined #melange | 05:48 | |
| ajaksu | Lennie: you might want to skip testing new patch until I test it :) | 05:55 |
| Lennie | ok | 05:56 |
| Lennie | lemme know | 05:56 |
| Lennie | so | 06:03 |
| Lennie | stupid thinking error fixed :) | 06:03 |
| ajaksu | whose? :) | 06:04 |
| Lennie | mine | 06:05 |
| Lennie | so how is the patch testing coming along ajaksu? | 06:12 |
| Lennie | jamtoday: anything you want to discuss? | 06:14 |
| * jamtoday is getting his flash drive out | 06:14 | |
| jamtoday | patches coming in minutes... | 06:15 |
| ajaksu | Lennie: found a stupid lack of support for the new fields in JS, fixing it :) | 06:15 |
| Lennie | jamtoday: as soon as the method for extending the form with a grade field is out there I'll get the GradingProjectSurvey up and runnign | 06:15 |
| jamtoday | Lennie: ok. I'm on it as soon as I publish the logic patch | 06:17 |
| Lennie | jamtoday: what's wrong with the logic atm? | 06:18 |
| jamtoday | it needed a lot of refactoring, all of the if/else for mentors and students subclassed, DRY methods | 06:18 |
| jamtoday | however, I am wondering if a ProjectSurvey is ever for a mentor | 06:18 |
| Lennie | nope is never :) | 06:18 |
| jamtoday | great, then this should work wonderfully | 06:18 |
| Lennie | atleast not in the current setup :P | 06:18 |
| jamtoday | it's functionally the same | 06:19 |
| Lennie | true | 06:19 |
| Lennie | but I don't want to think about it :D | 06:19 |
| Lennie | but what's there to subclass then? | 06:19 |
| jamtoday | btw I am taking a flight tomorrow, I am brewing coffee now so the plan is to sleep on my plane and pretend I have jet lag :) | 06:19 |
| jamtoday | the getProjects for student and mentors | 06:20 |
| jamtoday | the implementation was messy | 06:20 |
| Lennie | oh | 06:20 |
| jamtoday | and now it lends itself to a separate access check using the same method | 06:20 |
| Lennie | hmm | 06:20 |
| Lennie | but fetching those projects is already being done somewhere else | 06:20 |
| jamtoday | it is? | 06:21 |
| Lennie | yosh | 06:21 |
| Lennie | atleast for the student it is now | 06:21 |
| jamtoday | ln? | 06:22 |
| Lennie | project_survey view | 06:22 |
| Lennie | line 161 | 06:22 |
| jamtoday | setSurveyRecordGroup? | 06:22 |
| Lennie | ? | 06:23 |
| jamtoday | this is actually another spot I can subclass | 06:23 |
| jamtoday | oh, nevermind | 06:23 |
| Lennie | I'm not thinking about recorgroups now | 06:23 |
| Lennie | they are not important for gathering data :D | 06:23 |
| Lennie | atleast not on monday :) | 06:23 |
| Lennie | hurray for task api :P | 06:23 |
| Lennie | but yes | 06:24 |
| Lennie | certain functionality should move out of the survey logic | 06:24 |
| jamtoday | _selectProjects | 06:24 |
| Lennie | oui | 06:25 |
| jamtoday | this list view hasn't worked for me, perhaps I should revert and test this | 06:25 |
| jamtoday | this is preferrable to the dropdown? | 06:25 |
| Lennie | hmm | 06:25 |
| Lennie | it is easier for now | 06:25 |
| Lennie | atleast for me :P | 06:25 |
| jamtoday | but the other one already worked :) | 06:25 |
| jamtoday | anyways, I'll post the patch and we can revisit this | 06:26 |
| Lennie | hehe | 06:26 |
| Lennie | afaik the other one failed because you couldn't do the same survey twice for different projects | 06:26 |
| Lennie | that's why I decided to ahve the user select something up front :P | 06:26 |
| jamtoday | it should have let you....I programmed it to refresh the page with the project GET param, which would load the specified form | 06:27 |
| Lennie | yeah I was talking about a week ago | 06:27 |
| Lennie | I think you are a talking about yesterday's patch? | 06:28 |
| jamtoday | btw, this is also the patch for adding the grade Field | 06:28 |
| Lennie | if we are redirecting with get anyway | 06:28 |
| Lennie | I rather keep the list | 06:28 |
| Lennie | looks more bulky :D | 06:28 |
| Lennie | probably gives more information as well | 06:28 |
| jamtoday | ajax was a TODO, that would have been so nice and responsive... :) | 06:29 |
| Lennie | yes | 06:29 |
| Lennie | it can still be a TODO :) | 06:29 |
| Lennie | just not high on the prio list :P | 06:29 |
| jamtoday | I'll post three patches, two will be tiny changes to view and helper, and the big logic patch | 06:29 |
| Lennie | hmm | 06:30 |
| Lennie | ok | 06:30 |
| jamtoday | they're all needed for the grade field | 06:30 |
| Lennie | really? | 06:30 |
| jamtoday | the addGradeField is in logic, the getExtraFields is called from view | 06:30 |
| jamtoday | and actually since you don't have my current helper, you dont need the third patch | 06:30 |
| Lennie | the addgradefield is in the logic? | 06:31 |
| Lennie | but that's something view specific right? | 06:31 |
| jamtoday | yes, now that you put it that way.... | 06:32 |
| jamtoday | brb, making the changes | 06:32 |
| Lennie | I can imagine that the logic patch will remove all mentor student junk that is still in Survey logic | 06:33 |
| Lennie | that's for sure :) | 06:33 |
| Lennie | the getExtraFields or something will indeed have to be called from the view | 06:33 |
| Lennie | but does not need to be returned as a surveyrecord entry by the method in the helper | 06:34 |
| Lennie | because it is not in the survey schema | 06:34 |
| Lennie | the GradingProjectSurvey view can extract it on its own | 06:34 |
| Lennie | do whatever it wants with it and then store it :) | 06:34 |
| Lennie | as the boolean | 06:34 |
| Lennie | atleast that is what I had in mind | 06:35 |
| Lennie | don't know if it's reasonable | 06:35 |
| Lennie | but to me it sounds sane | 06:35 |
| jamtoday | I am calling getExtraFields from the takeGet | 06:38 |
| *** Lennie has quit IRC | 06:38 | |
| *** Lennie has joined #melange | 06:39 | |
| *** ChanServ sets mode: +o Lennie | 06:39 | |
| Lennie | so had to reboot :) | 06:40 |
| Lennie | my mac was somehow feeling slow | 06:40 |
| Lennie | must be the heat :P | 06:40 |
| Lennie | jamtoday: ping | 06:44 |
| Lennie | jamtoday: Does it make sense? | 06:44 |
| MatthewWilkes | Lennie: hah, we had the thunderstorm last night, it's livable now :) | 06:45 |
| Lennie | it has just rained here | 06:45 |
| Lennie | windows are open | 06:45 |
| Lennie | fresh air! | 06:45 |
| jamtoday | Lennie: so now there is two patches, the big one is grading_project_survey.py, and the one-line patch for survey view | 06:49 |
| jamtoday | because _takeGet needs to have the survey form passed | 06:49 |
| jamtoday | in the params | 06:49 |
| jamtoday | sound.....sane? | 06:49 |
| Lennie | hmm | 06:50 |
| Lennie | the later I'm not sure off | 06:50 |
| Lennie | I think Daniel did something else | 06:50 |
| jamtoday | hmmmm....in one of his latest patches? | 06:52 |
| Lennie | yes | 06:52 |
| Lennie | ajaksu: Ping | 06:52 |
| jamtoday | alright, i will post mine anyways | 06:53 |
| Lennie | ajaksu: have you tested your patch yet? I can't take the survey again :) | 06:53 |
| ajaksu | Lennie: pong | 06:53 |
| Lennie | soc/views/helper/surveys.py", line 653, in getSurveyResponseFromPost | 06:53 |
| Lennie | if name not in schema: | 06:53 |
| ajaksu | Lennie: I'm strugging with JS, so didn't get to that error yet, but I see why it happens (me being dumb) | 06:54 |
| Lennie | ok | 06:54 |
| Lennie | jamtoday: | 06:56 |
| Lennie | what does the big one for grading_project_survey do? | 06:56 |
| Lennie | I'm gonna get myself a cold shower and lunch | 06:59 |
| Lennie | ttys | 06:59 |
| jamtoday | internet fail | 07:00 |
| jamtoday | Lennie: uses _takeGet hook to call addGradeField, taken from logic where it shouldn't have been | 07:00 |
| ajaksu | yay for forgetting how to chain jquery selectors :/ | 07:02 |
| Lennie | jamtoday: sounds promising | 07:11 |
| Lennie | lunch time :) | 07:12 |
| *** jamtoday has quit IRC | 07:23 | |
| *** jamtoday has joined #melange | 07:47 | |
| jamtoday | internet router in a room where someone is sleeping. Never a good life design decision | 07:47 |
| Lennie | hehe | 07:47 |
| Lennie | ^^ | 07:47 |
| jamtoday | Lennie: you got the patch? | 07:47 |
| Lennie | yes I got it | 07:47 |
| Lennie | but it will be a few hours before I can review it | 07:48 |
| Lennie | maybe you want to get some sleep? | 07:48 |
| jamtoday | It would be a fun experiment to not sleep, but yes. agreed | 07:48 |
| jamtoday | not fun for survey takers, perhaps | 07:48 |
| Lennie | hehe | 07:48 |
| jamtoday | ttys | 07:49 |
| Lennie | yes | 07:49 |
| Lennie | hmm | 07:49 |
| Lennie | what's your plan for tomorow | 07:49 |
| Lennie | you needed to fly? | 07:49 |
| *** b0le has left #melange | 07:52 | |
| *** madrazr has quit IRC | 07:59 | |
| ajaksu | woot, Merio is giving us cross-browser UI clues! guy knows his JS and CSS :) | 08:39 |
| ajaksu | Lennie: can we kidnap Merio after GSoC ends, so Melange always has nice CSS and JS? :) | 08:40 |
| Lennie | hehe | 08:41 |
| Lennie | ajaksu: tbh we might yes | 08:41 |
| Lennie | ajaksu: depends on his performance :) | 08:41 |
| Lennie | ajaksu: ping | 08:44 |
| Lennie | I need another patch :D | 08:44 |
| ajaksu | Lennie: if I could vote, I'd vote for "he's helped me enough already" ;) | 08:44 |
| Lennie | ajaksu: your vote is welcome but he first has to go through GSoC :D | 08:44 |
| ajaksu | Lennie: for required/has_comment? sure | 08:44 |
| Lennie | no | 08:44 |
| Lennie | that is kinda working :) | 08:44 |
| Lennie | the problem is | 08:44 |
| Lennie | that we are now really using the django forms | 08:44 |
| Lennie | so | 08:44 |
| Lennie | error are now popping up because wrong widgets seem to be used | 08:45 |
| Lennie | when you create a survey | 08:45 |
| Lennie | with a questoin | 08:45 |
| Lennie | that allows checkboxes | 08:45 |
| Lennie | with pick_multi | 08:45 |
| Lennie | only one answer is returned by Django | 08:45 |
| Lennie | while I should be able to pick multiple | 08:46 |
| ajaksu | Lennie: ok, let me check it | 08:46 |
| Lennie | testcase is easy | 08:46 |
| Lennie | just choice question | 08:46 |
| Lennie | with pick multi | 08:46 |
| Lennie | (and what does pick_quant mean btw) | 08:46 |
| ajaksu | (OT: here's a preview of our export as JSON if we prettyprint it http://paste.pocoo.org/show/126495/ ) | 08:47 |
| tpb | Title: Paste #126495 | LodgeIt! (at paste.pocoo.org) | 08:47 |
| Lennie | oh | 08:47 |
| Lennie | I see another problem | 08:47 |
| Lennie | it doesn't even store pick_quant if I choose it | 08:47 |
| Lennie | it just sticks to pick_multi | 08:47 |
| ajaksu | Lennie: means 'pick_one as radio buttons', used to mean 'helper for rating-like questions' | 08:47 |
| Lennie | ? | 08:48 |
| ajaksu | Lennie: can you try pick_quant + radio_buttons? jamtoday was working on JS to couple the type to render, not sure it works | 08:48 |
| Lennie | aha | 08:49 |
| Lennie | thanks ^^ | 08:49 |
| Lennie | and selection only maps to select? | 08:50 |
| Lennie | which would be more clear if i'd dropdown :D | 08:50 |
| Lennie | any | 08:50 |
| Lennie | I can't select multiple items | 08:50 |
| Lennie | because the widget clean only returns one | 08:50 |
| Lennie | so probably wrong setting for that widget | 08:50 |
| Lennie | brb | 08:53 |
| Lennie | lemme know what you find :) | 08:53 |
| ajaksu | Lennie: cannot reproduce yet, still trying :) | 08:55 |
| Lennie | don't forget to update :) | 08:55 |
| Lennie | since it happens because I made changes to your patch | 08:55 |
| Lennie | so that it works with the cleaned data instead of sticking to the properties gotten from the method in the survey_helper | 08:55 |
| ajaksu | ok, updating :) | 08:57 |
| Lennie | to be clear | 08:57 |
| Lennie | it is now the form's responsibility | 08:57 |
| Lennie | to have right field names | 08:58 |
| Lennie | for things that need to be stored | 08:58 |
| Lennie | on POST | 08:58 |
| Lennie | we create a new empty form | 08:58 |
| Lennie | from the survey schema | 08:58 |
| Lennie | and then insert the POST data | 08:58 |
| Lennie | we're not calling getSurveyResponseFromPost anymore :) | 08:58 |
| ajaksu | so maybe we're missing a POST.getlist from getSurveyResponseFromPost ? | 08:59 |
| Lennie | ? | 09:00 |
| Lennie | we are not calling that method | 09:00 |
| Lennie | we don't use it | 09:00 |
| Lennie | the Django form just returns one choice | 09:00 |
| Lennie | because the widget might be wrong | 09:00 |
| Lennie | I've just removed the getSurveyReponseFromPost from our codebase :) | 09:00 |
| ajaksu | I meant: getSurveyResponseFromPost would call POST.getlist(field) for multidicts (as pick_multi), do we do that now? :) | 09:01 |
| Lennie | nope | 09:01 |
| Lennie | but I think Django has a widget for multi-selection | 09:01 |
| Lennie | you should pull the latest update in | 09:01 |
| Lennie | and take a look :) | 09:01 |
| Lennie | I'll brb | 09:01 |
| ajaksu | still going through your changes :) | 09:02 |
| ajaksu | k | 09:02 |
| Lennie | ok :) | 09:07 |
| Lennie | the system now works just like the rest of Melange | 09:08 |
| Lennie | it creates a form | 09:08 |
| Lennie | initialize it with post data | 09:08 |
| Lennie | do the cleaning | 09:08 |
| Lennie | and read all the fields if there is no FormValidationError | 09:08 |
| ajaksu | Lennie: fixed | 09:10 |
| Lennie | I'm interested in the cause :D | 09:10 |
| Lennie | what was it? | 09:10 |
| ajaksu | we should use getlist for multiple values per key :) | 09:11 |
| Lennie | and we're doing something else :P | 09:11 |
| Lennie | but we were doing it right in getSurveyResponse.... | 09:12 |
| Lennie | ? | 09:12 |
| ajaksu | yep, fix is making it right in getFields | 09:12 |
| ajaksu | btw, the widget we use is PickManyCheckbox(forms.CheckboxSelectMultiple) | 09:12 |
| Lennie | strange | 09:12 |
| Lennie | why doesn't it work... | 09:12 |
| ajaksu | POST isn't a plain dict, so POST[key] only retrieves one value from the possible many associated to key, IIUC... but I'm not sure there isn't a django idiom for this :) | 09:16 |
| Lennie | hmm | 09:16 |
| Lennie | Django doesn't take that into account? | 09:16 |
| Lennie | isn't it just because we are setting the data in getFields? | 09:16 |
| Lennie | instead of letting the request.POST intiate the form | 09:17 |
| Lennie | http://docs.djangoproject.com/en/dev/ref/forms/api/ | 09:17 |
| tpb | <http://ln-s.net/3Rx7> (at docs.djangoproject.com) | 09:17 |
| Lennie | I might be talking bullshyt (google Anathem if you think that this is a typo) | 09:18 |
| ajaksu | Lennie: I think you're right... I would try to learn the django way so we could get rid of or simplify getFields, if someone told me it was a priority ;) | 09:18 |
| Lennie | hmm | 09:19 |
| Lennie | go for it :)? | 09:19 |
| ajaksu | yay, learning django :) | 09:19 |
| Lennie | I commited your fix btw | 09:22 |
| Lennie | alteast it works now | 09:22 |
| Lennie | but anyhow | 09:22 |
| Lennie | the rest of Melange would do SurveyForm(request.POST) | 09:22 |
| Lennie | but ofcourse you need to generate the form first before doing that ^^ | 09:23 |
| ajaksu | Lennie: I think jamtoday had it working like that, but as we don't actually define a Form nor a Model with static fields, getFields was mostly a way to keep field order | 09:25 |
| Lennie | hmm | 09:25 |
| Lennie | field order can still be defined | 09:25 |
| ajaksu | yep, trying to figure that out but I know once I saw a way :) | 09:26 |
| ajaksu | in the django way, it seems my earlier idea of having SurveyForm and SurveyEditForm would make most other code in getFields unnecessary | 09:26 |
| Lennie | hmm | 09:29 |
| Lennie | not sure | 09:30 |
| Lennie | why the difference? | 09:30 |
| Lennie | what is the difference actualluy | 09:30 |
| Lennie | is one for creating and the other for taking a Survey? | 09:30 |
| ajaksu | getFields is currently responsible for rendering or not the edit fields (required and has_comment), plus showing the choice editor if in edit view | 09:31 |
| ajaksu | one for creating/editing, the other for taking | 09:31 |
| Lennie | would indeed sound nice | 09:35 |
| Lennie | if you could make that happen ^^ | 09:35 |
| Lennie | and if possible | 09:35 |
| Lennie | another patch | 09:35 |
| Lennie | the tooltips don't seem to be stored | 09:36 |
| Lennie | (yes we're getting into beauty issues now) | 09:36 |
| Lennie | which is kinda god :) | 09:36 |
| Lennie | good | 09:36 |
| ajaksu | explain "don't seem to be stored": do you want a field in edit view to set tooltip per question? | 09:37 |
| * ajaksu hopes so | 09:37 | |
| Lennie | we have that | 09:37 |
| Lennie | right? | 09:37 |
| Lennie | it says type custom tooltip here | 09:37 |
| Lennie | but the tooltip doesn't get saved | 09:37 |
| Lennie | :) | 09:37 |
| Lennie | and it seems that you can't set a tooltip for the selection questions? | 09:38 |
| ajaksu | Lennie: actually that would be a "Write a Custom Prompt For This Question..." and the custom prompt should be shown inside the input (I never liked this idea, would love to move to tooltip)... the not storing was probably me breaking JS by adding a fieldset | 09:40 |
| ajaksu | so I can fix saving the custom prompt, or that plus putting it on a tooltip, or those plus adding a custom prompt for choice questions :) | 09:41 |
| Lennie | hmm | 09:41 |
| Lennie | aha | 09:41 |
| Lennie | it was supposed to be in the question | 09:41 |
| Lennie | hmm | 09:41 |
| ajaksu | (which had the issue the prompt disappears when you start typing... maybe for text questions it could be shown in both places?) | 09:42 |
| * ajaksu fixes JS | 09:42 | |
| Lennie | It doesn't dissapear because you are typing | 09:43 |
| Lennie | it just fades because you haven't moved your mouse | 09:43 |
| Lennie | hmm | 09:44 |
| Lennie | I gotta go in a few mintues | 09:44 |
| Lennie | got some traveling to do | 09:44 |
| ajaksu | no, I meant the in-textarea prompt would disappear | 09:44 |
| Lennie | aha | 09:44 |
| Lennie | yes | 09:45 |
| Lennie | ofcourse :) | 09:45 |
| Lennie | tooltip would be better indeed | 09:45 |
| Lennie | or even both | 09:45 |
| Lennie | althouh tooltip alone is enough | 09:45 |
| Lennie | both might be messy :P | 09:45 |
| ajaksu | I have the unravelling of getFields and fixing custom prompts/tooltips storage/display in my todo list, if I fix those before you're back I can try the results view again :)( | 09:45 |
| Lennie | hehe | 09:45 |
| Lennie | thanks | 09:46 |
| Lennie | I always disliked getFields | 09:46 |
| Lennie | never really understood it :) | 09:46 |
| Lennie | when I get back in like 1.5h I'll do the grade field | 09:46 |
| Lennie | from James patch hopefully :) | 09:46 |
| ajaksu | great, I thought you were going for some > 8h travel, glad to know it's not that long :D | 09:47 |
| *** dr__house has joined #melange | 09:47 | |
| *** madrazr has joined #melange | 09:48 | |
| *** ChanServ sets mode: +v madrazr | 09:48 | |
| *** Lennie has quit IRC | 09:49 | |
| *** solydzajs has quit IRC | 10:00 | |
| *** solydzajs has joined #melange | 10:03 | |
| *** ChanServ sets mode: +o solydzajs | 10:03 | |
| *** dr__house has quit IRC | 10:06 | |
| *** solydzajs_ has joined #melange | 10:07 | |
| *** dr__house has joined #melange | 10:07 | |
| *** mithro has joined #melange | 10:14 | |
| *** ChanServ sets mode: +v mithro | 10:14 | |
| *** dr__house has quit IRC | 10:19 | |
| *** dr__house has joined #melange | 10:19 | |
| *** solydzajs_ has quit IRC | 10:19 | |
| *** solydzajs has quit IRC | 10:22 | |
| *** jamtoday has quit IRC | 10:28 | |
| *** Merio has joined #melange | 10:39 | |
| *** ChanServ sets mode: +v Merio | 10:39 | |
| ajaksu | Merio: huge thanks for taking a good look at out CSS, I'll be able to test safari on windows plus ie sometime today, so I might be able to apply your suggestions and figure out how non-hidden divs get hidden :) | 10:51 |
| *** Merio has quit IRC | 11:06 | |
| *** Merio has joined #melange | 11:14 | |
| *** ChanServ sets mode: +v Merio | 11:14 | |
| *** Lennie has joined #melange | 11:37 | |
| *** ChanServ sets mode: +o Lennie | 11:37 | |
| Lennie | oh no he's back! | 11:37 |
| Lennie | ajaksu: ping | 11:38 |
| ajaksu | Lennie: pong :) | 11:38 |
| ajaksu | can't make the project list view work, but I still expect subclass issues, namely regarding record creation :) | 11:39 |
| Lennie | ajaksu: what's happening :)? | 11:41 |
| Lennie | it worked here | 11:41 |
| ajaksu | empty projects list... created as a student and as a mentor. Hmmm did you set version to devvin? | 11:41 |
| *** Lennie has quit IRC | 11:42 | |
| *** Lennie has joined #melange | 11:43 | |
| *** ChanServ sets mode: +o Lennie | 11:43 | |
| Lennie | so | 11:43 |
| Lennie | stupid wireless :) | 11:43 |
| Lennie | tell me | 11:43 |
| Lennie | what's not happening? | 11:43 |
| ajaksu | empty projects list... created as a student and as a mentor. Hmmm did you set version to devvin? | 11:43 |
| Lennie | mwah that's not the problem | 11:47 |
| ajaksu | Lennie: maybe my seed_db created users/mentors/students are to blame, should I create them myself? :) | 11:47 |
| Lennie | I think that' s it :P | 11:47 |
| Lennie | you should login as the student! | 11:47 |
| Lennie | otherwise you'll see nothing i | 11:47 |
| Lennie | you'll only see your own projects | 11:47 |
| ajaksu | done that, from a different browser and a different account, created a project that is linked to that account, and still get an empty list :) | 11:48 |
| Lennie | are you 100% sure | 11:50 |
| Lennie | I just did it myself here and it worked :P | 11:50 |
| ajaksu | Lennie: 100% sure, but then the user was created by seed_db... I'll do the manual user creation after I send my first getfields patch :) | 11:50 |
| Lennie | hmm | 11:51 |
| Lennie | are you logged in as that user? | 11:51 |
| Lennie | and is that user really the student for that project | 11:51 |
| ajaksu | in opera, yes... and as another one in FF | 11:51 |
| Lennie | crazy | 11:51 |
| Lennie | works here :P | 11:51 |
| Lennie | we'll get to that later then :) | 11:51 |
| ajaksu | Lennie: that is a good question, it was created using that user's id, but I'm not sure he's the only one with that id, I'll test it soon :) | 11:52 |
| Lennie | you can screw things up with seed_db yes :P | 11:52 |
| Lennie | Erant: ping! | 11:57 |
| Lennie | Erant: are you alive on the other side of the ocean :)? | 11:58 |
| Lennie | ajaksu: I'll start integrating James's grading field patch | 12:01 |
| Lennie | so that I can get the last type working as well | 12:01 |
| Lennie | we might need to refactor that once you are done | 12:01 |
| ajaksu | Lennie: OK, I can merge once you commit or send my first patch now, whatever works best for you | 12:02 |
| Lennie | no rush | 12:02 |
| Lennie | just make sure all is fine on your end :) | 12:02 |
| Lennie | I'll have dinner first anyhow | 12:02 |
| ajaksu | Lennie: ok, I'll send a simple patch now but I'm willing to update it after you commit the grade changes, so no need to worry about it :) | 12:04 |
| Lennie | ajaksu: I've pushed his work | 12:24 |
| Lennie | ajaksu: The problem is that you now only have the field on GET | 12:24 |
| Lennie | but what you actually want is to have it on both GET and POST | 12:24 |
| Lennie | so that when the fields are retrieved from the cleaned for it is still there | 12:24 |
| Lennie | *form | 12:25 |
| ajaksu | Lennie: the grade field? | 12:25 |
| Lennie | yes | 12:25 |
| Lennie | I was hoping with your patch that we could just get the form in the Take method | 12:26 |
| Lennie | instead of in takeGet and takePost | 12:26 |
| ajaksu | so taking grading surveys now works? and it creates GradingProjectSurveyRecord ? | 12:26 |
| Lennie | so there is one place where it gets build | 12:26 |
| Lennie | no | 12:26 |
| Lennie | because I'm not reading the grade field because it is not in the form on a POST request :) | 12:27 |
| Lennie | it just shows now | 12:27 |
| Lennie | it is probably something that needs to move to the helper code | 12:27 |
| Lennie | anyhow | 12:27 |
| Lennie | I first want to review your work | 12:27 |
| Lennie | have it in there | 12:27 |
| Lennie | and then we can move the grading field to a good position | 12:27 |
| ajaksu | Lennie: ok, but I might be going for the long way in my patch, so I'm adding my work plan and you can tell me to focus on the grade taking issue :) | 12:27 |
| Lennie | what's your work plan? | 12:28 |
| ajaksu | mailing it to the list, but might make it private if you want to spare me from the shame :) | 12:30 |
| Lennie | lol | 12:30 |
| Lennie | go watch the youtube movie called | 12:30 |
| Lennie | Google I/O Myth of the Genius Programmer | 12:30 |
| Lennie | and then tell me what you need to do :) | 12:30 |
| madrazr | Lennie: movie? :P | 12:30 |
| Lennie | talk :) | 12:31 |
| Lennie | movie | 12:31 |
| Lennie | it's youtube :P | 12:31 |
| madrazr | Lennie: he he Ok | 12:31 |
| Lennie | Ajaksu how is it currently working? | 12:31 |
| Lennie | I have a good idea to get the grading forms working ^_^ | 12:35 |
| Lennie | but I'll wait for your work on Django forms | 12:35 |
| Lennie | if we are lucky it will all be up and running in a few hours | 12:35 |
| ajaksu | Lennie: ok, sent it, now I hope you can spot an easy jump instead of the progressive plan I have in mind, otherwise I'm merging methods :) | 12:37 |
| Lennie | ? | 12:37 |
| Lennie | he | 12:37 |
| Lennie | what are we talking about here :)? | 12:38 |
| ajaksu | work plan for having pure django forms instead of getfieldsform, sent mail with initial patch + plan :) | 12:38 |
| Lennie | ok | 12:40 |
| Lennie | dinner here now | 12:41 |
| Lennie | I suggest you take a break too :D | 12:41 |
| Lennie | unless you want to keep coding | 12:41 |
| Lennie | brb | 12:41 |
| ajaksu | k :) | 12:41 |
| *** florinciu has quit IRC | 12:47 | |
| *** MatthewWilkes has quit IRC | 13:02 | |
| *** Merio has quit IRC | 13:09 | |
| Lennie | ajaksu: ping | 13:20 |
| ajaksu | pong | 13:20 |
| Lennie | ready for some more coding :P | 13:20 |
| Lennie | hmm | 13:29 |
| ajaksu | Lennie: SurveyEditForm now only has the getFields method, time to see how we can populate fields as a normal django form would... | 13:30 |
| ajaksu | or a Dynaform :) | 13:30 |
| Lennie | :) | 13:31 |
| Lennie | hehe | 13:31 |
| Lennie | Don't go for dynaform | 13:31 |
| Lennie | I'm not a fan of that | 13:31 |
| Lennie | are you sure the form needs to have a Meta class btw/ | 13:31 |
| Lennie | ? | 13:31 |
| ajaksu | nope, but I'm not sure about anything modules :D | 13:31 |
| ajaksu | Lennie: the thing is, how do we add dynamic fields? where is the field kind information comming from, since we don't have them in a model? | 13:32 |
| Lennie | brb | 13:32 |
| Lennie | few min | 13:32 |
| ajaksu | k :) | 13:32 |
| Lennie | k | 13:37 |
| Lennie | maybe it's better for now to let getFields stay? | 13:38 |
| Lennie | and focus on the other issues? | 13:38 |
| Lennie | ajaksu: :) | 13:40 |
| ajaksu | let me take a look at our options, I promise I won't waste more than 20 minutes on this and if I get nothing usable I'll simply put InsertFields back (so we regain ordering) | 13:40 |
| Lennie | ok | 13:41 |
| Lennie | you have 20 min then :) | 13:41 |
| ajaksu | deal :) | 13:41 |
| Lennie | I do like the split :) | 13:42 |
| Lennie | however I do see chances for code reduction there | 13:43 |
| ajaksu | let me send the current version, SurveyEditForm is lean and mean, and SurveyTakeForm can have the same treatment :) | 13:44 |
| Lennie | my main concern with the one you send in is | 13:44 |
| Lennie | that a lot of code seems double | 13:44 |
| Lennie | and that surveyEditForm had getFields with post_dict | 13:44 |
| Lennie | while I'm not sure that will ever be used | 13:44 |
| Lennie | since that is just form creation when editing afaik | 13:45 |
| ajaksu | most of the double code is removed, but we still have a lot to gain from redundancy in SurveyTakeForm methods... | 13:45 |
| Lennie | ok | 13:46 |
| Lennie | I shouldn't take your 20 min away | 13:46 |
| Lennie | I'll keep quiet now | 13:46 |
| ajaksu | the post_dict for SurveyEditForm might be interesting if we ever want to validate the survey content, but I'll remove it for now :) | 13:46 |
| Lennie | gonna browse some code ;) | 13:46 |
| *** dhaun has joined #melange | 13:58 | |
| ajaksu | Lennie: time's up, what are the issues we should focus on? :) | 14:01 |
| Lennie | hehe | 14:01 |
| ajaksu | (I think I got a good lead at http://docs.djangoproject.com/en/dev/topics/forms/modelforms/#changing-the-order-of-fields ) | 14:01 |
| tpb | <http://ln-s.net/3S3p> (at docs.djangoproject.com) | 14:01 |
| Lennie | pj | 14:02 |
| Lennie | oh | 14:02 |
| Lennie | for ordering fields you can use base_fields.keyOrder right? | 14:03 |
| Lennie | or that yes | 14:03 |
| Lennie | so | 14:03 |
| Lennie | hmm | 14:03 |
| Lennie | do you want to send me a new version of the patch? | 14:03 |
| ajaksu | Lennie: not sure, didn't have time to actually test, will do so later :) | 14:03 |
| Lennie | is everything still working? | 14:04 |
| ajaksu | I've already sent the latest one... | 14:04 |
| ajaksu | well, edit form loses field ordering, otherwise yes... I can revert the ordering back in 30 sec :) | 14:04 |
| ajaksu | but if you want to commit that, I should add some docstrings to new schema methods :) | 14:05 |
| Lennie | hmm | 14:06 |
| Lennie | ordering needs to be preserved | 14:06 |
| Lennie | otherwise the questionair will look strange | 14:06 |
| ajaksu | worse: it's preserved in take view, but edit view is like you're on acid during an earthquake :) | 14:07 |
| Lennie | hehe | 14:07 |
| Lennie | you should take a look into ordering tomorrow I guess | 14:08 |
| Lennie | if nothing else pops up | 14:08 |
| Lennie | so first | 14:08 |
| Lennie | get the ordering back | 14:08 |
| Lennie | make sure docstrings are there :) | 14:08 |
| Lennie | and get me that patch | 14:08 |
| ajaksu | Lennie: yes, keyOrder seems to fit the bill too, so tomorrow it is! I'll revert ordering , add docstrings, run a last test and send the patch :) | 14:09 |
| Lennie | cool | 14:09 |
| Lennie | then we can talk some bugs | 14:09 |
| Lennie | and features | 14:09 |
| Lennie | I'll probably bother you with something else when you are busy (the grade field stuff) | 14:09 |
| ajaksu | ok :) | 14:10 |
| ajaksu | Lennie: everything works, except that I found an ordering buglet from our JS not following our change to as_twoline_table... can I add that to the 'latter' list? :) | 14:21 |
| Lennie | what does this bug cause? | 14:22 |
| ajaksu | new questions in an existing survey have wrong indexes, because we're counting table rows as if they were fields :) | 14:22 |
| Lennie | counting sucks :( | 14:23 |
| Lennie | does this cause any problems | 14:23 |
| Lennie | ? | 14:23 |
| ajaksu | well, now that you mention it, I think it doesn't, but counting does: indexes are stored on creation and based on rows count... now if you delete a couple of questions, new ones don't get added to the bottom but to the middle of the form | 14:24 |
| Lennie | hehe | 14:24 |
| Lennie | interesting | 14:24 |
| Lennie | added to the better to do tomorrow then later list :P | 14:25 |
| ajaksu | nothing you can't edit in plain text: http://paste.pocoo.org/show/126495/ | 14:25 |
| Lennie | *add it | 14:25 |
| tpb | Title: Paste #126495 | LodgeIt! (at paste.pocoo.org) | 14:25 |
| Lennie | ^^ | 14:25 |
| Lennie | ok so | 14:26 |
| Lennie | I'll can probably apply your patch without serious problems | 14:26 |
| Lennie | but in while I'm reviewing it might be best to take a look at the long question fields | 14:26 |
| Lennie | and comment field | 14:26 |
| Lennie | fields | 14:26 |
| Lennie | and ensure that their cleaning method returns a db.Text type | 14:27 |
| Lennie | instead of string | 14:27 |
| Lennie | because when strings are returned only 500 bytes are allowed in the datastore | 14:27 |
| *** dukeleto_ has joined #melange | 14:27 | |
| Lennie | now comments and long answers must be able to grow beyond this limit | 14:28 |
| Lennie | therefore it needs to return a db.Text type | 14:28 |
| ajaksu | hmm, I had to make schema text due to that limit, but I thought the long fields or comments would generate text themselves | 14:28 |
| ajaksu | I see the issue if they don't :) | 14:28 |
| Lennie | which is something else you can do with djanog :D | 14:28 |
| Lennie | for each text/comment field | 14:28 |
| Lennie | for each id they have | 14:29 |
| Lennie | you should add an | 14:29 |
| Lennie | clean_id field | 14:29 |
| Lennie | referencing a function that transforms the input to the db.Text :) | 14:29 |
| Lennie | http://docs.djangoproject.com/en/dev/ref/forms/validation/ | 14:29 |
| tpb | <http://ln-s.net/3S4P> (at docs.djangoproject.com) | 14:29 |
| Lennie | I think that might help | 14:30 |
| Lennie | is that something you want to take a look at ;D? | 14:30 |
| Lennie | after you have sent your patch ofcourse | 14:31 |
| ajaksu | Lennie: sent it, what kind of serious problems do you expect? | 14:34 |
| Lennie | with? | 14:34 |
| ajaksu | the patch I sent, (if anything take-related, I didn't change much if anything there) :) | 14:35 |
| Lennie | I hope to not find any problems :D | 14:35 |
| Lennie | anyhow | 14:35 |
| Lennie | the db.Text fix might be nice | 14:35 |
| Lennie | or | 14:35 |
| Lennie | radiobuttons ^_^ | 14:36 |
| ajaksu | looking into the validation docs, soon to open the appengine docs :) | 14:36 |
| Lennie | but not sure if that is still broken | 14:36 |
| Lennie | I'll get back to that soon | 14:36 |
| ajaksu | radiobuttons: looks, saving, everything? jamtoday had a fix for looks back then | 14:36 |
| Lennie | you take a look at db.Text :D | 14:36 |
| Lennie | saving | 14:36 |
| Lennie | (in taking) | 14:36 |
| ajaksu | ok, looking into db.Text :) | 14:36 |
| Lennie | the validation fails | 14:37 |
| Lennie | it says expecting list | 14:37 |
| Lennie | but it probably gets one item :) | 14:37 |
| Lennie | anyhow | 14:37 |
| Lennie | I'm fixing some other nasty form validation error | 14:37 |
| Lennie | and then reviewing/applying your patch | 14:37 |
| ajaksu | ok :) | 14:38 |
| Lennie | so | 14:38 |
| Lennie | fix pushed :) | 14:38 |
| *** Shaan7 has joined #melange | 14:40 | |
| Lennie | ajaksu: you mind if I remove this_user and project references? | 14:41 |
| Lennie | as well as getRoleSpecificFields | 14:41 |
| Lennie | ? | 14:42 |
| ajaksu | Lennie: not at all, thank you :) | 14:42 |
| Lennie | ah | 14:43 |
| Lennie | lovely | 14:43 |
| Lennie | the code quality is so much higher then a week ago :p | 14:44 |
| *** dukeleto has quit IRC | 14:45 | |
| Lennie | ajaksu: ping | 14:48 |
| Lennie | # TODO(ajaksu): this breaks ordering and blocks merging choice methods | 14:49 |
| Lennie | still valid? | 14:49 |
| ajaksu | Lennie: yup, I just flagged it but it's about option ordering in a select | 14:49 |
| Lennie | ok | 14:49 |
| Lennie | then I let it be :) | 14:50 |
| ajaksu | I guess james didn't know how to flag which option to select, I think it's 'initial' or something like that :) | 14:50 |
| Lennie | hmm | 14:50 |
| Lennie | sounds sane | 14:50 |
| Lennie | not completly sure :P | 14:50 |
| Lennie | # TODO(ajaksu): find a new way to keep fields in order | 14:50 |
| Lennie | think that suits the one at self.insertFields() better | 14:51 |
| ajaksu | still valid, I reverted to the old way, ah, yes, insertFields is better... | 14:51 |
| Lennie | hmm? | 14:52 |
| Lennie | I was just hinting that there was another todo about sorting above insertFields :) | 14:52 |
| Lennie | and that I added "new" | 14:52 |
| Lennie | I need three more docstrings :( | 14:52 |
| Lennie | for SurveyContentSchema | 14:52 |
| ajaksu | Lennie: for the schema class? | 14:52 |
| Lennie | ajaksu: what's a good one for the __init__? | 14:53 |
| Lennie | schema is the schema stored in SurveyContent? | 14:53 |
| ajaksu | Lennie: yes, it is... how about 'Set the dict this class encapsulates'? | 14:54 |
| ajaksu | (TBH, I expected we'd do something fancier than storing as-is with that initial dict) | 14:54 |
| Lennie | ok | 14:54 |
| Lennie | mwah | 14:54 |
| Lennie | can't have it all :) | 14:55 |
| Lennie | getType? | 14:55 |
| Lennie | get the type of a field right :)? | 14:55 |
| ajaksu | 'Fetch question type for field (one of short_answer, long_, pick_multi, pick_quant, selection).' | 14:56 |
| ajaksu | how did you guess that one? :D | 14:56 |
| Lennie | hehe | 14:57 |
| ajaksu | getRender: "Get rendering option for choice questions." | 14:57 |
| Lennie | I'm on stupid mode atm | 14:57 |
| ajaksu | (testing spike solution for db.Text) | 14:59 |
| Lennie | easy test case btw :D | 14:59 |
| Lennie | just type like crazy :P | 14:59 |
| Lennie | ajaksu: time to make patch notes for you ^^ | 15:02 |
| ajaksu | I think melange is stronger than my FF, which just crashed.... maybe a little too much typing :D | 15:04 |
| Lennie | rofmao! | 15:04 |
| ajaksu | Lennie: how about "HTTP response was too large: 18602605. The limit is: 10485760." ? :) | 15:06 |
| Lennie | ROFLMAO! | 15:06 |
| Lennie | you pasted way too much :P | 15:06 |
| Lennie | hahaha | 15:06 |
| Lennie | the awsome! | 15:06 |
| *** Shaan7 has quit IRC | 15:08 | |
| Lennie | ajaksu: your patch has been pushed | 15:08 |
| ajaksu | sweet :) | 15:08 |
| Lennie | how's db.text coming along? | 15:14 |
| Erant | You weird person. The moment you're on IRC, you're not on MSN, and vice-versa. | 15:14 |
| Lennie | hahaha | 15:14 |
| Lennie | I'm on mac | 15:14 |
| Erant | Can your poor MacBook nog handle more than two network threads? ;) | 15:14 |
| Lennie | MSN here sucs | 15:14 |
| Lennie | hehe | 15:14 |
| Lennie | ofcrouse | 15:14 |
| Erant | not* | 15:14 |
| Lennie | GTalk and MSN :P | 15:14 |
| Erant | Ohwow | 15:15 |
| Lennie | ERANT | 15:15 |
| Lennie | DUDE | 15:15 |
| Lennie | how's US :D? | 15:15 |
| Erant | Hot | 15:15 |
| Lennie | same here :) | 15:15 |
| Erant | Hotter | 15:15 |
| Lennie | in your new place? | 15:15 |
| Erant | Yah | 15:15 |
| Lennie | and? | 15:15 |
| Lennie | send pics plix | 15:15 |
| Erant | It's awesome | 15:15 |
| Lennie | I'll come visit :) | 15:15 |
| Erant | Pool, sauna, jacuzzi, etc. | 15:15 |
| Lennie | dude :P | 15:15 |
| Lennie | Why would I need a hotel :D | 15:15 |
| Erant | I know :) | 15:15 |
| ajaksu | Lennie: not that great, I'm not getting the clean_[field_id] method to be called for the dynamic props... | 15:15 |
| Lennie | I could just crash at your place :P | 15:15 |
| Erant | So I might go sit in the jacuzzi for a bit later. | 15:16 |
| Lennie | ajaksu: ouch that sucks :( | 15:16 |
| Erant | But the soon is absolutly hammering down right now | 15:16 |
| Erant | sun* | 15:16 |
| Erant | O_o | 15:16 |
| Lennie | Erant: hmm :) | 15:16 |
| Lennie | Erant: I'll visit your place when I'm over as well | 15:16 |
| Erant | Do that. | 15:16 |
| Lennie | Erant: and we'll go into town :) | 15:16 |
| Erant | Probably going to Santa Cruz tomorrow | 15:16 |
| Erant | We will. | 15:16 |
| Lennie | Erant: how was the flight? | 15:17 |
| Erant | Cupertino is boring. | 15:17 |
| Erant | Euh | 15:17 |
| Erant | Long. | 15:17 |
| Lennie | lets take this talk to pm ^_^ | 15:17 |
| Lennie | ajaksu: feel free to bother me with melange stuff again :p | 15:17 |
| ajaksu | Lennie: looks like overriding SurveyRecord's put to store long strings as TextProperty might work, testing... | 15:22 |
| Lennie | hmm | 15:23 |
| Lennie | that sounds nasty | 15:23 |
| Lennie | but there is no way that we can set the cleaner for any of those fields | 15:23 |
| Lennie | ? | 15:23 |
| ajaksu | Lennie: there might be, lemme look further... does this give you any ideas: 'If you want to validate a dynamic property value using a Property class, you can instantiate the Property class and call its validate() method on the value.' ? | 15:24 |
| ajaksu | (from expando's help) | 15:24 |
| Lennie | hmm | 15:24 |
| Lennie | db.text(value).validate() | 15:25 |
| Lennie | that's what they mean | 15:25 |
| Lennie | not too usefull | 15:25 |
| ajaksu | "Dynamic properties cannot use Property instances to perform validation, set defaults or apply automatic logic to values. Dynamic properties simply store values of the supported datastore types. See Types and Property Classes." http://code.google.com/appengine/docs/python/datastore/expandoclass.html | 15:25 |
| tpb | <http://ln-s.net/3S5r> (at code.google.com) | 15:25 |
| Lennie | yeah and text is a type? | 15:25 |
| Lennie | so you can still set it | 15:25 |
| Lennie | it just doesn't get auto-validated | 15:26 |
| ajaksu | hmm, how about overriding form's clean? | 15:26 |
| Lennie | I lied a bit with db.text.validate | 15:26 |
| Lennie | db.text is a datastory type | 15:26 |
| Lennie | TextProperty is the property | 15:26 |
| Lennie | ajaksu: if you can't get it to work with the fields :( | 15:27 |
| Lennie | so there is no way to add clean_id fields? | 15:29 |
| ajaksu | Lennie: I'm studying it further, found an article and was thinking the problem might be we define fields later in the game, so cleaning might need a hint to pick it up | 15:30 |
| Lennie | yes | 15:32 |
| Lennie | sounds sane | 15:32 |
| Lennie | I'm going to try and get the grade stuff working | 15:33 |
| Lennie | ajaksu: now there is only one place where a form get's initalized ^_^ | 15:59 |
| Lennie | ajaksu: I think I'm now going to let a method in the grading view hook into that and then it should work, "crosses fingers" | 16:00 |
| ajaksu | Lennie: cool, looking at the commit :) | 16:01 |
| Lennie | yeah I've added a TODO to fix the SurveyForm | 16:01 |
| Lennie | it should not try to confuse two sorts of data input | 16:01 |
| Lennie | if a record is specified then the record should be used | 16:02 |
| Lennie | if a post_dict is used | 16:02 |
| Lennie | then that should take precedence over the record | 16:02 |
| Lennie | and the record should NEVER be used then | 16:02 |
| ajaksu | not even if the user had some stored results and was adding something new? | 16:03 |
| Lennie | shouldn't that what he adds be in the post data ? | 16:03 |
| Lennie | shouldn't that to what he adds to also be in the post data | 16:03 |
| Lennie | * | 16:03 |
| Lennie | can you give me an example of where the behaviour you hinting at is needed? | 16:04 |
| ajaksu | say he had records for questions 1 and 2, now he's POSTing question 3... I think we need to merge both datasets, no? | 16:04 |
| *** Merio has joined #melange | 16:04 | |
| *** ChanServ sets mode: +v Merio | 16:04 | |
| Lennie | why? | 16:04 |
| Lennie | why isn't question 1 and 2 in the dataset? | 16:05 |
| Lennie | (in the POST data that is) | 16:05 |
| Lennie | they are also displayed on the form right? | 16:05 |
| Lennie | and that is what is being submitted? | 16:05 |
| ajaksu | Lennie: right, makes sense :) | 16:05 |
| * ajaksu should think before typing :) | 16:05 | |
| Lennie | mwah | 16:05 |
| Lennie | I can be wrong too :) | 16:05 |
| Lennie | just good to have someone else that thinks about the same subject as well | 16:06 |
| Lennie | for that matter I'm just dr__house I need someone to throw ideas off of | 16:06 |
| Lennie | :) | 16:06 |
| Lennie | little TV joke here :) | 16:07 |
| ajaksu | good, I'm a neverending source of ideas that need to be thrown away and I do know they deserve so (when pointed to why) :D | 16:08 |
| Lennie | ah | 16:09 |
| Lennie | we're not throwing all your idea away :D | 16:09 |
| Lennie | timeconstraints suck :P | 16:09 |
| Lennie | and there is a lot of your code in there as well :) | 16:09 |
| ajaksu | Lennie: don't worry, I'm really happy with throwing ideas away and working with consistent style... you have no idea how hard it was to juggle coding with bad designing before you got closer to surveys :) | 16:12 |
| Lennie | ah yes | 16:13 |
| Lennie | desperate times, desperate measures :) | 16:13 |
| Lennie | plot a course and go :p | 16:13 |
| ajaksu | Lennie: got 'a' solution to db.Text, but it's happening in survey_record logic... going upstream from there to see how close to the form I can get | 16:13 |
| Lennie | if prop instanceOf string :P | 16:14 |
| Lennie | len > 500 | 16:14 |
| Lennie | db.text | 16:14 |
| Lennie | something like that :P? | 16:14 |
| ajaksu | wow, you're doing a great dr house, yes, exactly that :) | 16:14 |
| Lennie | you watch gregory as well :)? | 16:15 |
| Lennie | The ep where he lured Cuddy into the xray chamber was awesome :P | 16:15 |
| Erant | Was that where the guy was unable to control what he said? | 16:16 |
| Lennie | yosh :D | 16:16 |
| Lennie | that one Erant | 16:16 |
| Erant | Seen it on the plane. | 16:16 |
| Lennie | hehe | 16:16 |
| Lennie | such a good joke that one :D | 16:16 |
| Erant | :P | 16:16 |
| ajaksu | occasionally, I don't have cable and don't watch much TV, but dr house is one of the things I watch when I can :) | 16:17 |
| dr__house | Lennie: pistons on the ferrari ;) | 16:17 |
| Lennie | :D | 16:19 |
| Lennie | ajaksu: House is awesome! | 16:19 |
| Lennie | ajaksu: ping! | 16:27 |
| ajaksu | Lennie: pong :) | 16:28 |
| Lennie | I need you to focus on something else :P | 16:28 |
| Lennie | pull the latest changes please | 16:29 |
| ajaksu | sure | 16:29 |
| ajaksu | ok :) | 16:29 |
| ajaksu | Lennie: got it | 16:32 |
| Lennie | ok | 16:33 |
| Lennie | I created a _getSurveyTakeForm | 16:33 |
| Lennie | GradingProjectSurvey view should overwrite this method | 16:33 |
| ajaksu | yup, returning surveys.SurveyTakeForm for now | 16:33 |
| *** madrazr1 has joined #melange | 16:33 | |
| Lennie | and retrieve the take form with the added grade field enabled | 16:33 |
| ajaksu | ah, ok, so override instead of extend :) | 16:33 |
| Lennie | yosh | 16:33 |
| *** madrazr has quit IRC | 16:33 | |
| *** madrazr1 is now known as madrazr | 16:34 | |
| Lennie | it should have all the normal survey fields | 16:34 |
| *** ChanServ sets mode: +v madrazr | 16:34 | |
| Lennie | but also include the grade field | 16:34 |
| Lennie | you can relocate the code currently in the ProjectGradingView to the helper if you'd wish | 16:34 |
| Lennie | it is basically an extension of the SurveyTakeForm :) | 16:34 |
| Lennie | (which means you might want to create a new subclass) | 16:35 |
| Lennie | or do something else that's nice | 16:35 |
| ajaksu | Lennie: ok, so goal is having the grading field appearing then stored, in a nice extension-based way... does that work for forms? :) | 16:36 |
| Lennie | I hope so :P | 16:36 |
| Lennie | it should atleast be possible to move the ProjectGradingSurvey code into the helper | 16:36 |
| Lennie | be it in a subclass that overrides getFields | 16:36 |
| *** dr__house has quit IRC | 16:36 | |
| Lennie | and just does the same work after it calls super.getFields | 16:37 |
| *** dr__house has joined #melange | 16:37 | |
| Lennie | but it should be possible somehow to take in a grade field at the end right? | 16:38 |
| ajaksu | Lennie: sure, I'm just thinking about how all the other fields come from surveycontent... and how easy things would be if grade was a simple field, but yes, it's possible :) | 16:39 |
| Lennie | hmm | 16:40 |
| Lennie | still feels like it should be possible to do it at the end | 16:45 |
| Lennie | we want it to come last anywya | 16:45 |
| ajaksu | working on it :) | 16:46 |
| Lennie | awesome! | 16:48 |
| Lennie | I'm conveying with madrazr about GHOP :) | 16:48 |
| Lennie | I'm going to get something to drink | 16:54 |
| Lennie | have you eaten yet ajaksu :)? | 16:55 |
| ajaksu | Lennie: some snacks 5h ago, time for a break... soon :) | 16:55 |
| Lennie | hehe | 16:55 |
| Lennie | breaks are important :) | 16:55 |
| madrazr | ajaksu: Oh! | 16:56 |
| madrazr | you seriously need a break! | 16:56 |
| Lennie | dr__house: you have to power to order people to take a break :p | 16:56 |
| madrazr | Lennie: was that said in reference to me? ;-) | 16:56 |
| Lennie | hmm | 16:57 |
| Lennie | no to ajaksu :P | 16:57 |
| Lennie | unless you are dr__house :D | 16:57 |
| madrazr | Lennie: ha ha ha :D | 16:57 |
| Lennie | but I thought that was your roomie/friend | 16:57 |
| madrazr | Lennie: he is my close friend :) | 16:57 |
| dr__house | Lennie: I have the power to ask people to go on a break? | 16:57 |
| Lennie | :) | 16:57 |
| Lennie | yes | 16:57 |
| Lennie | you are a doctor | 16:57 |
| madrazr | ROFL :D | 16:57 |
| madrazr | Lennie: I was actually waiting for a time to tell you how we get breaks here :D | 16:58 |
| madrazr | Lennie: it is also something related to power :P | 16:58 |
| madrazr | Lennie: power cuts for 6-7 hours during day time | 16:58 |
| madrazr | so forced breaks :P | 16:58 |
| Lennie | doctor > starship captain, Starfleet code section 12 paragraph 4 :) | 16:58 |
| Lennie | The Chief Medical Officer outranks the captain in health matters | 16:59 |
| madrazr | he he | 16:59 |
| Lennie | madrazr: you have power cuts a lot right? | 16:59 |
| madrazr | Lennie: simply too much :( | 17:00 |
| Lennie | sucks :0 | 17:00 |
| Lennie | you should buy an power storage device for your pc :) | 17:00 |
| Lennie | you can do 4 hours without power then | 17:00 |
| madrazr | Lennie: I am buying a lappy finally in a week or so :) | 17:01 |
| Lennie | :) | 17:01 |
| dr__house | madrazr: yeah like you are gonna get it this week :P | 17:02 |
| madrazr | dr__house: next week for sure! | 17:02 |
| dr__house | madrazr: this week ends tomorrow, and if you order tomorrow you might get it by 12-13 | 17:04 |
| Lennie | laptops only last for 2 hours on battery if you are lucky | 17:04 |
| Lennie | unless you get a macbook | 17:04 |
| Lennie | (praises his 6 hours of programming without power!) | 17:05 |
| madrazr | Lennie: he he, thats cool actually! | 17:05 |
| Lennie | but yes laptops are nice for uni as well :) | 17:05 |
| dr__house | Lennie: mine used to last me for 5-6 hours before my 9-cell battery died :( | 17:06 |
| Lennie | dr__house: a laptop or macbook? :) | 17:06 |
| Lennie | if it is a laptop then you had a good one D | 17:07 |
| dr__house | Lennie: laptop | 17:07 |
| dr__house | Lennie: Dell Inspiron 1525 | 17:07 |
| Lennie | nice :D | 17:08 |
| Lennie | +1 dr__house :) | 17:08 |
| Lennie | I'm off to take a drink now for real | 17:08 |
| Lennie | brb | 17:08 |
| dr__house | Lennie: yeah getting new batteries this week | 17:08 |
| dr__house | Lennie: sure :) | 17:08 |
| Lennie | :) | 17:16 |
| Lennie | so | 17:16 |
| Lennie | ajaksu: any ideas yet? :) | 17:16 |
| Lennie | dr__house: 9-cell ftw : | 17:16 |
| dr__house | yep :) | 17:17 |
| Lennie | dr__house: typically dell, did your battery blow up :P? | 17:17 |
| dr__house | Lennie: yeah, all of a sudden one day it says that the battery power has just 25 minutes on it | 17:17 |
| dr__house | :( | 17:17 |
| ajaksu | Lennie: the grade field is showing (was before I did anything other than pasting _getSurveyTakeForm in grading...survy view) | 17:17 |
| Lennie | yeah it showed :P | 17:18 |
| Lennie | because it was in _takeGet | 17:18 |
| Lennie | but it isn't supposed to use that for that purpose | 17:19 |
| ajaksu | Lennie: ok, so no, no ideas yet but working on it :) | 17:20 |
| Lennie | ok | 17:20 |
| Lennie | hmm | 17:25 |
| Lennie | nm stupid idea :D | 17:26 |
| Lennie | ajaksu: anything :D? | 17:42 |
| Lennie | ajaksu: Or should we discuss options? | 17:42 |
| ajaksu | Lennie: I can't generate a single grading_project_survey record, that's all I'm working with :/ | 17:43 |
| Lennie | oh | 17:44 |
| Lennie | I can take care of that | 17:44 |
| Lennie | as long as you can make the form show the field | 17:44 |
| Lennie | in get and post :) | 17:44 |
| Lennie | if the grade field is in there | 17:44 |
| Lennie | then I can take care of the rest | 17:44 |
| ajaksu | Lennie: duh, sorry then, fixing the field on post :) | 17:44 |
| Lennie | fixing field on post? | 17:45 |
| Lennie | hmm | 17:45 |
| Lennie | shouldn't it already be in there :P? | 17:45 |
| Lennie | are you using _getSurveyTakeForm ? | 17:45 |
| *** dhaun has quit IRC | 17:49 | |
| ajaksu | Lennie: sorry, forgot I turned off autohighlight for your nick... yes, I'm using it, and I'm going for a keyword to surveytakeform that adds the grade field | 17:59 |
| Lennie | ok | 18:01 |
| Lennie | hmm | 18:01 |
| Lennie | just subclass the form then | 18:01 |
| Lennie | makes it much cleaner? | 18:01 |
| Lennie | or isn't that possible?. | 18:01 |
| Lennie | I think in the future we'll be adding more required field surveys | 18:02 |
| ajaksu | Lennie: should be, now it works the ugly way let me try the nice way :) | 18:02 |
| Lennie | hehe | 18:02 |
| Lennie | it isn't that ugly | 18:03 |
| Lennie | I just wonder if it is possible | 18:03 |
| Lennie | we might want to extend the surveys to also work as application forms for organizations next year | 18:05 |
| Lennie | so we might be better of ensuring that we can add static fields :) | 18:06 |
| Lennie | because an org app form will have a bunch of them | 18:06 |
| ajaksu | yup :) | 18:07 |
| ajaksu | Lennie: it works! sorry for taking hours to make such a small improvement :( | 18:10 |
| Lennie | mwah :) | 18:10 |
| ajaksu | (ok, the solution to db.Text also occurred to me in the mean time :D) | 18:10 |
| Lennie | also good | 18:10 |
| Lennie | commit count )++ :) | 18:10 |
| Lennie | brb | 18:17 |
| Lennie | send it when ready :) | 18:17 |
| ajaksu | ok :) | 18:17 |
| Lennie | back | 18:27 |
| Lennie | so do you have a patch for me :D | 18:28 |
| Lennie | I want to get it in there so I can sleep :D | 18:28 |
| Lennie | and so Sverre can test the access checks tonight :) | 18:28 |
| ajaksu | Lennie: done diffing, sending :) | 18:30 |
| Lennie | awesome | 18:31 |
| Erant | Sleep? w0zzat | 18:31 |
| Lennie | something you shouldn't do yet | 18:31 |
| Lennie | mister jetlag :) | 18:31 |
| ajaksu | Lennie: sent | 18:32 |
| Lennie | k | 18:32 |
| *** dr__house has quit IRC | 18:33 | |
| Erant | Yeah, another good 8-9 hours. | 18:33 |
| Lennie | cool gonna take a look at it :) | 18:33 |
| Lennie | good luck with that Erant :p | 18:33 |
| Lennie | lol ajaksu why didn't you empty _takeGet :P | 18:35 |
| ajaksu | Lennie: because you still have to make taking work? :) | 18:36 |
| Lennie | might be true :P | 18:36 |
| Lennie | hmm | 18:46 |
| Lennie | the field does not get populated by request.POST | 18:47 |
| Lennie | (whishes Django was used :P) | 18:47 |
| ajaksu | getFields(post_dict=...? :) | 18:48 |
| Lennie | yes | 18:48 |
| Lennie | but the contents is not set :) | 18:48 |
| Lennie | because it is probably not in the schem | 18:48 |
| Lennie | a | 18:48 |
| Lennie | but it is in the form :P | 18:48 |
| ajaksu | I was thinking about that... maybe we should make the metaclass surveyrecord (so we'd have to pre-populate the record with the right fields from content)? | 18:50 |
| ajaksu | that would give us a mostly normal django form, from a weird origin but still :) | 18:50 |
| Lennie | sounds possible | 18:51 |
| *** madrazr has quit IRC | 18:52 | |
| ajaksu | so any new fields should be added to the blank record, leaving the form comfortable with being django :) | 18:52 |
| Lennie | hmm | 18:57 |
| Lennie | ajaksu: I'm too distracted :( | 18:57 |
| Lennie | 01:00 almost | 18:58 |
| Lennie | I'm heading to bed :P | 18:58 |
| Lennie | so close :( | 18:58 |
| ajaksu | Lennie: I know the feeling... I'll try to fix some other issues so you can start the day in a good mood :) | 18:59 |
| Lennie | or if you can fix the grade then you would start my day in a superb mood :P | 19:00 |
| Lennie | as long as it form gets through validatoin with grade field life is good | 19:00 |
| ajaksu | Lennie: ok, I'm closing in a db.Text solution and will be able to take a look at the grade field, but I'm starting to need some rest so no promises :) | 19:02 |
| Lennie | you are not the only one | 19:03 |
| Lennie | nn ajaksu | 19:03 |
| Lennie | was a good day | 19:04 |
| Lennie | thanks for the help | 19:04 |
| *** Lennie has quit IRC | 19:05 | |
| *** Merio has quit IRC | 20:29 | |
| *** SRabbelier has joined #melange | 21:26 | |
| *** ChanServ sets mode: +o SRabbelier | 21:26 | |
| SRabbelier | whow, it's quieter than ever in here | 21:27 |
| ajaksu | SRabbelier: wanna hack on surveys? :) | 21:29 |
| SRabbelier | ajaksu: yes, hence why I pinged you on gtalk :) | 21:29 |
| SRabbelier | at least... I think I did | 21:30 |
| SRabbelier | ajaksu: anyway, am working on the access check thingy | 21:30 |
| ajaksu | SRabbelier: looking at the code... | 21:31 |
| SRabbelier | ajaksu: access.py bottommost function | 21:32 |
| SRabbelier | ajaksu: http://code.google.com/p/soc/source/detail?r=f9119180c2947812cd549694e812c49876e35165 | 21:32 |
| tpb | <http://ln-s.net/3SGQ> (at code.google.com) | 21:32 |
| ajaksu | thanks :) | 21:32 |
| SRabbelier | ajaksu: np :) | 21:33 |
| ajaksu | SRabbelier: want me to test it, have any issues in mind? :) | 21:35 |
| SRabbelier | ajaksu: am addressing the comments on it now | 21:35 |
| SRabbelier | ajaksu: (and plug it in too) | 21:35 |
| * ajaksu realizes he's reimplemented (then discarded) isActivePeriod :D | 21:37 | |
| SRabbelier | ajaksu: lol :) | 21:37 |
| SRabbelier | ajaksu: you found it through my access check? | 21:37 |
| SRabbelier | ajaksu: I wish we had a more obvious and easy to use API | 21:37 |
| ajaksu | SRabbelier: yup... but to be honest, we might want to keep the different behavior for after-deadline (so user can see her record) | 21:38 |
| SRabbelier | ajaksu: ack | 21:38 |
| SRabbelier | ajaksu: would have to be different access check | 21:38 |
| SRabbelier | ajaksu: e.g., isSurveyResultReadable or something? | 21:39 |
| ajaksu | SRabbelier: might be a good choice, yes... I'll take a look at that once we have james's code for notifying menu in | 21:40 |
| SRabbelier | ajaksu: dude, seriously, you two are doing an awesome job, totally great! :) | 21:41 |
| SRabbelier | ajaksu: how's this measuring up to your initial goal of messing around with a GAE/Django/Python web app? :) | 21:42 |
| SRabbelier | ajaksu: I will bbl, buying nom | 21:44 |
| ajaksu | SRabbelier: this completely surpassed my expectations, been a wonderful ride so far! I'm not getting things done the right way as much as I'd like to (much better now that Lennie is around to give context on melange style) but man it's been so great :) | 21:44 |
| SRabbelier | ajaksu: sweet :D:D | 21:45 |
| SRabbelier | ajaksu: keep it up | 21:45 |
| SRabbelier | ajaksu: I'll bbl :) | 21:45 |
| ajaksu | SRabbelier: k, I'll be around :) | 21:46 |
| SRabbelier | ajaksu: pong | 22:40 |
| ajaksu | SRabbelier: err... ping? :) | 22:40 |
| SRabbelier | ajaksu: :P | 22:45 |
| SRabbelier | ajaksu: I'm back :) | 22:45 |
| SRabbelier | ajaksu: got some nommings | 22:46 |
| ajaksu | SRabbelier: I'm starting to need sleep, took me a minute to parse nommings as nom nom (usually I'd take less than... 45 secs? :D) | 22:47 |
| SRabbelier | ajaksu: hahah, how late is it over there? | 22:48 |
| ajaksu | SRabbelier: not too late, 23h48, but I'm staring at melange since 07:30 or so :) | 22:48 |
| SRabbelier | ajaksu: whow, you're dedicated dude :P | 22:49 |
| SRabbelier | ajaksu: did LH get you schwag yet? :P | 22:49 |
| ajaksu | SRabbelier: dedicated as in "it's my fail if we don't ship something acceptable"? is so, yes, I am :D | 22:50 |
| SRabbelier | ajaksu: yup :D | 22:50 |
| SRabbelier | ajaksu: that's about my definition of dedication :P | 22:50 |
| SRabbelier | ajaksu: so we should like, put on the melange blog: | 22:50 |
| SRabbelier | ajaksu: "surveys are not working yet, complain at ajaksu at gmail dot com"? ::P | 22:51 |
| ajaksu | SRabbelier: google swag as in 'promotional stuff'? do lots of free meals and a heck of a good time count as swag? :) | 22:51 |
| SRabbelier | ajaksu: yes, they do, :P | 22:52 |
| SRabbelier | ajaksu: do you at least have an OSPO shirt? :D | 22:52 |
| ajaksu | SRabbelier: yeah, guys on python-dev already know it: | 22:52 |
| ajaksu | [17:46:56] <Taggnostr> it could be better indeed, we should complain with ajaksu | 22:52 |
| SRabbelier | lolz :) | 22:52 |
| SRabbelier | s/with/at/ | 22:52 |
| SRabbelier | complain with == "complain together" :P | 22:52 |
| SRabbelier | ajaksu: unless you'll be complaining at yourself too xD | 22:53 |
| SRabbelier | ajaksu: in which case I guess it works :D | 22:53 |
| ajaksu | SRabbelier: I do have weird convos with myself on IRC, as I ofter re-read logs :D (hi me reading this in the future, did you get some sleep? why not get some more? you know you/I want it...) | 22:54 |
| ajaksu | *often | 22:54 |
| SRabbelier | LOL!!! | 22:54 |
| SRabbelier | s/ofter/often/ ? that's a weird typo, they're not even close by :P | 22:55 |
| ajaksu | I blame 're-read' :) | 22:57 |
| ajaksu | woot! fixed a nice bug :) | 23:05 |
| SRabbelier | ajaksu: a nice bug? :D | 23:23 |
| SRabbelier | isn't that a contradictio in termini? :P | 23:23 |
| ajaksu | SRabbelier: I didn't know its scientific name, are all nice bugs from genus Contradictio? :D | 23:27 |
| SRabbelier | ha! :P | 23:27 |
| SRabbelier | I forgot an s apparently | 23:27 |
| SRabbelier | http://nl.wikipedia.org/wiki/Contradictio_in_terminis | 23:27 |
| tpb | <http://ln-s.net/3SI6> (at nl.wikipedia.org) | 23:27 |
| SRabbelier | try this one instead | 23:28 |
| SRabbelier | http://en.wikipedia.org/wiki/Contradictio_in_terminis | 23:28 |
| tpb | <http://ln-s.net/3SI7> (at en.wikipedia.org) | 23:28 |
| SRabbelier | silly dutch search engine | 23:28 |
| ajaksu | SRabbelier: aha, then no, some bugs are nicer than others (says the issue tracker janitor) | 23:29 |
| SRabbelier | ajaksu: ehe, what defines a nice bug? | 23:30 |
| ajaksu | SRabbelier: it's a clear bug, that allows one to generally improve related code and has a report-confirm-test-patch-test-fix life cycle... now, don't ask me about nasty ones or I might give you a link :D | 23:33 |
| SRabbelier | ajaksu: hahaha, gotcha, the reproducable ones :) | 23:33 |
| ajaksu | SRabbelier: being deterministic as opposed to a heisenbug is nice, not making one think "we're all doomed, there's no way out of it" is kinda cool too :) | 23:36 |
| SRabbelier | ajaksu: hahah, yes, and debugging a web app is kinda hard :( | 23:38 |
| SRabbelier | ajaksu: lots of printfs :P | 23:38 |
| ajaksu | SRabbelier: yep, I got a nice remote debugger (in WingIDE) but I can't get it to work with GAE's auto reloading... still nice when it pinpoints an exception and opens a console to poke at variables | 23:39 |
| SRabbelier | ajaksu: ah, sweet, I don't have that :9 | 23:40 |
| SRabbelier | :( | 23:40 |
| ajaksu | SRabbelier: gf is kicking me out of the computer (thank heavens, I'm starting to get too sluggish), good night :) | 23:40 |
| SRabbelier | ajaksu: lol, good :0 | 23:40 |
| SRabbelier | ajaksu: have a good night :) | 23:40 |
| ajaksu | good night, Sverre :) | 23:44 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!