/srv/irclogs.ubuntu.com/2012/09/07/#ubuntu-unity.txt

bschaeferIll have to look at the code somemore...thought it could be very easy that I missed something when I removed a bunch of that code00:00
MCR1but do you understand my arguing ?00:00
bschaeferyeah00:00
MCR1I could not think of any situation where a hovering pointer over the launcher should still trigger a hide, it should not matter at all if the mouse is moved post reveal or not so long as the pointer is over the launcher which is checked there anyway...00:02
bschaeferMCR1, hmm the biggest thing is I just dosn't happen like that on my build...00:02
MCR1hmm, yeah - that is the strangest thingie...00:02
bschaeferMCR1, I think why it is there00:03
bschaeferMCR1, is move the the mouse over where the launcher would show...00:03
bschaeferThen open the dash00:03
bschaeferthen close the dash00:03
bschaeferand the launcher hides, with the mouse over00:03
bschaeferit00:03
MCR1I could reproduce it on several machines and this hunts me for since Precise alphas00:03
MCR1if you close the Dash, but the pointer still hovers the launcher it should imho not hide either00:04
bschaeferMCR1, hmm well make a bug :)00:04
bschaeferwhich then the design team can make the decision00:05
bschaefer(though since it is programmed it I would think they already did)00:05
* MCR1 is searching for his video-evidence :-D00:05
bschaeferMCR1, well make two bugs, one for that video you have00:06
bschaeferand when the mouse is over the launcher when the dash opens00:06
MCR1bschaefer: Too tired today, maybe I'll attack that tomorrow ;)00:08
bschaeferMCR1, alright, cool. Have a good night :)00:08
bschaeferMCR1, also 1307      neil.pa |       if (GetQuirk(MOUSE_MOVE_POST_REVEAL))00:09
bschaeferso it was added a while ago00:09
MCR11207 ?00:10
MCR1*130700:10
bschaeferrev 130700:11
MCR1ah00:11
bschaeferlike we are on 26xx or something like that00:11
MCR1line 58 is also strange...00:13
MCR1tomorrow... gn :) & thanx a lot for your patience with me, bschaefer ;)00:14
=== charles_ is now known as charles
mhr3bilal, ping07:02
MCR1JohnLea: Hi :) I would need your input here: https://bugs.launchpad.net/unity/+bug/104723208:47
ubot5Ubuntu bug 1047232 in Unity "Unity Launcher Autohide Behavior: The Launcher should never autohide if the mousepointer hovers over it" [Undecided,New]08:47
=== mmrazik is now known as mmrazi|otp
MCR1duflu: Mornin' :) Do you have any comments on https://code.launchpad.net/~mc-return/compiz/compiz.merge-replace-defines/+merge/122865  ?09:11
dufluMCR1: Sorry, out of time this week09:11
dufluIn a mad rush this afternoon09:11
MCR1mad rushes can be productive, so I will stop nerveing ;)09:12
MCR1*nerving09:12
JohnLeaMCR1; that bug looks like a dupe of bug #74570709:15
didrocksMCR1: see the MP, you are breaking one use case09:15
ubot5Launchpad bug 745707 in unity (Ubuntu) "Launcher - Launcher should never autohide when the cursor is positioned over the Launcher (e.g. When a spread ends)" [Medium,Triaged] https://launchpad.net/bugs/74570709:15
didrocksJohnLea: ^09:15
didrocksJohnLea: this will break the "super" key case09:15
JohnLeadidrocks; how will bug #745707 break the "super" key use case?09:16
ubot5Launchpad bug 745707 in unity (Ubuntu) "Launcher - Launcher should never autohide when the cursor is positioned over the Launcher (e.g. When a spread ends)" [Medium,Triaged] https://launchpad.net/bugs/74570709:16
didrocksJohnLea: see my comment on https://code.launchpad.net/~mc-return/unity/unity.merge-fix-launcher-should-never-autohide-when-pointer-hovers-it/+merge/12323109:16
JohnLeadidrocks; I don't think what you described in bug #123231 is that much of a problem, however well spotted and I think it would be better if we could add "This only applies when the Launcher is revealed by a pointer action, if the Launcher is revealed by a keyboard shortcut it should auto-hide as normal even if the mouse finds itself over the Launcher"  However this last statement is nice to have, not essential, and should not block this fix from l09:20
JohnLeaanding IMHO09:20
ubot5Launchpad bug 123231 in evolution-data-server "evolution-data-server-1.12 crashed with signal 5 in g_logv()" [Critical,Fix released] https://launchpad.net/bugs/12323109:20
JohnLeadidrocks; I'm updating the description of bug #74570709:20
ubot5Launchpad bug 745707 in unity (Ubuntu) "Launcher - Launcher should never autohide when the cursor is positioned over the Launcher (e.g. When a spread ends)" [Medium,Triaged] https://launchpad.net/bugs/74570709:20
didrocksJohnLea: the bug doesn't describe it, however MCR1's fix is breaking it09:20
didrocksJohnLea: so the fix has to be redone, otherwise, we will have this regression09:21
MCR1JohnLea, didrocks: To my understanding it would fix bug 74570709:21
JohnLeadidrocks; ok, MCR1 can you fix it? ;-)09:21
didrocksMCR1: see above + my comment ^09:22
didrocksMCR1: you still have that use case to work09:22
MCR1JohnLea: It would feel much better if the launcher would not hide in any case if the mouse hovers it, even if revealed by the Super key...09:23
didrocksMCR1: that was an explicit design use case09:23
didrocksthere was more case in the past, that was removed by design09:23
JohnLeadidrocks, MCR1; I have updated bug #745707 to include didrocks's point about keyboard shortcut triggered Launcher reveals, see bottom of the bug description.09:25
ubot5Launchpad bug 745707 in unity (Ubuntu) "Launcher - Launcher should never autohide when the cursor is positioned over the Launcher (e.g. When a spread ends)" [Medium,Triaged] https://launchpad.net/bugs/74570709:25
MCR1I understand, but it feels unnatural if something hides if the mousepointer hovers it, no matter how it is revealed, so maybe we should re-think this design09:25
* MCR1 reading09:25
MCR1a long bug report...09:27
* MCR1 needs some time to read and comprehend it all...09:27
JohnLeaMCR1; I think didrock's point is that if users are using the SUPER + NUMBER shortcut to select applications, they don't want the Launcher stuck out if their mouse pointer just happens to be in the Launcher area prior to SUPER being pressed09:28
MCR1JohnLea: Yes, I understand that case...09:29
=== mmrazi|otp is now known as mmrazik|lunch
MCR1JohnLea: Have you seen bug 1019453, btw. ?09:36
ubot5Launchpad bug 1019453 in unity (Ubuntu) "Unity Launcher Regression: Autohide Option makes the launcher not being used anymore, add Intellihide mode like all other docks/launchers have" [Wishlist,Incomplete] https://launchpad.net/bugs/101945309:36
JohnLeaMCR1; yes I have seen it ;-)09:37
MCR1^^^this would imho be the best behavior for launcher hiding...09:37
MCR1JohnLea: Ofc I am very interested in what you think about it also... :)09:40
JohnLeaMCR1; I personally really liked intellihide, I always ran it, and I liked how it surfaced the tiles in my launcher when there was space, and was revealable when there wasn't.  However it user tested very badly with everybody other than expert users, and when the decision was made to not have it on by default, the dev team were not willing to carry the cost of continuing to support the feature.09:55
sil2100JohnLea: what problems did normal users have with intellihide?09:56
MCR1JohnLea: There never was intellihide in Unity, but just window dodge. The difference between them is very small in terms, but very large in practical use.09:57
sil2100JohnLea: maybe all that was needed was a short tutorial on first ubuntu login? I think teaching people is much better than just removing complicated features - this doesn't lead to lives getting more convinient09:57
sil2100Progress requires learning...09:57
JohnLeasil2100; you should have seen it!  user after user would resort to moving there windows out of the way every time they needed to access the Launcher!  Most new users assumed this was how you revealed the launcher, and it was very painful to watch09:57
MCR1JohnLea: While dodge will hide from every window, Intellihide will just hide from active windows and automatically reveal the launcher again if other windows not touching it are active - That is a HUGE difference.09:58
sil2100I just feel that removing more and more useful features just because some people didn't know how to use them is a bit sad... Since its not that they didn't use it, they just didn't know how it works, right?09:58
MCR1JohnLea: Please do me a favor and simply install the package docky and set it to intellihide. Especially, but not only  on dual screens this makes a huge difference to simple window dodge...09:59
JohnLeasil2100; if we require someone to pick up the documentation, or complete a tutorial in order to use something we have really failed from a ux point of view.  What we want is a nice progressive learning curve, where users start off using the basic interactions, and learn the more complex interactions through usage.  The issue that killed intellihide was that when it stopped being the default option the dev team didn't want to continue to carry the10:00
JohnLeacost of supporting the code.10:00
JohnLeaMCR1; the problem with what you are describing is that most non-advanced users wouldn't get what was happening10:01
sil2100JohnLea: true, but what if we didn't require people to go through manuals or tutorials, just thinking of a way to *show* the newbie user on how to effectively use the operating system? Since anyway most non-advanced users had to learn the usage of a computer from books, courses or tutorials10:02
MCR1JohnLea: As I said before, there never was intellihide implemented in Unity, so no user was testing it ever...10:02
sil2100MCR1: there wasn't?10:02
didrocksMCR1: this was in the code, it was "dodge active window"10:02
JohnLeasil2100; there is also a UX cost to having options, just look at KDE.  However in this case it was the cost of maintaining the feature that killed it.10:02
MCR1JohnLea: Unity-2d has intellihide, Unity-3d once had window dodge...10:02
MCR1Unity-2d's autohide feature acts like intellihide IIRC...10:03
MCR1(it is a long time ago ;))10:03
didrocksMCR1: so it had it10:03
MCR1Unity-3d never had it10:03
didrocksdodge active window?10:04
didrocksI wrote this part of the code, I know what I wrote… :)10:04
MCR1Unity-2d had it until it was removed, but it was called autohide10:04
JohnLeasil2100; what we would like to do is to provide subtle visual cues to the user to aid discovery, some work to enable proximity effects which don't kill battery life is required to do some of these however10:04
didrocksMCR1: http://www.webupd8.org/2012/05/how-to-get-dodge-windows-and-minimize.html if you don't believe me10:05
sil2100JohnLea: there's a lot of things to consider here indeed..10:05
didrocksand look at the screenshot10:06
didrocksthere is "dodge active window"10:06
MCR1didrocks: This PPA was never official code - it was a remake - I never tested it myself, but it was a completely different re-implementation after Dodge was removed in Precise10:07
didrocksMCR1: the ppa just reverted the commit10:07
didrocksso not a completely different re-implementation10:07
didrocksMCR1: http://bazaar.launchpad.net/~unity-team/unity/branch-3.0/view/head:/unityshell.xml.in10:09
didrocksthis is unity oneiric10:09
didrockssee "Dodge Active Window"10:09
didrocksor you will tell that I faked the bzr repo? :)10:09
MCR1oh, *surprised*10:09
didrocksthanks for trusting me when I told I wrote it… :/10:09
MCR1didrocks: I am really sorry - I never saw this option before...10:10
didrockswell, it was there :)10:11
MCR1& ofc I trust you ;)10:11
didrocksthe logic is not a lot more compared to the first one: http://bazaar.launchpad.net/~unity-team/unity/branch-3.0/view/head:/src/Launcher.cpp#L189410:11
didrocksjust an "active" tag10:11
didrocksand the hidemachine had a state if we had that option enabled10:12
didrocks(which wasn't the default)10:12
MCR1maybe the problem was that dodge any window was the default... and only this got tested, but I feel :-[ now, that I somehow skipped this option in CCSM...10:15
MCR1great code, btw10:15
didrocksI loved the feature too, but as JohnLea said, user testing showed it was too complicated for them10:16
didrocksand the code is now buried…10:16
MCR1:(10:17
MCR1I know many people (especiall older ones) who have problems differentiating left and right mousebutton, but I still do not think that the right solution for this problem is to remove one of the buttons...10:19
MCR1*especially10:19
MCR1^^^this is no joke...10:19
didrockssome did it :-)10:23
MCR1:)10:24
JohnLeaMCR1, didrocks; personally I am a fan of three button mice, RiscOS style.  Left button primary action, middle button context menu (and no other app menus other than these context menus), right button secondary action.  And everything in the operating system and all applications conforming to this pattern.  I still miss RiscOS10:30
MCR1JohnLea: Unfortunately no experience at all with RiscOS :)10:31
JohnLeaMCR1; was way ahead of it's time in the early 90s, but got squeezed out by a combination on Windows, and inventing a chip that it named ARM, that quickly became worth more than the rest of the company.10:33
JohnLeaMCR1; it did a bunch  of UI  things that were very different from both windows and osx and worked very well10:34
MCR1JohnLea: From screenshots it reminds me a bit of the early Amiga days - those were glorious 8-)10:38
MCR1My first HDD had 20 Megabyte and was starting like a jetplane :)10:39
=== mmrazik|lunch is now known as mmrazik
=== MacSlow is now known as MacSlow|lunch
Night-hacksany way to rotate Unity to bottom in 12.04 64 bit ?11:16
popeyNight-hacks, No11:20
Night-hackspopey: really, that sucks !11:21
gordyou could turn your monitor on its side?11:45
Mirvdidrocks: can you merge lp:~timo-jyrinki/compiz/ubuntu_add_pythoncompizconfig_pc to lp:ubuntu/compiz, so that we can propose lp:~compiz-team/compiz/compiz.merge-minor-fixes for merging and compiz works again?11:53
didrockslet me look :)11:55
didrocksMirv: done11:57
Mirvdidrocks: thanks11:59
=== mmrazik is now known as mmrazik|otp
didrocksyw :)11:59
sil2100didrocks: I prepared the nux tarball btw - can you push, in the meantime, the changes to lp:~ubuntu-desktop/unity/precise ?11:59
sil2100didrocks: for the renaming?11:59
sil2100didrocks: https://code.launchpad.net/~sil2100/unity/precise-geis-rename11:59
didrockssil2100: the unity one are already done I think12:00
didrocksdid you check?12:00
sil2100... would be awesome if it was12:00
sil2100;)12:00
sil2100Awesome ;p12:00
sil2100didrocks: thanks!12:00
didrocks:)12:00
didrocksyw12:00
=== MacSlow|lunch is now known as MacSlow
=== mmrazik|otp is now known as mmrazik
sil2100didrocks: ok, I prepared and did a test build of the new nux tarball (with the renaming)12:32
sil2100didrocks: can I push and publish? ;)12:32
didrockssil2100: is there a bug to track the renaming?12:35
didrocksbut yeah, publish the nux tarball12:35
davidcallemhr3, hey, is there a libunity method to close a preview without closing the Dash?12:54
sil2100didrocks: new tarball released, here is the packaging: https://code.launchpad.net/~sil2100/nux/precise-newtarball-geis12:54
MirvMCR1: sam now added a python-compizconfig fix on top of your compiz.merge-minor-fixes, so he combination of that lp:~compiz-team/compiz/compiz.merge-minor-fixes would be going in and hopefully fixing everything (tm)..12:54
zygahey everyone, I have just noticed that classic desktop in 12.10 has wrong order of window control buttons -> in unity that order is [x] [-] [o] while in classic mode they are [x] [o] [-] -- is this by design?12:56
didrockssil2100: hum, in fact, I think you are missing a Breaks:12:59
didrockslike, if nux is updated12:59
didrocksand not unity12:59
didrockshum12:59
didrocksforget about that one12:59
didrockscan be fine if there are symlink in the new utouch* packages13:00
didrockssil2100: this has been the case, right?13:00
didrockssymlink from old naming to new naming?13:00
sil2100didrocks: I think a transitional package is there13:03
didrockssil2100: I'm speaking about symlink13:04
didrockslike if I update utouch* to the new naming13:04
didrockswith old nux and unity (because I couldn't upgrade everything in a row)13:04
didrocksis there a symlink to ensure that nux and unity are still running?13:05
sil2100fginther: ^13:05
fgintherdidrocks, sil2100 the old and new utouch libraries can coexist (i.e. libgeis1 and libutouch-geis1 can be installed at the same time). However, there is a conflicts and replaces on libutouch-geis-dev in libgeis-dev13:08
didrocksfginther: can we load in the same process the old and new one?13:09
didrocksfginther: like new nux using libgeis1 and old unity using libutouch-geis113:10
fgintherdidrocks, hmmm.. Yes that should work. Let me examine something to verify...13:11
didrocksthanks :)13:11
sil2100I need to get fginther's unity/5.0 rename changes merged in13:12
sil2100But the merger fails, looks to me like some stupid problem again13:12
fgintherdidrocks, Bonjour! Yes, you can mix new nux with old unity. nux only has s build-time dependency on libgeis-dev.13:16
fginthersil2100, what's the issue?13:17
didrocksfginther: excellent! thanks :)13:17
didrockssil2100: there is a missing thing in the unity packaging though, in that case13:18
didrocksif libutouch-geis-dev and libgeis-dev are conflicting13:18
didrockslet me fix that :)13:18
sil2100didrocks: thank you!13:28
sil2100:)13:28
didrockssil2100: not sure if you saw my question later today about the bug to track the renaming?13:30
didrockssil2100: we need one for SRUing I guess13:30
sil2100didrocks: hm, right13:32
sil2100fginther: is there a bug for that already?13:32
fginthersil2100, will this work? https://bugs.launchpad.net/ubuntu/+bug/103762113:33
ubot5Ubuntu bug 1037621 in Ubuntu Precise "[Needs-Packaging] geis (project name change)" [Undecided,In progress]13:33
didrocksI think something obvious with "renaming blablabla" would be better13:36
didrockswith a test case meaning, upgrading and see that there is no change, no regression and that multitouch is working13:36
fgintherdidrocks, ok, new bug on the way13:40
fgintherdidrocks, sil2100: https://bugs.launchpad.net/nux/+bug/104738513:52
ubot5Ubuntu bug 1047385 in Nux "Update dependency on renamed geis" [Undecided,New]13:52
tsdgeosmy first silly contribution to compiz so that i can at least compile it :D https://code.launchpad.net/~aacid/compiz/kde_needs_kdeworkspace/+merge/12327813:53
didrocksfginther: good, please add the packaging task, the unity ones as well, and nominate for precise13:53
didrocksfginther: put the quantal tasks as fix released13:53
mhr3davidcalle, no, user decides when to close it, not the scope14:10
mhr3why would you need that?14:10
davidcallemhr3, to do something to a result when an preview action is clicked, without loosing the context of the search. Possible use case : https://plus.google.com/u/0/117867558830601601230/posts/eLD7PMcMZ7M14:12
davidcallelosing*14:12
davidcallemhr3, another option (for this use case) would be to be able to update the content of the preivew (including actions), when an action is activated.14:13
mhr3davidcalle, oh so you want to go back to results after clicking?14:14
MCR1Mirv: thx 4 the info.14:14
davidcallemhr3, yeah14:14
mhr3yea, we should support that, and it might be possible somehow already :)14:15
mhr3maybe if you used goto dash uri14:15
davidcallemhr3, goto dash uri, I've seen that used somewhere...14:15
mhr3i'm not sure it'd work, but protocol-wise it's doable14:16
MCR1tsdgeos: Keep it coming ;)14:16
mhr3(doable right now)14:16
tsdgeosMCR1: jenkins complained https://code.launchpad.net/~aacid/compiz/kde_needs_kdeworkspace/+merge/123278 but but but, any idea why?14:17
tsdgeosmmrazik: ↑ ?14:17
davidcallemhr3, you mean something like Unity.ActivationResponse(goto_uri='' , handled = 1)14:18
mmraziktsdgeos: I'm just looking at it14:18
davidcallemhr3, nevermind, just seen what you are talking about14:19
MCR1tsdgeos: Have you rebased on latest lp:compiz ?14:19
MCR1tsdgeos: bzr merge lp:compiz14:20
MCR1bzr commit14:20
MCR1bzr push14:20
tsdgeoslet me see14:20
MCR1cp: cannot stat `debian/tmp/debian/tmp/usr/lib/pkgconfig/compizconfig-python.pc': No such file or directory14:20
MCR1dh_install: cp -a debian/tmp/debian/tmp/usr/lib/pkgconfig/compizconfig-python.pc debian/python-compizconfig//usr/lib/pkgconfig/ returned exit code 114:20
MCR1the fix for that is missing in your version ^^14:20
mmraziktsdgeos: that ie exactly where jenkins fails14:20
tsdgeosi see14:21
mmrazikMCR1: so it is in trunk already?14:21
MCR1I have not tested, but I think - 1 sec14:21
mmrazikMCR1, tsdgeos: well.. I just found a bug in the ci job config. It is not merging with trunk...14:22
MCR1mmrazik: yes14:22
mmrazikthat should be fixed14:22
mmrazikso let me re-trigger the job14:22
tsdgeosmmrazik: i'm useful!14:22
tsdgeos:D14:22
mmrazik:)14:22
MCR1mmrazik: yeah, you are right - I just got the mails that it did...14:23
fginthersil2100, didrocks can I ask you to please hold off work on the nux/unity for a moment14:23
davidcallemhr3, don't work, it just closes the Dash.14:23
MCR1mmrazik: no, it is in trunk (since 50 min)14:23
MCR1http://bazaar.launchpad.net/~compiz-team/compiz/0.9.8/revision/335214:24
mmrazikMCR1: ack. The compiz-ci was building just the branch from tsdgeos and it didn't merge it with lp:compiz14:24
mhr3davidcalle, what did you use?14:24
davidcallereturn Unity.ActivationResponse(goto_uri='', handled=3 ) and return Unity.ActivationResponse.new(Unity.HandledType.HIDE_DASH, '')14:24
davidcalleNot HIDE_DASH, bad copiy paste14:24
davidcallebut GOTO_DASH_URI14:25
mhr3davidcalle, try "home.lens" for the uri14:25
sil2100fginther: ugh? What's up?14:25
didrocksfginther: sure, and TBH I don't ming, tons of pings here14:26
davidcallemhr3, nope14:27
fginthersil2100, didrock chase found a potential issue that needs to be root cause14:28
mhr3davidcalle, so, yea... open a bug :)14:28
sil2100fginther: ok, keep me in touch then ;) Need any help?14:28
sil2100fginther: where is the issue btw.? Hope it's not in nux...14:32
fginthersil2100, I don't have many details yet, just that "- Verify gesture functionality" isn't working.14:35
sil2100Oh, ok14:36
=== yofel_ is now known as yofel
MCR1didrocks: Can and should I use the ubus_manager_ to check if the Dash has been activated with the shortcut key ?16:29
didrocksMCR1: not sure you will get this info, but I didn't look at ubus_manager for a long time :)16:30
bschaeferMCR1, it should send this message when the dash opens. UBUS_OVERLAY_SHOWN16:40
* bschaefer double checks it16:40
bschaeferyeah16:40
didrocksbschaefer: right, but it doesn't special case if it's opened with the shortcut key16:40
didrockswhich is what MCR1 wanted I guess16:41
bschaeferdidrocks, ooo, yeah...wait I think it does...16:41
* bschaefer goes to check16:41
MCR1bschaefer:  I thought more about using UBUS_DASH_EXTERNAL_ACTIVATION16:41
bschaeferubus.SendMessage(UBUS_PLACE_ENTRY_ACTIVATE_REQUEST, g_variant_new("(sus)", "home.lens", dash::NOT_HANDLED, ""));16:41
bschaeferLauncherController.cpp:72716:42
bschaeferMCR1, ^16:42
MCR1bschaefer: thx16:42
bschaeferMCR1, np :)16:42
mhr3and i'll be probably removing ubus_manager :P16:52
bschaefermhr3, please do...it just causes issues16:54
bschaefer:)16:54
bschaeferthings need to be re worked though....16:54
mhr3bschaefer, well, i mean like rewrite it, not get rid of it completely16:55
bschaefermhr3, haha, I was hoping you figured out how to do that16:55
* bschaefer thinks ubus ruins OO design 16:55
mhr3imo it's useful16:55
bschaeferyeaah16:55
bschaeferbut I wonder if things could be designed to fit better16:56
mhr3and it's message passing, isn't that base of OO? :)16:56
bschaeferyeah, but it likes a huge global variable16:56
bschaefererr16:57
bschaeferits talking with things it shouldn't be16:57
mhr3like?16:57
bschaeferlike when the Dash is about to show it sends this ubus message to everyone who is listening (which you never know when it'll get there)16:58
bschaeferwhich causes syncing problems16:58
mhr3fwiw one thing i'll change is that you'll be able to assign priority to the message when sending it16:58
bschaeferoo that would be awesome16:58
bschaeferbut its a queue!16:58
bschaeferpriority queue16:59
mhr3which from my tests is a huge win for responsiveness16:59
bschaefermax/min heap...hmm that should work :)16:59
bschaefermhr3, yeah, ubus does help a lot, but I keep running into times where I NEED the message to be sent and read before going on...16:59
mhr3well the listeners will still be invoked in order they registered17:00
mhr3but the high prio msg will be delivered sooner than the standard ones17:00
didrocksmhr3: do you have so much UBUS traffic to need that?17:00
didrockswas pretty low in my time :)17:01
bschaeferyeah, that would be awesome17:01
mhr3didrocks, no, but it uses idle which is not acceptable for everything17:01
didrocksyou want some sync messages?17:01
mhr3didrocks, didn't you read the bugs which say that dash opens >5seconds after super keypress17:01
bschaeferdidrocks, yes17:01
mhr3that's because it uses idle17:02
didrocksmhr3: didn't see that one TBH :)17:02
mhr3bschaefer, no, no, still fully async17:02
bschaefermhr3, I removed that17:02
* bschaefer always confuses async and sync17:02
didrocksI remember to have reported that more than one in my old system :)17:02
bschaeferbut I know the definition of both haha17:02
bschaefermhr3, that Idle in opening the dash is removed :)17:03
didrocksmhr3: hum, async and no idle to bring it back to the main loop, I'm confused :)17:03
bschaefer(or it will be soon)17:03
mhr3didrocks, async is fine as long as you can specify the prio17:03
mhr3and in compiz it's simple17:04
mhr3HIGH prio - do on next iteration, STANDARD - do when current processing finishes, LOW - do sometimes in the future when the app isn't doing anything (which is once in 30seconds on slow machines)17:05
mhr3right now it defaults to the last, and therefore it sucks17:05
bschaefermhr3, yea, if it would do it next iteration at times I would no longer hate ubus as it is in unity :)17:05
mhr3(ok the 30seconds was a bit exagerrated) :P17:06
bschaeferhaha ~1 second17:06
bschaeferis still to long17:06
didrocksmhr3: hum, I don't get the STANDARD one, you will get the mainloop more than once17:06
mhr3for user-initiated actions, indeed17:06
mhr3didrocks, compiz repaints and reading from sockets is done with standard17:07
bschaefermhr3, wait is this just some sort of dream or in the works?17:08
didrocksok17:08
didrocksmake sense now17:08
MCR1from reading your conversation above it seems to me that the ubus solution might not be the best for my problem17:08
mhr3otherwise said - the usual case is - HIGH: do on next iteration, STANDARD: do in next ~10 iterations, LOW: next ~100?17:08
didrocksmhr3: yeah, more clear that way :)17:09
didrocksyou are basically implemented a g_source? :)17:09
mhr3bschaefer, ehm.... :)17:09
mhr3didrocks, ubus uses gsources, so i'm just explaining how it fits into the global picture of processing events17:10
bschaefermhr3, well I would love to work on that part (If I ever get time) ... as it would be fun to implement a priority queue17:10
didrocksok :)17:10
mhr3bschaefer, you mean like auto q = std::queue<...>(); ?17:10
bschaefermhr3, that is a standard queue17:11
bschaeferFIFO17:11
mhr3oh.. priority17:11
mhr3sorry17:11
bschaeferI don't think that is in the standard...17:11
mhr3bschaefer, so yea auto q = std::priority_queue<...>(); :P17:11
bschaefermhr3, dam, it has been a while since I wrote a heap :(17:11
bschaefermhr3, but that is going to be better then what I could have made haha17:12
mhr3bschaefer, although data structures are awesome, don't expect you'll ever have to implement one :)17:12
bschaefermhr3, i know :) (I can dream though!)17:12
mhr3bschaefer, know their properties though, that's extermly important17:13
bschaefermhr3, yes it is! and awesome it uses a heap :)17:13
mhr3i never got to implementing heap at uni, i randomly picked hashtable instead :P17:14
bschaefermhr3, we got to make a process scheduler (not a real one haha)17:14
bschaeferthat was really fun17:14
mhr3oh that sounds nice17:14
bschaeferwhich in a sense is what you would want with the ubus17:15
bschaefereach new ubus message add to queue17:15
bschaeferlet the heap do its magic17:15
bschaefergrab front17:15
bschaeferthough I am not sure how ubus works with the main loop17:15
mhr3my plan say multimap17:16
mhr3msgs with same priority will be appended, high prio is put where it needs to17:17
bschaefernice, using that binary tree :)17:17
bschaeferhmm so a max of 3 nodes?17:17
bschaeferand each node having a queue?17:18
mhr3honestly i dunno what multimap uses17:18
bschaefer(maps use binary trees)17:18
mhr3but yea, i guess binary tree with lists as nodes17:18
bschaeferhmm that would also work, but if the priority queue is already in the standard it could make things easier17:19
bschaeferso you wouldn't have to make a map<int, queue>17:19
mhr3yea, i'll check it out17:19
bschaefermhr3, cool :) that sounds like fun haha17:19
mhr3anyway, eod for today :)17:19
bschaeferyeah, I should do some work :)17:19
bschaeferhave a good weekend!17:19
=== artfwo_ is now known as artfwo

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