fluid-work IRC Logs-2013-10-28

Owned by Fluid IRC Log Bot
Last updated: Oct 28, 2013
11 min read

[09:27:16 CDT(-0500)] <anastasiac> cindyli and Justin_o, I'm starting to look at documenting the composite panel work (which I see has been merged into master - yay!). I'm wondering if you two have a moment to chat about my initial thoughts

[09:28:42 CDT(-0500)] <Justin_o> anastasiac: sure, whenever you'd like

[09:28:59 CDT(-0500)] <anastasiac> cindyli, do you have a minute now?

[09:29:58 CDT(-0500)] <anastasiac> ah, she's not at her desk right now (smile)

[09:39:36 CDT(-0500)] <anastasiac> Justin_o and cindyli: The prefs framework message bundles contain strings that are apparently associated with the icons at the ends of the sliders. Are these strings supposed to be rendered into the markup in some way?

[09:45:14 CDT(-0500)] <anastasiac> Justin_o and cindyli: The textfieldSlider has been promoted to its own component, and it's a viewComponent, but it doesn't "come with" its own default HTML template. I wonder if it should?

[09:45:14 CDT(-0500)] <Justin_o> anastasiac: can you paste in an example

[10:26:20 CDT(-0500)] <anastasiac> Justin_o, cindyli, I'm re-thinking the idea of textfieldSlider having its own template. I checked all of our other components, and none of them – save Uploader and ToC – have their own templates.

[10:26:36 CDT(-0500)] <cindyli> ah ha

[10:26:56 CDT(-0500)] <anastasiac> and I've filed the string issue and added it to the iteration page: http://issues.fluidproject.org/browse/FLUID-5192

[10:26:56 CDT(-0500)] <Justin_o> anastasiac: yah.. we're pretty inconsistent eh.. i guess we'll figure that out for 2.0

[10:27:08 CDT(-0500)] <Justin_o> anastasiac: thanks

[10:28:35 CDT(-0500)] <colinclark> Hey jhernandez, are you around?

[10:50:09 CDT(-0500)] <jhernandez> colinclark: sure!

[10:50:44 CDT(-0500)] <jhernandez> I've heard that you were not able to build the android thing

[10:51:38 CDT(-0500)] <colinclark> It built fine, yes

[10:51:42 CDT(-0500)] <colinclark> I had trouble running it

[10:51:45 CDT(-0500)] <jhernandez> oh

[10:51:46 CDT(-0500)] <colinclark> I was hoping we could talk it through sometime soon

[10:51:59 CDT(-0500)] <jhernandez> of course

[10:52:01 CDT(-0500)] <colinclark> It failed silently while starting up Anode, I think

[10:52:07 CDT(-0500)] <colinclark> I saw the GPII intent

[10:52:10 CDT(-0500)] <colinclark> but no server running

[10:52:20 CDT(-0500)] <jhernandez> hmmm

[10:52:59 CDT(-0500)] <colinclark> I was so close! (smile)

[10:53:15 CDT(-0500)] <jhernandez> it's true that sometimes it doesn't starts properly, and you have to check for it

[10:54:00 CDT(-0500)] <jhernandez> either by looking at logcat, where you can see our beloved "Kettle Server is running on port: 8081" message

[10:55:04 CDT(-0500)] <jhernandez> or by directly ask the system if he's listening on port 8081 (netstat command)

[10:56:34 CDT(-0500)] <jhernandez> so I'm pretty sure that it should run well on your device

[10:57:59 CDT(-0500)] <jhernandez> anyway, any error will be raised on the logcat, so you will probably can see there if something is fucky [1]

[10:58:02 CDT(-0500)] <jhernandez> [1] http://i.imgur.com/IgU4Y.gif

[10:58:12 CDT(-0500)] <jhernandez> (tongue)

[11:13:05 CDT(-0500)] <colinclark> (smile) (smile)

[11:13:28 CDT(-0500)] <colinclark> jhernandez: I did look log cat the output of starting up the app, and it didn't give me any obvious failure messages

[11:14:16 CDT(-0500)] <colinclark> This was the log I received at the time: http://pastie.org/8422872

[11:14:28 CDT(-0500)] <colinclark> I can try to rebuild and push it again and let you know what happens if that helps, jhernandez

[11:18:12 CDT(-0500)] <jhernandez> hmm

[11:18:16 CDT(-0500)] <jhernandez> colinclark: it should work

[11:18:23 CDT(-0500)] <colinclark> (smile)

[11:19:05 CDT(-0500)] <jhernandez> usually, after "V/bridge-core( 3544): Env::~Env(): ret, this=0x738b7c30" you should start seeing our gpii's start lines

[11:20:10 CDT(-0500)] <jhernandez> as a quick test, start it, and if you don't see these lines on the logcat, start again

[11:20:14 CDT(-0500)] <jhernandez> oh!

[11:20:46 CDT(-0500)] <jhernandez> and I forget to say that, IIRC, on the instructions there is an importan phrase

[11:21:25 CDT(-0500)] <jhernandez> which says: "Take into account that the first time running, you need to manually start the GPII activity. ( This will be fixed in the future )"

[11:22:32 CDT(-0500)] <jhernandez> this is, go to the android menu, and start the GPII activity manually before running the "./android-gpii.sh start"

[11:39:56 CDT(-0500)] <colinclark> ok, this is interesting

[11:39:57 CDT(-0500)] <colinclark> let me try that

[11:47:56 CDT(-0500)] <jhernandez> colinclark: heh, I'm always saying interesting things (tongue) (tongue)

[11:57:39 CDT(-0500)] <colinclark> If I remember, I had to do a couple of other things, jhernandez

[11:57:45 CDT(-0500)] <colinclark> Something to do with jtar

[11:58:05 CDT(-0500)] <colinclark> and also changing an auto generated file to link against version 18 of the Android SDK

[12:01:07 CDT(-0500)] <jhernandez> oh, maybe was linking against 17?

[12:05:41 CDT(-0500)] <colinclark> yep, it was

[12:06:25 CDT(-0500)] <colinclark> jhernandez: This time, I'm getting actual error messages from Node.js!

[12:06:30 CDT(-0500)] <colinclark> That's an improvement!

[12:06:31 CDT(-0500)] <colinclark> http://pastie.org/8437754

[12:07:51 CDT(-0500)] <jhernandez> mmmmm

[12:07:55 CDT(-0500)] <jhernandez> sweeet!

[12:08:10 CDT(-0500)] <jhernandez> maybe a silly question, but ...

[12:08:19 CDT(-0500)] <colinclark> probably not a silly question (smile)

[12:08:26 CDT(-0500)] <jhernandez> heh

[12:08:38 CDT(-0500)] <jhernandez> it's about the js code of universal

[12:08:46 CDT(-0500)] <jhernandez> which version are you using?

[12:08:57 CDT(-0500)] <colinclark> let's check

[12:09:10 CDT(-0500)] <colinclark> It should be whatever version the script checked out for me

[12:09:15 CDT(-0500)] <jhernandez> and of course, which version (branch) of GPII/android are you using?

[12:09:15 CDT(-0500)] <colinclark> last Tuesday

[12:09:18 CDT(-0500)] <colinclark> let's check

[12:09:28 CDT(-0500)] <colinclark> Android master

[12:09:57 CDT(-0500)] <colinclark> Universal master

[12:10:25 CDT(-0500)] <colinclark> both seem to have the latest changes in them

[12:10:57 CDT(-0500)] <jhernandez> ok ok

[12:11:24 CDT(-0500)] <jhernandez> seems to me that the problem should be related to the latest changes on universal

[12:11:46 CDT(-0500)] <jhernandez> and it's not something related to our code from GPII/android

[12:12:24 CDT(-0500)] <jhernandez> basically, because we can see that Anode tries to run our GPII framework

[12:13:31 CDT(-0500)] <jhernandez> "Error invoking global function: gpii.server could not be located { type: 'initComponent',"

[12:13:55 CDT(-0500)] <jhernandez> this is an error from our GPII core framework

[12:14:00 CDT(-0500)] <jhernandez> not from Android

[12:14:05 CDT(-0500)] <jhernandez> ok

[12:15:10 CDT(-0500)] <jhernandez> later I'll try to build all the stuff from master, and will check if all is going fine

[12:15:37 CDT(-0500)] <colinclark> thankks

[12:15:41 CDT(-0500)] <jhernandez> btw, I've never seen this error message before

[12:15:50 CDT(-0500)] <colinclark> Maybe yzen has some thoughts on it?

[12:16:10 CDT(-0500)] <jhernandez> so, let's pray for not to be an Android 18-related issue :S

[12:16:13 CDT(-0500)] <jhernandez> maybe

[12:17:21 CDT(-0500)] <jhernandez> var yzen = fluid.require("yzen");

[12:17:22 CDT(-0500)] <jhernandez> (tongue)

[12:17:31 CDT(-0500)] <yzen> ha

[12:17:34 CDT(-0500)] <yzen> what's up jhernandez

[12:18:01 CDT(-0500)] <jhernandez> heh

[12:18:46 CDT(-0500)] <jhernandez> colinclark and I was wondering about if you have any opinion about that error message

[12:25:01 CDT(-0500)] <jhernandez> anyway

[12:25:09 CDT(-0500)] <jhernandez> yzen: colinclark I'll give it a try later

[12:25:40 CDT(-0500)] <yzen> oh

[12:25:42 CDT(-0500)] <yzen> right

[12:25:49 CDT(-0500)] <yzen> so gpii.server does not exist any more

[12:25:54 CDT(-0500)] <yzen> it's kettle onw

[12:25:55 CDT(-0500)] <yzen> now

[12:27:31 CDT(-0500)] <jhernandez> yzen: ofc!

[12:28:04 CDT(-0500)] <yzen> so you need to rename it in the list, and you need to make sure it's also available through npm

[12:31:44 CDT(-0500)] <jhernandez> just another thing ... find the differences between https://github.com/GPII/linux/blob/master/gpii.js and https://github.com/GPII/android/blob/master/gpii.js

[13:00:15 CDT(-0500)] <anastasiac> Justin_o, I've added another JIRA to the iteration page: http://issues.fluidproject.org/browse/FLUID-5194 I've tagged it blocker to make sure we think about it, but I suspect it's not a blocker

[13:01:11 CDT(-0500)] <Justin_o> anastasiac: thanks, it would be good to clean all that up

[13:55:20 CDT(-0500)] <anastasiac> cindyli and Justin_o, I'm looking at the composite panel stuff. I'm creating my own panel templates, naturally, so I've got a folder for them and I've set the 'templatePrefix' to reference my folder. But now, the path to the SeparatedPanelPrefsEditorFrame.html file is wrong, because it's looking in the same place as my custom templates. Overriding that one specific path is a bit onerous. I wonder if we should have separate "prefixes" for

[13:55:21 CDT(-0500)] <anastasiac> things integrators are likely to override (like panel templates), and b) things integrators are NOT likely to override (like the iframe)

[13:58:14 CDT(-0500)] <cindyli> anastasiac: are you using schema version?

[13:58:32 CDT(-0500)] <anastasiac> cindyli, yes, for now. I will also do this (eventually) with grades

[14:00:19 CDT(-0500)] <anastasiac> cindyli, is there some reasonably simple way for me to specify the path for the iFrame template only?

[14:00:32 CDT(-0500)] <cindyli> anastasiac: there's one place that the path to a custom template can be specified but not sure if you would be completely happy with that. For the composite panel - https://github.com/fluid-project/infusion/blob/master/src/framework/preferences/js/StarterSchemas.js#L156

[14:01:16 CDT(-0500)] <anastasiac> cindyli, isn't that the template for the container of a group?

[14:02:03 CDT(-0500)] <cindyli> yes, the group panel in my understanding is our so-call parent composite panel

[14:02:24 CDT(-0500)] <cindyli> for the regular or subpanel, to specify its own template: https://github.com/fluid-project/infusion/blob/master/src/framework/preferences/js/StarterSchemas.js#L149

[14:02:39 CDT(-0500)] <anastasiac> cindyli, the only thing I'm having trouble loading is SeparatedPanelPrefsEditorFrame.html

[14:02:59 CDT(-0500)] <anastasiac> it's looking for that file in my folder, the one I specified with the templatePrefix property

[14:03:01 CDT(-0500)] <cindyli> ah. ok. digging...

[14:12:47 CDT(-0500)] <cindyli> anastasiac: seems the path to iframe currently is not surfaced up by builder. It would be a deep nesting block to access it by passing in an option to the constructed prefsEditor grade. need a jira for it. which way do you think is more preferrable for integrators to supply this path? defining it in the aux schema, or as an option to the constructed grade.

[14:13:00 CDT(-0500)] <cindyli> hi Bosmon2

[14:14:13 CDT(-0500)] <anastasiac> cindyli, good question. hm… My first response is better in the schema. Just about everything is in the schema, so that feels more natural. But further thought might expose flaws in that logic, I didn't really think about it, it's just my first reaction

[14:15:02 CDT(-0500)] <anastasiac> also, cindyli, I think I've run into a bug in the composite stuff. I'm trying to have two composite panels, and the assembled selectors are getting confused. Maybe when you have a minute, you could come look at what I'm seeing?

[14:15:28 CDT(-0500)] <cindyli> coming, anastasiac.

[14:20:38 CDT(-0500)] <Justin_o> yzen: can you please take a look at my gpii/prefsEditor pull request when you have a chance. https://github.com/GPII/prefsEditors/pull/8 We'd like to use this updated version when we update chris and alex's pull requests.

[14:21:24 CDT(-0500)] <anastasiac> ok, cindyli, my test code is on its way via email

[14:22:00 CDT(-0500)] <cindyli> awesome. thanks, anastasiac

[14:35:08 CDT(-0500)] <jhernandez> colinclark: building android/master from scratch, I've found an issue with the jtar thing

[14:37:00 CDT(-0500)] <colinclark> cool

[14:37:05 CDT(-0500)] <colinclark> you saw the same thing as me, perhaps?

[14:37:14 CDT(-0500)] <colinclark> hey jhernandez, have you been talking to the TP Vision folks recently?

[14:37:52 CDT(-0500)] <jhernandez> the "source ./prebuild.sh" complained about it (can't create app/libs)

[14:38:41 CDT(-0500)] <jhernandez> so if you continue with the build, it won't build because it doesn't find the jtar tarball

[14:38:46 CDT(-0500)] <jhernandez> annnd

[14:39:05 CDT(-0500)] <jhernandez> about TPVision ...

[14:39:40 CDT(-0500)] <jhernandez> nope, I pinged him by skype last week but got no response from him

[14:39:49 CDT(-0500)] <jhernandez> this is on my android-related TODO list

[14:40:07 CDT(-0500)] <jhernandez> what do you need from them?=

[14:42:05 CDT(-0500)] <anastasiac> Justin_o, the build site demo of UIOptions has a problem with the links controls panel: the inputsLarger template isn't loading (an error in the console)

[14:42:12 CDT(-0500)] <anastasiac> not found

[14:43:47 CDT(-0500)] <Justin_o> anastasiac: (sad)

[14:44:00 CDT(-0500)] <Justin_o> anastasiac: could you please file a jira for that

[14:44:01 CDT(-0500)] <anastasiac> sorry :-/

[14:44:08 CDT(-0500)] <anastasiac> will do, Justin_o

[14:44:11 CDT(-0500)] <Justin_o> thnaks

[14:44:15 CDT(-0500)] <Justin_o> thanks

[14:44:21 CDT(-0500)] <Justin_o> sorry having trouble typing 'a'

[14:44:30 CDT(-0500)] <anastasiac> odd, Justin_o: I checked out master this morning, and it works locally...

[14:45:54 CDT(-0500)] <Justin_o> anastasiac: that's strange

[14:46:57 CDT(-0500)] <anastasiac> Justin_o, do you still want me to file it?

[14:47:01 CDT(-0500)] <Justin_o> anastasiac: i see what the problem is

[14:47:12 CDT(-0500)] <Justin_o> there's a typo on the file name.. using a lower case p instead of P

[14:47:30 CDT(-0500)] <anastasiac> ah, and my computer doesn't care

[14:48:10 CDT(-0500)] <anastasiac> Justin_o, should I file it? or would you or cindy just make the change in your next pull request?

[14:48:56 CDT(-0500)] <Justin_o> anastasiac: did you mind to file it? if you want to fix it right away i'll merge it in

[14:49:17 CDT(-0500)] <anastasiac> np, I'll file it and issue a pull request

[14:49:53 CDT(-0500)] <anastasiac> FYI, Justin_o, I've also files an improvement request: http://issues.fluidproject.org/browse/FLUID-5195 I could submit a pull request for that, too

[14:51:15 CDT(-0500)] <Justin_o> anastasiac: sure, thanks

[14:55:39 CDT(-0500)] <Justin_o> cindyli: the gpii/prefsEditor repo has been updated with the latest infusion and the updated Exploration Tool

[14:55:59 CDT(-0500)] <Justin_o> cindyli: We should use this now to update Chris and Alex's code

[14:56:05 CDT(-0500)] <cindyli> super, Justin_o

[14:56:32 CDT(-0500)] <Justin_o> cindyli: did you want me to try to merge it into my branch?

[14:57:00 CDT(-0500)] <cindyli> yes if it's not too much trouble, Justin_o

[14:58:31 CDT(-0500)] <Justin_o> cindyli: no problem

[14:58:55 CDT(-0500)] <cindyli> thanks, Justin_o

[14:59:45 CDT(-0500)] <anastasiac> Justin_o: file renames: https://github.com/fluid-project/infusion/pull/427

[15:09:37 CDT(-0500)] <Justin_o> cindyli: my branch has been updated

[15:10:00 CDT(-0500)] <cindyli> cool, Justin_o, i will merge in

[15:15:43 CDT(-0500)] <Justin_o> anastasiac: thanks for the pull request. I've merged it into the project repo

[15:19:14 CDT(-0500)] <anastasiac> thanks, Justin_o

[17:05:14 CDT(-0500)] <jhernandez> yzen: still around?¿

[17:07:46 CDT(-0500)] <yzen> jhernandez: on and off

[17:09:11 CDT(-0500)] <jhernandez> ok

[17:09:30 CDT(-0500)] <jhernandez> just wanted to say that there is not any problem with android

[17:18:07 CDT(-0500)] <yzen> jhernandez: sweet

[18:18:17 CDT(-0500)] <jhernandez> sgithens: around?

[18:19:15 CDT(-0500)] <sgithens> jnernandez: Wow! My IRC actually connected in tihs coffee shop in the background! I saw your emails yesterday but didn't have time to fully parse them yet

[18:19:25 CDT(-0500)] <jhernandez> heh

[18:19:50 CDT(-0500)] <jhernandez> no problem about the mails

[18:19:51 CDT(-0500)] <sgithens> What's happening

[18:20:31 CDT(-0500)] <jhernandez> I haven't checked your latest commits to gpii/android

[18:20:48 CDT(-0500)] <jhernandez> now I am checking it, and we're still having 2 problems

[18:20:56 CDT(-0500)] <sgithens> All I did was merge GPII-120 and your readme and path

[18:21:03 CDT(-0500)] <sgithens> Yeah, actually I found an issue while helping colin too

[18:21:09 CDT(-0500)] <jhernandez> yes!

[18:21:18 CDT(-0500)] <sgithens> Which ones did you find?

[18:21:18 CDT(-0500)] <jhernandez> one is the jtar damn thing

[18:21:19 CDT(-0500)] <sgithens> (smile)

[18:21:33 CDT(-0500)] <sgithens> I had to add in a thing to make the directory

[18:21:43 CDT(-0500)] <sgithens> because the libs dir wasn't checked in or created

[18:21:45 CDT(-0500)] <jhernandez> https://github.com/GPII/android/pull/5

[18:21:51 CDT(-0500)] <sgithens> so the 'cp' command was failing

[18:22:09 CDT(-0500)] <sgithens> hahhaha

[18:22:13 CDT(-0500)] <jhernandez> :]

[18:22:15 CDT(-0500)] <sgithens> yup

[18:22:21 CDT(-0500)] <sgithens> what was the other issue?

[18:22:26 CDT(-0500)] <jhernandez> the other was with our config

[18:22:31 CDT(-0500)] <sgithens> ah yes

[18:22:40 CDT(-0500)] <jhernandez> we're missing this one -> https://github.com/GPII/android/commit/98d2122768a44809af964c5557e9e19e7e641c46

[18:22:56 CDT(-0500)] <sgithens> aha!

[18:22:58 CDT(-0500)] <sgithens> thanks

[18:23:04 CDT(-0500)] <sgithens> I missed that

[18:23:24 CDT(-0500)] <jhernandez> you can add my repo, cherry-pick this commit, and put it into master

[18:23:35 CDT(-0500)] <sgithens> I'll try to get both those in before I leave. I'm flying to Cali in the morning and will be there until Sunday

[18:23:46 CDT(-0500)] <jhernandez> in fact, it's already into v0.2 branch

[18:24:02 CDT(-0500)] <jhernandez> wow, ok ok

[18:24:05 CDT(-0500)] <jhernandez> no problem

[18:24:26 CDT(-0500)] <sgithens> for a tissue bank even on Saturday. I hope to get some GPII hacking in, not sure how much though.

[18:24:26 CDT(-0500)] <sgithens> But I'll get those 2 simple things merged in

[18:24:44 CDT(-0500)] <jhernandez> thanks!

[18:25:37 CDT(-0500)] <sgithens> Thank you, for driving everything forward!

[18:26:01 CDT(-0500)] <jhernandez> anyway, when you have time, I'd like to talk to you about one of my pull request that I mention in the mail from saturday

[18:26:12 CDT(-0500)] <jhernandez> heh, my pleasure!

[18:26:31 CDT(-0500)] <sgithens> which number was it?

[18:26:57 CDT(-0500)] <jhernandez> actually, this one https://github.com/GPII/android/pull/4

[18:28:02 CDT(-0500)] <jhernandez> my doubt about the implementation is simple

[18:28:03 CDT(-0500)] <sgithens> I'm afraid to open thunder-bird, because it always destroys my wireless connection, I think because of how many requests it tries to make on opening

[18:28:14 CDT(-0500)] <jhernandez> heh

[18:28:17 CDT(-0500)] <jhernandez> no problem

[18:28:33 CDT(-0500)] <jhernandez> all the stuff is on the pull request I've just pasted

[18:29:11 CDT(-0500)] <jhernandez> basically, my question is about the implementation

[18:29:36 CDT(-0500)] <jhernandez> I've moved your AndroidFontSettings java code into a separate folder

[18:30:07 CDT(-0500)] <jhernandez> aand, about how I am extending the activity_manager.js source code

[18:30:48 CDT(-0500)] <jhernandez> I was wondering if we should split some of that source code into separate modules

[18:31:26 CDT(-0500)] <jhernandez> and keep the activity_manager.js source code

[18:31:39 CDT(-0500)] <jhernandez> clean

[18:32:31 CDT(-0500)] <sgithens> aha

[18:32:51 CDT(-0500)] <sgithens> I'll take a looksy

[18:33:03 CDT(-0500)] <jhernandez> thanks!

[18:33:20 CDT(-0500)] <sgithens> It's so wierd, because of the way their (androids) API's are, we might not be able to completely separate those things

[18:33:57 CDT(-0500)] <sgithens> Also, I feel like you are a roll, you pump out like a new settings handler backend every other day

[18:34:12 CDT(-0500)] <jhernandez> xDDD

[18:34:31 CDT(-0500)] <sgithens> It's completely awesome (smile)

[18:34:55 CDT(-0500)] <jhernandez> just trying to do my best

[18:34:57 CDT(-0500)] <jhernandez> :]

[18:35:51 CDT(-0500)] <jhernandez> in fact, the ALSA one, it's just a short-term solution

[18:36:43 CDT(-0500)] <jhernandez> I'd like to use a node.js port of pulseaudio, so we will be able to deal with app's volume levels, separately

[18:41:02 CDT(-0500)] <sgithens> that would be really cool!