fluid-work IRC Logs-2012-03-21
[10:21:58 CDT(-0500)] <anastasiac> michelled, cindyli1: I've created a couple of issues for the AuthTool work cindyli1 and I are working on: FLOE-1 and FLOE-2. michelled, please let me know if you think the issues contain enough information about where the work is happening.
[10:22:23 CDT(-0500)] <cindyli1> thanks, anastasiac
[10:30:06 CDT(-0500)] <michelled> alexn2: I'm going to close this pull request - could you please check if any of the code review comments have not been addressed? https://github.com/fluid-project/videoPlayer/pull/21
[10:30:21 CDT(-0500)] <michelled> alexn2: if they haven't been, please open JIRAs to cover them
[10:56:15 CDT(-0500)] <alexn2> michelled: I'm pretty sure all this feedback was incorporated into the component, so no need to create any new JIRAs
[10:56:28 CDT(-0500)] <michelled> great
[12:06:55 CDT(-0500)] <RUPO> greggy: sir ,should i have to describe/make a demo project(image editor) in the proposal?
[12:08:22 CDT(-0500)] <RUPO> waiting for your reply sir.
[12:11:34 CDT(-0500)] <greggy> in a meeting at the moment
[12:12:57 CDT(-0500)] <RUPO> sorry sir but didn't get you.
[12:13:19 CDT(-0500)] <greggy> talk to you later, I am in a meeting right w
[12:14:08 CDT(-0500)] <RUPO> ok sir
[12:16:45 CDT(-0500)] <greggy> RUPO: you should talk to the mentors for that project either jhung or Justin_o
[12:18:31 CDT(-0500)] <arunsur> colinclark, hi! I am interested in working on Web Based musical instruments. I went through flocking toolkit. really got me motivated.
[12:19:53 CDT(-0500)] <arunsur> colinclark, quickly jotted down some of my ideas on a wiki- http://goo.gl/eStKb . please do have a look. and let me know your suggestions
[12:23:10 CDT(-0500)] <jhung> Hi RUPO. I about to go into a meeting as well. I will get back to you in about 30 to 40 minutes?
[12:23:33 CDT(-0500)] <jhung> If you prefer, Rupo, post your questions here and I will look at the log when I'm done and get back to you.
[12:26:37 CDT(-0500)] <RUPO> jhung: sir,actally I am thinking that in my proposal for the Image Editor project wheather I have to make a demo of that or not?
[12:28:54 CDT(-0500)] <RUPO> furthermore, what informations should i give in the proposal,actually I have no idea how to write the proposal appropriately,please guide.
[12:42:52 CDT(-0500)] <jhung> Hi rupo. you can consult this guide to help you with your application: http://code.google.com/p/google-summer-of-code/wiki/AdviceforStudents
[12:43:38 CDT(-0500)] <jhung> A demo would be helpful if it will help illustrate your ideas, or demonstrate your abilities.
[12:55:06 CDT(-0500)] <jhung> rupo: see this, this is more appropriate http://www.google-melange.com/gsoc/org/google/gsoc2012/idi_idrc
[13:12:42 CDT(-0500)] <Bosmon> Justin_o - do you know if there is a development meeting today?
[13:36:59 CDT(-0500)] <colinclark> where is KASPAR?
[13:37:38 CDT(-0500)] <Bosmon> I seem him away on Skype.... was he meant to be here?
[13:38:02 CDT(-0500)] <colinclark> Justin_o is just informing me that KASPAR and I committed to doing a demo of GPII desktop personalization today for the community meeting
[13:38:06 CDT(-0500)] <colinclark> but of course it doesn't work yet
[13:38:07 CDT(-0500)] <colinclark> SHIP IT
[13:38:19 CDT(-0500)] <colinclark> We could certainly talk through the latest work on the architecture
[13:38:24 CDT(-0500)] <colinclark> yura and Bosmon
[13:38:29 CDT(-0500)] <colinclark> does that interest you?
[13:38:36 CDT(-0500)] <colinclark> or should we just cancel this week's community meeting?
[13:38:45 CDT(-0500)] <Bosmon> Sure, we could do that - but surely the most operative question is if it interests any conceivable audience _
[13:39:45 CDT(-0500)] <colinclark> Justin_o, michelled: Any thoughts?
[13:41:29 CDT(-0500)] <colinclark> Okay, I'm not hearing much
[13:41:34 CDT(-0500)] <colinclark> let's just cancel it and write some code
[13:41:37 CDT(-0500)] <colinclark> do some designs
[13:41:44 CDT(-0500)] <colinclark> whatever one does
[13:41:44 CDT(-0500)] <michelled> it is interesting to be sure, although given all our deadlines it might be better to do the demo and architecture talk at the same time
[13:43:46 CDT(-0500)] <colinclark> anastasiac: I saw this on Twitter, posted by Janet Swisher⦠https://ffeathers.wordpress.com/about-my-book/
[13:44:04 CDT(-0500)] <anastasiac> thanks, colinclark. cool
[13:48:09 CDT(-0500)] <Justin_o> fluid-everyone: unfortunately we'll be cancelling this weeks meeting as well.. if anyone has any suggestions though for what they'd like to cover in future weeks please let us know.. Next week yura will be talking about node.js
[13:55:24 CDT(-0500)] <yura> go yura
[13:57:00 CDT(-0500)] <jhung> @justin_o: I did a boom boom. I didn't realize that my editor was using Windows CR/LF and not Unix style LF. So the modifications I've done recently is "bad".
[13:57:53 CDT(-0500)] <anastasiac_> cindyli, I'm going to talk IDI site with jameswy, jessm and jvass right now. I'll come by after that
[13:58:09 CDT(-0500)] <cindyli> sure, anastasiac
[14:01:43 CDT(-0500)] <Justin_o> jhung: i like your twitteresque ping
[14:01:58 CDT(-0500)] <Justin_o> jhung: did you manage to fix the line endings
[14:04:09 CDT(-0500)] <jhung> @justin_o: I will fix the line endings in the next commit. #myMacHatesMe
[14:04:26 CDT(-0500)] <Justin_o> jhung: you just have to be nice to it
[14:04:27 CDT(-0500)] <jhung> @jhung: no I don't
[14:04:43 CDT(-0500)] <jhung> great. Now it's talking to me in IRC.
[14:05:52 CDT(-0500)] <Justin_o> jhung: should i start to get worried yet?
[14:08:55 CDT(-0500)] <jhung> justin_o: lol not yet. I think I have this under control. Constantly visiting BGR and tuaw seems to keep it satisfied.
[14:09:32 CDT(-0500)] <Justin_o> jhung:
[14:20:24 CDT(-0500)] <jhung> Hi Rupo. I responded while you were offline. You can see my responses here: http://wiki.fluidproject.org/display/fluid/fluid-work+IRC+Logs-2012-03-21
[14:20:56 CDT(-0500)] <RUPO> thank you sir.
[14:21:06 CDT(-0500)] <jhung> you're welcome
[14:24:02 CDT(-0500)] <michelled> Bosmon: it appears that the framework caching tests fail when I run them off the file system
[14:24:06 CDT(-0500)] <michelled> Bosmon: is that known?
[14:25:37 CDT(-0500)] <Bosmon> I believe that all tests which make use of the file system will fail in Chrome
[14:25:45 CDT(-0500)] <Bosmon> Which is "generally known"
[14:25:59 CDT(-0500)] <Bosmon> Justin_o has made a page indicating which options to use in various browsers to permit this
[14:26:09 CDT(-0500)] <Bosmon> Otherwise you can follow colinclark's suggestion of running them from a local web server
[14:26:25 CDT(-0500)] <Bosmon> At least in Firefox, the caching tests work ok from the filesystem for me
[14:27:32 CDT(-0500)] <michelled> I'm using FF
[14:27:40 CDT(-0500)] <michelled> I wonder why mine are failing
[14:37:26 CDT(-0500)] <Bosmon> michelled - what do they say
[14:37:37 CDT(-0500)] <Bosmon> I just checked the FLUID-4625 branch itself and that is fine too
[14:40:20 CDT(-0500)] <michelled> Bosmon: I checked with a local web server and they all pass fine
[14:40:35 CDT(-0500)] <Bosmon> Interesting
[14:40:40 CDT(-0500)] <Bosmon> What error did you get?
[14:40:57 CDT(-0500)] <michelled> I'm not getting errors - I'm getting failures in the tests for proleptick joinsets
[14:42:53 CDT(-0500)] <Bosmon> What does the failure say....
[14:43:01 CDT(-0500)] <Bosmon> And do they stay there even if you refresh the page?
[14:43:49 CDT(-0500)] <michelled> Expected count in final joinset
[14:43:49 CDT(-0500)] <michelled> Expected: 3 Result: 1
[14:43:50 CDT(-0500)] <michelled> Source: finalCallback([object Object])@file:///Users/michelled/Code/infusion/projectRepo/src/webapp/tests/framework-tests/core/js/CachingTests.js:157
[14:43:58 CDT(-0500)] <michelled> that's the first failure
[14:44:07 CDT(-0500)] <Bosmon> Very odd
[14:44:09 CDT(-0500)] <michelled> and it does reoccur when I refresh the page
[14:44:21 CDT(-0500)] <Bosmon> This is on FF on a Mac?
[14:44:37 CDT(-0500)] <michelled> yep
[14:44:48 CDT(-0500)] <michelled> FF 11
[14:44:58 CDT(-0500)] <Bosmon> Thankfully this is an error I could never be called upon to resolve
[14:44:59 CDT(-0500)] <michelled> I was also getting the failures on FF 10
[14:45:06 CDT(-0500)] <michelled> yes
[14:47:09 CDT(-0500)] <Bosmon> Given you can't even run software from the Fascist Empire in a VM : P
[14:47:27 CDT(-0500)] <michelled> Bosmon: all the tests pass in Safari
[14:47:37 CDT(-0500)] <michelled> and they pass in FF when I use a local web server
[14:47:42 CDT(-0500)] <michelled> so I've pushed your changes
[14:47:42 CDT(-0500)] <Bosmon> Awesome
[14:47:45 CDT(-0500)] <Bosmon> Thanks
[14:47:48 CDT(-0500)] <michelled> on to the next one
[14:48:02 CDT(-0500)] <Bosmon> It's great to have these sorted out
[14:48:10 CDT(-0500)] <Bosmon> I'll be able to turn to my UIO refactoring with Confidence : P
[15:02:10 CDT(-0500)] <michelled> Bosmon: would you consider the removal of applyInstantiator an API change?
[15:03:44 CDT(-0500)] <Bosmon> michelled - it was formerly commented as a non-API method
[15:03:51 CDT(-0500)] <Bosmon> So on those terms, I wouldn't
[15:06:13 CDT(-0500)] <michelled> Bosmon: I don't see it commented as non-API: https://github.com/fluid-project/infusion/blob/master/src/webapp/framework/core/js/FluidIoC.js#L919
[15:06:58 CDT(-0500)] <michelled> yura: have you ever run into a case where you called applyInstantiator directly?
[15:07:00 CDT(-0500)] <Bosmon> michelled - "glarg" : P
[15:07:10 CDT(-0500)] <Bosmon> Well, this was only a horrible oversight
[15:07:22 CDT(-0500)] <Bosmon> It calls "withInstantiator" which explains that it should never be used since it is stateful : P
[15:07:49 CDT(-0500)] <yura> michelled: sorry , i play by the rules
[15:07:59 CDT(-0500)] <Bosmon> yura, sure, sure
[15:08:08 CDT(-0500)] <michelled> yura: I don't know the rules
[15:08:24 CDT(-0500)] <yura> it's what Bosmon says
[15:08:25 CDT(-0500)] <yura>
[15:09:06 CDT(-0500)] <michelled> Justin_o: thoughts? can we ditch it? it's highly unlikely anyone has used it
[15:09:10 CDT(-0500)] <Bosmon> Consider yourself lucky I didn't abolish "resolveEnvironment" last week....... although that also doesn't seem to have a non-API comment!
[15:10:38 CDT(-0500)] <Bosmon> But yes, there are no uses of either "withInstantiator" or "applyInstantiator" within CSpace...
[15:11:13 CDT(-0500)] <Justin_o> Bosmon: what do you mean by "it was formerly commented as a non-API method"? How did it come to no longer be commented as such
[15:11:31 CDT(-0500)] <Bosmon> Justin_o - in theory, it came to be no longer commented as such because it was abolished
[15:11:41 CDT(-0500)] <Bosmon> But it turns out that I forgot to put the comment on it
[15:11:50 CDT(-0500)] <Bosmon> Hence this current discussion : P
[15:12:14 CDT(-0500)] <Justin_o> Bosmon: you mean it should have been removed before but it was missed?
[15:12:52 CDT(-0500)] <michelled> Justin_o: it should have been commented as non-API but wasn't
[15:12:54 CDT(-0500)] <Bosmon> Justin_o - I mean, it should have been commented before
[15:13:04 CDT(-0500)] <Bosmon> But in the pull request which michelled is reviewing now, it was abolished
[15:15:06 CDT(-0500)] <michelled> Justin_o: is seems to me that it should be safe to remove it, but I wanted your kingly opinion
[15:15:10 CDT(-0500)] <Justin_o> Bosmon, michelled: I guess i want to be careful not to set a precedent where we say oops that shouldn't have been an api lets just remove it
[15:15:23 CDT(-0500)] <michelled> that's a very good point
[15:16:59 CDT(-0500)] <michelled> we might be saved - it looks like it might not have shipped in 1.4
[15:17:23 CDT(-0500)] <Justin_o> michelled: that would be great. it would solve the whole issue
[15:18:11 CDT(-0500)] <michelled> Bosmon: were instantiates ever in another file? they certainly weren't in FluidIoC in 1.4
[15:18:55 CDT(-0500)] <Bosmon> I'm sure that's not right
[15:19:00 CDT(-0500)] <Bosmon> I can see them existing at least 9 months ago
[15:19:02 CDT(-0500)] <Bosmon> When was 1.4 released?
[15:19:09 CDT(-0500)] <michelled> https://github.com/fluid-project/infusion/blob/infusion-1.4/src/webapp/framework/core/js/FluidIoC.js
[15:19:19 CDT(-0500)] <RUPO> jhung: thank you for the support and replies
[15:20:22 CDT(-0500)] <Bosmon> michelled - They are there in that file
[15:20:38 CDT(-0500)] <Bosmon> Thankfully, "applyInstantiator" isn't
[15:21:20 CDT(-0500)] <michelled> ah, great - I must have misspelled instantiator in my search
[15:21:39 CDT(-0500)] <Bosmon> I should have named it, "thing" : P
[15:22:21 CDT(-0500)] <jhung> rupo: No problem.
[15:45:05 CDT(-0500)] <Bosmon> michelled - thanks for the merge
[15:45:08 CDT(-0500)] <Bosmon> Now for the big one...
[15:45:37 CDT(-0500)] <michelled> np
[15:47:05 CDT(-0500)] <michelled> ok Bosmon - this is the one with the conflict
[15:48:09 CDT(-0500)] <michelled> Bosmon: any chance you want to fix the conflict and push up the resolution to your branch?
[15:48:38 CDT(-0500)] <Bosmon> ok
[15:48:40 CDT(-0500)] <Bosmon> Which one?
[15:48:44 CDT(-0500)] <Bosmon> Oh
[15:48:47 CDT(-0500)] <michelled> 4633
[15:48:51 CDT(-0500)] <Bosmon> To merge up with current master?
[15:48:55 CDT(-0500)] <Bosmon> ok, cool
[15:48:55 CDT(-0500)] <michelled> yep
[15:49:26 CDT(-0500)] <Bosmon> It will merge, accompanied by the sounds of the infant Mozart...
[15:49:34 CDT(-0500)] <michelled>
[15:50:06 CDT(-0500)] <Bosmon> I wonder what it means, to write a Piano Concerto aged 11 : P
[15:51:00 CDT(-0500)] <michelled> Bosmon: how come the jslint options changed?
[15:51:13 CDT(-0500)] <Bosmon> michelled - could you remind me where the conflict is, quickly?
[15:51:46 CDT(-0500)] <michelled> Bosmon: it has to do with the changes to withInstantiator
[15:51:47 CDT(-0500)] <Bosmon> michelled - well, they had clearly never been applied before... I just reverted them to the ones used for IoC etc.
[15:52:03 CDT(-0500)] <Bosmon> I got bored applying the existing ones, we can certainly upgrade them further if you want : p
[15:52:17 CDT(-0500)] <Bosmon> But I thought it better to have linting to SOME standards rather than none at all
[15:52:50 CDT(-0500)] <michelled> I suppose the KING would want us to have a consistent linting standard
[16:03:36 CDT(-0500)] <Bosmon> avtar - I'm wondering whether we are meant to have Github integration working on our JIRA?
[16:03:45 CDT(-0500)] <Bosmon> I am noticing at the moment that the linkup doesn't seem to be working
[16:04:29 CDT(-0500)] <michelled> Bosmon: that might be a question for the king
[16:04:35 CDT(-0500)] <Bosmon> ok
[16:41:45 CDT(-0500)] <avtar> Bosmon: i can take a look at the jira/github issue
[16:42:03 CDT(-0500)] <Bosmon> avtar - thanks
[16:42:07 CDT(-0500)] <avtar> would it be ok if i get back to you by friday regarding this?
[16:42:12 CDT(-0500)] <Bosmon> No problem
[16:42:20 CDT(-0500)] <avtar> thanks