/srv/irclogs.ubuntu.com/2015/01/21/#ubuntu-autopilot.txt

=== charles_ is now known as charles
jhodappalesage, ping18:47
alesagejhodapp, hallo18:47
jhodappalesage, hey man, got an AP test problem I'm wondering if you can help shed some light on18:47
jhodappalesage, 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#L4018:48
alesagejhodapp, lay it on me18:48
jhodappalesage, 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#L5518:48
alesagejhodapp, ok18:49
jhodappalesage, It never seems to be able to get the dialog from the mediaplayer-app QML18:49
jhodappalesage, 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 working18:49
jhodappalesage, so the noMediaDialog one works18:49
alesagejhodapp, hmpf, I'd have to fire it up18:50
jhodappalesage, error dialog here: http://bazaar.launchpad.net/~jhodapp/mediaplayer-app/missing-codec-error-test/view/head:/src/qml/player/VideoPlayer.qml#L20318:50
jhodappalesage, 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 118:51
alesagejhodapp, need to close a couple of other things first :)18:52
jhodappalesage, sure thing, no rush18:52
jhodappalesage, probably just missing something very simple18:53
balloonsthomi, veebers are you guys open to autopilot having it's own site?19:58
thomiballoons: I think that would be a step backwards TBH20:07
thomiit'd make autopilot appear as a third class citizen20:07
thomiI don't understand what's so hard about making the current sphinx docs more integrated into the existing site20:08
thomiI don't think you need to mess around with json exporters and the like20:08
thomi(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
thomiballoons: 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 systems20:09
balloonsthomi, 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 everyone20:09
thomiWell... personally I think using a CMS for automatically generated documentation is a .... questionable choice20:10
balloonsbut I can easily forsee the generated html docs no longer existing and the content exported as json and forced into the site as-is20:10
* balloons argued for a static site20:11
thomiI'm not saying the whole site needs to be static20:11
thomijust that it's silly to turn automatically generated content into content managed by a CMS20:11
balloonslol.. keep things seperate; it's the sane way20:11
thomiapart from anything else, consider the overhead!20:11
balloonsthe cms system can handle the content to push if you wish20:11
* balloons looks at intersphinx20:12
thomithe only thing that's missing right now is for the autopilot docs to be linked to from other places20:12
balloonsthomi, 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 generating20:14
balloonsthey live on a separate server and I can't update them.20:15
balloonsnor is anyone keen to host them that way anymore20:15
thomiwell, 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
thomiI hope somehow that situation can be resolved. I'm not sure how we can ask people to use autopilot wihtout even hosting the documentation showehere20:18
balloonsthomi, 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:19
thomiI'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
balloonsok, so "prominent" and intact then, as in, as generated by sphinx directly, a cohesive whole20:21
thomiadditionally, 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 it20:22
veebersballoons: I'm just OTP at the moment, will join the discussion soon :-)20:22
thomiballoons: if you can do the import trick, and manage to keep everything intact, then I'll be happy20:22
thomiballoons: but TBH that seems like it'd be difficult to me20:22
thomiI 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
balloonsthomi, I'm really on your side on this, which is why this is so maddening20:23
thomisorry 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 now20:24
balloonsthomi, hah, you are absolutely fine.20:25
balloonsI 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 better20:26
balloonswe'll continue to look after it though20:27
veebersballoons: 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:40
balloonsveebers, I neither want autopilot to be third-tier, nor make you sad.20:42
veebersballoons: ^_^20:43
balloonsI 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 issues20:44
balloonsveebers, re: description, heh; https://bugs.launchpad.net/autopilot/+bug/141337421:08
ubot5Launchpad bug 1413374 in Autopilot "Launchpad description is lacking" [Undecided,New]21:08
veebersballoons: heh awesome thanks. I'll make sure that gets fixed21:08
balloonsyea, that way I didn't have to bug you and you don't need to drop everything now :-)21:08
balloonsveebers, thomi if you could recheck https://code.launchpad.net/~nskaggs/autopilot/apdocs-restruct/+merge/24659921:56
thomisure21:56
thomiballoons: all the "linkdescr" lines end in punctuation except "Tips to being out te test writer in you"21:58
thomiso that should have a '.'21:58
balloonskk21:58
thomioh,a nd the two in the Indices and tables section21:58
balloonskk21:58
veeberssure21:59
veebersballoons: lgtm, bottom approved.22:04
veebersheh, 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 course22:04
balloonsawesome, 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
balloonsveebers, sneaky me :-)22:04
veebersthomi: 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:05
thomiveebers: I don't see a question?22:06
veebersthomi: I set it as NF with a comment in the diff22:07
thomioh wait, old diff22:07
thomiveebers: ok22:07
thomiveebers: pushed22:09
* veebers looks22:09
veeberssweet, approved. Thanks22:10
balloonsalright, how's about https://code.launchpad.net/~nskaggs/autopilot/apdocs-install/+merge/247232 thomi and veebers ?22:26
* thomi looks22:26
thomiballoons: did you remove the additional sections? the diff has changed and now I'm confused :D22:27
balloonsthomi, 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 it22:28
thomiballoons: ahh, ok22:29
balloonsso the toc is now balanced again, and includes an install section :-)22:29
thomiballoons: LGTM22:29
balloonsohh right, need to add the ref for installing (i need it for the tutorial)22:31
balloonsawesome..22:32

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