#fluid-work

/

      • mrtyler joined the channel
      • the-t-in-rtf has quit
      • dandimitrov joined the channel
      • mrtyler has quit
      • georgitodorov joined the channel
      • dandimitrov has quit
      • [o__o] joined the channel
      • dandimitrov joined the channel
      • the-t-in-rtf joined the channel
      • dandimitrov has quit
      • dandimitrov joined the channel
      • dandimitrov
        Hey, jhernandez_, should we merge https://github.com/GPII/gpii-app/pull/40 for our tests (in the QSS PR) to run successfully in the CI :?
      • dandimitrov has quit
      • dandimitrov joined the channel
      • dandimitrov has quit
      • dandimitrov joined the channel
      • alf joined the channel
      • dandimitrov has quit
      • Bosmon has quit
      • dandimitrov joined the channel
      • dandimitrov has quit
      • Bosmon joined the channel
      • michelled joined the channel
      • dandimitrov joined the channel
      • cindyli joined the channel
      • cindyli
        jhernandez_: with PSP and QSS, when logging in with "alice", is "alice" displayed somewhere on the UI? I'm playing with gpii-app VM but not seeing the key "alice" is showing somewhere
      • Justin_o joined the channel
      • Bosmon: a question about snapshotter, i should create a file that contains all QSS supported settings. The QSS settings.json (https://github.com/GPII/gpii-app/pull/48/files#...) uses common terms such as "http://registry\\.gpii\\.net/common/language". But invokeSettingsHandlerGet() expects solutionId like "com.microsoft.windows.magnifier", so I used testData/solutions/win32.json5 to convert QSS su
      • pported common terms to solution ids and came up with this json content:
      • [
      • "com.microsoft.windows.screenDPI",
      • // not found in testData/solutions/win32.json5
      • // "http://registry.gpii.net/common/fontSize"
      • "eu.singularlogic.pixelsense.sociable",
      • "net.gpii.uioPlus",
      • "net.gpii.uioPlus",
      • "net.opendirective.maavis",
      • jhung joined the channel
      • IRC sends some errors when I pasted the json string. in case it didn't go thru. here's the pastebin for it - https://pastebin.com/7kiHCDSU
      • Bosmon: is what i'm doing correct?
      • dandimitrov_ joined the channel
      • Bosmon
        cindyli - no
      • cindyli
        :(
      • Bosmon
        Well, I'm not quite sure what you are doing : P
      • Why are you looking up all solution ids that support all common terms?
      • dandimitrov has quit
      • dandimitrov_ is now known as dandimitrov
      • cindyli
        because invokeSettingsHandlerGet() requires solution ids
      • not common terms
      • oh, maybe the matchmaking process can auto convert common terms to solutions
      • but matchmaking happens in the cloud and only for user defined preferences
      • Bosmon
        cindyli - yes ok
      • I guess you will have to go through one round of matchmaking in order to discover what solutions they would be allocated to
      • But it is ok to hard-code this for this current implementation
      • Given the set of preferences is fixed
      • Just go for either the builtin solutions or UIO+ for now
      • cindyli
        you mean hard code common terms or solutions?
      • ok, solutions
      • Bosmon
        cindyli - well in practice you are hardcoding both
      • Since the set supported by the QSS is not going to change quickly
      • Windows language support is coming here: https://github.com/GPII/universal/pull/534
      • cindyli
        Bosmon: there are multiple solutions for these common terms: font size, high contrast, self voicing. without matchmaking, how to decide which one to use?
      • Bosmon
        cindyli ^
      • Pick builtin, or UIO+
      • cindyli
        for example high contrast has
      • "net.opendirective.maavis",
      • "com.microsoft.windows.highContrast",
      • "net.gpii.uioPlus",
      • is "com.microsoft.windows.highContrast" the builtin?
      • Bosmon: i don't know enough to determine which one is builtin
      • Bosmon
        cindyli - yes, that is builtin
      • Anything that begins com.microsoft.windows is builtin
      • cindyli
        ok. thanks
      • Justin_o
        the-t-in-rtf: thanks for the update about gpii-grunt-lint-all. The latest dev release seems to be working. Do you know when you'll have a stable release out?
      • the-t-in-rtf
        Ask @Bosmon.
      • Bosmon
        ask me? : P
      • the-t-in-rtf
        The related pull is waiting on your input, dear.
      • the global excludes stuff there is new
      • dandimitrov
        cindyli: Regarding the keyed in user (“alice”), it is displayed in the tray icon’s context menu - right click on the tray icon
      • cindyli
        great. thanks, dandimitrov
      • dandimitrov: is the string "alice" the received key?
      • dandimitrov
        cindyli: looks like it
      • cindyli
        cool. thanks
      • dandimitrov
        Were you able to find it in your VM :?
      • cindyli
        yes, i did find it, dandimitrov
      • missed the right clicking at the first try :)
      • stegru
        Bosmon: the app zome is ready - but not ready for review, if you know what I mean
      • our baz
      • Bosmon
        zome?
      • clown joined the channel
      • alanharnum joined the channel
      • stegru
        oh, i've been working on the wrong thing?
      • sepidehshahi joined the channel
      • jhernandez_
        dandimitrov: sorry, I'm reading this now - re pull/40, yes, part of it. I can make a pull request against your branch if you want me to
      • but that will only fix the issues related to the creation of the installer
      • dandimitrov
        jhernandez_: that would be great
      • Justin_o
        jhernandez_: i filed another PR against gpii-chrome-extension. Are you able to review it https://github.com/GPII/gpii-chrome-extension/p... and there is the previous one still https://github.com/GPII/gpii-chrome-extension/p...
      • dandimitrov
        I think there are changes to the Build.ps1 script as well
      • g2szeto joined the channel
      • jhernandez_: In the last CI run only the create-vm step is failing because of the `test-windows.c` file
      • colinclark joined the channel
      • jhernandez_
        dandimitrov: right, let me create a tiny pr for you to incorporate into your branch
      • dandimitrov
        jhernandez_: great, thanks
      • cindyli
        Bosmon, colinclark, in a skype call that Greg and Bern had with me yesterday, they want to create 2 user keys and 20 empty user keys for the august demo. clown and I talked about the workflow for creating these keys. we're thinking to add a new testData/preferences/demoUserPrefs directory to hold preferences sets for these keys. convertPrefs.js will run in the same npm postinstall step as converting snapsets and other keys to convert
      • these user keys with "user" flag to build/dbData/demouser/. CD will pick up from this new build dir to load them into staging and production. do you think this workflow is ok?
      • clown: please correct me if i missed anything
      • btw, these 20 empty keys will just be jsons files with UUIDs as file names. their contents will be an empty javascript object
      • colinclark
        But isn't the point that we will give these empty user keys to people to actually use and try out?
      • if so, why would we load them using the same workflow as the canned preference sets?
      • cindyli
        colinclark: greg said he needs these keys just to save the prefs adjusted by QSS
      • colinclark
        (also, three Gs in Gregg)
      • cindyli
        haha, got it
      • i've been wrong with his name all the time, sorry, Gregg
      • :)
      • colinclark
        hahaha
      • cindyli
        our canned prefs sets in testData/preferences are loaded as snapsets, not user keys
      • this is a temp workflow for the august demo for loading test user keys
      • Greg has a different requirement for creating real user keys that will be handed out to users.
      • he will write that requirement in a doc for review
      • colinclark
        ok, i think whatever is easiest is probably fine
      • cindyli
        ok. i think so too
      • colinclark
        there's no risk that these keys will be written over in later deployments?
      • cindyli
        i think no, because our dataloading will only reload snapsets, not user keys. if later deployments, the same UUID is posted, couchdb will throw error
      • clown
        cindyli, colinclark, just got back from getting coffee: reading log.
      • colinclark
        ok, smart data loader :)
      • clown
        cindyli, your description of how I've handled the "demo user preferences" is correct.
      • clown wonders if colin thinks the data is smart, or the loader, or the coder
      • cindyli
        all of them, in my opinion :)
      • clown
        as for the workflow, my understanding is that we are doing it this way for the demo, and it's relatively easy for us to follow the cow path we created for the snapsets.
      • whether this is good approach in the long run is beyond me, at present. Likely not.
      • but I find it weird that the demo will not display "roy" as the name for a user's preferences (for example), but a UUID. Ah well, it's a demo.
      • clown goes back to figuring out AWS
      • dandimitrov has quit
      • the-t-in-rtf joined the channel
      • jhernandez_
        Justin_o: re the extension pulls, yes, happy to review them, but not sure by when
      • but will do my best to get to them ASAP
      • clown, cindyli: regarding your work on the dataloader, do you think that you can give us a quick tour on it during our apcp meeting today? I guess that I would also like to know how are we going to proceed with the upcoming demo
      • Justin_o
        jhernandez_: okay thanks.. i'm not exactly sure when it's needed.. it sounded like it might be needed for the august demo, but then in another call i had, it sounded like it wasn't will probably need to confirm that with Sandra
      • clown
        jhernandez_: quick tour — sure.
      • avtar joined the channel
      • jhernandez_
        Justin_o: yeah, it's hard for me to say too - maybe we should just double check with Sandra/Gregg
      • dandimitrov joined the channel
      • Justin_o
        yes, i think so
      • gtirloni joined the channel