fluid-work IRC Logs-2013-06-27

[10:19:49 CDT(-0500)] <yzen> cindyli: Justin_o: fyi: latest schema grade stuff is here: https://github.com/yzen/infusion/tree/FLUID-4903/ . I am now waiting on Bosmon and his work related to discovering the grades of a certain types

[10:20:47 CDT(-0500)] <cindyli> thanks, yzen

[10:21:57 CDT(-0500)] <Justin_o> yzen: thanks.. by the way.. are we supposed to be using "use strict;"

[10:30:29 CDT(-0500)] <colinclark> Justin_o: I really think we should be using "use strict" at this point

[10:30:46 CDT(-0500)] <colinclark> we need to only use it within our closure scope so it doesn't force other code in a concatenated file to be strict

[10:30:56 CDT(-0500)] <colinclark> but I believe it will catch a number of errors that are easy to make in JavaScript

[10:33:28 CDT(-0500)] <colinclark> hey kasparnet!

[10:33:41 CDT(-0500)] <kasparnet> dr. clarkster

[10:43:16 CDT(-0500)] <Justin_o> colinclark: that makes sense.. i guess we should update the other code in the repo

[10:44:57 CDT(-0500)] <colinclark> probably should at some point, yes

[10:46:49 CDT(-0500)] <colinclark> I guess I'll also finish my electrical project

[10:46:58 CDT(-0500)] <colinclark> oops

[10:47:00 CDT(-0500)] <colinclark> Well

[10:47:06 CDT(-0500)] <colinclark> everyone can know about my electrical project

[10:47:06 CDT(-0500)] <colinclark>

[10:47:08 CDT(-0500)] <colinclark> It's awesome

[10:47:34 CDT(-0500)] <colinclark> I've rewired almost my whole boat!

[10:53:40 CDT(-0500)] <Justin_o> colinclark: be careful.. i think on lake would be the last place i'd want to be doing electrical

[10:53:48 CDT(-0500)] <colinclark>

[10:54:06 CDT(-0500)] <colinclark> especially with the weather forecast this weekend!

[10:54:36 CDT(-0500)] <Justin_o> colinclark: yes.. looking pretty grey outside already

[12:08:42 CDT(-0500)] <Justin_o> colinclark, yzen: i filed a jira for switching our js files to use strict mode http://issues.fluidproject.org/browse/FLUID-5071

[12:09:51 CDT(-0500)] <colinclark> cool

[12:48:05 CDT(-0500)] <anastasiac> Justin_o, I think you may have pasted a wrong link into FLUID-5071 description

[12:49:16 CDT(-0500)] <Justin_o> anastasiac: oh

[12:49:28 CDT(-0500)] <Justin_o> yes you are right

[12:51:03 CDT(-0500)] <Justin_o> anastasiac: fixed now.. thanks for spotting that.. could have been worse i guess

[12:51:24 CDT(-0500)] <anastasiac> yes, who knows what you might have in your copy/paste buffer, Justin_o

[12:52:53 CDT(-0500)] <Justin_o> anastasiac: haha.. it's all work related

[13:03:36 CDT(-0500)] <Justin_o> yzen: for UIO will it no longer work without the i18n.js file

[13:03:43 CDT(-0500)] <Justin_o> i wonder if it should work by default without it

[13:05:03 CDT(-0500)] <Justin_o> just noticed that the manual test is missing it and UIO is failing to initialize without it http://issues.fluidproject.org/browse/FLUID-5072

[13:13:02 CDT(-0500)] <yzen> Justin_o: yes i think it must be included

[13:13:04 CDT(-0500)] <yzen> alwyas

[13:13:32 CDT(-0500)] <Justin_o> yzen: okay.. so it's a required dependency for UIO

[13:13:52 CDT(-0500)] <yzen> Justin_o: yes i think so

[13:14:05 CDT(-0500)] <michelled> cindyli: I'm looking at your pull request https://github.com/fluid-project/infusion/pull/338 and it doesn't apply cleanly now that we've pushed several other pull requests. could you update your 5010 branch to master please?

[13:14:12 CDT(-0500)] <Justin_o> yzen: okay..

[13:14:25 CDT(-0500)] <cindyli> sure, michelled

[13:14:31 CDT(-0500)] <michelled> thanks!

[13:21:23 CDT(-0500)] <Justin_o> yzen: did you want to review this pull request i just sent.. it's really small https://github.com/fluid-project/infusion/pull/351

[13:22:17 CDT(-0500)] <yzen> Justin_o: sure

[13:22:25 CDT(-0500)] <Justin_o> yzen: thanks

[13:25:55 CDT(-0500)] <Justin_o> cindyli: i just filed this jira.. you haven't filed something similar have you http://issues.fluidproject.org/browse/FLUID-5074

[13:26:56 CDT(-0500)] <cindyli> no, Justin_o

[13:27:23 CDT(-0500)] <Justin_o> cindyli: good thanks.. just wanted to make sure i wasn't duplicating anything

[13:30:28 CDT(-0500)] <cindyli> michelled: https://github.com/fluid-project/infusion/pull/338 is now up to date

[13:30:35 CDT(-0500)] <anastasiac> Justin_o, cindyli, are we still planning to have a schema meeting this afternoon?

[13:30:41 CDT(-0500)] <michelled> thanks cindyli

[13:30:44 CDT(-0500)] <cindyli> np

[13:31:10 CDT(-0500)] <cindyli> yes, anastasiac. we are waiting for Bosmon. he doesn't seem to be around

[13:31:15 CDT(-0500)] <anastasiac> okley

[13:31:38 CDT(-0500)] <Bosmon> Hi, I am sort of around, sorry

[13:31:42 CDT(-0500)] <Bosmon> I was a bit il

[13:33:11 CDT(-0500)] <Justin_o> Bosmon: are you feeling up for the meeting?

[13:33:17 CDT(-0500)] <Justin_o> we can do it tomorrow if you prefer

[13:33:19 CDT(-0500)] <Bosmon> Yes, just give me 5 mins

[13:33:33 CDT(-0500)] <Justin_o> Bosmon: okay let us know when you are ready.. i will have a hard deadline at 3:30 today though

[13:38:13 CDT(-0500)] <Bosmon> Justin_o - ok, I can have a bit of a meeting

[13:39:02 CDT(-0500)] <Justin_o> Bosmon: cool..

[13:39:27 CDT(-0500)] <Justin_o> fluid-everyone: we are about to start our meeting on uio schema.. would anyone like to join

[13:39:38 CDT(-0500)] <michelled> I'd like to

[13:39:40 CDT(-0500)] * anastasiac raises hand

[13:39:47 CDT(-0500)] <cindyli> me

[13:40:55 CDT(-0500)] <Justin_o> yzen: i'm assuming you are coming too..

[13:41:10 CDT(-0500)] <yzen> yes

[13:49:51 CDT(-0500)] <colinclark> oh, i'd be happy to join

[13:49:54 CDT(-0500)] <colinclark> if i'm not too late

[13:50:00 CDT(-0500)] <colinclark> Justin_o: ^

[13:50:28 CDT(-0500)] <Justin_o> colinclark: sure i'll call you in now

[14:20:57 CDT(-0500)] <kasparnet> bosmon: BASSMANNN!

[14:22:38 CDT(-0500)] <Bosmon> DAMMMN YOUUUUUU, KASPAARRRNETTTT!!!!

[14:30:15 CDT(-0500)] <anastasiac> hi, ramil, how are things going?

[14:30:26 CDT(-0500)] <kasparnet> oh, how I appreciate that welcome Bosmon

[14:30:37 CDT(-0500)] <kasparnet> so I'm looking at the ini settings handler

[14:30:43 CDT(-0500)] <ramil> I've tried a million different things, still won't work

[14:30:51 CDT(-0500)] <kasparnet> which currently allows for sections like: [a.b.c]

[14:30:59 CDT(-0500)] <anastasiac> ramil, are you still getting the same errors? or different errors?

[14:31:21 CDT(-0500)] <ramil> if i switched the infusion libs , i get different errors

[14:31:33 CDT(-0500)]

[14:31:45 CDT(-0500)] <ramil> what could cause the infusion to not be defined?

[14:32:09 CDT(-0500)] <anastasiac> ramil, when you update the lib, you don't get infusion not defined, do you?

[14:32:12 CDT(-0500)] <kasparnet> this is all fine, but with our extended support for subsections (denoted with [[ ]]), it becomes ambiguous

[14:32:44 CDT(-0500)] <kasparnet> eg: [a] [[b]] and [a.b] will suddenly be intepreted as the same json structure

[14:32:58 CDT(-0500)] <ramil> yes i think thats right anastasiac

[14:33:00 CDT(-0500)] <Bosmon> You should be cursed!

[14:33:33 CDT(-0500)] <anastasiac> ramil, if you're seeing an error that says "ASSERTION FAILED…", then you're making progress. It means that you just need to change your javascript code to use the new API

[14:33:40 CDT(-0500)] <kasparnet> Bosmon, hehe - well, you sort of started the conversation doing that

[14:33:46 CDT(-0500)] <anastasiac> which is unfortunately not documented

[14:33:50 CDT(-0500)] <anastasiac> yet

[14:33:55 CDT(-0500)] <anastasiac> because it's in flux

[14:33:58 CDT(-0500)] <anastasiac> :-?

[14:34:12 CDT(-0500)] <kasparnet> ... so my question is: do I allow this ambiguouity and just intepret it consistently (to [a] [[b]]) when encoding back to ini

[14:34:49 CDT(-0500)] <kasparnet> or would it be better practice to not interpret [a.b] when using the subsection option - and hence diverging from regular behavior

[14:34:51 CDT(-0500)] <anastasiac> ramil, if you look in the Infusion code at the UIOptions demo, you'll see the new API: https://github.com/fluid-project/infusion/blob/master/src/webapp/demos/uiOptions/js/uiOptionsDemo.js

[14:35:04 CDT(-0500)] <Bosmon> kasparnet - can you write out a few examples fully, perhaps on a wiki page?

[14:35:12 CDT(-0500)] <Bosmon> I'm not sure I completely grasp the encoding issues yet

[14:35:19 CDT(-0500)] <anastasiac> the html is here: https://github.com/fluid-project/infusion/blob/master/src/webapp/demos/uiOptions/html/uiOptions.html

[14:35:26 CDT(-0500)] <Bosmon> Perhaps just in a comment on the JIRA for the INI file handler

[14:35:33 CDT(-0500)] <kasparnet> ok will do

[14:35:43 CDT(-0500)] <kasparnet> eating first, then adding the text to JIRA

[14:35:48 CDT(-0500)] <kasparnet> will ping you when done

[14:36:46 CDT(-0500)] <anastasiac> ramil, you can see this demo in action here: http://build.fluidproject.org/infusion/demos/uiOptions/html/uiOptions.html

[14:37:13 CDT(-0500)] <ramil> anastasiac, looking at that html , it looks like it's calling demo.initSettingsStore();

[14:37:13 CDT(-0500)] <ramil> demo.initPageEnhancer();

[14:37:13 CDT(-0500)] <ramil> demo.initFatPanel(".flc-uiOptions-fatPanel");

[14:37:50 CDT(-0500)] <ramil> instead of the old code which starts with fluid.something

[14:38:05 CDT(-0500)] <anastasiac> right, ramil, that's part of the new API requirements. All those demo.() functions are defined in the js file I posted. Those demo.() functions call the fluid.* functions

[14:38:07 CDT(-0500)] <ramil> so fluid is being replaced by demo?

[14:38:18 CDT(-0500)] <anastasiac> sorry: demo.* is NOT part of the new API requirements

[14:38:23 CDT(-0500)] <anastasiac> I was not clear, ramil

[14:38:55 CDT(-0500)] <anastasiac> those three demo.* functions were written for the demo, to keep the HTML clean. If you look in the JS file, you'll see what those three functions do

[14:40:10 CDT(-0500)] <ramil> so basically in my javascript, there's only those few lines that deal directly with the Infusion.js so I just need to know what replaces them

[14:40:31 CDT(-0500)] <ramil> is this right?

[14:41:13 CDT(-0500)] <anastasiac> in your script, ramil, you need to accomplish what those three functions accomplish. You could create three similar functions, or put them in a single function

[14:58:35 CDT(-0500)] <Bosmon> IT'S DONE!!!!

[14:58:41 CDT(-0500)] <Bosmon> Congratulations to cindyli on her amazing branch

[14:58:52 CDT(-0500)] <Bosmon> And thanks to a FISH for great reviewing and pushing

[14:58:59 CDT(-0500)] <cindyli> YAY!!

[15:01:13 CDT(-0500)] <michelled> I didn't do much - all the reviewing and improvements were Bosmon and cindyli

[15:06:40 CDT(-0500)] <michelled> hey system64, how's it going?

[15:07:08 CDT(-0500)]