LBHB / NEMS0

THIS VERSION OF NEMS IS NO LONGER SUPPORTED. PLEASE USE THE NEW NEMS REPOSITORY OR INSTALL NEMS_DB TO GET NEMS0 SUPPORT.
GNU General Public License v3.0
8 stars 4 forks source link

Migrate nems.tf to nems.analysis #178

Open arrrobase opened 4 years ago

arrrobase commented 4 years ago
svdavid commented 4 years ago

Perhaps lower priority, but something to keep in mind with the overhaul: the fitter was implemented in TF1.0. Some backward-compatible routines are in place to take care of the fit, but if we're porting it into the nems.analysis / nems.fitters world, it may be worth using a more standard implementation. Also related is the question of if/how to use batching for situations where we don't have trials of uniform length (eg, from behavior data). Right now, the code is designed to have a bunch of batches of equal lenght, which correspond to distinct stimuli / trials.

arrrobase commented 4 years ago

Is it just the TF portion that doesn't support batches of different lengths, or all of NEMS?

svdavid commented 4 years ago

Only TF. the rest of NEMS doesn't have the concept of batches -- at least in this context. "batches" in nems are groups of cells the same stimulus, not subsets of data for one cell. so two meanings of batch, which may also be confusing.