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.
>>>
>>

Reply via email to