Re: [Rd] small bug in gl1ce, package lasso2 (PR#8280)

From: Berwin A Turlach <berwin_at_maths.uwa.edu.au>
Date: Fri 04 Nov 2005 - 07:45:53 GMT

>>>>> "BDR" == ripley <ripley@stats.ox.ac.uk> writes:

    BDR> Please do read the FAQ:

    BDR> Finally, check carefully whether the bug is with R, or a
    BDR> contributed package.  Bug reports on contributed packages
    BDR> should be sent first to the package maintainer, and only
    BDR> submitted to the R-bugs repository by package maintainers,
    BDR> mentioning the package in the subject line.

And Brian forgot to point out that you should probably not file a bug report on your bug report since somebody has to clean up after you. :) Now there is PR#8279 and PR#8280 to deal with, the second one should have been a follow up for the first.

    BDR> You are not the maintainer, so this is the wrong place. Well, I am not the maintainer either, but just a short note:

You should read the documentation of the argument 'standardize', whose default is TRUE:

standardize: logical flag: if 'TRUE', then the columns of the model

          matrix that correspond to parameters that are constrained
          are standardized to have empirical variance one.  The
          standardization is done after taking possible weights into
          account and after sweeping out variables whose parameters
          are not constrained.

If you have a constant term in your model, then you can not standardize it. So if you set 'sweep.out' to NULL and have a constant term in your model, then you should set standardize to FALSE. If you still want to standardize your non constant regressor variables, then you have to do it yourself and pass the variables as you want them to gl1ce.

IMO, there is no bug here, the functions were designed to work in this way.

Cheers,

        Berwin


R-devel@r-project.org mailing list
https://stat.ethz.ch/mailman/listinfo/r-devel Received on Fri Nov 04 18:51:00 2005

This archive was generated by hypermail 2.1.8 : Fri 04 Nov 2005 - 11:21:37 GMT