Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Demo failings #33

Closed
lionel-rigoux opened this issue Jul 6, 2017 · 3 comments
Closed

Demo failings #33

lionel-rigoux opened this issue Jul 6, 2017 · 3 comments

Comments

@lionel-rigoux
Copy link
Member

lionel-rigoux commented Jul 6, 2017

demo 13 14 16 17 reason
'demo_2DChoices' ? 🐞 😸 ? relies on ´quantile´
'demo_BSL' 😸 😸 😸 ? fixed by d1a6517 and by 905e04f
'demo_CI' ? 🐞 😸 ? relies on ´prctile´ from stats toolbox
'demo_CaBBI_FHN' ? 🐞 😸 ? relies on data fitting toolbox (fittype)
'demo_CaBBI_QGIF' ? 🐞 😸 ? relies on data fitting toolbox (fittype)
'demo_HRF_distributed' 😸 ? 😸 ? fixed by a8550d9
'demo_MFX' ? 🐞 😸 ? utilises cdf which depends on the stats toolbox
'demo_ToMgames' ? 😸 😸 ? plot problem + bug in multinomial sampling, fixed by d1a6517
'demo_bmc4glm' ? 🐞 😸 ? utilises cdf which depends on the stats toolbox
'demo_classification' 😸 😸 😸 ? dependency on zscore. Fixed by bc3e131
'demo_dcm_motorPremotor' 😸 ? 😸 ? fixed by 50c7aef
'demo_dcmonline' 😸 😸 😸 ? fixed by 57a9a81
'demo_dynLearningRate' ? 🐞 😸 ? utilises cdf which depends on the stats toolbox
'demo_interaction' 😸 😸 😸 😸 fixed (removed) by f7ec443
'demo_nullSpace' 😸 😸 😸 😸 fixed (removed) by 8a6f9e6
'demo_spm_hrf' 😸 😸 😸 😸 fixed (removed) by a8af462
'demo_susceptibility' 😸 🐞 😸 ? demo outdated + stochastic bug
'demo_test' 😸 😸 😸 😸 fixed (removed) by a7abf61
@jdaunize
Copy link
Contributor

jdaunize commented Jul 6, 2017

All the bug fixes performed under MATLAB 2013 need to be checked for other MATLAB versions!

@jdaunize
Copy link
Contributor

jdaunize commented Jul 7, 2017

Note: 'demo_susceptibility' runs OK on MATLAB 2013.

lionel-rigoux added a commit that referenced this issue Jul 7, 2017
- replace cdf calls by spm_cdf counterparts
- replace call to external nan* functions by spm couterparts
- add in-house simple  functions for quantile, ncfcdf, kstest
- fix typo and small issues in demos
lionel-rigoux added a commit that referenced this issue Jul 7, 2017
* extend check_struct to struct defaults

* enforce stricter checks of user defined model dimension

* fillInPriors makes use of VBA_priors to set default (#26)

* test commit (cosmetic change on VBA_hyperparameter.m)

* cleaning VBA goodness-of-fit summary statistics

WARNING: before, the field .R2 used to report the coef of determination
for continuous data, and the balanced accuracy for binary data. This has
now changed, and the fields fit.R2, fit.acc and fit.bacc always report
the same metric, namely: determination coefficient, classifical accuracy
(NaN for gaussian sources) and balanced classification accuracy  (NaN
for gaussian sources), respectively.

* correcting minor display bug

* minor changes in VBA results reporting

The changes actually deal with multi-source inversion.
NB: this fix does not solve graphical display issues with multi-source
inversion

* another set of bug fixes on VBA graphical results display

NB: still, this does not solve issues when VBA is in multi-source
mode....

* fix minor display bugs for VBA_hyperparameters.m

* bug fix for the zero prior variance case

Previous changes in VBA_prior and VBA_check unfortunately removed the
possibility to fix unknown model parameters to their prior value, which
was done by setting the corresponding prior variance to zero. This has
now been corrected back.

* major update of k-toM and BSL models

- correcting on k-ToM observation and evolution functions
- correcting on k-BSL observation and evolution functions
- updating "unwrap" functions for BSl and k-ToM learners
- adding "meta-ToM" learners (mixture of ToM and BSL experts)
- adding demonstration scripts

* updates on VBA statistical tools

- adding SEM_analysis0
- modifying a few GLM and RFT subfunctions

* a further series of minor bug fixes

- removed the 'extended' field in the options structure, and replaced it
with appropriate checks on the number of sources
- re-introduced params2update substructure after padding priors with
defaults
- overwrite ODE_posterior subfield in VBA_hyperparameters

* bug fixes in sparse inversion

- changed minimal gradient in sparseTransform for numerical convergence

* updates of a few demos

* update on VBA_MFX

- simplify priors checking
- fixed by-passing of VBA initialization
- eyeballing results in demo_MFX

* update VBA_MFX

- simplified priors checking
- fixed by-passing of VBA initialization step

* Update README.md

* Fix bernoulli.m clashing with Sym Toolbox (#30)

The symbolic toolbox has a bernoulli function which clashes with VBA's. This fix renames VBA's bernoulli.m file to VBA_bernoulli.m and updates references to bernoulli.m to reference the new function name.

* update on VBA_classification

- change in summary statistics
- change in results display
[ & minor update of unwrap_KBSLm ...]

* minor change in demos

* Small fix to mediation_contrast (#23)

* Fix for the wrong regression coefficient computation in stats&plots/mediationAnalysis0.m

* in stats/plots/RFT_* I fixed the call to cdf and icdf ('norm',..) to ('norm',..,0,1) because this command gives NaN values with matlab R2016a

* Fixed the numerical issues of 0/0 in RFT_Pval (when normcdf return 0 instead of a tiny number)

* Add nanxxx subfunction with VBA_ prefix, based on nansuite. Closes #31

* move nan subfunctions to avoid reinstall on pull

* update of k-ToM demos

* included commentaries in VBA_getDiagnostics

* accelerate VBA_MFX

bypass volterra kernel estimation within VB meta-loops

* accelerate VBA_hyperparameters

bypass volterra kernels estimation within VB meta-loop

* bug fixes

* a few more bug fixes on RFT (gaussian field)

* Fix display (issues #28 and #29) (#32)

Major changes fo the display system:
- merge extended and historical display function
- solve compatibility issues related to the 2014b graphical motor change
- numerous other bug fixes
- simplify figure identification for in-place redrawing
- cosmetic changes (plot arrangement, color consistency)
- add some documentation

* small trick to reduce demo run time

* add missing spm function

* remove legacy call to simulateNLSS_fb

* fix bug in when VBA_design_efficiency is given non array inputs

* remove dependency on zscore (stats toolbox)

* fix bug in VBA_sample for multinomial data

* add demo testing function

* bug fix on demo_spm_distributed

* fix display/verbose bugs

* remove demo_spm_hrf

no problematic code, no problem :)

* fix minor bug on demo_BSL

* remove dummy demo_test

useless

* remove demo_nullSpace

useless

* remove demo_interaction

useless

* added CDF of noncentral F-distrivution

* Remove external dependencies to fix #33

- replace cdf calls by spm_cdf counterparts
- replace call to external nan* functions by spm couterparts
- add in-house simple  functions for quantile, ncfcdf, kstest
- fix typo and small issues in demos
@lionel-rigoux
Copy link
Member Author

solved by 0b37168

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants