grinsted / gwmcmc

An implementation of the Goodman & Weare MCMC sampler for matlab
Other
51 stars 24 forks source link

Bracket for name value parameters? #6

Closed benlansdell closed 6 years ago

benlansdell commented 8 years ago

In gwmcmc.m

17: % [models,logP]=gwmcmc(minit,logPfuns,mccount,[Parameter,Value,Parameter,Value]);

Aren't the brackets a bit confusing, given the syntax is really:

[models,logP]=gwmcmc(minit, logPfuns, mccount, Parameter,Value, ...);

??

grinsted commented 8 years ago

I never considered it. My intended meaning is to show that the these arguments are optional. Maybe that is clear even without brackets.