I would also like to know how to get the tracker number for the patch file tracker.
Regards, Lakshmi Prasannakumar Bangalore On Wed, Oct 31, 2018 at 10:15 AM Lakshmi Prasannakumar < lakshmiprasannakuma...@gmail.com> wrote: > Hi Björn and Gábor, > > I've sent the patch file to the patch tracker. > Now I understand next this is to send you the timeline proposal for the > internship period , is it so? > The patch submission took me a little while since my system needed an > update due to ubuntu version becoming obsolete. > > Thanks, > Lakshmi Prasannakumar > Bangalore > > > On Tue, Oct 30, 2018 at 11:50 PM Lakshmi Prasannakumar < > lakshmiprasannakuma...@gmail.com> wrote: > >> Thanks Gábor, >> I'll make these changes and make a patch file and try to send across. >> >> Lakshmi Prasannakumar >> Bangalore >> >> >> On Tue, Oct 30, 2018 at 2:28 AM Gábor Boskovits <boskov...@gmail.com> >> wrote: >> >>> Hello Lakshmi, >>> >>> I did a preliminary review, that can make this contribution faster. >>> Here are my comments: >>> >>> (package >>> (name "r-weights") >>> (version "1.0") >>> (source >>> (origin >>> (method url-fetch) >>> (uri (cran-uri "weights" version)) >>> (sha256 >>> (base32 >>> "0186bfpkhxngrshac6bpg37alp6slwhwd43inrm8hqg0vhpfgc4c")))) >>> (build-system r-build-system) >>> (propagated-inputs >>> `(("r-gdata" ,r-gdata) >>> ("r-hmisc" ,r-hmisc) >>> ("r-mice" ,r-mice))) >>> (home-page >>> "http://cran.r-project.org/web/packages/weights") >>> (synopsis "Weighting and Weighted Statistics") >>> >>> Please use lower case for all words except the first. >>> The importer cannot find out which letters should be lower case, >>> this has to be done manually. >>> >>> (description >>> "Provides a variety of functions for producing simple weighted >>> statistics, such as weighted Pearson's correlations, partial >>> correlations, Chi-Squared statistics, histograms, and t-tests. Also >>> now includes some software for quickly recoding survey data and >>> plotting point estimates from interaction terms in regressions (and >>> multiply imputed regressions). NOTE: Weighted partial correlation >>> calculations pulled to address a bug.") >>> >>> Please, try to repharse this so that it has whole sentences, for >>> example like "This package provides...". >>> >>> (license gpl2+)) >>> >>> With these small modifications this looks good to me. >>> >>