barbaLab / nigeLab

Neurophysiological data analyses by engineers, for physiologists.
GNU General Public License v3.0
14 stars 2 forks source link

[BUG] #72

Closed arouseKUMC closed 1 year ago

arouseKUMC commented 4 years ago

Bug Report

Description

We were just trying to install and run the demo for the new version of nigeLab. We got an error where "saveLoc" was not a field in "tankObj.Paths" Do you have any idea what the issue is?

-Adam

Error Messages

-> Saving Tank params for TANK  (User: '')
Reference to non-existent field 'SaveLoc'.

Error in nigeLab.Tank/getParsFilename (line 727)
            f = nigeLab.utils.getUNCPath(tankObj.Paths.SaveLoc,...

Error in nigeLab.Tank/saveParams (line 456)
         fname_params = tankObj.getParsFilename();

Error in nigeLab.Tank/updateParams (line 93)
   tankObj.saveParams(tankObj.User,paramType);

Error in nigeLab.Tank (line 166)
         tankObj.updateParams('Tank'); % old maybe

Machine

m053m716 commented 4 years ago

@arouseKUMC Is the version you are running the most recent from the release tab?

arouseKUMC commented 4 years ago

Nope, sorry I had just cloned the main branch.

It seems to get further now with the version from the release. It looks like it saves .mat files. But then I do get this error:

Error using nigeLab.Block/checkActionIsValid (line 67) CAR (a necessary pre-processing step) is not yet completed. Cannot run doSD until this is complete, or requirement is changed in nigeLab.defaults.doActions

From: Maxwell Murphy notifications@github.com Sent: Friday, January 31, 2020 7:58 AM To: m053m716/ePhys_packages ePhys_packages@noreply.github.com Cc: Adam Rouse arouse@kumc.edu; Mention mention@noreply.github.com Subject: Re: [m053m716/ePhys_packages] [BUG] (#72)

@arouseKUMChttps://protect-us.mimecast.com/s/0hq4Cv2z9rIAzKN6tzan0H?domain=github.com Is the version you are running the most recent from the releasehttps://protect-us.mimecast.com/s/iyzOCqxrMmuXjl05SZSER9?domain=github.com tab?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://protect-us.mimecast.com/s/R51RCwpARvuy3981CKYcLF?domain=github.com, or unsubscribehttps://protect-us.mimecast.com/s/IVBWCxkB7wuxowMACRt9xy?domain=github.com.

m053m716 commented 4 years ago

OK cool, try running

linkToData(tankObj);

(There is a bug where doReReference is missing the update step that tells the Tank it has completed the re-referencing operation that is necessary prior to spike-detection in the default configuration).

m053m716 commented 4 years ago

I should add, with that v1.0.2-alpha release there is a known bug that probably messed up the doRawExtraction part. I'll try to get a more stable release on there today.

arouseKUMC commented 4 years ago

Ok, sounds good. Let me know when the next release comes out. No rush from our end.

-Adam

From: Maxwell Murphy notifications@github.com Sent: Friday, January 31, 2020 9:20 AM To: m053m716/ePhys_packages ePhys_packages@noreply.github.com Cc: Adam Rouse arouse@kumc.edu; Mention mention@noreply.github.com Subject: Re: [m053m716/ePhys_packages] [BUG] (#72)

I should add, with that v1.0.2-alpha release there is a known bug that probably messed up the doRawExtraction part. I'll try to get a more stable release on there today.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://protect-us.mimecast.com/s/DiMMCmZnMghWNV6LhOm38T?domain=github.com, or unsubscribehttps://protect-us.mimecast.com/s/_bmkCn5oNjtmN4xpsNyWo-?domain=github.com.

m053m716 commented 4 years ago

@arouseKUMC I was able to run through the demo_nigeLab.m script using release v1.1.0-alpha, so hopefully that is stable enough to get up and going. We just added some different indexing things and haven't tested 100% but looked like the DashBoard interface was more-or-less working and at least the pre-processing methods seem to be working.