| cjohnston | mhall119: can you approve it and we will deal with the look later? | 00:48 |
|---|---|---|
| cjohnston | id like to get it packaged and released | 00:48 |
| cjohnston | or can i approve it | 00:48 |
| mhall119 | cjohnston: which, the twitter branch? | 01:14 |
| cjohnston | ya | 01:14 |
| cjohnston | I just merged it | 01:14 |
| cjohnston | do you want to throw it up on an ec2 to test before the release? | 01:14 |
| mhall119 | bah, another failing test | 01:14 |
| cjohnston | from danilos' branch? | 01:16 |
| mhall119 | let me get a clean trunk | 01:17 |
| mhall119 | nope, still not passing | 01:18 |
| mhall119 | yeah, test_calculate_passes_with_multiple_plenary_rooms_if_editing | 01:18 |
| mhall119 | danilos: ^^ | 01:18 |
| mhall119 | cjohnston: are we deploying on lucid still? | 01:19 |
| cjohnston | afaik | 01:19 |
| cjohnston | mhall119: https://code.launchpad.net/~mhall119/summit/add-lunch can you mark abandoned please | 01:21 |
| cjohnston | mhall119: can you throw together a test fix please | 01:30 |
| mhall119 | cjohnston: let me look | 01:32 |
| mhall119 | cjohnston: danilos: nigelb: http://91.189.93.80:8000/uds-p/ | 02:35 |
| cjohnston | add a hash tag! | 02:36 |
| cjohnston | show all the goodness | 02:36 |
| cjohnston | mhall119: http://91.189.93.80:8000/uds-p/meeting/584/private-meeting/ not sure whats up with that | 02:38 |
| cjohnston | i guess we need if not meeting.private url, else no url | 02:39 |
| mhall119 | it wasn't private | 02:39 |
| mhall119 | if it was private, import_live_data wouldn't see it | 02:39 |
| cjohnston | so it was just called private meeting | 02:40 |
| mhall119 | yeah | 02:40 |
| mhall119 | see http://summit.ubuntu.com/uds-p/2011-11-01/ | 02:40 |
| cjohnston | pl | 02:41 |
| cjohnston | ok | 02:41 |
| mhall119 | cjohnston: easy fix to the test, nothing was wrong withthe code | 02:53 |
| mhall119 | I just pushed the fix to trunk | 02:54 |
| cjohnston | ok | 02:54 |
| cjohnston | you good then with packaging it up and shipping it out? | 02:54 |
| mhall119 | yup | 02:54 |
| mhall119 | ask danilos when would be a good day and time to put it live | 02:54 |
| mhall119 | then we'll see if IS can get it done then | 02:54 |
| cjohnston | he has a deadline of tomorrow night | 02:54 |
| mhall119 | works for me | 02:55 |
| cjohnston | mhall119: http://pad.ubuntu.com/yNPpIWZRAo | 03:01 |
| mhall119 | I think that's all we need | 03:02 |
| mhall119 | is that what's in our typical deployment requests? | 03:03 |
| cjohnston | I don't remember there being anything else.. I can't find an old email | 03:03 |
| cjohnston | I looked through the ltp one, and the only thing im missing is the i18n stuff | 03:03 |
| mhall119 | yeah | 03:04 |
| mhall119 | specify the exact version number of the production branch that should be deployed | 03:05 |
| cjohnston | that? | 03:06 |
| cjohnston | does rollback need the whole file or just the #? | 03:08 |
| mhall119 | also, add just the number, it'll revert any migrations higher than that number | 03:08 |
| mhall119 | s/also, add// | 03:08 |
| cjohnston | so like it is | 03:09 |
| mhall119 | that looks good now | 03:09 |
| cjohnston | i want to know how many lines of code this is | 03:10 |
| mhall119 | bzr diff -r 161..162 | 03:10 |
| cjohnston | i dont have enough scrollback | 03:11 |
| mhall119 | bzr diff -r 161 --diff-options="--normal" |wc -l | 03:15 |
| cjohnston | mhall119: we should probably try to fix bug #917674 as its a problem this uds | 03:15 |
| ubot4` | Launchpad bug 917674 in summit "Longer room names are blocked by the schedule (affects: 1) (heat: 6)" [Medium,Confirmed] https://launchpad.net/bugs/917674 | 03:15 |
| mhall119 | 4300 lines | 03:15 |
| cjohnston | this connect | 03:15 |
| cjohnston | whatever | 03:15 |
| cjohnston | thats it? | 03:15 |
| cjohnston | lol | 03:15 |
| mhall119 | we only have 12,413 lines of .py code | 03:17 |
| mhall119 | and 2006 lines of .html | 03:18 |
| cjohnston | so we edited almost 1/3 of the code | 03:20 |
| nigelb | morning *yawn* | 03:20 |
| cjohnston | hey nigelb | 03:20 |
| mhall119 | yup | 03:21 |
| mhall119 | morning nigelb | 03:21 |
| mhall119 | ready to deploy summit? | 03:21 |
| cjohnston | ready to deploy your participant fix ;-) | 03:21 |
| mhall119 | cjohnston: be sure to cc danilos on the RT email when you send it | 03:21 |
| nigelb | lolno. | 03:21 |
| nigelb | I'm reading up on formsets | 03:22 |
| cjohnston | mhall119: i forgot, but i have access to add him once its triaged | 03:22 |
| mhall119 | ok | 03:44 |
| mhall119 | cjohnston: you good for tonight? | 03:49 |
| danilos | mhall119, cjohnston: the problem is likely with the test stability in test_calculate_passes_with_multiple_plenary_rooms_if_editing | 07:50 |
| danilos | nigelb, hi :) | 07:51 |
| danilos | nigelb, did you make any progress with the formsets? If not, I'll need to look into that as well to ensure we can get it done by today | 08:17 |
| nigelb | danilos: hey, sorry, not yet. I'm tryign to see if I can spare some time at work today. | 08:23 |
| danilos | nigelb, right, do you have a branch I can start off or should I just start from trunk? | 08:23 |
| nigelb | danilos: I don't have anything productive, no | 08:23 |
| danilos | nigelb, ack, thanks, I'll also keep you posted if I make any progress | 08:24 |
| nigelb | ugh | 08:26 |
| nigelb | who broke summit production | 08:27 |
| nigelb | cjohnston: ^^^ | 08:27 |
| danilos | nigelb, probably a DB update not done, I'll check with IS | 08:35 |
| danilos | nigelb, btw, just adding 'participants' to the list of form fields is a minimal "add attendees" functionality as far as I can see, other than it being hard to use (Ctrl+clicking all around), any other reason why that was not considered? | 08:36 |
| danilos | ah, I see, saving doesn't really work | 08:37 |
| nigelb | yep | 08:37 |
| nigelb | :) | 08:37 |
| nigelb | I fell into the exact same trap | 08:37 |
| cjohnston | nigelb: dont know.. can you look into it | 10:58 |
| danilos | cjohnston, nigelb, heya, do you know where can we take a look at errors from the OOPSes? | 10:59 |
| danilos | cjohnston, nigelb: I am getting changes reverted, hopefully nothing blows up in that process either | 10:59 |
| danilos | cjohnston, nigelb: https://bugs.launchpad.net/summit/+bug/921502 | 11:00 |
| ubot4` | Launchpad bug 921502 in summit "Problems with latest deployment (affects: 1) (heat: 6)" [Undecided,New] | 11:00 |
| cjohnston | http://paste.ubuntu.com/816311/ | 11:00 |
| cjohnston | danilos: ^ | 11:00 |
| danilos | cjohnston, sounds similar | 11:01 |
| cjohnston | danilos: there are ~100 errors that have been spent out | 11:02 |
| cjohnston | the few ive looked at are pretty much the same as that one | 11:02 |
| cjohnston | all that no reverse match | 11:02 |
| danilos | cjohnston, right, I wonder why is this not failing locally | 11:02 |
| danilos | cjohnston, it originally 500-ed with the same error which I had a fix for cowboyed in, but then it started failing with an OOPS page | 11:03 |
| cjohnston | danilos: i assume you arent having any problems locally? | 11:06 |
| danilos | cjohnston, not with trunk, no | 11:06 |
| cjohnston | try production | 11:07 |
| danilos | cjohnston, haven't compared production | 11:07 |
| cjohnston | just incase its there please | 11:07 |
| cjohnston | I have to get the kids ready for school and leave for work in 20 minutes | 11:07 |
| cjohnston | im going to help as much as i can till then, but I dont know how much thatll be | 11:07 |
| cjohnston | one of the rrors AttributeError: 'NoneType' object has no attribute 'strftime' | 11:09 |
| cjohnston | http://paste.ubuntu.com/816320/ | 11:09 |
| cjohnston | i just fired up production locally and it works | 11:10 |
| cjohnston | danilos: its something specific with the summit page too it looks like... | 11:13 |
| cjohnston | .http://summit.ubuntu.com/lcq1-12/2012-02-09/ | 11:13 |
| cjohnston | nigelb: ^ | 11:13 |
| danilos | cjohnston, right, I don't know what even references the missing decorator since I don't see it when grepping, and it might be specific to wsgi setup which dev setup is not using | 11:15 |
| danilos | cjohnston, and locally it works | 11:15 |
| cjohnston | danilos: did they touch the wsgi files after pushing your latest fix? | 11:16 |
| danilos | cjohnston, nope, not that I know of | 11:16 |
| cjohnston | danilos: maybe try that? | 11:16 |
| danilos | cjohnston, also, I only asked for it to be cowboyed in | 11:16 |
| cjohnston | thats fine.. | 11:16 |
| danilos | cjohnston, that didn't help | 11:23 |
| cjohnston | danilos: g2g.. I'll be back as soon as I can | 11:24 |
| danilos | cjohnston, do you see any of the last OOPSes, I'd like to make sure I am looking at something after the change? | 11:24 |
| cjohnston | AttributeError: 'NoneType' object has no attribute 'strftime' | 11:25 |
| cjohnston | I forwarded you emails of the only different errors i can find | 11:26 |
| danilos | cjohnston, thanks | 11:34 |
| cjohnston | danilos: I'm curious whats up with the datetime thing | 11:37 |
| danilos | cjohnston, it just means the value is none | 11:37 |
| cjohnston | why tho | 11:37 |
| cjohnston | danilos: email | 11:41 |
| danilos | cjohnston, what worries me most is the fact that "decorators" appears in all the exceptions (except the date one) | 11:43 |
| cjohnston | dunno | 11:45 |
| danilos | actually, this might explain it, there are probably different templates in use on production instance | 11:45 |
| cjohnston | shouldn't be | 11:46 |
| cjohnston | is summit.schedule.decorators.meetinng normal or is that what you were referring to | 11:50 |
| danilos | cjohnston, I don't know if it's normal, but it seems as if it should be referring to something else (i.e. summit.schedule.views.meeting) | 11:53 |
| cjohnston | agreed | 11:53 |
| danilos | cjohnston, hum, hum, I wonder if the old base.html is still there and is seen before the newly moved base.html in common/ | 11:57 |
| cjohnston | do you have access to the server | 11:57 |
| danilos | cjohnston, nope, I have to go through IS for this | 11:58 |
| cjohnston | gotcha | 11:58 |
| cjohnston | try pinging I guess | 11:58 |
| cjohnston | says no vanguard tho | 11:59 |
| cjohnston | they bzr pull, could that leave an old one | 12:00 |
| danilos | nope, I've checked that, it's all fine | 12:08 |
| danilos | so no old templates are around | 12:08 |
| danilos | and this is obviously not from the old templates, I wonder what else could try to pull "summit" and "meeting" from summit.schedule.decorators | 12:08 |
| danilos | mhall119, ^ (when you show up) | 12:08 |
| cjohnston | danilos: are the imports on common.views correct | 12:15 |
| cjohnston | id think we would have local errors if it wasn't | 12:16 |
| danilos | cjohnston, yeah, also, note that every question requires some time for IS to respond | 12:17 |
| danilos | cjohnston, I think I'll re-ask for the reversal of the last landing, I want to focus on getting attendees adding working and then worry about deploying it all together | 12:18 |
| cjohnston | k | 12:19 |
| mhall119 | ok, what happened? | 12:59 |
| mhall119 | gah, why are reverse lookups failing? | 13:00 |
| cjohnston | mhall119: thoughts? | 13:12 |
| cjohnston | danilos: and i have both tried production locally and cant reproduce | 13:12 |
| cjohnston | he did already put in one fix | 13:12 |
| danilos | mhall119, I am not sure why would something try to get at summit.schedule.decorators.*, but it does and then fails | 13:14 |
| danilos | back-resolving URLs at least | 13:14 |
| danilos | it's fine to go there for decorators | 13:14 |
| danilos | but not for decorators.summit and decorators.meeting | 13:14 |
| danilos | mhall119, fwiw, if you've got time to look into this now, I was talking to ChrisS from IS (so he should be on top of things and might be in the process of reverting everything back) | 13:15 |
| cjohnston | there was an error with reverting back too | 13:16 |
| cjohnston | hes on lunch now | 13:16 |
| danilos | mhall119, (I am working on getting private attendees working) | 13:16 |
| cjohnston | http://paste.ubuntu.com/816407/ | 13:17 |
| mhall119 | cjohnston: danilos: using @summit_required decorator makes django think the view function exists in the decorators module (since it's wrapped by a decorator) | 13:23 |
| cjohnston | gotcha | 13:24 |
| cjohnston | is that whats causing the issue? | 13:24 |
| mhall119 | I don't think so | 13:26 |
| mhall119 | or it would cause it on localdev and staging too | 13:26 |
| mhall119 | http://summit.ubuntu.com/uds-p/2011-10-31/ works fine | 13:27 |
| mhall119 | so it can't be something from the base.html | 13:27 |
| mhall119 | somthing in summit.html is the problem | 13:27 |
| mhall119 | and also in meeting.html | 13:28 |
| cjohnston | ya | 13:30 |
| cjohnston | but what | 13:30 |
| mhall119 | that I don't know | 13:36 |
| mhall119 | oh hell, I bet I know what it is... | 13:42 |
| mhall119 | cjohnston: are they rolling it back or what? | 13:43 |
| cjohnston | mhall119: there was an error with the roleback | 13:43 |
| cjohnston | what is it | 13:43 |
| cjohnston | if you can come up with a fix, we wont rollback | 13:43 |
| mhall119 | using schedule.views.summit instead of summit.schedule.views.summit | 13:43 |
| cjohnston | mhall119: ill get them to change it and see | 13:45 |
| mhall119 | I'm making the changes locally | 13:46 |
| mhall119 | you want a patch or something they can merge into their production branch | 13:46 |
| cjohnston | yup | 13:49 |
| cjohnston | mhall119: somethings wrong with the css tho | 14:14 |
| cjohnston | danilos: its working | 14:14 |
| danilos | cjohnston, cool, thanks | 14:18 |
| mhall119 | cjohnston: please merge my branch into production and trunk | 14:24 |
| cjohnston | yup | 14:24 |
| cjohnston | already done | 14:24 |
| mhall119 | thanks | 14:32 |
| danilos | mhall119, cjohnston: btw, what happened with the colors on the wide display? also, if I am asked if we can get the wide display to be the default for Connect coming in 2 weeks, how easy would that be? | 15:27 |
| cjohnston | you dont want users to see the daily display? | 15:28 |
| mhall119 | danilos: just put /display at the end of the daily view | 15:29 |
| mhall119 | danilos: colors are defined in the Track record in the admin | 15:29 |
| cjohnston | danilos: the colors for the meetings? the tracks have to have colors defined | 15:29 |
| danilos | cjohnston, mhall119: I figured /display out (had to set connect.linaro.org to load proper URLs since they have changed), but if somebody (eg. Stephen) wants the old default back, I just wonder what should I say (because we are so close to the actual Connect, and the UI has changed significantly) | 15:31 |
| cjohnston | danilos: in order to see your private meetings you have to view the display | 15:31 |
| cjohnston | the daily | 15:31 |
| cjohnston | sorry | 15:31 |
| mhall119 | ^^ | 15:32 |
| danilos | thanks for the tips about colors | 15:32 |
| cjohnston | danilos: im going to start working on the layout issues on the linaro side | 15:32 |
| danilos | cjohnston, oh, so they don't show in the wide display for a day? | 15:32 |
| mhall119 | danilos: no | 15:32 |
| danilos | ok | 15:32 |
| mhall119 | they show as "Private Meeting" with no link or details | 15:32 |
| cjohnston | danilos: on the wide all it will say is Private Meeting, but you wont know who or where or what | 15:32 |
| mhall119 | and getting that working in render.py will be a nightmare | 15:33 |
| mhall119 | besides, the wide view horrible for desktop use anyway | 15:33 |
| danilos | mhall119, well, imho, it has its uses, I agree it's not perfect for looking at "your own" schedule | 15:34 |
| mhall119 | danilos: if they have a serious issue with the new schedule, I'll of course do what we can to help | 15:36 |
| mhall119 | but if it's just "omg change!", that's going to be hard to justify | 15:36 |
| danilos | btw, where did the BP link go? maybe it's just epiphany rendering issue | 15:36 |
| mhall119 | danilos: it's not in display anymore, it's on the meeting page | 15:36 |
| danilos | mhall119, well, if it's "omg change 10 days before Connect!", I'd call it a little bit more founded :) | 15:36 |
| mhall119 | display is just that, for display | 15:36 |
| mhall119 | danilos: in that case we can roll back to the previous production version until after connect | 15:37 |
| danilos | mhall119, right, and lose the private meeting stuff :) so, I won't speculate anymore and I'll hope nobody complains | 15:38 |
| mhall119 | me too :) | 15:38 |
| mhall119 | danilos: honestly if it's a deal breaker we can work something in | 15:38 |
| danilos | mhall119, btw, it'd be nice if BP links worked in the edit interface, I used to use the "display" interface to subscribe to sessions I care about by loading the blueprints directly from that one page | 15:38 |
| mhall119 | danilos: we can add it to the new schedule table | 15:39 |
| danilos | mhall119, yeah, I am not worried about that too much, just trying to be prepared for potential questions | 15:39 |
| danilos | mhall119, it would be nice if we can make the hover pop-up there stick and have link to a blueprint and similar | 15:40 |
| danilos | anyway, back to getting private attendees working: I got it mostly working, I just need to restrict the attendees to those for that particular summit | 15:41 |
| mhall119 | danilos: they'll "stick" as long as your mouse is over it | 15:42 |
| mhall119 | we can either add the blueprint link in the pop-up, or next to the meeting name in the table | 15:42 |
| cjohnston | mhall119: we need to restrict the people in managers and schedulers to attendees.. its way too long | 15:43 |
| danilos | mhall119, heh, I guess that depends on the font size and browser rendering, since there seems to be some space between it and session name for me and it disappears before I can move the mouse on top of it | 15:43 |
| mhall119 | cjohnston: we can just order them by username | 15:43 |
| mhall119 | danilos: ok, it's a pretty simple thing, we can introduce a jquery-ui widget in it's place in a future release | 15:44 |
| cjohnston | its still a gazillion people to try to pick 20 | 15:44 |
| mhall119 | cjohnston: technically it shouldn't be you that has to do it | 15:45 |
| cjohnston | already got a complaint | 15:45 |
| mhall119 | from who? | 15:47 |
| cjohnston | arwen | 15:48 |
| mhall119 | about setting managers? | 15:49 |
| mhall119 | cjohnston: you should have access to leads in the admin now | 15:51 |
| cjohnston | ty | 15:51 |
| === james_w` is now known as james_w | ||
| cjohnston | danilos: https://code.launchpad.net/~chrisjohnston/ubuntu-community-webthemes/linaro-fixes/+merge/90165 | 17:58 |
| cjohnston | We need to get those changes into production to unbreak the linaro side | 17:58 |
| danilos | cjohnston, looking, I'll have the attendee stuff shortly as well | 18:00 |
| cjohnston | awesome | 18:00 |
| danilos | cjohnston, looks good to me as well, nothing beats trying it out though :) | 18:09 |
| cjohnston | I'm not entirly sure how to try it, other than just replace ubuntu_website with it | 18:09 |
| cjohnston | danilos: im testing it locally | 18:19 |
| danilos | cjohnston, right, cool | 18:19 |
| cjohnston | danilos: everything looks good | 18:21 |
| danilos | cjohnston, great | 18:21 |
| cjohnston | mind merging it and asking IS to update the branch? | 18:21 |
| danilos | cjohnston, I am hoping to be done with my branch soon, so I'd rather focus on that still, and then get it all merged | 18:24 |
| cjohnston | ok | 18:24 |
| danilos | cjohnston, also, I don't have access to merge it anywhere myself | 18:24 |
| cjohnston | they are seperate branches and seperate process | 18:24 |
| cjohnston | processes | 18:24 |
| danilos | cjohnston, ah, right | 18:25 |
| cjohnston | your the owner to the team that is the owner of the branch | 18:25 |
| danilos | cjohnston, sure, I'll get that updated as well | 18:25 |
| cjohnston | :-) | 18:25 |
| cjohnston | ty | 18:25 |
| cjohnston | the private meeting page will thank you | 18:25 |
| cjohnston | lol | 18:25 |
| danilos | cjohnston, nigelb_, mhall119: https://code.launchpad.net/~danilo/summit/private-attendees/+merge/90183 | 18:41 |
| danilos | unfortunately, I didn't have any time to get the tests done (or even started) | 18:41 |
| cjohnston | danilos: ill test it | 18:43 |
| danilos | cjohnston, thanks, I've tested it as well, but I'll appreciate someone looking it over as well | 18:43 |
| cjohnston | ok | 18:45 |
| cjohnston | danilos: mind testing my two https://code.launchpad.net/summit/+activereviews | 18:45 |
| cjohnston | they are both just cleanup items that since we are going to push a release we might as well | 18:45 |
| danilos | cjohnston, LP is peculiarly slow for me now, pages loading... | 18:47 |
| cjohnston | hrm | 18:47 |
| cjohnston | danilos: so the linaro theme is good to go, just needs to be asked of IS | 18:48 |
| danilos | cjohnston, I also wonder if we need to update the theme or what for the top bar to point to lcq1-12 and not uds-q | 18:49 |
| cjohnston | danilos: i was trying to figure out how to do that | 18:49 |
| cjohnston | danilos: maybe pull the links out of base.html and put them into a header.html file? | 18:50 |
| cjohnston | danilos: are these particpants being marked as required? | 18:51 |
| danilos | cjohnston, yes | 18:52 |
| cjohnston | what did you mean about the displaying? | 18:52 |
| cjohnston | dude.. it looks awesome | 18:53 |
| cjohnston | mhall119: do you want to test it out? | 18:53 |
| cjohnston | ive created a meeting and added people, then gone back and edited the meeting, removed some of the initial people, and added some more people | 18:53 |
| mhall119 | cjohnston: if you're happy, I'm happy | 18:53 |
| cjohnston | ok | 18:54 |
| mhall119 | cjohnston: can we use his code as the basis for moving more stuff out of the admin? | 18:54 |
| cjohnston | mhall119: i think so | 18:55 |
| mhall119 | cool | 18:55 |
| cjohnston | danilos: im going to work on some help text too | 19:01 |
| danilos | cjohnston, I meant that I don't see on the regular meeting page who the attendees are, even if I am an admin or a participant | 19:02 |
| cjohnston | i saw it | 19:02 |
| cjohnston | below the pad | 19:02 |
| cjohnston | danilos: do you want us to add in the ability to define what etherpad to use for a summit? that way pad.l.o can be used? | 19:03 |
| danilos | cjohnston, yes, please! | 19:03 |
| cjohnston | ok | 19:04 |
| cjohnston | give me a bit and ill have a couple more MPs for review | 19:04 |
| knome | ministers of parliament? | 19:04 |
| danilos | cjohnston, ah, ok, just me being blind :) | 19:04 |
| danilos | knome, nope, Municipality Police (Officers) | 19:04 |
| knome | he | 19:05 |
| danilos | cjohnston, to get https://code.launchpad.net/~chrisjohnston/summit/921663/+merge/90185 to work I had to do one simple change (i.e. add ".count" or the if condition always passed) | 19:05 |
| cjohnston | hrm.. ok | 19:06 |
| cjohnston | it was workin fine for me | 19:06 |
| danilos | cjohnston, right, I don't know enough of the django templating to explain it, but without .count it didn't work (it showed all the empty "Lead:" lines on the tracks page), with it, it did (it should also be faster I suppose, though still far from optimal approach for rendering a page) | 19:08 |
| mhall119 | danilos: without .count, the variable you're testing is an instance of RelatedModelManager | 19:10 |
| mhall119 | as a non-None object, it will resolve to True in a logic test | 19:10 |
| danilos | mhall119, right, I assume something like that, I wonder why did it work for cjohnston in his testing | 19:10 |
| mhall119 | wait, having .all should accomplish the same thing | 19:11 |
| mhall119 | you don't need .all if you have .count | 19:11 |
| danilos | mhall119, I wonder if there's something faster than count() for django ORM (something like .any() in Storm) | 19:11 |
| mhall119 | .all returns (IIRC), a QuerySet instance | 19:11 |
| cjohnston | nigelb_: could you please smack tarmac | 19:11 |
| mhall119 | danilos: .count() in django ORM does the count in the SQL, so it's as fast as we can get | 19:12 |
| danilos | mhall119, you can be faster, you can do COUNT LIMIT 1, that's what Storm does ;) | 19:12 |
| mhall119 | unless you have .all() before .count(), in which case it's already loaded all the results | 19:12 |
| mhall119 | but COUNT without GROUP BY will always return just one result | 19:13 |
| knome | count(*) ? | 19:13 |
| danilos | mhall119, no, no, I mean you can LIMIT 1 the results that you are counting | 19:13 |
| mhall119 | oh, so count is 0 or 1, I gotcha | 19:13 |
| mhall119 | you can try track.lead_set.all.0 | 19:14 |
| mhall119 | that doesn't have the count, but should do LIMIT 1 | 19:14 |
| danilos | mhall119, right, that was just me being curious | 19:15 |
| danilos | cjohnston, anyway, I get just .all() to work now as well, maybe it's some browser caching on my side, sorry | 19:15 |
| mhall119 | danilos: it's not like it matters, since we're going to step through and display all the results anyway | 19:15 |
| danilos | mhall119, indeed | 19:16 |
| mhall119 | in fact, using .all in the if will have django cache the results for the inner for loop | 19:16 |
| mhall119 | so only one query | 19:16 |
| mhall119 | where as with .count it would be 2 | 19:16 |
| danilos | mhall119, yep, I was mostly worried that it didn't work, and it started working when I introduced the count, if we are to worry about performance, we shouldn't do a single query for each track :) | 19:16 |
| mhall119 | ah, the jfun of django internals | 19:17 |
| danilos | anyway, let me get the theme updated | 19:17 |
| danilos | mhall119, cjohnston: if you are fine with the private attendees change, can you perhaps arrange for it to be updated? | 19:19 |
| danilos | mhall119, cjohnston: btw, what was the problem with it in the morning? | 19:19 |
| cjohnston | schedule.views.something instead of summit.schedule.views.something | 19:20 |
| cjohnston | danilos: as soon as tarmac comes around it will merge the MP | 19:20 |
| cjohnston | I'm working right now on the help text | 19:20 |
| cjohnston | then ill go back and fix my track lead thing | 19:20 |
| cjohnston | then ill add the etherpad thing | 19:20 |
| cjohnston | then ill make someone do a bunch of reviews | 19:20 |
| cjohnston | then ill make it all go live | 19:21 |
| danilos | cjohnston, as mhall119 noted, nothing seems to need to change for the track lead thing | 19:24 |
| cjohnston | ok.. so then it can be merged as is? | 19:25 |
| cjohnston | i missed that convo | 19:25 |
| danilos | cjohnston, yes, it can | 19:25 |
| cjohnston | ok | 19:26 |
| cjohnston | mhall119: anything against renderablemixin? | 19:37 |
| danilos | cjohnston, I'll have to drop out in a minute or two, I'll come back online from home, but I'd appreciate if you guys can get the stuff merged into production branch and file an RT CCing me as well :) if not, I can do at least the part regarding the RT, but I can't land stuff to production branch | 19:41 |
| cjohnston | will do | 19:41 |
| cjohnston | :-) | 19:41 |
| danilos | cjohnston, thanks | 19:42 |
| cjohnston | danilos: still here | 19:43 |
| cjohnston | mhall119: do we want participants on private meetings to be required, or leave it unrequired so that they can edit it later | 19:44 |
| cjohnston | danilos: ^ | 19:46 |
| mhall119 | cjohnston: can we put the required flag on the form? | 20:03 |
| cjohnston | it currently is required | 20:04 |
| mhall119 | cjohnston: I don't have a problem with bringing renderablemixin to summit | 20:04 |
| mhall119 | cjohnston: that would seem a reasonable default to me, but why can't we have the checkbox like in the admin? | 20:04 |
| cjohnston | im trying to figure out how to make the page show it as required | 20:04 |
| cjohnston | no.. | 20:04 |
| cjohnston | requiring adding a participant | 20:04 |
| cjohnston | not marking a participant as required | 20:04 |
| mhall119 | oh, I understand now | 20:05 |
| mhall119 | I'd say no | 20:05 |
| cjohnston | hrm | 20:05 |
| cjohnston | i agree | 20:05 |
| cjohnston | i gotta figure out how | 20:05 |
| mhall119 | they're not very useful without a participant, but maybe they need to reserve a room for a hour to meet with someone not in LP or Summit | 20:05 |
| cjohnston | thats what im thinking | 20:06 |
| cjohnston | or if they dont know who will be involved | 20:06 |
| cjohnston | i think im gonna need some help mhall119 | 20:06 |
| mhall119 | does danilos have time? | 20:07 |
| mhall119 | I've got some catching up to do with my actual work | 20:07 |
| cjohnston | he is headed home | 20:09 |
| mhall119 | if it's not a bid deal for connect, we can fix it for uds-q | 20:09 |
| cjohnston | its missing a lable | 20:10 |
| cjohnston | label | 20:10 |
| cjohnston | for participant | 20:10 |
| cjohnston | s | 20:10 |
| cjohnston | i can wait for danilos | 20:11 |
| cjohnston | danilos: when you get back let me know please.. i requested that the theme be updated too | 20:26 |
| danilos | cjohnston, I've requested that earlier, lamont tells me it should be done, can you please confirm? | 20:30 |
| cjohnston | danilos: doesnt seem to be | 20:31 |
| cjohnston | there is now an RT for it | 20:31 |
| danilos | cjohnston, what in particular did you expect to change? | 20:32 |
| cjohnston | http://summit.linaro.org/lcq1-12/create_pm/ | 20:32 |
| cjohnston | http://summit.linaro.org/lcq1-12/2012-02-06/ | 20:32 |
| cjohnston | both should look nicer | 20:32 |
| danilos | that all looks fine to me (and it did earlier as well), let me try a different browser | 20:32 |
| cjohnston | im on chromium and it looks bad | 20:33 |
| cjohnston | it should pretty much match summit.u.c | 20:33 |
| danilos | cjohnston, it does for me, other than the footer | 20:34 |
| cjohnston | hrm | 20:34 |
| danilos | looks the same in chromium as well | 20:34 |
| danilos | then again, http://summit.linaro.org/media/css/ doesn't list the newly added 960.css | 20:38 |
| danilos | cjohnston, but neither does it from summit.ubuntu.com | 20:38 |
| cjohnston | danilos: does it look like http://ubuntuone.com/16tDRt3MhIBAz2sWeiR6l4 | 20:38 |
| danilos | cjohnston, ok, I was looking at the wrong path, it's all there: http://summit.linaro.org/ubuntu-website/media/css/ | 20:39 |
| cjohnston | hrm | 20:39 |
| danilos | cjohnston, nope, it looks like people.linaro.org/~danilo/summit/linaro-css.png | 20:41 |
| cjohnston | i wonder whats wrong with my browser | 20:42 |
| cjohnston | ive even cleared cache | 20:42 |
| cjohnston | oh well | 20:42 |
| danilos | cjohnston, you need to buy yourself a newer version, that one is broken :) I can sell you mine cheap | 20:43 |
| cjohnston | lol | 20:43 |
| cjohnston | danilos: can you fire up a branch that has your new attendee stuff, and merge in lp:~chrisjohnston/summit/this-adds-help-text | 20:44 |
| danilos | cjohnston, it'll take me a while to get everything set up on the laptop here, but sure | 20:45 |
| cjohnston | ok | 20:45 |
| danilos | cjohnston, actually, pulling the data for the sprint from LP and live instance is going to take a bit | 20:46 |
| cjohnston | danilos: i can give you a db | 20:46 |
| cjohnston | danilos: http://ubuntuone.com/0NNoVKVtXBx7l9jap6i3WG | 20:47 |
| danilos | cjohnston, thanks | 20:48 |
| cjohnston | danilos: its admin/password to get into the admin area, then you will have to add staff and can change agenda, or can change schedule, i forget which it is, to your user after you login for the first time | 20:51 |
| cjohnston | let me know when you get all that done please | 20:53 |
| danilos | cjohnston, I did get it all done | 20:58 |
| cjohnston | ok... | 20:58 |
| cjohnston | go to the create private meeting form in the ui please | 20:58 |
| danilos | cjohnston, I am on it, I can see the help icons, but the "Participants" label is gone | 20:59 |
| cjohnston | thats one of the two things i need help with | 20:59 |
| cjohnston | figure out how to get that back | 21:00 |
| cjohnston | and we want to make it not required to add a participant | 21:00 |
| danilos | cjohnston, that's not the right "required" param you are changing, that one is participation_essential basically | 21:03 |
| cjohnston | the one that i changed to false? | 21:04 |
| cjohnston | ok.. changed that back | 21:04 |
| danilos | cjohnston, http://paste.ubuntu.com/816981/ are the changes required to get the label back and make the field not be required | 21:07 |
| danilos | (the second part is restoring that required value that you already changed back) | 21:09 |
| danilos | cjohnston, I'm about to leave the computer, is there anything else I can help you with? | 21:21 |
| cjohnston | i just made an MP | 21:21 |
| cjohnston | thats it | 21:21 |
| cjohnston | it needs a review, I'll do the etherpad thing and then we will be ready to release | 21:21 |
| danilos | cjohnston, ack, is there someone else who can review it or do you want me to do it? | 21:22 |
| cjohnston | mhall119: might be able to later | 21:22 |
| cjohnston | not sure | 21:22 |
| danilos | cjohnston, for my personal sense, including so much new JS and CSS is not worth it for the differently colored tooltips, but that's just me :) | 21:25 |
| danilos | cjohnston, I won't look at those since I assume you just copied them directly | 21:26 |
| cjohnston | its all copied from loco.ubuntu.com | 21:26 |
| cjohnston | danilos: we are wanting to move 99% of the stuff out of the admin area, so they need to come at some point anyway | 21:27 |
| cjohnston | and if i don't do it now, i gotta figure out how to style to help text | 21:27 |
| danilos | cjohnston, right, I approved the code changes, I haven't looked at the JS nor CSS :) | 21:27 |
| cjohnston | ty | 21:28 |
| danilos | cjohnston, I reviewed the other outstanding MP you had as well | 21:31 |
| cjohnston | ty | 21:31 |
| danilos | cjohnston, I am off now, I hope I'll wake up to a fully updated production :P | 21:32 |
| danilos | have a nice day | 21:32 |
| cjohnston | have a good night | 21:32 |
| mhall119 | cjohnston: the staging box is updated to the latest trunk | 21:34 |
| cjohnston | cool | 21:43 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!