**** BEGIN LOGGING AT Fri Jan 08 02:59:58 2016 Jan 08 03:03:31 andolamin, pong Jan 08 03:03:55 whoops, kiddie bedtime,sorry Jan 08 06:49:40 elvispre: you are most likely correct; qtwebengine uses pulseaudio directly, without going through QtMultimedia, and therefore has its own set of codecs Jan 08 06:49:57 Or, maybe we simply have to activate it. Jan 08 08:08:08 morning Jan 08 08:28:09 Morning! Jan 08 08:30:11 Tofe: The dashboard problem I mentioned was me messing stuff up :P Jan 08 08:36:18 Herrie|Veer: ok, I'll let up fix it up then ! :p Jan 08 09:14:54 Tofe: For 979 I think we'll need a little fix. Jan 08 09:15:16 It works OK in luna-webappmgr just doesn't end up correctly in LNC in all cases. Jan 08 09:15:25 We have different scenarios: Jan 08 09:16:01 1. height is provided and no window params, so it should be pixel value (works correct) Jan 08 09:16:40 2. height is provided and window params metrics = units, luna webappmanager does the conversion to pixels but it doesn't end up in LNC Jan 08 09:17:45 3. window param dashHeight is provided in pixels, ends up correctly in LNC and gets converted to correct dpi pixels Jan 08 09:17:45 I don't understand, 2. is the basic scenario for gridunit height Jan 08 09:17:49 4. Jan 08 09:18:39 4. dashHeight and metrics window param are provided, lnc doesn't deal with this correctly currently. I'll propose a PR for this. Just not sure how to fix 2. Jan 08 09:19:07 Tofe: Luna-webappmanager does value * gridunit and outputs a pixel value in log Jan 08 09:19:17 However this doesn't get picked up by lnc Jan 08 09:21:17 See NotificationArea.qml Jan 08 09:21:59 And look for dashHeight which is set to 0 then does a few checks for window params and does a <= 0 and sets the default dash height Jan 08 09:22:49 So when luna-webappmanager correctly calculates it as 32 * 10 = 320 it will get the dashboardCardFixedHeight in LNC which is not correct Jan 08 09:47:13 Ok; I'll have to read that again. Jan 08 09:50:15 Tofe: See line 93 in NotificationArea.qml Jan 08 09:51:12 It only takes care of the fact when LuneOS_dashheight is specified, all other cases will be getting the default height. Jan 08 09:52:14 So either the proper height needs to be passed to LNC or you'd need to put it in LuneOS_dashheight when you calculate it in luna-webappmanager (which I think might not be ideal?) Jan 08 09:56:58 Initially, the height was used only for the popups, not for the dashboards Jan 08 09:57:29 The dashboards always have the same fixed size, apart if they specify a dashHeight property when opening it Jan 08 09:57:46 If noone uses dashHeight out there, we could unify all this a little bit Jan 08 10:04:01 Tofe: legacy uses dashHeight Jan 08 10:04:27 So we need to keep it working ;) Jan 08 10:04:41 And who uses height for a dashboard? Jan 08 10:05:29 Not sure would need to check apps :P Jan 08 10:06:04 Shouldn't be that hard to expose it and not worry about it anymore? The LNC part is fine anyway already :) Jan 08 10:06:21 But roughly, what you say is that : 4. means we don't handle metrics for dashHeight in LNC Jan 08 10:07:17 Tofe: This is a scenario I can think of where a dev would provide both metrics = units and dashHeight Jan 08 10:07:19 and 2. means someone uses height instead of dashHeight for a dashboard, which is an app mistake Jan 08 10:10:30 Tofe: Let me check what examples and documentation says ;) Jan 08 10:15:47 Having both dashHeight and height isn't such a good thing Jan 08 10:20:41 Tofe: You mean for a single call? Jan 08 10:20:49 Or as 2 different options? Jan 08 10:32:22 Tofe: Seems you're right. height shouldn't be specified for dashboards, only dashHeight. So seems DougReeder might have a mistake in his Testr code (though usually he's quite good with these things...) Jan 08 10:32:37 DougReeder: ^ Jan 08 10:36:49 Tofe: Looking back at the logs and LunaCE implementation dashHeight seems to be the only thing mentioned. Could be DougReeder has other references though. He's like morphis at times, fountain of knowledge going back to prehistoric Palm :P Jan 08 10:37:34 Still work require us to take care of scenario 3 and 4 in LNC for which I'll propose a small PR. Jan 08 10:38:43 ok Jan 08 10:50:03 Tofe: PR is in, put a comment as well Jan 08 12:15:29 Aaaah! A typo in the PR title! Jan 08 12:39:20 Tofe: LOL yeah I'll rework it a bit Jan 08 13:08:11 Tofe: Updated Jan 08 13:08:20 You saw nizovn's PR as well? Jan 08 13:15:03 Herrie|Veer: yes, didn't have really time to look at it yet Jan 08 13:15:06 but look ok Jan 08 13:15:08 looks Jan 08 13:22:03 Tofe: LOL I need more coffee, PR updated Jan 08 13:33:33 Herrie|Veer: is it tested / testable in any way? Jan 08 13:41:11 Tofe: Yeah :P Jan 08 13:41:25 DougReeder would need to update Testr anyway probably Jan 08 13:41:52 I tested it with locally updated Testr Jan 08 13:43:33 And tested that in emulator Jan 08 13:48:28 ok, merged Jan 08 14:05:28 http://lists.qt-project.org/pipermail/qtwebengine/2014-June/000061.html Jan 08 14:06:59 Hmmz I'm no expert Jan 08 14:07:34 With regards to licensing etc Jan 08 14:07:54 Don't we have mp3 support in gstreamer and/or pulseaudio already somehow? Jan 08 14:08:09 JaMa, morphis or ka6sox might know a thing about it? Jan 08 14:08:34 Herrie|Veer: we should Jan 08 14:08:38 atleast software encoding Jan 08 14:10:19 morphis: so, no licensing issue if we enable this by default in LuneOS, a priori ? Jan 08 14:11:59 that depends on which one we use Jan 08 14:12:31 I would like to see that as option for users to turn on Jan 08 14:12:43 to overcome the we-ship-it-but-are-not-allowed to Jan 08 14:12:52 That's how Qt does it too Jan 08 14:14:11 Seems it uses FFmpeg underneath Jan 08 14:20:14 morphis: Where you'd see this option? Settings or Tweaks or.... ? Jan 08 14:21:03 Herrie|Veer: first use and then settings too Jan 08 14:21:11 so the user can decide on first boot what he wants Jan 08 14:26:25 OK. Tofe: You know how to take care of this in back end? Jan 08 14:38:16 Not sure; depends if that can be enabled at runtime... Jan 08 15:22:25 After some reading, it seems the only simple way is the build option Jan 08 15:54:20 Tofe: Hmmmz Jan 08 15:54:56 That's not really good. I think this might only be a potential issue once we get into a commercial stage and start to ask money for LuneOS? Jan 08 15:56:28 At least that's what I gathered from FFmpeg Jan 08 18:29:18 I don't know very well; but as long as we don't sell any product (the end-user has to install LuneOS himself on the device), I guess we're quite ok Jan 08 18:30:09 Do you have some links to FFmpeg discussion about this? Jan 08 18:44:28 Tofe: At the bottom on https://www.ffmpeg.org/legal.html Jan 08 18:50:55 We might suffice with a disclaimer in FirstUse that cannot be bypassed and a tickbox needs ticking? Not pretty but well. Jan 08 19:12:44 I.e. That's the closest we can get to enabling it at runtime currently I guess ? Jan 08 19:12:52 DougReeder: You around? Jan 08 20:15:23 Herrie: note that in EU, software patents aren't valid :p Jan 08 20:17:29 Herrie: but if only for me, I could very well continue using a completely patent-free qtwebengine; we are not yet so well spread as to receive such kind of complaints, I think Jan 08 20:19:16 Tofe: We could make the tick box and page country dependant :p Jan 08 20:19:26 To annoy the US folks :P Jan 08 20:20:19 Well, anyway, for me it's not really an urgent issue :) Jan 08 20:20:52 Tofe: I agree. I think FFmpeg shouldn't really be an issue because we're using fully open source? Jan 08 20:21:18 Seems they're not happy when their stuff is included in non open-source projects :P Jan 08 20:21:51 Then again I'm in no way legal specialist and don't know ins and outs of all licenses. But yeah I guess we're OK seeing our userbase :P Jan 08 20:22:06 So you'll PR a change for it :P ? Jan 08 20:24:38 After I've finished tracking the CPU hog Jan 08 20:25:20 Tofe: Nothing suspicious patched upstream or for 5.6? Jan 08 20:25:28 Could be it'll sort itself :P Jan 08 20:28:27 I didn't look, but as there is already a QMake specific flag, I think they consider it handled already Jan 08 20:29:25 Ah, and for the CPU hog, I wondered: why did we prevent JustType to be launched at startup ? It would solve our cpu hog, and "preload" the JustType window... Jan 08 20:29:49 I find it weird that we start the mail app, but not justtype... Jan 08 20:44:36 Tofe: Well I think we can start it.. Might have been problems with window handling at that time? Jan 08 20:53:50 Not sure; maybe morphis would remember, but he's not around at the moment Jan 08 21:28:05 Tofe: So much has been changed/upgraded over time that I think it might be worth to just give it a go :-) Jan 08 21:28:22 yup Jan 08 21:28:36 I'll try tomorrow, it's just a one-liner Jan 08 21:28:47 (but I've forgotten where...) Jan 08 21:29:21 Same like I think we might want to revisit our decision to drop the Enyo 1 Calendar app. With all the great UI work done I think it'll behave a lot better :-) Just might need some tweaking for non-TP resolution. Jan 08 21:29:45 Will test that sometime :-) Jan 08 21:30:46 https://github.com/webOS-ports/luna-webappmanager/blob/master/src/qml/ApplicationContainer.qml#L70 disabling this one I think should be enough Jan 08 21:32:07 But I'm too tired right now Jan 08 21:32:20 Will stop for tonight I think :) Jan 08 21:51:53 Tofe: Same here :) Good night! Jan 08 22:07:22 * elvispre has an alarming alarm clock! Jan 08 22:07:54 Now I just need a way to stop it ringing once it has woken me up :) Jan 08 22:13:58 Elvispre: Pressing the buttons don't work? Jan 08 22:14:23 Herrie: I haven't seen the buttons for a while. Jan 08 22:15:16 A couple of weeks ago, the top edges of the buttons were visible in the "banner" (?), but now there is no banner at all. Jan 08 22:15:37 elvispre: Hmmmz... Might need to check that :) Jan 08 22:16:00 Could be we broke some things while trying to fix others :P Jan 08 22:16:19 Herrie: No rush. I can turn off the alarm by restarting ;) Jan 08 22:16:38 My impression was you were all working on banner stuff at the moment anyway. Jan 08 22:18:28 elvispre: Yeah it needs some tweaking. Jan 08 22:18:53 It should go silent with volume button too. Jan 08 22:19:48 Or I could use the mute switch. Jan 08 22:20:45 As soon as I get sound back I want to turn it off :P Jan 08 22:26:54 elvispre: Anything in journalctl after the alarm? Jan 08 22:33:56 Herrie: There are some activateDashAlarm logs by LWAM immediately before the (successful) ringAlarm stuff. I don't see any obvious errors that seem relevant. Jan 08 23:24:46 Herrie: Actually, I even got the alarm dashboard with the buttons once now. It took ages to render so I had muted the sound long before it appeared. Jan 08 23:27:00 I have created a PR (for core-apps webOS-ports:webOS-ports/master) or you might like to just try my forked code directly. Jan 08 23:27:53 * elvispre is going to set a LuneOS clock alarm to wake himself up in the morning. Jan 08 23:39:34 Herrie|Veer, ping? Jan 08 23:39:56 Herrie ping? Jan 08 23:40:28 Herrie's are dropping like flies **** BEGIN LOGGING AT Sat Jan 09 00:45:38 2016 **** ENDING LOGGING AT Sat Jan 09 02:59:58 2016