search

rietho / IPO

A Tool for automated Optimization of XCMS Parameters
http://bioconductor.org/packages/IPO/
Other
34 stars 20 forks source link

optimizeXcmsSet expects parameters that are not even used by and related to centWave #57

Closed etrh closed 6 years ago

etrh commented 6 years ago

When working with optimizeXcmsSet to optimize centWave parameters, optimizeXcmsSet requires several parameters that are not even related to centWave.

Error in checkParams(params, quantitative_parameters, qualitative_parameters,  : 
  The parameter(s) fwhm, sigma, max, steps, index is/are missing! Please specify; stopping!

Any plans to fix this?

rietho commented 6 years ago

Hi!

Unfortunately the package IPO is at the moment in an unsupported status. Thus, I can only offer very limited support.

However, it seems that there happened an error in your parameter specification. I guess that you specified a parameter step, which lets IPO think that you want to use the matchedFilter method. If you need further help, please post your full code, so that it's possible to see what might have gone wrong.

etrh commented 6 years ago

step was indeed the problem which I had erroneously added to the list of parameters. Removing it no longer throws an error. Thanks a lot.