openworm / org.geppetto

Geppetto is an open-source platform to build web-based applications to visualize and simulate neuroscience data and models.
http://geppetto.org
Other
209 stars 50 forks source link

Add ability to lazy load a value backend #504

Closed LordKrabo closed 8 years ago

LordKrabo commented 8 years ago

@niteshthali08 I am adding these to Sprint 75.

niteshthali08 commented 8 years ago

@tarelli : Could you please list set of actions on the UI which will invoke resolveImportType functionality ?

tarelli commented 8 years ago

@niteshthali08 at the moment it's just from the Javascript API, when you have an ImportType you can call resolve on it, see this

niteshthali08 commented 8 years ago

@tarelli : What would be the value of "String path" received by function "resolveImportValue(String path)" would look like ?

tarelli commented 8 years ago

@niteshthali08 the path of the variable or type that the value is attached to (e.g. nwbSample.response currently or a more complex one given the new design nwbSample.sweep100.stimulus.recording)

niteshthali08 commented 8 years ago

Thanks @tarelli, I need your help in understanding few things. Can we have a small meeting on Thursday or Friday ?

tarelli commented 8 years ago

Hey Nitesh, I'm out of commission for the rest of the week but @gidili can answer those questions as I've caught him up with what you are doing.

On Thu, Jun 16, 2016, 3:04 AM Nitesh Thali notifications@github.com wrote:

Thanks @tarelli https://github.com/tarelli, I need your help in understanding few things. Can we have a small meeting on Thursday or Friday ?

— You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub https://github.com/openworm/org.geppetto/issues/504#issuecomment-226369799, or mute the thread https://github.com/notifications/unsubscribe/AAE850Bgqx3ueTwd-1V8Ei5sPnEGZyVhks5qMK8RgaJpZM4IwEzi .

niteshthali08 commented 8 years ago

@gidili: are you available for some time during this week for a small meeting ?

gidili commented 8 years ago

@niteshthali08 yes! I'll send you an email to schedule

LordKrabo commented 8 years ago

Add this to sprint 76 @niteshthali08 @tarelli @gidili ?

niteshthali08 commented 8 years ago

@tarelli: I have committed the code. So I wanted to write a jUnit test case to call the functions on backend so that I can test them as they are executed. I just copy pasted code from one of the existing test case DataSourcesGeppettoManagerTest.java into ImportValueTest. I have created test2 folder inside resources with NWB stuff. Could you please help me in getting this work ? I want this test case to call the method resolveImportValue(path).

tarelli commented 8 years ago

@niteshthali08 I haven't found your test anywhere but I committed some code to show you some more pointers

niteshthali08 commented 8 years ago

@tarelli : Sorry for that there was a merge conflict while pushing which got neglected by me.

niteshthali08 commented 8 years ago

@tarelli : I have modified nwbModelInterpreter to return display text according to the proposed design for lazy loading. It would be great if we can have one small meeting at the earliest in coming week to get some pointers on implementing frontend.

niteshthali08 commented 8 years ago

I tried to write a test case to test out the resolveImportValue functionality. I created "ImportValueTest.java" but it is throwing an exception at the line given below. "appContext" is returned as null. public static Object getNewServiceInstance(String discoveryId) throws GeppettoInitializationException { ApplicationContext appContext = ApplicationListenerBean.getApplicationContext(discoveryId); ... }