fluid-work IRC Logs-2014-01-09
[08:23:15 CST(-0600)] <Bosmon> kasparnet - so I am just looking at https://github.com/GPII/windows/pull/42/files, who has just revealed that https://github.com/GPII/windows/pull/28/files was never merged
[08:23:19 CST(-0600)] <Bosmon> Which I find completely astonishing
[08:23:26 CST(-0600)] <Bosmon> Just going and looking to see what this all means
[08:27:56 CST(-0600)] <kasparnet> Bosmon: I think we had it in the pilot branch of olden times - and then it has just been laying there
[08:28:16 CST(-0600)] <kasparnet> Bosmon: good news is that nothing depends on it (yet) - but there are two pull requests pending that do
[08:33:05 CST(-0600)] <Bosmon> kasparnet - well, I'm still trying to piece it together
[08:33:23 CST(-0600)] <Bosmon> It seems we have had the gpii.lifecycleManager.registryResolver for years
[08:33:29 CST(-0600)] <Bosmon> Can it really be the case that it's just never been wired up?
[08:34:03 CST(-0600)] <Bosmon> I'm pretty sure it was initially, although using a very bad scheme
[08:34:08 CST(-0600)] <Bosmon> Was that code just removed at some point?
[08:37:39 CST(-0600)] <kasparnet> Bosmon: It probably was only ever wired up in the notorious Pilot branch – and then during the porting of stuff to master the demands block etc never made it across
[08:37:39 CST(-0600)] <kasparnet> that's my best guess at least
[08:40:19 CST(-0600)] <Bosmon> Well, that doesn't quite make sense
[08:40:29 CST(-0600)] <Bosmon> Since clearly it was invented to BE a "resolver"
[08:40:37 CST(-0600)] <Bosmon> I mean, we invented resolvers largely for this purpose
[08:41:48 CST(-0600)] <Bosmon> oh well
[08:42:09 CST(-0600)] <Bosmon> Hi yzen - I was just looking at the registryResolver together with kasparnet
[08:42:17 CST(-0600)] <Bosmon> And trying to figure out what on earth happened with this
[08:42:22 CST(-0600)] <yzen> yes
[08:42:26 CST(-0600)] <yzen> happened with what ?
[08:42:36 CST(-0600)] <Bosmon> Well, I can't understand how it could be that it was never wired up
[08:43:05 CST(-0600)] <yzen> i m was about to take a look at the pull
[08:43:14 CST(-0600)] <Bosmon> But I guess this stuff is just so so old
[08:43:23 CST(-0600)] <Bosmon> I am looking at this comment now that I put there in the LifecycleManager
[08:53:09 CST(-0600)] <cindyli> amilchev: you might have known that Justin_o_ and I are combing you and Chris' work together to comply with the design - http://wiki.fluidproject.org/download/attachments/34570511/PMTandPCP-for2pilots.pdf?version=2&modificationDate=1386791493919&api=v2. in particular to implement the switch in btw PMT and PCP. We wonder which branch has your latest PCP work? adjusters-pilots-2 or frames?
[08:57:20 CST(-0600)] <Bosmon> kasparnet, yzen - as far as I can see, the only thing wrong with pull 42 is that you have failed to apply the base grade "fluid.applyGradeLinkage"
[08:57:34 CST(-0600)] <Bosmon> Currently the "grade linkage" system is on an "opt-in" basis since it is quite expensive
[08:59:49 CST(-0600)] <yzen> Bosmon: i thought kasparnet addressed that
[09:01:57 CST(-0600)] <amilchev> cindyli: hi, right now the PCP frame is in adjuster-pulots-2
[09:04:11 CST(-0600)] <Bosmon> yzen - you're right
[09:04:27 CST(-0600)] <yzen> ok I'm going to take a look as well
[09:06:41 CST(-0600)] <Bosmon> Ah, yzen, kasparnet - the issue is that fluid.applyGradeLinkage is applied to the wrong component
[09:06:57 CST(-0600)] <Bosmon> It is applied to the "gpii.lifecycleManager.resolver" itself
[09:07:15 CST(-0600)] <Bosmon> Whereas it should really be applied to the resolverConfig component
[09:08:18 CST(-0600)] <yzen> yes that was one of them but i think he said that even adding it to standard resolver config
[09:08:29 CST(-0600)] <yzen> did not work
[09:08:32 CST(-0600)] <Bosmon> Ok
[09:08:45 CST(-0600)] <Bosmon> Well, perhaps it didn't - but it is currently missing from universal trunk
[09:09:47 CST(-0600)] <cindyli> got it. thanks, amilchev
[09:10:11 CST(-0600)] <yzen> Bosmon: ok ill update his/my pulls
[09:12:20 CST(-0600)] <Bosmon> When you add the linkage grade to the resolverConfig, you can also remove the silly 4-line comment above it
[09:12:50 CST(-0600)] <Bosmon> It might also be good to split "standardResolverConfig" into two, and apply the linkage grade to "resolverConfig" in a base grade
[09:16:06 CST(-0600)] <kasparnet> Bosmon, yzen: sorry - was afk didnt see comments till now
[09:16:27 CST(-0600)] <yzen> kasparnet: Bosmon i updated 115 for universal https://github.com/GPII/universal/pull/121/
[09:22:19 CST(-0600)] <yzen> yo kasparnet i know why it doesn't work
[09:31:58 CST(-0600)] <amilchev> cindyli: by the way, small change of branches - the combined adjusters (from the pull request) are still in adjusters-pilots-2, and the PCP frame is now in branch pcp-frame
[09:32:12 CST(-0600)] <amilchev> just letting you know
[09:32:41 CST(-0600)] <cindyli> thanks, amilchev, do i need to combine them together?
[09:33:31 CST(-0600)] <amilchev> cindyli: no, the pcp frame contains all adjusters, like here http://wiki.fluidproject.org/download/attachments/34570511/PMTandPCP-for2pilots.pdf?version=2&modificationDate=1386791493919&api=v2
[09:34:54 CST(-0600)] <cindyli> that's great. pcp-frame is the one we should use. thanks, amilchev
[09:55:05 CST(-0600)] <cindyli> amilchev: the translation in src/shared/frames/messages/de/frames.json is in english, is that what you want?
[09:55:28 CST(-0600)] <cindyli> same thing with the "more", "less" translation in src/shared/frames/messages/el/frames.json
[09:57:35 CST(-0600)] <amilchev> cindyli: didn't know about that, just used Chris' files, because I thought he had all the translations
[09:58:16 CST(-0600)] <amilchev> cindyli: you can use https://github.com/chrispetsos/prefsEditors/tree/frames-pilots-2/src/shared/frames/messages
[09:58:24 CST(-0600)] <amilchev> sorry about that
[09:58:42 CST(-0600)] <cindyli> no problem, thanks, amilchev
[10:06:46 CST(-0600)] <cindyli> amilchev: I have some conflicts when merging your pcp-frame branch into our combined branch which contains the pull request from Chris for the merged work from you both, especially with frames.css. I wonder if you have time to help solve that conflict?
[10:12:18 CST(-0600)] <amilchev> cindyli: I was about to leave about an hour ago ... is it possible that I just name the file differently so that the merge is easier and fix that after the EU meeting?
[10:13:01 CST(-0600)] <cindyli> sounds good, amilchev, if the renaming is not going to affect ur styling
[10:14:49 CST(-0600)] <amilchev> cindyli: no, I mean if frames.css is named pcpFrames.css, the merge would be successful and the styles still used .. as stupid as that is, sounds fast
[10:16:18 CST(-0600)] <amilchev> otherwise i won't be able to help you with the conflicts unfortunately
[10:18:28 CST(-0600)] <cindyli> amilchev: cool. i can do the rename if you are leaving
[10:18:54 CST(-0600)] <amilchev> thanks, cindyli, bye and hope everything's in place
[10:18:58 CST(-0600)] <cindyli> thanks for the help. sorry for keeping u so late. see you tomorrow
[11:09:17 CST(-0600)] <avtar> the-t-in-rtf: ping
[11:11:31 CST(-0600)] <colinclark> i'm rambling to the-t-in-rtf on Skype, avtar
[11:11:35 CST(-0600)] <colinclark> but we're almost done
[11:11:39 CST(-0600)] <avtar> ok
[11:13:20 CST(-0600)] <the-t-in-rtf> sorry, yes, almost done.
[11:14:32 CST(-0600)] <the-t-in-rtf> I don't seem to have your Skype details?
[11:16:47 CST(-0600)] <the-t-in-rtf> Anyway, I am ready when you are, just need to know how to reach you.
[11:18:13 CST(-0600)] <avtar> the-t-in-rtf: i sent you a contact request a few months ago but you never added me :'(
[11:18:20 CST(-0600)] <the-t-in-rtf> I am so sorry.
[11:19:06 CST(-0600)] <the-t-in-rtf> I see two avtar gils in Skype, one in the UK, one in India.
[11:19:21 CST(-0600)] <avtar> both are impostors
[11:19:27 CST(-0600)] <avtar> avtar.work
[12:10:59 CST(-0600)] <Justin_o_> kasparnet: i have a question about the common terms if you are available.
[12:11:37 CST(-0600)] <kasparnet> sure
[12:11:38 CST(-0600)] <kasparnet> Justin_o_:
[12:12:40 CST(-0600)] <Justin_o_> kasparnet: the prefsEditors save their models in a different format than what is in the common terms registry. yzen suggested we transform this when saving/fetching from the preferences server.
[12:13:02 CST(-0600)] <Justin_o_> kasparnet: yzen said this could be used to get the keys (URI) https://github.com/GPII/universal/blob/master/testData/ontologies/ISO24751-2/flat.json
[12:13:12 CST(-0600)] <Justin_o_> kasparnet: however i don't know what format the values should be in
[12:13:26 CST(-0600)] <Justin_o_> kasparnet: would you have something that specifies this?
[12:14:48 CST(-0600)] <kasparnet> Justin_o_: https://docs.google.com/spreadsheet/ccc?key=0AppduB_JZh5EdGltZnF3dVpKdXcxSVhEZ0VjZGY1U3c&usp=drive_web#gid=0 contains value ranges, etc
[12:15:12 CST(-0600)] <Justin_o_> kasparnet: thanks.. i don't have access to that file though, but have request access
[12:15:46 CST(-0600)] <kasparnet> if you private msg me the mail associated with your google account, i'll share
[12:16:13 CST(-0600)] <kasparnet> Justin_o_: what format are the prefseditors using?
[12:18:36 CST(-0600)] <Justin_o_> kasparnet: i sent you a pm with my e-mail address
[12:18:55 CST(-0600)] <cindyli> kasparnet: i sent you too
[12:19:03 CST(-0600)] <Justin_o_> kasparnet: this is what gets saved to the preferences server from the pmt. http://pastie.org/private/etbn769nwpiedkdof7lrq
[12:19:28 CST(-0600)] <kasparnet> cindyli, Justin_o_: looks like I dont have privs to share it, so created a copy: https://docs.google.com/spreadsheet/ccc?key=0AifE6R0ipKCjdFpDQnV2UkVjaFVBMFFObmlzVXNMTVE&usp=sharing
[12:19:42 CST(-0600)] <Bosmon> yzen, kasparnet - working on this now: http://issues.fluidproject.org/browse/FLUID-5246
[12:19:44 CST(-0600)] <Justin_o_> kasparnet: thanks
[12:19:45 CST(-0600)] <cindyli> thanks, kasparnet
[12:19:55 CST(-0600)] <kasparnet> cool - thanks Bosmon
[12:25:57 CST(-0600)] <Justin_o_> cindyli: another problem.. looks like the prefsFramework is dying on uri's
[12:26:37 CST(-0600)] <cindyli> oh no, coming, Justin_o_
[12:49:35 CST(-0600)] <Bosmon> yzen, kasparnet - I've pushed my fix here https://github.com/amb26/infusion/tree/FLUID-5246
[12:49:45 CST(-0600)] <Bosmon> But horrifyingly, the github UI for making a pull request appears to be broken
[12:49:56 CST(-0600)] <Bosmon> But perhaps you can do the review and merge anyway
[12:50:24 CST(-0600)] <Bosmon> At least the compare view is still working
[12:50:26 CST(-0600)] <Bosmon> https://github.com/amb26/infusion/compare/FLUID-5246
[13:00:05 CST(-0600)] <kasparnet> Bosmon: sure! Though I dont know much (read: anything) about the internal working of grades in infusion - but I can look over the code the best I can, run the tests, and check that it solves the problem I ran into
[13:00:37 CST(-0600)] <Bosmon> Well I guess it is mainly for yzen to merge it upstream first
[13:00:43 CST(-0600)] <Bosmon> And then you can update universal, etc.
[13:05:06 CST(-0600)] <kasparnet> Bosmon: ok - so would I do a first check, then pass on to yzen for merging? or would yura take care of review+merge and then I'll do update of references?
[13:05:35 CST(-0600)] <Bosmon> The latter, I think
[13:06:02 CST(-0600)] <Bosmon> I mean, the reported issue is clearly resolved, so unless this just goes on to expose a further issue, everything should be ok
[13:06:24 CST(-0600)] <kasparnet> ok cool, thanks for fixing this so quickly Bosmon
[13:06:58 CST(-0600)] <Bosmon> Let me know what I should be looking at next
[13:20:49 CST(-0600)] <Bosmon> kasparnet ^
[13:40:31 CST(-0600)] <Bosmon> ok, stupidly I couldn't make the pull request because I wasn't actually logged on
[13:40:44 CST(-0600)] <Bosmon> You would think that the UI would simply remove the entire button in that case....
[13:59:05 CST(-0600)] <Justin_o_> Bosmon, kasparnet: we have some questions about transformations
[14:00:39 CST(-0600)] <Bosmon> Hi Justin_o_
[14:01:01 CST(-0600)] <kasparnet> Justin_o_: sure
[14:01:50 CST(-0600)]
<Justin_o_> Bosmon, kasparnet: thanks.. so what we want to do is to transform something with a structure like
to {b: [
]
[14:02:08 CST(-0600)] <Justin_o_> }
[14:02:16 CST(-0600)] <Justin_o_> sorry, missed the closing bracket there
[14:02:25 CST(-0600)] <Bosmon> Justin_o_ - so are the keys "b" and "value" fixed?
[14:03:00 CST(-0600)] <Justin_o_> Bosmon: yes.. i believe so
[14:04:07 CST(-0600)] <Justin_o_> Bosmon: really we're trying to transform the model used by the prefs editors to one that matches the common terms.. currently when we save we get something that looks like this http://pastie.org/private/etbn769nwpiedkdof7lrq
[14:04:28 CST(-0600)]
<Bosmon> So I imagine in that case something as simple as
might do
[14:04:36 CST(-0600)] <Bosmon> Depending on exactly how kasparnet implemented the rules for arrays
[14:04:40 CST(-0600)] <Bosmon> He will have to remind us
[14:05:16 CST(-0600)] <Justin_o_> Bosmon: we do also have cases where "val" needs to be transformed.. for example magnification needs to be scaled
[14:20:40 CST(-0600)] <Justin_o_> Bosmon: thanks, so tha would work, however.... are new keys are actually url's but even though they are escaped.. it's causing the output to be objects instead of arrays with keys for the position.. e.g. 0
[14:21:03 CST(-0600)] <Justin_o_> seems like this is the case for anything that has a . in the new key name.. like "something
.a"
[14:21:18 CST(-0600)] <kasparnet> Bosmon, Justin_o_: sorry for being unresponsive - in a call - brb
[14:28:58 CST(-0600)] <michelled> Bosmon: I noticed that yzen has made some updates to his pull request: https://github.com/GPII/universal/pull/174
[14:29:05 CST(-0600)] <michelled> could you please take a look at it?
[14:29:18 CST(-0600)] <michelled> I'm happy to help test it, if you'd like
[14:36:00 CST(-0600)] <michelled> colinclark: I think GPII-251 is ready to be pushed
[14:38:17 CST(-0600)] <kasparnet> Hi Justin_o_
[14:38:22 CST(-0600)] <kasparnet> was just looking at the transformations
[14:39:05 CST(-0600)] <kasparnet> the function to make things array is: fluid.transforms.arrayValue (http://wiki.gpii.net/index.php/Architecture_-_Available_transformation_functions#To_Array_.28fluid.transforms.arrayValue.29)
[14:39:09 CST(-0600)] <Justin_o_> kasparnet: ah okay.. thanks.. so the simple syntax that Bosmon recommended would work if it wasn't for the escaped dots in our key. So we were looking at the arrayToObject transformer instead, but can't get it to work
[14:39:54 CST(-0600)] <kasparnet> ah - no, that one is a bit more complex
[14:40:41 CST(-0600)] <kasparnet> (and I keep forget how it's working )
[14:42:29 CST(-0600)] <Bosmon> Hi michelled - I was looking at pull 174 but I can't completely understand yzen's comment
[14:42:34 CST(-0600)] <Bosmon> on the file "input_TestCase12_new.json"
[14:42:47 CST(-0600)] <Bosmon> He has said "This file has an example of expressing a preference for specific type of a match maker. It is not actually used other than when logging the user in and out." - does this make sense to you?
[14:42:49 CST(-0600)] <Justin_o_> kasparnet: okay.. so arrayValue looks helpful.. are we able to transform the value inside of that too
[14:43:03 CST(-0600)] <Bosmon> Justin_o_ - of course all transformation expressions can cascade
[14:44:01 CST(-0600)] <Bosmon> One day sgithens will help us design the "transformation expression designer" : P
[14:44:08 CST(-0600)] <Bosmon> Hopefully, actually one day this year
[14:44:15 CST(-0600)] <Bosmon> As we begin to embark on the FINAL PROJECT
[14:44:16 CST(-0600)] <sgithens> Bosmon : TO END ALL DESIGNS!
[14:44:29 CST(-0600)] <sgithens> I"VE SEEN IT ALL MY LIFE!
[14:44:48 CST(-0600)] <sgithens> my ride is leaving in 4 minutes
[14:44:50 CST(-0600)] <sgithens> be back later
[14:44:56 CST(-0600)] <Bosmon> To PHILLERDELPPHIA?
[14:44:58 CST(-0600)] <sgithens> no
[14:45:00 CST(-0600)] <sgithens> home
[14:45:19 CST(-0600)] <sgithens> my bike is broke, I'm waiting for my new one to arrive, and the weather/roads are crap
[14:45:34 CST(-0600)]
<Justin_o_> Bosmon, kasparnet: how would i use those transforms with say this
to
[14:45:41 CST(-0600)] <sgithens> campus was closed for 2 days this week
[14:45:49 CST(-0600)] <Justin_o_> so i'd need to use the linear scale, but i don't see how to cascade those transformations
[14:46:03 CST(-0600)] <sgithens> 5 houses across the street from me lost power for 3 days, and several ended up with burst water pipes
[14:46:04 CST(-0600)] <Bosmon> Justin_o_ - you can write a "transform" block anywhere where you write a value
[14:46:07 CST(-0600)] <sgithens> it's been a mess
[14:46:12 CST(-0600)] <michelled> yzen, good timing - Bosmon was just saying that he needed some clarification on your comment on pull 174
[14:46:15 CST(-0600)] <Bosmon> Sorry to hear that, GITTHUEN
[14:46:25 CST(-0600)] <sgithens> THE BLUE HOUSE WAS UNSCATHED!
[14:46:32 CST(-0600)] <Bosmon> Excellent news
[14:46:36 CST(-0600)] <sgithens> except for a very very large tree limb that came down
[14:46:44 CST(-0600)] <sgithens> I'll need to deal with it, but it did minimal damage
[14:46:57 CST(-0600)] <sgithens> colinclark may still have my space station story
[14:47:06 CST(-0600)] <yzen> Bosmon: what was it that you wanted me to clarify ?
[14:47:15 CST(-0600)] <Bosmon> yzen: "This file has an example of expressing a preference for specific type of a match maker. It is not actually used other than when logging the user in and out."
[14:47:15 CST(-0600)] <kasparnet> Justin_o_: I would think it should be something like:
[14:47:24 CST(-0600)] <kasparnet> {
[14:47:24 CST(-0600)] <kasparnet> "b": {
[14:47:24 CST(-0600)] <kasparnet> "transform": {
[14:47:25 CST(-0600)] <kasparnet> "type": "fluid.transforms.arrayValue",
[14:47:27 CST(-0600)] <kasparnet> "input": {
[14:47:29 CST(-0600)] <kasparnet> "transform": {
[14:47:31 CST(-0600)] <kasparnet> "outputPath": "value",
[14:47:33 CST(-0600)] <kasparnet> "type": "fluid.transform.linearScale",
[14:47:35 CST(-0600)] <kasparnet> "valuePath": "display.magnification",
[14:47:37 CST(-0600)] <kasparnet> "factor": 0.1
[14:47:39 CST(-0600)] <kasparnet> }
[14:47:41 CST(-0600)] <kasparnet> }
[14:47:43 CST(-0600)] <kasparnet> }
[14:47:45 CST(-0600)] <kasparnet> }
[14:47:47 CST(-0600)] <kasparnet> }
[14:47:51 CST(-0600)] <kasparnet> ugh - sorry about that
[14:47:53 CST(-0600)] <kasparnet> http://piratepad.net/Qqb35JnkGC
[14:47:55 CST(-0600)] <colinclark> pastie is very helpful
[14:47:55 CST(-0600)] <Bosmon> So this comment does seem to imply that the file is used somehow
[14:47:56 CST(-0600)] <kasparnet> but it seems there might be an issue
[14:47:58 CST(-0600)] <Bosmon> That's fine, kasparnet
[14:48:06 CST(-0600)] <Bosmon> But I think you missed off the inputpath
[14:48:14 CST(-0600)] <Bosmon> Presumably the transform should say "inputPath": "a"
[14:48:33 CST(-0600)] <kasparnet> ah yes
[14:48:42 CST(-0600)] <Bosmon> I guess you just copied it from another example
[14:48:43 CST(-0600)] <kasparnet> but it seems to have difficulties creating things inside an array
[14:49:00 CST(-0600)] <yzen> Bosmon: sure so it means it can be used as any other item in preferences directory, by trying to login and logout as that user
[14:49:26 CST(-0600)] <Bosmon> yzen - ok - is this fact documented anywhere?
[14:49:56 CST(-0600)] <kasparnet> Bosmon, Justin_o_: if you try what I pasted here http://piratepad.net/Qqb35JnkGC it gives an empty array - which I dont believe it should
[14:50:00 CST(-0600)] <Bosmon> I guess there are lots and lots of such users
[14:50:09 CST(-0600)] <yzen> Bosmon: so far only 1
[14:50:10 CST(-0600)] <yzen> that one
[14:50:15 CST(-0600)] <Justin_o_> kasparnet:
[14:50:18 CST(-0600)] <Justin_o_> i'll try it
[14:50:28 CST(-0600)] <kasparnet> (you can use http://kasper.markus.dk/transformTester/ for a quick test)
[14:50:34 CST(-0600)] <Bosmon> yzen - well, I mean, the testData/preferences data is filled
[14:50:38 CST(-0600)] <kasparnet> it seems to have problems with objects inside arrays
[14:50:39 CST(-0600)] <Bosmon> kasparnet - excellent tool!
[14:51:13 CST(-0600)] <Justin_o_> kasparnet: thanks for the tool...
[14:51:24 CST(-0600)] <kasparnet> Bosmon: thanks - except the indentation and tab-usage is horribly frustrating
[14:51:27 CST(-0600)] <kasparnet>
[14:51:39 CST(-0600)] <Bosmon> yzen - my issue is how the user or anyone is intended to understand the purposes of this file
[14:51:41 CST(-0600)] <Bosmon> https://github.com/yzen/universal/blob/e285c7c0bb219f6b143e5374d5f48fb6dfd20c4e/testData/preferences/input_TestCase12_new.json
[14:51:50 CST(-0600)] <Bosmon> Its name suggests that it is a test case of some kind
[14:51:58 CST(-0600)] <Bosmon> But it is unclear what it is a test case for, and in what contexts it should be used
[14:52:12 CST(-0600)] <Bosmon> And, confusingly, it doesn't actually correspond to a test case or any acceptance script that we seem to have
[14:52:25 CST(-0600)] <yzen> Bosmon: ok i understand the name issue, i put at as is as nick (rule based mm) had it
[14:52:26 CST(-0600)] <kasparnet> I seem to remember that we somehow needed to specify on what levels we had arrays
[14:52:37 CST(-0600)] * kasparnet is looking through tests
[14:52:44 CST(-0600)] <Bosmon> kasparnet - ah, you mean the schema issue?
[14:52:54 CST(-0600)] <Bosmon> YOu would think that the "arrayValue" transform was perfectly good enough for outputing arrays
[14:53:03 CST(-0600)] <Bosmon> I mean, it's all in the name
[14:53:06 CST(-0600)] <kasparnet> Bosmon: well - it does output an array
[14:53:19 CST(-0600)] <kasparnet> the problem is trying do stuff inside the array that fails
[14:53:44 CST(-0600)] <kasparnet> a simple:
[14:53:47 CST(-0600)] <kasparnet> {
[14:53:48 CST(-0600)] <kasparnet> "b": {
[14:53:48 CST(-0600)] <kasparnet> "transform": {
[14:53:49 CST(-0600)] <kasparnet> "type": "fluid.transforms.arrayValue",
[14:53:51 CST(-0600)] <kasparnet> "inputPath": "a"
[14:53:53 CST(-0600)] <kasparnet> }
[14:53:55 CST(-0600)] <kasparnet> }
[14:53:57 CST(-0600)] <kasparnet> }
[14:53:59 CST(-0600)] <kasparnet> seem to work ok
[14:54:08 CST(-0600)] <Bosmon> kasparnet
[14:54:15 CST(-0600)] <Bosmon> Quit making stuff that fails, damn you
[14:54:30 CST(-0600)] <Bosmon> Ok, if one of you can write up a JIRA and make a test case pull, I will try to look at it very quickly
[14:54:38 CST(-0600)] <Bosmon> I assume this issue belongs in the "very very urgent" pile
[14:55:01 CST(-0600)] <yzen> Bosmon: so if i rename it into something meaningful would that be considered reasonable ?
[14:55:38 CST(-0600)] <Justin_o_> Bosmon: yes, very very urgent
[14:55:53 CST(-0600)] <Bosmon> yzen - well I guess the issue is, "where is it explained to the users of the system what users they can log on as and what effects they should observe"
[14:56:03 CST(-0600)] <Bosmon> I mean I guess this is really an issue for kasparnet more than for you
[14:56:03 CST(-0600)] <kasparnet> Bosmon, Justin_o_: ok, I'll do the JIRA and make a pull request with a failing test
[14:56:16 CST(-0600)] <kasparnet> michelled: will just do ^ and then get back to GPII-75
[14:56:17 CST(-0600)] <Justin_o_> kasparnet: thanks
[14:56:20 CST(-0600)] <Bosmon> Everything is an issue for kasparnet today
[14:56:40 CST(-0600)] <Bosmon> kasparnet - so what do you think is the purpose of all the files we have in https://github.com/yzen/universal/tree/e285c7c0bb219f6b143e5374d5f48fb6dfd20c4e/testData/preferences
[14:56:47 CST(-0600)] <Bosmon> And where do we explain to people what they all do?
[14:56:57 CST(-0600)] <yzen> sure but as for me, Bosmon , what would you like in terms of this pull request?
[14:57:15 CST(-0600)] <Bosmon> Well, let's get an answer to this question first, and then we will know what to do
[14:57:28 CST(-0600)] <Bosmon> In the meantime you can review https://github.com/fluid-project/infusion/pull/449
[14:58:14 CST(-0600)] <kasparnet> Bosmon: well - we're trying to move to a scheme where NP sets that test an application has that name (eg. easit1.json, easit2.json, maavis1.json, maavis2.json) ... and then we just have the old ones
[14:58:30 CST(-0600)] <yzen> Bosmon: ok
[14:58:31 CST(-0600)] <kasparnet> Bosmon: things like "audio.json" are a mystery to me as well
[14:58:52 CST(-0600)] <Bosmon> kasparnet - ok, so in theory you regard the introduction of this file "input_TestCase12_new.json" as unwelcome : P
[14:59:48 CST(-0600)] <kasparnet> haha - I do find that slightly confusing yes
[15:00:25 CST(-0600)] <kasparnet> but test_GPII-XYZ.json would probably be good
[15:01:20 CST(-0600)] <kasparnet> (if that's applicable to your case)
[15:01:40 CST(-0600)] <Bosmon> kasparnet - the name could be improved - but there still remains the issue of how people know what the files are good for
[15:01:50 CST(-0600)] <kasparnet> yeah
[15:01:53 CST(-0600)] <kasparnet> I agree
[15:01:56 CST(-0600)] <Bosmon> I assume that in theory, the ideal would be to have this added to an acceptance test
[15:02:01 CST(-0600)] <Bosmon> Do you agree?
[15:02:11 CST(-0600)] <kasparnet> all the NP sets?
[15:02:31 CST(-0600)] <Bosmon> Well, let's start with this file first
[15:02:34 CST(-0600)] <Bosmon> And then we can widen the question
[15:10:21 CST(-0600)] <Justin_o_> Bosmon: I've also filed a jira for the issue with the simple syntax you suggested, and have also submitted an accompanying pull request with a test case
[15:10:22 CST(-0600)] <Justin_o_> http://issues.fluidproject.org/browse/FLUID-5247
[15:10:31 CST(-0600)] <Bosmon> Thank you, Justin_o_
[15:10:43 CST(-0600)] <Justin_o_> Bosmon: if even this issue is solved i think we can write the needed model transformation
[15:13:15 CST(-0600)] <kasparnet> Bosmon: in terms of the transformation issue, would the fix be that the schema should be automatically generated on usage of arrayValue?
[15:18:27 CST(-0600)] <Bosmon> kasparnet - I'm not sure, you would think that no schema would be needed since the output value should be an array
[15:18:34 CST(-0600)] <Bosmon> I'll have to see what the fix entails when I look at the test case
[15:18:47 CST(-0600)] <kasparnet> yeah - I guess you got a point
[15:35:01 CST(-0600)] <kasparnet> I guess my tool is a bit outdated – at least I'm getting different results.. still wrong ones though
[15:46:05 CST(-0600)] <Bosmon> kasparnet - how do you host it?
[15:46:39 CST(-0600)] <kasparnet> it was just something I set up on my server during portugal - manually pulling in infusion
[15:46:51 CST(-0600)] <kasparnet> I should probably set up a cron script
[15:47:00 CST(-0600)] <Bosmon> Perhaps putting on "github pages" might lead to a way for it to be automatically updated
[15:47:17 CST(-0600)] <Bosmon> But then again, perhaps it might not : P
[15:51:30 CST(-0600)] <kasparnet> Bosmon: https://github.com/kaspermarkus/infusion/compare/FLUID-5248
[15:52:33 CST(-0600)] <kasparnet> Bosmon: I feel that I did something completely crazy though – my head is a destroyed right now – so it would probably make sense to just check that my test-case is even sensible
[15:52:50 CST(-0600)] <kasparnet> Bosmon: should I do an actual pull request somewhere?
[15:52:51 CST(-0600)] <Bosmon> kasparnet - is this trying to demonstrate the same issue as Justin_o_'s FLUID-5247?
[15:53:17 CST(-0600)] <Bosmon> Good grief
[15:53:22 CST(-0600)] <Bosmon> Neither of you read a single thing the other wrote
[15:53:27 CST(-0600)] <Bosmon> Let's see who made the better JIRA : P
[15:53:31 CST(-0600)] <kasparnet> lol
[15:53:38 CST(-0600)] <kasparnet> Bosmon: lets not
[15:53:42 CST(-0600)] <kasparnet> mine is horrific
[15:53:58 CST(-0600)] <kasparnet> and I doubt I can compete with the king when it comes to JIRAs
[15:54:08 CST(-0600)] <Bosmon> Well I think yours actually wins the prize
[15:54:15 CST(-0600)] <Bosmon> Since Justin_o_'s JIRA is actually self-contradictory
[15:54:30 CST(-0600)] <kasparnet> haha yay – take that Justin_o_
[15:55:31 CST(-0600)] <Bosmon> His description claims the issue is something to do with keys containing periods, but then his example doesn't demonstrate it
[15:57:10 CST(-0600)] <kasparnet> Justin_o_: ok... looks like I dont win the contest for speed though: " jobara added a commit an hour ago "
[15:57:12 CST(-0600)] <kasparnet> hehe
[16:00:27 CST(-0600)] <Bosmon> So kasparnet - rather than duplicating Justin_o_'s work, whilst we're waiting for him to come back, would you like to take a stab at the question above?
[16:00:54 CST(-0600)] <Bosmon> yzen has committed a file which we would like to know what you would prefer done to it
[16:00:58 CST(-0600)] <Bosmon> And then, all similar files
[16:01:34 CST(-0600)] <kasparnet> the NP set question?
[16:02:00 CST(-0600)] <Bosmon> Yes
[16:02:10 CST(-0600)] <michelled> Bosmon, kasparnet: Justin_o_ won't be back until tomorrow
[16:02:17 CST(-0600)] <Bosmon> Or more broadly, all of the files in https://github.com/yzen/universal/tree/e285c7c0bb219f6b143e5374d5f48fb6dfd20c4e/testData/preferences
[16:02:37 CST(-0600)] <Bosmon> It sounds like part of your preferred answer is that "these should be the same thing"
[16:02:54 CST(-0600)] <Bosmon> But presumably there is some value in the rest of the files, even though it seems in many cases it is now very unclear what exactly it is
[16:03:11 CST(-0600)] <Bosmon> But we can't just have people dumping random junk in here without a plan : P
[16:05:49 CST(-0600)] <Bosmon> kasparnet
[16:06:18 CST(-0600)] <kasparnet> well - I think in the longer term, we should have a folder with NP sets used for acceptance testing, a folder for NP sets used for demos, etc
[16:06:27 CST(-0600)] <Bosmon> ok
[16:06:40 CST(-0600)] <kasparnet> and then handle select the appropriate folder via configs
[16:06:40 CST(-0600)] <Bosmon> It sounds reasonable to organise this directory better
[16:06:45 CST(-0600)] <kasparnet> yeah
[16:07:00 CST(-0600)] <kasparnet> but perhaps we should leave the mess for now
[16:07:08 CST(-0600)] <Bosmon> Perhaps we shouldn't
[16:07:15 CST(-0600)] <kasparnet> I hate you
[16:07:16 CST(-0600)] <michelled> maybe we should open a JIRA for it
[16:07:18 CST(-0600)] <kasparnet>
[16:07:19 CST(-0600)] <Bosmon> Since we have a golden opportunity in this panic to actually get people off their arses
[16:07:25 CST(-0600)] <Bosmon> Perhaps we should DO IT NOW
[16:07:37 CST(-0600)] <Bosmon> Before the thing slides a further 7 months like that Windows pull request
[16:08:56 CST(-0600)] <kasparnet> I'm working 15 hour days and my todo list is still terribly long - and I am really scared to mess with something before the EC review
[16:09:15 CST(-0600)] <kasparnet> would it be ok if we create a JIRA and set it as fix-for pilots
[16:09:15 CST(-0600)] <kasparnet> (1 month from now)
[16:09:17 CST(-0600)] <Bosmon> ok
[16:09:21 CST(-0600)] <Bosmon> I'm making a JIRA now
[16:09:41 CST(-0600)] <kasparnet> thanks so much Bosmon – and sorry for countering actually cleaning up all this overdue stuff
[16:10:10 CST(-0600)] <kasparnet> there is a "release" called "Pilot 2" that you can put as fix-for .. that should ensure it's on our radar
[16:13:43 CST(-0600)] <Bosmon> ok
[16:14:05 CST(-0600)] <Bosmon> yzen - do you think you would have time to actually add an acceptance test for this NP set you added, or do we need to push that out?
[16:16:55 CST(-0600)] <yzen> Bosmon: i do not think we can add it, because we do not actually have a rule based match maker and i am not sure how to fully deploy it in addition to the fact that it's not part of universal
[16:17:23 CST(-0600)] <Bosmon> ok
[16:17:31 CST(-0600)] <Bosmon> I guess this will have to be pushed out "beyond the horizon"
[16:17:41 CST(-0600)] <kasparnet> michelled: I am trying to review GPII-75, but I'm getting an error on npm install in universal
[16:17:51 CST(-0600)] <Bosmon> What's the error, kasparnet?
[16:18:27 CST(-0600)] <kasparnet> npm ERR! fetch failed https://registry.npmjs.org/core-util-is/-/core-util-is-1.0.1.tgz
[16:18:27 CST(-0600)] <kasparnet> npm ERR! Error: 404 Not Found
[16:18:27 CST(-0600)] <kasparnet> npm ERR! at null.<anonymous> (C:\Program Files\nodejs\node_modules\npm\lib\utils\fetch.js:57:12)
[16:18:29 CST(-0600)] <kasparnet> npm ERR! at EventEmitter.emit (events.js:126:20)
[16:18:31 CST(-0600)] <kasparnet> npm ERR! at WriteStream.flush (fs.js:1529:12)
[16:18:33 CST(-0600)] <kasparnet> npm ERR! at Object.oncomplete (fs.js:297:15)
[16:18:35 CST(-0600)] <kasparnet> npm ERR! If you need help, you may report this log at:
[16:18:37 CST(-0600)] <kasparnet> npm ERR! <http://github.com/isaacs/npm/issues>
[16:18:39 CST(-0600)] <kasparnet> npm ERR! or email it to:
[16:18:41 CST(-0600)] <kasparnet> npm ERR! <npm-@googlegroups.com>
[16:19:30 CST(-0600)] <Bosmon> kasparnet - that URL seems to be working for me
[16:19:35 CST(-0600)] <Bosmon> I'd just delete and retry the install again
[16:20:06 CST(-0600)] <Bosmon> Have you received it more than once in a row?
[16:20:52 CST(-0600)]
<kasparnet> Bosmon: yeah, twice - and hitting the url in a browser gave me:
... but just reloaded the URL after your comment here and now it seems to work
[16:20:54 CST(-0600)] <kasparnet> so retrying
[16:21:29 CST(-0600)] <kasparnet> Bosmon: yay - it worked!
[16:21:35 CST(-0600)] <Bosmon> kasparnet - splendido
[16:21:42 CST(-0600)] <Bosmon> As I believe they say in DANEMARK
[16:22:42 CST(-0600)] <kasparnet> heh
[16:22:47 CST(-0600)] <kasparnet> always
[16:32:46 CST(-0600)] <Justin_o_> Bosmon: I won't be back till tomorrow but just poking in. The issue I was describing is where the new key contains a "." That is escaped. You can see in the test case the the left hand side had this. "labrador
.retriever". I haven't read kasparnets jira but I would assume its for a different issue.
[16:33:09 CST(-0600)] <Bosmon> Thanks Justin_o_ - your test case looks good and I am working on it now
[16:33:19 CST(-0600)] <Bosmon> But I am wondering what kasparnet's is about if it isn't about this
[16:34:19 CST(-0600)] <michelled> kasparnet: thanks for pushing 75 for windows
[16:34:25 CST(-0600)] <michelled> there's also one for linux https://github.com/GPII/linux/pull/39
[16:35:03 CST(-0600)] <michelled> note that I haven't tested it
[16:37:43 CST(-0600)] <avtar> kasparnet: i was getting a whack load of 404s with npm as well
[16:37:56 CST(-0600)] <avtar> just be persistent
[16:38:21 CST(-0600)] <Bosmon> Thanks for the merge of FLUID-5246, yzen!
[16:38:31 CST(-0600)] <yzen> Bosmon: thank you
[16:38:33 CST(-0600)] <avtar> yzen and i just
[16:38:37 CST(-0600)] <avtar> ugh
[16:38:55 CST(-0600)] <avtar> kasparnet: yzen and i just made the necessary changes to flowmanager.gpii.net
[16:39:52 CST(-0600)] <avtar> colin mentioned that you have a repository that contains a bunch of json docs which should replace the preference sets for preferences.gpii.net
[16:40:25 CST(-0600)] <Bosmon> avtar - who has such a repository!
[16:40:31 CST(-0600)] <avtar> kasparnet
[16:40:36 CST(-0600)] <michelled> I wonder if they mean this pull request? https://github.com/GPII/universal/pull/181
[16:40:37 CST(-0600)] <avtar> i was told
[16:40:39 CST(-0600)] <Bosmon> Amazing
[16:40:44 CST(-0600)] <Bosmon> Where is it, and what is in it
[16:41:03 CST(-0600)] <avtar> the first question i can't answer
[16:41:42 CST(-0600)] <avtar> regarding what's in it, preference sets with the new format that yzen was raving about
[16:42:06 CST(-0600)] <avtar> colin said those should replace what's in the current preferences server
[16:43:17 CST(-0600)] <Justin_o_> Bosmon: thanks. I think solving either or would allow us to keep going. Thanks for the help.
[16:45:00 CST(-0600)] <kasparnet> avtar: yeah - what should go into the preferences server is the content of the universal/testData/preferences folder – sec., let me dig up a link
[16:45:15 CST(-0600)] <kasparnet> all these: https://github.com/GPII/universal/tree/master/testData/preferences
[16:45:40 CST(-0600)] <kasparnet> but it should also include the ones from this: https://github.com/GPII/universal/pull/181
[16:46:49 CST(-0600)] <avtar> ok
[16:47:06 CST(-0600)] <avtar> so just to confirm
[16:47:13 CST(-0600)] <michelled> kasparnet: who is on tap to review 181?
[16:47:17 CST(-0600)] <kasparnet> I guess it might be easiest to just take the list from my branch: https://github.com/kaspermarkus/universal/tree/GPII-383/testData/preferences
[16:47:27 CST(-0600)] <michelled> I wonder if an updated pull request battle list needs to be sent out
[16:47:58 CST(-0600)] <michelled> kasparnet: I think we wanted to avoid using arbitrary branches at this point
[16:48:17 CST(-0600)] <avtar> kasparnet: the sets served by preferences.gpii.net should be deleted, the sets in universal should be redeployed, along with the ones from the pull request you just linked to
[16:48:19 CST(-0600)] <kasparnet> michelled: I'm not sure who would be best to review 181 ... it's gonna be nonsense for most people - as it's very MM specific for some of the NP sets
[16:48:41 CST(-0600)] <avtar> ok
[16:48:57 CST(-0600)] <kasparnet> avtar: yeah – though perhaps they should pass review first (the ones in my branch)
[16:48:59 CST(-0600)] <michelled> kasparnet: is it one that required multi reviewers?
[16:51:45 CST(-0600)] <kasparnet> michelled: no, not necessarily
[16:52:10 CST(-0600)] <kasparnet> michelled: the problem is that it's pretty hard to test
[16:52:24 CST(-0600)] <kasparnet> for most of the NP sets, one would need the RB and ST MM
[16:52:45 CST(-0600)] <kasparnet> for the backup sets, it's somewhat easier (though still a pain)
[16:53:02 CST(-0600)] <kasparnet> basically one would have to go through https://docs.google.com/document/d/1eJYRMp8JoJAMGmA7Y-eNMySiCRXgU4Uh3dwB0yp9EQs/edit?pli=1# and check that thing launches as expected
[16:55:09 CST(-0600)] <kasparnet> but even then, one would need some quite unvailable SP3 applications installed
[16:55:32 CST(-0600)] <kasparnet> michelled: perhaps this is best done, you and me together, tomorrow during our Meeting of Death®
[16:56:12 CST(-0600)] <kasparnet> (and perhaps have the merger do a sanity check of the 'code' before merge)
[16:59:22 CST(-0600)] <michelled> kasparnet: ok, I'm off to dance now - see you at the meeting of death
[17:00:06 CST(-0600)] <kasparnet> michelled: ok - enjoy the dancing and tty tomorrow
[17:00:21 CST(-0600)] <michelled> bye
[21:44:08 CST(-0600)] <colinclark> Bosmon: So how can I help with the latest in pull request review?
[21:45:12 CST(-0600)] <Bosmon> colinclark - I have one in for model transforms at the moment
[21:45:19 CST(-0600)] <colinclark> ok
[21:45:23 CST(-0600)] <colinclark> Shall I start there?
[21:45:30 CST(-0600)] <Bosmon> And there are of course those ones with grade linkage for the windows registry resolver
[21:45:38 CST(-0600)] <Bosmon> But how is our status with MATCHYMATCHYMATCHMAKERS?
[21:47:07 CST(-0600)] <colinclark> I am several hours behind on the IRC channel and apparently 65 emails behind
[21:47:15 CST(-0600)] <colinclark> so I guess you might be able to tell me
[21:47:29 CST(-0600)] <colinclark> Were you able to look at the Matchamaker proxy pull request?
[21:48:03 CST(-0600)] <Bosmon> Well, maybe
[21:48:13 CST(-0600)] <Bosmon> I think that is the one I made the fuss about the apparently superfluous file in?
[21:48:21 CST(-0600)] <colinclark> Paste me a link?
[21:49:37 CST(-0600)] <Bosmon> Starting at 15:52 today EST ....
[21:49:44 CST(-0600)] <Bosmon> Still trying to find the parent link
[21:49:52 CST(-0600)] <colinclark> I think I have most of that in my buffer
[21:49:54 CST(-0600)] <colinclark> no worries
[21:49:58 CST(-0600)] <Bosmon> https://github.com/GPII/universal/pull/174/files
[21:49:59 CST(-0600)] <Bosmon> This one
[21:50:30 CST(-0600)] <colinclark> yes, this naming issue is one that I had a long talk with Kasper
[21:50:37 CST(-0600)] <Bosmon> Today?
[21:50:44 CST(-0600)] <colinclark> no, at least a month ago
[21:50:48 CST(-0600)] <Bosmon> urgh
[21:50:49 CST(-0600)] <colinclark> and is, sadly, an example of a PiratePad that has vanished
[21:50:52 CST(-0600)] <Bosmon> One may as well write on the water : P
[21:51:01 CST(-0600)] <colinclark> I made detailed notes of our naming conventions
[21:51:05 CST(-0600)] <colinclark> we really do need that "publish to wiki" button
[21:51:19 CST(-0600)] <Bosmon> I tend to just keep mine open forever
[21:51:38 CST(-0600)] <colinclark> I lost some tabs with my recent kernel panics
[21:52:58 CST(-0600)] <Bosmon> So on the phone today you mentioned some complex workflow required for this pull
[21:53:02 CST(-0600)] <Bosmon> Which I failed to register completely
[21:53:09 CST(-0600)] <colinclark> I think the workflow, fortunately, isn't ours
[21:53:15 CST(-0600)] <colinclark> we just need to get it in
[21:53:31 CST(-0600)] <colinclark> from there, Nick and Andy can finish verifying that their Matchmaker servers work correctly with it
[21:53:38 CST(-0600)] <colinclark> and Avtar can deploy them
[21:55:31 CST(-0600)] <Bosmon> So, how can we verify this pull
[21:55:55 CST(-0600)] <Bosmon> Given that JURA's response when I asked him whether he could write an acceptance test for that stray file was, "I can't, because we don't have a rule-based matchmaker"
[22:00:30 CST(-0600)] <colinclark> But we do
[22:00:32 CST(-0600)] <colinclark> ok
[22:00:43 CST(-0600)] <colinclark> I think the answer is in an email from Nick
[22:01:00 CST(-0600)] <colinclark> and that I should be able to install the RBMM on a VM
[22:29:35 CST(-0600)] <Bosmon> Hey yzen
[22:30:30 CST(-0600)] <yzen> Hi Bosmon
[22:37:18 CST(-0600)] <colinclark> hey yzen
[22:37:31 CST(-0600)] <yzen> colinclark: hi