=== charles_ is now known as charles [18:47] alesage, ping [18:47] jhodapp, hallo [18:47] alesage, hey man, got an AP test problem I'm wondering if you can help shed some light on [18:48] alesage, take a look at this: http://bazaar.launchpad.net/~jhodapp/mediaplayer-app/missing-codec-error-test/view/head:/tests/autopilot/mediaplayer_app/tests/test_player.py#L40 [18:48] jhodapp, lay it on me [18:48] alesage, that function tries to get the error dialog, via this: http://bazaar.launchpad.net/~jhodapp/mediaplayer-app/missing-codec-error-test/view/head:/tests/autopilot/mediaplayer_app/emulators/main_window.py#L55 [18:49] jhodapp, ok [18:49] alesage, It never seems to be able to get the dialog from the mediaplayer-app QML [18:49] alesage, if you look at the test directly above that, it does the same thing and works, so I'm confused why this one isn't working [18:49] alesage, so the noMediaDialog one works [18:50] jhodapp, hmpf, I'd have to fire it up [18:50] alesage, error dialog here: http://bazaar.launchpad.net/~jhodapp/mediaplayer-app/missing-codec-error-test/view/head:/src/qml/player/VideoPlayer.qml#L203 [18:51] alesage, ok, if you've got a moment to try it out, you can get the latest media-hub and qtubuntu-media required to generate the errors from vivid silo 1 [18:52] jhodapp, need to close a couple of other things first :) [18:52] alesage, sure thing, no rush [18:53] alesage, probably just missing something very simple [19:58] thomi, veebers are you guys open to autopilot having it's own site? [20:07] balloons: I think that would be a step backwards TBH [20:07] it'd make autopilot appear as a third class citizen [20:08] I don't understand what's so hard about making the current sphinx docs more integrated into the existing site [20:08] I don't think you need to mess around with json exporters and the like [20:09] (besides, I think the existing API docs are less useful than the ones we generate, but that might be a QMl <-> python difference, I'm not sure) [20:09] balloons: intersphinx is a system for linking between sphinx doc trees - you should be able to build something similar to allow linking between docs built in separate systems [20:09] thomi, the site itself is more or less handled by others on my team, so I play a small role. That said, obviously having all content under the CMS is preferred. Having autopilot be special has caused issues for everyone [20:10] Well... personally I think using a CMS for automatically generated documentation is a .... questionable choice [20:10] but I can easily forsee the generated html docs no longer existing and the content exported as json and forced into the site as-is [20:11] * balloons argued for a static site [20:11] I'm not saying the whole site needs to be static [20:11] just that it's silly to turn automatically generated content into content managed by a CMS [20:11] lol.. keep things seperate; it's the sane way [20:11] apart from anything else, consider the overhead! [20:11] the cms system can handle the content to push if you wish [20:12] * balloons looks at intersphinx [20:12] the only thing that's missing right now is for the autopilot docs to be linked to from other places [20:14] thomi, the docs as they are are going away. As it stands, I don't know a way you'll end up with the html you're generating [20:15] they live on a separate server and I can't update them. [20:15] nor is anyone keen to host them that way anymore [20:18] well, it looks like I need to leave this to veebers. I'm really dissappointed that the new developer.ubuntu.com site clearly wasn't designed to support autopilot. [20:18] I hope somehow that situation can be resolved. I'm not sure how we can ask people to use autopilot wihtout even hosting the documentation showehere [20:19] thomi, I guess if I can get one thing from you it would be what you would like to see so I understand your viewpoint. Would it be to have developer.ubuntu.com/autopilot featuring the html generated docs, with links from the content on developer.u.c? [20:21] I'd like to see the ap docs (both API and narrative docs) be a prominent part of the site, AND to not have the current quality of the documentation retarded by the underlying technology used. [20:21] ok, so "prominent" and intact then, as in, as generated by sphinx directly, a cohesive whole [20:22] additionally, from a technical point of view, it seems *insane* to me that the plan is to take static generated content and import it into a CMS, especially since we update the docs reasonably frequently, but if you can achieve the goals above, I don't mind much how you do it [20:22] balloons: I'm just OTP at the moment, will join the discussion soon :-) [20:22] balloons: if you can do the import trick, and manage to keep everything intact, then I'll be happy [20:22] balloons: but TBH that seems like it'd be difficult to me [20:23] I mean, certainbly possible, but why go to all that trouble? But if there's some good reason that I don't understand, then go for it! [20:23] thomi, I'm really on your side on this, which is why this is so maddening [20:24] sorry if I seem overly grumpy. From my perspective, ,we were *strongly* encouraged to move the docs to developer.u.c, and it's been nothing but failure after failure, so I'm quite short-temered about it now [20:25] thomi, hah, you are absolutely fine. [20:26] I personally have no idea how you would make the magic import work, and frankly I'd rather we didn't do it. But yes, the re-design has made things worse rather than better [20:27] we'll continue to look after it though [20:40] balloons: heh, I don't think I can add much more from what thomis said. I reaction is we could always put the docs on readthedocs, become a 3rd tier citizen and maybe update the launchpad site to have more info. That would make me sad :-( [20:42] veebers, I neither want autopilot to be third-tier, nor make you sad. [20:43] balloons: ^_^ [20:44] I guess I am somewhat at a loss on how I can prevent that.. I feel like the current route make no one happy and is full of issues [21:08] veebers, re: description, heh; https://bugs.launchpad.net/autopilot/+bug/1413374 [21:08] Launchpad bug 1413374 in Autopilot "Launchpad description is lacking" [Undecided,New] [21:08] balloons: heh awesome thanks. I'll make sure that gets fixed [21:08] yea, that way I didn't have to bug you and you don't need to drop everything now :-) [21:56] veebers, thomi if you could recheck https://code.launchpad.net/~nskaggs/autopilot/apdocs-restruct/+merge/246599 [21:56] sure [21:58] balloons: all the "linkdescr" lines end in punctuation except "Tips to being out te test writer in you" [21:58] so that should have a '.' [21:58] kk [21:58] oh,a nd the two in the Indices and tables section [21:58] kk [21:59] sure [22:04] balloons: lgtm, bottom approved. [22:04] heh, I was looking at the MP while pulling the latest to build and got confused when there _was_ punctuation in my generated docs. Duh, you pushed the changes of course [22:04] awesome, I'll have at least one more for you this evening that does a little more expanding to the toc (add installation section) [22:04] veebers, sneaky me :-) [22:05] thomi: if you would like to yay/nay the question in your MP I would like to approve that too (https://code.launchpad.net/~thomir/autopilot/trunk-add-troubleshooting-docs/+merge/247093) [22:06] veebers: I don't see a question? [22:07] thomi: I set it as NF with a comment in the diff [22:07] oh wait, old diff [22:07] veebers: ok [22:09] veebers: pushed [22:09] * veebers looks [22:10] sweet, approved. Thanks [22:26] alright, how's about https://code.launchpad.net/~nskaggs/autopilot/apdocs-install/+merge/247232 thomi and veebers ? [22:26] * thomi looks [22:27] balloons: did you remove the additional sections? the diff has changed and now I'm confused :D [22:28] thomi, ahh, sorry.. I saw whitespace changes and fixed them quickly.. this mp depends on the one you just reviewed. I moved the faq install question, attempted to fix it, and added a couple other faq questions related to it [22:29] balloons: ahh, ok [22:29] so the toc is now balanced again, and includes an install section :-) [22:29] balloons: LGTM [22:31] ohh right, need to add the ref for installing (i need it for the tutorial) [22:32] awesome..