balloonsok https://code.launchpad.net/~nskaggs/autopilot/page-object-docs/+merge/246504 is ready now as well00:30
* veebers looks00:30
veebersballoons: Can you top approve this if you like (you're the 2nd reviewer): https://code.launchpad.net/~veebers/autopilot/adding-run-instructions-readme/+merge/24708800:32
balloonsveebers, I'd love too, but I have comments to change it . .00:34
veebersballoons: heh, even better :-)00:35
veebersthanks, will address soon00:36
balloonsoops, I realized I lied veebers  about docs/tutorial/running_ap.rst.. ignore that comment00:37
veebersballoons: hey, I get an error when building the docs of yours, also, not sure where to find the link to the content in the generated docs00:39
balloonsveebers, ohh which docs error? and I'll find the link one sec00:39
veebersballoons: this is the error I get:   ../page-object-docs/docs/tutorial/advanced_autopilot.rst:427: WARNING: Inline literal start-string without end-string.00:40
veebersballoons: nvm I found the link to the page, my bad00:42
balloonsok ;-)00:42
veebersgets me wondering if it should be more apparent. /me poinders00:42
veebersponders even00:42
balloonsveebers, weird I get no errors running my branch00:44
veebersballoons: hmm odd, this is what's I'm using to build: python3 setup.py build_sphinx -v -b html00:46
veebersI can see the error in the doc, not sure where it's come from though00:46
balloonsveebers, ahh I get it now on a fresh checkout.. interesting00:46
veebersballoons: that's odd as it's not something you changed :-\00:47
* balloons is just running sphinx-build -b html docs htmldocs00:47
balloonsI do see the error though.. simple fix00:48
balloonsshall I include it?00:48
balloonspushed 525 with the fix00:49
veeberssweet cheers00:50
balloonsveebers, "Text conflict in docs/tutorial/advanced_autopilot.rst"00:54
balloonsperhaps I should re-merge trunk'00:55
veebersballoons: ah right, thomi mentioned this morning that he's made changes to trunk that will need merging (whitespace changes)00:55
veebersalso, have reviewed branch, NF00:55
veebersballoons: hey it seems I've missed something completely (in your review) you say that link will die soon, where will it live after that? I thought there was just discussion as to the best way to get the docs in the right place00:59
balloonsveebers, basically the static html as it is cannot live on @ developer.ubuntu.com. It will be integrated, but not hosted as-is01:00
veebersballoons: also, as this is in the README it's not generated into html (or anything else) so there has to be a link to something there (as it remains just text)01:00
balloonsveebers, lol, long day.. right01:00
balloonsyou could just say refer to the docs then.. bad idea to put a link like that into a readme which will break and we'll never update01:00
veebersballoons: right, that's what you and thomis were talking about this morning. The documentation will still be available on dev.u.com right? /me feels he needs to re-read the backlog01:01
balloonsveebers, yes it will be there.. Ideally I'd like it to be there as-is, but as it stands that's not in the plan. The plan is to import it as json and redisplay it01:02
veebersballoons: ugh right. So it sucks that we can't put in the projects own readme the url to where the docs live. I can't just say "Check the docs" in the readme, how would a dev find that if it's not there? (I'm ranting a little, I know)01:03
balloonsveebers, yea I know. You could offer them the file based location01:05
veebersballoons: right I could but that feels a little dumb. I want them to see the good docs with it's links etc. I could always suggest building the docs and reading from there, but again it seems silly to do that if they exist on the web somewhere01:07
veebersballoons: so one thing I don't understand is, how come the link to the docs (on dev.u.com) will die or be unknown?01:09
balloonsveebers, on the self.wait_select_single, what would you like to see? you are confused as to what 'object' is?01:11
veebersballoons: well, object doesn't contain the method wait_select_wingle, nor do I see it defined in that snippet (or  implied) that it's implemented there01:12
balloonsveebers, the url is currently on a server I have no control over and can't/won't be updated. It will eventually migrate into d.u.c API pages.. that's the plan at least, and it's how AP must go in if it's to live on d.u.c01:13
balloonsI think it's ok as it's an advanced tutorial, and the source is given.. however, i'm happy to tweak it, I simply don't know how01:14
veebersballoons: but surely the url will be known right? Or are you stating that the current url and docs will be deleted before the migration?01:14
veebersballoons: I think a simple '. . .' above that code would be fine (considering it's an advanced tut.) much like line . . .01:14
balloonsveebers, that link won't be deleted until it's done.. I may leave it there for some time after even; who knows what we'll do with eventually. But it's a horrible link to want anyway, imho01:14
balloonsI'd rather see d.u.c/autopilot01:15
balloonsor something easy and sensible01:15
veebersdiffline 16201:15
veebersballoons: right that makes sense. So we can update the readme once the migration happens right? It means up to that point the docs are available (and you can learn the location from the readme) and then when the url changes we'll update the readme01:16
balloonsveebers, ok, hopefully everything is as you wanted now in that mp01:16
* veebers looks01:16
* veebers builds docs, looking good so far01:17
balloonsveebers, perhaps a custom url shortner?01:17
balloonsI'm ok with you doing it, it's your project so far be it from me to stop you, but I wanted you to be aware01:17
veebersballoons: sweet, bottom approved that MP of yours01:19
veebersballoons: I appreciate the heads up, it's just a little annoying that it seems hard to have the docs hosted somewhere (that should be an easy problem, right?)01:20
balloonsok, so one more for tomorrow01:20
veebersI think at this stage we'll use the existing url, then when the migration is finished and the autopilot docs are available there we'll update the url then (as it won't have broken inbetween)01:20
balloonsveebers, I agree it should be easy which is why I was pushing / asking if you wanted to do it yourselves somewhere.. imho, I'd rather see autopilot have it's own site, with docs, intro, etc01:20
veebersballoons: while that might be nice, it'll also introduce further maintenance issues than what we have. I'm not saying it will never happen, it's just not top priority at the moment01:21
balloonsyea, I know.. given the headaches though.. I'd rather own the silly thing01:21
balloonsYour 4 layers removed, but I'm still 301:22
balloons*you are01:22
balloonsok veebers, that's it for me and mp's.. will you try to actually land them today?01:23
balloonsI have the final big one I need to land and having the rest in makes it easier :-)01:24
* balloons notes he is rather unlikely to be ready tomorrow anyway so :-)01:24
veebersballoons: thomi tells me that there are issues at the moment with landing. But yes the intention it to get them merged into trunk.01:24
balloonskk.. enjoy and thanks!01:24
veebersballoons: I'm pretty sure the bottom approved stuff is/will be top approved so should get merged01:24
veebersnw, and thanks to you too ^_^01:25
balloonsveebers, we will be in touch on the docs thing.. please bug me on it, so I can do the same to those making site decisions :-)01:25
veebersballoons: don't worry, I will ;-)01:25
jgdxI'm seeing a failure in u-s-s AP tests on mako where a tap is interpreted as a long-tap11:34
jgdx… I think. A list element is highlighted then, after a couple of seconds, de-highlighted. It's very intermittent.11:35
jgdxTest is ubuntu_system_settings.tests.test_datetime.TimeDateTestCase.test_manual_tz_selection, and we're using scroll_to_and_click11:36
=== barry` is now known as barry
rpadovaniballoons, hey :-)17:59
balloonshowdy rpadovani17:59
rpadovaniballoons, do you remember the problem I talked to you about? I'm not able to debug it, so I setup a branch for you, if you want :-)17:59
rpadovaniballoons, https://code.launchpad.net/~rpadovani/ubuntu-calculator-app/bigNumber15012218:00
rpadovaniballoons, every time you run autopilot, there are random failures18:00
balloonsrpadovani, I remember you telling me to ignore it. I suppose you no longer want me to ignore it :-)18:00
balloonsthanks for the branch18:00
rpadovaniballoons, that is :D18:00
balloonsrpadovani, so is there a specific test that fails or ?18:00
rpadovaniballoons, that's the problem, it changes at every run18:01
rpadovaniballoons, 2 are the same, but are about precisions,18:01
rpadovaniballoons, I suppose (but it's only a supposition) that it takes more time to load the library when it uses bignumber, so it doesn't have time to load it18:01
balloonsrpadovani, if there's a good test to run over and over, that's what I was after18:04
rpadovaniballoons, I didn't find it, every time I try to run only the test failed in the previous run, it works well18:05
rpadovaniballoons, the last one failed on my pc is  ubuntu_calculator_app.tests.test_main.MainTestCase.test_small_numbers18:05
balloonsrpadovani, I'll run it a few times and see what happens. Might be a bit before I can have a look, otp18:15
rpadovaniballoons, thank you!18:15
balloonsyw.. I do love a good mystery18:15
balloonsrpadovani, first glance it definitely looks like keypresses aren't being recieved sometimes18:19
balloonsthe fix for that likely didn't work18:19
rpadovaniballoons, do you have in the log18:20
rpadovaniError: math.jsTypeError: Cannot call method 'eval' of undefined engine formula:0.000000001+118:20
balloonsrpadovani, yes it's gathering the logs as it runs18:23
rpadovaniballoons, so I think it's the problem, the var that manages the engine isn't set18:24
balloonsahh.. if you think it's a timing thing, add a sleep in the setup step and see what happens18:24
balloonsrpadovani, does that cause the autopilot.exceptions.StateNotFoundError: Object not found with name 'QQuickMouseArea' and properties {'id': 41} error?18:38
rpadovaniballoons, ehm , don't know?18:38
balloonsseems not18:38

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